fix(manifests): align kan-* change_policy profiles to user-app/default-v1 #753

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

Spec sources (whitelist)

  • modules/kan-mcp/module.yamlclassification.area: user-app, change_policy.profile: platform/default-v1
  • modules/kan-migrate/module.yaml — same
  • modules/kan-postgres/module.yaml — same
  • modules/kan-web/module.yaml — same
  • modules/kanboard/module.yaml — reference: correct (area: user-app + profile: user-app/default-v1)
  • Any 1-2 other user-app modules for verification of the correct pattern

Extracted context

4 kan-* modules are classified as area: user-app but carry change_policy.profile: platform/default-v1. This is inconsistent with 24 other user-app modules including kanboard which correctly use user-app/default-v1.

platform/default-v1 is for platform-tier modules (traefik, postgres, redis, honcho-*, infisical, forgejo, etc.).

Scope

Change change_policy.profile from platform/default-v1 to user-app/default-v1 in all 4 manifests.

Do NOT touch any other fields.

Acceptance criteria

  • All 4 kan-* modules validate: platformctl validate kan-mcp kan-migrate kan-postgres kan-web --json green
  • All 4 match kanboard pattern (area + profile consistent)

Do NOT read

  • Full repo — only the 5-6 files listed above

Agent notes

  • Recommended executor: Gemini 3.5 Flash
  • Size: Trivial (~8 LOC across 4 files)
  • Review tier: tier/trivial
  • Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §A4
## Spec sources (whitelist) - `modules/kan-mcp/module.yaml` — `classification.area: user-app`, `change_policy.profile: platform/default-v1` - `modules/kan-migrate/module.yaml` — same - `modules/kan-postgres/module.yaml` — same - `modules/kan-web/module.yaml` — same - `modules/kanboard/module.yaml` — reference: correct (`area: user-app` + `profile: user-app/default-v1`) - Any 1-2 other user-app modules for verification of the correct pattern ## Extracted context 4 kan-* modules are classified as `area: user-app` but carry `change_policy.profile: platform/default-v1`. This is inconsistent with 24 other user-app modules including `kanboard` which correctly use `user-app/default-v1`. `platform/default-v1` is for platform-tier modules (traefik, postgres, redis, honcho-*, infisical, forgejo, etc.). ## Scope Change `change_policy.profile` from `platform/default-v1` to `user-app/default-v1` in all 4 manifests. Do NOT touch any other fields. ## Acceptance criteria - [ ] All 4 kan-* modules validate: `platformctl validate kan-mcp kan-migrate kan-postgres kan-web --json` green - [ ] All 4 match kanboard pattern (area + profile consistent) ## Do NOT read - Full repo — only the 5-6 files listed above ## Agent notes - Recommended executor: Gemini 3.5 Flash - Size: Trivial (~8 LOC across 4 files) - Review tier: tier/trivial - Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §A4
Collaborator

Iskra judgment

Field Value
Target pdurlej/platform#issue#753
Priority p3
Action codex_candidate
Scores reach 3 / impact 3 / confidence 5
Piotr fit medium
Effort tiny
Labels judge/p3, judge/codex-candidate
Judge iskra via openclaw

Rationale: This is P3 Codex-ready manifest hygiene because four Kan user-app modules have inconsistent change-policy profiles and the fix is tightly scoped.

