Skip to content

fix(cli): reject conflicting output modes before I/O - #67

Merged
yusukebe merged 1 commit into
yusukebe:mainfrom
yearth:codex/reject-conflicting-output-modes
Aug 5, 2026
Merged

fix(cli): reject conflicting output modes before I/O#67
yusukebe merged 1 commit into
yusukebe:mainfrom
yearth:codex/reject-conflicting-output-modes

Conversation

@yearth

@yearth yearth commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reject conflicting output modes before source I/O
  • preserve supported JSON and TSV modifier combinations
  • prevent string-valued output flags from swallowing a following output flag

Tests

  • bunx tsc --noEmit
  • bunx oxfmt --check .
  • bun test --timeout 30000 (275 pass)
  • bun run build

yusukebe
yusukebe approved these changes Aug 5, 2026
View reviewed changes

@yusukebe yusukebe left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@yusukebe

yusukebe commented Aug 5, 2026

Copy link
Copy Markdown
Owner

@yearth Thanks!

@yusukebe
yusukebe merged commit d725f8f into yusukebe:main Aug 5, 2026
3 checks passed
@kiakiraki kiakiraki mentioned this pull request Aug 8, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yusukebe yusukebe yusukebe approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yearth @yusukebe