Skip to content

feat: CI sync check for bash/TS shared library parity (kaizen #347)#269

Merged
aviadr1 merged 1 commit intomainfrom
feat/ci-bash-ts-parity-check
Mar 21, 2026
Merged

feat: CI sync check for bash/TS shared library parity (kaizen #347)#269
aviadr1 merged 1 commit intomainfrom
feat/ci-bash-ts-parity-check

Conversation

@aviadr1
Copy link
Copy Markdown

@aviadr1 aviadr1 commented Mar 21, 2026

Summary

  • Adds vitest parity check test that parses function names from bash (.sh) and TypeScript (.ts) shared libraries and flags drift
  • Ports 3 missing parse-command functions to TS: extractGitCPath, detectGhRepo, getPrChangedFiles
  • Explicit exclusion list with documented reasons for intentionally asymmetric functions
  • Validates exclusions are not stale (excluded functions must exist in source)

Closes Garsson-io/kaizen#347

batch-260321-1108-3ef8/run-12

Test plan

  • All 85 hook tests pass (npx vitest run src/hooks/)
  • Parity check catches real drift (verified: 3 functions were missing before port)
  • Exclusions are validated (stale exclusions would fail the test)
  • New parse-command functions have unit tests

🤖 Generated with Claude Code

…bitai#347)

Adds a vitest test that parses function names from both bash and TS
shared libraries (state-utils, parse-command) and flags any functions
present in one but not the other. Explicit exclusions with reasons are
required for intentionally asymmetric functions.

Also ports 3 missing parse-command functions to TS:
- extractGitCPath: extract -C path from git commands
- detectGhRepo: detect GitHub repo from remote URL
- getPrChangedFiles: get changed files for PR commands

batch-260321-1108-3ef8/run-12

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aviadr1 aviadr1 enabled auto-merge (squash) March 21, 2026 13:31
@aviadr1 aviadr1 merged commit 0597c11 into main Mar 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[L2] CI sync check for bash/TS shared library parity

1 participant