@skillsmith/core 0.9.0 → 0.10.0

package/dist/src/journal/writer.js

@@ -0,0 +1,125 @@
+/**
+ * @fileoverview Change-journal writer — single-writer, fsync'd, hash-chained
+ *               append (SMI-5456 Wave 1 Step 3 / SMI-5470).
+ * @module @skillsmith/core/journal/writer
+ *
+ * P-5 single-writer invariant (docs/internal/implementation/smi-5456-skillsmith-agent-wave1.md
+ * "Shared-State / Coordination Audit"): the journal file is written by the
+ * apply-family MCP tools ONLY, and every writer in this process funnels
+ * through `appendJournalRecord`, which serializes behind ONE module-scoped
+ * promise queue. That queue — not any per-caller locking — is what makes
+ * concurrent `apply_namespace_rename` / `apply_recommended_edit` / `undo_apply`
+ * calls in the same server process produce well-formed, correctly-chained
+ * lines instead of interleaved/torn writes.
+ *
+ * Durability: each append opens the file, writes the line, calls
+ * `FileHandle.sync()` (fsync), then closes — so a record that returns
+ * successfully is durable on disk before the caller's apply/undo response
+ * goes out, matching the "fsync'd appends" requirement in the plan's P-5
+ * table.
+ */
+import { mkdir, open, readFile } from 'node:fs/promises';
+import { dirname } from 'node:path';
+import { JOURNAL_GENESIS_HASH, sha256Hex } from './hash.js';
+import { getJournalFilePath } from './path.js';
+/**
+ * Canonical (fixed-key-order) JSON of every hash-chain-relevant field,
+ * INCLUDING `prev_hash` but EXCLUDING `record_hash` itself. Plain
+ * `JSON.stringify` is sufficient canonicalization here because the object
+ * literal below is always constructed with the same key order — a full
+ * canonical-JSON library would be overkill for a single fixed shape.
+ */
+function canonicalFields(record) {
+    const ordered = {
+        schema: record.schema,
+        ts: record.ts,
+        session_id: record.session_id,
+        tool: record.tool,
+        action: record.action,
+        suggestion_id: record.suggestion_id,
+        target_path: record.target_path,
+        before_hash: record.before_hash,
+        after_hash: record.after_hash,
+        approval: record.approval,
+        backup_ref: record.backup_ref,
+        detail: record.detail,
+        prev_hash: record.prev_hash,
+    };
+    return JSON.stringify(ordered);
+}
+/**
+ * Recompute a record's `record_hash` from its other fields. Exported so the
+ * reader can independently verify each line without duplicating the
+ * canonicalization logic (drift between writer and reader would silently
+ * break every chain-verification call).
+ */
+export function computeRecordHash(record) {
+    return sha256Hex(canonicalFields(record));
+}
+export function isCorruptTailError(err) {
+    return (err instanceof Error &&
+        err.kind === 'journal.write.corrupt_tail');
+}
+async function getLastRecordHash(filePath) {
+    let raw;
+    try {
+        raw = await readFile(filePath, 'utf-8');
+    }
+    catch {
+        return JOURNAL_GENESIS_HASH;
+    }
+    const lines = raw.split('\n').filter((line) => line.trim().length > 0);
+    if (lines.length === 0)
+        return JOURNAL_GENESIS_HASH;
+    const lastLine = lines[lines.length - 1];
+    try {
+        const parsed = JSON.parse(lastLine);
+        if (typeof parsed.record_hash === 'string' && parsed.record_hash.length > 0) {
+            return parsed.record_hash;
+        }
+    }
+    catch {
+        // fall through to the corrupt-tail error below
+    }
+    const err = new Error('journal tail is corrupt or unparseable; refusing to append past an unverifiable chain head');
+    err.kind = 'journal.write.corrupt_tail';
+    throw err;
+}
+// P-5 single-writer invariant: every append in this process funnels through
+// this one queue, regardless of which apply-family tool called it.
+let writeQueue = Promise.resolve();
+/**
+ * Append a record to the journal. Serialized behind the module-scoped
+ * write queue and fsync'd before the returned promise resolves.
+ *
+ * Callers (the apply-family tools) should treat a rejection as fail-soft —
+ * the journal is an evidence trail, not a gate on the user's mutation —
+ * and log rather than propagate. See `tools/apply-journal.helpers.ts` in
+ * `@skillsmith/mcp-server` for the call-site convention.
+ */
+export function appendJournalRecord(input) {
+    const task = writeQueue.then(() => doAppend(input));
+    // A rejected append must not wedge the queue for subsequent callers.
+    writeQueue = task.then(() => undefined, () => undefined);
+    return task;
+}
+async function doAppend(input) {
+    const filePath = getJournalFilePath();
+    await mkdir(dirname(filePath), { recursive: true });
+    // Propagates as-is (including `JournalCorruptTailError`) — callers use
+    // `isCorruptTailError` to distinguish it from a generic I/O failure.
+    const prevHash = await getLastRecordHash(filePath);
+    const withoutHash = { ...input, prev_hash: prevHash };
+    const record = { ...withoutHash, record_hash: computeRecordHash(withoutHash) };
+    const line = JSON.stringify(record) + '\n';
+    const handle = await open(filePath, 'a');
+    try {
+        await handle.appendFile(line, 'utf-8');
+        await handle.sync();
+    }
+    finally {
+        await handle.close();
+    }
+    return record;
+}
+//# sourceMappingURL=writer.js.map

