Skip to content

feat(tool-script): add exec_command parameter schema with yield_time_ms and workdir - #2180

Merged
MiMoHardFather merged 1 commit into
mainfrom
feat/tool-script-exec-command-params
Aug 20, 2026
Merged

feat(tool-script): add exec_command parameter schema with yield_time_ms and workdir#2180
MiMoHardFather merged 1 commit into
mainfrom
feat/tool-script-exec-command-params

Conversation

@MiMoHardFather

Copy link
Copy Markdown
Collaborator

Summary

Define a typed ExecCommandParameters schema (cmd, yield_time_ms, workdir) so exec_command no longer leaks bash's internal parameter names to code-mode callers. The args are translated via execCommandArgs before dispatch.

Changes

  • Add ExecCommandParameters zod schema with cmd, yield_time_ms, and workdir fields
  • Add execCommandArgs() helper that maps { cmd, yield_time_ms, workdir }{ command, timeout, workdir, description }
  • Update generated type declarations to expose the new typed interface
  • Update tool catalog description for exec_command to document the new parameters
  • Update script guidance to reflect the new { cmd, yield_time_ms?, workdir? } interface
  • Add tests for parameter mapping and default yield_time_ms of 10000 ms

Test plan

  • exec_command maps to bash while direct bash remains backward compatible
  • exec_command defaults yield_time_ms to 10000 ms
  • Generated declarations use the new parameter schema
  • Typecheck passes

…ms and workdir

Define a typed ExecCommandParameters schema (cmd, yield_time_ms, workdir) so exec_command no longer leaks bash's internal parameter names to code-mode callers. The args are translated via execCommandArgs before dispatch. Also update the generated type declarations, tool catalog description, and script guidance to document the new interface.
@MiMoHardFather
MiMoHardFather merged commit fff7cff into main Aug 20, 2026
5 of 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.

1 participant