Skip to content

cluster: 6 Simple docs/pattern follow-ups (#96-backlog triage)#101

Merged
kiki830621 merged 7 commits into
mainfrom
idd/cluster-simple-docs-followups
May 19, 2026
Merged

cluster: 6 Simple docs/pattern follow-ups (#96-backlog triage)#101
kiki830621 merged 7 commits into
mainfrom
idd/cluster-simple-docs-followups

Conversation

@kiki830621
Copy link
Copy Markdown
Contributor

@kiki830621 kiki830621 commented May 19, 2026

Refs #60 #62 #63 #78 #90 #91

Summary

Cluster-PR for the 6 Simple-tier issues from the #96-backlog complexity triage (/idd-diagnose batch of 18 → 6 Simple / 12 Plan). All 6 are docs / pattern-copy changes with no behavior change — bundled into one PR with one combined verify. The 12 Plan-tier issues from the same triage are driven separately.

Cluster overview

# Change Commit
#60 batch-and-cluster.md — new ## Cluster-PR eligibility section + idd-implement cross-ref fead69c
#62 usecase-routing.md — decision-tree bulk-solve note 2d50108
#63 usecase-routing.md#44 chain-solve explicit URL link (row 27) ea94b08
#78 idd-issue/SKILL.md + CHANGELOG.md — v2.55.0 multi-finding migration note for CI callers dc947dd
#90 openspec/LANGUAGE.md (new file) — GitHub-side tracker marker convention 08f7c26
#91 spectra-archive/SKILL.mdStep 0: Bootstrap Stage Task List 9b6b360

Adaptations (recorded in per-issue Implementation Complete comments)

Checklist

  • Diagnose — 6 issues, Simple verdict each (batch complexity map)
  • Implement — 6 commits + R2 fix (e49d6cb), Refs #N each
  • Verify — 6-AI R1 CONDITIONAL (2 DA blocking) → R2 PASS (R2 master)
  • Pending: human review of this cluster PR + /idd-close #60 #62 #63 #78 #90 #91 after merge

Generated by /idd-implement cluster-PR mode. Do NOT add a GitHub close trailer (Closes/Fixes/Resolves) — IDD discipline requires manual /idd-close per issue after merge to enforce the checklist gate + per-issue closing summary.

@kiki830621
Copy link
Copy Markdown
Contributor Author

Verify Report — PR #101 (cluster: 6 Simple docs/pattern follow-ups) — Round 1

Engine

5 general-purpose Agents (Requirements / Logic / Security / Regression / Devil's Advocate) + Codex (gpt-5.5). Codex was still running at aggregation time — this master covers 5/6; a Codex addendum will follow if it surfaces anything new.

Aggregate verdict: 🟡 CONDITIONAL PASS — do not merge as-is

4 of 5 Claude reviewers PASS. Devil's Advocate surfaced 2 HIGH blocking findings, both independently confirmed by the coordinator (file-existence facts, not opinion). 4 of the 6 issues (#60 #62 #63 #78) are clean; 2 (#90 #91) need an R2 fix.

5-AI verdict

Reviewer Verdict Note
Requirements PASS 6/6 FULLY addressed; both adaptations (#78/#90) judged to satisfy issue intent. 2 cosmetic nits (see below)
Logic PASS #60 anchor exact-match ✓, #91 TaskCreate 1:1 with Steps 1-8 ✓, #62/#63 disjoint ✓, #90 valid markdown ✓
Security PASS / APPROVE docs-only; no secrets, no auto-close trap, LANGUAGE.md #NN uses literal letters (safe)
Regression PASS 0 blocking regressions; all edits land cleanly, KAC-conformant, no scope creep
Devil's Advocate DO NOT MERGE 2 HIGH blocking — see below. Conceded #60/#62/#63/#78

🔴 Blocking findings (DA, both coordinator-confirmed)

DA-1 — #91 fixed only one of two invocation surfaces (HIGH)

#91 added Step 0 to .claude/skills/spectra-archive/SKILL.md. But .claude/commands/spectra/archive.md also exists, has no Step 0, and does not delegate to the SKILL.md — confirmed:

  • .claude/commands/spectra/archive.md exists (5675 bytes), grep -c 'Step 0'0
  • grep 'skills/spectra-archive\|SKILL.md' in the command file → no delegation reference

/spectra:archive (the command form) resolves through the command file. So #91's Step 0 addition does not reach that invocation surface. #91's issue body + diagnosis both named only the SKILL.md — the command file was missed. (This is the same drift family as #93, but the command file is a 4th copy the #93 issue didn't enumerate.)

DA-2 — #90 created a file at a RESERVED path (HIGH)

#90 created openspec/LANGUAGE.md with a workflow convention. But openspec/LANGUAGE.md is already reserved — confirmed:

  • spectra-discuss SKILL.md:35 — "Try to read openspec/LANGUAGE.md. This file is the project's canonical vocabulary — terms with definition, avoid, why notes"
  • spectra-discuss has a "Vocabulary drift" mechanism (line 252-254) that captures TO openspec/LANGUAGE.md
  • Same reference in .claude/commands/spectra/discuss.md + .agents/skills/spectra-discuss/SKILL.md

openspec/LANGUAGE.md is meant to hold a controlled vocabulary, not a workflow convention. #90's file has zero vocabulary terms — spectra-discuss Step 0 will now load it as a "vocabulary file" with no vocabulary. Wrong home.

R2 fix plan (same PR, per verify-fix-in-same-PR discipline)

Non-blocking nits (Requirements, ACCEPTED for R2 or closing)

Cross-cutting (DA, acknowledged)


Next: R2 fixes #90 + #91 on this branch → re-verify R2.

…efs #90)

R1 verify Devil's Advocate (DA-2, confirmed blocking): openspec/LANGUAGE.md
is a reserved filename — spectra-discuss SKILL.md reads it as the project's
canonical vocabulary (terms with definition/avoid/why) and has a
vocabulary-drift capture mechanism targeting it. The R1 file placed a
workflow convention there with zero vocabulary terms, colliding with that
contract.

Relocate to openspec/CONVENTIONS.md — purpose-built filename for workflow
conventions, parallel to LANGUAGE.md in openspec/, not referenced (= not
reserved) by any skill. Content reframed to state the LANGUAGE.md vs
CONVENTIONS.md distinction explicitly.

R1 master verify: PR #101 #issuecomment-4485236848
@kiki830621
Copy link
Copy Markdown
Contributor Author

Verify Report — PR #101 — Round 2

Aggregate verdict: ✅ PASS — ready to merge

Both R1 blocking findings closed. R2 delta: 1 file relocate (#90) + 1 scoping resolution (#91, user decision). 4 issues (#60 #62 #63 #78) were clean in R1 and untouched by R2.

R1 → R2 closure

R1 finding Severity R2 status
DA-2 — #90 created openspec/LANGUAGE.md at a reserved path (spectra-discuss vocabulary file) HIGH CLOSED — commit e49d6cb relocates the convention to openspec/CONVENTIONS.md. Verified not reserved: grep -rn 'CONVENTIONS.md' across .claude/skills/, .claude/commands/, .agents/, plugins/, openspec/config.yaml → no references. git mv detected as 77%-similarity rename.
DA-1 — #91 fixed only the SKILL.md surface, not .claude/commands/spectra/archive.md HIGH RESOLVED (re-scoped) — see below

DA-1 resolution

On inspection the DA-1 finding was partially overstated. .claude/commands/spectra/archive.md is wrapped in <!-- SPECTRA:START v1.0.2 --><!-- SPECTRA:END --> tool-managed markers — the spectra CLI regenerates that region; hand-editing it is the actual anti-pattern. The SKILL.md surface #91 edited:

Decision (user): keep #91's SKILL.md edit; do not hand-edit the tool-managed command file. The command file's Step-0 gap is a facet of the spectra-archive copy divergence and is folded into #93 — whose 3-copy list is now corrected to 4 (the command file is the 4th). Recorded on:

Why R2 has no separate 6-AI re-ensemble

R2's code delta is a single file relocate (LANGUAGE.mdCONVENTIONS.md, content near-identical) that directly closes the named DA-2 finding — verifiable by the reserved-name grep above. #91 had no code change in R2 (user re-scoping decision). Re-running the 5-agent ensemble would re-review #60/#62/#63/#78 — unchanged since R1 PASS. Same rationale as PR #99 R3 (a finder-specified doc fix needs confirmation, not a fresh ensemble).

Non-blocking nits (carried from R1, ACCEPTED)

Process gap

  • Codex (6th reviewer) never returned — the gpt-5.5 xhigh background run was still executing long past the 5 Claude agents and did not complete before R2 aggregation. R1 + R2 verdicts rest on the 5 Claude reviewers (4 PASS + DA, with both DA findings independently coordinator-confirmed via file-existence checks). Cross-model blind verification was unavailable this run. Per idd-verify Step 2.5d, this is recorded explicitly rather than hidden inside an aggregate "PASS".

Final per-issue state

# R1 R2 Final
#60 PASS ✅ clean
#62 PASS ✅ clean
#63 PASS ✅ clean
#78 PASS ✅ clean
#90 🔴 DA-2 relocated to CONVENTIONS.md ✅ closed
#91 🔴 DA-1 re-scoped (SKILL.md edit stands; command file → #93) ✅ resolved

Next: human review of PR #101 → merge → /idd-close #60 #62 #63 #78 #90 #91 (per-issue closing summary).

@kiki830621 kiki830621 merged commit 0eb419c into main May 19, 2026
@kiki830621 kiki830621 deleted the idd/cluster-simple-docs-followups branch May 19, 2026 07:14
kiki830621 added a commit that referenced this pull request May 19, 2026
… (Refs #103)

README Version History had stopped at v2.62.0; plugin-update Phase 2.5
freshness check signals 1 + 3 + 6 all hit (latest version not in README,
CHANGELOG v2.64.0 not in README, Version History missing 2 same-major
versions). Extended the existing v2.56.0–v2.62.0 multi-version row to
v2.56.0–v2.64.0 and appended the two missing entries inline matching
the existing compression pattern.

v2.63.0: #96-backlog Simple cluster summary (6 docs/reference follow-ups
via cluster-PR #101). v2.64.0: IDD human-in-the-loop NSQL conformance
summary (F1–F4 + 3 follow-up issues filed).
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