Skip to content

Comments

feat: auto-register Complete<FieldName> flag completions#50

Merged
leodido merged 5 commits intomainfrom
codex/complete-hooks
Feb 23, 2026
Merged

feat: auto-register Complete<FieldName> flag completions#50
leodido merged 5 commits intomainfrom
codex/complete-hooks

Conversation

@leodido
Copy link
Owner

@leodido leodido commented Feb 21, 2026

Summary

  • auto-discover and auto-register optional Complete<FieldName> hooks during Define() for generated flags
  • validate completion hook signatures and return a dedicated validation error for invalid signatures
  • add coverage for completion registration and validation behavior
  • document the new hook pattern and update the full example to use CompleteTargetEnv

Validation

  • go test ./...
  • cd examples/full && go test ./...

@leodido leodido changed the title feat: auto-register Complete<FieldName> flag completions feat: auto-register Complete<FieldName> flag completions Feb 21, 2026
@leodido leodido force-pushed the codex/complete-hooks branch from 20578d7 to 0b110ce Compare February 21, 2026 17:47
@leodido leodido self-assigned this Feb 22, 2026
@leodido leodido merged commit 941a4a9 into main Feb 23, 2026
4 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