P1-07: first-class runtime objects + work pane (closes #30) #180

Merged
conductor merged 7 commits from p1-07-work-pane into master 2026-09-06 05:32:19 +01:00
Owner

Resumes the pinned P1-07 implementation (branch preserved, rebased onto master faed039).

What

  • Five typed runtime objects (MemoryQuery, Observation, VerificationResult, Outcome + envelope) in tool_protocol.py, round-trippable
  • Operator emission points: broker memory queries, verification results, task outcomes (incl. failed), degraded subsystems, compaction — wired into response envelope
  • WorkPane: third collapsible column (Ctrl+3) rendering runtime objects by event id — operational index, not narrative
  • Canary canary_p1_07.py registered in ci.yml same-commit; docs/runtime-objects.md; authority-map entry

Verification (local, this resume)

  • 9 frozen TDD tests green (red proven in original session); full suite 523 passed + 84 subtests, zero regressions
  • canary_p1_07 PASSED; canary_full_system PASSED; henxels exit 0; ruff check+format clean
  • Rebase deviations D1-D4 recorded in plan tail (f80abcd)

Closes #30.

Resumes the pinned P1-07 implementation (branch preserved, rebased onto master faed039). ## What - Five typed runtime objects (MemoryQuery, Observation, VerificationResult, Outcome + envelope) in tool_protocol.py, round-trippable - Operator emission points: broker memory queries, verification results, task outcomes (incl. failed), degraded subsystems, compaction — wired into response envelope - WorkPane: third collapsible column (Ctrl+3) rendering runtime objects by event id — operational index, not narrative - Canary canary_p1_07.py registered in ci.yml same-commit; docs/runtime-objects.md; authority-map entry ## Verification (local, this resume) - 9 frozen TDD tests green (red proven in original session); full suite 523 passed + 84 subtests, zero regressions - canary_p1_07 PASSED; canary_full_system PASSED; henxels exit 0; ruff check+format clean - Rebase deviations D1-D4 recorded in plan tail (f80abcd) Closes #30.
conductor added 4 commits 2026-09-06 04:31:01 +01:00
docs(plan): record P1-07 resume deviations D1-D4 (rebase onto faed039, suite 523)
Auto-label / priority-gate (pull_request) Skipped
Auto-label / auto-label (pull_request) Successful in 1s
CI Pipeline / lint (push) Successful in 22s
CI Pipeline / unit-tests (push) Successful in 28s
CI Pipeline / canary (push) Failing after 25s
CI Pipeline / best-practices (push) Successful in 1s
CI Pipeline / documentation (push) Successful in 2s
CI Pipeline / type-check (push) Successful in 20s
CI Pipeline / gitea-offline (push) Successful in 13s
CI Pipeline / required-checks (push) Skipped
CI Pipeline / lint (pull_request) Successful in 27s
CI Pipeline / unit-tests (pull_request) Successful in 36s
CI Pipeline / canary (pull_request) Failing after 23s
CI Pipeline / best-practices (pull_request) Successful in 2s
CI Pipeline / documentation (pull_request) Successful in 1s
CI Pipeline / type-check (pull_request) Successful in 20s
CI Pipeline / gitea-offline (pull_request) Successful in 13s
CI Pipeline / required-checks (pull_request) Skipped
f80abcdde1
gitea-actions Bot added the infrastructurememory labels 2026-09-06 04:31:06 +01:00
conductor added 1 commit 2026-09-06 04:58:26 +01:00
fix(p1-07): quality discharges I-1/I-2/I-3 — resume-path runtime objects, pane cap + channel scoping, straight-read verification link (PR #180)
CI Pipeline / unit-tests (push) Successful in 28s
CI Pipeline / canary (push) Failing after 24s
CI Pipeline / best-practices (push) Successful in 2s
CI Pipeline / documentation (push) Successful in 2s
CI Pipeline / type-check (push) Successful in 20s
CI Pipeline / gitea-offline (push) Successful in 13s
CI Pipeline / lint (pull_request) Successful in 21s
CI Pipeline / unit-tests (pull_request) Successful in 28s
CI Pipeline / canary (pull_request) Failing after 23s
CI Pipeline / best-practices (pull_request) Successful in 3s
CI Pipeline / documentation (pull_request) Successful in 2s
CI Pipeline / type-check (pull_request) Successful in 20s
CI Pipeline / gitea-offline (pull_request) Successful in 13s
CI Pipeline / required-checks (pull_request) Skipped
CI Pipeline / lint (push) Successful in 28s
CI Pipeline / required-checks (push) Skipped
2ac2c5e3c8
- I-2: resolve_pending_approval now emits Outcome/Verification/degraded observations via shared _attach_runtime_objects (approved tool calls were executed work but rendered nothing in the pane).
- I-3: WorkPane growth capped at 120 entries (oldest evicted); objects stamped with channel; visible_entries() exposes channel-scoped view.
- I-1: test asserts the envelope's verification_id equals the emitted verification's id directly (no recompute masking send_task back-fill).
- Red→green: 3 new frozen tests failed on missing behavior first (approval-resume emitted no objects; cap/channel absent).
Author
Owner

Quality gate (independent review) returned Approved with discharges. All three Importants discharged and pushed (2ac2c5e), each with a red-first regression test:

  • I-1: test_failed_verification_backfill_is_straight_read — envelope assertion, no recompute masking.
  • I-2: test_approval_resume_emits_outcome — resolve_pending_approval now emits Outcome/Verification/degraded via shared _attach_runtime_objects (approved tool calls are executed work; they now render in the pane).
  • I-3: test_work_pane_entries_capped_and_channel_scoped — WorkPane capped at 120 (oldest evicted), objects channel-stamped, visible_entries() channel-scoped.
    Minors 4-7 addressed where material (4): degraded failure path; paned channel stamping also corrects the two-view claim; no CoT leakage confirmed.

Local verification for 2ac2c5e: 526 passed + 84 subtests (full suite), canary_p1_07 PASS, ruff check+format clean, henxels exit 0 (21 pre-existing warnings only). Spec gate: PASS (3 minors, non-blocking).

Quality gate (independent review) returned **Approved with discharges**. All three Importants discharged and pushed (`2ac2c5e`), each with a red-first regression test: - **I-1**: `test_failed_verification_backfill_is_straight_read` — envelope assertion, no recompute masking. - **I-2**: `test_approval_resume_emits_outcome` — `resolve_pending_approval` now emits Outcome/Verification/degraded via shared `_attach_runtime_objects` (approved tool calls are executed work; they now render in the pane). - **I-3**: `test_work_pane_entries_capped_and_channel_scoped` — WorkPane capped at 120 (oldest evicted), objects channel-stamped, `visible_entries()` channel-scoped. Minors 4-7 addressed where material (4): degraded failure path; paned channel stamping also corrects the two-view claim; no CoT leakage confirmed. Local verification for `2ac2c5e`: 526 passed + 84 subtests (full suite), canary_p1_07 PASS, ruff check+format clean, henxels exit 0 (21 pre-existing warnings only). Spec gate: PASS (3 minors, non-blocking).
conductor added 1 commit 2026-09-06 05:24:02 +01:00
fix(p1-07): allowlist per-side border/outline CSS in tui_validator (CI canary fix)
CI Pipeline / lint (push) Successful in 22s
CI Pipeline / unit-tests (push) Successful in 34s
CI Pipeline / canary (push) Successful in 24s
CI Pipeline / best-practices (push) Successful in 2s
CI Pipeline / documentation (push) Successful in 2s
CI Pipeline / type-check (push) Successful in 20s
CI Pipeline / gitea-offline (push) Successful in 12s
CI Pipeline / required-checks (push) Skipped
CI Pipeline / lint (pull_request) Successful in 22s
CI Pipeline / unit-tests (pull_request) Successful in 28s
CI Pipeline / canary (pull_request) Successful in 22s
CI Pipeline / best-practices (pull_request) Successful in 2s
CI Pipeline / documentation (pull_request) Successful in 2s
CI Pipeline / type-check (pull_request) Successful in 21s
CI Pipeline / gitea-offline (pull_request) Successful in 13s
CI Pipeline / required-checks (pull_request) Successful in 1s
66e31c69e3
border-left is valid Textual 8.x CSS (verified against installed engine:
App CSS loads border-left/-right/-top/-bottom and outline per-sides fine);
the canary_tui_drift allowlist simply lacked them, so WorkPane's
border-left: solid $secondary failed CI (run 764). Adds the 8 per-side
properties; full drift canary 32/32, suite 526+84 green.
conductor added 1 commit 2026-09-06 05:24:48 +01:00
style(p1-07): ruff-format tui_validator.py (CI format gate on touched file)
CI Pipeline / lint (push) Successful in 21s
CI Pipeline / unit-tests (push) Successful in 29s
CI Pipeline / canary (push) Successful in 25s
CI Pipeline / best-practices (push) Successful in 2s
CI Pipeline / documentation (push) Successful in 2s
CI Pipeline / type-check (push) Successful in 20s
CI Pipeline / gitea-offline (push) Successful in 12s
CI Pipeline / required-checks (push) Skipped
CI Pipeline / lint (pull_request) Successful in 22s
CI Pipeline / unit-tests (pull_request) Successful in 28s
CI Pipeline / canary (pull_request) Successful in 22s
CI Pipeline / best-practices (pull_request) Successful in 2s
CI Pipeline / documentation (pull_request) Successful in 2s
CI Pipeline / type-check (pull_request) Successful in 20s
CI Pipeline / gitea-offline (pull_request) Successful in 13s
CI Pipeline / required-checks (pull_request) Successful in 0s
Auto-version and Release / release (pull_request) Failing after 5s
3ce4cde55b
conductor merged commit 451fbf525d into master 2026-09-06 05:32:19 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: conductor/rune#180