[skip ci] fix: batch fixes for issues #1887, 1888, 1892, 1893, 1895, 1896, 1897, 1904, 1905, 1909#385
Open
[skip ci] fix: batch fixes for issues #1887, 1888, 1892, 1893, 1895, 1896, 1897, 1904, 1905, 1909#385
Conversation
…, 1904, 1905, 1909 [skip ci] Fixes: - #1887: mcp get/add/remove without args now shows 'Please provide a server name' instead of generic clap error - #1888: debug skill with directory path now shows 'Path is a directory, not a file' instead of 'Is a directory' error - #1892: -c with invalid config key now shows 'Unknown configuration key' error instead of silently ignoring - #1893: import - now reads from stdin instead of trying to read literal file named '-' - #1895: pr without number now attempts to detect PR for current branch using gh CLI - #1896: github run now checks GITHUB_TOKEN env var, allows --dry-run without token for read-only ops - #1897: pr without number now shows helpful usage message instead of clap error - #1904: debug without subcommand now shows help and exits 0 instead of exit 2 - #1905: github without subcommand now shows help and exits 0 instead of exit 2 - #1909: sandbox without subcommand now shows help and exits 0 instead of exit 2 Also fixes pre-existing build issues: - Added trust_proxy field to RateLimitConfig - Fixed duplicate output field in run_cmd.rs - Added cookies feature to reqwest - Made install_panic_hook public
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.
Summary
Batch fix for multiple bounty issues.
Issues Fixed
Changes
Pre-existing Build Fixes
trust_proxyfield to RateLimitConfig in cortex-app-serveroutputfield in run_cmd.rs (renamed to output_file)cookiesfeature to reqwest dependencyinstall_panic_hookfunction public