KofTwentyTwo/license-tool
2
0 Fork 0
Code Issues 2 Pull Requests 9 Actions Packages Projects Releases 6 Wiki Activity

cmd: --quiet/--verbose are bound but never read (no-op flags) #34

New Issue
Closed
opened 2026-06-08 16:38:28 +00:00 by KofTwentyTwo · 0 comments
KofTwentyTwo commented 2026-06-08 16:38:28 +00:00 (Migrated from github.com)
Copy link
Copy source

Found during full-codebase review (see docs/review/03-cmd-tui-tests.md).

[MED] cmd/license-tool/commands.go: --quiet/-q and --verbose/-v are bound on the persistent flag set but read nowhere — pure no-op flags that imply behavior they do not deliver.

Fix: either wire them into output verbosity / resolver tracing, or remove them until implemented.

Related [LOW]: isWriteRefusal distinguishes exit 3 vs 4 by strings.Contains on error wording — brittle coupling to applier/config; prefer a typed sentinel error.

Found during full-codebase review (see docs/review/03-cmd-tui-tests.md). **[MED]** `cmd/license-tool/commands.go`: `--quiet`/`-q` and `--verbose`/`-v` are bound on the persistent flag set but read nowhere — pure no-op flags that imply behavior they do not deliver. Fix: either wire them into output verbosity / resolver tracing, or remove them until implemented. Related **[LOW]**: `isWriteRefusal` distinguishes exit 3 vs 4 by `strings.Contains` on error wording — brittle coupling to applier/config; prefer a typed sentinel error.
Sign in to join this conversation.
No Branch/Tag Specified
Labels
Clear labels
bug
Something isn't working
 dependencies
Pull requests that update a dependency file
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 github_actions
Pull requests that update GitHub Actions code
 go
Pull requests that update go code
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No labels bug
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
forge-recovery (Forge Recovery) james.maes (James Maes)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/license-tool#34