Skip to content

fix: remove list timeout regression and broken call command residue#220

Open
haosenwang1018 wants to merge 6 commits intophilschmid:mainfrom
haosenwang1018:fix/list-command-bun-timeout-and-config-help
Open

fix: remove list timeout regression and broken call command residue#220
haosenwang1018 wants to merge 6 commits intophilschmid:mainfrom
haosenwang1018:fix/list-command-bun-timeout-and-config-help

Conversation

@haosenwang1018
Copy link
Copy Markdown

What

  • remove a stray broken try block left in src/commands/call.ts
  • stop calling getInstructions() from list, which avoids unnecessary extra round-trips during cold starts
  • align the CLI help text with the actual config lookup precedence (-c/--config before MCP_CONFIG_PATH)

Why

This patch fixes two concrete problems and one docs/help mismatch:

  1. call.ts contained an invalid leftover block that breaks Bun parsing/runtime behavior.
  2. list was doing extra work that pushed the integration test over the timeout threshold in no-daemon cold-start scenarios.
  3. The built-in --help text still described config lookup precedence less precisely than the implementation and README.

Testing

  • bun test tests/integration/cli.test.ts tests/cli-errors.test.ts
  • result: 68 pass / 0 fail

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