@jmtrin/opencode-kevin 1.2.0 → 1.3.0

package/dist/plugin/Retrospective.js

@@ -1,216 +0,0 @@
-import { mkdirSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import { uuidv7 } from "./uuid.js";
-// v0.6.0 (K6-004 / plan §8.16) — exported so tests can assert that every
-// METRIC_KEYS entry has a label by iterating the keys (not by counting).
-export const METRIC_KEY_LABELS = {
-    tokens_injected_pre_prompt: "Tokens inyectados (pre-prompt)",
-    tokens_injected_compacting: "Tokens inyectados (compacting)",
-    reflections_throttled: "Reflexiones throttled",
-    duplicate_suppressions: "Supresiones por dedup",
-    tool_calls_deduped: "Tool calls deduped",
-    patterns_mined: "Patrones minados",
-    // BUG-014 — v0.4.0 keys (K4-006/K4-008/K4-025) had no Spanish labels,
-    // so the "## Métricas" section printed raw keys for them.
-    injections_total: "Inyecciones totales",
-    injections_effective: "Inyecciones efectivas",
-    injections_ineffective: "Inyecciones ineficaces",
-    patterns_promoted_new: "Patrones promovidos (nuevos)",
-    patterns_causal: "Patrones causales",
-    causal_links: "Enlaces causales",
-    memories_superseded: "Memorias superadas",
-    // v0.5.0 (K5-004 / plan §8.3) — the K5 metric keys need their own
-    // Spanish labels; the e2e BUG-014 regression test forbids raw-key
-    // fallback for any key in METRIC_KEYS.
-    injections_inconclusive: "Inyecciones sin conclusión",
-    injections_blocked_seen: "Inyecciones bloqueadas (ya vistas)",
-    injections_blocked_weak: "Inyecciones bloqueadas (débiles)",
-    injections_blocked_recurrence: "Inyecciones bloqueadas (recurrentes)",
-    injections_blocked_stale: "Inyecciones bloqueadas (obsoletas)",
-    injections_blocked_ignored: "Inyecciones bloqueadas (ignoradas)",
-    feedback_positive_total: "Feedback positivo total",
-    feedback_negative_total: "Feedback negativo total",
-    memories_archived: "Memorias archivadas",
-    // v0.6.0 (K6-004 / plan §8.16) — the K6 metric keys need their own
-    // Spanish labels; the audit regression forbids raw-key fallback for any
-    // key in METRIC_KEYS.
-    proposals_created: "Propuestas de curación creadas",
-    proposals_approved: "Propuestas aprobadas",
-    proposals_rejected: "Propuestas rechazadas",
-    artifact_writes_total: "Escrituras de artefacto (escritas)",
-    artifact_writes_noop: "Escrituras de artefacto (sin cambios)",
-    injections_blocked_confidence: "Inyecciones bloqueadas (confianza baja)",
-    // v0.7.0 (K7-004 / plan §8.3) — the K7 metric keys need their own
-    // Spanish labels; the audit regression forbids raw-key fallback for any
-    // key in METRIC_KEYS.
-    repo_facts_scanned: "Hechos del repositorio escaneados",
-    memories_contradicted: "Memorias contradichas",
-    conventions_mined: "Convenciones minadas",
-    conflicts_detected: "Conflictos detectados",
-    error_lessons_suppressed: "Lecciones de error suprimidas",
-    // v0.8.0 (K8-003 / plan §8.16) — the K8 metric keys need their own
-    // Spanish labels; the audit regression forbids raw-key fallback for any
-    // key in METRIC_KEYS.
-    shared_entries_total: "Entradas compartidas (totales)",
-    shared_entries_imported: "Entradas compartidas importadas",
-    shared_entries_exported: "Entradas compartidas exportadas",
-    okf_merge_folds: "Fusiones OKF (folds)",
-    rekey_events: "Re-keys de repositorio",
-    injections_from_shared: "Inyecciones desde la capa compartida",
-    // v0.9.0 (K9-003 / plan §8.16) — the K9 metric keys need their own
-    // Spanish labels; the audit regression forbids raw-key fallback for any
-    // key in METRIC_KEYS.
-    hook_fires_total: "Disparos de hooks (totales)",
-    hook_errors_total: "Errores de hooks",
-    hooks_dead_total: "Hooks muertos",
-    injections_suppressed_dead_hook: "Inyecciones suprimidas (hook muerto)",
-    native_registrations_total: "Registros nativos (totales)",
-    native_registration_failures: "Fallos de registro nativo",
-    // v1.0.0 (K10-005 / plan §6) — the six keys seeded by migration 011 section 3.
-    perf_samples_recorded: "Muestras de rendimiento registradas",
-    perf_budget_breaches: "Brechas de presupuesto de rendimiento",
-    dispose_fires_total: "Disparos de dispose (totales)",
-    dispose_misses_total: "Misses de dispose",
-    contract_digest_changes: "Cambios de digest de contrato",
-    bench_runs_total: "Ejecuciones de benchmark (totales)",
-    // v1.1.0 (K11-001 / plan §4) — drift metrics; labels required by BUG-014 regression.
-    bench_regression_failures: "Fallos de regresion de benchmark",
-    forget_requests_total: "Solicitudes de olvido totales",
-    forget_tombstones_published: "Tombstones publicados por olvido",
-    // v1.2.0 (K12-001 / plan §4) — surface metrics; labels required by BUG-014 regression.
-    tui_snapshots_flushed: "Snapshots TUI generados",
-    tui_actions_invoked: "Acciones TUI invocadas",
-};
-function originLabel(origin) {
-    if (origin === "reflector")
-        return "reflector";
-    if (origin === "pattern")
-        return "pattern";
-    if (origin === "retrospective")
-        return "retrospective";
-    return "agent";
-}
-export class Retrospective {
-    store;
-    memoryService;
-    retrospectivesDir;
-    metrics;
-    constructor(store, memoryService, options, metrics) {
-        this.store = store;
-        this.memoryService = memoryService;
-        this.retrospectivesDir =
-            options?.dir ?? join(homedir(), ".opencode-kevin", "retrospectives");
-        this.metrics = metrics ?? null;
-    }
-    async generate(sessionId) {
-        const existing = this.store
-            .prepare("SELECT file_path FROM retrospectives WHERE session_id = ?")
-            .get(sessionId);
-        if (existing?.file_path)
-            return existing.file_path;
-        const counts = this.store
-            .prepare(`SELECT
-					SUM(CASE WHEN success = 1 THEN 1 ELSE 0 END) AS success_count,
-					SUM(CASE WHEN success = 0 THEN 1 ELSE 0 END) AS failure_count,
-					COUNT(*) AS total
-				 FROM tool_calls WHERE session_id = ?`)
-            .get(sessionId);
-        const successCount = counts?.success_count ?? 0;
-        const failureCount = counts?.failure_count ?? 0;
-        const total = counts?.total ?? 0;
-        if (failureCount === 0)
-            return null;
-        const failedTools = this.store
-            .prepare(`SELECT tool, args_summary, error_type
-				 FROM tool_calls
-				 WHERE session_id = ? AND success = 0
-				 ORDER BY ts ASC`)
-            .all(sessionId);
-        const lessons = this.store
-            .prepare(`SELECT id, content, origin, fingerprint, project_id
-				 FROM memories
-				 WHERE type = 'error' AND source_session = ?
-				 ORDER BY created_at ASC`)
-            .all(sessionId);
-        const falsePositives = this.collectFalsePositives(sessionId, lessons);
-        const metricsSnapshot = this.metrics?.snapshot() ?? null;
-        const md = this.buildMarkdown(sessionId, total, successCount, failureCount, failedTools, lessons, falsePositives, metricsSnapshot);
-        mkdirSync(this.retrospectivesDir, { recursive: true });
-        const filePath = join(this.retrospectivesDir, `${sessionId}.md`);
-        writeFileSync(filePath, md, "utf8");
-        const id = uuidv7();
-        this.store
-            .prepare(`INSERT OR IGNORE INTO retrospectives
-				 (id, session_id, ts, failure_count, success_count, lessons_count, file_path, metadata)
-				 VALUES (?, ?, datetime('now'), ?, ?, ?, ?, ?)`)
-            .run(id, sessionId, failureCount, successCount, lessons.length, filePath, null);
-        return filePath;
-    }
-    collectFalsePositives(_sessionId, lessons) {
-        const reflectorLessons = lessons.filter((l) => l.origin === "reflector" && l.fingerprint !== null);
-        if (reflectorLessons.length === 0)
-            return [];
-        const result = [];
-        for (const lesson of reflectorLessons) {
-            const fp = lesson.fingerprint;
-            const row = this.store
-                .prepare(`SELECT COUNT(*) AS c FROM tool_calls
-				 WHERE COALESCE(error_fingerprint, fingerprint) = ?
-				   AND success = 0
-				   AND (project_id IS ? OR (project_id IS NULL AND ? IS NULL))`)
-                .get(fp, lesson.project_id, lesson.project_id);
-            const recurrenceCount = row?.c ?? 0;
-            if (recurrenceCount > 0) {
-                result.push({
-                    id: lesson.id,
-                    content: lesson.content,
-                    fingerprint: fp,
-                    project_id: lesson.project_id,
-                    recurrence_count: recurrenceCount,
-                });
-            }
-        }
-        return result;
-    }
-    buildMarkdown(sessionId, total, successCount, failureCount, failedTools, lessons, falsePositives, metricsSnapshot) {
-        const lines = [];
-        lines.push(`# Retrospective — Session ${sessionId}`);
-        lines.push("");
-        lines.push("## Resumen");
-        lines.push(`- Tool calls: ${total} (${successCount} ok, ${failureCount} failed)`);
-        lines.push("");
-        lines.push("## Tools que fallaron");
-        for (const ft of failedTools) {
-            const et = ft.error_type ?? "unknown";
-            const summary = ft.args_summary ?? "";
-            lines.push(`- ${ft.tool} (${et}): ${summary}`);
-        }
-        lines.push("");
-        lines.push("## Lecciones generadas");
-        for (const lesson of lessons) {
-            const label = originLabel(lesson.origin);
-            lines.push(`- [${label}] ${lesson.content}`);
-        }
-        lines.push("");
-        lines.push("## False-positive recap");
-        if (falsePositives.length === 0) {
-            lines.push("- Ninguna lección reflector-sourceada recurrrió en tool_calls.");
-        }
-        else {
-            for (const fp of falsePositives) {
-                lines.push(`- [reflector] ${fp.content} (fingerprint ${fp.fingerprint}, recurrencias: ${fp.recurrence_count})`);
-            }
-        }
-        lines.push("");
-        if (metricsSnapshot !== null) {
-            lines.push("## Métricas");
-            for (const [key, value] of Object.entries(metricsSnapshot)) {
-                const label = METRIC_KEY_LABELS[key] ?? key;
-                lines.push(`- ${label}: ${value}`);
-            }
-            lines.push("");
-        }
-        return lines.join("\n");
-    }
-}

package/dist/plugin/SharedLayer.d.ts

@@ -1,159 +0,0 @@
-import type { ArtifactWriter, WriteOutcome, WritePlan, WriteRequest } from "./ArtifactWriter.js";
-import { type ParseResult } from "./okf.js";
-export interface ImportReport {
-    path: string;
-    /** null when the file does not exist. */
-    fileHash: string | null;
-    parsed: number;
-    folded: number;
-    rejected: number;
-    /** rows inserted or updated in shared_entries. */
-    imported: number;
-    /** shared memories retired by an incoming tombstone. */
-    tombstoned: number;
-    /** true when fileHash matches the last okf_imports row. */
-    skipped: boolean;
-}
-/**
- * A planned (or applied) export (K8-020 / plan §5.5). The refusal is
- * carried by the write plan itself — `outcome: "refused"` with both
- * hashes audited — so a refused plan is indistinguishable from a
- * written one until `outcome` is read.
- */
-export interface ExportPlan {
-    path: string;
-    /** The whole-mode write request; applying re-plans through the single write funnel. */
-    request: WriteRequest;
-    /** The pure preview from planning time — diff and outcome, nothing written. */
-    write: WritePlan;
-    /** Entry ids in the merged corpus absent from the current file. */
-    entriesAdded: number;
-    /** Set by applyExport: the outcome of the actual write. */
-    applied?: WriteOutcome;
-}
-export declare class SharedLayer {
-    private readonly store;
-    private readonly repoId;
-    private readonly projectId;
-    private readonly version;
-    private readonly writer;
-    constructor(deps: {
-        store: StoreLike;
-        repoId: string;
-        projectId: string;
-        version: string;
-        writer: ArtifactWriter;
-    });
-    /**
-     * The single whole-file (mode `whole`) construction site in
-     * SharedLayer (K8-019 / K8-020, D8-08, asserted by
-     * tests/unit/single_write_path.test.ts): the OKF file is written
-     * through the ArtifactWriter's write funnel like every other file
-     * Kevin owns — there is no second write path.
-     */
-    private wholeRequest;
-    /**
-     * Pure export planner (K8-020 / plan §5.5): reads the current file
-     * and merges the given local memories' projections into the corpus,
-     * producing a {@link WritePlan} — but writes nothing. Refusals are
-     * audited like any other write: `after === before`, both hashes
-     * recorded (D6-04). The refusal ladder, in the order the code
-     * evaluates it — file-side refusals first (not_okf, version_ahead,
-     * repo_mismatch), then candidate-side refusals in the loop order
-     * (line_too_long, below_floor, not_curated, unknown_entry), then
-     * too_many_entries and finally parse_damaged.
-     */
-    planExport(memoryIds: string[], path: string): ExportPlan;
-    /**
-     * Tombstone-append planner (K8-020 / D8-09): emits one `tombstone`
-     * line per entry id, never deletes a line. The identifying triple is
-     * reconstructed from the local projection (a memory with
-     * `shared_entry_id`) so the line still passes parse's
-     * tamper-evident entry_id check; an id with no local projection is
-     * refused rather than emitting an invalid line.
-     */
-    planTombstone(entryIds: string[], path: string): ExportPlan;
-    /**
-     * v0.8.0 (BUG-003) — heal a stale `#repo` header after a rekey.
-     * Rekey changes the scope the file is written under; a stale first
-     * line would make every later planExport/planTombstone refuse with
-     * repo_mismatch forever. Only the header line is replaced; the entry
-     * bytes, line endings and trailing newline are preserved. The write
-     * goes through the single funnel (D8-08). Returns true when the
-     * header was rewritten. Never throws: a missing or unreadable file
-     * is nothing to heal.
-     */
-    healHeader(path: string, newRepoId: string): boolean;
-    /**
-     * Apply a previously planned export: the write goes through the
-     * single write funnel (re-planning against the file's current state),
-     * and the outcome is recorded on the returned plan (K8-020).
-     */
-    applyExport(plan: ExportPlan): ExportPlan;
-    /** Refusal plans are audited with both hashes (after === before). */
-    private refused;
-    /**
-     * Convert the requested local memories into OKF candidates and run
-     * the candidate-side refusals — line_too_long, below_floor,
-     * not_curated — in plan §5.5 order.
-     */
-    private candidateEntries;
-    private setting;
-    private readWithFlag;
-    /**
-     * Read and parse the shared file. Total: a missing or unreadable
-     * file yields an empty ParseResult, never a throw (D8-14).
-     */
-    read(path: string): ParseResult;
-    /**
-     * Idempotent and cheap to call (K8-016 / plan §5.5): hash the file
-     * first and return `{ skipped: true }` when the hash matches the
-     * most recent `okf_imports` row for this repo_id — the session.idle
-     * path costs one readFileSync plus one hash on an unchanged
-     * repository. A missing file is not an error: it returns
-     * `fileHash: null`, `parsed: 0`, and still writes an audit row.
-     */
-    import(path: string): ImportReport;
-    private writeImportRow;
-    /**
-     * Project every imported entry into memories (K8-017 / plan §5.5,
-     * D8-10): a shared memory IS a memory, so getRelevant(), rankScore(),
-     * the five gates, truth_penalty, ConflictDetector and every audit
-     * rollup keep working unchanged.
-     *
-     * - assert entries: inserted once, never updated — re-importing an
-     *   unchanged file must not create duplicates, bump evidence_count or
-     *   move updated_at. `fingerprint` is deliberately left NULL: it is a
-     *   different identity dimension (plan §3.3) and conflating it with
-     *   `shared_entry_id` would make the v0.4.0 supersede path fire on
-     *   unrelated entries (K8-017 acceptance).
-     * - tombstone entries: the ONE place in this codebase where the shared
-     *   layer may write memories.status. This is NOT a v0.7.0 Principle 24
-     *   violation: that principle forbids *contradiction* — a fuzzy
-     *   inference — from writing status, and a tombstone is the opposite:
-     *   an explicit, committed, human-reviewed decision that arrived
-     *   through a pull request (D8-09, plan §5.5).
-     *
-     * Returns the number of memories archived by tombstones.
-     */
-    private projectEntries;
-    /**
-     * Upsert every entry into shared_entries keyed on (repo_id,
-     * entry_id) — the UNIQUE index from migration 009 makes this an
-     * upsert rather than an append. `confidence` is the DERIVED value
-     * (deriveConfidence), never a transported float (plan §5.3).
-     * Returns the number of rows inserted or updated.
-     */
-    private upsertEntries;
-}
-/**
- * The Store surface SharedLayer needs. Kept structural so tests can
- * use the real Store without importing plugin internals.
- */
-export interface StoreLike {
-    prepare(sql: string): {
-        get(...params: unknown[]): unknown;
-        all(...params: unknown[]): unknown[];
-        run(...params: unknown[]): void;
-    };
-}