ci: use prepared-release-id to finalize release draft by aaronsteers · Pull Request #1083 · airbytehq/PyAirbyte

Aaron ("AJ") Steers (aaronsteers) · 2026-07-21T04:02:35Z

Summary

Migrates the two-step release-drafter workflow to aaronsteers/semantic-pr-release-drafter@v2.2.0 and finalizes the draft via prepared-release-id instead of re-pinning version.

Before, the finalize step re-discovered its draft by re-passing the version:

version: ${{ steps.not-ready-draft.outputs.tag-name }}

That relied on the eventually-consistent GET /releases list, which can miss a draft created seconds earlier → a duplicate draft. Now the finalize targets the exact release created by the first step via a strongly consistent point-read:

prepared-release-id: ${{ steps.not-ready-draft.outputs.id }}

The prepared release is the source of truth: its version, tag, prerelease state, and pinned commit SHA are reused as-is, so nothing can drift between the two steps. (The build step still consumes tag-name for UV_DYNAMIC_VERSIONING_BYPASS — unchanged.)

Requested by Aaron ("AJ") Steers (@aaronsteers), following up on aaronsteers/semantic-pr-release-drafter#57 (the prepared-release-id feature, released in v2.2.0).

Link to Devin session: https://app.devin.ai/sessions/c5aa9e420587440b9e2e5f4e92f65383

Summary by CodeRabbit

Chores
- Updated the release automation workflow to use the latest release-drafting action.
- Improved draft release finalization to reliably attach assets to the correct prepared release.

Important

Auto-merge enabled.

This PR is set to merge automatically when all requirements are met.

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

devin-ai-integration · 2026-07-21T04:02:39Z

🤖 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

github-actions · 2026-07-21T04:02:53Z

👋 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/1784606532-prepared-release-id' pyairbyte --help

# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1784606532-prepared-release-id'

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.

coderabbitai · 2026-07-21T04:05:09Z

📝 Walkthrough

Walkthrough

The release workflow upgrades semantic-pr-release-drafter to v2.2.0 and finalizes drafts using the prepared release identifier returned by the creation step.

Changes

Release drafter workflow

Layer / File(s)	Summary
Draft release creation and finalization `.github/workflows/release_drafter.yml`	Both release steps use `v2.2.0`; finalization reuses the prepared release identifier from the earlier step instead of passing a tag name.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

airbytehq/PyAirbyte#1049: Introduced the related “not-ready” release drafter workflow later updated here.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main workflow change: finalizing the release draft with prepared-release-id.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch devin/1784606532-prepared-release-id

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.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/release_drafter.yml:
- Line 34: Update both semantic-pr-release-drafter steps in the release workflow
to reference commit f9c46b42299873804471af7a5cb066d0a53d7e8c instead of the
mutable v2.2.0 tag, preserving the existing action configuration.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 5b48e202-1be6-4997-bafd-c00047bfc265

📥 Commits

Reviewing files that changed from the base of the PR and between 1d9b3ca and 716a6d5.

📒 Files selected for processing (1)

.github/workflows/release_drafter.yml

github-code-quality · 2026-07-21T04:09:59Z

Code Coverage Overview

Languages: Python

Python / code-coverage/pytest-fast

The overall coverage in commit 716a6d5 in the devin/1784606532-pre... branch is 67%. The coverage in commit d9f652f in the main branch is 65%.

Show a code coverage summary of the most impacted files.

File	`main` d9f652f	`devin/1784606532-pre...` 716a6d5	+/-
`airbyte/_util/api_util.py`	36%	37%	+1%
`airbyte/mcp/_tool_utils.py`	72%	79%	+7%
`airbyte/mcp/registry.py`	53%	70%	+17%
`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 commit 716a6d5 in the devin/1784606532-pre... branch is 67%. The coverage in commit d9f652f in the main branch is 65%.

Show a code coverage summary of the most impacted files.

File	`main` d9f652f	`devin/1784606532-pre...` 716a6d5	+/-
`airbyte/_util/api_util.py`	36%	37%	+1%
`airbyte/mcp/_tool_utils.py`	72%	79%	+7%
`airbyte/mcp/registry.py`	53%	70%	+17%
`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 commit 716a6d5 in the devin/1784606532-pre... branch is 72%. The coverage in commit d9f652f in the main branch is 71%.

Show a code coverage summary of the most impacted files.

File	`main` d9f652f	`devin/1784606532-pre...` 716a6d5	+/-
`airbyte/registry.py`	70%	72%	+2%
`airbyte/mcp/_tool_utils.py`	72%	79%	+7%
`airbyte/mcp/registry.py`	53%	70%	+17%
`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 21, 2026 04:29 UTC}

devin-ai-integration

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Copilot

Pull request overview

Updates the GitHub Actions release-drafter workflow to avoid duplicate draft releases by finalizing the exact “prepared” draft via prepared-release-id after building artifacts.

Changes:

Bumps aaronsteers/semantic-pr-release-drafter from v2.1.0 to v2.2.0.
Finalizes the prepared draft using prepared-release-id: ${{ steps.not-ready-draft.outputs.id }} instead of rediscovering by version.
Adds inline comments documenting the rationale (point-read consistency, reuse of version/tag/SHA).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ci: use prepared-release-id to finalize release draft

716a6d5

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

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

github-advanced-security AI found potential problems Jul 21, 2026

View reviewed changes

Comment thread .github/workflows/release_drafter.yml Dismissed

Comment thread .github/workflows/release_drafter.yml Dismissed

coderabbitai Bot requested changes Jul 21, 2026

View reviewed changes

Comment thread .github/workflows/release_drafter.yml

Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review July 21, 2026 04:19

Copilot AI review requested due to automatic review settings July 21, 2026 04:19

Copilot started reviewing on behalf of Aaron ("AJ") Steers (aaronsteers) July 21, 2026 04:20 View session

devin-ai-integration Bot reviewed Jul 21, 2026

View reviewed changes

coderabbitai Bot approved these changes Jul 21, 2026

View reviewed changes

Copilot AI reviewed Jul 21, 2026

View reviewed changes

Comment thread .github/workflows/release_drafter.yml

Comment thread .github/workflows/release_drafter.yml

Aaron ("AJ") Steers (aaronsteers) enabled auto-merge (squash) July 21, 2026 04:22

Aaron ("AJ") Steers (aaronsteers) merged commit 65e4309 into main Jul 21, 2026
30 checks passed

Aaron ("AJ") Steers (aaronsteers) deleted the devin/1784606532-prepared-release-id branch July 21, 2026 04:29

coderabbitai Bot mentioned this pull request Jul 24, 2026

ci: bump semantic-pr-release-drafter to v2.3.1 to unblock release draft finalize #1087

Merged

Uh oh!

Conversation

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

Uh oh!

Summary

Summary by CodeRabbit

Uh oh!

devin-ai-integration Bot commented Jul 21, 2026

🤖 Devin AI Engineer

Uh oh!

github-actions Bot commented Jul 21, 2026

👋 Greetings, Airbyte Team Member!

Testing This PyAirbyte Version

PR Slash Commands

Helpful Resources

Community Support

Uh oh!

Uh oh!

Uh oh!

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

Uh oh!

Walkthrough

Changes

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-code-quality Bot commented Jul 21, 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!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

✅ Devin Review: No Issues Found

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

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

edited by github-actions Bot

Loading

coderabbitai Bot commented Jul 21, 2026 •

edited

Loading

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

edited

Loading