Skip to content

Conversation

@drazisil-codecov
Copy link
Contributor

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

Summary

  • 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
  • Removes route from App.tsx, AI tabs from all navigation components, and related nav links

Context

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

  • Verify Codecov AI tab no longer appears in navigation
  • Verify /codecovai/:provider/:owner route is not accessible
  • Run existing tests to ensure no regressions

Note

Deprecates visible Codecov AI entry points across the app.

  • Removes lazy import and route for /codecovai/:provider/:owner from App.tsx
  • Drops Codecov AI tab from TabNavigation in AccountSettings, Analytics, Members, Owner, and Plan tabs; removes feature flag and Badge usage
  • Cleans up tests by removing AI tab mocks/expectations
  • Deletes codecovAI nav link from useNavLinks

Written by Cursor Bugbot for commit 3e9c344. 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.57%. Comparing base (bd05f38) to head (3e9c344).
✅ All tests successful. No failed tests found.

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              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
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%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.06% <ø> (-0.01%) ⬇️
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...3e9c344. Read the comment docs.

@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
02aead6 Mon, 26 Jan 2026 18:48:36 GMT Expired Expired
3e9c344 Mon, 26 Jan 2026 19:06:09 GMT Expired Expired
3e9c344 Mon, 26 Jan 2026 19:14:30 GMT Expired Expired
3e9c344 Mon, 26 Jan 2026 19:15:35 GMT Cloud Enterprise

@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.57%. Comparing base (bd05f38) to head (3e9c344).

✅ 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              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
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%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.06% <ø> (-0.01%) ⬇️
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...3e9c344. 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.

Copy link

@cursor cursor bot left a 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.

@drazisil-codecov drazisil-codecov force-pushed the joebecher/ccmrg-2033-hide-codecov-ai branch from 02aead6 to 20a2fa3 Compare January 26, 2026 18:54
@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    #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              
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
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%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.06% <ø> (-0.01%) ⬇️
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...3e9c344. Read the comment docs.

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
@drazisil-codecov drazisil-codecov force-pushed the joebecher/ccmrg-2033-hide-codecov-ai branch from 20a2fa3 to 3e9c344 Compare January 26, 2026 19:00
@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.57%. Comparing base (bd05f38) to head (3e9c344).

@@            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     
Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
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%) ⬇️

... and 42 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.06% <ø> (-0.01%) ⬇️
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...3e9c344. 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.

@sentry
Copy link

sentry bot commented Jan 27, 2026

Bundle Report

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

Detailed changes
Bundle name Size Change
gazebo-production-system 5.87MB -362.95kB (-5.82%) ⬇️
gazebo-production-esm 5.96MB -362.95kB (-5.74%) ⬇️

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 12.23kB -1.59%
assets/index.*.js -198 bytes 17.02kB -1.15%
assets/index.*.js 313 bytes 14.55kB 2.2%
assets/index.*.js -198 bytes 22.17kB -0.89%
assets/index.*.js -811 bytes 638.58kB -0.13%
assets/index.*.js -198 bytes 41.68kB -0.47%
assets/index.*.js -1 bytes 49.15kB -0.0%
assets/index.*.js (Deleted) -8.58kB 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.*.js:

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

Files in assets/index.*.js:

  • ./src/pages/PlanPage/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/App.tsx → Total Size: 8.1kB

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

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

Files in assets/index.*.js:

  • ./src/pages/AnalyticsPage/Tabs/Tabs.tsx → Total Size: 415 bytes
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 -200 bytes 11.59kB -1.7%
assets/index-legacy.*.js -800 bytes 716.05kB -0.11%
assets/index-legacy.*.js 321 bytes 13.1kB 2.51%
assets/index-legacy.*.js -201 bytes 16.6kB -1.2%
assets/index-legacy.*.js -212 bytes 41.18kB -0.51%
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/MembersPage/Tabs/Tabs.tsx → Total Size: 415 bytes

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

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

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

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

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

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

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

  • ./src/pages/AnalyticsPage/Tabs/Tabs.tsx → Total Size: 415 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.

4 participants