Skip to content

Bump base VSCode version for Quick Edit & Playground to 1.102.1#10047

Merged
penalosa merged 18 commits intomainfrom
penalosa/bump-vscode
Aug 11, 2025
Merged

Bump base VSCode version for Quick Edit & Playground to 1.102.1#10047
penalosa merged 18 commits intomainfrom
penalosa/bump-vscode

Conversation

@penalosa
Copy link
Contributor

@penalosa penalosa commented Jul 23, 2025

Bump the version of VSCode upon which we base our build of VSCode for Web, which is used in Quick Edit in the Cloudflare Dashboard as well as the Workers Playground.

How to test

  • Checkout this PR
  • Run pnpm -F "@cloudflare/quick-edit" run setup
  • Run pnpm -F "@cloudflare/quick-edit" run custom:build

    Warning, this takes about 15 minutes

  • Change packages/workers-playground/src/QuickEditor/VSCodeEditor.tsx#L18 to http://localhost:8787
  • Run (in parallel) pnpm -F workers-playground dev and pnpm -F "@cloudflare/quick-edit" exec wrangler dev
  • Open http://localhost:5173/playground and play around with the editor

  • Tests
    • Tests included
    • Tests not necessary because: This package needs to be manually tested
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: no user-facing changes
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: not a wrangler change

@penalosa penalosa requested a review from a team as a code owner July 23, 2025 13:28
@changeset-bot
Copy link

changeset-bot bot commented Jul 23, 2025

🦋 Changeset detected

Latest commit: c441fa0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudflare/quick-edit Minor

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jul 23, 2025

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@10047

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@10047

miniflare

npm i https://pkg.pr.new/miniflare@10047

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@10047

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@10047

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@10047

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@10047

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@10047

wrangler

npm i https://pkg.pr.new/wrangler@10047

commit: 45df43c

@penalosa penalosa added the preview:quick-edit Trigger a quick edit preview deploy label Jul 23, 2025
Comment on lines +47 to +48
with:
node-version: 22
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is required by VSCode's build process

readOnly: files.readOnly,
});
await this.writeFile(
Uri.parse(`cfs:/${this.rootFolder}/${path}`),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was previously missing the cfs:/ scheme. It worked in the older version of VSCode, but not in the newest. This took hours of stepping through compiled VSCode sources to find 😅

if (this.readRoot !== null) {
await this.readRoot(
await this.readDirectory(Uri.parse(`${this.rootFolder}/`))
await this.readDirectory(Uri.parse(`cfs:/${this.rootFolder}/`))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

As above

*
* Tips for matching in Quick Open:
* With the pattern, the picker will use the file name and file paths to score each entry. The score will determine the ordering and filtering.
* The scoring prioritizes prefix and substring matching. Then, it checks and it checks whether the pattern's letters appear in the same order as in the target (file name and path).
Copy link
Contributor

Choose a reason for hiding this comment

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

it checks and it checks

?

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Aug 8, 2025
@petebacondarwin
Copy link
Contributor

Also built and tested it locally. Seems fine

@penalosa penalosa force-pushed the penalosa/bump-vscode branch from 9c93b12 to 2a245ca Compare August 8, 2025 14:43
penalosa and others added 8 commits August 8, 2025 15:43
- Bumps quick-edit-extension and @cloudflare/quick-edit packages
- Updates VSCode base version with necessary compatibility fixes
- Consolidates patches for better maintainability

Co-Authored-By: pbacondarwin@cloudflare.com <pete@bacondarwin.com>
The quick-edit-extension package is private without a deploy script,
making it invalid for changesets. Only @cloudflare/quick-edit needs
to be versioned as it's the deployable package.

Co-Authored-By: pbacondarwin@cloudflare.com <pete@bacondarwin.com>
@penalosa penalosa force-pushed the penalosa/bump-vscode branch from 2a245ca to d9e3b8c Compare August 8, 2025 14:47
@petebacondarwin
Copy link
Contributor

@penalosa can you look into why the Deploy Pages Previews job is failing. Is it because this is now a Worker?

@penalosa
Copy link
Contributor Author

@penalosa can you look into why the Deploy Pages Previews job is failing. Is it because this is now a Worker?

To be honest, I'm now a bit stumped. The script isn't failing, it seems to be getting a SIGTERM from GitHub Actions. I thought it might be memory limit related and so bumped that, but that hasn't fixed it. Do you have any ideas?

@penalosa penalosa merged commit a4c4c5b into main Aug 11, 2025
59 of 76 checks passed
@penalosa penalosa deleted the penalosa/bump-vscode branch August 11, 2025 16:02
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview:quick-edit Trigger a quick edit preview deploy

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants