[xianjianlf2]

Summary

Fixes #8807.

Each worktree card resolved its branch to a PR by independently paginating the repo's full /pulls listing (state=all, up to MAX_PULL_REQUEST_PAGES = 5) with a 5s timeout. Gitea/Forgejo have no head-branch filter, a self-hosted Forgejo takes ~5s per page (reviewer data is loaded per PR), and a push event refreshes every card at once — one repo became hundreds of near-simultaneous requests whose responses were aborted right as they completed, and the burst OOM-killed the reporter's 512Mi Forgejo pod.

Fix

• One shared scan per repo (pull-request-scan-cache.ts): concurrent branch lookups await the same in-flight page walk, and the result is cached for 30s. A burst that refreshes N worktree cards now costs a single page walk instead of N.
• Negative results are cached too: a branch with no PR no longer rescans all 5 pages on every card refresh within the TTL.
• List timeout raised to 15s (other requests keep the 5s cap), so slow-but-successful pages are used instead of discarded and retried — the reporter observed Orca cancelling right as Forgejo finished.
• Cache invalidation on PR creation: after Orca creates a PR (and before the post-create findExistingPullRequest fallback), the scan is invalidated so the new PR is visible immediately rather than after the TTL.

The issue's remaining suggestion (state=open first with a lazy state=all fallback for merged detection) is noted as a possible follow-up; with the shared cache the marginal win is much smaller and it would double requests for merged-PR branches.

Testing

• New tests: three concurrent branch lookups (hit, hit, miss) issue exactly one list request; repeat lookups within the TTL issue none.
• vitest run src/main/gitea/ — 30 tests pass; npm run typecheck clean.

🤖 Generated with Claude Code

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: baaf17db-d0ca-4c3f-8871-1ab209db4dec

📥 Commits

Reviewing files that changed from the base of the PR and between 19f3d17 and 68616cd.

📒 Files selected for processing (2)

• src/main/gitea/client.test.ts
• src/main/gitea/pull-request-scan-cache.ts

---

📝 Walkthrough

Walkthrough

Adds a 30-second, repository-scoped cache for paginated Gitea pull-request scans, with concurrent request de-duplication, generation-based invalidation, bounded eviction, and reset support. Branch lookup uses the shared scanner with a longer list-request timeout. Pull-request creation flows invalidate cached scans before lookup and after successful creation. Tests cover cache reset, concurrent lookups, TTL reuse, failure retries, expiry, bounded size, and stale in-flight scans.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description covers summary and testing, but omits required Screenshots, AI Review Report, Security Audit, and Notes sections.	Add the missing template sections, or explicitly mark Screenshots as 'No visual change' and include AI review, security, and notes details.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the shared/cached Gitea /pulls scan change and its impact on refresh bursts.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e6a6fbb2-f8ba-4ac2-aa68-41d847276a05

📥 Commits

Reviewing files that changed from the base of the PR and between c12ade5 and fb1fde3.

📒 Files selected for processing (4)

• src/main/gitea/client.test.ts
• src/main/gitea/client.ts
• src/main/gitea/pull-request-creation.ts
• src/main/gitea/pull-request-scan-cache.ts

[xianjianlf2]

Fixed the invalidation race in the latest push: each repo key now carries a generation counter — invalidateGiteaPullRequestScan bumps it and drops the in-flight promise, and a scan only writes the cache if its generation is still current. Added a regression test that holds a scan in flight across an invalidation and asserts the post-create listing is what lands in the cache.