Repository navigation
io-cli 0.30.1 - #76
Merged
Merged
Conversation
0.30.0 documented `io skill add|list|remove` in the README and the CHANGELOG and shipped a working parse, a working plan and a working arm in the session - and a binary that answered `error: unrecognized subcommand 'skill'`. The argv door is the Subcommand enum in src/cli.rs and nothing had added `skill` to it. manage::parse's own surface list was missing it too, so even once clap routed the word the parse refused it with "is not a surface io manages". All 1,609 tests passed over this. Every one of them enters through manage::parse, and the routing that was missing sits in front of it. A subagent reported "no change needed in src/cli.rs" and that was accepted without checking, which is the failure this release documented repeatedly and then repeated. Found by running the published artifact. The post-release smoke is the last gate and the only one that sees the product the way a user meets it. The new gate asks clap itself which subcommands it will route and compares that against the surfaces manage::parse accepts, so the two cannot disagree again. Its first version was vacuous - the needle "io does not manage" does not appear in a refusal that reads "is not a surface io manages" - and is taken from the real bytes now. Verified end to end on the built binary: add writes the file and lists it as `yours`, remove deletes it, and the listing is empty of it afterwards.
io-cli 0.30.1
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
1 participant
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.
Release PR. Reviewed in #75.
io skilldid not exist. 0.30.0 documentedio skill add|list|removeand shipped a working parse, plan and session arm — and a binary that answerederror: unrecognized subcommand 'skill', because clap'sSubcommandenum never learned the word andmanage::parse's surface list hadn't either. All 1,609 tests passed over it; they all enter throughmanage::parse, which sits behind the missing routing. Found by running the published artifact.A test now asks clap itself which subcommands it routes and compares that against the surfaces
manage::parseaccepts.Gates: 99 binaries / 1,610 passed / 0 failed / 36 ignored;
clippy --all-targets -- -D warningsexit 0; fmt clean. Verified end to end on the built binary.