fix(CORRECTION-1/2): D2 producer contract + D3 telemetry wiring #313

Merged
conductor merged 1 commits from correction-1-d2d3 into master 2026-09-09 22:20:53 +01:00
Owner

CORRECTION-1 + CORRECTION-2: D2 producer contract, D3 telemetry wiring (branch correction-1-d2d3, commit f5ee557)

D1 reclassified (no code change): FIXED_DEPTH_ROUTE_INAPPLICABLE_TO_LOCAL_EVIDENCE_TASKS — EXPECTED_ROUTING_LIMITATION / ADAPTIVE_REQUIREMENT. OUTWARD external-evidence semantics preserved unchanged; resolution via ADAPT-2/3/4 routing (LOCAL_FIX and STANDARD_CHANGE bypass OUTWARD). The ADAPT-1 probe runs (#310/#311) stand as measured proof.

CORRECTION-1 (D2): producer contract enforced

  • Worker prompts (all 6 stage templates): artifact file must contain exactly one JSON document; StageCompleted markers to stdout only
  • _load_worker_artifact_file(): canonical pure-JSON; LEGACY_COMPATIBILITY_ONLY permits exactly the known StageCompleted: <STAGE> trailing sentinel (anchored regex — no arbitrary salvage); corrupt JSON never masked
  • Acceptance tests: pure JSON OK; exact sentinel OK; arbitrary trailing text rejected; corrupt JSON rejected; prompt contract verified for all stages

CORRECTION-2 (D3): telemetry wired into production

  • Relay marks tool windows observationally on tool events
  • Executor marks model windows around relay.run(); attempt threaded through _execute_stage → all 8 stage methods
  • PROVEN on a real production-path execution: ADVERSARIAL record wall=0.3006, model_time=0.3001, tool_time=0.3001 — overlapping windows, non-additive model respected
  • Telemetry failures never raise into the stage path

1163 tests pass (was 1158); ruff clean. ADAPT-1 restarts after this merges: fresh B1-v2/B2-v2/B3-v2 pre-registration, frozen SHA, no machinery changes.

## CORRECTION-1 + CORRECTION-2: D2 producer contract, D3 telemetry wiring (branch correction-1-d2d3, commit f5ee557) **D1 reclassified (no code change):** FIXED_DEPTH_ROUTE_INAPPLICABLE_TO_LOCAL_EVIDENCE_TASKS — EXPECTED_ROUTING_LIMITATION / ADAPTIVE_REQUIREMENT. OUTWARD external-evidence semantics preserved unchanged; resolution via ADAPT-2/3/4 routing (LOCAL_FIX and STANDARD_CHANGE bypass OUTWARD). The ADAPT-1 probe runs (#310/#311) stand as measured proof. **CORRECTION-1 (D2): producer contract enforced** - Worker prompts (all 6 stage templates): artifact file must contain exactly one JSON document; StageCompleted markers to stdout only - `_load_worker_artifact_file()`: canonical pure-JSON; LEGACY_COMPATIBILITY_ONLY permits exactly the known `StageCompleted: <STAGE>` trailing sentinel (anchored regex — no arbitrary salvage); corrupt JSON never masked - Acceptance tests: pure JSON OK; exact sentinel OK; arbitrary trailing text rejected; corrupt JSON rejected; prompt contract verified for all stages **CORRECTION-2 (D3): telemetry wired into production** - Relay marks tool windows observationally on tool events - Executor marks model windows around relay.run(); attempt threaded through _execute_stage → all 8 stage methods - PROVEN on a real production-path execution: ADVERSARIAL record wall=0.3006, model_time=0.3001, tool_time=0.3001 — overlapping windows, non-additive model respected - Telemetry failures never raise into the stage path 1163 tests pass (was 1158); ruff clean. **ADAPT-1 restarts after this merges: fresh B1-v2/B2-v2/B3-v2 pre-registration, frozen SHA, no machinery changes.**
conductor added 1 commit 2026-09-09 22:19:19 +01:00
fix(CORRECTION-1/2): D2 producer contract + sentinel-tolerant reader; D3 telemetry wired into production path
CI Pipeline / best-practices (push) Successful in 5s
CI Pipeline / documentation (push) Successful in 3s
Auto-label / priority-gate (pull_request) Skipped
Auto-label / auto-label (pull_request) Successful in 3s
CI Pipeline / canary (push) Successful in 35s
CI Pipeline / lint (push) Successful in 40s
CI Pipeline / type-check (push) Successful in 31s
CI Pipeline / gitea-offline (push) Successful in 15s
CI Pipeline / unit-tests (push) Successful in 59s
CI Pipeline / best-practices (pull_request) Successful in 3s
CI Pipeline / documentation (pull_request) Successful in 3s
CI Pipeline / lint (pull_request) Successful in 36s
CI Pipeline / canary (pull_request) Successful in 31s
Spec gate (headless) / spec-gate (pull_request) Successful in 6s
CI Pipeline / gitea-offline (pull_request) Successful in 15s
CI Pipeline / required-checks (push) Successful in 2s
CI Pipeline / unit-tests (pull_request) Successful in 52s
CI Pipeline / type-check (pull_request) Successful in 30s
CI Pipeline / required-checks (pull_request) Successful in 0s
Auto-version and Release / release (pull_request) Successful in 47s
f5ee557ee3
ADAPT-1 baseline corrections (per handoff: fix D2+D3, record D1 as
FIXED_DEPTH_ROUTE_INAPPLICABLE — no OUTWARD semantics change).

CORRECTION-1 (D2): producer contract enforced
- Worker prompts updated: artifact file must contain exactly one JSON
  document; StageCompleted markers go to stdout ONLY
- _load_worker_artifact_file(): canonical reader requires pure JSON;
  LEGACY_COMPATIBILITY_ONLY permits exactly the known 'StageCompleted:
  <STAGE>' trailing sentinel (regex-anchored, no arbitrary salvage);
  corrupt JSON is never masked
- Acceptance: pure JSON OK; exact sentinel OK; arbitrary trailing text
  rejected; corrupt JSON rejected; all 6 stage prompts carry the contract

CORRECTION-2 (D3): StageAttempt telemetry wired into real production path
- research_event_relay: marks tool windows on tool running/completed/error/
  failed events (observational, never changes classification)
- StageExecutor._run_relay_with_telemetry: marks model windows around
  relay.run(); attempt threaded through _execute_stage into all 8 stage
  methods (telemetry_attempt kwarg, default None)
- PROVEN on a real production-path execution: ADVERSARIAL record shows
  wall=0.3006, model_time=0.3001 (1 window), tool_time=0.3001 (1 window) —
  overlapping by design, non-additive model respected
- telemetry failures never raise into the stage path (observational)

D1 NOT a code correction: recorded as
FIXED_DEPTH_ROUTE_INAPPLICABLE_TO_LOCAL_EVIDENCE_TASKS —
EXPECTED_ROUTING_LIMITATION / ADAPTIVE_REQUIREMENT. OUTWARD external-
evidence semantics preserved unchanged.

1163 tests pass (was 1158); ruff clean.
gitea-actions Bot added the buginfrastructureperception labels 2026-09-09 22:19:28 +01:00

🤖 Headless spec gate (non-blocking, #204 first slice)

Spec-gate agent produced no text output — check runner logs.

Deterministic floors still govern the merge

This comment is advisory. Required checks (lint, tests, canaries, type-check, coverage, dependency lock, impact gate) remain the merge authority per docs/adversarial-review.md proportionality.

🤖 **Headless spec gate (non-blocking, #204 first slice)** Spec-gate agent produced no text output — check runner logs. <details><summary>Deterministic floors still govern the merge</summary> This comment is advisory. Required checks (lint, tests, canaries, type-check, coverage, dependency lock, impact gate) remain the merge authority per docs/adversarial-review.md proportionality.</details>
conductor merged commit a7ac3243db into master 2026-09-09 22:20:53 +01:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: conductor/rune#313