Skip to content

docs(upgrade): Claude Code files live in .claude/skills, not .claude/commands#3708

Merged
mnriem merged 1 commit into
github:mainfrom
jawwad-ali:docs/upgrade-claude-skills-path
Jul 24, 2026
Merged

docs(upgrade): Claude Code files live in .claude/skills, not .claude/commands#3708
mnriem merged 1 commit into
github:mainfrom
jawwad-ali:docs/upgrade-claude-skills-path

Conversation

@jawwad-ali

Copy link
Copy Markdown
Contributor

Problem

docs/upgrade.md points users at .claude/commands/ in three places when verifying/locating Claude Code's installed files:

  • line 310 — troubleshooting "verify files exist" (ls -la .claude/commands/ # Claude Code)
  • line 359 — "what gets overwritten" list
  • line 448 — a second "verify command files exist" block

But the Claude Code integration installs into .claude/skills (src/specify_cli/integrations/claude/__init__.py"dir": ".claude/skills"), and the "what gets kept" list earlier in this same doc (line 98) already correctly says .claude/skills/. So the doc is internally inconsistent, and the three troubleshooting checks tell users to ls a directory that does not exist for a Claude Code install.

Fix

Correct all three spots to .claude/skills/ (and reword one label to "command/skill files" to match line 98's phrasing). Documentation-only.

Verification

  • grep .claude/commands docs/upgrade.md → no matches remain.
  • Cross-checked against integrations/claude ("dir": ".claude/skills") and the already-correct line 98.
  • markdownlint-cli2 docs/upgrade.md → 0 issues.

AI-assisted: authored with Claude Code. I confirmed the install dir in the integration source and that line 98 of the same doc already uses .claude/skills/.

…commands

The Claude Code integration installs skills into `.claude/skills` (see
integrations/claude: `"dir": ".claude/skills"`), and the "what gets kept"
list earlier in this same doc already says `.claude/skills/`. But three
troubleshooting/reference spots still point users at `.claude/commands/`,
which does not exist for a Claude Code install -- so the "verify files
exist" checks list an empty/missing directory. Correct all three to
`.claude/skills/`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jawwad-ali
jawwad-ali requested a review from mnriem as a code owner July 24, 2026 08:46
@mnriem
mnriem requested a review from Copilot July 24, 2026 16:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Medium

@mnriem
mnriem merged commit ffe2a7f into github:main Jul 24, 2026
21 of 25 checks passed
@mnriem

mnriem commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Thank you!

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.

3 participants