@mmmbuto/nexuscli 0.7.0 → 0.7.2

package/lib/server/services/cli-loader.js

@@ -47,6 +47,8 @@ class CliLoader {
    */
   async loadMessagesFromCLI({
     sessionId,
+    threadId,          // optional native thread id (e.g., Codex exec thread)
+    sessionPath,       // alias for compatibility
     engine = 'claude',
     workspacePath,
     limit = DEFAULT_LIMIT,
@@ -59,6 +61,7 @@ class CliLoader {
 
     const startedAt = Date.now();
     const normalizedEngine = this._normalizeEngine(engine);
+    const nativeId = threadId || sessionPath || sessionId;
 
     let result;
     switch (normalizedEngine) {
@@ -67,7 +70,7 @@ class CliLoader {
         break;
 
       case 'codex':
-        result = await this.loadCodexMessages({ sessionId, limit, before, mode });
+        result = await this.loadCodexMessages({ sessionId, nativeId, limit, before, mode });
         break;
 
       case 'gemini':
@@ -174,22 +177,27 @@ class CliLoader {
   // CODEX - Load from ~/.codex/sessions/<sessionId>.jsonl
   // ============================================================
 
-  async loadCodexMessages({ sessionId, limit, before, mode }) {
-    const sessionFile = path.join(this.codexPath, 'sessions', `${sessionId}.jsonl`);
+  async loadCodexMessages({ sessionId, nativeId, limit, before, mode }) {
+    const baseDir = path.join(this.codexPath, 'sessions');
+    let sessionFile = path.join(baseDir, `${nativeId || sessionId}.jsonl`);
 
-    // Codex may not persist sessions locally - check if file exists
+    // If flat file missing, search nested rollout-* files by threadId
     if (!fs.existsSync(sessionFile)) {
-      // This is expected - Codex exec mode doesn't save history locally
-      console.log(`[CliLoader] Codex session file not found (expected): ${sessionFile}`);
+      sessionFile = this.findCodexSessionFile(baseDir, nativeId || sessionId);
+    }
+
+    // Codex exec may not persist sessions; handle gracefully
+    if (!sessionFile || !fs.existsSync(sessionFile)) {
+      console.log(`[CliLoader] Codex session file not found (id=${nativeId || sessionId})`);
       return this._emptyResult();
     }
 
     const rawMessages = await this._parseJsonlFile(sessionFile);
 
-    // Filter and normalize
+    // Normalize then filter only chat messages
     const messages = rawMessages
-      .filter(entry => entry.role === 'user' || entry.role === 'assistant')
-      .map(entry => this._normalizeCodexEntry(entry));
+      .map(entry => this._normalizeCodexEntry(entry))
+      .filter(msg => msg && (msg.role === 'user' || msg.role === 'assistant'));
 
     return this._paginateMessages(messages, limit, before, mode);
   }
@@ -198,13 +206,34 @@ class CliLoader {
    * Normalize Codex session entry to message shape
    */
   _normalizeCodexEntry(entry) {
-    const role = entry.role || 'assistant';
-    const created_at = entry.timestamp ? new Date(entry.timestamp).getTime() : Date.now();
-
-    // Codex may store content as string or object
+    // Skip non-chat bookkeeping events
+    const skipTypes = ['session_meta', 'turn_context', 'event_msg', 'token_count'];
+    if (skipTypes.includes(entry.type)) return null;
+
+    const role =
+      entry.role ||
+      entry.payload?.role ||
+      (entry.payload?.type === 'message' ? entry.payload.role : null) ||
+      entry.message?.role ||
+      'assistant';
+
+    const created_at = entry.timestamp
+      ? new Date(entry.timestamp).getTime()
+      : (entry.payload?.timestamp ? new Date(entry.payload.timestamp).getTime() : Date.now());
+
+    // Codex may store content in multiple shapes
     let content = '';
     if (typeof entry.content === 'string') {
       content = entry.content;
+    } else if (typeof entry.payload?.content === 'string') {
+      content = entry.payload.content;
+    } else if (Array.isArray(entry.payload?.content)) {
+      content = entry.payload.content
+        .map(block => block.text || block.message || block.title || '')
+        .filter(Boolean)
+        .join('\n');
+    } else if (entry.payload?.text) {
+      content = entry.payload.text;
     } else if (entry.message) {
       content = typeof entry.message === 'string' ? entry.message : JSON.stringify(entry.message);
     }
@@ -222,6 +251,39 @@ class CliLoader {
     };
   }
 
+  /**
+   * Find Codex rollout file by threadId within YYYY/MM/DD directories
+   */
+  findCodexSessionFile(baseDir, threadId) {
+    if (!threadId || !fs.existsSync(baseDir)) return null;
+    try {
+      const years = fs.readdirSync(baseDir);
+      for (const year of years) {
+        const yearPath = path.join(baseDir, year);
+        if (!fs.statSync(yearPath).isDirectory()) continue;
+        const months = fs.readdirSync(yearPath);
+        for (const month of months) {
+          const monthPath = path.join(yearPath, month);
+          if (!fs.statSync(monthPath).isDirectory()) continue;
+          const days = fs.readdirSync(monthPath);
+          for (const day of days) {
+            const dayPath = path.join(monthPath, day);
+            if (!fs.statSync(dayPath).isDirectory()) continue;
+            const files = fs.readdirSync(dayPath);
+            for (const file of files) {
+              if (file.endsWith('.jsonl') && file.includes(threadId)) {
+                return path.join(dayPath, file);
+              }
+            }
+          }
+        }
+      }
+    } catch (err) {
+      console.warn(`[CliLoader] Failed to search Codex session file: ${err.message}`);
+    }
+    return null;
+  }
+
   // ============================================================
   // GEMINI - Load from ~/.gemini/sessions/<sessionId>.jsonl
   // ============================================================

package/lib/server/services/codex-output-parser.js

@@ -263,13 +263,6 @@ class CodexOutputParser {
     return this.usage;
   }
 
-  /**
-   * Get thread ID (native Codex session ID)
-   */
-  getThreadId() {
-    return this.threadId;
-  }
-
   /**
    * Reset parser state for new request
    */
@@ -277,7 +270,6 @@ class CodexOutputParser {
     this.buffer = '';
     this.finalResponse = '';
     this.usage = null;
-    this.threadId = null;
     this.pendingCommands.clear();
   }
 }

package/lib/server/services/context-bridge.js

@@ -139,13 +139,11 @@ class ContextBridge {
     for (let i = messages.length - 1; i >= 0; i--) {
       const msg = messages[i];
 
-      // For code-focused engines, compress assistant responses to code only
-      // BUT always keep user messages for context continuity
+      // For code-focused engines, filter out non-code content
       let content = msg.content;
-      if (config.codeOnly && msg.role === 'assistant') {
-        const codeContent = this.extractCodeContent(content);
-        // Only use code-only if there's actual code, otherwise keep truncated original
-        content = codeContent || (content.length > 500 ? content.substring(0, 500) + '...' : content);
+      if (config.codeOnly) {
+        content = this.extractCodeContent(content);
+        if (!content) continue; // Skip if no code
       }
 
       // Truncate long messages

package/lib/server/services/gemini-wrapper.js

@@ -67,15 +67,15 @@ class GeminiWrapper {
    *
    * @param {Object} params
    * @param {string} params.prompt - User message/prompt
-   * @param {string} params.threadId - Native Gemini session ID for resume
+   * @param {string} params.sessionId - Session UUID (for logging)
    * @param {string} [params.model='gemini-3-pro-preview'] - Model name
    * @param {string} [params.workspacePath] - Workspace directory
    * @param {Function} [params.onStatus] - Callback for status events (SSE streaming)
-   * @returns {Promise<{text: string, usage: Object, sessionId: string}>}
+   * @returns {Promise<{text: string, usage: Object}>}
    */
   async sendMessage({
     prompt,
-    threadId,
+    sessionId,
     model = DEFAULT_MODEL,
     workspacePath,
     onStatus
@@ -87,23 +87,16 @@ class GeminiWrapper {
       const cwd = workspacePath || this.workspaceDir;
 
       // Build CLI arguments
-      // If threadId exists, use --resume to continue native session
+      // Note: cwd is set in pty.spawn() options, no need for --include-directories
       const args = [
         '-y',                              // YOLO mode - auto-approve all actions
         '-m', model,                       // Model selection
         '-o', 'stream-json',               // JSON streaming for structured events
+        prompt                             // Prompt as positional argument
       ];
 
-      // Add resume flag if continuing existing session
-      if (threadId) {
-        args.push('--resume', threadId);
-      }
-
-      // Add prompt as positional argument
-      args.push(prompt);
-
       console.log(`[GeminiWrapper] Model: ${model}`);
-      console.log(`[GeminiWrapper] ThreadId: ${threadId || '(new session)'}`);
+      console.log(`[GeminiWrapper] Session: ${sessionId}`);
       console.log(`[GeminiWrapper] CWD: ${cwd}`);
       console.log(`[GeminiWrapper] Prompt length: ${prompt.length}`);
 
@@ -177,7 +170,6 @@ class GeminiWrapper {
 
         resolve({
           text: finalResponse,
-          sessionId: parser.getSessionId(), // Native Gemini session ID for resume
           usage: {
             prompt_tokens: promptTokens,
             completion_tokens: completionTokens,

package/lib/server/services/workspace-manager.js

@@ -396,7 +396,7 @@ class WorkspaceManager {
    * @returns {string}
    */
   getSessionPath(workspacePath) {
-    // Convert /var/www/myapp → -var-www-myapp
+    // Convert /home/user/myproject → -home-user-myproject
     const projectDir = workspacePath.replace(/\//g, '-').replace(/^-/, '');
     return path.join(this.claudePath, 'projects', projectDir);
   }

package/lib/server/tests/performance.test.js

@@ -45,7 +45,7 @@ describe('Performance Benchmarks', () => {
 
   test('Workspace validation should be fast', async () => {
     const manager = new WorkspaceManager();
-    const testPath = '/var/www/myapp';
+    const testPath = '/home/user/myproject';
 
     const start = Date.now();
     const validated = await manager.validateWorkspace(testPath);

package/lib/server/tests/services.test.js

@@ -16,7 +16,7 @@ describe('WorkspaceManager', () => {
 
   test('should validate workspace path', async () => {
     // Test with allowed path
-    const validPath = '/var/www/myapp';
+    const validPath = '/home/user/myproject';
     const result = await manager.validateWorkspace(validPath);
     expect(result).toBe(validPath);
   });
@@ -147,7 +147,7 @@ describe('SummaryGenerator', () => {
 describe('Integration - Service Interactions', () => {
   test('WorkspaceManager should use consistent path resolution', async () => {
     const manager = new WorkspaceManager();
-    const testPath = '/var/www/myapp';
+    const testPath = '/home/user/myproject';
     const validated = await manager.validateWorkspace(testPath);
     expect(validated).toBe(testPath);
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mmmbuto/nexuscli",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "NexusCLI - TRI CLI Control Plane (Claude/Codex/Gemini)",
   "main": "lib/server/server.js",
   "bin": {

package/lib/server/db.js.old

@@ -1,225 +0,0 @@
-const initSqlJs = require('sql.js');
-const path = require('path');
-const fs = require('fs');
-
-// Detect environment (Linux vs Termux)
-const isTermux = process.env.PREFIX?.includes('com.termux');
-
-// Database directory
-const dbDir = isTermux
-  ? path.join(process.env.HOME, '.nexuscli')
-  : process.env.NEXUSCLI_DB_DIR || '/var/lib/nexuscli';
-
-// Database file path
-const dbPath = path.join(dbDir, 'nexuscli.db');
-
-// Ensure directory exists
-if (!fs.existsSync(dbDir)) {
-  fs.mkdirSync(dbDir, { recursive: true });
-  console.log(`✅ Created database directory: ${dbDir}`);
-}
-
-let db = null;
-
-// Initialize database
-async function initDb() {
-  const SQL = await initSqlJs();
-
-  // Load existing database or create new
-  if (fs.existsSync(dbPath)) {
-    const buffer = fs.readFileSync(dbPath);
-    db = new SQL.Database(buffer);
-    console.log(`✅ Database loaded: ${dbPath}`);
-  } else {
-    db = new SQL.Database();
-    console.log(`✅ Database created: ${dbPath}`);
-  }
-
-  // Initialize schema
-  initSchema();
-
-  // Auto-save every 5 seconds
-  setInterval(saveDb, 5000);
-
-  return db;
-}
-
-// Save database to file
-function saveDb() {
-  if (!db) return;
-  const data = db.export();
-  const buffer = Buffer.from(data);
-  fs.writeFileSync(dbPath, buffer);
-}
-
-// Initialize schema
-function initSchema() {
-  db.run(`
-    -- Conversations table
-    CREATE TABLE IF NOT EXISTS conversations (
-      id TEXT PRIMARY KEY,
-      title TEXT NOT NULL,
-      created_at INTEGER NOT NULL,
-      updated_at INTEGER NOT NULL,
-      metadata TEXT
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_conversations_updated_at
-    ON conversations(updated_at DESC);
-
-    -- Messages table
-    CREATE TABLE IF NOT EXISTS messages (
-      id TEXT PRIMARY KEY,
-      conversation_id TEXT NOT NULL,
-      role TEXT NOT NULL CHECK(role IN ('user', 'assistant', 'system')),
-      content TEXT NOT NULL,
-      created_at INTEGER NOT NULL,
-      metadata TEXT,
-      FOREIGN KEY (conversation_id) REFERENCES conversations(id) ON DELETE CASCADE
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_messages_conversation_id
-    ON messages(conversation_id);
-
-    CREATE INDEX IF NOT EXISTS idx_messages_created_at
-    ON messages(created_at ASC);
-
-    -- Jobs table
-    CREATE TABLE IF NOT EXISTS jobs (
-      id TEXT PRIMARY KEY,
-      conversation_id TEXT,
-      message_id TEXT,
-      node_id TEXT NOT NULL,
-      tool TEXT NOT NULL,
-      command TEXT NOT NULL,
-      status TEXT NOT NULL CHECK(status IN ('queued', 'executing', 'completed', 'failed', 'cancelled')),
-      exit_code INTEGER,
-      stdout TEXT,
-      stderr TEXT,
-      duration INTEGER,
-      created_at INTEGER NOT NULL,
-      started_at INTEGER,
-      completed_at INTEGER,
-      FOREIGN KEY (conversation_id) REFERENCES conversations(id) ON DELETE SET NULL,
-      FOREIGN KEY (message_id) REFERENCES messages(id) ON DELETE SET NULL
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_jobs_conversation_id
-    ON jobs(conversation_id);
-
-    CREATE INDEX IF NOT EXISTS idx_jobs_status
-    ON jobs(status);
-
-    CREATE INDEX IF NOT EXISTS idx_jobs_created_at
-    ON jobs(created_at DESC);
-
-    -- Users table
-    CREATE TABLE IF NOT EXISTS users (
-      id TEXT PRIMARY KEY,
-      username TEXT UNIQUE NOT NULL,
-      password_hash TEXT NOT NULL,
-      role TEXT NOT NULL DEFAULT 'user' CHECK(role IN ('admin', 'user')),
-      is_locked INTEGER NOT NULL DEFAULT 0,
-      failed_attempts INTEGER NOT NULL DEFAULT 0,
-      last_failed_attempt INTEGER,
-      locked_until INTEGER,
-      created_at INTEGER NOT NULL,
-      last_login INTEGER
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_users_username
-    ON users(username);
-
-    -- Login attempts table (for rate limiting)
-    CREATE TABLE IF NOT EXISTS login_attempts (
-      id INTEGER PRIMARY KEY AUTOINCREMENT,
-      ip_address TEXT NOT NULL,
-      username TEXT,
-      success INTEGER NOT NULL DEFAULT 0,
-      timestamp INTEGER NOT NULL
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_login_attempts_ip
-    ON login_attempts(ip_address, timestamp DESC);
-
-    CREATE INDEX IF NOT EXISTS idx_login_attempts_timestamp
-    ON login_attempts(timestamp DESC);
-
-    -- Nodes table (optional - for multi-node setups)
-    CREATE TABLE IF NOT EXISTS nodes (
-      id TEXT PRIMARY KEY,
-      hostname TEXT NOT NULL,
-      ip_address TEXT,
-      status TEXT NOT NULL CHECK(status IN ('online', 'offline', 'error')),
-      capabilities TEXT,
-      last_heartbeat INTEGER,
-      created_at INTEGER NOT NULL
-    );
-
-    CREATE INDEX IF NOT EXISTS idx_nodes_status
-    ON nodes(status);
-  `);
-
-  console.log('✅ Database schema initialized');
-}
-
-// Prepare statement (sql.js uses db.prepare)
-function prepare(sql) {
-  const stmt = db.prepare(sql);
-  return {
-    run: (...params) => {
-      stmt.bind(params);
-      stmt.step();
-      stmt.reset();
-      saveDb(); // Auto-save on write
-    },
-    get: (...params) => {
-      stmt.bind(params);
-      const result = stmt.step() ? stmt.getAsObject() : null;
-      stmt.reset();
-      return result;
-    },
-    all: (...params) => {
-      stmt.bind(params);
-      const results = [];
-      while (stmt.step()) {
-        results.push(stmt.getAsObject());
-      }
-      stmt.reset();
-      return results;
-    }
-  };
-}
-
-// Graceful shutdown
-process.on('exit', () => {
-  if (db) {
-    saveDb();
-    db.close();
-    console.log('✅ Database connection closed');
-  }
-});
-
-process.on('SIGINT', () => {
-  if (db) {
-    saveDb();
-    db.close();
-    console.log('✅ Database connection closed (SIGINT)');
-  }
-  process.exit(0);
-});
-
-process.on('SIGTERM', () => {
-  if (db) {
-    saveDb();
-    db.close();
-  }
-});
-
-// Export db object (initialized async)
-module.exports = {
-  initDb,
-  getDb: () => db,
-  prepare,
-  saveDb
-};