package/dist/src/journal/writer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"writer.js","sourceRoot":"","sources":["../../../src/journal/writer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAA;AACxD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AAEnC,OAAO,EAAE,oBAAoB,EAAE,SAAS,EAAE,MAAM,WAAW,CAAA;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,WAAW,CAAA;AAG9C;;;;;;GAMG;AACH,SAAS,eAAe,CAAC,MAA0C;IACjE,MAAM,OAAO,GAAG;QACd,MAAM,EAAE,MAAM,CAAC,MAAM;QACrB,EAAE,EAAE,MAAM,CAAC,EAAE;QACb,UAAU,EAAE,MAAM,CAAC,UAAU;QAC7B,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,MAAM,EAAE,MAAM,CAAC,MAAM;QACrB,aAAa,EAAE,MAAM,CAAC,aAAa;QACnC,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,UAAU,EAAE,MAAM,CAAC,UAAU;QAC7B,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,UAAU,EAAE,MAAM,CAAC,UAAU;QAC7B,MAAM,EAAE,MAAM,CAAC,MAAM;QACrB,SAAS,EAAE,MAAM,CAAC,SAAS;KAC5B,CAAA;IACD,OAAO,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,CAAA;AAChC,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,iBAAiB,CAAC,MAA0C;IAC1E,OAAO,SAAS,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC,CAAA;AAC3C,CAAC;AAeD,MAAM,UAAU,kBAAkB,CAAC,GAAY;IAC7C,OAAO,CACL,GAAG,YAAY,KAAK;QACnB,GAAwC,CAAC,IAAI,KAAK,4BAA4B,CAChF,CAAA;AACH,CAAC;AAED,KAAK,UAAU,iBAAiB,CAAC,QAAgB;IAC/C,IAAI,GAAW,CAAA;IACf,IAAI,CAAC;QACH,GAAG,GAAG,MAAM,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;IACzC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,oBAAoB,CAAA;IAC7B,CAAC;IAED,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC,CAAA;IACtE,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,oBAAoB,CAAA;IAEnD,MAAM,QAAQ,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAE,CAAA;IACzC,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAA2B,CAAA;QAC7D,IAAI,OAAO,MAAM,CAAC,WAAW,KAAK,QAAQ,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC5E,OAAO,MAAM,CAAC,WAAW,CAAA;QAC3B,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,+CAA+C;IACjD,CAAC;IAED,MAAM,GAAG,GAAG,IAAI,KAAK,CACnB,4FAA4F,CAClE,CAAA;IAC5B,GAAG,CAAC,IAAI,GAAG,4BAA4B,CAAA;IACvC,MAAM,GAAG,CAAA;AACX,CAAC;AAED,4EAA4E;AAC5E,mEAAmE;AACnE,IAAI,UAAU,GAAkB,OAAO,CAAC,OAAO,EAAE,CAAA;AAEjD;;;;;;;;GAQG;AACH,MAAM,UAAU,mBAAmB,CAAC,KAAyB;IAC3D,MAAM,IAAI,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAA;IACnD,qEAAqE;IACrE,UAAU,GAAG,IAAI,CAAC,IAAI,CACpB,GAAG,EAAE,CAAC,SAAS,EACf,GAAG,EAAE,CAAC,SAAS,CAChB,CAAA;IACD,OAAO,IAAI,CAAA;AACb,CAAC;AAED,KAAK,UAAU,QAAQ,CAAC,KAAyB;IAC/C,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAA;IACrC,MAAM,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IAEnD,uEAAuE;IACvE,qEAAqE;IACrE,MAAM,QAAQ,GAAG,MAAM,iBAAiB,CAAC,QAAQ,CAAC,CAAA;IAElD,MAAM,WAAW,GAAuC,EAAE,GAAG,KAAK,EAAE,SAAS,EAAE,QAAQ,EAAE,CAAA;IACzF,MAAM,MAAM,GAAkB,EAAE,GAAG,WAAW,EAAE,WAAW,EAAE,iBAAiB,CAAC,WAAW,CAAC,EAAE,CAAA;IAE7F,MAAM,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,GAAG,IAAI,CAAA;IAC1C,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAA;IACxC,IAAI,CAAC;QACH,MAAM,MAAM,CAAC,UAAU,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QACtC,MAAM,MAAM,CAAC,IAAI,EAAE,CAAA;IACrB,CAAC;YAAS,CAAC;QACT,MAAM,MAAM,CAAC,KAAK,EAAE,CAAA;IACtB,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC"}

package/dist/src/paywall-triggers/index.d.ts

@@ -0,0 +1,13 @@
+/**
+ * @fileoverview Public barrel for the paywall trigger-state store (SMI-5456
+ *               Wave 1 Step 5, QD-3). See `store.ts`'s module header for the
+ *               Wave-1 scope limitation on in-conversation dismissal
+ *               recording, and `path.ts`'s header for why this module is
+ *               `paywall-triggers/`, not `triggers/` (a pre-existing,
+ *               unrelated module owns that name).
+ * @module @skillsmith/core/paywall-triggers
+ */
+export { TRIGGER_DISMISSAL_THRESHOLD, TRIGGER_MUTE_DAYS, emptyTriggerState, type TriggerId, type TriggerState, } from './types.js';
+export { TRIGGER_STATE_DIR_ENV_VAR, getNudgeCooldownState, getTriggerStateDirForTests, getTriggerStateFilePath, } from './path.js';
+export { canShowTrigger, isTriggerMuted, loadTriggerState, recordImpression, recordTriggerDismissal, } from './store.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/src/paywall-triggers/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/paywall-triggers/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EACL,2BAA2B,EAC3B,iBAAiB,EACjB,iBAAiB,EACjB,KAAK,SAAS,EACd,KAAK,YAAY,GAClB,MAAM,YAAY,CAAA;AAEnB,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,0BAA0B,EAC1B,uBAAuB,GACxB,MAAM,WAAW,CAAA;AAElB,OAAO,EACL,cAAc,EACd,cAAc,EACd,gBAAgB,EAChB,gBAAgB,EAChB,sBAAsB,GACvB,MAAM,YAAY,CAAA"}

package/dist/src/paywall-triggers/index.js

