@lumenflow/cli 2.7.0 → 2.9.0

package/dist/wu-done-auto-cleanup.js

@@ -0,0 +1,139 @@
+/**
+ * WU-1366: Auto cleanup after wu:done success
+ *
+ * Provides functions to run state cleanup automatically after successful wu:done.
+ * Cleanup is non-fatal: errors are logged but do not block completion.
+ *
+ * The cleanup.trigger config option controls when cleanup runs:
+ * - 'on_done': Run after wu:done success (default)
+ * - 'on_init': Run during lumenflow init
+ * - 'manual': Only run via pnpm state:cleanup
+ *
+ * @see {@link packages/@lumenflow/core/src/state-cleanup-core.ts} - Core cleanup orchestration
+ * @see {@link packages/@lumenflow/core/src/lumenflow-config-schema.ts} - CleanupConfigSchema
+ */
+import { getConfig } from '@lumenflow/core/dist/lumenflow-config.js';
+import { cleanupState } from '@lumenflow/core/dist/state-cleanup-core.js';
+import { cleanupSignals } from '@lumenflow/memory/dist/signal-cleanup-core.js';
+import { cleanupMemory } from '@lumenflow/memory/dist/mem-cleanup-core.js';
+import { archiveWuEvents } from '@lumenflow/core/dist/wu-events-cleanup.js';
+import fg from 'fast-glob';
+import { readFile } from 'node:fs/promises';
+import { parse as parseYaml } from 'yaml';
+import path from 'node:path';
+import { LOG_PREFIX, EMOJI } from '@lumenflow/core/dist/wu-constants.js';
+/**
+ * Active WU statuses that should protect signals
+ */
+const ACTIVE_WU_STATUSES = ['in_progress', 'blocked'];
+/**
+ * Get active WU IDs (in_progress or blocked) by scanning WU YAML files.
+ *
+ * @param baseDir - Base directory
+ * @returns Set of active WU IDs
+ */
+async function getActiveWuIds(baseDir) {
+    const activeIds = new Set();
+    try {
+        const config = getConfig({ projectRoot: baseDir });
+        const wuDir = path.join(baseDir, config.directories.wuDir);
+        // Find all WU YAML files
+        const wuFiles = await fg('WU-*.yaml', { cwd: wuDir });
+        for (const file of wuFiles) {
+            try {
+                const filePath = path.join(wuDir, file);
+                const content = await readFile(filePath, 'utf-8');
+                const wu = parseYaml(content);
+                if (wu.id && wu.status && ACTIVE_WU_STATUSES.includes(wu.status)) {
+                    activeIds.add(wu.id);
+                }
+            }
+            catch {
+                // Skip files that fail to parse
+                continue;
+            }
+        }
+    }
+    catch {
+        // If we can't read WU files, return empty set (safer to remove nothing)
+    }
+    return activeIds;
+}
+/**
+ * Check if auto cleanup should run based on config.
+ *
+ * @returns true if cleanup.trigger is 'on_done' or not set (default)
+ */
+export function shouldRunAutoCleanup() {
+    try {
+        const config = getConfig();
+        const trigger = config.cleanup?.trigger;
+        // Default to 'on_done' if not set
+        if (!trigger) {
+            return true;
+        }
+        return trigger === 'on_done';
+    }
+    catch {
+        // If config can't be loaded, default to running cleanup
+        return true;
+    }
+}
+/**
+ * Run state cleanup automatically after wu:done success.
+ *
+ * This function is non-fatal: errors are logged as warnings but do not throw.
+ * Cleanup respects the config.cleanup.trigger setting.
+ *
+ * @param baseDir - Base directory for cleanup operations
+ * @returns Promise that resolves when cleanup completes (or is skipped)
+ */
+export async function runAutoCleanupAfterDone(baseDir) {
+    // Check if cleanup should run
+    if (!shouldRunAutoCleanup()) {
+        return;
+    }
+    try {
+        console.log(`${LOG_PREFIX.DONE} ${EMOJI.INFO} Running auto state cleanup...`);
+        const result = await cleanupState(baseDir, {
+            dryRun: false,
+            // Inject real cleanup functions
+            cleanupSignals: async (dir, opts) => cleanupSignals(dir, {
+                dryRun: opts.dryRun,
+                getActiveWuIds: () => getActiveWuIds(dir),
+            }),
+            cleanupMemory: async (dir, opts) => cleanupMemory(dir, {
+                dryRun: opts.dryRun,
+            }),
+            archiveEvents: async (dir, opts) => archiveWuEvents(dir, {
+                dryRun: opts.dryRun,
+            }),
+        });
+        if (result.success) {
+            const typesStr = result.summary.typesExecuted.join(', ');
+            console.log(`${LOG_PREFIX.DONE} ${EMOJI.SUCCESS} State cleanup complete: ` +
+                `${formatBytes(result.summary.totalBytesFreed)} freed [${typesStr}]`);
+        }
+        else {
+            // Partial success - some cleanups failed
+            const errorMsgs = result.errors.map((e) => `${e.type}: ${e.message}`).join(', ');
+            console.warn(`${LOG_PREFIX.DONE} ${EMOJI.WARNING} State cleanup partial: ${errorMsgs}`);
+        }
+    }
+    catch (err) {
+        // Non-fatal: log warning but don't throw
+        const message = err instanceof Error ? err.message : String(err);
+        console.warn(`${LOG_PREFIX.DONE} ${EMOJI.WARNING} Could not run auto state cleanup: ${message}`);
+    }
+}
+/**
+ * Format bytes as human-readable string
+ */
+function formatBytes(bytes) {
+    const BYTES_PER_KB = 1024;
+    if (bytes < BYTES_PER_KB) {
+        return `${bytes} B`;
+    }
+    const kb = (bytes / BYTES_PER_KB).toFixed(1);
+    return `${kb} KB`;
+}

