Skip to content

fix(deps): update open package to resolve browser launch on KDE Plasma 6#242

Open
YatWoirat wants to merge 1 commit intocodemie-ai:mainfrom
YatWoirat:fix/sso-linux-browser
Open

fix(deps): update open package to resolve browser launch on KDE Plasma 6#242
YatWoirat wants to merge 1 commit intocodemie-ai:mainfrom
YatWoirat:fix/sso-linux-browser

Conversation

@YatWoirat
Copy link
Copy Markdown

Summary

Updates the open package from v8.4.2 to v10.2.0 to resolve a silent failure during browser-based authentication flows on Linux with KDE Plasma 6.

Motivation

When using codemie setup with the SSO provider on Linux with KDE Plasma 6, the browser failed to open, causing the setup process to hang and eventually timeout.

This occurs because open@8.4.2 relies on an outdated bundled xdg-open script that explicitly checks for KDE_SESSION_VERSION 4 or 5. Since Plasma 6 sets this variable to 6, the switch statement had no match. The script executed nothing, exited with 0 (success), causing the silent failure.

Changes

  • Bumped open from ^8.4.2 to ^10.2.0 in package.json.
  • open v10 updates the internal xdg-open utility to v1.2.1, as well as includes other Linux/macOS fixes.

Testing

  • Ran npm run test:run locally to ensure no tests are broken.
  • Tested codemie setup locally on a KDE Plasma 6 Wayland system; verified the browser opens correctly.
  • Confirmed open v10.x is fully compatible with Node >= 20.0.0 and pure ESM.
  • Ran npm run ci:full.

Related Issues

Relates to #203
Relates to #205

The open package in v8.4.2 relies on an outdated bundled xdg-open script
that lacks a case for KDE_SESSION_VERSION=6, causing it to fail silently
on KDE Plasma 6 during SSO authentication.
Bumping to v10.2.0 updates the internal xdg-open utility to v1.2.1,
which natively supports Plasma 6. This update also includes broader Linux
compatibility fixes from the upstream package that may help with similar
issues on other setups.

Relates to codemie-ai#203
Relates to codemie-ai#205
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.

1 participant