Skip to content

Refactor command syntax in README.md to use colon namespacing for consistency#26

Merged
amir-yogev-gh merged 3 commits into
flightctl:mainfrom
amir-yogev-gh:update-command-syntax-docs
Apr 20, 2026
Merged

Refactor command syntax in README.md to use colon namespacing for consistency#26
amir-yogev-gh merged 3 commits into
flightctl:mainfrom
amir-yogev-gh:update-command-syntax-docs

Conversation

@amir-yogev-gh

@amir-yogev-gh amir-yogev-gh commented Apr 20, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates the documentation for several workflows to clarify feature descriptions and standardize command usage instructions across different environments. The main focus is on making the documentation more concise, accurate, and consistent, especially regarding how to invoke workflow commands in both Cursor and Claude Code environments.

Documentation improvements:

  • Expanded workflow descriptions in README.md to clarify the steps and features for Bugfix, PRD, Design, and CVE Fix workflows, including new phases and automation details.

Command usage standardization:

  • Updated all usage instructions to show a unified command syntax (e.g., /workflow:command) that works in both Cursor and Claude Code, replacing previous environment-specific examples and references.

These changes make the documentation easier to follow and ensure users can quickly understand and use the workflows in their preferred environment.

Summary by CodeRabbit

  • Documentation
    • Unified usage to slash-prefixed commands (e.g., /bugfix:assess, /cve-fix:patch, /ai-ready:update) and removed separate Cursor/Claude Code instruction blocks.
    • Expanded workflow docs: bugfix now lists assess→diagnose→submit PR, iterative PR feedback and unattended mode.
    • PRD/design flows now include revise→publish PR→respond to reviewers and sync to Jira.
    • CVE flow adds backporting and closing Jira tickets.

@coderabbitai

coderabbitai Bot commented Apr 20, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 58b53ab7-e47a-4674-a71e-6137f6caeb7c

📥 Commits

Reviewing files that changed from the base of the PR and between 025980f and c374200.

📒 Files selected for processing (4)
  • README.md
  • ai-ready/README.md
  • design/README.md
  • prd/README.md
✅ Files skipped from review due to trivial changes (2)
  • prd/README.md
  • design/README.md

Walkthrough

Standardized README command syntax to slash-prefixed /workflow:phase forms across multiple workflows and expanded several workflow descriptions to add explicit phases (iterative feedback, publish/respond PR steps, backports, Jira syncing, and unattended chaining).

Changes

Cohort / File(s) Summary
Root README updates
README.md
Expanded workflow descriptions for Bugfix, PRD, Design, and CVE Fix: added explicit assessment/revise/publish/respond steps, unattended chaining for Bugfix, backporting and Jira ticket closure for CVE Fix.
Workflow README docs
ai-ready/README.md, cve-fix/README.md, triage/README.md, design/README.md, prd/README.md
Replaced Cursor/Claude Code invocation examples with unified slash-prefixed command syntax (e.g., /ai-ready:update, /cve-fix:backport, /triage:scan, /design:publish, /prd:revise). Minor textual clarifications to flows and phases.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures the main focus of the changeset: standardizing command syntax across multiple README files using colon namespacing (e.g., /workflow:command), which is a central and consistent theme throughout all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (2)
README.md (1)

121-124: Consider explaining the command format mechanism.

Line 121 states the format "works in both Cursor and Claude Code" but doesn't explain why or how this colon-namespaced syntax achieves cross-environment compatibility. Users transitioning from the old @workflow/commands/... syntax might benefit from a brief explanation.

📝 Suggested clarification
-Invoke a workflow command (works in both Cursor and Claude Code):
+Invoke workflow commands using colon-namespaced syntax (works in both Cursor and Claude Code):
 
 - `/bugfix:assess`, `/bugfix:diagnose`, `/bugfix:fix`, ...
 - `/docs-writer:gather`, `/docs-writer:plan`, `/docs-writer:draft`, ...

Alternatively, you could add a brief note explaining that this syntax is recognized by both environments' command dispatchers, unlike the previous environment-specific formats.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 121 - 124, The README mentions the colon-namespaced
commands like `/bugfix:assess` and `/docs-writer:plan` but doesn't explain how
they work across Cursor and Claude Code; add a short clarification that this
colon syntax is recognized by both environments' command dispatchers (unified
command format) and contrasts it with the older `@workflow/commands/...`
environment-specific form so readers understand that `/namespace:action` is the
cross-environment mechanism and will be routed by both Cursor and Claude Code
command handlers.
ai-ready/README.md (1)

7-7: Inconsistent command format in prose.

Line 7 uses "The /update command" while line 16 documents the full form /ai-ready:update. For consistency with other workflow READMEs (cve-fix, triage), prose and section headers typically use either the bare command name or describe it as a "phase" rather than including the slash prefix.

📝 Suggested alignment options

Option 1 (matches cve-fix/triage pattern):

-The `/update` command scans a codebase and:
+The `update` command scans a codebase and:

Option 2 (more explicit):

-The `/update` command scans a codebase and:
+The update phase scans a codebase and:
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@ai-ready/README.md` at line 7, The README uses two different command formats
(`/update` vs `/ai-ready:update`); pick a single style and make the prose
consistent by replacing the `/update` occurrence with the bare workflow name
ai-ready:update (remove the leading slash) and ensure headers and other
references use the same form (or consistently refer to it as the "update phase")
so the file matches the cve-fix/triage README pattern.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 21: Update the README "Design" workflow sentence to explicitly map each
high-level action to its phase/command names: append the command names in
parentheses for each step (e.g., "draft a technical design document (/draft)",
"decompose work into Jira-ready epics and stories (/decompose)", "manage review
via GitHub PRs (/revise)"), and add the missing phases for publishing and
handling responses by mentioning "/publish" and "/respond" for publishing final
outputs and responding to follow-ups; edit the line that currently begins with
"**Design** -- Design-and-decompose workflow" to include these command
annotations so all seven phases (/ingest, /draft, /decompose, /revise, /publish,
/respond, /sync) are explicitly referenced.

---

Nitpick comments:
In `@ai-ready/README.md`:
- Line 7: The README uses two different command formats (`/update` vs
`/ai-ready:update`); pick a single style and make the prose consistent by
replacing the `/update` occurrence with the bare workflow name ai-ready:update
(remove the leading slash) and ensure headers and other references use the same
form (or consistently refer to it as the "update phase") so the file matches the
cve-fix/triage README pattern.

In `@README.md`:
- Around line 121-124: The README mentions the colon-namespaced commands like
`/bugfix:assess` and `/docs-writer:plan` but doesn't explain how they work
across Cursor and Claude Code; add a short clarification that this colon syntax
is recognized by both environments' command dispatchers (unified command format)
and contrasts it with the older `@workflow/commands/...` environment-specific
form so readers understand that `/namespace:action` is the cross-environment
mechanism and will be routed by both Cursor and Claude Code command handlers.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 077b4011-ca2f-4e98-a6f6-ed87749acddf

📥 Commits

Reviewing files that changed from the base of the PR and between efa70b2 and 025980f.

📒 Files selected for processing (4)
  • README.md
  • ai-ready/README.md
  • cve-fix/README.md
  • triage/README.md

Comment thread README.md Outdated
@amir-yogev-gh amir-yogev-gh requested a review from adalton April 20, 2026 18:50

@adalton adalton 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.

Individual workflow READMEs now have stale descriptions

The root README expands several workflow descriptions but the individual README files weren't updated to match:

  • prd/README.md:3 still says "...and manages review via GitHub PRs"
  • design/README.md:3 still says "...takes a PRD...breaks work into Jira-ready epics and stories, and manages review via GitHub PRs"

These now diverge from the more detailed descriptions in the root README.

Comment thread README.md
Comment thread README.md Outdated
@amir-yogev-gh amir-yogev-gh merged commit 2949e88 into flightctl:main Apr 20, 2026
4 checks passed
@amir-yogev-gh amir-yogev-gh deleted the update-command-syntax-docs branch April 20, 2026 19:56
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