package/dist/wu-done.js

@@ -95,6 +95,8 @@ import { SpawnStatus } from '@lumenflow/core/dist/spawn-registry-schema.js';
 // WU-2022: Feature accessibility validation (blocking)
 import { validateExposure, validateFeatureAccessibility, } from '@lumenflow/core/dist/wu-validation.js';
 import { ensureCleanWorktree } from './wu-done-check.js';
+// WU-1366: Auto cleanup after wu:done success
+import { runAutoCleanupAfterDone } from './wu-done-auto-cleanup.js';
 // WU-1588: Memory layer constants
 const MEMORY_SIGNAL_TYPES = {
     WU_COMPLETION: 'wu_completion',
@@ -1091,7 +1093,6 @@ function recordTransactionState(id, wuPath, stampPath, backlogPath, statusPath)
  * @param {string} backlogPath - Path to backlog.md (WU-1230)
  * @param {string} statusPath - Path to status.md (WU-1230)
  */
-// eslint-disable-next-line sonarjs/cognitive-complexity -- Pre-existing complexity, refactor tracked separately
 async function rollbackTransaction(txState, wuPath, stampPath, backlogPath, statusPath) {
     console.error(`\n${LOG_PREFIX.DONE} ${EMOJI.WARNING} ROLLING BACK TRANSACTION (WU-755 + WU-1230 + WU-1255 + WU-1280)...`);
     // WU-1280: ATOMIC ROLLBACK - Clean git state FIRST, then restore files
@@ -1272,7 +1273,6 @@ function runWUValidator(doc, id, allowTodo = false, worktreePath = null) {
  * @param {string|null} overrideReason - Reason for override
  * @returns {{valid: boolean, error: string|null, auditEntry: object|null}}
  */
-// eslint-disable-next-line sonarjs/cognitive-complexity -- Pre-existing complexity, refactor tracked separately
 async function checkOwnership(id, doc, worktreePath, overrideOwner = false, overrideReason = null) {
     // Missing worktree means WU was not claimed properly (unless escape hatch applies)
     if (!worktreePath || !existsSync(worktreePath)) {
@@ -2085,7 +2085,9 @@ async function main() {
                     if (lane)
                         releaseLaneLock(lane, { wuId: id });
                 }
-                catch { }
+                catch {
+                    // Intentionally ignore lock release errors during cleanup
+                }
                 process.exit(EXIT_CODES.SUCCESS);
             }
         }
@@ -2096,7 +2098,9 @@ async function main() {
                 if (lane)
                     releaseLaneLock(lane, { wuId: id });
             }
-            catch { }
+            catch {
+                // Intentionally ignore lock release errors during error handling
+            }
             // WU-1811: Check if cleanup is safe before removing worktree
             // If cleanupSafe is false (or undefined), preserve worktree for recovery
             if (err.cleanupSafe === false) {
@@ -2180,6 +2184,9 @@ async function main() {
     // WU-1983: Migration deployment nudge - only if supabase paths in code_paths
     const codePaths = docMain.code_paths || [];
     await printMigrationDeploymentNudge(codePaths, mainCheckoutPath);
+    // WU-1366: Auto state cleanup after successful completion
+    // Non-fatal: errors are logged but do not block completion
+    await runAutoCleanupAfterDone(mainCheckoutPath);
 }
 /**
  * WU-1983: Print migration deployment nudge when WU includes supabase changes.

package/dist/wu-edit.js

@@ -59,7 +59,6 @@ import { normalizeWUSchema } from '@lumenflow/core/dist/wu-schema-normalization.
 import { lintWUSpec, formatLintErrors } from '@lumenflow/core/dist/wu-lint.js';
 // WU-1329: Import path existence validators for strict validation
 import { validateCodePathsExistence, validateTestPathsExistence, } from '@lumenflow/core/dist/wu-preflight-validators.js';
-/* eslint-disable security/detect-object-injection */
 const PREFIX = LOG_PREFIX.EDIT;
 /**
  * WU-1039: Validate which edits are allowed on done WUs
@@ -271,13 +270,11 @@ const EDIT_OPTIONS = {
  * @param {string} newInitId - New initiative ID
  * @returns {Array<string>} Array of relative file paths that were modified
  */
-// eslint-disable-next-line sonarjs/cognitive-complexity -- Pre-existing complexity, refactor tracked separately
 function updateInitiativeWusArrays(worktreePath, wuId, oldInitId, newInitId) {
     const modifiedFiles = [];
     // Remove from old initiative if it exists and is different from new
     if (oldInitId && oldInitId !== newInitId) {
         const oldInitPath = join(worktreePath, INIT_PATHS.INITIATIVE(oldInitId));
-        // eslint-disable-next-line security/detect-non-literal-fs-filename
         if (existsSync(oldInitPath)) {
             try {
                 const oldInit = readInitiative(oldInitPath, oldInitId);
@@ -296,7 +293,6 @@ function updateInitiativeWusArrays(worktreePath, wuId, oldInitId, newInitId) {
     }
     // Add to new initiative
     const newInitPath = join(worktreePath, INIT_PATHS.INITIATIVE(newInitId));
-    // eslint-disable-next-line security/detect-non-literal-fs-filename
     if (existsSync(newInitPath)) {
         try {
             const newInit = readInitiative(newInitPath, newInitId);
@@ -333,7 +329,6 @@ function validateInitiativeFormat(initId) {
  */
 function validateInitiativeExists(initId) {
     const initPath = INIT_PATHS.INITIATIVE(initId);
-    // eslint-disable-next-line security/detect-non-literal-fs-filename
     if (!existsSync(initPath)) {
         die(`Initiative not found: ${initId}\n\nFile does not exist: ${initPath}`);
     }
@@ -464,11 +459,9 @@ function normalizeWUDates(wu) {
  */
 function validateWUEditable(id) {
     const wuPath = WU_PATHS.WU(id);
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool validates WU files
     if (!existsSync(wuPath)) {
         die(`WU ${id} not found at ${wuPath}\n\nEnsure the WU exists and you're in the repo root.`);
     }
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool validates WU files
     const content = readFileSync(wuPath, { encoding: FILE_SYSTEM.ENCODING });
     const wu = parseYAML(content);
     // WU-1929: Done WUs allow initiative/phase edits only (metadata reassignment)
@@ -505,7 +498,6 @@ function validateWUEditable(id) {
  * @param {string} id - WU ID (for error messages)
  */
 function validateWorktreeExists(worktreePath, id) {
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool validates worktree paths
     if (!existsSync(worktreePath)) {
         die(`Cannot edit WU ${id}: worktree path missing from disk.\n\n` +
             `Expected worktree at: ${worktreePath}\n\n` +
@@ -580,7 +572,6 @@ async function applyEditsInWorktree({ worktreePath, id, updatedWU }) {
     normalizeWUDates(updatedWU);
     // Emergency fix Session 2: Use centralized stringifyYAML helper
     const yamlContent = stringifyYAML(updatedWU);
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool writes WU files
     writeFileSync(wuPath, yamlContent, { encoding: FILE_SYSTEM.ENCODING });
     console.log(`${PREFIX} ✅ Updated ${id}.yaml in worktree`);
     // Format the file
@@ -660,11 +651,9 @@ export function mergeStringField(existing, newValue, shouldReplace) {
  */
 function loadSpecFile(specPath, originalWU) {
     const resolvedPath = resolve(specPath);
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool validates spec files
     if (!existsSync(resolvedPath)) {
         die(`Spec file not found: ${resolvedPath}`);
     }
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool validates spec files
     const specContent = readFileSync(resolvedPath, {
         encoding: FILE_SYSTEM.ENCODING,
     });
@@ -1040,7 +1029,6 @@ async function main() {
                     normalizeWUDates(normalizedWU);
                     // Emergency fix Session 2: Use centralized stringifyYAML helper
                     const yamlContent = stringifyYAML(normalizedWU);
-                    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool writes WU files
                     writeFileSync(wuPath, yamlContent, { encoding: FILE_SYSTEM.ENCODING });
                     console.log(`${PREFIX} ✅ Updated ${id}.yaml in micro-worktree`);
                     // WU-1929: Handle bidirectional initiative updates

package/dist/wu-preflight.js

@@ -27,7 +27,6 @@ import { PATTERNS, EXIT_CODES, LOG_PREFIX, EMOJI } from '@lumenflow/core/dist/wu
 import { defaultWorktreeFrom, WU_PATHS } from '@lumenflow/core/dist/wu-paths.js';
 import { readWURaw } from '@lumenflow/core/dist/wu-yaml.js';
 import { die } from '@lumenflow/core/dist/error-handler.js';
-/* eslint-disable security/detect-non-literal-fs-filename */
 /**
  * Detect worktree path from WU YAML or calculate from lane
  * @param {string} id - WU ID

package/dist/wu-prep.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console -- CLI command uses console for status output */
 /**
  * WU Prep Helper (WU-1223)
  *

package/dist/wu-proto.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console -- CLI tool requires console output */
 /**
  * WU Proto Helper (WU-1359)
  *

package/dist/wu-spawn.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console -- CLI tool requires console output */
 /**
  * WU Spawn Helper
  *
@@ -43,7 +42,6 @@ import { minimatch } from 'minimatch';
 // WU-2252: Import invariants loader for spawn output injection
 import { loadInvariants, INVARIANT_TYPES } from '@lumenflow/core/dist/invariants-runner.js';
 import { validateSpawnArgs, generateExecutionModeSection, generateThinkToolGuidance, recordSpawnToRegistry, formatSpawnRecordedMessage, } from '@lumenflow/core/dist/wu-spawn-helpers.js';
-// eslint-disable-next-line sonarjs/deprecation -- legacy factory used by CLI spawns
 import { SpawnStrategyFactory } from '@lumenflow/core/dist/spawn-strategy.js';
 import { getConfig } from '@lumenflow/core/dist/lumenflow-config.js';
 import { generateClientSkillsGuidance, generateSkillsSelectionSection, resolveClientConfig, } from '@lumenflow/core/dist/wu-spawn-skills.js';
@@ -1331,7 +1329,6 @@ async function main() {
         }
     }
     // Create strategy
-    // eslint-disable-next-line sonarjs/deprecation -- legacy factory used by CLI spawns
     const strategy = SpawnStrategyFactory.create(clientName);
     const clientContext = { name: clientName, config: resolveClientConfig(config, clientName) };
     if (clientName === 'codex-cli' || args.codex) {

package/dist/wu-unblock.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console -- CLI tool requires console output */
 /**
  * WU Unblock Helper
  *
@@ -55,7 +54,6 @@ function branchExists(branch) {
 function createWorktree(doc, worktreePath, branchName) {
     if (!worktreePath)
         die('Worktree path required to create a worktree');
-    // eslint-disable-next-line security/detect-non-literal-fs-filename -- CLI tool checks worktree
     if (existsSync(worktreePath)) {
         console.warn(`${PREFIX} Worktree ${worktreePath} already exists; skipping creation.`);
         return;

package/dist/wu-validate.js

@@ -112,7 +112,6 @@ function validateAllWUs({ strict = true } = {}) {
 /**
  * Main entry point
  */
-// eslint-disable-next-line sonarjs/cognitive-complexity -- Pre-existing complexity, refactor tracked separately
 async function main() {
     const args = createWUParser({
         name: 'wu-validate',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumenflow/cli",
-  "version": "2.7.0",
+  "version": "2.9.0",
   "description": "Command-line interface for LumenFlow workflow framework",
   "keywords": [
     "lumenflow",
@@ -99,6 +99,7 @@
     "lumenflow-init": "./dist/init.js",
     "lumenflow": "./dist/init.js",
     "lumenflow-doctor": "./dist/doctor.js",
+    "lumenflow-commands": "./dist/commands.js",
     "lumenflow-release": "./dist/release.js",
     "lumenflow-docs-sync": "./dist/docs-sync.js",
     "lumenflow-sync-templates": "./dist/sync-templates.js",
@@ -131,7 +132,8 @@
     "lane-health": "./dist/lane-health.js",
     "lane-suggest": "./dist/lane-suggest.js",
     "state-cleanup": "./dist/state-cleanup.js",
-    "state-doctor": "./dist/state-doctor.js"
+    "state-doctor": "./dist/state-doctor.js",
+    "lumenflow-integrate": "./dist/commands/integrate.js"
   },
   "files": [
     "dist",
@@ -149,11 +151,11 @@
     "pretty-ms": "^9.2.0",
     "simple-git": "^3.30.0",
     "yaml": "^2.8.2",
-    "@lumenflow/core": "2.7.0",
-    "@lumenflow/memory": "2.7.0",
-    "@lumenflow/metrics": "2.7.0",
-    "@lumenflow/initiatives": "2.7.0",
-    "@lumenflow/agent": "2.7.0"
+    "@lumenflow/core": "2.9.0",
+    "@lumenflow/metrics": "2.9.0",
+    "@lumenflow/memory": "2.9.0",
+    "@lumenflow/agent": "2.9.0",
+    "@lumenflow/initiatives": "2.9.0"
   },
   "devDependencies": {
     "@vitest/coverage-v8": "^4.0.17",