Skip to content

chore: Remove Codecov AI code and assets#3969

Open
drazisil-codecov wants to merge 3 commits intomainfrom
joebecher/ccmrg-2033-remove-codecov-ai-from-the-ui
Open

chore: Remove Codecov AI code and assets#3969
drazisil-codecov wants to merge 3 commits intomainfrom
joebecher/ccmrg-2033-remove-codecov-ai-from-the-ui

Conversation

@drazisil-codecov
Copy link
Contributor

@drazisil-codecov drazisil-codecov commented Jan 26, 2026

Summary

  • Complete cleanup of Codecov AI feature following the UI hiding
  • Removes all underlying code and assets that were left intact in the "hide" PR

Changes

  • Remove CodecovAIPage component and all sub-components
  • Remove codecovAI GraphQL hooks and services (useCodecovAIInstallation, useCodecovAIInstalledRepos)
  • Remove codecovAI assets (PR review and test gen example images)
  • Remove GH_APP_AI config from config.js

Dependencies

This PR depends on #3970 being merged first (Hide Codecov AI from the UI)

Related to CCMRG-2033

Test plan

  • Verify all Codecov AI files are removed
  • Run tests to ensure no broken imports or references
  • Confirm build succeeds

Note

Fully removes the deprecated Codecov AI feature from the app.

  • Deletes CodecovAIPage and all subcomponents, GraphQL hooks (useCodecovAIInstallation, useCodecovAIInstalledRepos), tests, and assets
  • Removes /codecovai/:provider/:owner route and AI-related tabs/feature-flag usage from Account, Analytics, Members, Owner, and Plan tabs
  • Drops AI-specific nav/static links (codecovAI, codecovAIAppInstallation, codecovAIDocs) and GH_APP_AI from config.js
  • Updates tests to reflect removed tabs/routes and simplified tab rendering

Written by Cursor Bugbot for commit 3b23bed. This will update automatically on new commits. Configure here.

@linear
Copy link

linear bot commented Jan 26, 2026

@sentry
Copy link

sentry bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.59%. Comparing base (bd05f38) to head (3b23bed).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3969      +/-   ##
==========================================
+ Coverage   98.57%   98.59%   +0.02%     
==========================================
  Files         826      818       -8     
  Lines       15232    15134      -98     
  Branches     4466     4442      -24     
==========================================
- Hits        15015    14922      -93     
+ Misses        210      205       -5     
  Partials        7        7              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
