28 Commits

• Add PowerShell installer wrapper and update documentation (#532)

  * Add install.ps1 PowerShell wrapper and tests
  
  Add a Windows-native PowerShell wrapper (install.ps1) that resolves symlinks and delegates to the Node-based installer runtime. Update README with PowerShell usage examples and cross-platform npx entrypoint guidance. Point the ecc-install bin to the Node installer (scripts/install-apply.js) in package.json (and refresh package-lock), include install.ps1 in package files, and add tests: a new install-ps1.test.js and a tweak to install-sh.test.js to skip on Windows. These changes provide native Windows installer support while keeping npm-compatible cross-platform invocation.
  
  * Improve tests for Windows HOME/USERPROFILE
  
  Make tests more cross-platform by ensuring HOME and USERPROFILE are kept in sync and by normalizing test file paths for display.
  
  - tests/lib/session-adapters.test.js: set USERPROFILE when temporarily setting HOME and restore previous USERPROFILE on teardown.
  - tests/run-all.js: use a normalized displayPath (forward-slash separated) for logging and error messages so output is consistent across platforms.
  - tests/scripts/ecc.test.js & tests/scripts/session-inspect.test.js: build envOverrides from options.env and add HOME <-> USERPROFILE fallbacks so spawned child processes receive both variables when only one is provided.
  
  These changes prevent test failures and inconsistent logs on Windows where USERPROFILE is used instead of HOME.
  
  * Fix Windows paths and test flakiness
  
  Improve cross-platform behavior and test stability.
  
  - Remove unused createLegacyInstallPlan import from install-lifecycle.js.
  - Change resolveInstallConfigPath to use path.normalize(path.join(cwd, configPath)) to produce normalized relative paths.
  - Tests: add toBashPath and normalizedRelativePath helpers to normalize Windows paths for bash and comparisons.
  - Make cleanupTestDir retry rmSync on transient Windows errors (EPERM/EBUSY/ENOTEMPTY) with short backoff using sleepMs.
  - Ensure spawned test processes receive USERPROFILE and convert repo/detect paths to bash format when invoking bash.
  
  These changes reduce Windows-specific failures and flakiness in the test suite and tidy up a small unused import.

  Fredrik Hallerös · 2026-03-16 13:35:17 -07:00
  17a6ef4edb
• fix(harness-audit): make scoring deterministic with scripted rubric (#524)
  Chandan Semwal · 2026-03-16 13:34:30 -07:00
  0ff1b594d0
• fix: resolve all CI test failures (19 fixes across 6 files) (#519)

  - canonical-session: fall back to JSON file recording when the loaded
    state-store module has no writer methods (factory vs instance)
  - install-executor: skip node_modules and .git dirs in listFilesRecursive
    to prevent ETIMEDOUT copying thousands of .opencode dependency files
  - ecc.js: increase maxBuffer to 10MB for spawned subcommands to prevent
    ENOBUFS on large install plan JSON output
  - install-apply.test: update Cursor and Antigravity path assertions to
    match flattened rule layout and remapped dirs (workflows, skills)
  - ecc.test: increase maxBuffer in test runner to handle large output
  - orchestrate-codex-worker.sh: guard against unreadable task file before
    cat, write failure status and handoff artifacts on early exit

  Affaan Mustafa · 2026-03-16 01:59:53 -07:00
  b48930974b
• feat: record canonical session snapshots via adapters (#511)
  Affaan Mustafa · 2026-03-16 01:35:45 -07:00
  426fc54456
• feat: strengthen install lifecycle and target adapters (#512)

  * fix: strengthen install lifecycle adapters
  
  * fix: restore template content on uninstall

  Affaan Mustafa · 2026-03-15 21:47:31 -07:00
  131f977841
• feat: wire manifest resolution into install execution (#509)
  Affaan Mustafa · 2026-03-15 21:47:22 -07:00
  1e0238de96
• feat: self-improving skills loop — observe, inspect, amend, evaluate

  - Add skill health observation layer (execution logging, success/failure tracking)
  - Add skill health inspector (trace recurring failures across runs)
  - Add amendify mechanism (propose SKILL.md patches from failure evidence)
  - Add evaluation scaffolding (compare amended vs original performance)
  - Wire into session-inspect CLI: skills:health, skills:amendify, skills:evaluate
  - 1145/1145 tests passing (+3 new)

  Affaan Mustafa · 2026-03-14 23:21:18 -07:00
  c53bba9e02
• feat: expand session adapter registry with structured targets

  - Registry accepts { type, value } structured targets
  - Add --list-adapters and --target-type CLI flags to session-inspect
  - Export adapter type from claude-history and dmux-tmux adapters
  - 71 new session adapter tests, 34 new session-inspect tests
  - All 1142 tests passing

  Affaan Mustafa · 2026-03-14 19:09:26 -07:00
  2b2777915e
• feat: orchestration harness, selective install, observer improvements
  Affaan Mustafa · 2026-03-14 12:55:25 -07:00
  4e028bd2d2
• fix: stabilize windows hook and claw tests
  Affaan Mustafa · 2026-03-05 12:37:24 -08:00
  d1f44e89e2
• feat: deliver v1.8.0 harness reliability and parity updates
  Affaan Mustafa · 2026-03-04 14:48:06 -08:00
  48b883d741
• fix: address CodeRabbit review — deduplicate prompt, fix skill count

  - Swap loadHistory/appendTurn order to prevent user message appearing
    twice in the prompt (once in history, once as USER MESSAGE)
  - Calculate actual loaded skill count via fs.existsSync instead of
    counting requested skill names (banner now reflects reality)
  - Add err.stack to test harness error output for better debugging

  andydiaz122 · 2026-02-24 10:20:17 -05:00
  e0b3a7be65
• feat: add NanoClaw agent REPL — persistent session-aware CLI for ECC

  Implements a barebones agent loop that delegates to `claude -p` with
  markdown-as-database session persistence and ECC skill context loading.
  Zero external dependencies, ~264 lines of pure Node.js CommonJS.
  
  - scripts/claw.js: core module (storage, context, delegation, REPL)
  - commands/claw.md: slash command definition with usage docs
  - tests/scripts/claw.test.js: 14 unit tests covering all modules
  - package.json: add claw script and files entry
  - tests/run-all.js: register claw tests in test manifest

  andydiaz122 · 2026-02-15 12:02:19 -05:00
  853c64d7c1
• test: add 3 tests for stdin 1MB overflow and analyzePhase async method (round 87)

  - post-edit-format.js: verify MAX_STDIN truncation at 1MB limit
  - post-edit-typecheck.js: verify MAX_STDIN truncation at 1MB limit
  - skill-create-output.js: test analyzePhase() returns Promise and writes output

  Affaan Mustafa · 2026-02-13 12:42:20 -08:00
  9b286ab3f8
• fix: use nullish coalescing for confidence default + add 3 tests (round 85)

  Fix confidence=0 showing 80% instead of 0% in patterns() (|| → ??).
  Test evaluate-session.js config parse error catch, getSessionIdShort
  fallback at root CWD, and precise confidence=0 assertion.

  Affaan Mustafa · 2026-02-13 12:11:26 -08:00
  8cacf0f6a6
• test: cover setGlobal/setProject catch blocks and session-start main().catch

  - setup-package-manager: setGlobal catch when HOME is non-directory (ENOTDIR)
  - setup-package-manager: setProject catch when CWD is read-only (EACCES)
  - session-start: main().catch handler when ensureDir throws (exit 0, don't block)
  
  Total tests: 825 → 828

  Affaan Mustafa · 2026-02-13 09:55:00 -08:00
  241c35a589
• test: add 3 tests for setup-pm --project success, demo export, --list marker (Round 68)
  Affaan Mustafa · 2026-02-13 07:23:16 -08:00
  5031a84d6e
• test: add --global success, bare PM name, and source label tests (Round 62)

  - setup-package-manager.test.js: --global npm writes config and exits 0
  - setup-package-manager.test.js: bare PM name sets global preference
  - setup-package-manager.test.js: --detect with env var shows source 'environment'
  
  791 tests total, all passing.

  Affaan Mustafa · 2026-02-13 06:49:29 -08:00
  85a86f6747
• test: add search scope, path utility, and zero-value analysis tests (Round 54)

  - getAllSessions search matches only shortId, not title/content
  - getSessionPath returns absolute path with correct directory structure
  - analysisResults handles zero values for all data fields without crash

  Affaan Mustafa · 2026-02-13 05:43:29 -08:00
  3bc0929c6e
• test: add setup-package-manager marker uniqueness and list completeness tests

  Round 45: verify --detect shows exactly one (current) marker and
  --list includes all four PMs with Lock file + Install entries.

  Affaan Mustafa · 2026-02-13 04:47:30 -08:00
  f6ebc2a3c2
• fix: correct box() off-by-one width calculation in skill-create-output

  The box() helper produced lines that were width+1 characters instead of
  the requested width. Adjusted all three formulas (top border, middle
  content, bottom border) by -1 each. Added 4 tests verifying box width
  accuracy across instincts(), analysisResults(), and nextSteps() output.

  Affaan Mustafa · 2026-02-13 04:05:12 -08:00
  b497135b95
• fix: header subtitle width mismatch in skill-create-output; add 9 tests (Round 34)

  - Fix subtitle padding 55→59 so line 94 matches 64-char border width
  - Add 4 header width alignment tests (skill-create-output)
  - Add 3 getExecCommand non-string args tests (package-manager)
  - Add 2 detectFromPackageJson non-string type tests (package-manager)

  Affaan Mustafa · 2026-02-13 03:58:16 -08:00
  554b5d6704
• fix: reject flags passed as package manager names in setup-package-manager CLI

  When --global or --project was followed by another flag (e.g., --global --project),
  the flag was treated as a package manager name. Added pmName.startsWith('-') check
  to both handlers. Added 20 tests across 4 test files covering argument validation,
  ensureDir error propagation, runCommand stderr handling, and saveAliases failure paths.

  Affaan Mustafa · 2026-02-13 03:37:46 -08:00
  167b105cac
• test: add 22 tests for validators, skill-create-output, and package-manager edge cases
  Affaan Mustafa · 2026-02-13 03:02:28 -08:00
  b3db83d018
• fix: clamp progressBar to prevent RangeError on overflow, add 10 tests

  progressBar() in skill-create-output.js could crash with RangeError when
  percent > 100 because repeat() received a negative count. Fixed by
  clamping filled to [0, width].
  
  New tests:
  - progressBar edge cases: 0%, 100%, and >100% confidence
  - Empty patterns/instincts arrays
  - post-edit-format: null tool_input, missing file_path, prettier failure
  - setup-package-manager: --detect output completeness, current marker

  Affaan Mustafa · 2026-02-13 02:01:57 -08:00
  5398ac793d
• fix: box() off-by-one alignment, add 5 tests for readStdinJson and box alignment

  - skill-create-output.js: fix top border being 1 char narrower than
    middle/bottom lines (width - title - 5 → width - title - 4)
  - Add box alignment regression test verifying all lines have equal width
  - Add 4 readStdinJson tests via subprocess (valid JSON, invalid JSON,
    empty stdin, nested objects) — last untested exported utility function
  - All 338 tests passing

  Affaan Mustafa · 2026-02-12 16:53:06 -08:00
  5107b3669f
• fix: remove unused imports in test files (ESLint)

  - validators.test.js: remove unused execSync (only execFileSync used)
  - skill-create-output.test.js: remove unused path module

  Affaan Mustafa · 2026-02-12 16:51:40 -08:00
  64796f99be
• fix: add word boundary to dev server hook regex, fix box() crash, add 27 tests

  - hooks.json: add \b word boundary anchors to dev server blocking regex
    to prevent false positives matching "npm run develop", "npm run devtools" etc.
  - skill-create-output.js: guard box() horizontal repeat with Math.max(0, ...)
    to prevent RangeError when title exceeds container width
  - Add 13 tests for setup-package-manager.js CLI argument parsing
  - Add 14 tests for skill-create-output.js SkillCreateOutput class
  - All 333 tests passing

  Affaan Mustafa · 2026-02-12 16:46:06 -08:00
  924bac4ddf