Skip to content

Commit fba0b3e

Browse files
SK-2832: drop serviceaccount severity tag from code-review
Security coverage for serviceaccount/** is already provided by /code-review Step 3 with risk-appropriate severities, so the extra 🔐 tag + blocking severity-floor was unnecessary and could turn minor nits into blockers. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent c1bf5fc commit fba0b3e

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

.claude/commands/code-review.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,6 @@ Read the file `.claude/commands/code-security.md` and follow all of its instruct
7777

7878
**If `GITHUB_ACTIONS` is set:** apply that command's **PR / CI mode** — report only issues introduced by added (`+`) lines; do not raise pre-existing vulnerabilities or whole-project checks the diff does not touch. Do **not** print code-security's standalone per-finding blocks, summary, or risk rating — collect its findings into the single consolidated report defined in **Output (PR / CI mode)** below.
7979

80-
**Serviceaccount emphasis:** when a changed file is under `serviceaccount/**` (auth, credentials, JWT, token lifecycle), prefix its findings with 🔐 and treat any security finding there as **at least High severity** (blocking). This replaces the need for a separate serviceaccount security job.
81-
8280
---
8381

8482
## Final Verdict (local mode only)

0 commit comments

Comments
 (0)