Skip to content

Conversation

@office-exsitu
Copy link

Summary of changes

  • Apply repository formatting standards across JS/JSON and configs using npm run format (wp-scripts)
  • Attempt PHP code style fixes via npm run lint:php:fix (skipped: phpcbf not available in this environment)
  • Add WARP.md with development workflow and common commands for Warp terminal users
  • Keep package/composer manifests and lockfiles in sync and formatted

Technical implementation details

  • Ran npm ci to ensure local formatter availability
  • Executed npm run format (wp-scripts) which formatted:
    • .wp-env.json, package.json, package-lock.json, pnpm-lock.yaml
    • src/settings/* files (JS)
    • JSON schemas in includes/schemas/*
  • PHP auto-fix step attempted via npm run lint:php:fix but phpcbf is not installed on this machine; no PHP auto-fixes were applied
  • Staged and committed all resulting changes in a single commit

Diff summary (vs origin/trunk)

11 files changed, 44294 insertions(+), 39191 deletions(-)

Key files touched:

  • .wp-env.json
  • WARP.md (new)
  • composer.json, composer.lock
  • includes/schemas/mcp-2025-06-18.json
  • includes/schemas/wordpress-mcp-extensions.json
  • package.json, package-lock.json, pnpm-lock.yaml
  • src/settings/AuthenticationTokensTab.js, src/settings/DocumentationTab.js

Testing steps

  1. Install dependencies
    • composer install
    • npm install
  2. Build or start the admin UI (optional)
    • npm run start (dev) or npm run build
  3. Run PHP unit tests
    • vendor/bin/phpunit --testsuite wordpress-mcp
  4. Sanity check admin screen:
    • Open WP Admin → Settings → WordPress MCP and ensure UI loads without errors

Related issues/tickets

  • N/A (please link any relevant issues)

- Apply wp-scripts format across JS/JSON and configs

- Attempted PHP code style fixes via npm run lint:php:fix (phpcbf not available in this environment)

- Add WARP.md with repository development workflow and commands (Warp guidance)

- Keep package.json/composer.json and lockfiles formatted to match project conventions
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