[Sprint 19] Polish category UX, test naming, and documentation

[mpaulosky]

Follow-up from PR #340 Aragorn gate. Copilot review surfaced minor UX/polish items that did not block merge but should be addressed:

UI semantics (Legolas)

• src/Web/Features/BlogPosts/Create/Create.razor (~L46, L114) and src/Web/Features/BlogPosts/Edit/Edit.razor (~L74, L117): The Category field shows a required asterisk and the submit guard blocks any save (including draft) whenever categories exist, but the helper copy says categories are needed before publishing. Either:
  • Make CategoryId required only when IsPublished == true, or
  • Update the copy + asterisk so the requirement is unconditional.
• Create.razor / Edit.razor: If GetCategoriesQuery fails, _categories stays empty silently and posts can be saved with CategoryId = null despite the UI marking it required. Surface load failures via _error and/or block submit until categories load.
• src/Web/Features/Categories/List/Index.razor (~L138): LoadAsync does not clear _error on success and does not reset _categories on failure; can leave stale UI state. Clear _error on success and set _categories = [] on failure.

Naming / docs (low priority)

• tests/Web.Tests/Features/Categories/Commands/UpdateCategoryCommandValidatorTests.cs actually tests EditCategoryCommandValidator — rename file/class/namespace to Edit* for consistency.
• src/AppHost/MongoDbResourceBuilderExtensions.cs (~L292): seed log says "inserted" but is now an upsert; change wording to "upserted" / "inserted/updated".
• .squad/skills/gh-pr-comments-fallback/SKILL.md (~L15, L36): grammar — "due to" not "due".
• .squad/agents/legolas/history.md (~L946): placeholder date 2025-07-XX; correct to actual date.

Source

PR #340 Copilot automated review — full thread on the PR.

Suggested routing

• UI items → Legolas
• Test rename → Gimli
• AppHost log wording → Sam
• .squad/ doc/grammar fixes → Frodo

[github-actions]

📋 Assigned to Legolas (Frontend / Blazor)

Issue: #341 — [Sprint 19] Polish category UX, test naming, and documentation

Legolas 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:
Legolas, work on issue #341

[github-actions]

📋 Assigned to Frodo (Tech Writer)

Issue: #341 — [Sprint 19] Polish category UX, test naming, and documentation

Frodo 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:
Frodo, work on issue #341

[github-actions]

📋 Assigned to Gimli (Tester)

Issue: #341 — [Sprint 19] Polish category UX, test naming, and documentation

Gimli 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:
Gimli, work on issue #341

[mpaulosky]

Triage: Multi-Owner Polish Follow-up

✅ Milestone corrected — Sprint 19: Polish & Documentation

Work Breakdown & Owners

Work Item	Owner	Scope
Category UX semantics (Create/Edit asterisk, validation flow, error handling)	@legolas	~L46, L114, L74, L117 in Create/Edit.razor; L138 in Categories/List
Test file rename (UpdateCategory → EditCategory for consistency)	@gimli	Low priority; naming convention alignment
AppHost seed log wording ("inserted" → "upserted")	@sam	1-line fix; L292 MongoDbResourceBuilderExtensions.cs
.squad/ doc fixes (grammar "due" → "due to", placeholder date correction)	@frodo	~L15, L36 in SKILL.md; L946 in history.md

Notes

• This is a bundled follow-up — keep together as one polish milestone
• Priority: Low-to-medium (Copilot gate findings, not blockers)
• No new architecture — all fixes are UX/naming/docs alignment
• Independent slices — each owner can work in parallel

[github-actions]

📋 Assigned to Sam (Backend / .NET)

Issue: #341 — [Sprint 19] Polish category UX, test naming, and documentation

Sam 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:
Sam, work on issue #341