Skip to content

Add path-scoped .claude/rules#101

Merged
GeorgeJahad merged 10 commits intoarmadaproject:masterfrom
sudiptob2:setup/claude-rules
Feb 17, 2026
Merged

Add path-scoped .claude/rules#101
GeorgeJahad merged 10 commits intoarmadaproject:masterfrom
sudiptob2:setup/claude-rules

Conversation

@sudiptob2
Copy link
Collaborator

@sudiptob2 sudiptob2 commented Feb 13, 2026

What: Add slash commands (/implement, /address-review, /issue) and extract path-scoped .claude/rules for a more structured Claude Code workflow.

Why: Common tasks like picking up GitHub issues, addressing PR review comments, and drafting issues were manual and repetitive. Testing, config, and version-specific guidance was embedded inline in CLAUDE.md, making it verbose. Extracting rules into path-scoped files surfaces the right context only when editing relevant files.

Changes:

  • Add /implement command: fetches a GitHub issue, creates a branch from latest master (with remote selection), plans the work, executes step-by-step with programmer approval, commits incrementally, and generates a PR summary
  • Add /address-review command: walks through unaddressed PR review comments one by one (fix, reply, or skip), commits fixes, and posts replies with confirmation
  • Add /issue command: drafts a GitHub issue markdown file from user-provided context and saves to plans/
  • Extract testing patterns, config entry conventions, and version-specific SparkSubmit rules into .claude/rules/ with path-scoped triggers
  • Scope gh CLI permissions in settings.json to specific subcommands instead of a blanket gh * wildcard
  • Trim CLAUDE.md by replacing inline code examples with references to the new rule files

How to verify:

  1. Run /implement <issue-number> and verify it walks through all 5 phases (fetch, branch, plan, execute, summary)
  2. Run /address-review <pr-number> on a PR with review comments
  3. Edit a file under src/test/ and confirm testing rules context is surfaced
  4. Check .claude/settings.json for scoped gh permissions

Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
…specific guidance

Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
@sudiptob2 sudiptob2 marked this pull request as ready for review February 17, 2026 17:11
Copy link
Collaborator

@GeorgeJahad GeorgeJahad left a comment

Choose a reason for hiding this comment

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

I haven't had a chance to experiment with all the features but it looks good to me.

Thanks @sudiptob2 !

@GeorgeJahad GeorgeJahad merged commit 515ce0b into armadaproject:master Feb 17, 2026
12 checks passed
@sudiptob2 sudiptob2 deleted the setup/claude-rules branch February 18, 2026 05:18
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