scene-capability-engine 3.6.38 → 3.6.44

package/lib/state/sce-state-store.js

@@ -162,6 +162,7 @@ class SceStateStore {
       migration_records: {},
       auth_leases: {},
       auth_events: [],
+      interactive_approval_events: {},
       sequences: {
         scene_next: 1,
         spec_next_by_scene: {},
@@ -294,6 +295,34 @@ class SceStateStore {
       CREATE INDEX IF NOT EXISTS idx_auth_event_stream_ts
         ON auth_event_stream(event_timestamp);
 
+      CREATE TABLE IF NOT EXISTS interactive_approval_event_projection (
+        event_id TEXT PRIMARY KEY,
+        workflow_id TEXT,
+        event_timestamp TEXT NOT NULL,
+        event_type TEXT NOT NULL,
+        action TEXT,
+        actor TEXT,
+        actor_role TEXT,
+        from_status TEXT,
+        to_status TEXT,
+        blocked INTEGER,
+        reason TEXT,
+        audit_file TEXT,
+        line_no INTEGER,
+        raw_json TEXT NOT NULL,
+        source TEXT,
+        indexed_at TEXT NOT NULL
+      );
+
+      CREATE INDEX IF NOT EXISTS idx_interactive_approval_event_projection_workflow_ts
+        ON interactive_approval_event_projection(workflow_id, event_timestamp DESC);
+
+      CREATE INDEX IF NOT EXISTS idx_interactive_approval_event_projection_actor_action_ts
+        ON interactive_approval_event_projection(actor, action, event_timestamp DESC);
+
+      CREATE INDEX IF NOT EXISTS idx_interactive_approval_event_projection_blocked_ts
+        ON interactive_approval_event_projection(blocked, event_timestamp DESC);
+
       CREATE TABLE IF NOT EXISTS timeline_snapshot_registry (
         snapshot_id TEXT PRIMARY KEY,
         created_at TEXT NOT NULL,
@@ -1007,6 +1036,30 @@ class SceStateStore {
     };
   }
 
+  _mapInteractiveApprovalEventProjectionRow(row) {
+    if (!row) {
+      return null;
+    }
+    return {
+      event_id: normalizeString(row.event_id),
+      workflow_id: normalizeString(row.workflow_id) || null,
+      event_timestamp: normalizeIsoTimestamp(row.event_timestamp) || null,
+      event_type: normalizeString(row.event_type),
+      action: normalizeString(row.action) || null,
+      actor: normalizeString(row.actor) || null,
+      actor_role: normalizeString(row.actor_role) || null,
+      from_status: normalizeString(row.from_status) || null,
+      to_status: normalizeString(row.to_status) || null,
+      blocked: normalizeBooleanValue(row.blocked, false),
+      reason: normalizeString(row.reason) || null,
+      audit_file: normalizeString(row.audit_file) || null,
+      line_no: normalizeNonNegativeInteger(row.line_no, 0),
+      raw: parseJsonSafe(row.raw_json, null),
+      source: normalizeString(row.source) || null,
+      indexed_at: normalizeIsoTimestamp(row.indexed_at) || null
+    };
+  }
+
   _mapTimelineSnapshotRow(row) {
     if (!row) {
       return null;
@@ -2151,6 +2204,218 @@ class SceStateStore {
       .filter(Boolean);
   }
 
+  async clearInteractiveApprovalEventProjection(options = {}) {
+    const auditFileFilter = normalizeString(options.auditFile || options.audit_file);
+
+    if (this._useMemoryBackend()) {
+      if (!auditFileFilter) {
+        this._memory.interactive_approval_events = {};
+        return { success: true, removed: 0 };
+      }
+      let removed = 0;
+      for (const [eventId, item] of Object.entries(this._memory.interactive_approval_events || {})) {
+        if (normalizeString(item.audit_file) === auditFileFilter) {
+          delete this._memory.interactive_approval_events[eventId];
+          removed += 1;
+        }
+      }
+      return { success: true, removed };
+    }
+
+    if (!await this.ensureReady()) {
+      return null;
+    }
+
+    if (auditFileFilter) {
+      const info = this._db
+        .prepare('DELETE FROM interactive_approval_event_projection WHERE audit_file = ?')
+        .run(auditFileFilter);
+      return {
+        success: true,
+        removed: normalizeNonNegativeInteger(info && info.changes, 0)
+      };
+    }
+
+    const info = this._db
+      .prepare('DELETE FROM interactive_approval_event_projection')
+      .run();
+    return {
+      success: true,
+      removed: normalizeNonNegativeInteger(info && info.changes, 0)
+    };
+  }
+
+  async upsertInteractiveApprovalEventProjection(records = [], options = {}) {
+    const source = normalizeString(options.source) || 'jsonl.interactive-approval-events';
+    const auditFile = normalizeString(options.auditFile || options.audit_file) || null;
+    const nowIso = this.now();
+    const normalizedRecords = Array.isArray(records)
+      ? records.map((item, index) => ({
+        event_id: normalizeString(item && item.event_id),
+        workflow_id: normalizeString(item && item.workflow_id) || null,
+        event_timestamp: normalizeIsoTimestamp(item && (item.event_timestamp || item.timestamp), nowIso) || nowIso,
+        event_type: normalizeString(item && item.event_type),
+        action: normalizeString(item && item.action) || null,
+        actor: normalizeString(item && item.actor) || null,
+        actor_role: normalizeString(item && item.actor_role) || null,
+        from_status: normalizeString(item && item.from_status) || null,
+        to_status: normalizeString(item && item.to_status) || null,
+        blocked: normalizeBooleanValue(item && item.blocked, false),
+        reason: normalizeString(item && item.reason) || null,
+        audit_file: normalizeString(item && (item.audit_file || item.auditFile)) || auditFile,
+        line_no: normalizeNonNegativeInteger(item && (item.line_no || item.lineNo), index + 1),
+        raw_json: JSON.stringify(item && typeof item === 'object' ? item : {}),
+        source,
+        indexed_at: nowIso
+      }))
+        .filter((item) => item.event_id && item.event_type)
+      : [];
+
+    if (this._useMemoryBackend()) {
+      for (const item of normalizedRecords) {
+        this._memory.interactive_approval_events[item.event_id] = { ...item };
+      }
+      return {
+        success: true,
+        written: normalizedRecords.length,
+        total: Object.keys(this._memory.interactive_approval_events || {}).length
+      };
+    }
+
+    if (!await this.ensureReady()) {
+      return null;
+    }
+
+    const statement = this._db.prepare(`
+      INSERT OR REPLACE INTO interactive_approval_event_projection(
+        event_id, workflow_id, event_timestamp, event_type, action, actor, actor_role,
+        from_status, to_status, blocked, reason, audit_file, line_no, raw_json, source, indexed_at
+      )
+      VALUES(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+    `);
+
+    this._withTransaction(() => {
+      for (const item of normalizedRecords) {
+        statement.run(
+          item.event_id,
+          item.workflow_id,
+          item.event_timestamp,
+          item.event_type,
+          item.action,
+          item.actor,
+          item.actor_role,
+          item.from_status,
+          item.to_status,
+          item.blocked ? 1 : 0,
+          item.reason,
+          item.audit_file,
+          item.line_no,
+          item.raw_json,
+          item.source,
+          item.indexed_at
+        );
+      }
+    });
+
+    const totalRow = this._db
+      .prepare('SELECT COUNT(*) AS total FROM interactive_approval_event_projection')
+      .get();
+
+    return {
+      success: true,
+      written: normalizedRecords.length,
+      total: normalizeNonNegativeInteger(totalRow && totalRow.total, 0)
+    };
+  }
+
+  async listInteractiveApprovalEventProjection(options = {}) {
+    const limit = normalizeInteger(options.limit, 100);
+    const workflowId = normalizeString(options.workflowId || options.workflow_id);
+    const actor = normalizeString(options.actor);
+    const action = normalizeString(options.action);
+    const eventType = normalizeString(options.eventType || options.event_type);
+    const auditFile = normalizeString(options.auditFile || options.audit_file);
+    const blockedFilter = options.blocked === undefined || options.blocked === null
+      ? null
+      : normalizeBooleanValue(options.blocked, false);
+
+    if (this._useMemoryBackend()) {
+      let rows = Object.values(this._memory.interactive_approval_events || {}).map((item) => ({ ...item }));
+      if (workflowId) {
+        rows = rows.filter((item) => normalizeString(item.workflow_id) === workflowId);
+      }
+      if (actor) {
+        rows = rows.filter((item) => normalizeString(item.actor) === actor);
+      }
+      if (action) {
+        rows = rows.filter((item) => normalizeString(item.action) === action);
+      }
+      if (eventType) {
+        rows = rows.filter((item) => normalizeString(item.event_type) === eventType);
+      }
+      if (auditFile) {
+        rows = rows.filter((item) => normalizeString(item.audit_file) === auditFile);
+      }
+      if (blockedFilter !== null) {
+        rows = rows.filter((item) => normalizeBooleanValue(item.blocked, false) === blockedFilter);
+      }
+      rows.sort((left, right) => (Date.parse(right.event_timestamp || '') || 0) - (Date.parse(left.event_timestamp || '') || 0));
+      if (limit > 0) {
+        rows = rows.slice(0, limit);
+      }
+      return rows.map((row) => this._mapInteractiveApprovalEventProjectionRow(row)).filter(Boolean);
+    }
+
+    if (!await this.ensureReady()) {
+      return null;
+    }
+
+    let query = `
+      SELECT event_id, workflow_id, event_timestamp, event_type, action, actor, actor_role,
+             from_status, to_status, blocked, reason, audit_file, line_no, raw_json, source, indexed_at
+      FROM interactive_approval_event_projection
+    `;
+    const clauses = [];
+    const params = [];
+    if (workflowId) {
+      clauses.push('workflow_id = ?');
+      params.push(workflowId);
+    }
+    if (actor) {
+      clauses.push('actor = ?');
+      params.push(actor);
+    }
+    if (action) {
+      clauses.push('action = ?');
+      params.push(action);
+    }
+    if (eventType) {
+      clauses.push('event_type = ?');
+      params.push(eventType);
+    }
+    if (auditFile) {
+      clauses.push('audit_file = ?');
+      params.push(auditFile);
+    }
+    if (blockedFilter !== null) {
+      clauses.push('blocked = ?');
+      params.push(blockedFilter ? 1 : 0);
+    }
+    if (clauses.length > 0) {
+      query += ` WHERE ${clauses.join(' AND ')}`;
+    }
+    query += ' ORDER BY event_timestamp DESC, line_no DESC';
+    if (limit > 0) {
+      query += ' LIMIT ?';
+      params.push(limit);
+    }
+
+    const rows = this._db.prepare(query).all(...params);
+    return rows
+      .map((row) => this._mapInteractiveApprovalEventProjectionRow(row))
+      .filter(Boolean);
+  }
+
   async upsertTimelineSnapshotIndex(records = [], options = {}) {
     const source = normalizeString(options.source) || 'file.timeline.index';
     const nowIso = this.now();

package/lib/state/state-migration-manager.js

@@ -723,8 +723,12 @@ async function runStateDoctor(options = {}, dependencies = {}) {
       syncStatus = 'source-parse-error';
     } else if (sourceCount === 0 && targetCount === 0) {
       syncStatus = 'empty';
+    } else if (sourceCount === 0 && targetCount > 0) {
+      syncStatus = 'sqlite-only';
     } else if (targetCount < sourceCount) {
       syncStatus = 'pending-migration';
+    } else if (targetCount > sourceCount) {
+      syncStatus = 'sqlite-ahead';
     }
     return {
       id: component.id,
@@ -750,22 +754,37 @@ async function runStateDoctor(options = {}, dependencies = {}) {
   if (checks.some((item) => item.sync_status === 'source-parse-error')) {
     blocking.push('source-parse-error');
   }
+  if (checks.some((item) => item.sync_status === 'sqlite-ahead')) {
+    blocking.push('sqlite-ahead');
+  }
+  if (checks.some((item) => item.sync_status === 'sqlite-only')) {
+    blocking.push('sqlite-only');
+  }
 
   const alerts = checks
     .filter((item) => item.sync_status === 'pending-migration')
     .map((item) => `pending migration: ${item.id}`);
+  alerts.push(...checks
+    .filter((item) => item.sync_status === 'missing-source')
+    .map((item) => `missing source: ${item.id}`));
 
   if (runtime.timeline && runtime.timeline.consistency && runtime.timeline.consistency.status === 'pending-sync') {
     alerts.push('runtime timeline index pending-sync');
   }
   if (runtime.timeline && runtime.timeline.consistency && runtime.timeline.consistency.status === 'sqlite-ahead') {
-    alerts.push('runtime timeline index sqlite-ahead');
+    blocking.push('runtime timeline index sqlite-ahead');
+  }
+  if (runtime.timeline && runtime.timeline.consistency && runtime.timeline.consistency.status === 'sqlite-only') {
+    blocking.push('runtime timeline index sqlite-only');
   }
   if (runtime.scene_session && runtime.scene_session.consistency && runtime.scene_session.consistency.status === 'pending-sync') {
     alerts.push('runtime scene-session index pending-sync');
   }
   if (runtime.scene_session && runtime.scene_session.consistency && runtime.scene_session.consistency.status === 'sqlite-ahead') {
-    alerts.push('runtime scene-session index sqlite-ahead');
+    blocking.push('runtime scene-session index sqlite-ahead');
+  }
+  if (runtime.scene_session && runtime.scene_session.consistency && runtime.scene_session.consistency.status === 'sqlite-only') {
+    blocking.push('runtime scene-session index sqlite-only');
   }
 
   const summary = summarizeDoctorChecks(checks, alerts, blocking);
@@ -792,7 +811,9 @@ function summarizeDoctorChecks(checks = [], alerts = [], blocking = []) {
   const pendingComponents = normalizedChecks.filter((item) => item.sync_status === 'pending-migration').length;
   const syncedComponents = normalizedChecks.filter((item) => item.sync_status === 'synced').length;
   const sqliteOnlyComponents = normalizedChecks.filter((item) => item.sync_status === 'sqlite-only').length;
+  const sqliteAheadComponents = normalizedChecks.filter((item) => item.sync_status === 'sqlite-ahead').length;
   const missingSourceComponents = normalizedChecks.filter((item) => item.sync_status === 'missing-source').length;
+  const parseErrorComponents = normalizedChecks.filter((item) => item.sync_status === 'source-parse-error').length;
   const driftRecords = normalizedChecks.reduce((sum, item) => {
     const source = normalizeCount(item.source_record_count);
     const target = normalizeCount(item.sqlite_record_count);
@@ -804,7 +825,9 @@ function summarizeDoctorChecks(checks = [], alerts = [], blocking = []) {
     synced_components: syncedComponents,
     pending_components: pendingComponents,
     sqlite_only_components: sqliteOnlyComponents,
+    sqlite_ahead_components: sqliteAheadComponents,
     missing_source_components: missingSourceComponents,
+    source_parse_error_components: parseErrorComponents,
     total_source_records: sourceRecords,
     total_sqlite_records: sqliteRecords,
     total_record_drift: driftRecords,
@@ -878,6 +901,7 @@ async function collectRuntimeDiagnostics(dependencies = {}) {
     const sceneIndex = await sessionStore.getSceneIndexDiagnostics();
     runtime.scene_session = {
       read_preference: normalizeString(sceneIndex.read_preference) || 'file',
+      read_source: normalizeString(sceneIndex.read_source) || 'file',
       consistency: {
         status: normalizeString(sceneIndex.status) || 'unknown',
         file_index_count: normalizeCount(sceneIndex.file_scene_count),
@@ -889,6 +913,7 @@ async function collectRuntimeDiagnostics(dependencies = {}) {
   } catch (_error) {
     runtime.scene_session = {
       read_preference: 'file',
+      read_source: 'unavailable',
       consistency: {
         status: 'unavailable',
         file_index_count: 0,

package/lib/state/state-storage-policy.js

@@ -0,0 +1,179 @@
+'use strict';
+
+const DEFAULT_STATE_STORAGE_POLICY = Object.freeze({
+  schema_version: '1.0',
+  strategy: 'selective-sqlite-advancement',
+  tiers: {
+    'file-source': {
+      description: 'Canonical file-backed storage for low-cardinality config, raw evidence, audit streams, and recovery-oriented payloads.'
+    },
+    'sqlite-index': {
+      description: 'SQLite index/registry layer for file-backed resources that need high-frequency filtering, sorting, and cross-run aggregation.'
+    },
+    'derived-sqlite-projection': {
+      description: 'Disposable SQLite projection rebuilt from canonical files for append-only streams with query pressure.'
+    }
+  },
+  admission: {
+    required_signals: [
+      'cross-run or cross-session query pressure is proven',
+      'file scans are materially weaker than indexed filtering/sorting',
+      'sqlite content remains rebuildable from a canonical source',
+      'operator diagnostics and reconcile path are defined before rollout'
+    ],
+    deny_if_any: [
+      'resource is the only copy of raw audit or evidence payload',
+      'resource is low-cardinality personal workspace or preference state',
+      'human-readable diff and manual recovery are more valuable than query speed',
+      'migration would introduce silent source-of-truth cutover'
+    ],
+    future_candidate_checklist: [
+      'identify canonical source path or stream',
+      'document expected query patterns and consumers',
+      'define rebuild and reconcile semantics',
+      'define release gate or audit behavior for drift states',
+      'document why existing file-only storage is insufficient'
+    ]
+  },
+  component_scope: [
+    {
+      component_id: 'collab.agent-registry',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/config/agent-registry.json',
+      sqlite_tables: ['agent_runtime_registry'],
+      rationale: 'Registry-style lookup with repeated status and capability queries.'
+    },
+    {
+      component_id: 'runtime.timeline-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/timeline/index.json',
+      sqlite_tables: ['timeline_snapshot_registry'],
+      rationale: 'Timeline index benefits from filtered and cross-session reads while file snapshots remain recoverable source artifacts.'
+    },
+    {
+      component_id: 'runtime.scene-session-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/session-governance/scene-index.json',
+      sqlite_tables: ['scene_session_cycle_registry'],
+      rationale: 'Scene/session lookups have query pressure and consistency checks but still rely on file session payloads.'
+    },
+    {
+      component_id: 'errorbook.entry-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/errorbook/index.json',
+      sqlite_tables: ['errorbook_entry_index_registry'],
+      rationale: 'Promoted errorbook registry queries benefit from indexed status and quality filtering.'
+    },
+    {
+      component_id: 'errorbook.incident-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/errorbook/staging/index.json',
+      sqlite_tables: ['errorbook_incident_index_registry'],
+      rationale: 'Incident staging state requires queryable triage views without replacing raw incident artifacts.'
+    },
+    {
+      component_id: 'governance.spec-scene-overrides',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/spec-governance/spec-scene-overrides.json',
+      sqlite_tables: ['governance_spec_scene_override_registry'],
+      rationale: 'Override lookups are registry-like and join naturally with other governance indexes.'
+    },
+    {
+      component_id: 'governance.scene-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/spec-governance/scene-index.json',
+      sqlite_tables: ['governance_scene_index_registry'],
+      rationale: 'Scene governance summaries are better served by indexed counts and status filters.'
+    },
+    {
+      component_id: 'release.evidence-runs-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/reports/release-evidence/handoff-runs.json',
+      sqlite_tables: ['release_evidence_run_registry'],
+      rationale: 'Release evidence run summaries need fast historical querying while release assets remain file-backed.'
+    },
+    {
+      component_id: 'release.gate-history-index',
+      tier: 'sqlite-index',
+      canonical_source: 'file',
+      source_path: '.sce/reports/release-evidence/release-gate-history.json',
+      sqlite_tables: ['release_gate_history_registry'],
+      rationale: 'Gate history is registry-shaped and queried by tag, pass/fail, and drift metrics.'
+    }
+  ],
+  resource_rules: [
+    {
+      rule_id: 'workspace-personal-state',
+      tier: 'file-source',
+      explicit_paths: ['~/.sce/workspace-state.json'],
+      derived_projection_allowed: false,
+      source_replacement_allowed: false,
+      rationale: 'Personal workspace selection and preferences are low-cardinality, atomic, and not worth migrating into SQLite.'
+    },
+    {
+      rule_id: 'append-only-report-streams',
+      tier: 'file-source',
+      path_patterns: ['.sce/reports/**/*.jsonl'],
+      derived_projection_allowed: true,
+      source_replacement_allowed: false,
+      rationale: 'Raw governance and evidence streams must stay append-only files; projection is allowed only for query acceleration.'
+    },
+    {
+      rule_id: 'append-only-audit-streams',
+      tier: 'file-source',
+      path_patterns: ['.sce/audit/**/*.jsonl'],
+      derived_projection_allowed: true,
+      source_replacement_allowed: false,
+      rationale: 'Audit streams remain canonical evidence and should never become SQLite-only write paths.'
+    },
+    {
+      rule_id: 'timeline-snapshot-payloads',
+      tier: 'file-source',
+      path_patterns: ['.sce/timeline/snapshots/**'],
+      derived_projection_allowed: false,
+      source_replacement_allowed: false,
+      rationale: 'Timeline snapshots are recovery-oriented payload artifacts, not registry data.'
+    },
+    {
+      rule_id: 'session-payload-artifacts',
+      tier: 'file-source',
+      path_patterns: ['.sce/session-governance/sessions/**'],
+      derived_projection_allowed: false,
+      source_replacement_allowed: false,
+      rationale: 'Session payloads must stay file-backed for recovery, archive, and manual debugging.'
+    },
+    {
+      rule_id: 'release-evidence-assets',
+      tier: 'file-source',
+      path_patterns: [
+        '.sce/reports/release-evidence/**/*.json',
+        '.sce/reports/release-evidence/**/*.md',
+        '.sce/reports/release-evidence/**/*.jsonl',
+        '.sce/reports/release-evidence/**/*.lines'
+      ],
+      derived_projection_allowed: true,
+      source_replacement_allowed: false,
+      rationale: 'Release evidence assets remain portable files even when selected summary indexes are mirrored into SQLite.'
+    }
+  ]
+});
+
+const REQUIRED_COMPONENT_IDS = Object.freeze(DEFAULT_STATE_STORAGE_POLICY.component_scope.map((item) => item.component_id));
+
+function cloneStateStoragePolicyDefaults() {
+  return JSON.parse(JSON.stringify(DEFAULT_STATE_STORAGE_POLICY));
+}
+
+module.exports = {
+  DEFAULT_STATE_STORAGE_POLICY,
+  REQUIRED_COMPONENT_IDS,
+  cloneStateStoragePolicyDefaults
+};

package/lib/upgrade/migration-engine.js

@@ -300,10 +300,10 @@ class MigrationEngine {
     
     try {
       // Check if .sce/ directory exists
-      const kiroPath = path.join(projectPath, '.sce');
-      const kiroExists = await pathExists(kiroPath);
+      const scePath = path.join(projectPath, '.sce');
+      const sceExists = await pathExists(scePath);
       
-      if (!kiroExists) {
+      if (!sceExists) {
         errors.push('.sce/ directory not found');
         return { success: false, errors, warnings };
       }
@@ -320,7 +320,7 @@ class MigrationEngine {
       const requiredDirs = ['specs', 'steering', 'tools', 'backups'];
       
       for (const dir of requiredDirs) {
-        const dirPath = path.join(kiroPath, dir);
+        const dirPath = path.join(scePath, dir);
         const exists = await pathExists(dirPath);
         
         if (!exists) {
@@ -337,7 +337,7 @@ class MigrationEngine {
       ];
       
       for (const file of requiredSteeringFiles) {
-        const filePath = path.join(kiroPath, file);
+        const filePath = path.join(scePath, file);
         const exists = await pathExists(filePath);
         
         if (!exists) {

package/lib/upgrade/migrations/1.0.0-to-1.1.0.js

@@ -26,10 +26,10 @@ module.exports = {
     const changes = [];
     
     try {
-      const kiroPath = path.join(projectPath, '.sce');
+      const scePath = path.join(projectPath, '.sce');
       
       // 1. Ensure backups/ directory exists
-      const backupsPath = path.join(kiroPath, 'backups');
+      const backupsPath = path.join(scePath, 'backups');
       const backupsExists = await pathExists(backupsPath);
       
       if (!backupsExists) {
@@ -39,7 +39,7 @@ module.exports = {
       
       // 2. Ensure version.json has correct structure
       // (This is handled by VersionManager, but we verify it here)
-      const versionPath = path.join(kiroPath, 'version.json');
+      const versionPath = path.join(scePath, 'version.json');
       const versionExists = await pathExists(versionPath);
       
       if (versionExists) {

package/lib/utils/tool-detector.js

@@ -64,21 +64,21 @@ async function detectKiroIDE(projectPath) {
   let confidence = 'low';
 
   // Check for .sce directory
-  const kiroDir = path.join(projectPath, '.sce');
-  if (await fs.pathExists(kiroDir)) {
+  const sceDir = path.join(projectPath, '.sce');
+  if (await fs.pathExists(sceDir)) {
     indicators.push('.sce directory exists');
     detected = true;
     confidence = 'medium';
   }
 
   // Check for SCE-specific files
-  const kiroFiles = [
+  const sceFiles = [
     '.sce/steering',
     '.sce/specs',
     '.sce/tools'
   ];
 
-  for (const file of kiroFiles) {
+  for (const file of sceFiles) {
     const filePath = path.join(projectPath, file);
     if (await fs.pathExists(filePath)) {
       indicators.push(`${file} exists`);

package/lib/utils/validation.js

@@ -21,11 +21,11 @@ async function validateProjectStructure(projectPath) {
   const warnings = [];
   
   try {
-    const kiroPath = path.join(projectPath, '.sce');
+    const scePath = path.join(projectPath, '.sce');
     
     // Check if .sce/ directory exists
-    const kiroExists = await pathExists(kiroPath);
-    if (!kiroExists) {
+    const sceExists = await pathExists(scePath);
+    if (!sceExists) {
       errors.push('.sce/ directory not found');
       return { success: false, errors, warnings };
     }
@@ -39,7 +39,7 @@ async function validateProjectStructure(projectPath) {
     ];
     
     for (const dir of requiredDirs) {
-      const dirPath = path.join(kiroPath, dir.path);
+      const dirPath = path.join(scePath, dir.path);
       const exists = await pathExists(dirPath);
       
       if (!exists) {
@@ -60,7 +60,7 @@ async function validateProjectStructure(projectPath) {
     ];
     
     for (const file of requiredSteeringFiles) {
-      const filePath = path.join(kiroPath, file.path);
+      const filePath = path.join(scePath, file.path);
       const exists = await pathExists(filePath);
       
       if (!exists) {
@@ -78,7 +78,7 @@ async function validateProjectStructure(projectPath) {
     ];
     
     for (const file of requiredToolFiles) {
-      const filePath = path.join(kiroPath, file.path);
+      const filePath = path.join(scePath, file.path);
       const exists = await pathExists(filePath);
       
       if (!exists) {