chore: remove simulate_old_pagination test helper and its test

[detail-app]

Remove the test helper simulate_old_pagination and the old_pagination_drops_resolved_bugs_issue_300 test. simulate_old_pagination re-implemented the old shared-offset fetch_page_multi_status algorithm that was deleted from production in PR #301 in favor of the offset+limit/drain approach, and the test was its only consumer. Neither guarded or exercised any live production code path — the replacement new_pagination_* tests cover the current algorithm.

History

• Last materially changed May 2026 (fix: correct multi-status pagination in bugs list #301 @devin-ai-integration[bot])

---

Dead Code PRs can be configured here.

---

Summary by cubic

Removes the simulate_old_pagination test helper and its only test, which re-implemented the deleted shared-offset pagination algorithm. The replacement new_pagination_* tests already cover the current offset+limit/drain behavior, so this dead code no longer guards any live production path.

^{Written for commit 15604d5. Summary will update on new commits.}

[aviator-app]

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.

---

See the real-time status of this PR on the Aviator webapp.

Use the Aviator Chrome Extension to see the status of your PR within GitHub.

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

[github-actions]

This PR has been inactive for 7 days and will be closed in 7 days if there is no activity.