Skip to content

feat - incorporated mcp server request configs for cli mode #373

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
+380 −12
Open

feat - incorporated mcp server request configs for cli mode #373

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
1aaf641
feat - incorporated mcp server request configs for cli mode
Kavyapriya-1804 May 3, 2025
b38b6fa
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 4, 2025
b1110c4
Merge branch 'main' into feat-cli-server-request-configs
KavyapriyaJG May 10, 2025
90dc9d0
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 11, 2025
a76776f
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 12, 2025
a4d2ab2
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 19, 2025
003ea14
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 28, 2025
bc62929
Merge remote-tracking branch 'upstream/main' into feat-cli-server-req…
Kavyapriya-1804 May 28, 2025
0ffcd81
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 May 30, 2025
470db15
Merge branch 'main' into feat-cli-server-request-configs
cliffhall May 30, 2025
79b1b10
Merge remote-tracking branch 'upstream/main' into feat-cli-server-req…
Kavyapriya-1804 Jun 3, 2025
bc470f6
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 15, 2025
d8a7534
feat - Added test scripts for the CLI server configs
Kavyapriya-1804 Jun 15, 2025
bfa29e7
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 18, 2025
cecbc77
feat - Moved timeout tests to cli tests
Kavyapriya-1804 Jun 18, 2025
3940f6b
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 19, 2025
cbe58f9
fix - removed test logs
Kavyapriya-1804 Jun 21, 2025
c97b5c9
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 21, 2025
02a0e3c
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 23, 2025
5fb9790
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jun 25, 2025
2ef0c0f
Merge from main
olaservo Jun 29, 2025
e003134
Merge branch 'main' into feat-cli-server-request-configs
olaservo Jun 29, 2025
ff846b6
Fix formatting
olaservo Jun 29, 2025
302257b
Update cli-tests.js
olaservo Jun 29, 2025
d797947
Merge branch 'feat-cli-server-request-configs' of https://github.com/…
olaservo Jun 30, 2025
f4f4779
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jul 5, 2025
72ac117
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jul 7, 2025
94cd40a
Merge branch 'main' into feat-cli-server-request-configs
cliffhall Jul 7, 2025
ea6af4d
Merge branch 'main' into feat-cli-server-request-configs
Kavyapriya-1804 Jul 14, 2025
29947ea
feat: updated documentation for CLI req timeout configs
Kavyapriya-1804 Jul 14, 2025
470b19d
Merge branch 'main' into feat-cli-server-request-configs
olaservo Jul 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -299,6 +299,15 @@ npx @modelcontextprotocol/inspector --cli node build/index.js
# With config file
npx @modelcontextprotocol/inspector --cli --config path/to/config.json --server myserver

# With request timeout configs (in milliseconds) - request timeout
npx modelcontextprotocol/inspector --cli node build/index.js --method tools/call --tool-name longRunningOperation --tool-arg duration=5 steps=5 --request-timeout 15000

# With request timeout configs (in milliseconds) - reset timeout on progress
npx modelcontextprotocol/inspector --cli node build/index.js --method tools/call --tool-name longRunningOperation --tool-arg duration=15 steps=5 --reset-timeout-on-progress true

# With request timeout configs (in milliseconds) - max total timeout
npx modelcontextprotocol/inspector --cli node build/index.js --method tools/call --tool-name longRunningOperation --tool-arg duration=15 steps=5 --max-total-timeout 30000

Comment on lines +308 to +310
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--max-total-timeout really only makes sense when combined with --request-timeout and --reset-timeout-on-progress

Suggested change
# With request timeout configs (in milliseconds) - max total timeout
npx modelcontextprotocol/inspector --cli node build/index.js --method tools/call --tool-name longRunningOperation --tool-arg duration=15 steps=5 --max-total-timeout 30000
# With request timeout configs (in milliseconds) - max total timeout (used with request timeout and reset timeout on progress)
npx modelcontextprotocol/inspector --cli node build/index.js --method tools/call --tool-name longRunningOperation --tool-arg duration=35 steps=5 --reset-timeout-on-progress true --request-timeout 7000 --max-total-timeout 35000

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. This can be done.

# List available tools
npx @modelcontextprotocol/inspector --cli node build/index.js --method tools/list

Expand Down
Loading