Skip to content

chore: Bump airbyte-cdk to 7.3.9#835

Merged
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
mainfrom
devin/1761176493-bump-cdk-to-7.3.9
Oct 23, 2025
Merged

chore: Bump airbyte-cdk to 7.3.9#835
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
mainfrom
devin/1761176493-bump-cdk-to-7.3.9

Conversation

@aaronsteers

@aaronsteers Aaron ("AJ") Steers (aaronsteers) commented Oct 22, 2025

Copy link
Copy Markdown
Member

chore: Bump airbyte-cdk to 7.3.9

Summary

This PR updates the airbyte-cdk dependency from version >=6.61.6,<7.0.0 to >=7.3.9,<8.0.0. This is a major version bump from CDK 6.x to 7.x, which may include breaking changes according to semantic versioning.

The update also brings in several transitive dependency changes:

  • jsonref: >=0.2,<0.3>=1,<2 (major version bump)
  • whenever: >=0.6.16,<0.7.0>=0.7.3,<0.9.0
  • Removed numpy version constraint (was <2)

Review & Testing Checklist for Human

Given this is a major version bump with potential breaking changes, please verify:

  • Run the full test suite locally (poe test) to catch any breaking changes from the CDK upgrade
  • Review the CDK migration guide for documented breaking changes between 6.x and 7.x
  • Test core functionality manually: Try reading from a source, writing to cache, and basic operations to ensure nothing is broken
  • Monitor CI results once they complete - pay particular attention to integration tests
  • Review transitive dependency changes in the lock file diff to ensure no unexpected issues

Notes

Summary by CodeRabbit

  • Chores
    • Updated airbyte-cdk dependency to a newer version range.

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration

Copy link
Copy Markdown
Contributor
Original prompt from AJ Steers
@Devin - Please bump to the latest CDK dependency in PyAirbyte
Thread URL: https://airbytehq-team.slack.com/archives/D089P0UPVT4/p1761176349579049?thread_ts=1761176349.579049

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 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 and CI monitoring

@github-actions

Copy link
Copy Markdown

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

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/1761176493-bump-cdk-to-7.3.9' pyairbyte --help

# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1761176493-bump-cdk-to-7.3.9'

Helpful Resources

PR Slash Commands

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

  • /fix-pr - Fixes most formatting and linting issues
  • /poetry-lock - Updates poetry.lock file
  • /test-pr - Runs tests with the updated PyAirbyte

Community Support

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

📝 Edit this welcome message.

@coderabbitai

coderabbitai Bot commented Oct 22, 2025

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This PR updates the airbyte-cdk dependency constraint in pyproject.toml from version >=6.61.6,<7.0.0 to >=7.3.9,<8.0.0, raising the minimum supported version and widening the acceptable range to the next major version.

Changes

Cohort / File(s) Summary
Dependency version bump
pyproject.toml
Updated airbyte-cdk dependency constraint from >=6.61.6,<7.0.0 to >=7.3.9,<8.0.0

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The PR title "chore: Bump airbyte-cdk to 7.3.9" directly and accurately describes the primary change in the pull request. The changeset updates the airbyte-cdk dependency constraint from version 6.x to 7.3.9, and the title clearly communicates this specific update with appropriate precision. It uses a standard commit convention prefix (chore:), is concise and readable, and provides enough specificity that a teammate reviewing the commit history would immediately understand the purpose of the change.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devin/1761176493-bump-cdk-to-7.3.9

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0f7f3c7 and e1fe09e.

⛔ Files ignored due to path filters (1)
  • poetry.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2024-10-10T16:17:57.989Z
Learnt from: aaronsteers
PR: airbytehq/PyAirbyte#417
File: airbyte/cli.py:503-504
Timestamp: 2024-10-10T16:17:57.989Z
Learning: In the PyAirbyte project, support for Python versions earlier than 3.10 is not necessary, as the project requires Python 3.10 or newer.

Applied to files:

  • pyproject.toml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: Pytest (All, Python 3.10, Windows)
  • GitHub Check: Pytest (All, Python 3.11, Ubuntu)
  • GitHub Check: Pytest (All, Python 3.11, Windows)
  • GitHub Check: Pytest (All, Python 3.10, Ubuntu)
  • GitHub Check: Pytest (Fast)
  • GitHub Check: Pytest (No Creds)
🔇 Additional comments (1)
pyproject.toml (1)

20-20: Verify codebase compatibility with airbyte-cdk 7.3.9 and test coverage.

The dependency constraint correctly bumps airbyte-cdk from v6.61.6 to v7.3.9, but since this is a major version upgrade, there's potential for breaking changes. A few questions:

  1. Have you verified that all code in the repo using airbyte-cdk APIs has been updated for compatibility? We're only seeing the dependency constraint change here; do code changes exist elsewhere in the PR to support v7.x?

  2. Did the full test suite pass locally? The PR description mentions running poe test as part of the checklist—can you confirm the results?

  3. Have you reviewed the CDK migration guide for breaking changes and verified they don't affect PyAirbyte? (Also, the transitive dependency changes—particularly jsonref jumping from 0.2.x to 1.x—could have cascading effects. Wdyt on whether these need explicit testing?)

  4. Regarding the removed numpy version constraint: Was this constraint necessary for compatibility, or is it safe to allow any version of numpy?


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

PyTest Results (Fast Tests Only, No Creds)

304 tests  ±0   304 ✅ ±0   5m 51s ⏱️ + 1m 21s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit e1fe09e. ± Comparison against base commit 0f7f3c7.

@github-actions

Copy link
Copy Markdown

PyTest Results (Full)

373 tests  ±0   357 ✅ +2   24m 28s ⏱️ -29s
  1 suites ±0    16 💤 ±0 
  1 files   ±0     0 ❌  - 2 

Results for commit e1fe09e. ± Comparison against base commit 0f7f3c7.

@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 01ffecc into main Oct 23, 2025
23 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1761176493-bump-cdk-to-7.3.9 branch October 23, 2025 00:13
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.

1 participant