chore(lint): enable A003, PLW0108 rules (zero-violation easy wins) by aaronsteers · Pull Request #1058 · airbytehq/PyAirbyte

Aaron ("AJ") Steers (aaronsteers) · 2026-07-06T04:15:59Z

Summary

Remove A003 and PLW0108 from the ignore list. Both rules are already selected via their parent categories (A and PL), so removing the ignores enables enforcement.

A003: Prevents class attributes from shadowing Python builtins like type, id, list
PLW0108: Flags unnecessary lambdas like lambda x: fn(x) which should just be fn

Both rules have zero existing violations — this is a config-only change with no code impact.

Link to Devin session: https://app.devin.ai/sessions/6ae2048fc77a4f278aee3dd022384f4a
Requested by: Aaron ("AJ") Steers (@aaronsteers)

Un-ignore A003 (no builtin shadowing in classes) and PLW0108 (no unnecessary lambdas). Both have 0 existing violations, so this is config-only with no code changes needed. Co-Authored-By: AJ Steers <aj@airbyte.io>

Auto-fix stale noqa comments that suppress rules not triggered on those lines. These accumulate as code changes and become misleading. Co-Authored-By: AJ Steers <aj@airbyte.io>

devin-ai-integration · 2026-07-06T04:16:02Z

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

Disable automatic comment, CI, and merge conflict monitoring

coderabbitai · 2026-07-06T04:16:08Z

Warning

Review limit reached

@devin-ai-integration[bot], you've reached your PR review limit, so we couldn't start this review.

Next review available in: 7 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 5244bd04-e94f-45f4-9dd5-b31dbeaff9af

📥 Commits

Reviewing files that changed from the base of the PR and between 15ca9ab and 2958c66.

📒 Files selected for processing (1)

.ruff.toml

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch devin/1783311246-ruff-easy-wins

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

github-actions · 2026-07-06T04:16:17Z

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

Testing This PyAirbyte Version

You can test this version of PyAirbyte using the following:

# Run PyAirbyte CLI from this branch:
uvx --from 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1783311246-ruff-easy-wins' pyairbyte --help

# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1783311246-ruff-easy-wins'

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

/fix-pr - Fixes most formatting and linting issues
/uv-lock - Updates uv.lock file
/test-pr - Runs tests with the updated PyAirbyte
/prerelease - Builds and publishes a prerelease version to PyPI

📚 Show Repo Guidance

Helpful Resources

Community Support

Questions? Join the #pyairbyte channel in our Slack workspace.

📝 Edit this welcome message.

…ix)" This reverts commit e03b9ac.

Per review feedback: remove lines from ignore entirely instead of commenting them out. Both rules are already selected via their parent categories (A and PL). Co-Authored-By: AJ Steers <aj@airbyte.io>

github-code-quality · 2026-07-06T04:30:32Z

Code Coverage Overview

Languages: Python

Python / code-coverage/pytest-fast

The overall coverage in the devin/1783311246-ruf... branch is 67%. The coverage in the main branch is 65%.

Show a code coverage summary of the most impacted files.

File	main `d9f652f`	devin/1783311246-ruf... `2958c66`	+/-
`airbyte/_util/api_util.py`	36%	37%	+1%
`airbyte/registry.py`	70%	72%	+2%
`airbyte/mcp/_tool_utils.py`	72%	77%	+5%
`airbyte/mcp/int...c_history_ui.py`	0%	36%	+36%
`airbyte/mcp/int...hared_models.py`	0%	81%	+81%
`airbyte/cloud/models.py`	0%	91%	+91%
`airbyte/mcp/int..._registry_ui.py`	0%	92%	+92%
`airbyte/mcp/int...nc_status_ui.py`	0%	97%	+97%
`airbyte/mcp/int...ive/__init__.py`	0%	100%	+100%
`airbyte/mcp/int...tive/_prefab.py`	0%	100%	+100%

Python / code-coverage/pytest-no-creds

The overall coverage in the devin/1783311246-ruf... branch is 67%. The coverage in the main branch is 65%.

Show a code coverage summary of the most impacted files.

File	main `d9f652f`	devin/1783311246-ruf... `2958c66`	+/-
`airbyte/_util/api_util.py`	36%	37%	+1%
`airbyte/registry.py`	70%	72%	+2%
`airbyte/mcp/_tool_utils.py`	72%	77%	+5%
`airbyte/mcp/int...c_history_ui.py`	0%	36%	+36%
`airbyte/mcp/int...hared_models.py`	0%	81%	+81%
`airbyte/cloud/models.py`	0%	91%	+91%
`airbyte/mcp/int..._registry_ui.py`	0%	92%	+92%
`airbyte/mcp/int...nc_status_ui.py`	0%	97%	+97%
`airbyte/mcp/int...ive/__init__.py`	0%	100%	+100%
`airbyte/mcp/int...tive/_prefab.py`	0%	100%	+100%

