From fc5e3e8d6cbadd8b370591c3ccda97229040ebba Mon Sep 17 00:00:00 2001 From: adilallo <39313955+adilallo@users.noreply.github.com> Date: Tue, 18 Aug 2026 18:42:22 -0600 Subject: [PATCH] Keep ticket IDs out of comments, commits, and PR copy. Branch names still carry the ticket; the code and PR prose should not. Co-authored-by: Cursor --- .cursor/rules/coding-guidelines.mdc | 10 +++++++++- AGENTS.md | 2 ++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/.cursor/rules/coding-guidelines.mdc b/.cursor/rules/coding-guidelines.mdc index 3119783..74401cb 100644 --- a/.cursor/rules/coding-guidelines.mdc +++ b/.cursor/rules/coding-guidelines.mdc @@ -47,7 +47,15 @@ When your changes create orphans: The test: Every changed line should trace directly to the user's request. -## 4. Goal-Driven Execution +## 4. Comments, commits, and PRs + +**Let the code talk. Keep tickets out of prose.** + +- No comment when the names already say it. +- No Linear IDs (`CR-123`) in comments, commit messages, or PR titles/bodies. Branch names still use `adilallo//CR--…` (see `branch-naming.mdc`). Link the issue from Linear, not from the PR copy. +- Don't narrate Figma→code mappings the prop already encodes (`palette="inverse"` needs no "this is Invert" comment). + +## 5. Goal-Driven Execution **Define success criteria. Loop until verified.** diff --git a/AGENTS.md b/AGENTS.md index 766871d..b987047 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -46,6 +46,8 @@ priors — they reflect deliberate decisions. 6. **No new pathname-sniffing chrome.** Compose chrome via group/nested layouts, not `usePathname()` checks. (`ConditionalNavigation` is the sole tolerated exception — it carries SSR session state.) +7. **No Linear ticket IDs in comments, commits, or PR copy.** Branch names + still carry `CR-`. See `coding-guidelines.mdc`. ## Legacy / scaffolding