feat(status): add handoff contract registry #137

Merged
pdurlej merged 1 commit from codex/handoff-contract-registry into main 2026-06-23 20:48:49 +02:00
Collaborator

Summary

  • add a first-class read-only external handoff contract registry and patchwarden handoff-contracts
  • back the registry with spec/handoffs/external.v0.json plus JSON Schema
  • make patchwarden status and docs/status.html consume the registry instead of owning duplicated handoff data

Verification

  • PYTHONPATH=src:. python3 -m unittest tests.test_handoff_contract_registry tests.test_cli_status tests.test_status_html tests.test_docs_module_inventory tests.test_artifact_schema_contract
  • PYTHONPATH=src:. python3 -m unittest discover -s tests
  • git diff --check
## Summary - add a first-class read-only external handoff contract registry and `patchwarden handoff-contracts` - back the registry with `spec/handoffs/external.v0.json` plus JSON Schema - make `patchwarden status` and `docs/status.html` consume the registry instead of owning duplicated handoff data ## Verification - `PYTHONPATH=src:. python3 -m unittest tests.test_handoff_contract_registry tests.test_cli_status tests.test_status_html tests.test_docs_module_inventory tests.test_artifact_schema_contract` - `PYTHONPATH=src:. python3 -m unittest discover -s tests` - `git diff --check`
feat(status): add handoff contract registry
All checks were successful
fallow-py / fallow-py-advisory (pull_request) Successful in 15s
863d57de85
Author
Collaborator

Patchwarden contract dry-run for head 863d57de854f12e143231dca915bdcba79bbab5b blocked only on policy_governance, as expected for changes under src/patchwarden/, tests/, spec/, and docs/.

Local verification passed:

  • PYTHONPATH=src:. python3 -m unittest tests.test_handoff_contract_registry tests.test_cli_status tests.test_status_html tests.test_docs_module_inventory tests.test_artifact_schema_contract — 61 OK
  • PYTHONPATH=src:. python3 -m unittest discover -s tests — 630 OK
  • git diff --check — OK

Manual/operator-preference path is intentional for this governance-surface PR. Required external check observed in the dry-run: fallow-py / fallow-py-advisory (pull_request) success on this head.

Patchwarden contract dry-run for head `863d57de854f12e143231dca915bdcba79bbab5b` blocked only on `policy_governance`, as expected for changes under `src/patchwarden/`, `tests/`, `spec/`, and `docs/`. Local verification passed: - `PYTHONPATH=src:. python3 -m unittest tests.test_handoff_contract_registry tests.test_cli_status tests.test_status_html tests.test_docs_module_inventory tests.test_artifact_schema_contract` — 61 OK - `PYTHONPATH=src:. python3 -m unittest discover -s tests` — 630 OK - `git diff --check` — OK Manual/operator-preference path is intentional for this governance-surface PR. Required external check observed in the dry-run: `fallow-py / fallow-py-advisory (pull_request)` success on this head.
pdurlej deleted branch codex/handoff-contract-registry 2026-06-23 20:48:49 +02:00
Collaborator

🏛️ Architect (loop) — endorse. handoff_contracts.py is read-only (registry / hash / render — no write/approval/merge surface; the docstrings restate the write-free / never-merge boundaries), and the win is real SSoT de-duplication: status + status.html now consume one registry (spec/handoffs/external.v0.json + schema) instead of each owning a parallel copy. Pairs with the #136 schema-presence guard so the registry can't advertise an undocumented contract. No core-logic touched, 630 green. No changes requested. — claude (architect loop)

🏛️ Architect ✅ (loop) — endorse. `handoff_contracts.py` is read-only (registry / hash / render — no write/approval/merge surface; the docstrings restate the write-free / never-merge boundaries), and the win is real SSoT de-duplication: `status` + `status.html` now *consume* one registry (`spec/handoffs/external.v0.json` + schema) instead of each owning a parallel copy. Pairs with the #136 schema-presence guard so the registry can't advertise an undocumented contract. No core-logic touched, 630 green. No changes requested. — claude (architect loop)
Sign in to join this conversation.
No reviewers
No labels
agent/claude-code
agent/codex
agent/gemini
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
area:business-model
area:competitive
area:discovery
area:forgejo
area:metrics
area:product-strategy
area:v0-core
cagan-grade-approved
client:platform
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
kind:artifact
kind:decision
kind:dogfood
kind:epic
kind:implementation
kind:research
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
priority:p0
priority:p1
priority:p2
priority:p3
ready-for-agent
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
size/large
size/medium
size/small
size/tiny
size/unknown
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:blocked-on-discovery
status:cagan-grade-review-pending
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:needs-operator-decision
status:operator-needed
status:parked
tier:0-anchor
tier:0-platform-substrate
tier:1-core
tier:1-iskra-value-layer
tier:2-supporting
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
wave:1-foundation
wave:2-positioning
wave:3-validation
wave:4-economics
wave:5-operating
No milestone
No project
No assignees
2 participants
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
pdurlej/patchwarden!137
No description provided.