7 Commits

• Fix stdin overflow bug in hook scripts - truncate chunks to stay within MAX_STDIN limit

  Co-authored-by: pangerlkr <73515951+pangerlkr@users.noreply.github.com>

  copilot-swe-agent[bot] · 2026-02-18 07:40:12 +00:00
  1eca3c9130
• fix: nullish coalescing in evaluate-session config, narrow pre-compact glob, add 11 tests

  - evaluate-session.js: || 10 → ?? 10 for min_session_length (0 is valid)
  - pre-compact.js: *.tmp → *-session.tmp to match only session files
  - 11 new tests: config loading (min=0, null, custom path, invalid JSON),
    session-end update path (timestamp, template replace, preserve content),
    pre-compact glob specificity, extractSessionSummary edge cases

  Affaan Mustafa · 2026-02-13 02:42:01 -08:00
  4ff4872bf3
• fix: 3 bugs fixed, stdin encoding hardened, 37 CI validator tests added

  Bug fixes:
  - utils.js: glob-to-regex conversion now escapes all regex special chars
    (+, ^, $, |, (), {}, [], \) before converting * and ? wildcards
  - validate-hooks.js: escape sequence processing order corrected —
    \\\\ now processed before \\n and \\t to prevent double-processing
  - 6 hooks: added process.stdin.setEncoding('utf8') to prevent
    multi-byte UTF-8 character corruption at chunk boundaries
    (check-console-log, post-edit-format, post-edit-typecheck,
    post-edit-console-warn, session-end, evaluate-session)
  
  New tests (37):
  - CI validator test suite (tests/ci/validators.test.js):
    - validate-agents: 9 tests (real project, frontmatter parsing,
      BOM/CRLF, colons in values, missing fields, non-md skip)
    - validate-hooks: 13 tests (real project, invalid JSON, invalid
      event types, missing fields, async/timeout validation, inline JS
      syntax, array commands, legacy format)
    - validate-skills: 6 tests (real project, missing SKILL.md, empty
      files, non-directory entries)
    - validate-commands: 5 tests (real project, empty files, non-md skip)
    - validate-rules: 4 tests (real project, empty files)
  
  Total test count: 228 (up from 191)

  Affaan Mustafa · 2026-02-12 16:08:49 -08:00
  492c99ac24
• fix: 2 bugs fixed, 17 tests added for hook scripts

  Bug fixes:
  - evaluate-session.js: whitespace-tolerant regex for counting user
    messages in JSONL transcripts (/"type":"user"/ → /"type"\s*:\s*"user"/)
  - session-end.js: guard against null elements in content arrays
    (c.text → (c && c.text) to prevent TypeError)
  
  New tests (17):
  - evaluate-session: whitespace JSON regression test
  - session-end: null content array elements regression test
  - post-edit-console-warn: 5 tests (warn, skip non-JS, clean files,
    missing file, stdout passthrough)
  - post-edit-format: 3 tests (empty stdin, non-JS skip, invalid JSON)
  - post-edit-typecheck: 4 tests (empty stdin, non-TS skip, missing file,
    no tsconfig)
  
  Total test count: 191 (up from 164)

  Affaan Mustafa · 2026-02-12 16:02:31 -08:00
  90ea2f327c
• fix: migrate hooks to stdin JSON input, fix duplicate main() calls, add threshold validation

  - Migrate session-end.js and evaluate-session.js from CLAUDE_TRANSCRIPT_PATH
    env var to stdin JSON transcript_path (correct hook input mechanism)
  - Remove duplicate main() calls that ran before stdin was read, causing
    session files to be created with empty data
  - Add range validation (1-10000) on COMPACT_THRESHOLD in suggest-compact.js
    to prevent negative or absurdly large thresholds
  - Add integration/hooks.test.js to tests/run-all.js so CI runs all 97 tests
  - Update evaluate-session.sh to parse transcript_path from stdin JSON
  - Update hooks.test.js to pass transcript_path via stdin instead of env var
  - Sync .cursor/ copies

  Affaan Mustafa · 2026-02-12 15:33:55 -08:00
  ed7ec29ead
• fix: add try-catch to inline hooks, fix schema drift

  - Wrap JSON.parse in try-catch for all 6 inline hooks in hooks.json
    (dev-server blocker, tmux reminder, git-push reminder, doc blocker,
    PR create logger, build analysis) — previously unguarded JSON.parse
    would crash on empty/malformed stdin, preventing data passthrough
  - Add config parse error logging to evaluate-session.js
  - Fix plugin.schema.json: author can be string or {name,url} object,
    add version (semver pattern), homepage, keywords, skills, agents
  - Fix package-manager.schema.json: add setAt (date-time) field and
    make packageManager required to match actual code behavior

  Affaan Mustafa · 2026-02-12 14:38:00 -08:00
  6686cb9bda
• 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