-
Notifications
You must be signed in to change notification settings - Fork 36
feat: Hide Codecov AI from the UI #3970
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.
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3970 +/- ##
==========================================
- Coverage 98.57% 98.57% -0.01%
==========================================
Files 826 826
Lines 15232 15214 -18
Branches 4466 4455 -11
==========================================
- Hits 15015 14997 -18
Misses 210 210
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3970 +/- ##
==========================================
- Coverage 98.57% 98.57% -0.01%
==========================================
Files 826 826
Lines 15232 15214 -18
Branches 4458 4447 -11
==========================================
- Hits 15015 14997 -18
Misses 210 210
Partials 7 7
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
02aead6 to
20a2fa3
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #3970 +/- ##
==========================================
- Coverage 98.57% 98.57% -0.01%
==========================================
Files 826 826
Lines 15232 15214 -18
Branches 4466 4455 -11
==========================================
- Hits 15015 14997 -18
Misses 210 210
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Remove Codecov AI tab and route from the UI as the first step in deprecating the feature. This hides the feature from users while keeping the underlying code intact for a follow-up cleanup PR. Changes: - Remove CodecovAIPage route from App.tsx - Remove Codecov AI tab from all navigation tabs - Remove codecovAI nav link from useNavLinks.ts - Update tests to remove AI feature flag checks Note: Nav links used by internal page components (codecovAIDocs, codecovAIAppInstallation) are preserved to avoid breaking tests for the still-existing (but inaccessible) CodecovAIPage. Closes CCMRG-2033
20a2fa3 to
3e9c344
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #3970 +/- ##
==========================================
+ Coverage 96.48% 98.57% +2.08%
==========================================
Files 826 826
Lines 15232 15214 -18
Branches 4458 4455 -3
==========================================
+ Hits 14697 14997 +300
+ Misses 476 210 -266
+ Partials 59 7 -52
... and 42 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Bundle ReportChanges will decrease total bundle size by 725.9kB (-5.78%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: gazebo-production-esmAssets Changed:
Files in
Files in
Files in
Files in
Files in
view changes for bundle: gazebo-production-systemAssets Changed:
Files in
Files in
Files in
Files in
Files in
|
Summary
App.tsx, AI tabs from all navigation components, and related nav linksContext
This is part 1 of a 2-part PR split requested by @thomasrockhu-codecov. Part 2 (full code cleanup) will follow in a separate PR.
Related to CCMRG-2033
Test plan
/codecovai/:provider/:ownerroute is not accessibleNote
Deprecates visible Codecov AI entry points across the app.
/codecovai/:provider/:ownerfromApp.tsxTabNavigationin AccountSettings, Analytics, Members, Owner, and Plan tabs; removes feature flag andBadgeusagecodecovAInav link fromuseNavLinksWritten by Cursor Bugbot for commit 3e9c344. This will update automatically on new commits. Configure here.