fix(manifests): downgrade vault criticality core→standard (sunset module) #754

Closed
opened 2026-06-08 23:07:14 +02:00 by ollama · 1 comment
Collaborator

Spec sources (whitelist)

  • modules/vault/module.yaml line ~15 — criticality: core but lifecycle: sunset
  • modules/INDEX.yaml — vault entry for reference

Extracted context

modules/vault/module.yaml is the only sunset module with criticality: core. Its deprecation target 2026-05-30 is already past (M04 Vault sunset complete). All postgres/redis/edge modules are core; vault should be standard like other sunset modules.

Scope

Single-line change in modules/vault/module.yaml:

  • criticality: corecriticality: standard

Also update modules/INDEX.yaml vault entry if needed.

Acceptance criteria

  • Vault criticality is standard, not core
  • platformctl validate vault --json green
  • INDEX.yaml vault entry consistent

Do NOT read

  • Full repo — only vault module files
  • Do NOT touch vault-bootstrap (separate issue)

Agent notes

  • Recommended executor: Gemini 3.5 Flash
  • Size: Trivial (~2 LOC)
  • Review tier: tier/trivial
  • Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §A3
## Spec sources (whitelist) - `modules/vault/module.yaml` line ~15 — `criticality: core` but `lifecycle: sunset` - `modules/INDEX.yaml` — vault entry for reference ## Extracted context `modules/vault/module.yaml` is the only sunset module with `criticality: core`. Its deprecation target 2026-05-30 is already past (M04 Vault sunset complete). All postgres/redis/edge modules are core; vault should be standard like other sunset modules. ## Scope Single-line change in `modules/vault/module.yaml`: - `criticality: core` → `criticality: standard` Also update `modules/INDEX.yaml` vault entry if needed. ## Acceptance criteria - [ ] Vault criticality is `standard`, not `core` - [ ] `platformctl validate vault --json` green - [ ] INDEX.yaml vault entry consistent ## Do NOT read - Full repo — only vault module files - Do NOT touch vault-bootstrap (separate issue) ## Agent notes - Recommended executor: Gemini 3.5 Flash - Size: Trivial (~2 LOC) - Review tier: tier/trivial - Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §A3
Owner

Closing as already implemented on current main. Evidence: modules/vault/module.yaml now has criticality: standard, lifecycle: sunset, and runtime.orchestrator: external; modules/INDEX.yaml also records vault criticality standard. Verification before close: UV_CACHE_DIR=/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane --extra dev platformctl validate all --json returned exitCode: 0 on all modules.

Closing as already implemented on current main. Evidence: `modules/vault/module.yaml` now has `criticality: standard`, `lifecycle: sunset`, and `runtime.orchestrator: external`; `modules/INDEX.yaml` also records vault criticality `standard`. Verification before close: `UV_CACHE_DIR=/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane --extra dev platformctl validate all --json` returned `exitCode: 0` on all modules.
Sign in to join this conversation.
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
2 participants
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#754
No description provided.