4 Commits

• fix: improve error handling, fix bugs, and optimize core libraries

  utils.js:
  - Fix countInFile: enforce global flag on regex to prevent silent
    under-counting (match() without /g returns only first match)
  - Add 5s timeout to readStdinJson to prevent hooks hanging forever
  - Handle EEXIST race condition in ensureDir
  - Pre-compile regex patterns in getGitModifiedFiles to avoid N*M
    compilations and catch invalid patterns before filtering
  - Add JSDoc documentation to all improved functions
  
  session-manager.js:
  - Fix getSessionById triple file read: pass pre-read content to
    getSessionStats instead of re-reading from disk
  - Allow getSessionStats to accept content string directly
  
  session-aliases.js:
  - Wrap temp file cleanup in try/catch to prevent cascading errors
  
  check-console-log.js:
  - Refactor to use shared utils (isGitRepo, getGitModifiedFiles, log)
    instead of raw execSync calls
  - Add exclusion patterns for test files, config files, and scripts/
    where console.log is intentional
  
  session-end.js:
  - Log count of skipped unparseable transcript lines for diagnostics
  
  suggest-compact.js:
  - Guard against NaN from corrupted counter files
  
  package-manager.js:
  - Remove dead fallbackOrder parameter (unused after #162 fix)

  Affaan Mustafa · 2026-02-12 07:06:53 -08:00
  18c5a76a96
• fix: eliminate child process spawns during session startup (#162)

  getAvailablePackageManagers() spawned where.exe/which for each package
  manager (npm, pnpm, yarn, bun). During SessionStart hooks, these 4+
  child processes combined with Bun's own initialization exceeded the
  spawn limit on Windows, freezing the terminal.
  
  Fix: Remove process spawning from the hot path. Steps 1-5 of detection
  (env var, project config, package.json, lock file, global config) already
  cover all file-based detection. If none match, default to npm without
  spawning. Also fix getSelectionPrompt() to list supported PMs without
  checking availability.

  Affaan Mustafa · 2026-02-12 00:01:23 -08:00
  75ab8e6194
• fix: resolve ESLint errors and update tests for project-name fallback

  - Fix 16 ESLint no-unused-vars errors across hook scripts and tests
  - Add eslint-disable comment for intentional control-regex in ANSI stripper
  - Update session file test to use getSessionIdShort() instead of hardcoded 'default'
    (reflects PR #110's project-name fallback behavior)
  - Add marketing/ to .gitignore (local drafts)
  - Add skill-create-output.js (terminal output formatter)
  
  All 69 tests now pass. CI should be green.

  Affaan Mustafa · 2026-01-29 02:58:51 -08:00
  a44a0553bb
• feat: cross-platform support with Node.js scripts

  - Rewrite all bash hooks to Node.js for Windows/macOS/Linux compatibility
  - Add package manager auto-detection (npm, pnpm, yarn, bun)
  - Add scripts/lib/ with cross-platform utilities
  - Add /setup-pm command for package manager configuration
  - Add comprehensive test suite (62 tests)
  
  Co-authored-by: zerx-lab

  zerx-lab · 2026-01-22 23:08:07 -08:00
  970f8bf884