pdurlej/platform
1
1
Fork 0
Code Issues 44 Pull requests 1 Projects Releases Packages Wiki Activity Actions

docs(identity): document W4b Forgejo MCP reality #439

Closed
codex wants to merge 1 commit from codex/w4b/identity-mcp-reality-check into main
pull from: codex/w4b/identity-mcp-reality-check
merge into: pdurlej:main
Conversation 1 Commits 1 Files changed 5 +170 -12
codex commented 2026-05-24 19:46:52 +02:00
Collaborator
Copy link

Canary status: missing — docs/spec reality-check PR; rely on required Forgejo checks before merge

Canary Context Pack

Product story

W4 needs to close the agent identity risk without launching a new MCP router runtime component. The current safe standard already exists: direct Forgejo API calls with actor PATs from macOS Keychain, with identity verification before and after writes.

What changed

  • Added docs/specs/forgejo-mcp-identity-split-v0/04-current-reality.md.
  • Marked the MCP router plan as a future implementation path, not a W4 blocker.
  • Updated the spec README/tasks/plan with the current standard.
  • Updated W4 output to record W4b reality-check prep.

Why it changed

#56 is valid as a future identity-boundary issue, but W4 does not need the full router to avoid the known failure mode. docs/forgejo-agent-operations.md already forbids MCP writes when they authenticate as pdurlej; this PR makes that reality explicit inside the #56 spec kit.

Files touched

  • docs/specs/forgejo-mcp-identity-split-v0/04-current-reality.md
  • docs/specs/forgejo-mcp-identity-split-v0/README.md
  • docs/specs/forgejo-mcp-identity-split-v0/02-plan.md
  • docs/specs/forgejo-mcp-identity-split-v0/03-tasks.md
  • state/cycle/W4-secrets-access-hardening-output.md

Relevant context

  • #56 Forgejo MCP identity split
  • docs/forgejo-agent-operations.md canonical direct-curl write path
  • W4 secrets/access hardening
  • ADR-0010 cousin identity
  • ADR-0018 agent-visible root causes

Runtime evidence

None. This PR is docs/spec only and performs no runtime mutation.

Known constraints

The MCP router is deferred, not rejected. Admin MCP disabled-by-default remains the desired future invariant.

Explicit out-of-scope

  • No MCP implementation.
  • No runtime service.
  • No token movement or rotation.
  • No Forgejo MCP write usage.
  • No changes to macOS Keychain, BW, Infisical, or RS2000.

Requested decision

Merge as W4b reality check. Treat #56 as future Milestone 06/router work rather than a W4 blocker.

Merge blockers

Any statement that weakens the current rule against MCP writes as pdurlej, or any accidental token/secret disclosure.

Spec sources read

  • docs/forgejo-agent-operations.md — current canonical direct-curl write path and identity rules
  • docs/specs/forgejo-mcp-identity-split-v0/README.md — current spec-kit status and scope
  • docs/specs/forgejo-mcp-identity-split-v0/00-constitution.md — identity invariants
  • docs/specs/forgejo-mcp-identity-split-v0/01-specify.md — acceptance criteria
  • docs/specs/forgejo-mcp-identity-split-v0/02-plan.md — router plan
  • docs/specs/forgejo-mcp-identity-split-v0/03-tasks.md — slice plan
  • state/cycle/W4-secrets-access-hardening-output.md — W4 handling for #56

Refs #56

