Skip to content

Commit d27ca54

Browse files
committed
chore: Switch finding-id format to mi-*/I-* to avoid Windows case-insensitive collisions
1 parent 6fc6b86 commit d27ca54

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

.claude/rules/commit.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ Rules that apply to every commit made in this repository.
1414
## Content to avoid
1515

1616
- Do NOT include workflow-specific labels such as "Review response", "レビュー対応", or similar. The commit message should describe **what the code does now**, not the process that produced it.
17-
- Do NOT include finding IDs from review documents (e.g. `C-1`, `M-1`, `m-1`, `I-1`). These are internal to the review workflow and have no meaning outside of it.
17+
- Do NOT include finding IDs from review documents (e.g. `C-1`, `M-1`, `mi-1`, `I-1`). These are internal to the review workflow and have no meaning outside of it.
1818

1919
## Trailers
2020

.claude/skills/parallel-review/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ Aggregation procedure:
136136
1. Read each reviewer file.
137137
2. Deduplicate — when multiple reviewers point out the same location with the same intent, merge them into one entry and record the originating reviewers.
138138
3. Group by severity (Critical → Major → Minor → Info).
139-
4. Within each group, assign finding-ids (C-1, C-2, ..., M-1, ..., m-1, ..., i-1).
139+
4. Within each group, assign finding-ids (Critical: C-1, C-2, ...; Major: M-1, M-2, ...; Minor: mi-1, mi-2, ...; Info: I-1, I-2, ...).
140140
5. Read `.claude/skills/parallel-review/templates/review-doc.md` to grasp the template skeleton, then Write to {final_doc_path} following the format rules in this SKILL § Format Rules.
141141

142142
Return value: {doc_path, findings_total, severity_counts: {critical, major, minor, info}, duplicates_merged}

.claude/skills/review-respond/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ Launch procedure:
128128
```
129129
You are responsible for parsing the review document. Read {document_path} and Write the metadata for each finding to {tmp_dir}/findings.json.
130130
131-
Extraction targets: Critical / Major / Minor sections (skip Info). For each finding extract id (C-1, M-1, m-1, etc.) / severity / location / description (body up to the markers) / current_meta (current values of triage / estimate / status / verification; when the same field appears multiple times, take the last value).
131+
Extraction targets: Critical / Major / Minor sections (skip Info). For each finding extract id (C-1, M-1, mi-1, etc.) / severity / location / description (body up to the markers) / current_meta (current values of triage / estimate / status / verification; when the same field appears multiple times, take the last value).
132132
133133
stage classification (based on current_meta):
134134
- markers empty → pending_triage

0 commit comments

Comments
 (0)