fix(prd-7): M5 code quality review — remove dead re-exports, verify type safety

- Removed unused re-exports (CommentUpdateInput, ExecutionDetails,
  ensureProperlyEncodedUrl) from forgejo/comment-logic.ts
- Verified no `any` type escapes in Forgejo-specific code
- Confirmed naming conventions consistent with upstream
- No dead code, unused imports, or unreachable branches
- All complex decisions documented with comments
- Typecheck, format, and 834 tests pass clean

Progress: 83% complete — M6 (documentation accuracy) remaining

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

prds/7-senior-review-forgejo.md

@@ -92,7 +92,7 @@ Perform a thorough, documented code review of all Forgejo-related changes. The r
   - Verify mocks accurately represent Forgejo API responses
   - **Deliverable**: Test coverage improved, gaps documented
 
-- [ ] **M5: Code quality & maintainability review**
+- [x] **M5: Code quality & maintainability review**
 
   - TypeScript strictness: are types accurate or using `any` / type assertions?
   - Naming conventions: consistent with the rest of the codebase?

src/forgejo/comment-logic.ts

@@ -12,13 +12,6 @@
 
 import { getServerUrl } from "../github/api/config";
 
-// Re-export types so consumers don't need to import from two places
-export type {
-  CommentUpdateInput,
-  ExecutionDetails,
-} from "../github/operations/comment-logic";
-export { ensureProperlyEncodedUrl } from "../github/operations/comment-logic";
-
 // ---------------------------------------------------------------------------
 // Run index helper
 // ---------------------------------------------------------------------------