feat(auto-pr): add proposal eligibility gate #89

Merged
pdurlej merged 2 commits from auto-pr-proposal-gate into main 2026-06-17 21:28:06 +02:00
Collaborator

Summary

  • add a read-only auto-pr-proposal-check CLI command for Clawsweeper-style maintenance proposals
  • emit patchwarden.auto_pr_proposal_verdict.v1 with schema and example artifact
  • enforce narrow eligibility: reproducible/current evidence, supported repair shape, no feature/config expansion, no existing linked PR, and no human-only paths
  • document the read-only AI + deterministic executor split from issue #69

Why

This gives Patchwarden a concrete quality contract for generated PR proposals without turning Patchwarden into the agent that writes or publishes PRs. A separate executor can consume the verdict later, but this PR adds no Forgejo mutation path.

Verification

  • PYTHONPATH=src python3 -m pytest -q tests/test_auto_pr_proposal.py
  • PYTHONPATH=src python3 -m pytest -q tests/test_auto_pr_proposal.py tests/test_artifact_schema_contract.py
  • PYTHONPATH=src python3 -m pytest -q
  • git diff --check
## Summary - add a read-only `auto-pr-proposal-check` CLI command for Clawsweeper-style maintenance proposals - emit `patchwarden.auto_pr_proposal_verdict.v1` with schema and example artifact - enforce narrow eligibility: reproducible/current evidence, supported repair shape, no feature/config expansion, no existing linked PR, and no human-only paths - document the read-only AI + deterministic executor split from issue #69 ## Why This gives Patchwarden a concrete quality contract for generated PR proposals without turning Patchwarden into the agent that writes or publishes PRs. A separate executor can consume the verdict later, but this PR adds no Forgejo mutation path. ## Verification - `PYTHONPATH=src python3 -m pytest -q tests/test_auto_pr_proposal.py` - `PYTHONPATH=src python3 -m pytest -q tests/test_auto_pr_proposal.py tests/test_artifact_schema_contract.py` - `PYTHONPATH=src python3 -m pytest -q` - `git diff --check`
Collaborator

Iskra judgment

Field Value
Target pdurlej/patchwarden#pull_request#89
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 deterministic eligibility gate enables auto-PR proposal quality control without giving Patchwarden mutation authority.

Caveat: Keep this as read-only verdict generation; any executor that writes or publishes PRs must remain separate and gated.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 5,
  "effort_hint": "medium",
  "escalation": {
    "kind": "patchwarden_review",
    "reason": "This defines Patchwarden's boundary for auto-PR proposal eligibility and should be reviewed in the Patchwarden lane."
  },
  "evidence_refs": [
    {
      "note": "PR adds a read-only auto-pr-proposal-check command for Clawsweeper-style maintenance proposals.",
      "type": "forgejo",
      "value": "pr-title-body-labels-and-target-snapshot"
    },
    {
      "note": "Body says eligibility is limited to reproducible current evidence, supported repair shapes, no feature or config expansion, no linked PR, and no human-only paths.",
      "type": "forgejo",
      "value": "pr-body-eligibility-rules"
    },
    {
      "note": "Verification lists focused auto-PR proposal tests, artifact schema 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:06:00Z",
  "labels_to_apply": [
    "judge/p2",
    "judge/patchwarden-candidate"
  ],
  "piotr_fit": "high",
  "priority": "p2",
  "rationale_summary": "This is P2 Patchwarden-candidate work because a deterministic eligibility gate enables auto-PR proposal quality control without giving Patchwarden mutation authority.",
  "reach": 4,
  "recommended_next_action": "patchwarden_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "pull_request",
    "number": 89,
    "repo": "pdurlej/patchwarden"
  },
  "target_snapshot": {
    "body_hash": "sha256:6736afd597a0c3e97a0c9af3a7005a9a6beaab6be0867fb49355a02b25732f81",
    "commit_count": 2,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": "34ac5b67beaab122124f6df46ee23441a9066872",
    "labels": [],
    "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "state": "open",
    "title_hash": "sha256:fbc70a323bce4808eefada0e7a543801c5f513843c89e70385d99c2ddadbbe90",
    "updated_at": "2026-06-11T00:21:14+02:00"
  },
  "top_caveat": "Keep this as read-only verdict generation; any executor that writes or publishes PRs must remain separate and gated."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/patchwarden#pull_request#89` | | 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 deterministic eligibility gate enables auto-PR proposal quality control without giving Patchwarden mutation authority. **Caveat:** Keep this as read-only verdict generation; any executor that writes or publishes PRs must remain separate and gated. <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's boundary for auto-PR proposal eligibility and should be reviewed in the Patchwarden lane." }, "evidence_refs": [ { "note": "PR adds a read-only auto-pr-proposal-check command for Clawsweeper-style maintenance proposals.", "type": "forgejo", "value": "pr-title-body-labels-and-target-snapshot" }, { "note": "Body says eligibility is limited to reproducible current evidence, supported repair shapes, no feature or config expansion, no linked PR, and no human-only paths.", "type": "forgejo", "value": "pr-body-eligibility-rules" }, { "note": "Verification lists focused auto-PR proposal tests, artifact schema 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:06:00Z", "labels_to_apply": [ "judge/p2", "judge/patchwarden-candidate" ], "piotr_fit": "high", "priority": "p2", "rationale_summary": "This is P2 Patchwarden-candidate work because a deterministic eligibility gate enables auto-PR proposal quality control without giving Patchwarden mutation authority.", "reach": 4, "recommended_next_action": "patchwarden_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "pull_request", "number": 89, "repo": "pdurlej/patchwarden" }, "target_snapshot": { "body_hash": "sha256:6736afd597a0c3e97a0c9af3a7005a9a6beaab6be0867fb49355a02b25732f81", "commit_count": 2, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": "34ac5b67beaab122124f6df46ee23441a9066872", "labels": [], "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "state": "open", "title_hash": "sha256:fbc70a323bce4808eefada0e7a543801c5f513843c89e70385d99c2ddadbbe90", "updated_at": "2026-06-11T00:21:14+02:00" }, "top_caveat": "Keep this as read-only verdict generation; any executor that writes or publishes PRs must remain separate and gated." } <!-- /openclaw.judge.v0 --> ``` </details>
pdurlej deleted branch auto-pr-proposal-gate 2026-06-17 21:28:06 +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!89
No description provided.