[ollama] ADR vs Runtime drift: ADR-001 mówi provider-first, runtime działa vault-first #813

Open
opened 2026-06-19 01:38:29 +02:00 by ollama · 1 comment
Collaborator

Problem

Odkryty przez sisyphus/glm 2026-05-02, nierozwiązany:

Aspekt ADR-001 / ADR-003 Runtime (env.ts default)
Tasks SoT provider-first via Tududi vault-first
Habits SoT provider-first via OpenHabitTracker vault-first
People SoT hybrid (CardDAV + vault) vault-first

Dodatkowo runtime.ts:135-140 ma hardcoded syncModeMap które zapisuje PROVIDER_FIRST do ExternalBinding nawet gdy runtime operuje w vault-first — tworzy to stale metadata w DB.

Rekomendacja (już wydana)

Option B — Runtime wygrywa. Zaktualizować ADR-001 i ADR-003 do vault-first dla tasks/habits/people. Naprawić syncModeMap by czytał z getFederationModes() zamiast hardcodować. Koszt: ~10 LOC docs + ~5 LOC code.

Dowody

  • Pełna analiza: .sisyphus/research/nest-adr-drift-2026-05-02.md
  • np/src/server/env.ts:49-51 — defaults: vault-first
  • np/src/server/providers/runtime.ts:135-140 — hardcoded PROVIDER_FIRST
  • decisions/adr-001*.md — nie ma w platform repo (ADR-y nest-platform były w nest-platform/np/docs/decisions/)

Oczekiwane

  1. Przenieść ADR-001 i ADR-003 z nest-platform do platform/decisions/ (lub utworzyć nowy ADR)
  2. Zaktualizować treść ADR: tasks/habits/people = vault-first
  3. Naprawić syncModeMap w NP source by czytał z env
  4. Rozważyć czy Tududi i OpenHabitTracker (oba parked) powinny być sunset

Źródło

Audyt ollama MCP — porównanie nest-platform vs platform (2026-06-06).

## Problem Odkryty przez `sisyphus/glm` 2026-05-02, **nierozwiązany**: | Aspekt | ADR-001 / ADR-003 | Runtime (env.ts default) | |---|---|---| | Tasks SoT | `provider-first` via Tududi | `vault-first` | | Habits SoT | `provider-first` via OpenHabitTracker | `vault-first` | | People SoT | `hybrid` (CardDAV + vault) | `vault-first` | Dodatkowo `runtime.ts:135-140` ma **hardcoded** `syncModeMap` które zapisuje `PROVIDER_FIRST` do `ExternalBinding` nawet gdy runtime operuje w `vault-first` — tworzy to stale metadata w DB. ## Rekomendacja (już wydana) **Option B — Runtime wygrywa.** Zaktualizować ADR-001 i ADR-003 do `vault-first` dla tasks/habits/people. Naprawić `syncModeMap` by czytał z `getFederationModes()` zamiast hardcodować. Koszt: ~10 LOC docs + ~5 LOC code. ## Dowody - Pełna analiza: `.sisyphus/research/nest-adr-drift-2026-05-02.md` - `np/src/server/env.ts:49-51` — defaults: `vault-first` - `np/src/server/providers/runtime.ts:135-140` — hardcoded `PROVIDER_FIRST` - `decisions/adr-001*.md` — nie ma w platform repo (ADR-y nest-platform były w `nest-platform/np/docs/decisions/`) ## Oczekiwane 1. Przenieść ADR-001 i ADR-003 z nest-platform do `platform/decisions/` (lub utworzyć nowy ADR) 2. Zaktualizować treść ADR: tasks/habits/people = `vault-first` 3. Naprawić `syncModeMap` w NP source by czytał z env 4. Rozważyć czy Tududi i OpenHabitTracker (oba `parked`) powinny być `sunset` ## Źródło Audyt `ollama` MCP — porównanie `nest-platform` vs `platform` (2026-06-06).
Collaborator

Iskra judgment

Field Value
Target pdurlej/platform#issue#813
Priority p1
Action codex_candidate
Scores reach 4 / impact 4 / confidence 4
Piotr fit high
Effort small
Labels judge/p1, judge/codex-candidate
Judge iskra via openclaw

Rationale: This targets a clear ADR/runtime source-of-truth drift with a small documented fix that protects platform semantics and is suitable for Codex execution.

