Skip to content

add hr and reports command groups#20

Open
sumedhkane03 wants to merge 1 commit intoRippling:mainfrom
sumedhkane03:feat/hr-reports-cli
Open

add hr and reports command groups#20
sumedhkane03 wants to merge 1 commit intoRippling:mainfrom
sumedhkane03:feat/hr-reports-cli

Conversation

@sumedhkane03
Copy link

@sumedhkane03 sumedhkane03 commented Feb 24, 2026

Summary

  • add new top-level hr and reports command groups using existing authenticated API client patterns
  • implement employee directory, time-off, payroll, and reports list/run/export command flows with guarded fallback endpoint handling
  • fix CLI entrypoint registration so poetry run rippling ... includes all registered commands, and add CliRunner tests + README usage docs

Test plan

  • poetry run pytest
  • poetry run rippling -h
  • poetry run rippling hr -h
  • poetry run rippling reports -h

Made with Cursor


Note

Medium Risk
Adds new command surfaces that call authenticated internal endpoints and write export files, so incorrect endpoint selection/response handling could cause runtime errors or unexpected output, though changes are isolated to new modules and include basic tests.

Overview
Adds new top-level rippling hr and rippling reports command groups, including HR employee directory (list/get), time-off and payroll summary queries, and report list/run/export workflows with basic output formatting.

Introduces hr_reports_utils helpers to build an authenticated API client (optionally with role/company headers), parse JSON params, and try multiple candidate internal endpoints with graceful fallback when an org lacks access.

Updates CLI startup to register commands at import time (so poetry run rippling ... exposes all commands), adds CliRunner tests covering help text and key error/success paths, and expands README.md with usage examples and testing instructions.

Written by Cursor Bugbot for commit f55c332. Configure here.

Introduce new authenticated hr and reports command surfaces for employee, payroll, time-off, and report workflows by reusing existing API/auth patterns. Also fix CLI command registration for packaged entrypoints and add tests/docs for the new commands.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sumedhkane03 sumedhkane03 requested review from a team as code owners February 24, 2026 19:49
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before Autofix could start.

Comment @cursor review or bugbot run to trigger another review on this PR

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