pdurlej/patchwarden

Fork 0

feat(v0): parse Forgejo pull_request event fixtures #31

New issue

Closed

opened 2026-05-26 02:14:40 +02:00 by codex · 2 comments

codex commented

2026-05-26 02:14:40 +02:00

Collaborator

Goal

Prepare service/webhook mode by parsing Forgejo event payloads as data, without running an HTTP server yet.

Scope

Add a small parser module for Forgejo pull_request event JSON.
Add CLI command such as patchwarden forgejo-event --event-file ... that emits normalized PR metadata JSON for the offline pipeline.
Cover opened/synchronize/reopened/labeled/unlabeled event shapes in fixtures.

Acceptance criteria

Parser extracts repo, PR number, title, author, base/head SHA, labels, and event action.
Invalid/non-PR events fail clearly with exit code 2 through CLI.
Output can be used by the offline pipeline metadata issue once both are merged.

No-go

No webhook listener or signature verification.
No network calls.
No Actions-specific assumptions beyond Forgejo event JSON shape.

Spec sources

src/patchwarden/forgejo_client.py
src/patchwarden/pipeline.py
Platform workflow event usage as inspiration: pdurlej/platform:.forgejo/workflows/patchwarden-client-dry-run.yml

## Goal Prepare service/webhook mode by parsing Forgejo event payloads as data, without running an HTTP server yet. ## Scope - Add a small parser module for Forgejo `pull_request` event JSON. - Add CLI command such as `patchwarden forgejo-event --event-file ...` that emits normalized PR metadata JSON for the offline pipeline. - Cover opened/synchronize/reopened/labeled/unlabeled event shapes in fixtures. ## Acceptance criteria - Parser extracts repo, PR number, title, author, base/head SHA, labels, and event action. - Invalid/non-PR events fail clearly with exit code `2` through CLI. - Output can be used by the offline pipeline metadata issue once both are merged. ## No-go - No webhook listener or signature verification. - No network calls. - No Actions-specific assumptions beyond Forgejo event JSON shape. ## Spec sources - `src/patchwarden/forgejo_client.py` - `src/patchwarden/pipeline.py` - Platform workflow event usage as inspiration: `pdurlej/platform:.forgejo/workflows/patchwarden-client-dry-run.yml`

codex added the

labels

2026-05-26 02:14:40 +02:00

claude referenced this issue

2026-05-27 12:08:15 +02:00

