Skip to content

Refactor CLI options and fix TypeScript errors#3

Merged
mgreen-simplethread merged 2 commits into
mgreen-simplethread:mainfrom
mikedamage:bugfix/cli-typescript-errors
Jan 15, 2026
Merged

Refactor CLI options and fix TypeScript errors#3
mgreen-simplethread merged 2 commits into
mgreen-simplethread:mainfrom
mikedamage:bugfix/cli-typescript-errors

Conversation

@mikedamage
Copy link
Copy Markdown
Contributor

Summary

  • Convert bin/sundrop.js to TypeScript (bin/sundrop.ts)
  • Consolidate CLI options into typed options object for better type inference
  • Add demandOption and runtime validation for required --out flag
  • Fix type error in resolvePackage with non-null assertion
  • Add CLAUDE.md with project workflow instructions

Details

The CLI was refactored to use a centralized options object instead of chained yargs methods. This improves TypeScript type inference and eliminates the Promise | object union type issues that were causing compilation errors.

Test plan

  • bun run typecheck passes with no errors
  • All 71 unit tests pass
  • bun run format applied

🤖 Generated with Claude Code

mikedamage and others added 2 commits January 15, 2026 09:48
- Convert bin/sundrop.js to TypeScript (bin/sundrop.ts)
- Consolidate CLI options into typed options object for better type inference
- Add demandOption and runtime validation for required --out flag
- Fix type error in resolvePackage with non-null assertion
- Add CLAUDE.md with project workflow instructions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mgreen-simplethread mgreen-simplethread merged commit bf5e501 into mgreen-simplethread:main Jan 15, 2026
1 check 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.

2 participants