Skip to content

fix: match help topic names to CLI command names#263

Merged
maximelb merged 1 commit intocli-v2from
fix/help-topic-singular-plural
Mar 19, 2026
Merged

fix: match help topic names to CLI command names#263
maximelb merged 1 commit intocli-v2from
fix/help-topic-singular-plural

Conversation

@maximelb
Copy link
Copy Markdown
Contributor

Summary

  • Help topics were registered with plural names (e.g. outputs, sensors) but CLI commands use singular names (output, sensor), so limacharlie help output failed while limacharlie help outputs worked
  • Renamed all 7 mismatched topic keys to match their CLI command names: outputsoutput, sensorssensor, eventsevent, detectionsdetection, extensionsextension, adaptersadapter, downloadsdownload
  • Added singular/plural fallback in get_help_topic() and get_cheatsheet() so both forms resolve (e.g. limacharlie help output and limacharlie help outputs both work)

Test plan

  • Existing TestHelpTopicFallthrough tests pass
  • Verified both singular and plural forms resolve for all renamed topics
  • Manual: limacharlie help output, limacharlie help sensor, etc.

🤖 Generated with Claude Code

Help topics were registered with plural names (e.g. "outputs", "sensors")
but the CLI commands use singular names ("output", "sensor"), so
`limacharlie help output` failed while `limacharlie help outputs` worked.

Renamed topic keys to match their CLI commands and added singular/plural
fallback in get_help_topic() and get_cheatsheet() so both forms work.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@maximelb maximelb requested a review from steveatlc March 19, 2026 22:45
@maximelb maximelb marked this pull request as ready for review March 19, 2026 22:45
@maximelb maximelb requested a review from dzimine-lc March 19, 2026 22:46
@maximelb maximelb merged commit 1ab2967 into cli-v2 Mar 19, 2026
1 check passed
@maximelb maximelb deleted the fix/help-topic-singular-plural branch March 19, 2026 22:46
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