feat(#461): add /fullsend namespace prefix for slash commands#660
feat(#461): add /fullsend namespace prefix for slash commands#660rh-hemartin wants to merge 1 commit into
Conversation
Adds `/fullsend triage`, `/fullsend code`, `/fullsend review`, and `/fullsend fix` as the recommended command form to avoid collisions with other AI tools (e.g., Qodo) that also respond to `/review`. Bare commands remain as aliases — no breaking change. Updates shim template, self-shim, fix.yml instruction parsing, user docs, and glossary. Closes #461 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Do we want to remove the normal commands? |
|
@rh-hemartin yes, I think we should just remove the old un-namespaced commands. |
|
Stylistically, do you like any of these more than another?
|
|
I like the fully spelled commands with space, but it is a matter of preference. |
ggallen
left a comment
There was a problem hiding this comment.
We should not use the version with the space. It could cause problems if we want to actually add a prompt after the command.
And we should also not use /fullsend. We had a long discussion about stages where the number of characters was deemed significant (i.e. fix vs. refinement). I don't see why we would then want to add an 8-character prefix on every command.
|
Oh, that's a good point. Like |
|
Superseded by #742 — clean branch with just the /fs- namespace change. |
Summary
/fullsend triage,/fullsend code,/fullsend review,/fullsend fixas namespaced command forms to avoid collisions with other AI tools (e.g., Qodo) that also respond to/review/triage,/code,/review,/fix) remain as aliases — no breaking changesedpattern to strip/fullsend fixprefix from human instructionsChanged files
internal/scaffold/fullsend-repo/templates/shim-workflow.yaml— namespaced patterns in all 4 slash-command jobs.github/workflows/fullsend.yaml— same (this repo's own shim)internal/scaffold/fullsend-repo/.github/workflows/fix.yml— instruction prefix strippingdocs/guides/user/bugfix-workflow.md— updated command table and inline referencesdocs/glossary.md— updated Slash Command definitionTest plan
make lintpassesgo test ./internal/scaffold/...passesmake go-testpasses (all packages)/fullsend reviewon a PR comment in an enrolled repo, verify dispatch triggers/reviewon same repo, verify bare alias still worksCloses #461
🤖 Generated with Claude Code