Skip to content

Conversation

dcs-soni
Copy link
Contributor

@dcs-soni dcs-soni commented Oct 8, 2025

Description

The VS Code task: vscode-extension:continue-ui:build is failing on Windows due to incorrect path resolution when combining absolute paths with the cwd option.

Here is the error:

 Error: Cannot find module 'C:\Users\[UserName]\Desktop...\continue\extensions\vscode\Users[UserName]Desktop...continue\extensions\vscode\scripts\prepackage.js'
      at Module._resolveFilename (node:internal/modules/cjs/loader:1186:15)
      at Module._load (node:internal/modules/cjs/loader:1012:27)
      at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:158:12)
      at node:internal/main/run_main_module:30:49 {
   code: 'MODULE_NOT_FOUND',
   requireStack: []

The path is duplicated and concatenated incorrectly, which thus causing this error. Windows path resolution attempts to combine the absolute path with the cwd, resulting in a malformed path. This blocks development on a Windows machine. This affects only windows users though the fix improves cross-platform consistency.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

After this fix:

  • Tested on Windows 11 - task now runs successfully, and the build is complete.

Summary by cubic

Fixed the VS Code build task on Windows by switching to a relative script path. This prevents malformed paths when combined with cwd and improves cross-platform consistency.

  • Bug Fixes
    • Changed task args to "scripts/prepackage.js" instead of "${workspaceFolder}/extensions/vscode/scripts/prepackage.js".

@dcs-soni dcs-soni requested a review from a team as a code owner October 8, 2025 06:39
@dcs-soni dcs-soni requested review from Patrick-Erichsen and removed request for a team October 8, 2025 06:39
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 8, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Copy link
Collaborator

@Patrick-Erichsen Patrick-Erichsen left a comment

Choose a reason for hiding this comment

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

Worked for me, thanks for the fix!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 13, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 13, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 33fc0f4 into continuedev:main Oct 13, 2025
51 of 57 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 13, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2025
@sestinj
Copy link
Contributor

sestinj commented Oct 14, 2025

🎉 This PR is included in version 1.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Oct 15, 2025

🎉 This PR is included in version 1.25.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Oct 15, 2025

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants