[Upgrade] Migrate Tellent HR to new Authentication #18913

jcortes · 2025-10-30T21:21:34Z

WHY

Resolves #18904

Summary by CodeRabbit

Bug Fixes
- GraphQL endpoint now uses subdomain-based URLs for more reliable API calls.
- API authentication header standardized to a lowercase key for consistent requests.
Chores
- Bumped package and component version numbers across the KiwiHR integration for release management.

vercel · 2025-10-30T21:21:39Z

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

2 Skipped Deployments

Project	Deployment	Preview	Comments	Updated (UTC)
pipedream-docs	Ignored			Oct 31, 2025 4:56pm
pipedream-docs-redirect-do-not-edit	Ignored			Oct 31, 2025 4:56pm

coderabbitai · 2025-10-30T21:21:53Z

Walkthrough

KiwiHR component versions bumped to 1.0.0; GraphQL endpoint now built from the configured subdomain (https://{subdomain}.kiwihr.com/api/graphql); API key header changed to lowercase x-api-key. No other behavioral changes reported.

Changes

Cohort / File(s)	Summary
Actions version bumps `components/kiwihr/actions/create-employee/create-employee.mjs`, `components/kiwihr/actions/update-employee-record/update-employee-record.mjs`	`version` updated from `"0.0.2"` → `"1.0.0"`
Source version bump `components/kiwihr/sources/new-employee/new-employee.mjs`	`version` updated from `"0.0.1"` → `"1.0.0"`
Package manifest version `components/kiwihr/package.json`	Top-level `version` updated from `"0.1.1"` → `"1.0.0"`
Authentication migration / app changes `components/kiwihr/kiwihr.app.mjs`	`_baseUrl()` now builds `https://${subdomain}.kiwihr.com/api/graphql`; `_headers()` uses `x-api-key` (lowercase). No signature changes.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Component as KiwiHR Component
    participant App as kiwihr.app.mjs
    participant GraphQL as kiwihr GraphQL

    Note over Component,App: Prepare request using this.kiwihr.$auth
    Component->>App: provide query/data
    App->>App: construct URL: https://${subdomain}.kiwihr.com/api/graphql
    App->>GraphQL: POST\nHeader: x-api-key: <api_key>\nBody: GraphQL query
    GraphQL-->>App: Response
    App-->>Component: Response payload

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Review attention items:
- components/kiwihr/kiwihr.app.mjs — validate subdomain handling, URL construction, header casing, and error behavior when auth props are missing.
- components/kiwihr/package.json — confirm intentional major version bump.
- Action/source files — ensure only version strings were changed.

Poem

🐰 I hopped through lines of code anew,
Subdomains guiding every view,
A tiny header lowered with grace,
Versions aligned in tidy place,
🥕 Cheers from a rabbit, light and true.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title "[Upgrade] Migrate Tellent HR to new Authentication" clearly reflects the primary objective of the PR, which is to migrate the KiwiHR integration to use the new authentication approach. The title is concise and appropriately conveys that this is a feature upgrade focused on authentication changes. While the title doesn't specify the technical details (subdomain-based URL and header format changes), it sufficiently communicates the main purpose for developers scanning the pull request history.
Linked Issues Check	✅ Passed	The changes in `components/kiwihr/kiwihr.app.mjs` directly implement the core requirements from issue #18904: the URL construction now uses the subdomain pattern (`https://${subdomain}.kiwihr.com/api/graphql`) and the API key header is now lowercase (`x-api-key`), both matching the required authentication strategy. The version bumps across all components signal a coordinated migration of the KiwiHR integration, and since these components depend on the app configuration file for authentication, they will automatically use the new authentication approach. The changes satisfy the primary coding-related objectives of the linked issue.
Out of Scope Changes Check	✅ Passed	All changes in the PR are directly related to the authentication migration objective. The functional changes in `kiwihr.app.mjs` implement the new auth strategy, and the version bumps across all component files (creating a consistent v1.0.0 across the integration) represent a reasonable practice for releasing a major feature update. These version increments are typical for coordinated releases and serve to signal the significance of the authentication migration to users of the integration.
Description Check	✅ Passed	The PR description follows the repository's template by including a "WHY" section with a reference to issue #18904. While the description is minimal and could benefit from more detail about the specific changes, it adequately documents the purpose by linking to the issue that contains the full requirements and context. The description is not vague or generic; it directly references the problem being solved.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch upgrade-kiwihr-url-and-headers-migration

📜 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 91118cc and 766b7cf.

📒 Files selected for processing (5)

components/kiwihr/actions/create-employee/create-employee.mjs (1 hunks)
components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1 hunks)
components/kiwihr/kiwihr.app.mjs (1 hunks)
components/kiwihr/package.json (1 hunks)
components/kiwihr/sources/new-employee/new-employee.mjs (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

components/kiwihr/actions/create-employee/create-employee.mjs
components/kiwihr/package.json

⏰ 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). (4)

GitHub Check: pnpm publish
GitHub Check: Lint Code Base
GitHub Check: Publish TypeScript components
GitHub Check: Verify TypeScript components

🔇 Additional comments (2)

components/kiwihr/sources/new-employee/new-employee.mjs (1)

9-9: Version bump is appropriate.

