docs: rewrite roadmap and prepare v5.3.2 by flyingrobots · Pull Request #19 · git-stunts/git-cas

flyingrobots · 2026-03-15T20:31:58Z

Summary

rewrite ROADMAP.md around the real shipped state at v5.3.1
formalize the human CLI/TUI vs planned git cas agent split
replace loose backlog/vision buckets with sequenced milestones M17-M25
refresh STATUS.md to match the roadmap and current release line
harden slow vault rotation tests with explicit per-test timeouts

Testing

npx eslint .
npm test
GIT_STUNTS_DOCKER=1 npx vitest run test/integration
bunx vitest run test/unit
GIT_STUNTS_DOCKER=1 bunx vitest run test/integration
deno run -A npm:vitest run test/unit
GIT_STUNTS_DOCKER=1 deno run -A npm:vitest run test/integration

Summary by CodeRabbit

Documentation
- Clarified README JSON output wording; reorganized STATUS with a refreshed roadmap, interface strategy, and versioning layout.
Bug Fixes
- Improved CLI exit behavior to flush output and handle broken-pipe scenarios; version reporting now reflects package version.
Tests
- Extended several test timeouts and hardened integration tests to invoke the CLI more reliably.
Chores
- Added test-runner workspace/config, new Vitest config, Unreleased changelog entry, and bumped package version.

coderabbitai · 2026-03-15T20:32:17Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b428cd4d-9fcd-45d7-ab99-c04ea0a93361

📥 Commits

Reviewing files that changed from the base of the PR and between 7525cc3 and 47d603a.

📒 Files selected for processing (1)

STATUS.md

🚧 Files skipped from review as they are similar to previous changes (1)

STATUS.md

📝 Walkthrough

Walkthrough

Added a new bin/io.js for broken‑pipe detection and flush‑before‑exit utilities; wired it into the CLI startup/exit flow. Made runAction accept an injectable setExitCode. Replaced shell-based test invocations with spawn helpers, extended several test timeouts, added Vitest workspace/config, and updated docs/metadata.

Changes

Cohort / File(s)	Summary
StdIO & CLI wiring `bin/io.js`, `bin/git-cas.js`, `bin/actions.js`	Add `bin/io.js` exports (`isBrokenPipeError`, `installBrokenPipeHandlers`, `flushStdioAndExit`); `bin/git-cas.js` reads version from package.json, installs broken‑pipe handlers, and uses flush-before-exit; `runAction` gains injectable `setExitCode`.
Integration tests — spawn helpers `test/integration/vault-cli.test.js`, `test/integration/round-trip.test.js`, `test/integration/vault.test.js`	Replace shell `execSync` with `spawnSync` helpers (`runCli`, `cli`, `initBareRepo`), call CLI with argv arrays, and tighten error handling/assertions.
Unit tests — IO & runAction `test/unit/cli/io.test.js`, `test/unit/cli/actions.test.js`, `test/unit/cli/version.test.js`, `test/unit/cli/vitest-workspace.test.js`	Add tests for broken‑pipe handlers and flush/exit behavior; update tests to inject/assert `setExitCode`; add CLI --version and Vitest workspace assertions.
Test timeouts & config `test/unit/domain/services/rotateVaultPassphrase.test.js`, `test/unit/facade/ContentAddressableStore.rotation.test.js`, `test/unit/vault/VaultService.test.js`, `vitest.config.js`, `vitest.workspace.js`	Introduce `LONG_TEST_TIMEOUT_MS = 15000` and apply to several tests; add `vitest.config.js` and `vitest.workspace.js` with integration fileParallelism disabled.
Docs & metadata `README.md`, `STATUS.md`, `CHANGELOG.md`, `package.json`, `jsr.json`	Minor README wording tweak; major STATUS.md restructuring; add Unreleased section to CHANGELOG; bump package version to 5.3.2 and update jsr.json.
Misc `.dockerignore`	Add `.git` to `.dockerignore`.

Sequence Diagram(s)

sequenceDiagram
  rect rgba(220,230,255,0.5)
    participant CLI as CLI (bin/git-cas.js)
    participant IO as IO Module (bin/io.js)
    participant STDOUT as stdout
    participant STDERR as stderr
    participant Proc as process/exit
  end

  CLI->>IO: installBrokenPipeHandlers()
  IO->>STDOUT: attach 'error' listener
  IO->>STDERR: attach 'error' listener

  Note over STDOUT,STDERR: On EPIPE error
  STDOUT-->>IO: 'error' (EPIPE)
  IO->>Proc: call exit(getExitCode())
  CLI->>IO: flushStdioAndExit(code)
  IO->>STDOUT: flushStream()
  IO->>STDERR: flushStream()
  IO->>Proc: exit(code)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

feat: M8 Spit Shine + M9 Cockpit — code review polish & CLI improvements #11 — Overlaps on CLI exit handling and runAction injectable setExitCode.
feat(cas): M2 Boomerang — restore, CLI, integration tests #2 — Related modifications to CLI startup/version/stdio behavior used by the IO utilities.
refactor: extract helpers from facade + cleanup #16 — Overlapping refactors around vault/key rotation and related test/config updates.

