jared/cc-os

Watch

Fork 0

POODR map T1: settle build risks R1 (LoopLedger keying) + R2 (empty tools allowlist) #459

New issue

Closed

opened 2026-08-21 16:21:16 +00:00 by jared · 2 comments

jared commented

2026-08-21 16:21:16 +00:00

Owner

Map: #458

Investigate per spec §"Risks to settle": R1 — confirm bound spends key per edge, not per target (loop_ledger.rb, routing_applier.rb); add a regression test if per-edge; committee decision if per-target. R2 — determine whether tools: [] is valid agent frontmatter; if not, Read-only fallback + decision-log entry.

Spec: docs/os-sdlc-rebuild/poodr-map-spec.md (ticket 1); blueprint sections cited there are authoritative.

Map: #458 Investigate per spec §"Risks to settle": R1 — confirm bound spends key per edge, not per target (loop_ledger.rb, routing_applier.rb); add a regression test if per-edge; committee decision if per-target. R2 — determine whether `tools: []` is valid agent frontmatter; if not, Read-only fallback + decision-log entry. Spec: docs/os-sdlc-rebuild/poodr-map-spec.md (ticket 1); blueprint sections cited there are authoritative.

jared added the

plugin/os-sdlc

label

2026-08-21 16:21:16 +00:00

jared referenced this issue

2026-08-21 16:21:20 +00:00

POODR map T2: commit maps/poodr-implementation.yaml + validator test #460

jared added the

wayfinder:task

label

2026-08-21 16:22:30 +00:00

jared commented

2026-08-21 17:24:14 +00:00

Author

Owner

Work started via /loop build session (spec §7 ticket 1). Investigating R1 (LoopLedger keying) and R2 (empty tools allowlist).

jared commented

2026-08-21 17:29:41 +00:00

Author

Owner

Resolution

Done: Settled both build risks. R1: LoopLedger keys bound spends per edge (implementation_id, from_node, verdict) at loop_ledger.rb:46-53 — the required behavior; added regression test test_edges_from_different_nodes_sharing_a_target_keep_separate_counters. R2: Claude Code honors tools: [] (zero-tool agent, documented since v2.1.208); change-interpreter keeps its empty allowlist unchanged.

Evidence: Commit 7662b7a. Suite green: 852 runs, 2150 assertions, 0 failures. Rubocop clean on touched file. Rulings logged as D01/D02 in docs/os-sdlc-rebuild/poodr-map-decisions.md. Note: spec named routing_applier.rb, which does not exist; routing logic lives in next_step.rb (recorded in D01).

Follow-ups: none

## Resolution **Done:** Settled both build risks. R1: LoopLedger keys bound spends per edge (implementation_id, from_node, verdict) at loop_ledger.rb:46-53 — the required behavior; added regression test test_edges_from_different_nodes_sharing_a_target_keep_separate_counters. R2: Claude Code honors tools: [] (zero-tool agent, documented since v2.1.208); change-interpreter keeps its empty allowlist unchanged. **Evidence:** Commit 7662b7a. Suite green: 852 runs, 2150 assertions, 0 failures. Rubocop clean on touched file. Rulings logged as D01/D02 in docs/os-sdlc-rebuild/poodr-map-decisions.md. Note: spec named routing_applier.rb, which does not exist; routing logic lives in next_step.rb (recorded in D01). **Follow-ups:** none

jared closed this issue

2026-08-21 17:29:41 +00:00

jared referenced this issue from a commit

2026-08-21 20:00:42 +00:00

test+docs(os-sdlc): #459 settle R1/R2 — per-edge ledger regression test, decision log D01-D02 (#458)

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

jared/cc-os#459

No description provided.

Rows
Columns