Skip to content

fix: drop node-gyp workarounds now that better-sqlite3 uses prebuilds - #5412

Closed
jonkoops wants to merge 2 commits into
redhat-developer:release-1.10from
jonkoops:fix/yarn-install-scripts-release-1.10
Closed

jonkoops wants to merge 2 commits into
redhat-developer:release-1.10from
jonkoops:fix/yarn-install-scripts-release-1.10

Conversation

@jonkoops

Copy link
Copy Markdown
Contributor

Backport of #5405 to release-1.10.

#5365 already bumped better-sqlite3 to v13 with prebuilds on this branch. This drops the leftover root node-gyp devDependency and dependenciesMeta, removes the e2e-tests postinstall Playwright hook (and --mode=skip-build in the e2e lint workflow), and updates docs to match.

release-1.10 does not have enableScripts: true in .yarnrc.yml or the cluster-free harness from main, so those parts of #5405 do not apply here.

RHDHBUGS-3725

Backport of redhat-developer#5405 to
release-1.10.

redhat-developer#5365 bumped better-sqlite3 to v13 with prebuilds; remove the root
node-gyp devDependency and dependenciesMeta, the e2e-tests postinstall
Playwright hook, and --mode=skip-build in CI. Update docs to document
the explicit playwright install step and drop node-gyp prerequisites.

Ref: RHDHBUGS-3725
Signed-off-by: Jon Koops <jonkoops@gmail.com>
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@github-actions

Copy link
Copy Markdown
Contributor

The container image build workflow finished with status: failure.

The root keytar built:false override is still required on release-1.10:
without it, yarn install in the offline hermetic build tries to compile
keytar and fails on missing libsecret-1.
@jonkoops

Copy link
Copy Markdown
Contributor Author

Closing: release-1.10 does not have enableScripts or cluster-free E2E, so #5405 does not fix a broken scenario here. The partial backport also regressed hermetic builds by removing keytar dependenciesMeta without an equivalent on main.

@jonkoops jonkoops closed this Sep 15, 2026
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

The container image build workflow finished with status: failure.

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