8 Commits

Author	SHA1	Message	Date
Affaan Mustafa	27dce7794a	fix: reject empty/invalid array commands in hooks validator, add 19 tests ... validate-hooks.js: Empty arrays [] and arrays with non-string elements (e.g., [123, null]) passed command validation due to JS truthiness of empty arrays (![] === false). Added explicit length and element type checks. 19 new tests covering: non-array event type values, null/string matcher entries, string/number top-level data, empty string/array commands, non-string array elements, non-string type field, non-number timeout, timeout boundary (0), unwrapped hooks format, legacy format error paths, empty agent directory, whitespace-only command files, valid skill refs, mixed valid/invalid rules and skills.	2026-02-13 02:33:40 -08:00
Affaan Mustafa	2dbba8877b	fix: reject whitespace-only command/field values in CI validators, add 10 tests ... validate-hooks.js: whitespace-only command strings now fail validation validate-agents.js: whitespace-only model/tools values now fail validation	2026-02-13 02:09:22 -08:00
Affaan Mustafa	492c99ac24	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)	2026-02-12 16:08:49 -08:00
Affaan Mustafa	40a4fafa7f	fix: add async/timeout to hooks schema and validate in CI ... - hooks.schema.json: add async (boolean) and timeout (number) properties to hookItem definition, matching fields used in hooks.json - validate-hooks.js: validate async and timeout types when present - hooks.test.js: add SessionEnd to required event types check	2026-02-12 15:58:59 -08:00
Affaan Mustafa	639c9aaca3	fix: Windows path support, error handling, and dedup in validators ... - session-manager.js: fix getSessionStats path detection to handle Windows paths (C:\...) in addition to Unix paths (/) - package-manager.js: add try-catch to setPreferredPackageManager for consistent error handling with setProjectPackageManager - validate-hooks.js: extract duplicated hook entry validation into reusable validateHookEntry() helper - Update .d.ts JSDoc for both fixes	2026-02-12 15:57:20 -08:00
Affaan Mustafa	76b271ab6b	fix: 6 bugs fixed, 67 tests added for session-manager and session-aliases ... Bug fixes: - utils.js: prevent duplicate 'g' flag in countInFile regex construction - validate-agents.js: handle CRLF line endings in frontmatter parsing - validate-hooks.js: handle \t and \\ escape sequences in inline JS validation - session-aliases.js: prevent NaN in date sort when timestamps are missing - session-aliases.js: persist rollback on rename failure instead of silent loss - session-manager.js: require absolute paths in getSessionStats to prevent content strings ending with .tmp from being treated as file paths New tests (164 total, up from 97): - session-manager.test.js: 27 tests covering parseSessionFilename, parseSessionMetadata, getSessionStats, CRUD operations, getSessionSize, getSessionTitle, edge cases (null input, non-existent files, directories) - session-aliases.test.js: 40 tests covering loadAliases (corrupted JSON, invalid structure), setAlias (validation, reserved names), resolveAlias, listAliases (sort, search, limit), deleteAlias, renameAlias, updateAliasTitle, resolveSessionAlias, getAliasesForSession, cleanupAliases, atomic write Also includes hook-generated improvements: - utils.d.ts: document that readStdinJson never rejects - session-aliases.d.ts: fix updateAliasTitle type to accept null - package-manager.js: add try-catch to setProjectPackageManager writeFile	2026-02-12 15:50:04 -08:00
Affaan Mustafa	911d38f686	feat: add 3 new skills, JS syntax validation in hooks CI, and edge case tests ... - New skills: api-design, database-migrations, deployment-patterns - validate-hooks.js: validate inline JS syntax in node -e hook commands - utils.test.js: edge case tests for findFiles with null/undefined inputs - README: update skill count to 35, add new skills to directory tree	2026-02-12 15:24:28 -08:00
Roei Bar Aviv	7c0bc25982	feat: add comprehensive CI/CD pipeline ... Adds GitHub Actions workflows for CI, maintenance, and releases with multi-platform testing matrix.	2026-01-28 23:05:43 -08:00