Skip to content

Correcting the srt cli path in the published package.#289008

Merged
dileepyavan merged 1 commit intomainfrom
DileepY/srt_path
Jan 20, 2026
Merged

Correcting the srt cli path in the published package.#289008
dileepyavan merged 1 commit intomainfrom
DileepY/srt_path

Conversation

@dileepyavan
Copy link
Copy Markdown
Member

srt cli path is incorrect for the published package.

Copilot AI review requested due to automatic review settings January 20, 2026 03:43
@dileepyavan dileepyavan enabled auto-merge (squash) January 20, 2026 03:44
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 20, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Terminal sandbox runtime (srt) executable path so the correct CLI entrypoint is used when running from the published package layout.

Changes:

  • Switches _srtPath from node_modules/.bin/srt to the package’s actual bin target at @anthropic-ai/sandbox-runtime/dist/cli.js.
  • Adds an inline comment documenting the expected location of the srt CLI.

@dileepyavan dileepyavan merged commit acc954b into main Jan 20, 2026
27 of 28 checks passed
@dileepyavan dileepyavan deleted the DileepY/srt_path branch January 20, 2026 04:05
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants