docs(minimax-multimodal-toolkit): clarify sandbox env setup#55
Open
JithendraNara wants to merge 1 commit intoMiniMax-AI:mainfrom
Open
docs(minimax-multimodal-toolkit): clarify sandbox env setup#55JithendraNara wants to merge 1 commit intoMiniMax-AI:mainfrom
JithendraNara wants to merge 1 commit intoMiniMax-AI:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Improves the MiniMax multimodal skill setup guidance to account for sandboxed AI runtimes where updating shell profiles doesn’t affect already-running IDE/agent sessions, and updates the environment check script to print actionable sandbox-aware remediation.
Changes:
- Added “sandboxed agent” guidance to
SKILL.mdfor bothMINIMAX_API_HOSTandMINIMAX_API_KEYsetup. - Updated the “missing key/host” remediation steps to explicitly mention restarting the IDE/agent or configuring env vars in tool-specific settings.
- Enhanced
scripts/check_environment.shfailure messages to include sandbox-aware troubleshooting (restart / tool env settings).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| skills/minimax-multimodal-toolkit/SKILL.md | Adds sandbox-aware setup notes and remediation steps for API host/key configuration. |
| skills/minimax-multimodal-toolkit/scripts/check_environment.sh | Improves missing env var failure output to guide users in sandboxed runtimes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
What
minimax-multimodal-toolkitthat exportingMINIMAX_API_KEYorMINIMAX_API_HOSTto~/.zshrc/~/.bashrcmay not affect an already-running sandboxed agent sessionscripts/check_environment.shprint sandbox-aware remediation instead of only suggestingexport ...Why
Issue #41 reports a real usability gap: users can follow the current instructions, save
MINIMAX_API_KEYto their shell profile, and still see the skill fail inside a sandboxed agent runtime because the running app never inherited the new environment.This PR is scoped to one documentation/runtime-guidance improvement in
minimax-multimodal-toolkit: make the missing-key path actionable for sandboxed AI tools.Fixes #41