baldart 4.85.0 → 4.87.0

package/CHANGELOG.md

@@ -5,6 +5,33 @@ All notable changes to BALDART will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [4.87.0] - 2026-07-01
+
+**Plan-review gate promoted to the cross-tool SSOT (AGENTS.md primitive `1.3.0`).** The "have the plan reviewed by `plan-auditor` + `doc-reviewer` before presenting it" rule lived only in the `CLAUDE.md` stub's Plan Mode — so Codex never ran it. Now that both agents exist on Codex (transpiled to `.codex/agents/`), the tool-agnostic principle is promoted into `AGENTS.md`: **MUST have any non-trivial plan reviewed by `plan-auditor` + `doc-reviewer` (in parallel), fold in their feedback, note the review gate in the plan, and present the ALREADY-reviewed plan — never a raw one.** The Claude-only `EnterPlanMode`/`ExitPlanMode` mechanics stay in `CLAUDE.md`; `AGENTS.md` carries the principle so both tools honor it.
+
+**MINOR** — cross-tool enrichment of the shipped skeleton; no config key, no CLI/layout change, contamination-clean. Consumers pick it up on the next `npx baldart update`. `primitive_version` (AGENTS) 1.2.0 → 1.3.0; CLAUDE stub unchanged (1.1.0).
+
+### Changed
+
+- **`framework/templates/primitives/AGENTS.md`** — added the plan-review MUST after the `codebase-architect` MUST. `primitive_version` → 1.3.0.
+- **`framework/templates/primitives/AGENTS.CHANGELOG.md`** — `1.3.0` entry.
+
+## [4.86.0] - 2026-07-01
+
+**AGENTS.md primitive regains the universal working-discipline rules (primitive `1.2.0`).** The lean v4.83.0 rewrite of the skeleton dropped a handful of everyday-habit MUSTs to stay short — but those are exactly the "conventions an agent can't infer and gets wrong" the file exists to carry. Restored as compact, stack-agnostic rules in the `AGENTS.md` skeleton (the cross-tool SSOT):
+
+- **File navigation**: MUST find real paths via Glob/Grep before reading — never guess a path from naming conventions.
+- **Investigation before claiming**: check git history (`git log --all --grep`) + the actual source before asserting a feature does or does not exist.
+- **Implementation completeness**: MUST verify every plan item / acceptance criterion is wired and functional before marking work DONE — cross-check against the code, not the intent.
+- **Testing convention** (Workflow Gates): when a test fails, fix the *code* not the test (unless the test is wrong); for a new feature, write the failing test first.
+
+**MINOR** — enriches shipped skeleton content; no config key, no CLI/layout change; contamination-clean (no project tokens). Consumers pick it up on the next `npx baldart update` (byte-stable on everything else; adopted overlays untouched). `primitive_version` (AGENTS) 1.1.0 → 1.2.0; CLAUDE stub unchanged (1.1.0).
+
+### Changed
+
+- **`framework/templates/primitives/AGENTS.md`** — added the file-navigation + investigation + implementation-completeness MUSTs and the testing convention in Workflow Gates. `primitive_version` → 1.2.0.
+- **`framework/templates/primitives/AGENTS.CHANGELOG.md`** — `1.2.0` entry.
+
 ## [4.85.0] - 2026-07-01
 
 **Root-file primitives gain universal delegation + quality rules (primitive `1.1.0`).** The v4.83.0 skeletons pointed to `.claude/agents/REGISTRY.md` for agent routing but never stated the single highest-frequency rule in the always-loaded file: *which agent writes what*. The `AGENTS.md` primitive (the cross-tool SSOT Codex also reads) now carries a **MUST**: plan-and-delegate, never write substantial code inline — **code / logic / tests → `coder`**, **UI → `ui-expert`** (dual-tool: Claude Task `subagent_type` / Codex custom agent by name; every other agent routes via `REGISTRY.md`). The `CLAUDE.md` stub's Plan Mode names both agents to match (was `coder` only).

package/VERSION

@@ -1 +1 @@
-4.85.0
+4.87.0

package/framework/templates/primitives/AGENTS.CHANGELOG.md

