Skip to content

Conversation

@4geru
Copy link
Contributor

@4geru 4geru commented Sep 6, 2025

Summary

  • Add environment variable examples to the MCP Inspector command in both
    English and Japanese README files
  • Show how to properly pass CHANNEL_ACCESS_TOKEN and
    DESTINATION_USER_ID when using the inspector

Changes

  • Updated README.md to include -e flags for required environment
    variables
  • Updated README.ja.md with the same environment variable documentation
    in Japanese

Copy link
Contributor

@eucyt eucyt left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution. I've commented 🙏

Comment on lines 180 to 182
npx @modelcontextprotocol/inspector node dist/index.js
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN"
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
npx @modelcontextprotocol/inspector node dist/index.js
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN"
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"
npx @modelcontextprotocol/inspector node dist/index.js \
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN" \
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"

Comment on lines 186 to 188
npx @modelcontextprotocol/inspector node dist/index.js
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN"
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
npx @modelcontextprotocol/inspector node dist/index.js
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN"
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"
npx @modelcontextprotocol/inspector node dist/index.js \
-e CHANNEL_ACCESS_TOKEN="YOUR_CHANNEL_ACCESS_TOKEN" \
-e DESTINATION_USER_ID="YOUR_DESTINATION_USER_ID"

@4geru 4geru force-pushed the 4geru/add-inspector-env-variables branch from ae06e6d to a0b806a Compare October 22, 2025 00:15
@4geru
Copy link
Contributor Author

4geru commented Oct 22, 2025

@eucyt Sorry to late, please review it again.

Copy link
Contributor

@eucyt eucyt left a comment

Choose a reason for hiding this comment

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

Thank you!

@eucyt eucyt added this pull request to the merge queue Oct 22, 2025
Merged via the queue into line:main with commit 92d53f8 Oct 22, 2025
6 checks passed
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.

2 participants