Skip to content

fix(dashboard): revalidate GitHub caches when installation access cha…#142

Merged
stylessh merged 1 commit intomainfrom
cursor/defecb70
Apr 17, 2026
Merged

fix(dashboard): revalidate GitHub caches when installation access cha…#142
stylessh merged 1 commit intomainfrom
cursor/defecb70

Conversation

@stylessh
Copy link
Copy Markdown
Owner

@stylessh stylessh commented Apr 17, 2026

…nges

Subscribe pulls/issues mine and repos.list caches to installationAccess signals, bust installation access row on refresh, so setup changes reflect immediately instead of waiting for unrelated cache TTLs.

Summary

Changes

Test Plan

  • [ ]

Screenshots

Before After
Previous UI After UI

Summary by CodeRabbit

  • Bug Fixes
    • Improved synchronization of GitHub data across repositories, pull requests, and issues to ensure accurate and current information is displayed, particularly when access permissions change.

…nges

Subscribe pulls/issues mine and repos.list caches to installationAccess
signals, bust installation access row on refresh, so setup changes reflect
immediately instead of waiting for unrelated cache TTLs.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
diffkit 86015e0 Apr 17 2026, 12:28 AM

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 17, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b78ed233-6091-442d-b13a-485a0eacdc7e

📥 Commits

Reviewing files that changed from the base of the PR and between ce9a5a8 and 86015e0.

📒 Files selected for processing (1)
  • apps/dashboard/src/lib/github.functions.ts

📝 Walkthrough

Walkthrough

Updated cache revalidation logic for GitHub API calls by extending revalidation signals to include installation access status. The refreshInstallationAccess function now validates GitHub context availability and explicitly busts cached data, while repo listing and pull/issue queries now participate in revalidation when installation access changes.

Changes

Cohort / File(s) Summary
Cache Revalidation Wiring
apps/dashboard/src/lib/github.functions.ts
Extended signalKeys for pulls.mine, issues.mine, and repos.list cached queries to include installationAccess signal; enhanced refreshInstallationAccess to load GitHub context with early return, explicitly bust cached installationAccess row, and update debug messaging.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete. While it includes a custom summary, most required template sections (Changes, Test Plan) are left empty or contain only placeholders. Fill in the Changes section with specific modifications made, and provide actual test cases or scenarios in Test Plan instead of leaving unchecked placeholders.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: revalidating GitHub caches when installation access changes, which is precisely what the changeset accomplishes.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/defecb70

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant