chore(modules): add first metadata completion batch #577

Merged
pdurlej merged 1 commit from deepseek/m05-first-10-modules into main 2026-05-28 23:13:37 +02:00
Collaborator

Canary status: missing - fire canary 3+3 manually before merge

Summary

Closes #561.

Adds the first 10-module source/artifact metadata completion batch for M05. This is metadata-only: no runtime tags changed, no compose changes, no digest changes, no service restart path, and no upgrade execution.

Modules updated

  • dashboard
  • shelfmark
  • excalidraw-app
  • excalidraw-room
  • matrix-well-known
  • ntfy
  • np-meerkat-frontend
  • coredns
  • mirotalk-sfu
  • voice-transcription

Selection rationale

These modules already had concrete runtime.image_observed values and did not require touching Forgejo, secrets/auth modules, DB-heavy modules, or runtime topology. Public/stateless modules are included only where the change is strictly catalog metadata copied from already observed runtime state.

Digest-only images are marked as manual/pinned instead of pretending there is a known upstream tag. Local images are marked platform-local and platform-local-build.

Validation

  • UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json - pass
  • UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli update-report all --json - pass

Update-report after this batch: withSource=19, withArtifact=19, missingSource=69, missingArtifact=69.

Spec sources read

  • Issue #561 - task scope and acceptance criteria.
  • schema/module.schema.json - source/artifact field contract.
  • Existing completed module manifests for local convention examples.
  • The 10 changed module manifests - exact observed image/source fields.

Out of scope

  • Runtime upgrades.
  • Compose changes.
  • Digest pinning changes.
  • Module loader behavior.
  • Secrets/auth module cleanup.
  • DB-heavy module metadata.

Canary Context Pack

Product story

M05 needs enough source/artifact metadata to make M09 upgrade work systematic instead of ad hoc. This batch increases coverage without changing runtime behavior.

What changed

Added spec.source and spec.artifact metadata for 10 selected modules.

Why it changed

M09 upgrade packets need stable current/candidate source and artifact fields.

Files touched

10 modules/*/module.yaml manifests.

Relevant context

M05 module metadata issues #558-#561 and module schema source/artifact contract.

Runtime evidence

No runtime action. Metadata is copied from existing runtime.image_observed values already present in manifests.

Known constraints

This PR intentionally avoids runtime-impacting modules where metadata requires deeper source archaeology.

Explicit out-of-scope

No runtime migration, no upgrade, no compose edit, no service restart.

Requested decision

Approve as metadata-only M05 batch after required canary/review.

Merge blockers

Schema validation failure, incorrect metadata shape, or evidence that this accidentally changes runtime behavior.

Canary status: missing - fire canary 3+3 manually before merge ## Summary Closes #561. Adds the first 10-module source/artifact metadata completion batch for M05. This is metadata-only: no runtime tags changed, no compose changes, no digest changes, no service restart path, and no upgrade execution. ## Modules updated - `dashboard` - `shelfmark` - `excalidraw-app` - `excalidraw-room` - `matrix-well-known` - `ntfy` - `np-meerkat-frontend` - `coredns` - `mirotalk-sfu` - `voice-transcription` ## Selection rationale These modules already had concrete `runtime.image_observed` values and did not require touching Forgejo, secrets/auth modules, DB-heavy modules, or runtime topology. Public/stateless modules are included only where the change is strictly catalog metadata copied from already observed runtime state. Digest-only images are marked as manual/pinned instead of pretending there is a known upstream tag. Local images are marked `platform-local` and `platform-local-build`. ## Validation - `UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` - pass - `UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli update-report all --json` - pass Update-report after this batch: `withSource=19`, `withArtifact=19`, `missingSource=69`, `missingArtifact=69`. ## Spec sources read - Issue #561 - task scope and acceptance criteria. - `schema/module.schema.json` - source/artifact field contract. - Existing completed module manifests for local convention examples. - The 10 changed module manifests - exact observed image/source fields. ## Out of scope - Runtime upgrades. - Compose changes. - Digest pinning changes. - Module loader behavior. - Secrets/auth module cleanup. - DB-heavy module metadata. ## Canary Context Pack ### Product story M05 needs enough source/artifact metadata to make M09 upgrade work systematic instead of ad hoc. This batch increases coverage without changing runtime behavior. ### What changed Added `spec.source` and `spec.artifact` metadata for 10 selected modules. ### Why it changed M09 upgrade packets need stable current/candidate source and artifact fields. ### Files touched 10 `modules/*/module.yaml` manifests. ### Relevant context M05 module metadata issues #558-#561 and module schema source/artifact contract. ### Runtime evidence No runtime action. Metadata is copied from existing `runtime.image_observed` values already present in manifests. ### Known constraints This PR intentionally avoids runtime-impacting modules where metadata requires deeper source archaeology. ### Explicit out-of-scope No runtime migration, no upgrade, no compose edit, no service restart. ### Requested decision Approve as metadata-only M05 batch after required canary/review. ### Merge blockers Schema validation failure, incorrect metadata shape, or evidence that this accidentally changes runtime behavior.
chore(modules): add first metadata completion batch
Some checks failed
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s
platformctl plan / auto-apply scope (pull_request) Successful in 26s
base-is-main / guard (pull_request) Successful in 1s
canary-required / canary (pull_request) Successful in 14s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 22s
patchwarden-pr-sanity / sanity (pull_request) Failing after 2m16s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 5s
0907146137
pdurlej deleted branch deepseek/m05-first-10-modules 2026-05-28 23:13:37 +02:00
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 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!577
No description provided.