Skip to content

Stop stats after missing transcript#71

Open
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/stop-stats-after-missing-transcript
Open

Stop stats after missing transcript#71
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/stop-stats-after-missing-transcript

Conversation

@w287346141
Copy link
Copy Markdown

Summary

  • return immediately after reporting a missing transcript path in the stats command
  • add a regression test that mocks process.exit and verifies the command does not continue into readFileSync

Verification

  • npm.cmd test -- --run tests/stats-command.test.ts
  • npm.cmd run lint -- src/cli/commands/stats.ts tests/stats-command.test.ts
  • npm.cmd run typecheck
  • npm run verify (pre-push hook)

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