fix(patchwarden): use patchwarden actor for safe-docs review #834
No reviewers
Labels
No labels
W6d-automerge-calibration
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
automerge-candidate
class/security-sensitive
cutover-gate
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
iterating
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
large-impact
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
meta
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
needs-operator-decision
needs-triage
not-ready
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
operator-emotional
owner-attention
phase/02
phase/03
priority:p0
priority:p1
priority:p2
priority:p3
proposed
ready-for-agent
ready-for-operator
recovery
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
risk/exposure
risk/process
risk/product
risk/runtime
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:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tier/full
tier/lite
tier/stacked
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
No milestone
No project
No assignees
4 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
pdurlej/platform!834
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "codex/patchwarden-review-actor"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Canary status: required — Patchwarden PR sanity workflow change
Summary
patchwarden-pr-sanityuseFORGEJO_TOKEN_PATCHWARDENas the formal safe-docs/status review actorWhy
Patchwarden can already post a clean PASS sanity comment, but Platform PRs can still be blocked by protected-branch approval rules. This moves the Platform dogfood flow closer to the June 2026 Patchwarden vision: a clean safe-docs/status PR gets a real Patchwarden-owned approval review, while merge authority stays with branch protection/automerge.
Checks
python3 -m pytest control-plane/platformctl/tests/test_pr_sanity.py -qgit diff --checkRuntime / secrets
No runtime mutation. No secret values printed, written, or committed. This only changes the workflow's actor selection when a runner already has
FORGEJO_TOKEN_PATCHWARDEN.Refs pdurlej/platform#823
Refs pdurlej/patchwarden#237
Patchwarden PR sanity
Operator signal: ⚠️ WAIT - automation should rerun or inspect reviewer health before owner attention.
Automerge signal: ❌ NOT READY - no unattended merge or APPROVED review should be published.
Verdict: ⚠️ WAIT - deterministic checks are clean, but reviewer health is degraded; this is not an approval signal.
Next step: Rerun PR sanity or inspect the degraded reviewer lane before unattended merge.
83446dd02f7a2ea705a37875eca81d6ae12e1e270e1eligible_sanity_cleandegradedpresentglm-5.2:cloud,deepseek-v4-pro:cloud,kimi-k2.7-code:cloudWhat I checked
9clean3Approval Handoff
not_ready_degraded_reviewer_healthSignal Board
cleandegradednot_ready_degraded_reviewer_healthineligibleautomation first.forgejo/workflows/patchwarden-client-dry-run.yml,.forgejo/workflows/patchwarden-pr-sanity.yml,control-plane/platformctl/ci/automerge_readiness.py,control-plane/platformctl/ci/pr_sanity.py, ... (+2 more)class/security-sensitiveclass/security-sensitiveis present.🧭 Merge authority: branch protection and automerge controller remain authoritative.
Required Fixes
No deterministic blockers.
Non-blocking Notes
infosensitive-path-touchedSensitive path touched -.forgejo/workflows/patchwarden-client-dry-run.yml.forgejo/workflows/patchwarden-client-dry-run.ymlinfosensitive-path-touchedSensitive path touched -.forgejo/workflows/patchwarden-pr-sanity.yml.forgejo/workflows/patchwarden-pr-sanity.ymlReviewer Details
Model reviewer lanes
global-glm/glm-5.2:cloudskipped-global-deepseek/deepseek-v4-pro:cloudskipped-redteam/kimi-k2.7-code:cloudskipped-Policy notes
Operator live approval present for #834: branch is current and all latest CI/Patchwarden contexts are green. Approval recorded by Iskra as merge actor for the Patchwarden review-actor repair.