spine-framework 0.3.46 → 0.3.48

package/.framework/cli/commands/migrate.ts

@@ -90,10 +90,39 @@ async function migrateCommand(options: MigrateOptions): Promise<void> {
     process.exit(1)
   }
 
+  // Create migration tracking table if it doesn't exist
+  try {
+    await client.query(`
+      CREATE TABLE IF NOT EXISTS _spine_migrations (
+        id SERIAL PRIMARY KEY,
+        filename TEXT UNIQUE NOT NULL,
+        applied_at TIMESTAMPTZ DEFAULT NOW()
+      )
+    `)
+  } catch {
+    // Non-fatal: proceed without tracking
+  }
+
+  // Load already-applied migrations
+  let appliedMigrations = new Set<string>()
+  try {
+    const { rows } = await client.query('SELECT filename FROM _spine_migrations')
+    appliedMigrations = new Set(rows.map((r: { filename: string }) => r.filename))
+  } catch {
+    // Non-fatal: proceed without tracking
+  }
+
   let applied = 0
+  let skipped = 0
   let failed = 0
 
   for (const { name, path } of migrationFiles) {
+    if (appliedMigrations.has(name)) {
+      console.log(`  Skipping ${name} (already applied)`)
+      skipped++
+      continue
+    }
+
     const sql = readFileSync(path, 'utf8')
     process.stdout.write(`  Applying ${name}... `)
 
@@ -103,6 +132,11 @@ async function migrateCommand(options: MigrateOptions): Promise<void> {
       for (const stmt of statements) {
         await client.query(stmt)
       }
+      // Record as applied
+      await client.query(
+        `INSERT INTO _spine_migrations (filename) VALUES ($1) ON CONFLICT (filename) DO NOTHING`,
+        [name]
+      )
       console.log('✓')
       applied++
     } catch (err: any) {
@@ -114,7 +148,7 @@ async function migrateCommand(options: MigrateOptions): Promise<void> {
 
   await client.end()
 
-  console.log(`\n  ${applied} applied, ${failed} failed`)
+  console.log(`\n  ${applied} applied, ${skipped} skipped, ${failed} failed`)
 
   if (failed > 0) {
     console.error('\nSome migrations failed. Check errors above.')

package/.framework/migrations/002_rls_and_functions.sql

@@ -41,7 +41,7 @@ BEGIN
   -- Look up internal person ID from auth_uid
   SELECT p.id INTO v_person_id
   FROM public.people p
-  WHERE p.auth_uid = v_auth_id::text AND p.is_active = true
+  WHERE p.auth_uid = v_auth_id AND p.is_active = true
   LIMIT 1;
   
   RETURN v_person_id;

package/.framework/migrations/006_fix_current_actor_id.sql

@@ -0,0 +1,21 @@
+-- Patch: fix uuid = text operator error in current_actor_id()
+-- The previous version cast v_auth_id to text before comparing with auth_uid (uuid),
+-- which causes PostgreSQL error 42883. This re-creates the function with matching types.
+
+CREATE OR REPLACE FUNCTION public.current_actor_id()
+RETURNS uuid AS $$
+DECLARE
+  v_auth_id UUID;
+  v_person_id UUID;
+BEGIN
+  v_auth_id := auth.uid();
+
+  -- Look up internal person ID from auth_uid
+  SELECT p.id INTO v_person_id
+  FROM public.people p
+  WHERE p.auth_uid = v_auth_id AND p.is_active = true
+  LIMIT 1;
+
+  RETURN v_person_id;
+END;
+$$ LANGUAGE plpgsql SECURITY DEFINER STABLE;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spine-framework",
-  "version": "0.3.46",
+  "version": "0.3.48",
   "description": "Multi-tenant, modular application platform for modern SaaS systems",
   "type": "module",
   "bin": {
@@ -20,6 +20,7 @@
     ".framework/migrations/003_system_data.sql",
     ".framework/migrations/004_enhancements.sql",
     ".framework/migrations/005_auth_user_trigger.sql",
+    ".framework/migrations/006_fix_current_actor_id.sql",
     ".framework/public/",
     ".framework/scripts/",
     ".framework/seeds/",