Skip to content

feat(rocketlane): Rocketlane integration — 64 tools across projects, tasks, phases, fields, time tracking, spaces, and invoices#5709

Open
waleedlatif1 wants to merge 3 commits into
stagingfrom
worktree-rocketlane-integration
Open

feat(rocketlane): Rocketlane integration — 64 tools across projects, tasks, phases, fields, time tracking, spaces, and invoices#5709
waleedlatif1 wants to merge 3 commits into
stagingfrom
worktree-rocketlane-integration

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • New Rocketlane integration: 64 tools covering projects, tasks, phases, custom fields, time entries, time-offs, users, spaces, space documents, resource allocations, and invoices
  • API-key auth (api-key header), all request/response shapes authored against the official OpenAPI definitions at developer.rocketlane.com — no guessed schemas
  • Block with per-operation subblocks, enum dropdowns matching spec values exactly, advanced-mode filters, and mutually-exclusive time-entry source selection
  • BlockMeta with 8 templates and 6 skills grounded in documented Rocketlane use cases
  • Generated docs page (integrations/rocketlane.mdx, all 64 actions)

Type of Change

  • New feature

Testing

  • Full typecheck clean, biome clean, check:api-validation and check:bare-icons pass, block/registry test suites pass (339 tests)
  • Every tool independently validated against the embedded OpenAPI specs by parallel review passes (params, URLs, methods, body nesting, response mapping, block wiring)

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

…tasks, phases, fields, time tracking, spaces, and invoices
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 16, 2026 10:26am

Request Review

@cursor

cursor Bot commented Jul 16, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Docs-only icon and navigation wiring with no runtime API or auth changes in this diff.

Overview
Adds Rocketlane branding to the docs site so the integration page and blocks can render the correct logo.

Introduces a new RocketlaneIcon SVG in icons.tsx, registers it on the rocketlane block type in icon-mapping.ts, and inserts rocketlane into the integrations docs sidebar order in meta.json.

Reviewed by Cursor Bugbot for commit f682759. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit 2faa233. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a full Rocketlane integration for workflow automation. The main changes are:

  • Adds 64 tools for projects, tasks, phases, fields, time tracking, spaces, allocations, and invoices.
  • Adds API-key authentication and Rocketlane request and response types.
  • Adds the Rocketlane block, operation controls, templates, skills, registries, and icons.
  • Adds generated integration documentation for all actions.

Confidence Score: 5/5

This looks safe to merge.

  • The latest fixes preserve explicit empty values for direct update-tool calls.
  • Missing user references are rejected before time-off and placeholder requests are built.
  • The time-entry-category operation is consistently connected across the selector, parameter mapping, access list, and registry.
  • No blocking issues remain in the reviewed changes.

Important Files Changed

Filename Overview
apps/sim/tools/rocketlane/update_time_entry.ts Preserves explicit empty activity names and notes while retaining the API-required date and minutes.
apps/sim/tools/rocketlane/update_project.ts Preserves explicit empty values for the updated optional project fields.
apps/sim/tools/rocketlane/create_time_off.ts Rejects time-off requests that do not identify a user.
apps/sim/tools/rocketlane/assign_placeholders.ts Rejects placeholder assignments that do not identify a user.
apps/sim/blocks/blocks/rocketlane.ts Defines the Rocketlane operation selector, parameter mappings, credentials, and tool access.
apps/sim/tools/registry.ts Registers the new Rocketlane tools for workflow execution.
apps/docs/content/docs/en/integrations/rocketlane.mdx Documents the Rocketlane integration and its 64 actions.

Reviews (7): Last reviewed commit: "chore(rocketlane): trigger fresh review ..." | Re-trigger Greptile

Comment thread apps/sim/tools/rocketlane/update_time_entry.ts
Comment thread apps/sim/tools/rocketlane/update_time_entry.ts Outdated
@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a full Rocketlane integration for workflow automation. The main changes are:

  • 65 registered tools for projects, tasks, phases, fields, time tracking, spaces, allocations, and invoices.
  • A Rocketlane block with operation-specific fields, templates, and skills.
  • API-key authentication through Rocketlane's api-key header.
  • Registry, integration catalog, icon, and generated documentation updates.

Confidence Score: 4/5

The missing operation and invalid empty-user request paths need fixes before merging.

  • One registered tool cannot be selected from the workflow editor.
  • Two mutation tools can send empty user references allowed by their input definitions.
  • Project updates silently ignore an empty external reference intended to clear the field.
  • Runtime type conversion and API-key header handling follow the existing execution path.

apps/sim/blocks/blocks/rocketlane.ts, apps/sim/tools/rocketlane/create_time_off.ts, apps/sim/tools/rocketlane/assign_placeholders.ts, apps/sim/tools/rocketlane/update_project.ts

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/rocketlane.ts Adds the Rocketlane block and runtime parameter mapping, but omits one registered operation from the selector.
apps/sim/tools/rocketlane/create_time_off.ts Creates time-off requests but allows an empty user reference to reach the API.
apps/sim/tools/rocketlane/assign_placeholders.ts Adds placeholder assignment support but does not enforce a user identifier.
apps/sim/tools/rocketlane/update_project.ts Updates project fields but drops an empty external reference intended to clear the value.
apps/sim/tools/registry.ts Registers the new Rocketlane tool surface, including the category-listing tool missing from the block selector.
apps/sim/tools/rocketlane/types.ts Adds shared Rocketlane API types, error parsing, response mapping, and header construction.
apps/sim/lib/integrations/integrations.json Adds Rocketlane catalog metadata but lists 64 operations while 65 tools are registered.
apps/docs/content/docs/en/integrations/rocketlane.mdx Documents the user-facing Rocketlane actions but mirrors the incomplete 64-operation catalog.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  UI[Rocketlane block] --> CFG[Runtime parameter mapping]
  CFG --> REG[Tool registry]
  REG --> EXEC[Generic tool executor]
  EXEC -->|api-key header| API[Rocketlane API]
  API --> RESP[Mapped workflow output]
  CAT[Catalog and documentation] -. describes .-> UI
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart LR
  UI[Rocketlane block] --> CFG[Runtime parameter mapping]
  CFG --> REG[Tool registry]
  REG --> EXEC[Generic tool executor]
  EXEC -->|api-key header| API[Rocketlane API]
  API --> RESP[Mapped workflow output]
  CAT[Catalog and documentation] -. describes .-> UI
Loading

Reviews (2): Last reviewed commit: "feat(rocketlane): Rocketlane integration..." | Re-trigger Greptile

Comment thread apps/sim/blocks/blocks/rocketlane.ts
Comment thread apps/sim/tools/rocketlane/create_time_off.ts
Comment thread apps/sim/tools/rocketlane/assign_placeholders.ts
Comment thread apps/sim/tools/rocketlane/update_project.ts Outdated
…user reference for time-off and placeholder assignment
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit dc0e403. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit dc0e403. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

Comment thread apps/sim/tools/rocketlane/update_time_entry.ts
Comment thread apps/sim/tools/rocketlane/update_project.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit dc0e403. Configure here.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit f682759. Configure here.

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