CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

Fix CI: package-lock.json, WPCS step, @wordpress/scripts upgrade #3944

Merged⚡ AI-generatedXLccantynz wants to mergeclaude/tradequote-ai-plugin-spec-ADZYGmainopened May 30, 20260/4 tasks
ccantynzcommented May 30, 2026

Originally written by @ccantynz-alt on GitHub.
Imported from https://github.com/ccantynz-alt/-AI-Quote-to-Invoice-for-Trades/pull/4


$(cat <<'EOF'

Summary

  • Add tradequote-ai/package-lock.json so npm ci can run in GitHub Actions (was missing, causing build failure)
  • Add tradequote-ai/.npmrc with legacy-peer-deps=true for @wordpress/scripts compatibility
  • Upgrade @wordpress/scripts from ^27.0.0 to ^30.27.0 to resolve AJV peer dependency conflict on Node 22
  • Add ajv@8 as an explicit dependency (required by ajv-keywords)
  • Rewrite WPCS CI step to use composer global require instead of the broken $(wp-coding-standards location) tool syntax
  • Exclude .npmrc from the release zip

Test plan

  • CI "Build React bundle" job passes (npm ci + npm run build)
  • CI "PHP syntax check" job passes on PHP 8.1, 8.2, 8.3
  • CI "WordPress Coding Standards" job completes without error (non-blocking || true)
  • Release workflow still packages plugin zip correctly

https://claude.ai/code/session_01SYko81ETdN3uyZ8GVdH1s5 EOF )


Generated by Claude Code

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts