[ollama] Git sidecar (automatyczne snapshoty vaulta) — brak w platform #814

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

Problem

W nest-platform istniał src/server/git-sidecar/service.ts — automatyczne snapshoty Git vaulta po każdej operacji CRUD (entity.created, entity.updated, destructive.confirm). W platform nie ma śladu tej funkcjonalności:

  • Moduł np nie wspomina o git sidecar
  • Nie ma osobnego modułu np-git-sidecar
  • Nie ma konfiguracji GIT_SIDECAR_ENABLED w Infisical
  • module.yaml np nie deklaruje zależności od git

Kontekst

Git sidecar był ważnym elementem NP: zapewniał historię zmian vaulta w Git, co dawało możliwość rollbacku i audytu poza Postgresem. W nest-platform był opcjonalny (GIT_SIDECAR_ENABLED=false default).

Oczekiwane

  1. Ustalić czy git sidecar nadal działa na produkcji (sprawdzić logi NP na RS2000)
  2. Jeśli nie działa: zdecydować czy przywrócić jako osobny moduł np-git-sidecar
  3. Jeśli działa: dodać do module.yaml np jako zależność lub capability
  4. Alternatywnie: uznać za świadomie usunięte i udokumentować decyzję

Źródło

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

## Problem W `nest-platform` istniał `src/server/git-sidecar/service.ts` — automatyczne snapshoty Git vaulta po każdej operacji CRUD (entity.created, entity.updated, destructive.confirm). W `platform` **nie ma śladu** tej funkcjonalności: - Moduł `np` nie wspomina o git sidecar - Nie ma osobnego modułu `np-git-sidecar` - Nie ma konfiguracji `GIT_SIDECAR_ENABLED` w Infisical - `module.yaml` np nie deklaruje zależności od git ## Kontekst Git sidecar był ważnym elementem NP: zapewniał historię zmian vaulta w Git, co dawało możliwość rollbacku i audytu poza Postgresem. W nest-platform był opcjonalny (`GIT_SIDECAR_ENABLED=false` default). ## Oczekiwane 1. Ustalić czy git sidecar nadal działa na produkcji (sprawdzić logi NP na RS2000) 2. Jeśli nie działa: zdecydować czy przywrócić jako osobny moduł `np-git-sidecar` 3. Jeśli działa: dodać do `module.yaml` np jako zależność lub capability 4. Alternatywnie: uznać za świadomie usunięte i udokumentować decyzję ## Źródło Audyt `ollama` MCP — porównanie `nest-platform` vs `platform` (2026-06-06).
Collaborator

Iskra judgment

Field Value
Target pdurlej/platform#issue#814
Priority p2
Action operator_needed
Scores reach 3 / impact 4 / confidence 3
Piotr fit high
Effort medium
Labels judge/p2, judge/operator-needed
Judge iskra via openclaw

Rationale: This is a meaningful runtime governance question because loss of vault git snapshots could weaken rollback and auditability, but it needs operator-confirmed production evidence before implementation.

Caveat: The packet identifies a plausible missing capability, but no fresh production log or repository inspection was performed here.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 3,
  "effort_hint": "medium",
  "escalation": {
    "kind": "operator",
    "reason": "The next step requires checking live NP runtime behavior and deciding whether git-sidecar removal was intentional."
  },
  "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/p2",
    "judge/operator-needed"
  ],
  "piotr_fit": "high",
  "priority": "p2",
  "rationale_summary": "This is a meaningful runtime governance question because loss of vault git snapshots could weaken rollback and auditability, but it needs operator-confirmed production evidence before implementation.",
  "reach": 3,
  "recommended_next_action": "operator_needed",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "issue",
    "number": 814,
    "repo": "pdurlej/platform"
  },
  "target_snapshot": {
    "body_hash": "sha256:fa48aaa211ab929185faba7e89c8f30edcbe4c2508530d9133033ac0cfe63707",
    "commit_count": null,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": null,
    "labels": [
      "agent/ollama",
      "domain:infra",
      "domain:runtime",
      "flow/intake",
      "kind/research",
      "priority:p2",
      "source/agent-generated",
      "tier:2-tools-products-modules"
    ],
    "labels_hash": "sha256:4640d68db1492c08003d4a5a65f63999263425f66c0576d7b918f6d614feae1f",
    "state": "open",
    "title_hash": "sha256:83dd5aaf6b4ecd9ae28198af3ec55c43b8b333592ed74307ceba806616c2379b",
    "updated_at": "2026-06-19T01:38:42+02:00"
  },
  "top_caveat": "The packet identifies a plausible missing capability, but no fresh production log or repository inspection was performed here."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/platform#issue#814` | | Priority | p2 | | Action | operator_needed | | Scores | reach 3 / impact 4 / confidence 3 | | Piotr fit | high | | Effort | medium | | Labels | `judge/p2`, `judge/operator-needed` | | Judge | `iskra` via `openclaw` | **Rationale:** This is a meaningful runtime governance question because loss of vault git snapshots could weaken rollback and auditability, but it needs operator-confirmed production evidence before implementation. **Caveat:** The packet identifies a plausible missing capability, but no fresh production log or repository inspection was performed here. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 3, "effort_hint": "medium", "escalation": { "kind": "operator", "reason": "The next step requires checking live NP runtime behavior and deciding whether git-sidecar removal was intentional." }, "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/p2", "judge/operator-needed" ], "piotr_fit": "high", "priority": "p2", "rationale_summary": "This is a meaningful runtime governance question because loss of vault git snapshots could weaken rollback and auditability, but it needs operator-confirmed production evidence before implementation.", "reach": 3, "recommended_next_action": "operator_needed", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 814, "repo": "pdurlej/platform" }, "target_snapshot": { "body_hash": "sha256:fa48aaa211ab929185faba7e89c8f30edcbe4c2508530d9133033ac0cfe63707", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [ "agent/ollama", "domain:infra", "domain:runtime", "flow/intake", "kind/research", "priority:p2", "source/agent-generated", "tier:2-tools-products-modules" ], "labels_hash": "sha256:4640d68db1492c08003d4a5a65f63999263425f66c0576d7b918f6d614feae1f", "state": "open", "title_hash": "sha256:83dd5aaf6b4ecd9ae28198af3ec55c43b8b333592ed74307ceba806616c2379b", "updated_at": "2026-06-19T01:38:42+02:00" }, "top_caveat": "The packet identifies a plausible missing capability, but no fresh production log or repository inspection was performed here." } <!-- /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#814
No description provided.