[ollama] Destructive operations confirm flow — brak w platform #816

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

Problem

W nest-platform istniał rozbudowany system destructive change confirm flow:

  • Tokeny DestructiveChangeToken w Prisma (hash, expiresAt, confirmedAt)
  • Operacje: DELETE, ARCHIVE, MERGE, BULK_REWRITE
  • 2-etapowy proces: prepare (generuje token) → confirm (wykonuje operację)
  • Pełny audyt (destructive.prepare, destructive.confirm)
  • Soft-delete przez moveVaultRecordToTrash
  • MERGE łączyło body/tagi/linki wielu encji w jedną

W platform nie ma śladu tego mechanizmu:

  • Moduł np nie wspomina o destructive flow
  • Nie ma osobnego modułu
  • Brak w module.yaml

Kontekst

To była kluczowa funkcja bezpieczeństwa NP — operacje destrukcyjne wymagały 2-etapowego potwierdzenia z 20-minutowym timeoutem. Chroniło to przed przypadkowym usunięciem danych przez agentów AI.

Oczekiwane

  1. Ustalić czy destructive flow nadal działa w NP na produkcji
  2. Jeśli tak: dodać do dokumentacji modułu np
  3. Jeśli nie: rozważyć przywrócenie jako część NP lub osobny moduł
  4. Alternatywnie: udokumentować świadomą decyzję o usunięciu

Źródło

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

## Problem W `nest-platform` istniał rozbudowany system **destructive change confirm flow**: - Tokeny `DestructiveChangeToken` w Prisma (hash, expiresAt, confirmedAt) - Operacje: `DELETE`, `ARCHIVE`, `MERGE`, `BULK_REWRITE` - 2-etapowy proces: `prepare` (generuje token) → `confirm` (wykonuje operację) - Pełny audyt (`destructive.prepare`, `destructive.confirm`) - Soft-delete przez `moveVaultRecordToTrash` - MERGE łączyło body/tagi/linki wielu encji w jedną W `platform` **nie ma śladu** tego mechanizmu: - Moduł `np` nie wspomina o destructive flow - Nie ma osobnego modułu - Brak w `module.yaml` ## Kontekst To była kluczowa funkcja bezpieczeństwa NP — operacje destrukcyjne wymagały 2-etapowego potwierdzenia z 20-minutowym timeoutem. Chroniło to przed przypadkowym usunięciem danych przez agentów AI. ## Oczekiwane 1. Ustalić czy destructive flow nadal działa w NP na produkcji 2. Jeśli tak: dodać do dokumentacji modułu `np` 3. Jeśli nie: rozważyć przywrócenie jako część NP lub osobny moduł 4. Alternatywnie: udokumentować świadomą decyzję o usunięciu ## Źródło Audyt `ollama` MCP — porównanie `nest-platform` vs `platform` (2026-06-06).
Collaborator

Iskra judgment

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

Rationale: This safety-policy gap can affect destructive operations by agents, so it should be checked and either documented or restored before runtime confidence is assumed.

Caveat: The packet asserts absence in platform but does not include a live code search or production behavior check.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 3,
  "effort_hint": "medium",
  "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-22T00:00:00Z",
  "labels_to_apply": [
    "judge/p1",
    "judge/codex-candidate"
  ],
  "piotr_fit": "high",
  "priority": "p1",
  "rationale_summary": "This safety-policy gap can affect destructive operations by agents, so it should be checked and either documented or restored before runtime confidence is assumed.",
  "reach": 3,
  "recommended_next_action": "codex_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "issue",
    "number": 816,
    "repo": "pdurlej/platform"
  },
  "target_snapshot": {
    "body_hash": "sha256:c19daaa6a7a9a47b1da8540ec65579be8bb872163f31f14856087dafc2c2f997",
    "commit_count": null,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": null,
    "labels": [
      "agent/ollama",
      "domain:runtime",
      "flow/intake",
      "kind/research",
      "priority:p2",
      "source/agent-generated",
      "tier:2-tools-products-modules",
      "type:policy"
    ],
    "labels_hash": "sha256:3f40f80745df74c065bb66234700e592e1d412cb6af258f168895b7f368ea9e6",
    "state": "open",
    "title_hash": "sha256:bb1981a15cc603e692e88f77eee7f655df31188d9ad99cf0d5dad35989f3e653",
    "updated_at": "2026-06-19T01:38:56+02:00"
  },
  "top_caveat": "The packet asserts absence in platform but does not include a live code search or production behavior check."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/platform#issue#816` | | Priority | p1 | | Action | codex_candidate | | Scores | reach 3 / impact 4 / confidence 3 | | Piotr fit | high | | Effort | medium | | Labels | `judge/p1`, `judge/codex-candidate` | | Judge | `iskra` via `openclaw` | **Rationale:** This safety-policy gap can affect destructive operations by agents, so it should be checked and either documented or restored before runtime confidence is assumed. **Caveat:** The packet asserts absence in platform but does not include a live code search or production behavior check. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 3, "effort_hint": "medium", "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-22T00:00:00Z", "labels_to_apply": [ "judge/p1", "judge/codex-candidate" ], "piotr_fit": "high", "priority": "p1", "rationale_summary": "This safety-policy gap can affect destructive operations by agents, so it should be checked and either documented or restored before runtime confidence is assumed.", "reach": 3, "recommended_next_action": "codex_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 816, "repo": "pdurlej/platform" }, "target_snapshot": { "body_hash": "sha256:c19daaa6a7a9a47b1da8540ec65579be8bb872163f31f14856087dafc2c2f997", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [ "agent/ollama", "domain:runtime", "flow/intake", "kind/research", "priority:p2", "source/agent-generated", "tier:2-tools-products-modules", "type:policy" ], "labels_hash": "sha256:3f40f80745df74c065bb66234700e592e1d412cb6af258f168895b7f368ea9e6", "state": "open", "title_hash": "sha256:bb1981a15cc603e692e88f77eee7f655df31188d9ad99cf0d5dad35989f3e653", "updated_at": "2026-06-19T01:38:56+02:00" }, "top_caveat": "The packet asserts absence in platform but does not include a live code search or production behavior check." } <!-- /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#816
No description provided.