Skip to content

Conversation

@wojtekn
Copy link
Collaborator

@wojtekn wojtekn commented Nov 21, 2025

Motivation for the change, related issues

Fixes logging in the CLI blueprints handler to display "Skipping SQLite integration plugin setup..." when --skip-sqlite-setup is used, instead of always showing "Fetching SQLite integration plugin..." regardless of the skip option.

Implementation details

Modified packages/playground/cli/src/blueprints-v1/blueprints-v1-handler.ts to conditionally log the appropriate message based on skipSqliteSetup.

Testing Instructions (or ideally a Blueprint)

To test this change:

  1. Set up Playground repository locally
  2. Run the CLI with --skip-sqlite-setup flag and verify the log shows "Skipping..." message
  3. Run the CLI without the flag and verify it shows "Fetching..." message

Example commands:

# Should show "Skipping SQLite integration plugin setup..."
node --no-warnings=ExperimentalWarning  --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts ./packages/playground/cli/src/cli.ts server --skip-sqlite-setup

# Should show "Fetching SQLite integration plugin..."
node --no-warnings=ExperimentalWarning  --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts ./packages/playground/cli/src/cli.ts server

@wojtekn wojtekn requested review from adamziel and bgrgicak November 21, 2025 10:14
@wojtekn wojtekn self-assigned this Nov 21, 2025
@adamziel adamziel merged commit b24ed74 into WordPress:trunk Nov 21, 2025
32 checks passed
@adamziel
Copy link
Collaborator

That's great, thank you!

@wojtekn wojtekn deleted the fix/sqlite-plugin-fetching-log-message branch November 21, 2025 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants