Skip to content

feat(skill): rename proceed-with-claude-recommendation → proceed-with-the-recommendation#36

Merged
naimkatiman merged 2 commits intomainfrom
feat/rename-proceed-skill
Apr 28, 2026
Merged

feat(skill): rename proceed-with-claude-recommendation → proceed-with-the-recommendation#36
naimkatiman merged 2 commits intomainfrom
feat/rename-proceed-skill

Conversation

@naimkatiman
Copy link
Copy Markdown
Owner

Summary

Identifier-only rename of the featured companion skill so it can be reused from non-Claude agents (Codex, Gemini CLI, etc.) without carrying Claude branding in the name.

  • proceed-with-claude-recommendationproceed-with-the-recommendation across file paths, frontmatter `name:`, slash command, install snippets, and cross-references
  • 5 file/dir renames + 13 content updates (18 files, +56 / −49)
  • Body language about "Claude-emitted recommendation" intentionally untouched — that agent-genericization pass is a separate follow-up PR
  • Past CHANGELOG v3.3.0 entries and historical GitHub commit URLs in TESTING.md preserved (correct historical record)
  • New `[Unreleased]` CHANGELOG entry added documenting the rename + the deferred-genericization boundary

Migration note for old-name installs

Anyone with the previous standalone install needs to re-run the install snippet under the new path:

```bash
SKILL=proceed-with-the-recommendation
mkdir -p ~/.claude/skills/$SKILL
curl -L https://raw.githubusercontent.com/naimkatiman/continuous-improvement/main/skills/\$SKILL.md
-o ~/.claude/skills/$SKILL/SKILL.md
```

Test plan

  • `npm run build` — manifests regenerated, no diff drift in `plugins/`
  • `npm test` — 238/238 passing
  • `node bin/check-docs-substrings.mjs` — all 72 assertions match
  • Final straggler grep — only intentional historical refs remain (CHANGELOG v3.3.0 entries + 2 historical GitHub SHA URLs in TESTING.md)

…he-recommendation

Renames the slash command, skill, and test docs from proceed-with-claude-recommendation to proceed-with-the-recommendation across all locations (commands/, skills/, plugins/continuous-improvement/, docs/testing/). No content changes — pure file renames to remove "claude" from the command name.
… to proceed-with-the-recommendation

Renames the skill identifier across CHANGELOG, CONTRIBUTING, README, commands, plugins, skills, install scripts, tests, and generated artifacts. Updates frontmatter `name:`, slash command invocations, install snippets, file paths, and cross-references. No body content changes — pure identifier rename to drop Claude-specific branding from the skill name.
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request renames the proceed-with-claude-recommendation skill to proceed-with-the-recommendation to remove Claude-specific branding, updating all file paths, identifiers, and documentation accordingly. Feedback focuses on ensuring a smooth migration for existing users by adding cleanup steps to the installation snippets and scripts, and updating the changelog to advise removing legacy directories to avoid command conflicts.

Comment thread README.md
Comment on lines +120 to 123
SKILL=proceed-with-the-recommendation
mkdir -p ~/.claude/skills/$SKILL
curl -L https://raw.githubusercontent.com/naimkatiman/continuous-improvement/main/skills/$SKILL.md \
-o ~/.claude/skills/$SKILL/SKILL.md
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Since the skill identifier has changed, users who re-run this snippet will end up with two versions of the skill installed. Because they share trigger phrases like "do all of it", this will cause conflicts in Claude Code. Adding a cleanup step to the snippet ensures a smooth migration.

Suggested change
SKILL=proceed-with-the-recommendation
mkdir -p ~/.claude/skills/$SKILL
curl -L https://raw.githubusercontent.com/naimkatiman/continuous-improvement/main/skills/$SKILL.md \
-o ~/.claude/skills/$SKILL/SKILL.md
SKILL=proceed-with-the-recommendation
rm -rf ~/.claude/skills/proceed-with-claude-recommendation
mkdir -p ~/.claude/skills/$SKILL
curl -L https://raw.githubusercontent.com/naimkatiman/continuous-improvement/main/skills/$SKILL.md \
-o ~/.claude/skills/$SKILL/SKILL.md

Comment thread CHANGELOG.md
## [Unreleased]

### Changed
- **Renamed `proceed-with-claude-recommendation` → `proceed-with-the-recommendation`** — drops Claude-specific branding from the skill identifier so the same skill can be installed into non-Claude agents (Codex, Gemini CLI, etc.). Identifier-only rename: file paths, frontmatter `name:`, slash command, install snippets, and cross-references updated. Body language about "Claude-emitted recommendation" is intentionally untouched in this release; that agent-genericization pass is a separate follow-up. Old-name installations need to re-run the install snippet under the new path.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

It is recommended to explicitly advise users to remove the legacy skill directory. If both the old and new versions coexist in ~/.claude/skills/, they will both be loaded by the agent, leading to command conflicts due to overlapping trigger phrases (e.g., "do all of it").

Suggested change
- **Renamed `proceed-with-claude-recommendation``proceed-with-the-recommendation`** — drops Claude-specific branding from the skill identifier so the same skill can be installed into non-Claude agents (Codex, Gemini CLI, etc.). Identifier-only rename: file paths, frontmatter `name:`, slash command, install snippets, and cross-references updated. Body language about "Claude-emitted recommendation" is intentionally untouched in this release; that agent-genericization pass is a separate follow-up. Old-name installations need to re-run the install snippet under the new path.
- **Renamed `proceed-with-claude-recommendation``proceed-with-the-recommendation`** — drops Claude-specific branding from the skill identifier so the same skill can be installed into non-Claude agents (Codex, Gemini CLI, etc.). Identifier-only rename: file paths, frontmatter `name:`, slash command, install snippets, and cross-references updated. Body language about "Claude-emitted recommendation" is intentionally untouched in this release; that agent-genericization pass is a separate follow-up. Old-name installations need to re-run the install snippet under the new path and remove the legacy directory to avoid command conflicts.

Comment thread bin/install.mjs
"continuous-improvement.md",
"planning-with-files.md",
"proceed-with-claude-recommendation.md",
"proceed-with-the-recommendation.md",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

By removing the old filename from COMMAND_FILES, the uninstallAll function will no longer clean up the legacy /proceed-with-claude-recommendation.md command file. Keeping it in the list allows the uninstaller to remove the orphan file from the user's commands directory, while the installer will safely skip it since the source file no longer exists in the repository.

Suggested change
"proceed-with-the-recommendation.md",
"proceed-with-the-recommendation.md",
"proceed-with-claude-recommendation.md", // Legacy name for cleanup

Comment thread src/bin/install.mts
"continuous-improvement.md",
"planning-with-files.md",
"proceed-with-claude-recommendation.md",
"proceed-with-the-recommendation.md",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Keeping the legacy filename in this list ensures that uninstallAll can clean up the old command file from previous installations.

  "proceed-with-the-recommendation.md",
  "proceed-with-claude-recommendation.md", // Legacy name for cleanup

@naimkatiman naimkatiman merged commit f0a7068 into main Apr 28, 2026
4 checks passed
@naimkatiman naimkatiman deleted the feat/rename-proceed-skill branch April 28, 2026 09:19
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.

1 participant