Skip to content

fix: stabilize CLI processing errors - #27

Merged
rogerchappel merged 3 commits into
mainfrom
agent/oss-fc00092fcd69-cli-error-boundary
Sep 1, 2026
Merged

rogerchappel merged 3 commits into
mainfrom
agent/oss-fc00092fcd69-cli-error-boundary

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • add CLI regressions for malformed TypeScript analysis and failed baseline persistence in text and JSON modes
  • handle analysis, report generation, and baseline persistence through one error boundary
  • delay coverage output until baseline persistence succeeds and document the stable failure-output contract

Verification

  • Tests or checks run: npm ci --ignore-scripts; npx vitest run src/cli.test.ts; npm run check; npm run lint; npm run release:check; git diff --check origin/main...HEAD
  • Manual review completed: reviewed all three commits, the final diff, and text/JSON failure streams

Risk Level

  • Low
  • Medium
  • High

Notes: The change only affects CLI failure handling. Successful report and baseline output remain unchanged.

Rollback Plan

  • Revert the three commits in reverse order.

Human Decision Needed

  • None
  • Maintainer review
  • Product/design decision
  • Security/privacy review
  • Other:

Commit-by-commit summary

  • bda8eea test: cover CLI processing failures
  • 6ffa29c fix: unify CLI processing errors
  • 4a5df02 docs: define CLI failure output

@rogerchappel

Copy link
Copy Markdown
Owner Author

Automated merge note

  • Triage class: auto-merge
  • Summary: Unifies CLI handling for analysis and baseline-save failures, adds text/JSON regression coverage, and documents the failure-output contract (3 files, +71/-21).
  • Checks run: GitHub Actions test on Node 20.19.0, 22, and 24 completed SUCCESS; final diff reread; branch protection verified (one approving review, no required status checks); all commit author/committer identities verified as Roger Chappel miscanalysis@gmail.com.
  • Rebased/CI-repaired: No; neither was required.
  • Verified head SHA: 4a5df0275f0ad261a52a399e4876eb0fa79ba8f7.

@rogerchappel
rogerchappel merged commit 4e78158 into main Sep 1, 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.

1 participant