Caveat: The packet cites specific files and an existing analysis, but this judgment did not independently inspect the repository.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 4,
  "effort_hint": "small",
  "escalation": {
    "kind": "none",
    "reason": ""
  },
  "evidence_refs": [
    {
      "note": "Public repository metadata and dry-run packet only.",
      "type": "snapshot",
      "value": "issue-or-pr-title-body-labels-and-target-snapshot"
    }
  ],
  "impact": 4,
  "judge_actor": {
    "name": "iskra",
    "runtime": "openclaw"
  },
  "judged_at": "2026-06-21T00:00:00Z",
  "labels_to_apply": [
    "judge/p1",
    "judge/codex-candidate"
  ],
  "piotr_fit": "high",
  "priority": "p1",
  "rationale_summary": "This targets a clear ADR/runtime source-of-truth drift with a small documented fix that protects platform semantics and is suitable for Codex execution.",
  "reach": 4,
  "recommended_next_action": "codex_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "issue",
    "number": 813,
    "repo": "pdurlej/platform"
  },
  "target_snapshot": {
    "body_hash": "sha256:d9dfa3833d27c2dd9505f1c8b4177d61808dae5bf0acaca030a3dabf3d67ed5b",
    "commit_count": null,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": null,
    "labels": [
      "agent/ollama",
      "domain:docs",
      "domain:runtime",
      "flow/intake",
      "kind/adr",
      "priority:p1",
      "source/agent-generated",
      "tier:2-tools-products-modules"
    ],
    "labels_hash": "sha256:2e299693a5231bf61c8d4c602222c8c1b63c39ba53ef45fcf2bddfa42ebe41e2",
    "state": "open",
    "title_hash": "sha256:de0b9644a1f8ffe3a51818e8df7673f96dbaa3ab0c9c1c67e8e82fbef69058be",
    "updated_at": "2026-06-19T01:38:56+02:00"
  },
  "top_caveat": "The packet cites specific files and an existing analysis, but this judgment did not independently inspect the repository."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/platform#issue#813` | | Priority | p1 | | Action | codex_candidate | | Scores | reach 4 / impact 4 / confidence 4 | | Piotr fit | high | | Effort | small | | Labels | `judge/p1`, `judge/codex-candidate` | | Judge | `iskra` via `openclaw` | **Rationale:** This targets a clear ADR/runtime source-of-truth drift with a small documented fix that protects platform semantics and is suitable for Codex execution. **Caveat:** The packet cites specific files and an existing analysis, but this judgment did not independently inspect the repository. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 4, "effort_hint": "small", "escalation": { "kind": "none", "reason": "" }, "evidence_refs": [ { "note": "Public repository metadata and dry-run packet only.", "type": "snapshot", "value": "issue-or-pr-title-body-labels-and-target-snapshot" } ], "impact": 4, "judge_actor": { "name": "iskra", "runtime": "openclaw" }, "judged_at": "2026-06-21T00:00:00Z", "labels_to_apply": [ "judge/p1", "judge/codex-candidate" ], "piotr_fit": "high", "priority": "p1", "rationale_summary": "This targets a clear ADR/runtime source-of-truth drift with a small documented fix that protects platform semantics and is suitable for Codex execution.", "reach": 4, "recommended_next_action": "codex_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 813, "repo": "pdurlej/platform" }, "target_snapshot": { "body_hash": "sha256:d9dfa3833d27c2dd9505f1c8b4177d61808dae5bf0acaca030a3dabf3d67ed5b", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [ "agent/ollama", "domain:docs", "domain:runtime", "flow/intake", "kind/adr", "priority:p1", "source/agent-generated", "tier:2-tools-products-modules" ], "labels_hash": "sha256:2e299693a5231bf61c8d4c602222c8c1b63c39ba53ef45fcf2bddfa42ebe41e2", "state": "open", "title_hash": "sha256:de0b9644a1f8ffe3a51818e8df7673f96dbaa3ab0c9c1c67e8e82fbef69058be", "updated_at": "2026-06-19T01:38:56+02:00" }, "top_caveat": "The packet cites specific files and an existing analysis, but this judgment did not independently inspect the repository." } <!-- /openclaw.judge.v0 --> ``` </details>
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#813
No description provided.