Caveat: Keep the change limited to the four profile fields and verify validation without changing classification, ownership, or platform-tier policy.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 5,
  "effort_hint": "tiny",
  "escalation": {
    "kind": "none",
    "reason": ""
  },
  "evidence_refs": [
    {
      "note": "Issue identifies four kan-* user-app module manifests using platform/default-v1 change policy instead of user-app/default-v1.",
      "type": "forgejo",
      "value": "issue-title-body-labels-and-target-snapshot"
    },
    {
      "note": "Body cites kanboard and other user-app modules as the correct reference pattern.",
      "type": "forgejo",
      "value": "issue-body-reference-pattern"
    },
    {
      "note": "Scope is limited to changing change_policy.profile in exactly four manifests without touching other fields.",
      "type": "forgejo",
      "value": "issue-body-scope"
    }
  ],
  "impact": 3,
  "judge_actor": {
    "name": "iskra",
    "runtime": "openclaw"
  },
  "judged_at": "2026-06-15T01:03:00Z",
  "labels_to_apply": [
    "judge/p3",
    "judge/codex-candidate"
  ],
  "piotr_fit": "medium",
  "priority": "p3",
  "rationale_summary": "This is P3 Codex-ready manifest hygiene because four Kan user-app modules have inconsistent change-policy profiles and the fix is tightly scoped.",
  "reach": 3,
  "recommended_next_action": "codex_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "issue",
    "number": 753,
    "repo": "pdurlej/platform"
  },
  "target_snapshot": {
    "body_hash": "sha256:c07b0032e70cf4d5dd9486286773ccef234a2d4a620f953a545c4692e6782385",
    "commit_count": null,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": null,
    "labels": [],
    "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "state": "open",
    "title_hash": "sha256:3446ad0625f3c10c558d31e26fb1303713383cbcbabbb34c7578297bb296dc20",
    "updated_at": "2026-06-08T23:09:24+02:00"
  },
  "top_caveat": "Keep the change limited to the four profile fields and verify validation without changing classification, ownership, or platform-tier policy."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/platform#issue#753` | | Priority | p3 | | Action | codex_candidate | | Scores | reach 3 / impact 3 / confidence 5 | | Piotr fit | medium | | Effort | tiny | | Labels | `judge/p3`, `judge/codex-candidate` | | Judge | `iskra` via `openclaw` | **Rationale:** This is P3 Codex-ready manifest hygiene because four Kan user-app modules have inconsistent change-policy profiles and the fix is tightly scoped. **Caveat:** Keep the change limited to the four profile fields and verify validation without changing classification, ownership, or platform-tier policy. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 5, "effort_hint": "tiny", "escalation": { "kind": "none", "reason": "" }, "evidence_refs": [ { "note": "Issue identifies four kan-* user-app module manifests using platform/default-v1 change policy instead of user-app/default-v1.", "type": "forgejo", "value": "issue-title-body-labels-and-target-snapshot" }, { "note": "Body cites kanboard and other user-app modules as the correct reference pattern.", "type": "forgejo", "value": "issue-body-reference-pattern" }, { "note": "Scope is limited to changing change_policy.profile in exactly four manifests without touching other fields.", "type": "forgejo", "value": "issue-body-scope" } ], "impact": 3, "judge_actor": { "name": "iskra", "runtime": "openclaw" }, "judged_at": "2026-06-15T01:03:00Z", "labels_to_apply": [ "judge/p3", "judge/codex-candidate" ], "piotr_fit": "medium", "priority": "p3", "rationale_summary": "This is P3 Codex-ready manifest hygiene because four Kan user-app modules have inconsistent change-policy profiles and the fix is tightly scoped.", "reach": 3, "recommended_next_action": "codex_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 753, "repo": "pdurlej/platform" }, "target_snapshot": { "body_hash": "sha256:c07b0032e70cf4d5dd9486286773ccef234a2d4a620f953a545c4692e6782385", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [], "labels_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "state": "open", "title_hash": "sha256:3446ad0625f3c10c558d31e26fb1303713383cbcbabbb34c7578297bb296dc20", "updated_at": "2026-06-08T23:09:24+02:00" }, "top_caveat": "Keep the change limited to the four profile fields and verify validation without changing classification, ownership, or platform-tier policy." } <!-- /openclaw.judge.v0 --> ``` </details>
Owner

Closing as already implemented on current main. Evidence: kan-mcp, kan-migrate, kan-postgres, and kan-web all now use change_policy.profile: user-app/default-v1, matching the user-app pattern. 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: `kan-mcp`, `kan-migrate`, `kan-postgres`, and `kan-web` all now use `change_policy.profile: user-app/default-v1`, matching the user-app pattern. 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
3 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#753
No description provided.