Python / code-coverage/pytest

The overall coverage in the devin/1783311246-ruf... branch is 72%. The coverage in the main branch is 71%.

Show a code coverage summary of the most impacted files.

File	main `d9f652f`	devin/1783311246-ruf... `2958c66`	+/-
`airbyte/registry.py`	70%	72%	+2%
`airbyte/mcp/_tool_utils.py`	72%	77%	+5%
`airbyte/mcp/server.py`	69%	76%	+7%
`airbyte/mcp/int...c_history_ui.py`	0%	36%	+36%
`airbyte/mcp/int...hared_models.py`	0%	81%	+81%
`airbyte/mcp/int..._registry_ui.py`	0%	92%	+92%
`airbyte/cloud/models.py`	0%	93%	+93%
`airbyte/mcp/int...nc_status_ui.py`	0%	97%	+97%
`airbyte/mcp/int...ive/__init__.py`	0%	100%	+100%
`airbyte/mcp/int...tive/_prefab.py`	0%	100%	+100%

_{Updated July 06, 2026 04:47 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.}

devin-ai-integration Bot and others added 2 commits July 6, 2026 04:14

chore(lint): enable A003, PLW0108 rules (zero-violation easy wins)

2a40243

Un-ignore A003 (no builtin shadowing in classes) and PLW0108 (no unnecessary lambdas). Both have 0 existing violations, so this is config-only with no code changes needed. Co-Authored-By: AJ Steers <aj@airbyte.io>

style(lint): remove 134 unused noqa directives (RUF100 auto-fix)

e03b9ac

Auto-fix stale noqa comments that suppress rules not triggered on those lines. These accumulate as code changes and become misleading. Co-Authored-By: AJ Steers <aj@airbyte.io>

Aaron ("AJ") Steers (aaronsteers) self-assigned this Jul 6, 2026

Revert "style(lint): remove 134 unused noqa directives (RUF100 auto-f…

f131d66

…ix)" This reverts commit e03b9ac.

devin-ai-integration Bot changed the title ~~chore(lint): enable A003, PLW0108 rules and clean up stale noqa directives~~ chore(lint): enable A003, PLW0108 rules (zero-violation easy wins) Jul 6, 2026

fix(lint): remove A003, PLW0108 from ignore list cleanly

2958c66

Per review feedback: remove lines from ignore entirely instead of commenting them out. Both rules are already selected via their parent categories (A and PL). Co-Authored-By: AJ Steers <aj@airbyte.io>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore(lint): enable A003, PLW0108 rules (zero-violation easy wins)#1058

chore(lint): enable A003, PLW0108 rules (zero-violation easy wins)#1058
Aaron ("AJ") Steers (aaronsteers) wants to merge 4 commits into
mainfrom
devin/1783311246-ruff-easy-wins

Aaron ("AJ") Steers (aaronsteers) commented Jul 6, 2026 •

edited by devin-ai-integration Bot

Loading

Uh oh!

devin-ai-integration Bot commented Jul 6, 2026

Uh oh!

coderabbitai Bot commented Jul 6, 2026 •

edited

Loading

Review limit reached

Uh oh!

github-actions Bot commented Jul 6, 2026

Testing This PyAirbyte Version

PR Slash Commands

Helpful Resources

Community Support

Uh oh!

github-code-quality Bot commented Jul 6, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

Aaron ("AJ") Steers (aaronsteers) commented Jul 6, 2026 • edited by devin-ai-integration Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

devin-ai-integration Bot commented Jul 6, 2026

🤖 Devin AI Engineer

Uh oh!

coderabbitai Bot commented Jul 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Uh oh!

github-actions Bot commented Jul 6, 2026

👋 Greetings, Airbyte Team Member!

Testing This PyAirbyte Version

PR Slash Commands

Helpful Resources

Community Support

Uh oh!

github-code-quality Bot commented Jul 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Coverage Overview

Python / code-coverage/pytest-fast

Python / code-coverage/pytest-no-creds

Python / code-coverage/pytest

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Aaron ("AJ") Steers (aaronsteers) commented Jul 6, 2026 •

edited by devin-ai-integration Bot

Loading

coderabbitai Bot commented Jul 6, 2026 •

edited

Loading

github-code-quality Bot commented Jul 6, 2026 •

edited

Loading