three map nodes under-enrich relative to their charters (programmer in coding.yaml, contract-auditor and test-reviewer in poodr-implementation.yaml) #684

New issue

Closed

opened 2026-09-24 17:11:07 +00:00 by jared · 1 comment

jared commented

2026-09-24 17:11:07 +00:00

Owner

Problem

The #676 audit compared enrich lists to charters looking for excess. It also found the reverse in three nodes, where the charter reads a fact the map never supplies:

coding.yaml:11 programmer gets [ticket_brief]; programmer.md:16-20 lists seam-designer, change-auditor, test diff, and test files as inputs.
poodr-implementation.yaml:163 contract-auditor gets [ticket_brief, gate_results]; contract-auditor.md:15 checks the diff against every WORK item in the change-auditor work list.
poodr-implementation.yaml:62 test-reviewer gets [ticket_brief, red_tests, test_changes]; test-reviewer.md:16 reasons about WORK items and retired_tests.
In each case the role either fails the charter rule silently or relies on the handoff field by luck.

Reproduce

n/a: design audit finding, not a runtime defect.

Approach

Per node, decide whether the map or the charter is right, and change one of them. coding.yaml has no change-auditor or seam-designer nodes, so the programmer charter must state those inputs as conditional. The two poodr-implementation nodes likely need the same enrich entries their poodr-ticket-implementation twins carry.

Required work:

One decision and edit per node, recorded in the ticket.
Map tests stay green.

Evidence

#676 audit notes on programmer, contract-auditor, and test-reviewer rows.

Origin

Trigger: #676 enrich-fact audit, run in session 9434289b
Improvised this session: none
Chain: oversized or duplicated brief facts <- enrich lists chosen per node without a per-role byte audit <- DESIGN (map enrich model)
Root candidate: this ticket
Where: the map/charter/composer paths named in the draft
Parent: #676
Session: 9434289b-c48f-4a8a-8add-0d82068e2c3f
Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/9434289b-c48f-4a8a-8add-0d82068e2c3f.jsonl

Skeptic verdict: CREATE (os-backlog:ticket-skeptic, session 9434289b, citations verified).

## Problem The #676 audit compared enrich lists to charters looking for excess. It also found the reverse in three nodes, where the charter reads a fact the map never supplies: - coding.yaml:11 programmer gets `[ticket_brief]`; programmer.md:16-20 lists seam-designer, change-auditor, test diff, and test files as inputs. - poodr-implementation.yaml:163 contract-auditor gets `[ticket_brief, gate_results]`; contract-auditor.md:15 checks the diff against every WORK item in the change-auditor work list. - poodr-implementation.yaml:62 test-reviewer gets `[ticket_brief, red_tests, test_changes]`; test-reviewer.md:16 reasons about WORK items and retired_tests. In each case the role either fails the charter rule silently or relies on the handoff field by luck. ## Reproduce n/a: design audit finding, not a runtime defect. ## Approach Per node, decide whether the map or the charter is right, and change one of them. coding.yaml has no change-auditor or seam-designer nodes, so the programmer charter must state those inputs as conditional. The two poodr-implementation nodes likely need the same enrich entries their poodr-ticket-implementation twins carry. Required work: - [ ] One decision and edit per node, recorded in the ticket. - [ ] Map tests stay green. ## Evidence #676 audit notes on programmer, contract-auditor, and test-reviewer rows. ## Origin - Trigger: #676 enrich-fact audit, run in session 9434289b - Improvised this session: none - Chain: oversized or duplicated brief facts <- enrich lists chosen per node without a per-role byte audit <- DESIGN (map enrich model) - Root candidate: this ticket - Where: the map/charter/composer paths named in the draft - Parent: #676 - Session: 9434289b-c48f-4a8a-8add-0d82068e2c3f - Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/9434289b-c48f-4a8a-8add-0d82068e2c3f.jsonl Skeptic verdict: CREATE (os-backlog:ticket-skeptic, session 9434289b, citations verified).

jared added the

plugin/os-sdlc

filed-by/agent

labels

2026-09-24 17:11:07 +00:00

jared commented

2026-09-24 20:10:15 +00:00

Author

Owner

Resolution

Done: programmer charter marks map-provided inputs conditional; contract-auditor and test-reviewer enrich change-auditor in poodr-implementation.yaml

Evidence: shipped in commit d15b8119 (#691)

Follow-ups: none

## Resolution **Done:** programmer charter marks map-provided inputs conditional; contract-auditor and test-reviewer enrich change-auditor in poodr-implementation.yaml **Evidence:** shipped in commit d15b8119 (#691) **Follow-ups:** none

jared closed this issue

2026-09-24 20:10:15 +00:00