PR_26175_ALFA_051-idea-board-game-hub-row-expectation

[ToolboxAid]

Summary

• Aligns the Idea Board cross-flow Game Hub expanded-row expectation with current Game Hub rendering.
• Expects source-idea and readiness-output rows, removing the stale summary-row assertion from this test path.

Validation

• PASS: node --check tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
• PASS: npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --workers=1

Package

• tmp/PR_26175_ALFA_051-idea-board-game-hub-row-expectation_delta.zip

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 654cf47afd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Correct the stale expectation scope claim

This stale three-row expectation is not limited to the Idea Board cross-flow path: tests/playwright/tools/GameHubMockRepository.spec.mjs:571-577 still asserts the same summary/source-idea/readiness-output expanded rows, while the current renderExpandedGameRow only renders source-idea and readiness-output. As written, the report can mark the PR complete while leaving the Game Hub parent/child table lane with the same stale assertion that this change is meant to resolve.

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

Match the active BUILD source of truth

The repo workflow says the BUILD doc is the source of truth, but at this commit docs_build/dev/BUILD_PR.md still defines PR_26175_ALFA_047-theme-v2-svg-icon-registry with SVG registry targets and validation, while this run adds ALFA_051 reports and changes IdeaBoardTableNotes.spec.mjs instead. In contexts where the BUILD gates are enforced, this work is outside the active exact scope and skips the required validation, so either update the BUILD doc for ALFA_051 or move these changes to the correct build before marking it PASS.

Useful? React with 👍 / 👎.