Skip to content

Conversation

@SamuelLHuber
Copy link
Contributor

@SamuelLHuber SamuelLHuber commented Jan 7, 2026

Summary

  • Fixed a bug where option values (e.g., --transformation my-value) were incorrectly detected as positional arguments, causing a misleading "Options must come before positional args" hint
  • Changed pipelines create to use positional name argument instead of --name option, unifying all create commands to the same pattern

Changes

File Change
src/lib/argGuards.ts Skip values for unknown options (not just those in optionPatterns)
src/commands/pipelines.ts Change --name option to positional arg
test/cli-usability.test.ts Add test for unknown option value skipping
README.md Update example syntax
CLAUDE.md Add development guidance
.changeset/fix-option-parsing.md Changeset for patch release

New unified command pattern

All create commands now follow:

indexingco <resource> create [options] <name>

Example:

# Before (inconsistent)
indexingco pipelines create --name my-pipeline --transformation foo ...

# After (consistent with filters/transformations)
indexingco pipelines create --transformation foo ... my-pipeline

Test plan

  • All 23 tests pass
  • Build succeeds
  • Manual test of pipelines create with new syntax

🤖 Generated with Claude Code

@SamuelLHuber SamuelLHuber force-pushed the fix/cli-option-parsing-and-arg-consistency branch 3 times, most recently from dbff202 to c5209f8 Compare January 7, 2026 14:20
- Fix argGuards to skip values for unknown options (not just those in optionPatterns)
- Change `pipelines create` to use positional `name` arg instead of `--name` option
- This unifies all `create` commands to use the same pattern:
  `indexingco <resource> create [options] <name>`
- Add test case for unknown option value skipping
- Update README with new syntax
- Add CLAUDE.md with development guidance

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@SamuelLHuber SamuelLHuber force-pushed the fix/cli-option-parsing-and-arg-consistency branch from c5209f8 to 36ba4b8 Compare January 7, 2026 14:27
@SamuelLHuber SamuelLHuber merged commit d0c5801 into master Jan 7, 2026
5 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.

2 participants