7 Commits

• fix: add missing validation in renameAlias, add 6 tests

  renameAlias was missing length (>128), reserved name, and empty string
  validation that setAlias enforced. This inconsistency allowed renaming
  aliases to reserved names like 'list' or 'delete'.
  
  Also adds tests for:
  - renameAlias empty string, reserved name, and length limit
  - validate-skills whitespace-only SKILL.md rejection
  - validate-rules whitespace-only file and recursive subdirectory scan

  Affaan Mustafa · 2026-02-13 01:05:59 -08:00
  654731f232
• fix: use valid model name in colon-in-values frontmatter test

  The test was using 'claude-sonnet-4-5-20250929' which isn't in VALID_MODELS
  (haiku/sonnet/opus). Use 'sonnet' with a description field containing
  colons to properly test colon handling in frontmatter values.

  Affaan Mustafa · 2026-02-12 17:33:17 -08:00
  39280e251b
• test: add 6 tests for command validation and session content verification

  - validate-commands: creates: line skipping, valid cross-refs, unclosed
    code blocks, valid workflow diagrams
  - session-end: backtick escaping in session files, tools/files in output

  Affaan Mustafa · 2026-02-12 17:15:21 -08:00
  35aed05903
• test: add regression tests for empty frontmatter field rejection

  Add 2 tests verifying validate-agents correctly rejects agents with
  empty model and empty tools values in YAML frontmatter.

  Affaan Mustafa · 2026-02-12 16:58:27 -08:00
  e5f1c58c11
• 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
• test: add cross-reference validation tests for validate-commands

  - Add runValidatorWithDirs() helper for multi-constant overrides
  - Test broken command references (e.g., /nonexistent-cmd)
  - Test broken agent path references (e.g., agents/fake-agent.md)
  - Test fenced code block exclusion (refs inside ``` are skipped)
  - Test broken workflow agent references (e.g., planner -> ghost-agent)
  - Total tests: 261 → 287 (+26)

  Affaan Mustafa · 2026-02-12 16:26:24 -08:00
  6fa3bfe71d
• 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