docs: fix /implement references to /code in user guide - #392
Conversation
The bugfix workflow guide referenced `/implement` as the slash command for the code agent, but the actual wired command in the shim workflow is `/code`. Update all three occurrences to match the deployed behavior. Made-with: Cursor
Site previewPreview: https://967aa1eb-site.fullsend-ai.workers.dev Commit: |
![fullsend-ai-review[bot]](https://avatars.githubusercontent.com/in/3478501?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review: #392
Head SHA: a5509cc
Timestamp: 2026-04-23T00:00:00Z
Outcome: approve
Summary
Clean documentation fix. All three /implement → /code replacements in the bugfix workflow user guide are correct — the deployed shim workflow (internal/scaffold/fullsend-repo/templates/shim-workflow.yaml lines 67–68) confirms /code is the actual wired command. No correctness, security, or injection concerns. Approving.
Findings
Info
- [intent-alignment]
docs/ADRs/0002-initial-fullsend-design.md— ADR 0002 still references/implementin ~6 places. This PR correctly scopes itself to the user guide, and ADRs are typically historical records, but if the team treats ADRs as living documents the terminology inconsistency is worth a follow-up.
Footer
Outcome: approve
This review applies to SHA a5509ccdd4d9262d61bc54d5f75ab757762dca85. Any push to the PR head clears this review and requires a new evaluation.
|
I think I just got this in at #427 |
|
Closing this PR as the work was done here #427 |
2 participants
The bugfix workflow guide referenced
/implementas the slash command for the code agent, but the actual wired command in the shim workflow is/code. Update all three occurrences to match the deployed behavior.Made-with: Cursor