pdurlej/platform

Fork 0

docs(patchwarden): smoke contract-pr workflow #785

Merged

pdurlej merged 1 commit from codex/orders/patchwarden-contract-smoke into main

2026-06-10 23:57:28 +02:00

codex commented

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

Collaborator

Canary status: missing — this PR is the live Patchwarden contract-pr smoke target

Canary Context Pack

Product story

Verify that the platform dogfood workflow now produces the new Patchwarden Contract Run signal after platform PR #782 landed.

What changed

Added one docs/status-only note under state/cycle/ describing the expected smoke signal.

Why it changed

This gives Forgejo Actions a harmless PR to run .forgejo/workflows/patchwarden-client-dry-run.yml against, so we can observe whether patchwarden contract-pr emits artifacts and, when runner-local token material exists, publishes patchwarden/contract plus the marker-backed comment.

Files touched

state/cycle/W6d-patchwarden-contract-pr-smoke-2026-06-10.md

Relevant context

pdurlej/patchwarden#84 — Contract Run evaluator.
pdurlej/patchwarden#85 — patchwarden contract-pr runner.
pdurlej/platform#782 — platform workflow wiring.

Runtime evidence

Pre-PR local check: git diff --check passed. Runtime evidence is expected from this PR's Forgejo workflow run.

Known constraints

This PR should not be used as proof of automerge readiness policy. It only tests the workflow signal from contract-pr.

Explicit out-of-scope

No workflow changes.
No branch protection changes.
No automerge policy changes.
No token/secret changes.

Requested decision

Observe the workflow output, patchwarden/contract status, and Patchwarden comment/artifacts. Merge only if the smoke behaves as expected.

Merge blockers

patchwarden-client-dry-run fails technically.
The contract status/comment is absent when the runner-local Patchwarden token should be available.
The PR touches anything outside the intended docs/status note.

Spec sources read

AGENTS.md — platform PR/canary rules.
state/cycle/ convention from existing cycle notes.
.forgejo/workflows/patchwarden-client-dry-run.yml — smoke target from #782.

No closing issue keyword: this is a smoke PR for the just-merged Patchwarden dogfood wiring.

Canary status: missing — this PR is the live Patchwarden contract-pr smoke target ## Canary Context Pack ### Product story Verify that the platform dogfood workflow now produces the new Patchwarden Contract Run signal after platform PR #782 landed. ### What changed Added one docs/status-only note under `state/cycle/` describing the expected smoke signal. ### Why it changed This gives Forgejo Actions a harmless PR to run `.forgejo/workflows/patchwarden-client-dry-run.yml` against, so we can observe whether `patchwarden contract-pr` emits artifacts and, when runner-local token material exists, publishes `patchwarden/contract` plus the marker-backed comment. ### Files touched - `state/cycle/W6d-patchwarden-contract-pr-smoke-2026-06-10.md` ### Relevant context - `pdurlej/patchwarden#84` — Contract Run evaluator. - `pdurlej/patchwarden#85` — `patchwarden contract-pr` runner. - `pdurlej/platform#782` — platform workflow wiring. ### Runtime evidence Pre-PR local check: `git diff --check` passed. Runtime evidence is expected from this PR's Forgejo workflow run. ### Known constraints This PR should not be used as proof of automerge readiness policy. It only tests the workflow signal from `contract-pr`. ### Explicit out-of-scope - No workflow changes. - No branch protection changes. - No automerge policy changes. - No token/secret changes. ### Requested decision Observe the workflow output, `patchwarden/contract` status, and Patchwarden comment/artifacts. Merge only if the smoke behaves as expected. ### Merge blockers - `patchwarden-client-dry-run` fails technically. - The contract status/comment is absent when the runner-local Patchwarden token should be available. - The PR touches anything outside the intended docs/status note. ## Spec sources read - `AGENTS.md` — platform PR/canary rules. - `state/cycle/` convention from existing cycle notes. - `.forgejo/workflows/patchwarden-client-dry-run.yml` — smoke target from #782. No closing issue keyword: this is a smoke PR for the just-merged Patchwarden dogfood wiring.

codex added 1 commit

2026-06-10 23:52:20 +02:00

docs(patchwarden): smoke contract-pr workflow

base-is-main / guard (pull_request) Successful in 1s

Details

canary-required / collect-diff (pull_request) Successful in 4s

Details

patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s

Details

patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s

Details

canary-required / canary (pull_request) Has been skipped

Details

patchwarden-client-dry-run / dry-run (pull_request) Failing after 20s

Details

patchwarden-pr-sanity / sanity (pull_request) Successful in 1m27s

Details

6e9b071fe7

Add a tiny W6d docs/status note to trigger the platform Patchwarden contract-pr workflow after PR #782.

pdurlej merged commit 1b8dde27c3 into main

2026-06-10 23:57:28 +02:00

pdurlej deleted branch codex/orders/patchwarden-contract-smoke

2026-06-10 23:57:28 +02:00

pdurlej referenced this pull request from a commit

2026-06-10 23:57:29 +02:00

Merge pull request 'docs(patchwarden): smoke contract-pr workflow' (#785) from codex/orders/patchwarden-contract-smoke into main

patchwarden commented

2026-06-11 00:20:05 +02:00

First-time contributor

Patchwarden PR sanity

Status: advisory_findings
PR: 785
Commit: 6e9b071fe769084de2fdb133e6d001a579ab83eb
Security-sensitive label: missing
Authority: advisory model review plus deterministic blockers only
3+3 canary: still alive; this does not replace it

Deterministic findings

No deterministic findings.

Model reviewers

`global-glm` / `glm-5.1:cloud`

Status: ok
Verdict: NOT_OK
high Empty diff prevents verification of claimed changes
- Evidence: Diff section is empty despite PR description claiming file 'state/cycle/W6d-patchwarden-contract-pr-smoke-2026-06-10.md' was added
- Next: Confirm diff rendering is correct; if file exists, rebase or force-push to refresh diff; reviewer must verify actual file content before merge

`global-deepseek` / `deepseek-v4-pro:cloud`

Status: ok
Verdict: OK
Findings: none

`redteam` / `kimi-k2.6:cloud`

Status: ok
Verdict: OK
Findings: none

Policy notes

GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot.
Optional red-team model is enabled only when PLATFORMCTL_PR_SANITY_REDTEAM_MODEL is configured.
Auto-merge is not enabled here.

# Patchwarden PR sanity - Status: `advisory_findings` - PR: `785` - Commit: `6e9b071fe769084de2fdb133e6d001a579ab83eb` - Security-sensitive label: `missing` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings No deterministic findings. ## Model reviewers ### `global-glm` / `glm-5.1:cloud` - Status: `ok` - Verdict: `NOT_OK` - **`high`** Empty diff prevents verification of claimed changes - Evidence: `Diff section is empty despite PR description claiming file 'state/cycle/W6d-patchwarden-contract-pr-smoke-2026-06-10.md' was added` - Next: Confirm diff rendering is correct; if file exists, rebase or force-push to refresh diff; reviewer must verify actual file content before merge ### `global-deepseek` / `deepseek-v4-pro:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `redteam` / `kimi-k2.6:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ## Policy notes - GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot. - Optional red-team model is enabled only when `PLATFORMCTL_PR_SANITY_REDTEAM_MODEL` is configured. - Auto-merge is not enabled here.