Skip to content

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

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

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

Conversation

@leodido
Copy link
Copy Markdown
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
Comment thread README.md Outdated
@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

2 participants