fix: handle missing 'module'/'class' keys in custom plugin config by sentry[bot] · Pull Request #770 · codecov/codecov-cli · GitHub

This PR addresses a KeyError: 'module' that occurred when a custom plugin was defined in codecov.yml without specifying the module or class keys. Previously, _load_plugin_from_yaml would directly subscript plugin_dict['module'] and plugin_dict['class'], leading to an unhandled KeyError if these keys were absent.

The fix adds an early validation check within _load_plugin_from_yaml. It now verifies the presence of both module and class keys. If either is missing, it logs an informative error message using click.secho and gracefully returns a NoopPlugin(), preventing the CLI from crashing. This approach is consistent with the existing error handling patterns within the function for other plugin loading issues.

Fixes CLI-BX

This PR was automatically generated by Sentry. You can adjust this setting at any time.

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
723	1	722	1

View the full list of 1 ❄️ flaky test(s)

api.temp.calculator.test_calculator::test_divide

Flake rate in main: 100.00% (Passed 0 times, Failed 1726 times)

Stack Traces | 0.001s run time

def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

fix: handle missing 'module'/'class' keys in custom plugin config

c1ac25e

fix: handle missing 'module'/'class' keys in custom plugin config by sentry[bot] · Pull Request #770 · codecov/codecov-cli · GitHub

Uh oh!

fix: handle missing 'module'/'class' keys in custom plugin config - #770

fix: handle missing 'module'/'class' keys in custom plugin config#770
sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix/plugin-keyerror

sentry Bot commented Aug 22, 2026

Uh oh!

codecov Bot commented Aug 22, 2026 •
edited
Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants

fix: handle missing 'module'/'class' keys in custom plugin config by sentry[bot] · Pull Request #770 · codecov/codecov-cli · GitHub

Uh oh!

fix: handle missing 'module'/'class' keys in custom plugin config - #770

fix: handle missing 'module'/'class' keys in custom plugin config#770sentry[bot] wants to merge 1 commit intomaincodecov/codecov-cli:mainfrom seer/fix/plugin-keyerrorcodecov/codecov-cli:seer/fix/plugin-keyerrorCopy head branch name to clipboard

Conversation

sentry Bot commented Aug 22, 2026

Uh oh!

codecov Bot commented Aug 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ 1 Tests Failed:

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants

fix: handle missing 'module'/'class' keys in custom plugin config#770
sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix/plugin-keyerror

codecov Bot commented Aug 22, 2026 •
edited
Loading