Skip to content

Update SQLite refresh script to support the SQLite monorepo#3452

Merged
JanJakes merged 2 commits intotrunkfrom
update-sqlite-refresh
Apr 1, 2026
Merged

Update SQLite refresh script to support the SQLite monorepo#3452
JanJakes merged 2 commits intotrunkfrom
update-sqlite-refresh

Conversation

@JanJakes
Copy link
Copy Markdown
Member

@JanJakes JanJakes commented Apr 1, 2026

Summary

  • The sqlite-database-integration repository was restructured into a monorepo (WordPress/sqlite-database-integration#334). The plugin now needs to be built from a subpackage.
  • The refresh script now shallow-clones the SQLite repository and runs its build script (bin/build-sqlite-plugin-zip.sh) to produce a self-contained plugin ZIP with the driver symlink resolved.
  • For older tagged versions (pre-monorepo), the script falls back to downloading the GitHub archive, which still works correctly.

Test plan

  • Verified npx nx run playground-wordpress-builds:bundle-sqlite-database succeeds for both trunk and v2.1.16
  • Verified the trunk ZIP contains the correct flat plugin structure with db.copy, load.php, and a real wp-includes/database/ directory
  • Verified the v2.1.16 ZIP is identical to the previous version (84 KB)
  • Verify Playground boots correctly with the new trunk ZIP (run npm run dev and confirm SQLite works)

@JanJakes JanJakes requested review from a team, bgrgicak and Copilot April 1, 2026 11:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the SQLite integration plugin refresh flow to work with the upstream sqlite-database-integration monorepo by building a ZIP from a shallow clone, while keeping a fallback path for older (pre-monorepo) releases.

Changes:

  • Switch refresh script from downloading GitHub archives to shallow-cloning and running the upstream build script when available
  • Add fallback to archive download for older tags that don’t contain the build script
  • Update the expected ZIP size for trunk in module details

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

File Description
packages/playground/wordpress-builds/src/sqlite-database-integration/get-sqlite-driver-module-details.ts Updates expected size metadata for the trunk ZIP
packages/playground/wordpress-builds/build/refresh-sqlite-integration-plugin.js Implements clone+build flow for monorepo versions with a legacy archive-download fallback

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JanJakes JanJakes removed the request for review from bgrgicak April 1, 2026 11:29
JanJakes added 2 commits April 1, 2026 14:01
Done via: npx nx run playground-wordpress-builds:bundle-sqlite-database
@JanJakes JanJakes force-pushed the update-sqlite-refresh branch from a327517 to 6ee2478 Compare April 1, 2026 12:13
@JanJakes
Copy link
Copy Markdown
Member Author

JanJakes commented Apr 1, 2026

I'll merge this one to re-enable the SQLite refresh job.

@JanJakes JanJakes merged commit 9255570 into trunk Apr 1, 2026
137 of 139 checks passed
@JanJakes JanJakes deleted the update-sqlite-refresh branch April 1, 2026 14:48
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.

2 participants