Canary status: missing — docs/spec reality-check PR; rely on required Forgejo checks before merge ## Canary Context Pack ### Product story W4 needs to close the agent identity risk without launching a new MCP router runtime component. The current safe standard already exists: direct Forgejo API calls with actor PATs from macOS Keychain, with identity verification before and after writes. ### What changed - Added `docs/specs/forgejo-mcp-identity-split-v0/04-current-reality.md`. - Marked the MCP router plan as a future implementation path, not a W4 blocker. - Updated the spec README/tasks/plan with the current standard. - Updated W4 output to record W4b reality-check prep. ### Why it changed #56 is valid as a future identity-boundary issue, but W4 does not need the full router to avoid the known failure mode. `docs/forgejo-agent-operations.md` already forbids MCP writes when they authenticate as `pdurlej`; this PR makes that reality explicit inside the #56 spec kit. ### Files touched - `docs/specs/forgejo-mcp-identity-split-v0/04-current-reality.md` - `docs/specs/forgejo-mcp-identity-split-v0/README.md` - `docs/specs/forgejo-mcp-identity-split-v0/02-plan.md` - `docs/specs/forgejo-mcp-identity-split-v0/03-tasks.md` - `state/cycle/W4-secrets-access-hardening-output.md` ### Relevant context - #56 Forgejo MCP identity split - `docs/forgejo-agent-operations.md` canonical direct-curl write path - W4 secrets/access hardening - ADR-0010 cousin identity - ADR-0018 agent-visible root causes ### Runtime evidence None. This PR is docs/spec only and performs no runtime mutation. ### Known constraints The MCP router is deferred, not rejected. Admin MCP disabled-by-default remains the desired future invariant. ### Explicit out-of-scope - No MCP implementation. - No runtime service. - No token movement or rotation. - No Forgejo MCP write usage. - No changes to macOS Keychain, BW, Infisical, or RS2000. ### Requested decision Merge as W4b reality check. Treat #56 as future Milestone 06/router work rather than a W4 blocker. ### Merge blockers Any statement that weakens the current rule against MCP writes as `pdurlej`, or any accidental token/secret disclosure. ## Spec sources read - `docs/forgejo-agent-operations.md` — current canonical direct-curl write path and identity rules - `docs/specs/forgejo-mcp-identity-split-v0/README.md` — current spec-kit status and scope - `docs/specs/forgejo-mcp-identity-split-v0/00-constitution.md` — identity invariants - `docs/specs/forgejo-mcp-identity-split-v0/01-specify.md` — acceptance criteria - `docs/specs/forgejo-mcp-identity-split-v0/02-plan.md` — router plan - `docs/specs/forgejo-mcp-identity-split-v0/03-tasks.md` — slice plan - `state/cycle/W4-secrets-access-hardening-output.md` — W4 handling for #56 Refs #56
docs(identity): document W4b Forgejo MCP reality
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
Details
canary-required / 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-pr-sanity / sanity (pull_request) Successful in 23s
Details
af35450039
codex commented 2026-05-24 20:19:25 +02:00
Author
Collaborator
Copy link

Integrator note: this fork output has been consolidated into #443.

Please merge #443 instead of this PR. Keeping this PR open temporarily as fallback/source evidence until #443 lands.

