Repository navigation
Conversation
Move every @cloudflare/computer dependency, and Think's peer range, to
^0.10.0. Computer 0.9 renamed the host module prefix from ws: to
codemode:, and 0.10 renames WorkerJavaScriptBackend to CodemodeBackend
(@cloudflare/computer/backends/codemode), changes its default id to
"codemode", and reshapes the tool sets' exec option to
{ backends, output, metadata }.
- Import CodemodeBackend from @cloudflare/computer/backends/codemode.
- Every CodemodeBackend uses the default "codemode" id instead of
"js" or "javascript", and backend references follow.
- Rename ws:git and ws:tools to codemode:git and codemode:tools in
backend modules, isolate imports, tool descriptions and docs.
- The pi and opencode harness examples pass exec: { backends } instead
of the deprecated shell option.
- Add a Think changeset for the new peer range.
Point every @cloudflare/computer dependency at the pkg.pr.new preview of cloudflare/computer#305 through a root pnpm override, until 0.10.0 is published.
🦋 Changeset detectedLatest commit: e54e610 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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
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.
Computer 0.10 renames the JavaScript backend and its host modules, so Think, its docs and the examples need to follow before they can move to it. This upgrades every
@cloudflare/computerdependency, and Think's peer range, to^0.10.0.Draft until Computer 0.10.0 is published. The top commit points everything at the preview build of cloudflare/computer#305 so CI can run now, and gets dropped before merging.
Summary
"codemode"instead of setting"js"or"javascript", and backend references follow.exec: { backends }instead of the deprecatedshelloption.^0.10.0.Evidence
With the preview build:
pnpm run build,pnpm run check, andnx affected -t testall pass (27 projects, including Think's workspace tools tests running code on the"codemode"backend).Merge Danger
Door: two-way
Blast Radius: Think users
Think's peer range moves to
^0.10.0, so upgrading Think means upgrading Computer, and their code needs the same renames.Before merging
TEMPcommit.pnpm installand commit the lockfile.