[Sprint] Comments: Refactor handlers to use ObjectId and return Result<T>

[mpaulosky]

Problem

Comment handlers have the same ObjectId/Result problems. Note: Comments currently use ObjectId in their query type but string in commands — inconsistency needs resolution.

Depends On

Depends on: Foundation issue (standardize ObjectId in Shared commands/queries)

Files to Modify

• src/Api/Handlers/Comments/GetCommentHandler.cs — return Task<Result>
• src/Api/Handlers/Comments/ListCommentsHandler.cs — return Task<Result<IReadOnlyList>>
• src/Api/Handlers/Comments/CreateCommentHandler.cs — return Task<Result>
• src/Api/Handlers/Comments/UpdateCommentHandler.cs — return Task<Result>
• src/Api/Handlers/Comments/DeleteCommentHandler.cs — return Task<Result>
• src/Api/Handlers/Comments/CommentEndpoints.cs — map Result → HTTP responses

Acceptance Criteria

• All 5 Comment handlers return Task<Result>
• ObjectId/string inconsistency in Comment commands/queries resolved consistently
• CommentEndpoints maps Result → proper HTTP status codes
• Solution builds with zero errors and zero warnings

[github-actions]

🏗️ Squad Triage — Aragorn (Lead)

Issue: #87 — [Sprint] Comments: Refactor handlers to use ObjectId and return Result
Assigned to: Legolas (Frontend Dev)
Reason: Issue relates to frontend/UI work

@copilot evaluation: No strong capability match — routed to squad member.

Team roster:

• Aragorn (Lead) → label: squad:aragorn
• Legolas (Frontend Dev) → label: squad:legolas
• Sam (Backend Dev) → label: squad:sam
• Gimli (Tester) → label: squad:gimli
• Boromir (DevOps) → label: squad:boromir
• Frodo (Tech Writer) → label: squad:frodo
• Gandalf (Security Officer) → label: squad:gandalf
• Ralph (Work Monitor) → label: squad:ralph
• @copilot (🤖 Coding Agent) → label: squad:@copilot
• @copilot (Coding Agent) → label: squad:copilot

To reassign, remove the current squad:* label and add the correct one.

[github-actions]

📋 Assigned to Sam (Backend Dev)

Issue: #87 — [Sprint] Comments: Refactor handlers to use ObjectId and return Result

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 #87

[mpaulosky]

Resolved in PR #92 (merged to main).