src/config.js 95.83% <ø> (ø)
src/pages/AccountSettings/shared/Header/Header.tsx 100.00% <ø> (ø)
src/pages/AnalyticsPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/MembersPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/OwnerPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/PlanPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/services/navigation/useNavLinks.ts 99.02% <ø> (-0.01%) ⬇️
src/services/navigation/useStaticNavLinks.ts 98.83% <ø> (-0.03%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.11% <ø> (+0.04%) ⬆️
Services 99.32% <ø> (-0.01%) ⬇️
Shared 99.06% <ø> (ø)
UI 99.01% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd05f38...3b23bed. Read the comment docs.

@codecov-notifications
Copy link

codecov-notifications bot commented Jan 26, 2026

Codecov Report

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

@@            Coverage Diff             @@
##             main    #3969      +/-   ##
==========================================
+ Coverage   98.57%   98.59%   +0.02%     
==========================================
  Files         826      818       -8     
  Lines       15232    15134      -98     
  Branches     4466     4434      -32     
==========================================
- Hits        15015    14922      -93     
+ Misses        210      205       -5     
  Partials        7        7              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
src/config.js 95.83% <ø> (ø)
src/pages/AccountSettings/shared/Header/Header.tsx 100.00% <ø> (ø)
src/pages/AnalyticsPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/MembersPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/OwnerPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/PlanPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/services/navigation/useNavLinks.ts 99.02% <ø> (-0.01%) ⬇️
src/services/navigation/useStaticNavLinks.ts 98.83% <ø> (-0.03%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.11% <ø> (+0.04%) ⬆️
Services 99.32% <ø> (-0.01%) ⬇️
Shared 99.06% <ø> (ø)
UI 99.01% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd05f38...3b23bed. Read the comment docs.

@codecov-public-qa
Copy link

codecov-public-qa bot commented Jan 26, 2026

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.59%. Comparing base (bd05f38) to head (3b23bed).

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3969      +/-   ##
==========================================
+ Coverage   98.57%   98.59%   +0.02%     
==========================================
  Files         826      818       -8     
  Lines       15232    15134      -98     
  Branches     4458     4442      -16     
==========================================
- Hits        15015    14922      -93     
+ Misses        210      205       -5     
  Partials        7        7              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
src/config.js 95.83% <ø> (ø)
src/pages/AccountSettings/shared/Header/Header.tsx 100.00% <ø> (ø)
src/pages/AnalyticsPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/MembersPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/OwnerPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/PlanPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/services/navigation/useNavLinks.ts 99.02% <ø> (-0.01%) ⬇️
src/services/navigation/useStaticNavLinks.ts 98.83% <ø> (-0.03%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.11% <ø> (+0.04%) ⬆️
Services 99.32% <ø> (-0.01%) ⬇️
Shared 99.06% <ø> (ø)
UI 99.01% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd05f38...3b23bed. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codecov-qa
Copy link

codecov-qa bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.59%. Comparing base (bd05f38) to head (3b23bed).

@@            Coverage Diff             @@
##             main    #3969      +/-   ##
==========================================
+ Coverage   96.48%   98.59%   +2.11%     
==========================================
  Files         826      818       -8     
  Lines       15232    15134      -98     
  Branches     4458     4434      -24     
==========================================
+ Hits        14697    14922     +225     
+ Misses        476      205     -271     
+ Partials       59        7      -52     
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
src/config.js 95.83% <ø> (ø)
src/pages/AccountSettings/shared/Header/Header.tsx 100.00% <ø> (ø)
src/pages/AnalyticsPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/MembersPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/OwnerPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/pages/PlanPage/Tabs/Tabs.tsx 100.00% <ø> (ø)
src/services/navigation/useNavLinks.ts 99.02% <ø> (-0.01%) ⬇️
src/services/navigation/useStaticNavLinks.ts 98.83% <ø> (-0.03%) ⬇️

... and 42 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.11% <ø> (+0.04%) ⬆️
Services 99.32% <ø> (-0.01%) ⬇️
Shared 99.06% <ø> (+2.14%) ⬆️
UI 99.01% <ø> (+20.18%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd05f38...3b23bed. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codecov-releaser
Copy link
Collaborator

codecov-releaser commented Jan 26, 2026

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
48008ef Mon, 26 Jan 2026 17:16:00 GMT Expired Expired
6c584e7 Mon, 26 Jan 2026 18:51:07 GMT Expired Expired
87bf33c Mon, 26 Jan 2026 19:00:17 GMT Expired Expired
3b23bed Mon, 26 Jan 2026 19:08:49 GMT Expired Expired
3b23bed Mon, 26 Jan 2026 19:14:34 GMT Cloud Enterprise

@thomasrockhu-codecov
Copy link
Contributor

thomasrockhu-codecov commented Jan 26, 2026

Can we split this into 2 PRs: 1 that hides Codecov AI from view, and then 1 that actually cleans up and removes all the code?

@drazisil-codecov drazisil-codecov force-pushed the joebecher/ccmrg-2033-remove-codecov-ai-from-the-ui branch from 48008ef to 6c584e7 Compare January 26, 2026 18:44
@drazisil-codecov drazisil-codecov changed the title feat: Remove Codecov AI from the UI chore: Remove Codecov AI code and assets Jan 26, 2026
@drazisil-codecov
Copy link
Contributor Author

Addressed @thomasrockhu-codecov's feedback - split this into 2 PRs:

  1. feat: Hide Codecov AI from the UI #3970 - Hide Codecov AI from the UI (removes route and tabs, quick visibility fix)
  2. This PR - Complete cleanup (removes all underlying code and assets)

Please merge #3970 first, then this PR.

@drazisil-codecov drazisil-codecov force-pushed the joebecher/ccmrg-2033-remove-codecov-ai-from-the-ui branch from 6c584e7 to 87bf33c Compare January 26, 2026 18:55
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
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 and related static nav links
- Update tests to remove AI feature flag checks

Closes CCMRG-2033
Complete cleanup of Codecov AI feature following the UI hiding in the
previous PR.

Changes:
- Remove CodecovAIPage component and all sub-components
- Remove codecovAI GraphQL hooks and services
- Remove codecovAI assets (PR review and test gen example images)
- Remove GH_APP_AI config

This is part 2 of the Codecov AI deprecation.
Closes CCMRG-2033
@drazisil-codecov drazisil-codecov force-pushed the joebecher/ccmrg-2033-remove-codecov-ai-from-the-ui branch from 87bf33c to 3b23bed Compare January 26, 2026 19:04
@sentry
Copy link

sentry bot commented Jan 27, 2026

Bundle Report

Changes will decrease total bundle size by 726.97kB (-5.79%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 5.87MB -363.49kB (-5.83%) ⬇️
gazebo-production-esm 5.95MB -363.49kB (-5.75%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js -198 bytes 17.02kB -1.15%
assets/index.*.js -1 bytes 49.15kB -0.0%
assets/index.*.js -198 bytes 41.68kB -0.47%
assets/index.*.js 313 bytes 14.55kB 2.2%
assets/index.*.js -198 bytes 22.17kB -0.89%
assets/index.*.js -198 bytes 12.23kB -1.59%
assets/index.*.js -1.15kB 638.24kB -0.18%
assets/index.*.js (Deleted) -8.58kB 0 bytes -100.0% 🗑️
assets/index.*.css -199 bytes 101.24kB -0.2%
assets/pr-review-example-light-mode.*.png (Deleted) -123.0kB 0 bytes -100.0% 🗑️
assets/pr-review-example-dark-mode.*.png (Deleted) -122.96kB 0 bytes -100.0% 🗑️
assets/test-gen-example-dark-mode.*.png (Deleted) -53.63kB 0 bytes -100.0% 🗑️
assets/test-gen-example-light-mode.*.png (Deleted) -53.49kB 0 bytes -100.0% 🗑️

Files in assets/index.*.js:

  • ./src/pages/PlanPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index.*.js:

  • ./src/pages/AnalyticsPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index.*.js:

  • ./src/pages/OwnerPage/Tabs/Tabs.tsx → Total Size: 747 bytes

Files in assets/index.*.js:

  • ./src/pages/MembersPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index.*.js:

  • ./src/services/navigation/useNavLinks.ts → Total Size: 21.35kB

  • ./src/pages/AccountSettings/shared/Header/Header.tsx → Total Size: 417 bytes

  • ./src/App.tsx → Total Size: 8.1kB

  • ./src/services/navigation/useStaticNavLinks.ts → Total Size: 16.58kB

  • ./src/config.js → Total Size: 2.18kB

view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js -215 bytes 20.89kB -1.02%
assets/index-legacy.*.js -212 bytes 41.18kB -0.51%
assets/index-legacy.*.js -201 bytes 16.6kB -1.2%
assets/index-legacy.*.js -200 bytes 11.59kB -1.7%
assets/index-legacy.*.js 321 bytes 13.1kB 2.51%
assets/index-legacy.*.js -1.33kB 715.51kB -0.19%
assets/index-legacy.*.js (Deleted) -8.57kB 0 bytes -100.0% 🗑️
assets/pr-review-example-light-mode.*.png (Deleted) -123.0kB 0 bytes -100.0% 🗑️
assets/pr-review-example-dark-mode.*.png (Deleted) -122.96kB 0 bytes -100.0% 🗑️
assets/test-gen-example-dark-mode.*.png (Deleted) -53.63kB 0 bytes -100.0% 🗑️
assets/test-gen-example-light-mode.*.png (Deleted) -53.49kB 0 bytes -100.0% 🗑️

Files in assets/index-legacy.*.js:

  • ./src/pages/OwnerPage/Tabs/Tabs.tsx → Total Size: 747 bytes

Files in assets/index-legacy.*.js:

  • ./src/pages/AnalyticsPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index-legacy.*.js:

  • ./src/pages/PlanPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index-legacy.*.js:

  • ./src/pages/MembersPage/Tabs/Tabs.tsx → Total Size: 415 bytes

Files in assets/index-legacy.*.js:

  • ./src/config.js → Total Size: 2.19kB

  • ./src/services/navigation/useStaticNavLinks.ts → Total Size: 16.58kB

  • ./src/App.tsx → Total Size: 8.24kB

  • ./src/services/navigation/useNavLinks.ts → Total Size: 21.37kB

  • ./src/pages/AccountSettings/shared/Header/Header.tsx → Total Size: 417 bytes

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.

3 participants