docs(secrets): record W4a direct PAT removal #437

Merged
pdurlej merged 1 commit from codex/w4a/remove-direct-pat-report into main 2026-05-24 19:19:50 +02:00
Collaborator

Canary status: missing — state/report-only documentation PR; rely on required Forgejo checks before merge

Canary Context Pack

Product story

W4a closes the deploy-runner direct PAT fallback removal with operator-visible evidence. The deploy lane should now depend on Infisical Token Auth in normal state, with rollback documented but not active.

What changed

  • Added state/reports/w4a-direct-pat-removal-2026-05-24.md with runtime evidence, smoke result, and rollback.
  • Updated state/cycle/W4-secrets-access-hardening-output.md to mark W4a removal complete.
  • Updated state/STATUS_NOW.md with the latest W4a evidence and next steps.

Why it changed

Operator approved w4a-remove-direct-pat-approved; the fallback was removed and a no-op auto-apply smoke confirmed Infisical Token Auth was used.

Files touched

  • state/reports/w4a-direct-pat-removal-2026-05-24.md
  • state/cycle/W4-secrets-access-hardening-output.md
  • state/STATUS_NOW.md

Relevant context

  • W4 secrets/access hardening
  • Issue #265 Token Auth migration
  • Issue #274 Token Auth rotation reminder

Runtime evidence

  • Backup: /var/lib/forgejo-deploy-runner/platformctl-deploy.env.w4a-20260524T170930Z.bak
  • Removed direct fallback key: PLATFORMCTL_FORGEJO_TOKEN
  • Restarted only forgejo-deploy-runner.service
  • Smoke: platformctl-auto-apply.yml run #1948 for matrix-well-known succeeded
  • Run log marker: forgejo_token_source=infisical-token-auth
  • Direct fallback marker count: 0
  • Unhealthy containers: 0
  • Watchdog positive restarts last 30m: 0

Known constraints

No secret values are stored in the repo report. #274 stays open for the 2026-08-12 Token Auth rotation.

Explicit out-of-scope

  • No further runtime mutation in this PR.
  • No Infisical token rotation.
  • No broader W4b/W4c identity or secret-pipeline changes.

Requested decision

Merge the evidence/report PR so #265 can be closed cleanly.

Merge blockers

Any contradiction in runtime evidence, accidental secret disclosure, or failing required checks.

Spec sources read

  • state/STATUS_NOW.md — current operator-facing status
  • state/cycle/W4-secrets-access-hardening-output.md — W4 execution state
  • state/reports/w4a-deploy-runner-token-auth-2026-05-24.md — W4a pre-removal evidence

Closes #265

Canary status: missing — state/report-only documentation PR; rely on required Forgejo checks before merge ## Canary Context Pack ### Product story W4a closes the deploy-runner direct PAT fallback removal with operator-visible evidence. The deploy lane should now depend on Infisical Token Auth in normal state, with rollback documented but not active. ### What changed - Added `state/reports/w4a-direct-pat-removal-2026-05-24.md` with runtime evidence, smoke result, and rollback. - Updated `state/cycle/W4-secrets-access-hardening-output.md` to mark W4a removal complete. - Updated `state/STATUS_NOW.md` with the latest W4a evidence and next steps. ### Why it changed Operator approved `w4a-remove-direct-pat-approved`; the fallback was removed and a no-op auto-apply smoke confirmed Infisical Token Auth was used. ### Files touched - `state/reports/w4a-direct-pat-removal-2026-05-24.md` - `state/cycle/W4-secrets-access-hardening-output.md` - `state/STATUS_NOW.md` ### Relevant context - W4 secrets/access hardening - Issue #265 Token Auth migration - Issue #274 Token Auth rotation reminder ### Runtime evidence - Backup: `/var/lib/forgejo-deploy-runner/platformctl-deploy.env.w4a-20260524T170930Z.bak` - Removed direct fallback key: `PLATFORMCTL_FORGEJO_TOKEN` - Restarted only `forgejo-deploy-runner.service` - Smoke: `platformctl-auto-apply.yml` run #1948 for `matrix-well-known` succeeded - Run log marker: `forgejo_token_source=infisical-token-auth` - Direct fallback marker count: `0` - Unhealthy containers: `0` - Watchdog positive restarts last 30m: `0` ### Known constraints No secret values are stored in the repo report. #274 stays open for the 2026-08-12 Token Auth rotation. ### Explicit out-of-scope - No further runtime mutation in this PR. - No Infisical token rotation. - No broader W4b/W4c identity or secret-pipeline changes. ### Requested decision Merge the evidence/report PR so #265 can be closed cleanly. ### Merge blockers Any contradiction in runtime evidence, accidental secret disclosure, or failing required checks. ## Spec sources read - `state/STATUS_NOW.md` — current operator-facing status - `state/cycle/W4-secrets-access-hardening-output.md` — W4 execution state - `state/reports/w4a-deploy-runner-token-auth-2026-05-24.md` — W4a pre-removal evidence Closes #265
docs(secrets): record W4a direct PAT removal
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Has been skipped
patchwarden-pr-sanity / sanity (pull_request) Successful in 22s
19be0e9917
Sign in to join this conversation.
No reviewers
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
1 participant
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/platform!437
No description provided.