fix(tool): point DEFAULT_INTEGRATION_ID at Python Sandbox, not Slack (test) - #1027
Merged
Conversation
Tool.DEFAULT_INTEGRATION_ID was 686432941223092cb4294d3f, which is the Slack integration on both dev and prod. Any script tool created via aix.Tool(code=...) was silently connected to Slack: the agent payload carried the full SLACK_* action catalog, empty parameters, and the user's code never executed. Use 688779d8bfb8e46c273982ca (Python Sandbox, verified on dev and prod; same ID v1 ModelFactory.PYTHON_SANDBOX_ID already uses), add a unit test locking the script path to the sandbox integration, and fix test comments/names that mislabeled the Slack ID as the Script integration.
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
1 participant
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.
Clean cherry-pick of the fix in #1025 onto
test(replaces #1026, which pulled in unpromoted development commits).Tool.DEFAULT_INTEGRATION_IDpointed at the Slack integration (686432941223092cb4294d3f) instead of Python Sandbox —aix.Tool(code=...)connected user code to Slack, so agents carried the full SLACK_* action catalog and the script never ran. Switched to688779d8bfb8e46c273982ca(verified on dev and prod), with a unit test locking it in.🤖 Generated with Claude Code
https://claude.ai/code/session_01D7Zr94ozn8WzV5qv1B17EL