Board: freeze provider-neutral observation contract - #956
Draft
jeffhuber wants to merge 3 commits into
Draft
Conversation
CODE_MOWER_BUILDER:codex
CODE_MOWER_BUILDER:codex
CODE_MOWER_BUILDER:codex
Contributor
Author
Claude audit (merge-authority lane)Head SHA: Claude Audit: PASS Summary: New, additive, self-contained board_observation contract module with a closed JSON-Schema, extensive validation logic, and 17 valid + 13 adversarial fixtures exercising the invariants (timestamp ordering, identity binding, freshness/staleness, primary-route determinism, measurement null-vs-zero semantics, privacy scrubbing of display labels). package_manifest.py change is purely additive. No correctness, security, or contract-violation blockers found. Findings: none. |
This was referenced Sep 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Board needs one closed local read model for session and work activity before presentation or producer work can safely begin. This defines
code_mower.boardObservation.v1, with exact session/work/run and repository/worktree binding, the existing remote-session public lifecycle projection, per-source freshness and coverage, separate requested and observed evidence, deterministic next-actor routing, and explicit unavailable measurements.The offline fixtures cover assigned through merged work, waiting and failure paths, stale and unavailable sources, unlinked process observations, and a strictly covered no-work state. Adversarial cases fail closed on enum/field drift, cross-session or worktree joins, stale live claims, timestamp and head mismatches, unauthorized display labels, missing measurements serialized as zero, and oversized identifiers.
This PR adds no Board UI, lifecycle producer, Slack mapping, cloud event or field, network action, provider mutation, or lease mutation. It remains draft pending refresh onto current main and renewed exact-head validation. The Devin release and qualification campaign are complete. Contract/fixture implementation may merge after its dependencies, independent audit, tests, CI, and gate pass; Graphify release ordering continues to gate Board release #952 rather than this implementation PR.
Validation at exact head
0feb5f98374622956bbc71158c6a1147a0df253c:code-mower/gatepassedCloses #946
Current completion gate
Continue this existing PR under the updated #900/#945 completion plan; do not duplicate it. The review and validation above belong to the recorded historical head. Refresh the branch, rerun the eligible independent exact-head audit and relevant tests/CI/authoritative gate, then reassess readiness. #935 is complete through merged #973 and its resolver is reused by the later #949 producer integration. This roadmap update changes no code, head, draft status, or audit verdict.