fix: validator test messages, Issue init-only fields, IssueEntity missing props, exception middleware, squad-ci wiring

[Copilot]

Addresses remaining review feedback from PR #23 across five areas.

Changes

• Validator tests: Fixed assertions to match actual FluentValidation messages ("greater than or equal to 1" / "between 1 and 100" vs the incorrect "greater than 0")

• Issue record immutability: Soft-delete audit fields (IsArchived, ArchivedBy, ArchivedAt) changed from { get; set; } to { get; init; }, consistent with record semantics

• IssueEntity / IssueRepository:

  • Added missing ArchivedBy and ArchivedAt properties to IssueEntity (previously referenced in ToDomain() but undeclared — compilation error)
  • Enum.Parse<IssueStatus> → Enum.TryParse with ignoreCase: true and IssueStatus.Open fallback to avoid 500s on unexpected stored values

• Exception middleware: Added UseExceptionHandler to Program.cs mapping FluentValidation.ValidationException → 400 with structured error body; previously these would surface as 500s despite endpoints declaring 400

• squad-ci.yml: Replaced no-op echo placeholder with actual dotnet restore / build / test pipeline

// Before — throws on unknown/mismatched casing stored in MongoDB
Status: Enum.Parse<IssueStatus>(Status),

// After — safe fallback, case-insensitive
Status: Enum.TryParse<IssueStatus>(Status, ignoreCase: true, out var parsedStatus)
    ? parsedStatus
    : IssueStatus.Open,

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[github-actions]

Test Results Summary

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit b01fd49. ± Comparison against base commit 633e702.

[Copilot]

Copilot wasn't able to review any files in this pull request.