Repository navigation
Fit the help in 80 columns and fill gaps in the docs - #34
Merged
Merged
Conversation
Options with long descriptions, like --tool, --session and mcp's --timeout, ran past 80 columns, up to 110, and wrapped wherever the terminal broke them. The summary wraps each description under itself now, and a test checks every command's help, the inspect usage line included, which got split in two.
neorepl complete printed nothing and exited 0 for a prefix with no completions, where apropos, doc and the other lookups exit 1 when they find nothing, and neither its help nor macroexpand's had an exit status paragraph. complete reports not-found the way the lookups do now, in every format, and a server with no completion op says so rather than passing for one with nothing to complete. Both helps say what the statuses mean.
The README listed the full-screen inspector's keys but left out P, which turns pretty printing on and off, and g, which shows the value again, while the inspector's own help line has them, and the arrows that do what Enter and Backspace do. It also said neorepl inspect does the same, which it does for pushing, popping, paging and defining only.
The skill didn't say which server the commands use when there are several, or that -p picks another, nor what happens to the agent session when the server restarts, usually on another port, and it gave exit 1 as the code, a test or check failing only, where a lookup finding nothing and a server failing to start exit 1 too.
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.
The help wraps option descriptions to fit in 80 columns, with a test over every command, and
completeexits 1 when nothing completes, like the other lookups. The README gets the inspector'sPandg, and the skill covers several servers, restarts and what exit 1 means.complete --format jsonwith nothing to complete returns anot-foundobject now rather than[], matchingapropos.