moflo 4.8.21 → 4.8.22

package/src/@claude-flow/shared/dist/events/event-store.js

@@ -44,22 +44,11 @@ export class EventStore extends EventEmitter {
     async initialize() {
         if (this.initialized)
             return;
-        // Load sql.js WASM — prefer local node_modules copy over remote URL
+        // Load sql.js WASM
         this.SQL = await initSqlJs({
             locateFile: this.config.wasmPath
                 ? () => this.config.wasmPath
-                : (file) => {
-                    // Try to resolve from node_modules first (works in Node.js)
-                    try {
-                        const sqlJsDir = require.resolve('sql.js');
-                        const { dirname, join } = require('node:path');
-                        const localPath = join(dirname(sqlJsDir), file);
-                        if (require('node:fs').existsSync(localPath))
-                            return localPath;
-                    }
-                    catch { /* fallback below */ }
-                    return `https://sql.js.org/dist/${file}`;
-                },
+                : (file) => `https://sql.js.org/dist/${file}`,
         });
         // Load existing database if exists
         if (this.config.databasePath !== ':memory:' && existsSync(this.config.databasePath)) {
@@ -121,11 +110,11 @@ export class EventStore extends EventEmitter {
         // Set version on event
         event.version = nextVersion;
         // Insert event
-        const stmt = `
-      INSERT INTO events (
-        id, type, aggregate_id, aggregate_type, version, timestamp,
-        source, payload, metadata, causation_id, correlation_id
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+        const stmt = `
+      INSERT INTO events (
+        id, type, aggregate_id, aggregate_type, version, timestamp,
+        source, payload, metadata, causation_id, correlation_id
+      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
     `;
         this.db.run(stmt, [
             event.id,
@@ -261,10 +250,10 @@ export class EventStore extends EventEmitter {
      */
     async saveSnapshot(snapshot) {
         this.ensureInitialized();
-        const stmt = `
-      INSERT OR REPLACE INTO snapshots (
-        aggregate_id, aggregate_type, version, state, timestamp
-      ) VALUES (?, ?, ?, ?, ?)
+        const stmt = `
+      INSERT OR REPLACE INTO snapshots (
+        aggregate_id, aggregate_type, version, state, timestamp
+      ) VALUES (?, ?, ?, ?, ?)
     `;
         this.db.run(stmt, [
             snapshot.aggregateId,
@@ -281,9 +270,7 @@ export class EventStore extends EventEmitter {
     async getSnapshot(aggregateId) {
         this.ensureInitialized();
         const stmt = this.db.prepare('SELECT * FROM snapshots WHERE aggregate_id = ? ORDER BY version DESC LIMIT 1');
-        stmt.bind([aggregateId]);
-        const hasRow = stmt.step();
-        const row = hasRow ? stmt.getAsObject() : null;
+        const row = stmt.getAsObject([aggregateId]);
         stmt.free();
         if (!row || Object.keys(row).length === 0) {
             return null;
@@ -303,7 +290,6 @@ export class EventStore extends EventEmitter {
         this.ensureInitialized();
         // Total events
         const totalStmt = this.db.prepare('SELECT COUNT(*) as count FROM events');
-        totalStmt.step();
         const totalRow = totalStmt.getAsObject();
         totalStmt.free();
         const totalEvents = totalRow.count || 0;
@@ -325,12 +311,10 @@ export class EventStore extends EventEmitter {
         aggStmt.free();
         // Timestamp range
         const rangeStmt = this.db.prepare('SELECT MIN(timestamp) as oldest, MAX(timestamp) as newest FROM events');
-        rangeStmt.step();
         const rangeRow = rangeStmt.getAsObject();
         rangeStmt.free();
         // Snapshot count
         const snapshotStmt = this.db.prepare('SELECT COUNT(*) as count FROM snapshots');
-        snapshotStmt.step();
         const snapshotRow = snapshotStmt.getAsObject();
         snapshotStmt.free();
         return {
@@ -362,20 +346,20 @@ export class EventStore extends EventEmitter {
         if (!this.db)
             return;
         // Events table
-        this.db.run(`
-      CREATE TABLE IF NOT EXISTS events (
-        id TEXT PRIMARY KEY,
-        type TEXT NOT NULL,
-        aggregate_id TEXT NOT NULL,
-        aggregate_type TEXT NOT NULL,
-        version INTEGER NOT NULL,
-        timestamp INTEGER NOT NULL,
-        source TEXT NOT NULL,
-        payload TEXT NOT NULL,
-        metadata TEXT,
-        causation_id TEXT,
-        correlation_id TEXT
-      )
+        this.db.run(`
+      CREATE TABLE IF NOT EXISTS events (
+        id TEXT PRIMARY KEY,
+        type TEXT NOT NULL,
+        aggregate_id TEXT NOT NULL,
+        aggregate_type TEXT NOT NULL,
+        version INTEGER NOT NULL,
+        timestamp INTEGER NOT NULL,
+        source TEXT NOT NULL,
+        payload TEXT NOT NULL,
+        metadata TEXT,
+        causation_id TEXT,
+        correlation_id TEXT
+      )
     `);
         // Indexes for performance
         this.db.run('CREATE INDEX IF NOT EXISTS idx_aggregate_id ON events(aggregate_id)');
@@ -385,14 +369,14 @@ export class EventStore extends EventEmitter {
         this.db.run('CREATE INDEX IF NOT EXISTS idx_version ON events(version)');
         this.db.run('CREATE UNIQUE INDEX IF NOT EXISTS idx_aggregate_version ON events(aggregate_id, version)');
         // Snapshots table
-        this.db.run(`
-      CREATE TABLE IF NOT EXISTS snapshots (
-        aggregate_id TEXT PRIMARY KEY,
-        aggregate_type TEXT NOT NULL,
-        version INTEGER NOT NULL,
-        state TEXT NOT NULL,
-        timestamp INTEGER NOT NULL
-      )
+        this.db.run(`
+      CREATE TABLE IF NOT EXISTS snapshots (
+        aggregate_id TEXT PRIMARY KEY,
+        aggregate_type TEXT NOT NULL,
+        version INTEGER NOT NULL,
+        state TEXT NOT NULL,
+        timestamp INTEGER NOT NULL
+      )
     `);
         if (this.config.verbose) {
             console.log('[EventStore] Schema created successfully');

package/src/@claude-flow/shared/dist/events/event-store.test.js

@@ -112,18 +112,14 @@ describe('EventStore', () => {
             expect(events).toHaveLength(3);
         });
         it('should replay from specific version', async () => {
-            // Create multiple events on the same aggregate so versions increment past 2
-            await eventStore.append(createAgentSpawnedEvent('agent-1', 'coder', 'core', [])); // v1
-            await eventStore.append(createAgentStartedEvent('agent-1')); // v2
-            await eventStore.append(createTaskCreatedEvent('task-1', 'implementation', 'Task', 'Desc', 'high', [])); // v1
+            await eventStore.append(createAgentSpawnedEvent('agent-1', 'coder', 'core', []));
+            await eventStore.append(createAgentStartedEvent('agent-1'));
+            await eventStore.append(createTaskCreatedEvent('task-1', 'implementation', 'Task', 'Desc', 'high', []));
             const events = [];
             for await (const event of eventStore.replay(2)) {
                 events.push(event);
             }
-            // replay(2) returns events where per-aggregate version >= 2
-            // Only agent-1's second event (v2) qualifies
-            expect(events.length).toBeGreaterThanOrEqual(1);
-            expect(events[0].version).toBeGreaterThanOrEqual(2);
+            expect(events.length).toBeGreaterThanOrEqual(2);
         });
     });
     describe('Snapshots', () => {

package/src/@claude-flow/shared/dist/hooks/executor.js

@@ -205,8 +205,6 @@ export class HookExecutor {
         const results = [];
         let currentContext = { ...initialContext };
         let totalExecutionTime = 0;
-        let totalHooksExecuted = 0;
-        let totalHooksFailed = 0;
         let aborted = false;
         for (const event of events) {
             if (aborted) {
@@ -215,8 +213,6 @@ export class HookExecutor {
             const result = await this.execute(event, currentContext, options);
             results.push(...result.results);
             totalExecutionTime += result.totalExecutionTime;
-            totalHooksExecuted += result.hooksExecuted;
-            totalHooksFailed += result.hooksFailed;
             // Merge context for next event
             if (result.finalContext) {
                 currentContext = { ...currentContext, ...result.finalContext };
@@ -226,12 +222,13 @@ export class HookExecutor {
                 break;
             }
         }
+        const hooksFailed = results.filter(r => !r.success).length;
         return {
-            success: totalHooksFailed === 0 && !aborted,
+            success: hooksFailed === 0 && !aborted,
             results: options.collectResults ? results : [],
             totalExecutionTime,
-            hooksExecuted: totalHooksExecuted,
-            hooksFailed: totalHooksFailed,
+            hooksExecuted: results.length,
+            hooksFailed,
             aborted,
             finalContext: currentContext,
         };

package/src/@claude-flow/shared/dist/hooks/safety/file-organization.js

@@ -142,7 +142,7 @@ export class FileOrganizationHook {
             blocked = true;
             blockReason = `Source files should not be written to root folder. Suggested: ${fileTypeInfo.directories[0]}`;
             suggestedDirectory = fileTypeInfo.directories[0];
-            suggestedPath = path.join(suggestedDirectory, fileName).replace(/\\/g, '/');
+            suggestedPath = path.join(suggestedDirectory, fileName);
             issues.push({
                 type: 'root-write',
                 severity: 'error',

package/src/@claude-flow/shared/dist/hooks/safety/git-commit.js

@@ -401,9 +401,9 @@ export class GitCommitHook {
      */
     formatForGit(message) {
         // Escape for heredoc usage
-        return `$(cat <<'EOF'
-${message}
-EOF
+        return `$(cat <<'EOF'
+${message}
+EOF
 )`;
     }
     /**

package/src/@claude-flow/shared/dist/hooks/verify-exports.test.js

@@ -22,8 +22,8 @@ describe('Hooks Module Exports', () => {
     it('should export types (type-only imports)', () => {
         expect(true).toBe(true);
     });
-    it('should create instances from exported factories', async () => {
-        const { createHookRegistry, createHookExecutor } = await import('./index.js');
+    it('should create instances from exported factories', () => {
+        const { createHookRegistry, createHookExecutor } = require('./index.js');
         const registry = createHookRegistry();
         expect(registry).toBeDefined();
         expect(typeof registry.register).toBe('function');
@@ -32,8 +32,8 @@ describe('Hooks Module Exports', () => {
         expect(executor).toBeDefined();
         expect(typeof executor.execute).toBe('function');
     });
-    it('should have all 26 hook events defined', async () => {
-        const { HookEvent } = await import('./index.js');
+    it('should have all 26 hook events defined', () => {
+        const { HookEvent } = require('./index.js');
         const expectedEvents = [
             'PreToolUse',
             'PostToolUse',
@@ -68,8 +68,8 @@ describe('Hooks Module Exports', () => {
             expect(HookEvent[event]).toBeDefined();
         }
     });
-    it('should have all 5 priority levels defined', async () => {
-        const { HookPriority } = await import('./index.js');
+    it('should have all 5 priority levels defined', () => {
+        const { HookPriority } = require('./index.js');
         expect(HookPriority.Critical).toBe(1000);
         expect(HookPriority.High).toBe(500);
         expect(HookPriority.Normal).toBe(0);

package/src/@claude-flow/shared/dist/utils/secure-logger.js

@@ -191,7 +191,7 @@ export function createSecureLogger(prefix, config) {
 /**
  * Default logger instance
  */
-export const logger = createSecureLogger('moflo');
+export const logger = createSecureLogger('claude-flow');
 /**
  * Sanitize an error for safe logging/display
  */