The major version bump from "0.0.1" to "1.0.0" correctly reflects the breaking authentication changes in this PR.

components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1)

9-9: Version bump is appropriate.

The major version bump from "0.0.2" to "1.0.0" correctly reflects the breaking authentication changes in this PR.

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

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ca39807 and 7cdae65.

📒 Files selected for processing (5)

components/kiwihr/actions/create-employee/create-employee.mjs (1 hunks)
components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1 hunks)
components/kiwihr/kiwihr.app.mjs (1 hunks)
components/kiwihr/package.json (1 hunks)
components/kiwihr/sources/new-employee/new-employee.mjs (1 hunks)

⏰ 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). (4)

GitHub Check: Lint Code Base
GitHub Check: Verify TypeScript components
GitHub Check: pnpm publish
GitHub Check: Publish TypeScript components

🔇 Additional comments (4)

components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1)

9-9: LGTM! Appropriate version bump.

The patch version increment correctly reflects the authentication migration changes without breaking the action's interface.

components/kiwihr/package.json (1)

3-3: LGTM! Appropriate package version bump.

The minor version increment correctly reflects the authentication migration changes across the KiwiHR integration.

components/kiwihr/actions/create-employee/create-employee.mjs (1)

9-9: LGTM! Appropriate version bump.

The patch version increment correctly reflects the authentication migration changes without breaking the action's interface.

components/kiwihr/sources/new-employee/new-employee.mjs (1)

9-9: LGTM! Appropriate version bump.

The patch version increment correctly reflects the authentication migration changes without breaking the source's interface.

components/kiwihr/kiwihr.app.mjs

GTFalcao

This will break existing flows when updated - I think it should get a major version bump (i.e. 1.0.0)

jcortes · 2025-10-31T14:22:13Z

/approve

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 72b025d and 91118cc.

📒 Files selected for processing (5)

components/kiwihr/actions/create-employee/create-employee.mjs (1 hunks)
components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1 hunks)
components/kiwihr/kiwihr.app.mjs (1 hunks)
components/kiwihr/package.json (1 hunks)
components/kiwihr/sources/new-employee/new-employee.mjs (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (3)

components/kiwihr/kiwihr.app.mjs
components/kiwihr/package.json
components/kiwihr/sources/new-employee/new-employee.mjs

🔇 Additional comments (2)

components/kiwihr/actions/create-employee/create-employee.mjs (1)

9-9: LGTM! Version bump aligns with authentication migration.

The version bump to "0.0.3" appropriately indicates that this action now works with the updated authentication approach implemented in the kiwihr app connector.

components/kiwihr/actions/update-employee-record/update-employee-record.mjs (1)

9-9: LGTM! Version bump aligns with authentication migration.

The version bump to "0.0.3" appropriately indicates that this action now works with the updated authentication approach implemented in the kiwihr app connector.

components/kiwihr/actions/update-employee-record/update-employee-record.mjs

jcortes · 2025-10-31T16:57:08Z

/approve

jcortes self-assigned this Oct 30, 2025

pipedream-component-development requested a review from GTFalcao October 30, 2025 21:22

coderabbitai bot reviewed Oct 30, 2025

View reviewed changes

components/kiwihr/kiwihr.app.mjs Show resolved Hide resolved

GTFalcao reviewed Oct 30, 2025

View reviewed changes

jcortes force-pushed the upgrade-kiwihr-url-and-headers-migration branch 2 times, most recently from 72b025d to 91118cc Compare October 31, 2025 14:21

pipedream-component-development requested a review from GTFalcao October 31, 2025 14:22

coderabbitai bot reviewed Oct 31, 2025

View reviewed changes

components/kiwihr/actions/update-employee-record/update-employee-record.mjs Show resolved Hide resolved

[Upgrade] Migrate Tellent HR to new Authentication

766b7cf

jcortes force-pushed the upgrade-kiwihr-url-and-headers-migration branch from 91118cc to 766b7cf Compare October 31, 2025 16:56

GTFalcao approved these changes Oct 31, 2025

View reviewed changes

jcortes merged commit 54c05a8 into master Oct 31, 2025
10 checks passed

jcortes deleted the upgrade-kiwihr-url-and-headers-migration branch October 31, 2025 17:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Upgrade] Migrate Tellent HR to new Authentication #18913

[Upgrade] Migrate Tellent HR to new Authentication #18913

Uh oh!

jcortes commented Oct 30, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

vercel bot commented Oct 30, 2025 •

edited

Loading

Uh oh!

coderabbitai bot commented Oct 30, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

GTFalcao left a comment

Uh oh!

jcortes commented Oct 31, 2025

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

jcortes commented Oct 31, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[Upgrade] Migrate Tellent HR to new Authentication #18913

[Upgrade] Migrate Tellent HR to new Authentication #18913

Uh oh!

Conversation

jcortes commented Oct 30, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

WHY

Summary by CodeRabbit

Uh oh!

vercel bot commented Oct 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Oct 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

GTFalcao left a comment

Choose a reason for hiding this comment

Uh oh!

jcortes commented Oct 31, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

jcortes commented Oct 31, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jcortes commented Oct 30, 2025 •

edited by coderabbitai bot

Loading

vercel bot commented Oct 30, 2025 •

edited

Loading

coderabbitai bot commented Oct 30, 2025 •

edited

Loading