@@ -4,6 +4,28 @@ Formato: [Keep a Changelog](https://keepachangelog.com/) · [SemVer](https://sem
 Questo è il changelog INTERNO dello skeleton spedito (`primitive_version`), separato dal
 file `AGENTS.md` generato nel consumer e indipendente dal `VERSION` del framework.
 
+## 1.3.0 — 2026-07-01
+
+- **Plan-review gate promoted to the cross-tool SSOT**: MUST have any non-trivial plan
+  reviewed by `plan-auditor` + `doc-reviewer` (in parallel) and fold in their feedback
+  BEFORE presenting it for approval — present the already-reviewed plan, never a raw one.
+  Was Claude-only (CLAUDE.md Plan Mode, before `ExitPlanMode`); now in AGENTS.md so Codex
+  runs it too (both agents exist on Codex via the transpiler). The `EnterPlanMode`/
+  `ExitPlanMode` mechanics stay in CLAUDE.md; AGENTS.md carries the tool-agnostic principle.
+
+## 1.2.0 — 2026-07-01
+
+- Restored the universal **working-discipline** rules the lean v1.0.0 rewrite had dropped
+  (they prevent concrete, recurring failures and are exactly the "conventions an agent
+  can't infer" the file should carry):
+  - MUST find real paths via Glob/Grep before reading — never guess from naming; check git
+    history + source before claiming a feature exists or not.
+  - MUST verify every plan item / acceptance criterion is wired + functional before DONE
+    (cross-check against code, not intent).
+  - Testing convention in Workflow Gates: fix the **code** not the test (unless the test is
+    wrong); for a new feature, write the failing test first.
+- All stack-agnostic; no project tokens (contamination-clean).
+
 ## 1.1.0 — 2026-07-01
 
 - **Implementation-delegation MUST**: plan-and-delegate, never write substantial code

package/framework/templates/primitives/AGENTS.md

@@ -1,4 +1,4 @@
-<!-- baldart-primitive: name=AGENTS primitive_version=1.1.0
+<!-- baldart-primitive: name=AGENTS primitive_version=1.3.0
      This is the SHIPPED SKELETON. The writer (src/utils/root-primitives.js) strips
      this banner, resolves every {{ slot }} / {{> partial }} / {{#flag}} from
      baldart.config.yml (+ package.json/README), merges .baldart/overlays/AGENTS.md,
@@ -28,6 +28,10 @@ mechanics — this file remains the single source of truth for the protocol.
   (Claude Code: Task tool `subagent_type: codebase-architect`; Codex: the custom agent by
   name at `.codex/agents/codebase-architect.toml`). Only if NEITHER mechanism exists,
   degrade explicitly to inline retrieval (code-graph → LSP → Grep) and SAY SO.
+- MUST have any non-trivial plan reviewed BEFORE presenting it for approval — run
+  `plan-auditor` + `doc-reviewer` (in parallel), incorporate their feedback, and note in the
+  plan that the review gate was honored. Present the ALREADY-reviewed plan, never a raw one.
+  (Claude Code enforces this inside plan mode, before `ExitPlanMode` — see `CLAUDE.md`.)
 - MUST delegate implementation to the domain agent — plan and delegate, do NOT write
   substantial code inline: **code / logic / tests → `coder`**; **UI (components, pages,
   styling, visual output) → `ui-expert`**. Same dual-tool invocation as `codebase-architect`
@@ -51,6 +55,11 @@ mechanics — this file remains the single source of truth for the protocol.
   `backup/<YYYYMMDD>-<reason>` tag first). Deleting an already-merged branch is routine.
 - MUST NOT hardcode secrets / tokens / API keys (env vars only), expose internal stack
   traces in responses, or log PII. Validate & parse every external / user input at the boundary.
+- MUST find real file paths via Glob/Grep before reading — never guess a path from naming
+  conventions. Check git history (`git log --all --grep`) and the actual source before
+  claiming a feature does or does not exist.
+- MUST verify every plan item / acceptance criterion is wired and functional before marking
+  work DONE — cross-check each against the code, not the intent.
 - MUST run the testing gates before DONE (see § Workflow Gates).
 
 **Feature-gated (active only when the flag is `true` in `baldart.config.yml`):**
@@ -93,6 +102,9 @@ run a full build every commit.
 **Pre-PR (once, before opening a PR):** full build (must pass) · unit tests (if they exist) ·
 E2E for critical-flow changes.
 
+When a test fails, fix the **code**, not the test — unless the test itself is wrong. For a
+new feature with tests, prefer writing the failing test first, then the minimum code to pass.
+
 ## Quality bar (SHOULD)
 
 - SHOULD respect performance budgets — avoid unbounded or N+1 data access on list paths

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "baldart",
-  "version": "4.85.0",
+  "version": "4.87.0",
   "description": "Claude Agent Framework - Reusable framework for coordinating AI agents and humans in software projects",
   "bin": {
     "baldart": "./bin/baldart.js"