@@ -0,0 +1,13 @@
+/**
+ * @fileoverview Public barrel for the paywall trigger-state store (SMI-5456
+ *               Wave 1 Step 5, QD-3). See `store.ts`'s module header for the
+ *               Wave-1 scope limitation on in-conversation dismissal
+ *               recording, and `path.ts`'s header for why this module is
+ *               `paywall-triggers/`, not `triggers/` (a pre-existing,
+ *               unrelated module owns that name).
+ * @module @skillsmith/core/paywall-triggers
+ */
+export { TRIGGER_DISMISSAL_THRESHOLD, TRIGGER_MUTE_DAYS, emptyTriggerState, } from './types.js';
+export { TRIGGER_STATE_DIR_ENV_VAR, getNudgeCooldownState, getTriggerStateDirForTests, getTriggerStateFilePath, } from './path.js';
+export { canShowTrigger, isTriggerMuted, loadTriggerState, recordImpression, recordTriggerDismissal, } from './store.js';
+//# sourceMappingURL=index.js.map

package/dist/src/paywall-triggers/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/paywall-triggers/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EACL,2BAA2B,EAC3B,iBAAiB,EACjB,iBAAiB,GAGlB,MAAM,YAAY,CAAA;AAEnB,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,0BAA0B,EAC1B,uBAAuB,GACxB,MAAM,WAAW,CAAA;AAElB,OAAO,EACL,cAAc,EACd,cAAc,EACd,gBAAgB,EAChB,gBAAgB,EAChB,sBAAsB,GACvB,MAAM,YAAY,CAAA"}

package/dist/src/paywall-triggers/path.d.ts

@@ -0,0 +1,47 @@
+/**
+ * Trigger-state file layout (SMI-5456 Wave 1 Step 5, QD-3).
+ *
+ * `~/.skillsmith/paywall-triggers/<triggerId>.json` — one file per trigger,
+ * mirroring the per-resource-directory convention already used by
+ * `journal/path.ts` (`~/.skillsmith/journal/`) and
+ * `telemetry/agent-marker.ts` (`~/.skillsmith/agent-markers/`). Named
+ * `paywall-triggers` (not `triggers`) both in the module path and the
+ * on-disk directory to avoid any collision with the PRE-EXISTING, unrelated
+ * `packages/core/src/triggers/` module (`TriggerDetector`/`ContextScorer` —
+ * proactive skill-suggestion detection, nothing to do with PRD §8.1's T1-T4
+ * paywall triggers).
+ *
+ * Format compatibility with the SessionStart hook's nudge cooldown
+ * (the per-harness `hooks/<harness>/session-start.sh` script, generated by
+ * the — locked for this step — `agent-pack/hooks.ts`): that hook writes a
+ * PLAIN epoch-seconds integer to
+ * `~/.skillsmith/agent-nudge.state` (`SKILLSMITH_AGENT_NUDGE_STATE`
+ * override), because it is self-contained POSIX sh with no JSON tooling
+ * available. This module's {@link getNudgeCooldownState} reads that EXACT
+ * file/format/override so a future consumer (a report, a diagnostic) can
+ * inspect the onboarding-nudge cooldown through the same TS surface as the
+ * T1-T4 paywall triggers, without asking the hook to change its wire format.
+ * The T1-T4 trigger files themselves are independent, JSON-shaped, and own
+ * their own directory — see the module header of `store.ts` for why.
+ *
+ * @module @skillsmith/core/paywall-triggers/path
+ */
+import type { TriggerId } from './types.js';
+/** Env var overriding the trigger-state directory (test isolation). */
+export declare const TRIGGER_STATE_DIR_ENV_VAR = "SKILLSMITH_TRIGGER_STATE_DIR";
+export declare function getTriggerStateFilePath(triggerId: TriggerId): string;
+export declare function getTriggerStateDirForTests(): string;
+/**
+ * Read the onboarding SessionStart hook's nudge-cooldown stamp — the SAME
+ * file (`~/.skillsmith/agent-nudge.state`, override
+ * `SKILLSMITH_AGENT_NUDGE_STATE`) `hooks.ts`'s `renderSessionStartHook`
+ * writes a bare epoch-seconds integer into. Read-only: this module never
+ * writes this file — that stays the hook's job.
+ *
+ * @returns the last-nudge epoch-seconds, or null if the file is absent,
+ *   empty, or not a valid non-negative integer (matches the hook's own
+ *   tolerant `case "" | *[!0-9]*) last=0` parsing — malformed content is
+ *   treated as "no prior nudge", never an error).
+ */
+export declare function getNudgeCooldownState(): number | null;
+//# sourceMappingURL=path.d.ts.map

package/dist/src/paywall-triggers/path.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"path.d.ts","sourceRoot":"","sources":["../../../src/paywall-triggers/path.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AAMH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,YAAY,CAAA;AAE3C,uEAAuE;AACvE,eAAO,MAAM,yBAAyB,iCAAiC,CAAA;AAOvE,wBAAgB,uBAAuB,CAAC,SAAS,EAAE,SAAS,GAAG,MAAM,CAEpE;AAED,wBAAgB,0BAA0B,IAAI,MAAM,CAEnD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,qBAAqB,IAAI,MAAM,GAAG,IAAI,CAcrD"}

package/dist/src/paywall-triggers/path.js

@@ -0,0 +1,73 @@
+/**
+ * Trigger-state file layout (SMI-5456 Wave 1 Step 5, QD-3).
+ *
+ * `~/.skillsmith/paywall-triggers/<triggerId>.json` — one file per trigger,
+ * mirroring the per-resource-directory convention already used by
+ * `journal/path.ts` (`~/.skillsmith/journal/`) and
+ * `telemetry/agent-marker.ts` (`~/.skillsmith/agent-markers/`). Named
+ * `paywall-triggers` (not `triggers`) both in the module path and the
+ * on-disk directory to avoid any collision with the PRE-EXISTING, unrelated
+ * `packages/core/src/triggers/` module (`TriggerDetector`/`ContextScorer` —
+ * proactive skill-suggestion detection, nothing to do with PRD §8.1's T1-T4
+ * paywall triggers).
+ *
+ * Format compatibility with the SessionStart hook's nudge cooldown
+ * (the per-harness `hooks/<harness>/session-start.sh` script, generated by
+ * the — locked for this step — `agent-pack/hooks.ts`): that hook writes a
+ * PLAIN epoch-seconds integer to
+ * `~/.skillsmith/agent-nudge.state` (`SKILLSMITH_AGENT_NUDGE_STATE`
+ * override), because it is self-contained POSIX sh with no JSON tooling
+ * available. This module's {@link getNudgeCooldownState} reads that EXACT
+ * file/format/override so a future consumer (a report, a diagnostic) can
+ * inspect the onboarding-nudge cooldown through the same TS surface as the
+ * T1-T4 paywall triggers, without asking the hook to change its wire format.
+ * The T1-T4 trigger files themselves are independent, JSON-shaped, and own
+ * their own directory — see the module header of `store.ts` for why.
+ *
+ * @module @skillsmith/core/paywall-triggers/path
+ */
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { getConfigDir } from '../config/index.js';
+/** Env var overriding the trigger-state directory (test isolation). */
+export const TRIGGER_STATE_DIR_ENV_VAR = 'SKILLSMITH_TRIGGER_STATE_DIR';
+function getTriggerStateDir() {
+    const override = process.env[TRIGGER_STATE_DIR_ENV_VAR];
+    return override && override.length > 0 ? override : join(getConfigDir(), 'paywall-triggers');
+}
+export function getTriggerStateFilePath(triggerId) {
+    return join(getTriggerStateDir(), `${triggerId}.json`);
+}
+export function getTriggerStateDirForTests() {
+    return getTriggerStateDir();
+}
+/**
+ * Read the onboarding SessionStart hook's nudge-cooldown stamp — the SAME
+ * file (`~/.skillsmith/agent-nudge.state`, override
+ * `SKILLSMITH_AGENT_NUDGE_STATE`) `hooks.ts`'s `renderSessionStartHook`
+ * writes a bare epoch-seconds integer into. Read-only: this module never
+ * writes this file — that stays the hook's job.
+ *
+ * @returns the last-nudge epoch-seconds, or null if the file is absent,
+ *   empty, or not a valid non-negative integer (matches the hook's own
+ *   tolerant `case "" | *[!0-9]*) last=0` parsing — malformed content is
+ *   treated as "no prior nudge", never an error).
+ */
+export function getNudgeCooldownState() {
+    const override = process.env.SKILLSMITH_AGENT_NUDGE_STATE;
+    const path = override && override.length > 0 ? override : join(getConfigDir(), 'agent-nudge.state');
+    if (!existsSync(path))
+        return null;
+    let raw;
+    try {
+        raw = readFileSync(path, 'utf-8').trim();
+    }
+    catch {
+        return null;
+    }
+    if (!/^[0-9]+$/.test(raw))
+        return null;
+    const value = Number(raw);
+    return Number.isSafeInteger(value) ? value : null;
+}
+//# sourceMappingURL=path.js.map

package/dist/src/paywall-triggers/path.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"path.js","sourceRoot":"","sources":["../../../src/paywall-triggers/path.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AAEH,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAA;AAClD,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAA;AAEhC,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AAGjD,uEAAuE;AACvE,MAAM,CAAC,MAAM,yBAAyB,GAAG,8BAA8B,CAAA;AAEvE,SAAS,kBAAkB;IACzB,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,CAAA;IACvD,OAAO,QAAQ,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,EAAE,EAAE,kBAAkB,CAAC,CAAA;AAC9F,CAAC;AAED,MAAM,UAAU,uBAAuB,CAAC,SAAoB;IAC1D,OAAO,IAAI,CAAC,kBAAkB,EAAE,EAAE,GAAG,SAAS,OAAO,CAAC,CAAA;AACxD,CAAC;AAED,MAAM,UAAU,0BAA0B;IACxC,OAAO,kBAAkB,EAAE,CAAA;AAC7B,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,qBAAqB;IACnC,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,4BAA4B,CAAA;IACzD,MAAM,IAAI,GACR,QAAQ,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,EAAE,EAAE,mBAAmB,CAAC,CAAA;IACxF,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAA;IAClC,IAAI,GAAW,CAAA;IACf,IAAI,CAAC;QACH,GAAG,GAAG,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,IAAI,EAAE,CAAA;IAC1C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAA;IACb,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC;QAAE,OAAO,IAAI,CAAA;IACtC,MAAM,KAAK,GAAG,MAAM,CAAC,GAAG,CAAC,CAAA;IACzB,OAAO,MAAM,CAAC,aAAa,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAA;AACnD,CAAC"}

package/dist/src/paywall-triggers/store.d.ts

@@ -0,0 +1,75 @@
+/**
+ * Paywall trigger-state store (SMI-5456 Wave 1 Step 5, QD-3).
+ *
+ * PRD §8.1 defines four paywall triggers (T1 version currency, T2 quota
+ * forecast, T3 sharing, T4 security-depth disclosure) with two frequency
+ * rules: at most one upgrade prompt per session, and two dismissals of the
+ * same trigger suppress it for 30 days. This module persists the state those
+ * rules need — per-trigger last-shown session id, dismissal count, and mute
+ * expiry — under `~/.skillsmith/paywall-triggers/` (deliberately NOT
+ * `~/.skillsmith/triggers/` — see `path.ts`'s header for the pre-existing,
+ * unrelated `packages/core/src/triggers/` module this avoids colliding with).
+ *
+ * WAVE-1 SCOPE LIMITATION (read before wiring this into anything):
+ * The trigger RULES live in the pack's prompt wording (PRD §6 change 6) —
+ * the agent, running inside whatever harness's model, decides in
+ * conversation whether to mention a trigger, and the user's response
+ * ("no thanks") is plain conversational text the agent sees, not a
+ * structured tool call. There is currently NO MCP tool or CLI command that
+ * reports a T1/T2/T4 dismissal from that conversation into this store — no
+ * Wave-1 exclusion-9 tool covers it (PRD §10 exclusion 9's only two
+ * acknowledged exceptions are the undo tool and the Wave-2 stats surface).
+ * That means: {@link recordTriggerDismissal} is real, tested infrastructure
+ * with NO live caller in Wave 1. This is intentional staged infrastructure
+ * (the same shape the change-journal module shipped in — Step 3, before the
+ * apply tools became its first real caller in a later step) — NOT an
+ * "unconsumed dismissal-recording pathway" in the house anti-pattern sense,
+ * because nothing in this codebase currently calls it as if dismissals were
+ * being captured. Do not present in-conversation dismissal tracking as a
+ * shipped product behavior (docs, marketing copy, the eval-matrix report)
+ * until a real consumer exists. What DOES already work end-to-end in Wave 1:
+ * the session-cap half via {@link canShowTrigger}/{@link recordImpression} —
+ * though even that has no live caller yet either, since trigger *display*
+ * also currently lives entirely in the pack's prompt wording. Both halves
+ * are ready for whichever surface (a future tool, or a CLI diagnostic) needs
+ * them; the eval-matrix worker (Step 6) should treat "trigger frequency cap
+ * enforced server-side" as NOT YET TRUE for the published Tier badges.
+ *
+ * Single-writer-per-trigger-file note: like the journal (P-5), each trigger
+ * file is read-modify-written directly (not append-only, since there's only
+ * ever one live state per trigger, not a growing log) — concurrent writers
+ * to the SAME trigger id are not expected in Wave 1 (no live caller), so no
+ * write-queue serialization is implemented here; add one if/when a consumer
+ * with real concurrency (e.g. two parallel MCP requests) lands.
+ *
+ * @module @skillsmith/core/paywall-triggers/store
+ */
+import { type TriggerId, type TriggerState } from './types.js';
+/** Load a trigger's persisted state, or a fresh empty state if none exists or it is corrupt. */
+export declare function loadTriggerState(triggerId: TriggerId): TriggerState;
+/**
+ * Is `triggerId` currently muted (30-day suppression after
+ * {@link TRIGGER_DISMISSAL_THRESHOLD} dismissals)?
+ *
+ * @param now - injectable clock for tests.
+ */
+export declare function isTriggerMuted(state: TriggerState, now?: number): boolean;
+/**
+ * P-5 "one prompt per session enforced from persisted state, not memory":
+ * true when `triggerId` has NOT already been shown in `sessionId` AND is not
+ * currently muted. Concurrent sessions may each independently see `true` and
+ * both prompt once — accepted, documented (P-5 table).
+ */
+export declare function canShowTrigger(triggerId: TriggerId, sessionId: string, now?: number): boolean;
+/** Record that `triggerId` was shown in `sessionId` (persisted — the session cap). */
+export declare function recordImpression(triggerId: TriggerId, sessionId: string): TriggerState;
+/**
+ * Record a dismissal. On reaching {@link TRIGGER_DISMISSAL_THRESHOLD}
+ * dismissals, sets a {@link TRIGGER_MUTE_DAYS}-day mute from `now`.
+ *
+ * See the module header: no Wave-1 caller reports a real in-conversation
+ * dismissal today. This function is correct, tested library code — not an
+ * inert stub — for whichever surface becomes that caller.
+ */
+export declare function recordTriggerDismissal(triggerId: TriggerId, now?: number): TriggerState;
+//# sourceMappingURL=store.d.ts.map

package/dist/src/paywall-triggers/store.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"store.d.ts","sourceRoot":"","sources":["../../../src/paywall-triggers/store.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6CG;AAMH,OAAO,EAIL,KAAK,SAAS,EACd,KAAK,YAAY,EAClB,MAAM,YAAY,CAAA;AAInB,gGAAgG;AAChG,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,SAAS,GAAG,YAAY,CAiBnE;AAQD;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,KAAK,EAAE,YAAY,EAAE,GAAG,GAAE,MAAmB,GAAG,OAAO,CAErF;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAC5B,SAAS,EAAE,SAAS,EACpB,SAAS,EAAE,MAAM,EACjB,GAAG,GAAE,MAAmB,GACvB,OAAO,CAIT;AAED,sFAAsF;AACtF,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,MAAM,GAAG,YAAY,CAKtF;AAED;;;;;;;GAOG;AACH,wBAAgB,sBAAsB,CACpC,SAAS,EAAE,SAAS,EACpB,GAAG,GAAE,MAAmB,GACvB,YAAY,CAQd"}

package/dist/src/paywall-triggers/store.js

@@ -0,0 +1,122 @@
+/**
+ * Paywall trigger-state store (SMI-5456 Wave 1 Step 5, QD-3).
+ *
+ * PRD §8.1 defines four paywall triggers (T1 version currency, T2 quota
+ * forecast, T3 sharing, T4 security-depth disclosure) with two frequency
+ * rules: at most one upgrade prompt per session, and two dismissals of the
+ * same trigger suppress it for 30 days. This module persists the state those
+ * rules need — per-trigger last-shown session id, dismissal count, and mute
+ * expiry — under `~/.skillsmith/paywall-triggers/` (deliberately NOT
+ * `~/.skillsmith/triggers/` — see `path.ts`'s header for the pre-existing,
+ * unrelated `packages/core/src/triggers/` module this avoids colliding with).
+ *
+ * WAVE-1 SCOPE LIMITATION (read before wiring this into anything):
+ * The trigger RULES live in the pack's prompt wording (PRD §6 change 6) —
+ * the agent, running inside whatever harness's model, decides in
+ * conversation whether to mention a trigger, and the user's response
+ * ("no thanks") is plain conversational text the agent sees, not a
+ * structured tool call. There is currently NO MCP tool or CLI command that
+ * reports a T1/T2/T4 dismissal from that conversation into this store — no
+ * Wave-1 exclusion-9 tool covers it (PRD §10 exclusion 9's only two
+ * acknowledged exceptions are the undo tool and the Wave-2 stats surface).
+ * That means: {@link recordTriggerDismissal} is real, tested infrastructure
+ * with NO live caller in Wave 1. This is intentional staged infrastructure
+ * (the same shape the change-journal module shipped in — Step 3, before the
+ * apply tools became its first real caller in a later step) — NOT an
+ * "unconsumed dismissal-recording pathway" in the house anti-pattern sense,
+ * because nothing in this codebase currently calls it as if dismissals were
+ * being captured. Do not present in-conversation dismissal tracking as a
+ * shipped product behavior (docs, marketing copy, the eval-matrix report)
+ * until a real consumer exists. What DOES already work end-to-end in Wave 1:
+ * the session-cap half via {@link canShowTrigger}/{@link recordImpression} —
+ * though even that has no live caller yet either, since trigger *display*
+ * also currently lives entirely in the pack's prompt wording. Both halves
+ * are ready for whichever surface (a future tool, or a CLI diagnostic) needs
+ * them; the eval-matrix worker (Step 6) should treat "trigger frequency cap
+ * enforced server-side" as NOT YET TRUE for the published Tier badges.
+ *
+ * Single-writer-per-trigger-file note: like the journal (P-5), each trigger
+ * file is read-modify-written directly (not append-only, since there's only
+ * ever one live state per trigger, not a growing log) — concurrent writers
+ * to the SAME trigger id are not expected in Wave 1 (no live caller), so no
+ * write-queue serialization is implemented here; add one if/when a consumer
+ * with real concurrency (e.g. two parallel MCP requests) lands.
+ *
+ * @module @skillsmith/core/paywall-triggers/store
+ */
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { dirname } from 'node:path';
+import { getTriggerStateFilePath } from './path.js';
+import { TRIGGER_DISMISSAL_THRESHOLD, TRIGGER_MUTE_DAYS, emptyTriggerState, } from './types.js';
+const MUTE_DURATION_MS = TRIGGER_MUTE_DAYS * 24 * 60 * 60 * 1000;
+/** Load a trigger's persisted state, or a fresh empty state if none exists or it is corrupt. */
+export function loadTriggerState(triggerId) {
+    const path = getTriggerStateFilePath(triggerId);
+    if (!existsSync(path))
+        return emptyTriggerState(triggerId);
+    try {
+        const parsed = JSON.parse(readFileSync(path, 'utf-8'));
+        if (parsed.triggerId !== triggerId)
+            return emptyTriggerState(triggerId);
+        return {
+            schema: 1,
+            triggerId,
+            lastSessionShown: typeof parsed.lastSessionShown === 'string' ? parsed.lastSessionShown : null,
+            dismissalCount: typeof parsed.dismissalCount === 'number' ? parsed.dismissalCount : 0,
+            mutedUntil: typeof parsed.mutedUntil === 'number' ? parsed.mutedUntil : null,
+        };
+    }
+    catch {
+        return emptyTriggerState(triggerId);
+    }
+}
+function saveTriggerState(state) {
+    const path = getTriggerStateFilePath(state.triggerId);
+    mkdirSync(dirname(path), { recursive: true, mode: 0o700 });
+    writeFileSync(path, JSON.stringify(state, null, 2) + '\n', { mode: 0o600 });
+}
+/**
+ * Is `triggerId` currently muted (30-day suppression after
+ * {@link TRIGGER_DISMISSAL_THRESHOLD} dismissals)?
+ *
+ * @param now - injectable clock for tests.
+ */
+export function isTriggerMuted(state, now = Date.now()) {
+    return state.mutedUntil !== null && now < state.mutedUntil;
+}
+/**
+ * P-5 "one prompt per session enforced from persisted state, not memory":
+ * true when `triggerId` has NOT already been shown in `sessionId` AND is not
+ * currently muted. Concurrent sessions may each independently see `true` and
+ * both prompt once — accepted, documented (P-5 table).
+ */
+export function canShowTrigger(triggerId, sessionId, now = Date.now()) {
+    const state = loadTriggerState(triggerId);
+    if (isTriggerMuted(state, now))
+        return false;
+    return state.lastSessionShown !== sessionId;
+}
+/** Record that `triggerId` was shown in `sessionId` (persisted — the session cap). */
+export function recordImpression(triggerId, sessionId) {
+    const state = loadTriggerState(triggerId);
+    const next = { ...state, lastSessionShown: sessionId };
+    saveTriggerState(next);
+    return next;
+}
+/**
+ * Record a dismissal. On reaching {@link TRIGGER_DISMISSAL_THRESHOLD}
+ * dismissals, sets a {@link TRIGGER_MUTE_DAYS}-day mute from `now`.
+ *
+ * See the module header: no Wave-1 caller reports a real in-conversation
+ * dismissal today. This function is correct, tested library code — not an
+ * inert stub — for whichever surface becomes that caller.
+ */
+export function recordTriggerDismissal(triggerId, now = Date.now()) {
+    const state = loadTriggerState(triggerId);
+    const dismissalCount = state.dismissalCount + 1;
+    const mutedUntil = dismissalCount >= TRIGGER_DISMISSAL_THRESHOLD ? now + MUTE_DURATION_MS : state.mutedUntil;
+    const next = { ...state, dismissalCount, mutedUntil };
+    saveTriggerState(next);
+    return next;
+}
+//# sourceMappingURL=store.js.map

package/dist/src/paywall-triggers/store.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"store.js","sourceRoot":"","sources":["../../../src/paywall-triggers/store.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6CG;AAEH,OAAO,EAAE,UAAU,EAAE,SAAS,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,SAAS,CAAA;AAC5E,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AAEnC,OAAO,EAAE,uBAAuB,EAAE,MAAM,WAAW,CAAA;AACnD,OAAO,EACL,2BAA2B,EAC3B,iBAAiB,EACjB,iBAAiB,GAGlB,MAAM,YAAY,CAAA;AAEnB,MAAM,gBAAgB,GAAG,iBAAiB,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAA;AAEhE,gGAAgG;AAChG,MAAM,UAAU,gBAAgB,CAAC,SAAoB;IACnD,MAAM,IAAI,GAAG,uBAAuB,CAAC,SAAS,CAAC,CAAA;IAC/C,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,iBAAiB,CAAC,SAAS,CAAC,CAAA;IAC1D,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAA0B,CAAA;QAC/E,IAAI,MAAM,CAAC,SAAS,KAAK,SAAS;YAAE,OAAO,iBAAiB,CAAC,SAAS,CAAC,CAAA;QACvE,OAAO;YACL,MAAM,EAAE,CAAC;YACT,SAAS;YACT,gBAAgB,EACd,OAAO,MAAM,CAAC,gBAAgB,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,CAAC,IAAI;YAC9E,cAAc,EAAE,OAAO,MAAM,CAAC,cAAc,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC;YACrF,UAAU,EAAE,OAAO,MAAM,CAAC,UAAU,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI;SAC7E,CAAA;IACH,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,iBAAiB,CAAC,SAAS,CAAC,CAAA;IACrC,CAAC;AACH,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAmB;IAC3C,MAAM,IAAI,GAAG,uBAAuB,CAAC,KAAK,CAAC,SAAS,CAAC,CAAA;IACrD,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,CAAA;IAC1D,aAAa,CAAC,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,CAAA;AAC7E,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,cAAc,CAAC,KAAmB,EAAE,MAAc,IAAI,CAAC,GAAG,EAAE;IAC1E,OAAO,KAAK,CAAC,UAAU,KAAK,IAAI,IAAI,GAAG,GAAG,KAAK,CAAC,UAAU,CAAA;AAC5D,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,cAAc,CAC5B,SAAoB,EACpB,SAAiB,EACjB,MAAc,IAAI,CAAC,GAAG,EAAE;IAExB,MAAM,KAAK,GAAG,gBAAgB,CAAC,SAAS,CAAC,CAAA;IACzC,IAAI,cAAc,CAAC,KAAK,EAAE,GAAG,CAAC;QAAE,OAAO,KAAK,CAAA;IAC5C,OAAO,KAAK,CAAC,gBAAgB,KAAK,SAAS,CAAA;AAC7C,CAAC;AAED,sFAAsF;AACtF,MAAM,UAAU,gBAAgB,CAAC,SAAoB,EAAE,SAAiB;IACtE,MAAM,KAAK,GAAG,gBAAgB,CAAC,SAAS,CAAC,CAAA;IACzC,MAAM,IAAI,GAAiB,EAAE,GAAG,KAAK,EAAE,gBAAgB,EAAE,SAAS,EAAE,CAAA;IACpE,gBAAgB,CAAC,IAAI,CAAC,CAAA;IACtB,OAAO,IAAI,CAAA;AACb,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,sBAAsB,CACpC,SAAoB,EACpB,MAAc,IAAI,CAAC,GAAG,EAAE;IAExB,MAAM,KAAK,GAAG,gBAAgB,CAAC,SAAS,CAAC,CAAA;IACzC,MAAM,cAAc,GAAG,KAAK,CAAC,cAAc,GAAG,CAAC,CAAA;IAC/C,MAAM,UAAU,GACd,cAAc,IAAI,2BAA2B,CAAC,CAAC,CAAC,GAAG,GAAG,gBAAgB,CAAC,CAAC,CAAC,KAAK,CAAC,UAAU,CAAA;IAC3F,MAAM,IAAI,GAAiB,EAAE,GAAG,KAAK,EAAE,cAAc,EAAE,UAAU,EAAE,CAAA;IACnE,gBAAgB,CAAC,IAAI,CAAC,CAAA;IACtB,OAAO,IAAI,CAAA;AACb,CAAC"}

package/dist/src/paywall-triggers/store.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=store.test.d.ts.map

package/dist/src/paywall-triggers/store.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"store.test.d.ts","sourceRoot":"","sources":["../../../src/paywall-triggers/store.test.ts"],"names":[],"mappings":""}

package/dist/src/paywall-triggers/store.test.js

@@ -0,0 +1,121 @@
+/**
+ * @fileoverview Tests for the paywall trigger-state store (SMI-5456 Wave 1
+ *               Step 5, QD-3). Covers the P-5 "Trigger state" invariant:
+ *               "One prompt per session enforced from persisted state, not
+ *               memory" (cap) + "30-day mute expiry" test.
+ * @module @skillsmith/core/paywall-triggers/store.test
+ */
+import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { afterEach, beforeEach, describe, expect, it } from 'vitest';
+import { canShowTrigger, getNudgeCooldownState, isTriggerMuted, loadTriggerState, recordImpression, recordTriggerDismissal, TRIGGER_DISMISSAL_THRESHOLD, TRIGGER_MUTE_DAYS, TRIGGER_STATE_DIR_ENV_VAR, } from './index.js';
+let stateDir;
+let prevEnv;
+let prevNudgeEnv;
+beforeEach(() => {
+    stateDir = mkdtempSync(join(tmpdir(), 'skillsmith-paywall-triggers-'));
+    prevEnv = process.env[TRIGGER_STATE_DIR_ENV_VAR];
+    process.env[TRIGGER_STATE_DIR_ENV_VAR] = stateDir;
+    prevNudgeEnv = process.env.SKILLSMITH_AGENT_NUDGE_STATE;
+});
+afterEach(() => {
+    rmSync(stateDir, { recursive: true, force: true });
+    if (prevEnv !== undefined)
+        process.env[TRIGGER_STATE_DIR_ENV_VAR] = prevEnv;
+    else
+        delete process.env[TRIGGER_STATE_DIR_ENV_VAR];
+    if (prevNudgeEnv !== undefined)
+        process.env.SKILLSMITH_AGENT_NUDGE_STATE = prevNudgeEnv;
+    else
+        delete process.env.SKILLSMITH_AGENT_NUDGE_STATE;
+});
+describe('session cap (persisted, not memory)', () => {
+    it('allows showing a trigger the first time in a session', () => {
+        expect(canShowTrigger('T1', 'session-a')).toBe(true);
+    });
+    it('blocks a second impression of the same trigger in the same session', () => {
+        recordImpression('T1', 'session-a');
+        expect(canShowTrigger('T1', 'session-a')).toBe(false);
+    });
+    it('the cap is read from persisted state, not in-memory — a fresh loadTriggerState call after recordImpression reflects it', () => {
+        recordImpression('T2', 'session-x');
+        const state = loadTriggerState('T2');
+        expect(state.lastSessionShown).toBe('session-x');
+    });
+    it('allows the SAME trigger again in a DIFFERENT session (documented: concurrent sessions may each prompt once)', () => {
+        recordImpression('T1', 'session-a');
+        expect(canShowTrigger('T1', 'session-b')).toBe(true);
+    });
+    it('caps are independent per trigger id', () => {
+        recordImpression('T1', 'session-a');
+        expect(canShowTrigger('T3', 'session-a')).toBe(true);
+    });
+});
+describe('30-day mute expiry', () => {
+    it('does not mute after one dismissal', () => {
+        const state = recordTriggerDismissal('T4');
+        expect(state.dismissalCount).toBe(1);
+        expect(isTriggerMuted(state)).toBe(false);
+    });
+    it(`mutes after ${TRIGGER_DISMISSAL_THRESHOLD} dismissals`, () => {
+        recordTriggerDismissal('T4');
+        const state = recordTriggerDismissal('T4');
+        expect(state.dismissalCount).toBe(TRIGGER_DISMISSAL_THRESHOLD);
+        expect(isTriggerMuted(state)).toBe(true);
+    });
+    it('a muted trigger cannot be shown, even in a session it was never shown in', () => {
+        recordTriggerDismissal('T1');
+        recordTriggerDismissal('T1');
+        expect(canShowTrigger('T1', 'brand-new-session')).toBe(false);
+    });
+    it(`is still muted 1ms before ${TRIGGER_MUTE_DAYS} days elapse`, () => {
+        const now = 1_000_000;
+        recordTriggerDismissal('T2', now);
+        recordTriggerDismissal('T2', now);
+        const almostExpired = now + TRIGGER_MUTE_DAYS * 24 * 60 * 60 * 1000 - 1;
+        const state = loadTriggerState('T2');
+        expect(isTriggerMuted(state, almostExpired)).toBe(true);
+    });
+    it(`the mute has expired exactly at ${TRIGGER_MUTE_DAYS} days`, () => {
+        const now = 1_000_000;
+        recordTriggerDismissal('T2', now);
+        recordTriggerDismissal('T2', now);
+        const exactlyExpired = now + TRIGGER_MUTE_DAYS * 24 * 60 * 60 * 1000;
+        const state = loadTriggerState('T2');
+        expect(isTriggerMuted(state, exactlyExpired)).toBe(false);
+    });
+    it('canShowTrigger allows the trigger again once the mute has expired', () => {
+        const now = 1_000_000;
+        recordTriggerDismissal('T3', now);
+        recordTriggerDismissal('T3', now);
+        const afterMute = now + TRIGGER_MUTE_DAYS * 24 * 60 * 60 * 1000 + 1;
+        expect(canShowTrigger('T3', 'any-session', afterMute)).toBe(true);
+    });
+    it('dismissal count keeps accumulating past the threshold (never resets)', () => {
+        recordTriggerDismissal('T1');
+        recordTriggerDismissal('T1');
+        const state = recordTriggerDismissal('T1');
+        expect(state.dismissalCount).toBe(3);
+    });
+});
+describe('getNudgeCooldownState — format compatibility with the SessionStart hook', () => {
+    it('returns null when the nudge-state file does not exist', () => {
+        const nudgeFile = join(stateDir, 'agent-nudge.state');
+        process.env.SKILLSMITH_AGENT_NUDGE_STATE = nudgeFile;
+        expect(getNudgeCooldownState()).toBeNull();
+    });
+    it('reads a plain epoch-seconds integer written by the hook (no JSON, no wrapper)', () => {
+        const nudgeFile = join(stateDir, 'agent-nudge.state');
+        process.env.SKILLSMITH_AGENT_NUDGE_STATE = nudgeFile;
+        writeFileSync(nudgeFile, '1735689600');
+        expect(getNudgeCooldownState()).toBe(1735689600);
+    });
+    it('treats malformed content as "no prior nudge" (matches the hook\'s own tolerant parsing)', () => {
+        const nudgeFile = join(stateDir, 'agent-nudge.state');
+        process.env.SKILLSMITH_AGENT_NUDGE_STATE = nudgeFile;
+        writeFileSync(nudgeFile, 'not-a-number');
+        expect(getNudgeCooldownState()).toBeNull();
+    });
+});
+//# sourceMappingURL=store.test.js.map