[Sprint 19] stabilize SeedMyBlogData concurrent invocation in AppHost.Tests

[mpaulosky]

Summary

AppHost.Tests (Aspire + Playwright E2E) is red on both PR #283 and PR #285 because SeedMyBlogData Concurrent Invocations Allow Only One Run is failing in both branches.

Current diagnosis says this is a shared red-branch / shared AppHost test instability, not two separate PR regressions.

Impact

• Blocks PR fix(build): clear remaining Release analyzer warnings #283 from merge readiness
• Blocks PR fix(headers): preserve original year when normalizing headers #285 from merge readiness
• Makes CI signal noisy because the same shared failure appears on multiple branches

Acceptance Criteria

• Reproduce SeedMyBlogData Concurrent Invocations Allow Only One Run reliably in a CI-like AppHost test run
• Identify the root cause in AppHost startup/seeding, concurrency handling, or the test harness
• Fix or harden the failing path so AppHost.Tests (Aspire + Playwright E2E) is green for this scenario
• Verify PR fix(build): clear remaining Release analyzer warnings #283 and PR fix(headers): preserve original year when normalizing headers #285 are no longer blocked by this specific failure
• Document the root cause and any follow-up hardening work in this issue or a linked follow-up issue

Context

• Failing check: AppHost.Tests (Aspire + Playwright E2E)
• Failing test: SeedMyBlogData Concurrent Invocations Allow Only One Run
• Blocking PRs: fix(build): clear remaining Release analyzer warnings #283, fix(headers): preserve original year when normalizing headers #285
• Related non-priority branch: PR feat(theme): centralise table, form, and alert CSS into input.css — #276 #277 already has an explicit not-merge-ready comment and is not the current unblock target

Routing

Per .squad/routing.md, build/test gate failures route to Aragorn for ownership/triage.

Sprint

Sprint 19

[github-actions]

📋 Assigned to Aragorn (Lead / Architect)

Issue: #286 — [Sprint 19] stabilize SeedMyBlogData concurrent invocation in AppHost.Tests

Aragorn will pick this up in the next Copilot session.

For Copilot coding agent: If enabled, this issue will be worked automatically.
Otherwise, start a Copilot session and say:
Aragorn, work on issue #286

[mpaulosky]

Status update: the validated AppHost seed-concurrency fix has now been ported onto both blocked PR branches.\n\n- PR #283 / branch includes commit \n- PR #285 / branch includes commit \n\nBoth branches passed local Release build + AppHost.Tests validation before push. GitHub is currently rerunning the affected checks.

[mpaulosky]

Correction to the previous status note: the ported fix commits are:\n\n- PR #283 on branch squad/280-cleanup-release-build-warnings includes commit a3ca073\n- PR #285 on branch squad/284-normalize-copyright-header-updates includes commit d3beb06\n\nBoth branches passed local Release build and AppHost.Tests validation before push, and GitHub is rerunning the affected checks now.

[mpaulosky]

🤖 Ralph Board Update

✅ Issue resolved: AppHost.Tests concurrent invocation fix validated and merged.

Resolution summary:

• PR fix(build): clear remaining Release analyzer warnings #283 (fix: clear Release analyzer warnings) — MERGED
• PR fix(headers): preserve original year when normalizing headers #285 (fix: preserve original year in headers) — MERGED
• Both included validated AppHost.Tests concurrent-invocation hardening
• AppHost.Tests now green on both branches
• CI signal restored

Closing issue as COMPLETED.