Skip to content

feat(autofix): Bypass cache in UI to get latest check value #105110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Mihir-Mavalankar merged 3 commits into from
Dec 17, 2025
Merged

feat(autofix): Bypass cache in UI to get latest check value #105110

Mihir-Mavalankar merged 3 commits into from
Dec 17, 2025

Conversation

@Mihir-Mavalankar
Copy link
Contributor

@Mihir-Mavalankar Mihir-Mavalankar commented Dec 16, 2025 •
edited
Loading

PR Description

  • Added skip_cache parameter to has_project_connected_repos() to allow bypassing the cache when freshness matters like in the UI. Updated the setup check endpoint to use skip_cache=True so the UI always reflects the latest current repo configuration state as soon as the user updates settings.
  • We still need a cache for hot paths like post_process (called for every autofix automation run). Reduced TTL to 15 minutes though. We can always increase it if needed.
  • Alternative: I can remove the cache completely and we can add it back if it's too much load on autofix.

@Mihir-Mavalankar Mihir-Mavalankar self-assigned this Dec 16, 2025
@Mihir-Mavalankar Mihir-Mavalankar requested a review from a team as a code owner December 16, 2025 23:47
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 16, 2025
@codecov
Copy link

codecov bot commented Dec 17, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files 
@@            Coverage Diff            @@
##           master   #105110    +/-   ##
=========================================
  Coverage   80.62%    80.62%            
=========================================
  Files        9419      9417     -2     
  Lines      403879    403768   -111     
  Branches    25637     25626    -11     
=========================================
- Hits       325624    325539    -85     
+ Misses      77786     77760    -26     
  Partials      469       469

@Mihir-Mavalankar Mihir-Mavalankar merged commit 8f82855 into master Dec 17, 2025
67 checks passed
@Mihir-Mavalankar Mihir-Mavalankar deleted the cache-bypass-for-ui branch December 17, 2025 16:34
@github-actions github-actions bot locked and limited conversation to collaborators Jan 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@Zylphrex Zylphrex Zylphrex approved these changes

Assignees

@Mihir-Mavalankar Mihir-Mavalankar

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Mihir-Mavalankar @Zylphrex