carapace-sh / carapace / 30113885906

61%

master: 61%

Pull #1227

github

fix: gate flag-argument completion on AcceptsNext and clean up code

- Fix failing TestAcceptAttached: traverse.go now checks AcceptsNext()
  before completing a flag argument in the next-arg position, so flags
  with AcceptAttached-only style no longer incorrectly offer value
  completions when the user types `-3 <TAB>`.
- Replace magic numbers in internalActions.go with named pflagfork
  constants (NameAsShorthand, Default) via GetMode().
- Revert go.mod/example-nonposix go.mod local replace directives back
  to versioned carapace-pflag; keep local override in go.work only.
- Add named constants for ArgumentStyle bitmask values in pflagfork.
- Simplify lookupPosixShorthandArg variable names and remove redundant
  hasNext logic.
- Clarify comments in traverse.go and lookupPosixLonghandArg.

Assisted-by: Crush:glm-5.2

93 of 105 new or added lines in 6 files covered. (88.57%)

3905 of 6379 relevant lines covered (61.22%)

1.03 hits per line