docs(v0): add platform dogfood playbook (closes #32) #51

Iskra commented

2026-06-08 03:05:17 +02:00

Collaborator

{
"confidence": 5,
"effort_hint": "medium",
"escalation": {
"kind": "patchwarden_review",
"reason": "Patchwarden parser work should stay aligned with offline pipeline contracts."
},
"evidence_refs": [
{
"note": "Issue prepares service mode by parsing Forgejo pull_request event JSON offline.",
"type": "forgejo",
"value": "issue-title-body-labels-and-target-snapshot"
},
{
"note": "Scope adds parser and CLI output for normalized PR metadata without HTTP server or network calls.",
"type": "forgejo",
"value": "issue-body-scope"
},
{
"note": "Acceptance requires clear extraction fields and exit code 2 for invalid non-PR events.",
"type": "forgejo",
"value": "issue-body-acceptance"
}
],
"impact": 4,
"judge_actor": {
"name": "iskra",
"runtime": "openclaw"
},
"judged_at": "2026-06-08T01:02:00Z",
"labels_to_apply": [
"judge/p2",
"judge/patchwarden-candidate"
],
"piotr_fit": "high",
"priority": "p2",
"rationale_summary": "This is P2 Patchwarden foundation work because offline Forgejo event parsing is a clean prerequisite for later webhook or service mode.",
"reach": 3,
"recommended_next_action": "patchwarden_candidate",
"rerun_reason": "no_prior_judgment",
"schema": "openclaw.judge.v0",
"target": {
"kind": "issue",
"number": 31,
"repo": "pdurlej/patchwarden"
},
"target_snapshot": {
"body_hash": "sha256:fb702393fd68e94d1a8c1c96f0d5862b4962179c72084e315719dc3d3afc5c5f",
"commit_count": null,
"evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
"head_sha": null,
"labels": [
"area:forgejo",
"area:v0-core",
"kind:implementation",
"ready-for-agent"
],
"labels_hash": "sha256:ed53cff2546340696edabb4dfba9cf4615d23bc93102790acf2eed910440a261",
"state": "open",
"title_hash": "sha256:3cd873dace17c376dcbc58c8cfbee9597a2486260f45c5d80e9872ff9dddce78",
"updated_at": "2026-05-27T12:08:15+02:00"
},
"top_caveat": "Keep this offline and fixture-driven; do not smuggle in webhook listener or network behavior."
}

{ "confidence": 5, "effort_hint": "medium", "escalation": { "kind": "patchwarden_review", "reason": "Patchwarden parser work should stay aligned with offline pipeline contracts." }, "evidence_refs": [ { "note": "Issue prepares service mode by parsing Forgejo pull_request event JSON offline.", "type": "forgejo", "value": "issue-title-body-labels-and-target-snapshot" }, { "note": "Scope adds parser and CLI output for normalized PR metadata without HTTP server or network calls.", "type": "forgejo", "value": "issue-body-scope" }, { "note": "Acceptance requires clear extraction fields and exit code 2 for invalid non-PR events.", "type": "forgejo", "value": "issue-body-acceptance" } ], "impact": 4, "judge_actor": { "name": "iskra", "runtime": "openclaw" }, "judged_at": "2026-06-08T01:02:00Z", "labels_to_apply": [ "judge/p2", "judge/patchwarden-candidate" ], "piotr_fit": "high", "priority": "p2", "rationale_summary": "This is P2 Patchwarden foundation work because offline Forgejo event parsing is a clean prerequisite for later webhook or service mode.", "reach": 3, "recommended_next_action": "patchwarden_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 31, "repo": "pdurlej/patchwarden" }, "target_snapshot": { "body_hash": "sha256:fb702393fd68e94d1a8c1c96f0d5862b4962179c72084e315719dc3d3afc5c5f", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [ "area:forgejo", "area:v0-core", "kind:implementation", "ready-for-agent" ], "labels_hash": "sha256:ed53cff2546340696edabb4dfba9cf4615d23bc93102790acf2eed910440a261", "state": "open", "title_hash": "sha256:3cd873dace17c376dcbc58c8cfbee9597a2486260f45c5d80e9872ff9dddce78", "updated_at": "2026-05-27T12:08:15+02:00" }, "top_caveat": "Keep this offline and fixture-driven; do not smuggle in webhook listener or network behavior." }

Iskra added the

judge/p2

judge/patchwarden-candidate

labels

2026-06-08 03:05:17 +02:00

Iskra referenced this issue from pdurlej/judging-claw

2026-06-08 03:05:17 +02:00

[Judging Claw] patchwarden_candidate for pdurlej/patchwarden#31 (p2) #63

claude referenced this issue

2026-06-23 07:21:59 +02:00

feat(controller): PW-G009–G012 + PW-G016 — ClawSweeper controller autonomy loop #107

codex referenced this issue from a pull request that will close it,

2026-06-23 09:52:19 +02:00

Implement Patchwarden June vision slices #111

codex commented

2026-06-23 09:52:31 +02:00

Author

Collaborator

Addressed by #111.

Close basis: patchwarden forgejo-event --event-file ... now normalizes offline Forgejo pull_request event fixtures (opened, synchronize, reopened, labeled, unlabeled) into the same PR metadata shape consumed by pipeline --metadata-file and contract-pr --metadata-file. Non-PR events and missing changed_files fail with exit 2. HTTP listener and signature verification remain separate future service-mode work, not part of this issue.

Addressed by #111. Close basis: `patchwarden forgejo-event --event-file ...` now normalizes offline Forgejo `pull_request` event fixtures (`opened`, `synchronize`, `reopened`, `labeled`, `unlabeled`) into the same PR metadata shape consumed by `pipeline --metadata-file` and `contract-pr --metadata-file`. Non-PR events and missing `changed_files` fail with exit 2. HTTP listener and signature verification remain separate future service-mode work, not part of this issue.

codex added the

flow/review

label

2026-06-23 09:52:34 +02:00