test: remove orphaned duplicate untriaged route test

[joryirving]

Summary

• Delete src/app/api/issues/untriaged.test.ts — an orphaned duplicate that imports the same ./untriaged/route GET handler already covered by src/app/api/issues/untriaged/route.test.ts.

Verification

• Surviving untriaged/route.test.ts runs green (8 tests).

Closes #501

[its-saffron]

AI Automated Review

Full PR review.

Analysis engine: MiniMax-M2.7@https://litellm.jory.dev/v1 (anthropic) — escalated (fast_low_confidence)

PR Review: PR 514 — Remove Orphaned Duplicate Untriaged Route Test

Recommendation: APPROVE

This PR cleanly deletes the orphaned duplicate test file src/app/api/issues/untriaged.test.ts as requested in issue PR 501. All CI checks pass (typecheck, tests, build, lint, docker build).

---

Change-by-Change Findings

File: src/app/api/issues/untriaged.test.ts (deleted)

• Removes 286 lines of test code that was a duplicate of src/app/api/issues/untriaged/route.test.ts
• Both files imported and tested the same ./untriaged/route GET handler
• The flat-path test was a leftover from an older routing structure (visible in git history: commits from June 2026 addressing TypeScript errors, import paths, and missing mocks)
• Surviving untriaged/route.test.ts covers the same handler with 8 tests and runs green per PR verification

---

Standards Compliance

✅ No repository standard violations — This is a test cleanup deletion with no impact on production code, environment variables, database schema, or security boundaries.

---

Linked Issue Fit

✅ Issue PR 501 acceptance criteria met exactly:

• Problem stated: Flat untriaged.test.ts and nested untriaged/route.test.ts both test the same handler
• Fix specified: Delete the flat untriaged.test.ts
• PR action: Exactly 1 file deleted (src/app/api/issues/untriaged.test.ts)
• No misinterpretation — PR does precisely what the issue requested

---

Evidence Provider Findings

No evidence providers were configured for this review.

---

Tool Harness Findings

Tool harness verification confirmed:

• git_status_short: Clean working directory (only untracked corpus files)
• git_diff_stat: No uncommitted changes
• Git log history for both the deleted file and the untriaged/ directory shows the expected evolution, including the nested route.test.ts being the canonical test file since route coverage was added (PR 441)

---

Unknowns / Needs Verification

None. The diff, CI results, and git history are consistent and sufficient to verify this change.

---

Summary

This is a straightforward cleanup PR: deleting a redundant test file that was superseded by the nested route.test.ts. The PR body accurately describes the change, CI is green, and issue PR 501 is fully addressed. No blockers.