Poem

🐰 I hopped through streams with careful cheer,

Caught broken pipes and kept exits clear,
Spawned new tests that speak in arrays,
Timeouts stretched to patient days,
A little rabbit signs this cheer. 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'docs: rewrite roadmap and prepare v5.3.2' accurately summarizes the main changes: documentation updates to the roadmap/status and version bump to 5.3.2.
Docstring Coverage	✅ Passed	Docstring coverage is 80.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat/roadmap-dual-cli-surfaces

📝 Coding Plan

Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 35: Update the user-facing bullet that starts with "Verify & JSON output"
to fix the grammar: replace "provides convenience structured output" with
"provides convenient structured output" (or an equivalent phrasing). Locate the
bullet text containing "git cas verify" and the "--json" mention and change that
fragment only so the rest of the sentence remains unchanged.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 92e634d9-5c3a-470c-af00-3e1596e38750

📥 Commits

Reviewing files that changed from the base of the PR and between d63de0b and 50708d7.

📒 Files selected for processing (6)

README.md
ROADMAP.md
STATUS.md
test/unit/domain/services/rotateVaultPassphrase.test.js
test/unit/facade/ContentAddressableStore.rotation.test.js
test/unit/vault/VaultService.test.js

README.md

flyingrobots · 2026-03-15T21:56:15Z

@coderabbitai review please 🥕

coderabbitai · 2026-03-15T21:56:30Z

@flyingrobots Sure thing! Kicking off a review of PR #19 now. 🥕

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

flyingrobots · 2026-03-16T00:39:06Z

@coderabbitai review please 🥕

coderabbitai · 2026-03-16T00:39:25Z

@flyingrobots Sure! Triggering another review of PR #19 now. 🥕

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@STATUS.md`:
- Around line 20-29: The "Recently Shipped" table mixes release versions with
milestone IDs that look chronological (e.g., `v3.1.0` with M13 and `v4.0.0` with
M14 appearing alongside M10–M12), so add a brief clarifying note above or below
the table stating that milestone IDs (M10, M11, M12, M13, M14, etc.) are
thematic/non-linear and do not imply chronological order, or alternatively
reorder the rows by milestone sequence; update the table caption or a one-line
parenthetical (e.g., "Milestone IDs are thematic and non-sequential") and ensure
the note references the milestone labels (M10–M14) and versions (`v3.1.0`,
`v4.0.0`, `v5.0.0`, etc.) so readers aren't confused.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6a2ebf1e-801e-4150-be9d-abbebded724a

📥 Commits

Reviewing files that changed from the base of the PR and between 053ea77 and 7525cc3.

📒 Files selected for processing (9)

CHANGELOG.md
STATUS.md
bin/git-cas.js
jsr.json
package.json
test/unit/cli/version.test.js
test/unit/cli/vitest-workspace.test.js
vitest.config.js
vitest.workspace.js

✅ Files skipped from review due to trivial changes (3)

CHANGELOG.md
jsr.json
package.json

🚧 Files skipped from review as they are similar to previous changes (1)

vitest.config.js

STATUS.md

docs: rewrite roadmap around dual CLI surfaces

50708d7

coderabbitai bot reviewed Mar 15, 2026

View reviewed changes

README.md Outdated Show resolved Hide resolved

flyingrobots added 2 commits March 15, 2026 14:54

Fix: stabilize multi-runtime CLI test runs

c3e7271

docs: fix README json output wording

053ea77

flyingrobots added 2 commits March 15, 2026 17:38

fix: harden cli version and vitest projects

00def38

chore: start v5.3.2 development line

7525cc3

coderabbitai bot reviewed Mar 16, 2026

View reviewed changes

STATUS.md Show resolved Hide resolved

docs: clarify milestone ordering in status

47d603a

flyingrobots changed the title ~~docs: rewrite roadmap around dual CLI surfaces~~ docs: rewrite roadmap and prepare v5.3.2 Mar 16, 2026

flyingrobots merged commit b445f66 into main Mar 16, 2026
6 checks passed

flyingrobots deleted the feat/roadmap-dual-cli-surfaces branch March 16, 2026 00:59

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs: rewrite roadmap and prepare v5.3.2#19

docs: rewrite roadmap and prepare v5.3.2#19
flyingrobots merged 6 commits intomainfrom
feat/roadmap-dual-cli-surfaces

flyingrobots commented Mar 15, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Mar 15, 2026 •

edited

Loading

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

flyingrobots commented Mar 15, 2026

Uh oh!

coderabbitai bot commented Mar 15, 2026

Uh oh!

flyingrobots commented Mar 16, 2026

Uh oh!

coderabbitai bot commented Mar 16, 2026

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

flyingrobots commented Mar 15, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Testing

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Mar 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

flyingrobots commented Mar 15, 2026

Uh oh!

coderabbitai bot commented Mar 15, 2026

Uh oh!

flyingrobots commented Mar 16, 2026

Uh oh!

coderabbitai bot commented Mar 16, 2026

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

flyingrobots commented Mar 15, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Mar 15, 2026 •

edited

Loading