Repository navigation
Show the program's name in help output - #119
Conversation
app.HelpName was set to '-', so every help screen called the command '-': 'NAME: - - Standard Notes CLI', 'USAGE: - [global options]', and the same for each subcommand's help. The README's first instruction is sn --help, so this was the first thing a new user saw. Removing the line lets urfave/cli default HelpName to app.Name, which is already 'sn'. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The change is minimal, scoped to help output naming, and no remaining references or behavioral risks were found in the updated code.
Review effort: Lite
Findings: None
What changed in this PR
Removes an explicit app.HelpName = "-" override so urfave/cli help output uses the application name (app.Name, already set to "sn") rather than displaying - for the program and subcommands.
Changes:
- Deleted the
app.HelpName = "-"assignment to allow the default help naming behavior. - As a result,
sn --helpand subcommand help (sn add --help, etc.) display the correct program name.
| File | Description |
|---|---|
| cmd/sncli/main.go | Removes the HelpName override so help output reflects app.Name (sn). |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Not up to standards ⛔🟢 Coverage
|
| Metric | Results |
|---|---|
| Coverage variation | ✅ +0.00% coverage variation (-1.00%) |
| Diff coverage | ✅ ∅ diff coverage |
Coverage variation details
Coverable lines Covered lines Coverage Common ancestor commit (10a5e38) 8826 899 10.19% Head commit (7119ab8) 8825 (-1) 899 (+0) 10.19% (+0.00%) Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch:
<coverage of head commit> - <coverage of common ancestor commit>
Diff coverage details
Coverable lines Covered lines Diff coverage Pull request (#119) 0 0 ∅ (not applicable) Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified:
<covered lines added or modified>/<coverable lines added or modified> * 100%
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
app.HelpNamewas set to"-", so help called the command-everywhere:Subcommands inherited it, so
sn add --helpread- add - add items. The README tells a new user to runsn --helpas their first step, which made this the first thing they saw.Deleting the line lets urfave/cli default
HelpNametoapp.Name, which is already"sn":and
sn add --helpnow readssn add - add items.Nothing referenced
HelpNameanywhere else.go build,go vetandgo test ./...pass; verified by building both versions and comparing their output.🤖 Generated with Claude Code