Repository navigation
docs(cli): sync interactive menu options and introduce CLI history (#432) - #433
Merged
rosspeili merged 2 commits intoOct 14, 2026
Merged
Conversation
…ble (ARPAHLS#432) Synchronize interactive splash menu commands in docs/usage/cli.md with current options (adding 9 / evm and 10 / addressbook). Add anchor links to command groups and commands for seamless cross-doc referencing. Update _HELP_MENU and HELP_GROUPS in skillware/cli.py to include EVM and addressbook topics, correcting option indices and range notices (1-10). Append verified CLI provenance history table documenting 25 key commits from v0.3.5 through v0.4.7 with accurate commit hashes, dates, and authors. Update docs/usage/README.md and CHANGELOG.md. Add regression tests in tests/test_cli.py.
rosspeili
requested changes
Oct 14, 2026
rosspeili
left a comment
Contributor
There was a problem hiding this comment.
Thanks @rin-katsuragi! The code fix in skillware/cli.py swapping EVM/Addressbook to align _HELP_MENU and the updated menu tables in docs/usage/cli.md look great and good thinking ahead.
Before we can merge, we need to polish the CLI history table:
- Table Schema Consistency: Align the columns with the established standard used across all
docs/skills/*/*.mdhistory tables:
| Commit | Description | Date | Version | Contributors | - Contributor Attribution & Links: Several commits were authored by community contributors but are currently attributed to Ross. Also, contributors must be linked to their GitHub profiles (
[@handle](https://github.com/handle)):5e524a1,5b68b78,0bd2410,dd90896,04a3d69were authored by @rizzoMartin.93d0d9ewas authored by @uttam12331.2f20e20was authored by @blade-dancer.- Link @harshthakur3 for
5fcf76a.
- Commit Date & Test Assertion: Commit
dd90896is14 Jun 2026(not 31 May). Please update both the doc row and the test assertion intests/test_cli.py(assert "14 Jun 2026" in cli_md).
Contributor
Author
|
Thanks for the review and guidance @rosspeili! I am so sorry for messing that up. I updated with a new commit. Hope this is better now. |
Contributor
|
Nw at all @rin-katsuragi, this LGTM. Merging as is. Well done <3 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Synchronizes
docs/usage/cli.mdwith the current interactive splash menu options and grouped help topics, addresses underlying indexing drift inskillware/cli.py, and introduces the CLI provenance history table.9(evm) and10(addressbook), updating option6(help) to reference both new command groups.docs/usage/cli.mdand addedskillware addressbooktodocs/usage/README.md.HELP_GROUPSand_HELP_MENUinskillware/cli.pyto 14 distinct topics, wiringevmandaddressbookinto the help submenu and correcting option range notices from1-8to1-10.dd90896date to 31 May 2026).tests/test_cli.pycovering help topic mapping integrity and CLI docs synchronization.Type of Change
skills/skills/skillware/core/loader, env, adaptersskillware/cli.py,docs/usage/cli.mdexamples/*.py, agent loops,examples/README.mdpyproject.toml,MANIFEST.inChecklist (all PRs)
Fixes #432)python -m black --check .,flake8, andruff check .pass locally (or CI-equivalent subset)pytest skills/andpytest tests/pass locally when relevantCHANGELOG.mdupdated under[Unreleased]when user-visible behavior changesexamples/README.mdupdated if this PR adds, renames, or removes a runnable scriptpytest tests/test_registry_docs.pywhen skills, examples index, or agent-loops matrix changedpytest tests/test_skill_docs.pywhen catalog Usage Examples or provider snippets changedRelated Issues
Fixes #432