Skip to content

Community Hub: list the built-in skills as installed - #7

Merged
EveAeternam merged 2 commits into
mainfrom
claude/skills-review-eruted
Oct 13, 2026
Merged

EveAeternam merged 2 commits into
mainfrom
claude/skills-review-eruted

Conversation

@EveAeternam

Copy link
Copy Markdown
Collaborator

Requested by Evie · project thread

What changed and why

Before: the two skills Shell:B ships (verify-chat, audit-code) weren't in the community repo. Once they are (ShellB-App/Shell-B-Skills#1), the Hub would list them as not installed, and pressing Install would fail with "You already have a skill called 'verify-chat'".

After: the Hub shows them as Installed. Installing one under the same name says it comes with Shell:B and suggests another name if you want to try the community copy.

How: community._installed() maps each skill whose origin is builtin to its skill:shellb-skills:skills/<name> item id. install() gives a clearer error for that case. There's also a changelog entry for the new community skills.

How you tested it

  • scripts/check.sh steps run in a cloud container: py_compile, pyflakes, and the full suite (429 tests). Two tests fail here and fail the same way on main without this change: smoke_test /api/models (no Ollama in the container) and integrity_test cell backups (runs as root). tsc wasn't run because no UI changed.
  • New community_test.Builtins test fails without the fix and passes with it.
  • User-visible change: entry added at the top of docs/changelog.json
  • New UI strings: none
  • Manual or API docs: no route, tool or table changed
  • No secrets, chats or personal data in the diff

New dependencies

None.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JN8DiWpoqx3bUXAAS9ZCn9


Generated by Claude Code

verify-chat and audit-code ship with Shell:B and are now mirrored in
Shell-B-Skills. The Hub marks those items installed, and installing one
under the same name explains it is built in.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JN8DiWpoqx3bUXAAS9ZCn9
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JN8DiWpoqx3bUXAAS9ZCn9
@EveAeternam EveAeternam self-assigned this Oct 13, 2026
@EveAeternam
EveAeternam marked this pull request as ready for review October 13, 2026 02:57
@EveAeternam
EveAeternam merged commit 82a5bc0 into main Oct 13, 2026
2 checks passed
EveAeternam pushed a commit that referenced this pull request Oct 13, 2026
docs/changelog.json: both sides added an entry at the top; this branch's first, then main's.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018go55qk4ihWMhkzn8r7rpK
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants