clementine-agent 1.0.98 → 1.0.99

package/dist/vault-migrations/runner.d.ts

@@ -5,10 +5,19 @@
  * Each exports a `migration` object conforming to VaultMigration.
  * State is tracked in ~/.clementine/.vault-migrations.json.
  */
-import type { VaultMigrationSummary } from './types.js';
+import type { MigrationContext, VaultMigrationSummary } from './types.js';
 /**
- * Run all pending vault migrations against the user's vault.
- * Idempotent — safe to call multiple times.
+ * Run all pending migrations. Each is dispatched based on its shape:
+ *   - VaultMigration (no `kind` field) → apply(vaultDir)
+ *   - Migration       (has `kind` field) → apply(MigrationContext)
+ *
+ * Idempotent — safe to call multiple times. State is tracked in
+ * `~/.clementine/.vault-migrations.json` (kept this filename for back-compat).
+ */
+export declare function runMigrations(ctx: MigrationContext, backupDir?: string): Promise<VaultMigrationSummary>;
+/**
+ * Back-compat wrapper: existing call sites pass just vaultDir. Builds a
+ * default MigrationContext from BASE_DIR and PKG_DIR.
  */
 export declare function runVaultMigrations(vaultDir: string, backupDir?: string): Promise<VaultMigrationSummary>;
 //# sourceMappingURL=runner.d.ts.map

package/dist/vault-migrations/runner.js

@@ -8,7 +8,7 @@
 import { existsSync, readFileSync, writeFileSync, mkdirSync, readdirSync, copyFileSync } from 'node:fs';
 import path from 'node:path';
 import pino from 'pino';
-import { VAULT_MIGRATIONS_STATE } from '../config.js';
+import { BASE_DIR, PKG_DIR, VAULT_MIGRATIONS_STATE } from '../config.js';
 const logger = pino({ name: 'clementine.vault-migrations' });
 /** Load the migration state file. Returns empty state if missing or corrupt. */
 function loadState() {
@@ -41,6 +41,7 @@ function backupFile(filePath, backupDir) {
 /**
  * Discover all migration modules in the compiled dist/vault-migrations/ directory.
  * Returns them sorted by filename (numeric prefix ensures correct order).
+ * Accepts both VaultMigration (legacy) and Migration (multi-target) shapes.
  */
 async function discoverMigrations() {
     const migrations = [];
@@ -64,16 +65,24 @@ async function discoverMigrations() {
             }
         }
         catch (err) {
-            logger.warn({ file, err }, 'Failed to load vault migration');
+            logger.warn({ file, err }, 'Failed to load migration');
         }
     }
     return migrations;
 }
+/** Discriminator — true if migration uses the new MigrationContext shape. */
+function isContextMigration(m) {
+    return typeof m.kind === 'string';
+}
 /**
- * Run all pending vault migrations against the user's vault.
- * Idempotent — safe to call multiple times.
+ * Run all pending migrations. Each is dispatched based on its shape:
+ *   - VaultMigration (no `kind` field) → apply(vaultDir)
+ *   - Migration       (has `kind` field) → apply(MigrationContext)
+ *
+ * Idempotent — safe to call multiple times. State is tracked in
+ * `~/.clementine/.vault-migrations.json` (kept this filename for back-compat).
  */
-export async function runVaultMigrations(vaultDir, backupDir) {
+export async function runMigrations(ctx, backupDir) {
     const summary = {
         applied: [],
         skipped: [],
@@ -87,16 +96,14 @@ export async function runVaultMigrations(vaultDir, backupDir) {
     if (migrations.length === 0)
         return summary;
     for (const migration of migrations) {
-        // Skip if already recorded as applied
         if (appliedIds.has(migration.id)) {
             summary.alreadyRun.push(migration.id);
             continue;
         }
         try {
-            // Back up target files before migration
-            if (backupDir) {
-                // Best-effort backup of the vault's 00-System dir (most common target)
-                const systemDir = path.join(vaultDir, '00-System');
+            // Best-effort backup of the vault's 00-System dir for vault-style migrations
+            if (backupDir && !isContextMigration(migration)) {
+                const systemDir = path.join(ctx.vaultDir, '00-System');
                 if (existsSync(systemDir)) {
                     const systemFiles = readdirSync(systemDir).filter(f => f.endsWith('.md'));
                     for (const f of systemFiles) {
@@ -104,36 +111,36 @@ export async function runVaultMigrations(vaultDir, backupDir) {
                     }
                 }
             }
-            const result = migration.apply(vaultDir);
-            if (result.applied) {
+            const result = isContextMigration(migration)
+                ? await migration.apply(ctx)
+                : migration.apply(ctx.vaultDir);
+            const r = result;
+            if (r.applied) {
                 summary.applied.push(migration.id);
-                state.applied.push({
-                    id: migration.id,
-                    appliedAt: new Date().toISOString(),
-                    result: 'applied',
-                });
-                logger.info({ id: migration.id, details: result.details }, 'Vault migration applied');
+                state.applied.push({ id: migration.id, appliedAt: new Date().toISOString(), result: 'applied' });
+                logger.info({ id: migration.id, details: r.details }, 'Migration applied');
             }
-            else if (result.skipped) {
+            else if (r.skipped) {
                 summary.skipped.push(migration.id);
-                // Record as applied so we don't re-check every update
-                state.applied.push({
-                    id: migration.id,
-                    appliedAt: new Date().toISOString(),
-                    result: 'skipped',
-                });
-                logger.info({ id: migration.id, details: result.details }, 'Vault migration skipped (already present)');
+                state.applied.push({ id: migration.id, appliedAt: new Date().toISOString(), result: 'skipped' });
+                logger.info({ id: migration.id, details: r.details }, 'Migration skipped (already present)');
             }
         }
         catch (err) {
             const errMsg = String(err).slice(0, 200);
             summary.failed.push(migration.id);
             summary.errors.push({ id: migration.id, error: errMsg });
-            // Do NOT record as applied — will retry on next update
-            logger.warn({ id: migration.id, err }, 'Vault migration failed');
+            logger.warn({ id: migration.id, err }, 'Migration failed');
         }
     }
     saveState(state);
     return summary;
 }
+/**
+ * Back-compat wrapper: existing call sites pass just vaultDir. Builds a
+ * default MigrationContext from BASE_DIR and PKG_DIR.
+ */
+export async function runVaultMigrations(vaultDir, backupDir) {
+    return runMigrations({ vaultDir, baseDir: BASE_DIR, pkgDir: PKG_DIR }, backupDir);
+}
 //# sourceMappingURL=runner.js.map

package/dist/vault-migrations/types.d.ts

@@ -1,10 +1,27 @@
 /**
- * Vault migration system — types and interfaces.
+ * Migration system — types and interfaces.
  *
- * Vault migrations ship structural changes to user vault files (SOUL.md,
- * AGENTS.md, etc.) alongside code updates. Each migration is idempotent
- * and runs once during `clementine update`.
+ * Migrations ship structural changes to user data files (SOUL.md, AGENTS.md,
+ * advisor rules, prompt overrides, clementine.json, etc.) alongside code
+ * updates. Each migration is idempotent and runs once during `clementine update`.
+ *
+ * Two shapes coexist for back-compat:
+ *   - VaultMigration: takes vaultDir only (the original shape, used by 0001-0003)
+ *   - Migration: takes a MigrationContext { vaultDir, baseDir, pkgDir } so a
+ *     migration can touch advisor-rules/, prompt-overrides/, clementine.json,
+ *     etc. — anything under ~/.clementine/, not just the vault.
+ *
+ * Discriminator: `kind` field. Vault-style migrations omit it.
  */
+export interface MigrationContext {
+    /** ~/.clementine/vault */
+    vaultDir: string;
+    /** ~/.clementine/  (data home — config, state, cache, advisor-rules, etc.) */
+    baseDir: string;
+    /** Package install root (where dist/ lives). For migrations that ship templates. */
+    pkgDir: string;
+}
+/** Original vault-only migration shape. Existing 0001-0003 use this. */
 export interface VaultMigration {
     /** Unique ID matching the filename (e.g., "0001-add-execution-framework"). */
     id: string;
@@ -13,6 +30,15 @@ export interface VaultMigration {
     /** Apply the migration. Must be idempotent — safe to re-run. */
     apply: (vaultDir: string) => MigrationResult;
 }
+/** Multi-target migration shape. Use for any data outside vault/. */
+export interface Migration {
+    /** Discriminator. Vault-only migrations omit this and use VaultMigration. */
+    kind: 'vault' | 'config' | 'advisor-rules' | 'prompt-overrides' | 'multi';
+    id: string;
+    description: string;
+    apply: (ctx: MigrationContext) => MigrationResult | Promise<MigrationResult>;
+}
+export type AnyMigration = VaultMigration | Migration;
 export interface MigrationResult {
     /** True if changes were written to disk. */
     applied: boolean;
@@ -39,4 +65,6 @@ export interface VaultMigrationSummary {
         error: string;
     }>;
 }
+/** Alias — same shape, generalized name now that migrations aren't vault-only. */
+export type MigrationSummary = VaultMigrationSummary;
 //# sourceMappingURL=types.d.ts.map

package/dist/vault-migrations/types.js

@@ -1,9 +1,17 @@
 /**
- * Vault migration system — types and interfaces.
+ * Migration system — types and interfaces.
  *
- * Vault migrations ship structural changes to user vault files (SOUL.md,
- * AGENTS.md, etc.) alongside code updates. Each migration is idempotent
- * and runs once during `clementine update`.
+ * Migrations ship structural changes to user data files (SOUL.md, AGENTS.md,
+ * advisor rules, prompt overrides, clementine.json, etc.) alongside code
+ * updates. Each migration is idempotent and runs once during `clementine update`.
+ *
+ * Two shapes coexist for back-compat:
+ *   - VaultMigration: takes vaultDir only (the original shape, used by 0001-0003)
+ *   - Migration: takes a MigrationContext { vaultDir, baseDir, pkgDir } so a
+ *     migration can touch advisor-rules/, prompt-overrides/, clementine.json,
+ *     etc. — anything under ~/.clementine/, not just the vault.
+ *
+ * Discriminator: `kind` field. Vault-style migrations omit it.
  */
 export {};
 //# sourceMappingURL=types.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.0.98",
+  "version": "1.0.99",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",