Skip to content

Conversation

@zanesq
Copy link
Collaborator

@zanesq zanesq commented Oct 21, 2025

Summary

Canary builds for win and linux are breaking from package upgrades with

An unhandled rejection has occurred inside Forge:
Error: 
1. Cannot find module @rollup/rollup-linux-x64-gnu. npm has a bug related to optional dependencies (https://github.com/npm/cli/issues/4828). Please try `npm i` again after removing both package-lock.json and node_modules directory.
2. Unexpected module status 5. Cannot require() ES Module /home/runner/work/goose/goose/ui/desktop/node_modules/vite/dist/node/index.js because it is not yet fully loaded. This may be caused by a race condition if the module is simultaneously dynamically import()-ed via Promise.all(). Try await-ing the import() sequentially in a loop instead. (from /home/runner/work/goose/goose/ui/desktop/node_modules/@electron-forge/plugin-vite/dist/VitePlugin.js)

Root cause is
npm 10.9.2 (current) has a bug with optional dependencies
npm 11.3.0+ fixed this bug
Node 22.17.1 ships with npm 10.9.2 by default

Rolling back for a quick fix for now until we can upgrade Node in CI

@zanesq zanesq merged commit 11e71ee into main Oct 21, 2025
14 checks passed
@zanesq zanesq deleted the zane/rollback-vite-electron-upgrades branch October 21, 2025 16:43
katzdave added a commit that referenced this pull request Oct 21, 2025
* 'main' of github.com:block/goose:
  roll back vite and electron package upgrades breaking canary win and linux (#5292)
  Revert "Revert "Rewrite extension management tools"" (#5273)
  improvement: propagate McpErrors directly into ToolCallResult (#5289)
wpfleger96 added a commit that referenced this pull request Oct 22, 2025
* main: (33 commits)
  Add Web Accessibility Auditor recipe to cookbook (#5318)
  To do mcp tutorial (#5317)
  workflows: add a manual trigger option to pr-smoke-test (#5302)
  documenting `goose recipe list` command (#5278)
  add a system prompt snapshot test (#5305)
  fix: handle non exist subrecipe path (#5287)
  Next camp (#5237)
  more lowercasing of TabItem labels (#5307)
  modified docs/tutorials/cicd Github Action's install path to follow download_cli script (#5240)
  Fix artifact download to work across workflow runs (#5304)
  Added extension search (#5283)
  docs: lowercase 'goose' in TabItem labels for consistency (#5297)
  feat(prompts): add format to save code snippet (#5007)
  fix: use Windows-compatible default path for CLI installation (#5221)
  feat: add Test Coverage Optimizer recipe (#5118) (#5272)
  Upgrade node to fix canary (#5301)
  Remove reliance on localstorage for pendingScheduleDeepLink when scheduling a recipe (#5290)
  Add historical tracking with trend indicators using artifacts (#5295)
  roll back vite and electron package upgrades breaking canary win and linux (#5292)
  Revert "Revert "Rewrite extension management tools"" (#5273)
  ...
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Oct 25, 2025
…linux (block#5292)

Signed-off-by: Blair Allan <Blairallan@icloud.com>
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Nov 29, 2025
…linux (block#5292)

Signed-off-by: Blair Allan <Blairallan@icloud.com>
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.

3 participants