fix: clarify that sandbox functions are only callable via execute_typescript#8096
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 29c50c39b8
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
dale-lakes
force-pushed
the
fix/clarify-execute-only-functions
branch
from
29c50c3 to
a886301
Compare
dale-lakes
changed the title
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a88630108e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Use is_first_class_extension() to filter the function list. Extensions with unprefixed_tools=true (Developer, Analyze) have their tools registered as direct top-level tools — listing them as sandbox-only would incorrectly suppress valid direct calls. Only functions from non-first-class extensions (Todo, Autovisualiser, Memory, etc.) are labeled as execute_typescript-only. Signed-off-by: Dale Lakes <6843636+spitfire55@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
dale-lakes
force-pushed
the
fix/clarify-execute-only-functions
branch
from
a886301 to
6b2b092
Compare
…escript (aaif-goose#8096) Signed-off-by: Dale Lakes <6843636+spitfire55@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
2 participants
The MOIM info-msg lists sandbox functions (Todo.todoWrite, Autovisualiser.renderMermaid, etc.) as "Available functions" with no indication they can't be called directly. Models attempt direct tool calls, which fail with
-32002: Tool not found.Uses
is_first_class_extension()to filter the function list. Extensions withunprefixed_tools=true(Developer, Analyze) have their tools registered as direct top-level tools and are excluded from the sandbox-only label. Only functions from non-first-class extensions are labeled as execute_typescript-only.