Skip to content

Change Authorization header in Airbyte components#19734

Merged
michelle0927 merged 8 commits intomasterfrom
issue-19365
Jan 26, 2026
Merged

Change Authorization header in Airbyte components#19734
michelle0927 merged 8 commits intomasterfrom
issue-19365

Conversation

@michelle0927
Copy link
Collaborator

@michelle0927 michelle0927 commented Jan 19, 2026

Resolves #19365

Summary by CodeRabbit

  • New Features

    • List workspaces action now displays dynamic response showing the count of workspaces retrieved.
  • Style

    • Updated workspace field labels for improved consistency and clarity.
  • Chores

    • Bumped action versions (0.0.2 → 0.0.3) and package version (0.1.0 → 0.2.0).
    • Updated platform dependencies.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
pipedream-docs-redirect-do-not-edit Ignored Ignored Jan 26, 2026 3:28pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 19, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Multiple Airbyte components were updated with version bumps across action files. Authorization mechanism switched from API key to OAuth access token in the main app configuration. The app file refactored URL construction to use template literals and updated field label capitalization. Platform dependency and package version were incremented.

Changes

Cohort / File(s) Summary
Action Version Updates
components/https_airbyte_com/actions/create-workspace/create-workspace.mjs, components/https_airbyte_com/actions/delete-workspace/delete-workspace.mjs, components/https_airbyte_com/actions/update-workspace/update-workspace.mjs
Version property incremented from 0.0.2 to 0.0.3 across three workspace action files
Dynamic Summary Output
components/https_airbyte_com/actions/list-workspaces/list-workspaces.mjs
Version bumped to 0.0.3; summary message refactored from static string to dynamic pluralized output: "Successfully retrieved X workspace(s)"
App Configuration & Authorization
components/https_airbyte_com/https_airbyte_com.app.mjs
Authorization header source switched from api_key to oauth_access_token; URL construction refactored to template literals; field labels updated to title case ("Workspace Name", "Include Deleted")
Package Configuration
components/https_airbyte_com/package.json
Package version bumped from 0.1.0 to 0.2.0; @pipedream/platform dependency updated from ^1.6.8 to ^3.1.1

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (2 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description only contains 'Resolves #19365' and does not follow the required template with a 'WHY' section explaining the rationale. Add a 'WHY' section to the description template explaining the motivation behind the authorization header change and API migration.
Out of Scope Changes check ❓ Inconclusive Several changes appear tangential to the core objective: label capitalization updates, URL construction refactoring, and platform dependency upgrade may exceed the stated scope. Clarify whether label capitalization, URL template literal refactoring, and dependency upgrades are necessary for the authorization header migration or should be separated.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The PR title clearly describes the main change: switching from API key to OAuth access token authorization in Airbyte components.
Linked Issues check ✅ Passed The PR successfully implements the requirements from issue #19365: updated authorization to use oauth_access_token, refactored the app file, and bumped versions accordingly.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
components/https_airbyte_com/actions/update-workspace/update-workspace.mjs (1)

2-2: Address or remove the debug comment indicating a bug.

This Portuguese comment translates to: "Returns success, but is creating new workspaces instead of updating them." If this is a known bug, it should be tracked in an issue and either fixed or documented. If the issue has been resolved, remove this comment.

Would you like me to open an issue to track this bug, or has this been resolved with the API/auth changes?

🤖 Fix all issues with AI agents
In `@components/https_airbyte_com/package.json`:
- Line 3: The package.json version was bumped to "0.2.0" for the new OAuth
feature but you must also ensure the `@pipedream/platform` dependency is updated
to the project-standard ^3.1.1 and confirm component action compatibility;
update the "@pipedream/platform" entry in package.json if not already, run the
component's action unit/integration tests and manual smoke tests for actions
(particularly OAuth flows) and address any API/typing changes surfaced by
platform 3.1.1 in the action handlers (search for files referencing
`@pipedream/platform` or OAuth logic).

lcaresia
lcaresia previously approved these changes Jan 22, 2026
Copy link
Collaborator

@lcaresia lcaresia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vunguyenhung
Copy link
Contributor

Hi everyone, all test cases are passed! Ready for release!

Test reports

@michelle0927
Copy link
Collaborator Author

/approve

@michelle0927
Copy link
Collaborator Author

/approve

@michelle0927 michelle0927 merged commit e8c2c68 into master Jan 26, 2026
9 checks passed
@michelle0927 michelle0927 deleted the issue-19365 branch January 26, 2026 17:30
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.

[FEATURE] Change Authorization header in Airbyte components

3 participants