feat(core): add static module registry #88

Merged
pdurlej merged 9 commits from core-module-registry into main 2026-06-17 21:27:43 +02:00
Collaborator

Summary

  • add a static patchwarden-core module registry for the current Contract Run modules
  • add a JSON Schema for patchwarden.core_module_registry.v1
  • move Contract Run module ids/version/kind to core_modules.py so emitted module_results are backed by first-class Core definitions
  • document that this is a read-only Core registry, not dynamic plugin loading

Why

This is the first implementation slice for treating Patchwarden Core modules as product surface: Patchwarden defines how verdict artifacts are produced, while agents remain outside that boundary as sensors/executors under policy.

Verification

  • PYTHONPATH=src python3 -m pytest -q tests/test_core_module_registry.py tests/test_contract_run.py tests/test_cli_evaluate.py
  • PYTHONPATH=src python3 -m pytest -q
  • git diff --check
## Summary - add a static `patchwarden-core` module registry for the current Contract Run modules - add a JSON Schema for `patchwarden.core_module_registry.v1` - move Contract Run module ids/version/kind to `core_modules.py` so emitted `module_results` are backed by first-class Core definitions - document that this is a read-only Core registry, not dynamic plugin loading ## Why This is the first implementation slice for treating Patchwarden Core modules as product surface: Patchwarden defines how verdict artifacts are produced, while agents remain outside that boundary as sensors/executors under policy. ## Verification - `PYTHONPATH=src python3 -m pytest -q tests/test_core_module_registry.py tests/test_contract_run.py tests/test_cli_evaluate.py` - `PYTHONPATH=src python3 -m pytest -q` - `git diff --check`
Collaborator

Iskra judgment

Field Value
Target pdurlej/patchwarden#pull_request#88
Priority p2
Action patchwarden_candidate
Scores reach 4 / impact 4 / confidence 5
Piotr fit high
Effort medium
Labels judge/p2, judge/patchwarden-candidate
Judge iskra via openclaw

Rationale: This is P2 Patchwarden-candidate work because a static Core registry turns contract modules into explicit product surface without adding dynamic plugin risk.

Caveat: Keep the registry read-only and static for now; do not blur it into dynamic plugin loading.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 5,
  "effort_hint": "medium",
  "escalation": {
    "kind": "patchwarden_review",
    "reason": "This defines Patchwarden Core module registry as product surface and should be reviewed in the Patchwarden lane."
  },
  "evidence_refs": [
    {
      "note": "PR adds a static patchwarden-core module registry for current Contract Run modules.",
      "type": "forgejo",
      "value": "pr-title-body-labels-and-target-snapshot"
    },
    {
      "note": "Body states module ids, versions, and kinds move into core_modules.py so emitted module_results are backed by first-class Core definitions.",
      "type": "forgejo",
      "value": "pr-body-summary"
    },
    {
      "note": "Verification lists focused core module registry and contract tests, full pytest, and diff check.",
      "type": "check",
      "value": "pr-body-verification-summary"
    }
  ],
  "impact": 4,
  "judge_actor": {
    "name": "iskra",
    "runtime": "openclaw"
  },
  "judged_at": "2026-06-11T01:05:00Z",
  "labels_to_apply": [
    "judge/p2",
    "judge/patchwarden-candidate"
  ],
  "piotr_fit": "high",
  "priority": "p2",
  "rationale_summary": "This is P2 Patchwarden-candidate work because a static Core registry turns contract modules into explicit product surface without adding dynamic plugin risk.",
  "reach": 4,
  "recommended_next_action": "patchwarden_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "pull_request",
    "number": 88,
    "repo": "pdurlej/patchwarden"
  },
  "target_snapshot": {
    "body_hash": "sha256:40666a42eead6c9e96ca3bdff75ff6ab8b0966864a61915b6d0d8e62c55b4f3b",
    "commit_count": 8,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": "c400d49469b47760b55db9848f8d604080367be6",
    "labels": [],
    "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "state": "open",
    "title_hash": "sha256:efde4a4e60474237739343daa055d6bd56eb1c49362f061ba9eff4d8e3a5b1ef",
    "updated_at": "2026-06-11T01:45:04+02:00"
  },
  "top_caveat": "Keep the registry read-only and static for now; do not blur it into dynamic plugin loading."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/patchwarden#pull_request#88` | | Priority | p2 | | Action | patchwarden_candidate | | Scores | reach 4 / impact 4 / confidence 5 | | Piotr fit | high | | Effort | medium | | Labels | `judge/p2`, `judge/patchwarden-candidate` | | Judge | `iskra` via `openclaw` | **Rationale:** This is P2 Patchwarden-candidate work because a static Core registry turns contract modules into explicit product surface without adding dynamic plugin risk. **Caveat:** Keep the registry read-only and static for now; do not blur it into dynamic plugin loading. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 5, "effort_hint": "medium", "escalation": { "kind": "patchwarden_review", "reason": "This defines Patchwarden Core module registry as product surface and should be reviewed in the Patchwarden lane." }, "evidence_refs": [ { "note": "PR adds a static patchwarden-core module registry for current Contract Run modules.", "type": "forgejo", "value": "pr-title-body-labels-and-target-snapshot" }, { "note": "Body states module ids, versions, and kinds move into core_modules.py so emitted module_results are backed by first-class Core definitions.", "type": "forgejo", "value": "pr-body-summary" }, { "note": "Verification lists focused core module registry and contract tests, full pytest, and diff check.", "type": "check", "value": "pr-body-verification-summary" } ], "impact": 4, "judge_actor": { "name": "iskra", "runtime": "openclaw" }, "judged_at": "2026-06-11T01:05:00Z", "labels_to_apply": [ "judge/p2", "judge/patchwarden-candidate" ], "piotr_fit": "high", "priority": "p2", "rationale_summary": "This is P2 Patchwarden-candidate work because a static Core registry turns contract modules into explicit product surface without adding dynamic plugin risk.", "reach": 4, "recommended_next_action": "patchwarden_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "pull_request", "number": 88, "repo": "pdurlej/patchwarden" }, "target_snapshot": { "body_hash": "sha256:40666a42eead6c9e96ca3bdff75ff6ab8b0966864a61915b6d0d8e62c55b4f3b", "commit_count": 8, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": "c400d49469b47760b55db9848f8d604080367be6", "labels": [], "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "state": "open", "title_hash": "sha256:efde4a4e60474237739343daa055d6bd56eb1c49362f061ba9eff4d8e3a5b1ef", "updated_at": "2026-06-11T01:45:04+02:00" }, "top_caveat": "Keep the registry read-only and static for now; do not blur it into dynamic plugin loading." } <!-- /openclaw.judge.v0 --> ``` </details>
pdurlej deleted branch core-module-registry 2026-06-17 21:27:43 +02:00
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
3 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!88
No description provided.