fix(infra): resolve remaining database runtime issues (#348)

[mpaulosky]

Summary

Closes #348

Working as Boromir (DevOps/Infra) + Sam (Backend) + Gimli (Tester).

Root Cause

After PR #346 (mongo:7 + mongo-data-v7 fix) was merged, MongoDB containers continued crashing in the running environment. Investigation revealed the active Aspire AppHost was built from the main repo's stale local dev branch — 2 commits behind origin/dev, still launching mongo:8.2 → exit 139 (SIGSEGV/AVX).

Identified via: ps aux | grep AppHost.dll — DLL path revealed the wrong repo root.

Changes

Boromir (DevOps/Infra)

• Pulled and rebuilt main repo AppHost (git pull origin dev + dotnet build) — 0 errors, MongoDB now uses mongo:7 + mongo-data-v7.
• Added rule 6 to .squad/skills/mongodb-dba-patterns/SKILL.md: sync main repo before starting Aspire; use ps aux | grep AppHost.dll to identify the active build.
• Added investigation findings to boromir/history.md.

Sam (Backend)

• Full backend audit: MongoDbBlogPostRepository, MongoDbCategoryRepository, BlogDbContext, Program.cs — all correct. No production code change required.
• Added findings to sam/history.md.

Gimli (Tester)

• Added SeedMyBlogData_Makes_Seeded_Posts_Visible_On_The_Blog_Page integration test to MongoSeedDataIntegrationTests — asserts seeded posts are visible via the real AppHost-wired Web/MongoDB path.
• Added findings to gimli/history.md.

Validation

• MongoDbContainerConfigurationTests — 4/4 ✅
• Web.Tests.Integration (Testcontainers) — 29/29 ✅
• All pre-push gates passed (markdownlint, build, unit tests, integration tests) ✅
• AppHost.Tests build: 0 errors, 0 warnings ✅

Volume State

Volume	Status
mongo-data	⚠️ FCV-contaminated (mongo:8.2 data, UUID-format). Do not use with mongo:7.
mongo-data-v7	✅ Clean, numeric-ident format, lock file 0 bytes. Safe for mongo:7.

[github-actions]

🏗️ PR Added to Squad Triage Queue

This PR has been labeled with squad and added to the triage queue.

Next steps:

• The squad Lead will review and assign to an appropriate team member
• A squad:member label will be added after triage

If you know which squad member should handle this, you can add the appropriate squad:member label yourself.

[Copilot]

Pull request overview

Adds runtime regression coverage for the MongoDB/Aspire database path after PR #346 and documents the operational lesson (stale main-repo build silently running pre-#346 infra) in the MongoDB DBA skill and agent histories. No production code change.

Changes:

• New SeedMyBlogData_Makes_Seeded_Posts_Visible_On_The_Blog_Page integration test in MongoSeedDataIntegrationTests that boots the real AppHost, seeds Mongo, and asserts seeded titles appear on /blog (with an /alive-based readiness probe).
• New rule #6 in .squad/skills/mongodb-dba-patterns/SKILL.md requiring git pull origin dev + dotnet build and ps aux | grep AppHost.dll to verify the active Aspire build matches the current worktree.
• History entries for Boromir, Sam, and Gimli capturing the investigation, findings, and standing learnings.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
tests/AppHost.Tests/MongoSeedDataIntegrationTests.cs	Adds end-to-end test seeding Mongo then asserting /blog HTML contains seeded titles, plus a WaitForWebReadyAsync helper.
.squad/skills/mongodb-dba-patterns/SKILL.md	Adds rule #6 on syncing the main repo and using ps aux to identify the running AppHost DLL.
.squad/agents/sam/history.md	Logs Sam's backend audit findings concluding no production code change is needed.
.squad/agents/gimli/history.md	Logs Gimli's reasoning for the new runtime read regression test.
.squad/agents/boromir/history.md	Logs Boromir's stale-dev-branch root cause, remediation steps, and lessons.

[github-actions]

Test Results Summary

480 tests  +1   479 ✅ +1   32s ⏱️ +4s
  6 suites ±0     1 💤 ±0 
  6 files   ±0     0 ❌ ±0 

Results for commit deee1ce. ± Comparison against base commit 883137f.

♻️ This comment has been updated with latest results.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.14%. Comparing base (883137f) to head (deee1ce).

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #349   +/-   ##
=======================================
  Coverage   80.14%   80.14%           
=======================================
  Files          68       68           
  Lines        1682     1682           
  Branches      201      201           
=======================================
  Hits         1348     1348           
  Misses        244      244           
  Partials       90       90           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.