Skip to content

Fix bench component load before dependencies#943

Open
chubes4 wants to merge 1 commit into
mainfrom
fix/issue-942-bench-primary-before-deps
Open

Fix bench component load before dependencies#943
chubes4 wants to merge 1 commit into
mainfrom
fix/issue-942-bench-primary-before-deps

Conversation

@chubes4

@chubes4 chubes4 commented Jun 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Load/include the primary bench component immediately after ensuring it is active and before activating dependency plugins.
  • Keep hook snapshots ahead of component/dependency includes so deferred plugins_loaded/init callbacks are still replayed.
  • Update the bench smoke fixture so the dependency activates inside wordpress.bench and fails if it cannot see the primary component API.

Fixes #942.

Verification

  • npm run build
  • npx tsx scripts/bench-bootstrap-files-smoke.ts
  • npx tsx scripts/recipe-bench-smoke.ts
  • npm run check progressed through build, bench, recipe, and many smoke checks, but hit the local 10-minute shell timeout during recipe-interruption-artifacts-smoke with no assertion failure shown before timeout.
  • npx tsx scripts/recipe-interruption-artifacts-smoke.ts also hit a local 5-minute shell timeout with no output.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Drafted the bench load-order fix, targeted regression fixture updates, smoke verification, and PR description for Chris to review.

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.

wordpress.bench should load the component plugin before dependency activation

1 participant