docs(minimax-multimodal-toolkit): clarify sandbox env setup#55
docs(minimax-multimodal-toolkit): clarify sandbox env setup#55JithendraNara 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.
|
Fresh recheck on current I verified current
Given that, this PR looks functionally superseded by current |
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