Integrator note: this fork output has been consolidated into #443. Please merge #443 instead of this PR. Keeping this PR open temporarily as fallback/source evidence until #443 lands.
codex closed this pull request 2026-05-24 20:25:50 +02:00
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
Required
Details
canary-required / 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-pr-sanity / sanity (pull_request) Successful in 23s
Required
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
W6d-automerge-calibration

   
agent/claude-code
Vistula actor: Claude Code.

  
agent/codex
Vistula actor: Codex.

  
agent/hermes
Vistula actor: Hermes Upstream.

  
agent/iskra
Vistula actor: Iskra.

  
agent/ollama
Vistula actor: Ollama or compatible model.

  
agent/patchwarden
Vistula actor: Patchwarden.

  
automerge-candidate
Candidate for narrow W6d autonomous merge readiness pilot

  
class/security-sensitive
Security-sensitive class of service: smallest coherent PRs and full canary

  
cutover-gate
blocks production declare / final RS2000 replacement

  
dependency/blocked
Vistula dependency state: blocked.

  
dependency/blocks-others
Vistula dependency state: blocks other work.

  
dependency/cross-repo
Vistula dependency scope: cross-repository.

  
dependency/needs-confirmation
Vistula dependency state: needs confirmation.

  
domain:agents
Agent identity, delegation, subagents, routing, or workflows.

  
domain:ci
CI, checks, Actions, runners, or release automation.

  
domain:docs
Documentation, packets, practices, or source-of-truth text.

  
domain:forgejo
Forgejo issues, PRs, labels, repo settings, or identity.

  
domain:infra
Infrastructure, storage, backup, DNS, network, or host substrate.

  
domain:memory
Memory, recall, write gates, salience, or Honcho-adjacent behavior.

  
domain:runtime
Runtime services, deploys, daemons, timers, or live behavior.

  
domain:signal
Signal UX, delivery, context, or outbound behavior.

  
domain:ux
Product UX, operator experience, or conversation ergonomics.

  
flow/architecture
Vistula lifecycle: architecture phase in progress.

  
flow/blocked
Vistula lifecycle: blocked on dependency or decision.

  
flow/deployed
Vistula lifecycle: deployed to runtime where applicable.

  
flow/done
Vistula lifecycle: work merged or otherwise done.

  
flow/implementation
Vistula lifecycle: implementation phase in progress.

  
flow/intake
Vistula lifecycle: incoming signal not refined yet.

  
flow/maintained
Vistula lifecycle: maintained/steady state.

  
flow/observed
Vistula lifecycle: observed after delivery.

  
flow/ready
Vistula lifecycle: ready for architecture or implementation.

  
flow/refining
Vistula lifecycle: Hermes/spec refinement in progress.

  
flow/retired
Vistula lifecycle: retired or intentionally closed.

  
flow/review
Vistula lifecycle: review phase in progress.

  
iterating
Orchestrator/Codex actively amending in response to review — operator can ignore until resolved

  
judge/codex-candidate
Candidate for Codex implementation.

  
judge/hermes-candidate
Candidate for Hermes discovery/research.

  
judge/low-confidence
Judgment confidence is low.

  
judge/needs-refinement
Needs clearer evidence or scope.

  
judge/operator-needed
Needs Piotr decision or input.

  
judge/p0
Urgent, operator attention or unblock now.

  
judge/p1
High value, schedule soon.

  
judge/p2
Useful, normal queue.

  
judge/p3
Low priority, keep but do not chase.

  
judge/park
Parked from current attention.

  
judge/patchwarden-candidate
Needs Patchwarden merge-safety attention.

  
judge/stale-priority
Existing priority judgment appears stale.

  
kind/adr
Vistula work kind: architecture decision record.

  
kind/bug
Vistula work kind: bug fix.

  
kind/chore
Vistula work kind: chore.

  
kind/feature
Vistula work kind: feature.

  
kind/infra
Vistula work kind: infrastructure.

  
kind/ops
Vistula work kind: operations.

  
kind/refactor
Vistula work kind: refactor.

  
kind/research
Vistula work kind: research/discovery.

  
large-impact
Per charter §3 review-by-impact: large impact (≥300 LOC OR new module OR identity/secret change OR architectural pivot) — Oracle pre-review recommended

  
merge/auto
Vistula merge mode: automated merge.

  
merge/manual
Vistula merge mode: manual merge.

  
merge/manual-dependency-conflict
Manual merge reason: dependency conflict.

  
merge/manual-failing-tests
Manual merge reason: failing tests.

  
merge/manual-merge-conflict
Manual merge reason: merge conflict.

  
merge/manual-missing-review
Manual merge reason: missing review.

  
merge/manual-operator-preference
Manual merge reason: operator preference.

  
merge/manual-red-zone
Manual merge reason: red-zone risk.

  
merge/manual-security-sensitive
Manual merge reason: security-sensitive work.

  
merge/manual-unclear-scope
Manual merge reason: unclear scope.

  
merge/manual-unknown
Manual merge reason: unknown.

  
meta
meta-decomposition issue; first agent decomposes into atomic children

  
mode:operator-only
Apply step requires explicit operator approval.

  
mode:patchwarden-iskra-approved
Work eligible for Patchwarden/Iskra approval inside repo policy.

  
mode:safe-auto
Low-risk work eligible for lightweight autonomous handling.

  
needs-operator-decision
PR is blocked on operator yes/no decision — fresh-Claude/Codex won't make this call alone

  
needs-triage
decomposing agent has open questions / unknowns

  
not-ready
reviewed but blocked on a precondition

  
observed/erroring
Vistula observation: erroring.

  
observed/needs-followup
Vistula observation: needs follow-up.

  
observed/pending
Vistula observation: pending.

  
observed/retire-candidate
Vistula observation: retire candidate.

  
observed/unused
Vistula observation: unused.

  
observed/used
Vistula observation: used.

  
operator-emotional
operator-emotional importance (Iskra memory etc.)

  
owner-attention
owner must decide; blocks default path

  
phase/02
Phase 02 cataloging

  
phase/03
Phase 03 control plane (platformctl)

  
priority:p0
Immediate safety, uptime, data, or operator-blocking issue.

  
priority:p1
Important active work; should be pulled soon.

  
priority:p2
Useful normal-priority work.

  
priority:p3
Opportunistic cleanup, research, or backlog work.

  
proposed
auto-generated by meta-decomposition; awaiting human review

  
ready-for-agent
reviewed and pickable; agents may claim per cookbook

  
ready-for-operator
PR is ready for operator review/merge — orchestrator finished its part

  
recovery
disaster recovery / restore semantics relevant

  
review:claude-reviewed
Reviewed by Claude-family agent.

  
review:codex-reviewed
Reviewed by Codex-family agent.

  
review:dziadek-reviewed
Reviewed by Dziadek pinch-point reviewer.

  
review:needs-human
Needs human review before merge or apply.

  
risk/exposure
incorrectly public/private/tailnet/auth/routed

  
risk/process
workflow/review/orchestration may produce bad outcomes

  
risk/product
work may have lost owner/user value

  
risk/runtime
platform may not run, recover, validate, behave reproducibly

  
safety:external-write
May write to external systems, users, rooms, repos, or services.

  
safety:no-prod-mutation
Must not mutate production or live external state.

  
safety:prod-impact
May affect production/runtime behavior.

  
safety:secret-touch
Touches secrets, credentials, auth, or secret-adjacent config.

  
size/large
Vistula size: large.

  
size/medium
Vistula size: medium.

  
size/small
Vistula size: small.

  
size/tiny
Vistula size: tiny.

  
size/unknown
Vistula size: not classified yet.

  
source/adr
Vistula source: ADR.

  
source/agent-generated
Vistula source: agent generated.

  
source/manual
Vistula source: manual entry.

  
source/operator-chat
Vistula source: operator chat.

  
source/voice-note
Vistula source: voice note.

  
status:blocked
Blocked by missing dependency, evidence, access, or operator decision.

  
status:codex-ready
Ready for a Codex implementation pass.

  
status:merged:pending-evidence
Merged but waiting for live or artifact evidence before closure.

  
status:needs-evidence
Needs proof, logs, tests, or operator-visible evidence.

  
status:operator-needed
Requires explicit operator input before safe progress.

  
status:parked
Intentionally deferred; not active campaign work.

  
tier/full
Full review tier per ADR-0007

  
tier/lite
Lite review tier per ADR-0007

  
tier/stacked
Stacked PR (base != main) escape hatch per ADR-0017. Requires consolidator-PR plan in PR body.

  
tier:0-platform-substrate
Data, secrets, backup, storage, or critical platform substrate.

  
tier:1-iskra-value-layer
Iskra/OpenClaw behavior, memory, runtime, or value-layer work.

  
tier:2-tools-products-modules
Tools, products, modules, experiments, and lower-blast-radius work.

  
type:bug
Incorrect behavior or regression.

  
type:chore
Maintenance, cleanup, metadata, or operational task.

  
type:docs
Documentation-only work.

  
type:feat
New capability or product behavior.

  
type:policy
Governance, protocol, boundary, or decision-contract change.

  
type:research
Investigation, spike, or evidence-gathering task.

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
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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!439
No description provided.