Skip to content

Breezy HR - Get Candidate#20476

Merged
GTFalcao merged 11 commits intoPipedreamHQ:masterfrom
RoboSourceTeam:breezy-hr-get-candidate
Apr 15, 2026
Merged

Breezy HR - Get Candidate#20476
GTFalcao merged 11 commits intoPipedreamHQ:masterfrom
RoboSourceTeam:breezy-hr-get-candidate

Conversation

@caleblehman-robosource
Copy link
Copy Markdown
Contributor

@caleblehman-robosource caleblehman-robosource commented Apr 1, 2026

WHY

Allow the user to get details about a candidate given a companyID, positionID, and candidateID

Summary by CodeRabbit

  • New Features

    • Added a "Get Candidate" action to retrieve candidate details by company, position, and candidate ID.
    • Expanded Breezy HR integration with company, position, and candidate selectors and new API-backed lookup capabilities.
  • Chores

    • Bumped Breezy HR component version to 0.7.0.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pipedream-docs-redirect-do-not-edit Ignored Ignored Apr 15, 2026 0:06am

Request Review

@pipedream-component-development
Copy link
Copy Markdown
Collaborator

Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified.

@pipedream-component-development
Copy link
Copy Markdown
Collaborator

Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: f5bcd1ab-e967-4430-b694-16973d53d729

📥 Commits

Reviewing files that changed from the base of the PR and between ed1e3f1 and 4351523.

📒 Files selected for processing (1)
  • components/breezy_hr/actions/get-candidate/get-candidate.mjs

Walkthrough

Adds a new "Get Candidate" action and expands the Breezy HR app with REST helpers, prop definitions for companyId/positionId/candidateId, listing methods, and a candidate retrieval method; package version bumped.

Changes

Cohort / File(s) Summary
Get Candidate Action
components/breezy_hr/actions/get-candidate/get-candidate.mjs
New read-only action breezy_hr-get-candidate that accepts companyId, positionId, candidateId, calls this.breezyHr.getCandidate, exports a $summary, and returns the candidate data.
Breezy HR App + API helpers
components/breezy_hr/breezy_hr.app.mjs
Added REST utilities (_accessToken, _headers, _makeRequest, _normalizeArray, _entityId), dynamic propDefinitions for companyId/positionId/candidateId with options() that call API methods, and implemented listCompanies, listPositions, listPositionCandidates, and getCandidate. Removed authKeys().
Package metadata
components/breezy_hr/package.json
Bumped component version from 0.6.0 to 0.7.0.

Sequence Diagram(s)

sequenceDiagram
  participant Action as "Get Candidate Action"
  participant App as "BreezyHR App (this.breezyHr)"
  participant API as "Breezy HR API"

  Action->>App: getCandidate({ $, companyId, positionId, candidateId })
  App->>App: _makeRequest(path: /company/.../candidate/...)
  App->>API: HTTP GET /company/{companyId}/position/{positionId}/candidate/{candidateId}\n(headers: Authorization)
  API-->>App: 200 OK (candidate data)
  App-->>Action: returns data
  Action->>Action: $.export("$summary", "Candidate retrieved successfully")
  Action-->>$: returns data
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Breezy HR - Get Candidate' directly and clearly summarizes the main change—adding a new Get Candidate action to the Breezy HR integration.
Description check ✅ Passed The pull request description includes the WHY section explaining the feature's purpose, though it lacks details about implementation approach, testing, or impacts.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Copy Markdown
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: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@components/breezy_hr/actions/get-candidate/get-candidate.mjs`:
- Around line 50-55: The step summary currently embeds candidate identifiers via
the computed fullName and label and exports them with $.export("$summary",
`Retrieved candidate ${label}`); remove the sensitive data from the summary by
replacing that export with a generic message (e.g., "Candidate retrieved
successfully") and stop referencing fullName/label/candidateId in the summary;
leave any internal logging that needs identifiers separate and ensure only
non-PII text is exported by $.export("$summary").

In `@components/breezy_hr/package.json`:
- Line 3: Add a new CHANGELOG.md for the 0.7.0 release: create a changelog file
for the breezy_hr component that follows the project template, include the
header and a new section for version "0.7.0" with the release date placeholder
(YYYY-MM-DD) and an "Added" subsection containing a bullet describing the new
feature/action; ensure the file name is CHANGELOG.md and the version string
matches "0.7.0" from package.json.
🪄 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: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: d93eaeb8-f3cc-4799-b06d-5d8888922752

📥 Commits

Reviewing files that changed from the base of the PR and between bd6602f and ed1e3f1.

📒 Files selected for processing (3)
  • components/breezy_hr/actions/get-candidate/get-candidate.mjs
  • components/breezy_hr/breezy_hr.app.mjs
  • components/breezy_hr/package.json

Comment thread components/breezy_hr/actions/get-candidate/get-candidate.mjs Outdated
Comment thread components/breezy_hr/package.json
Copy link
Copy Markdown
Collaborator

@GTFalcao GTFalcao left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, looks good to me!

@GTFalcao GTFalcao merged commit 77e545d into PipedreamHQ:master Apr 15, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for PR Review to Done in Component (Source and Action) Backlog Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

User submitted Submitted by a user

Development

Successfully merging this pull request may close these issues.

6 participants