Skip to content

[AAASM-3941] ✨ (rules): Add org-level .claude/rules for AI-assisted development#27

Merged
Chisanan232 merged 7 commits into
masterfrom
v0.1.0/AAASM-3941/design_claude_rules
Jul 6, 2026
Merged

[AAASM-3941] ✨ (rules): Add org-level .claude/rules for AI-assisted development#27
Chisanan232 merged 7 commits into
masterfrom
v0.1.0/AAASM-3941/design_claude_rules

Conversation

@Chisanan232

@Chisanan232 Chisanan232 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Jira Ticket

Summary

Changes

  • .claude/rules/01-security.md — secrets/credentials/PII handling, dangerous commands (no curl | bash, no --no-verify, no force-push to main/master), never commit .env/credentials; notes the AAASM-3926 hard-enforcement scope boundary.
  • .claude/rules/02-git-workflow.md — branch naming, one worktree per ticket (stacked on a dependency's branch when one exists), atomic gitmoji commits, PR-only merges, PR title/body conventions.
  • .claude/rules/03-coding-standards.md — cross-repo standards; references (does not restate) CLAUDE.md's existing "document the WHY, not the WHAT" section; minimal/scoped changes, no unrequested refactors.
  • .claude/rules/04-agent-escalation.md — when an AI agent must stop and ask instead of guessing (uncertainty, blockers, hard-to-reverse actions), and how to phrase the escalation.
  • .claude/rules/05-context-boundary.md — secrets/credentials out of prompt/commit context; no leaking private-repo contents into public-repo artifacts.
  • .claude/rules/06-audit-trail.md — the existing lightweight traceability pattern (branch↔ticket, PR↔Jira link, start/PR-opened Jira comments) already used on AAASM-3938/3939/3940, documented rather than replaced with something heavier.
  • CLAUDE.md — one-line update so the "AI tooling" section no longer says .claude/rules/ is empty, now that AAASM-3941 populates it (.claude/skills/ remains pending AAASM-3942).

Validation

  • Content reviewed for accuracy
  • Links verified
  • No unintended scope changes

Checklist

  • PR title follows: [<ticket ID>] <GitEmoji> (<change scope category>): <key point as summary>
  • PR body follows this template

@Chisanan232

Copy link
Copy Markdown
Contributor Author

Claude Code review — AAASM-3941

CI: Red, same org-wide Actions billing-block (see #25) — not caused by this change.

Scope vs ticket: Matches AAASM-3941's AC — all 6 rule files present (01-security06-audit-trail), framed as advisory guidance, with 01-security.md explicitly deferring hard-enforcement (settings.json allow/deny) to sibling Epic AAASM-3926 so the two Epics don't produce conflicting rule ownership. Verified via diff: zero files under .claude/skills/ touched (no overlap with #28, which ran in parallel).

Side effects: The one CLAUDE.md edit in this PR only rewords the sentence AAASM-3940 itself just added ("rules populated, skills still empty") — verified no pre-existing (pre-Epic) content touched.

Note for whoever merges this stack: #27 and #28 both edit that same CLAUDE.md sentence independently (diamond branches off #26, never merged into each other). Whichever of #27/#28 merges to master second will still show "skills still empty" or an equivalent stale claim once both have landed. Recommend a 1-line follow-up fixup commit on master right after both merge — trivial, not worth blocking either PR over.

Frontend: N/A.

Recommend: ready to merge, after #26.

@Chisanan232 Chisanan232 force-pushed the v0.1.0/AAASM-3941/design_claude_rules branch from fa6f76f to 1c6565a Compare July 6, 2026 00:23
@Chisanan232 Chisanan232 merged commit eabc30c into master Jul 6, 2026
@Chisanan232 Chisanan232 deleted the v0.1.0/AAASM-3941/design_claude_rules branch July 6, 2026 00: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.

1 participant