feat(pitchbook): add PitchBook integration - #6876
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryCursor Bugbot is generating a summary for commit 574b8aa. Configure here. |
574b8aa to
fb191a1
Compare
fb191a1 to
3ba5ca2
Compare
|
@cursor review |
Greptile SummaryThe PR adds a complete PitchBook integration spanning its workflow block, 91 API tools, generated metadata and output schemas, registry wiring, icons, fixtures, tests, deployment configuration, and user documentation.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/blocks/blocks/pitchbook.ts | Defines the PitchBook workflow block, operation selection, inputs, parameter mapping, and output exposure. |
| apps/sim/tools/pitchbook/utils.ts | Centralizes PitchBook authentication headers, query construction, and sanitized API error handling. |
| apps/sim/tools/pitchbook/pitchbook.test.ts | Adds integration wiring coverage for registry alignment, URL and parameter construction, response normalization, and 401 credential scrubbing. |
| apps/sim/tools/registry.ts | Registers the new PitchBook tools for generic workflow execution. |
| apps/docs/content/docs/en/integrations/pitchbook.mdx | Documents authentication, filtering conventions, inputs, and outputs for the PitchBook operations. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
User[Workflow user or agent] --> Block[PitchBook block]
Block --> Registry[Tool registry]
Registry --> Tool[Selected PitchBook tool]
Tool --> API[PitchBook Public API v2]
API --> Normalize[Response normalization]
Normalize --> Output[Workflow outputs]
Reviews (2): Last reviewed commit: "feat(pitchbook): add PitchBook integrati..." | Re-trigger Greptile
3ba5ca2 to
f94a5e2
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ 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 f94a5e2. Configure here.
Summary
Notes
Authorization: PB-Token {key}, not a bearer token. A 401 from PitchBook echoes the submitted key back in its message, so the error extractor replaces that one case with a fixed string rather than surfacing it into logs and block output.>2023-01-01,1^500), so filter values pass through untouched. Any documented filter without a dedicated field goes throughAdditional Filtersas JSON.activeContracton Contracts History is tri-state (active only / past only / omitted returns all), so it is a dropdown rather than a switch — a switch cannot express the omitted case.GET /limits. 15 arrays keep an untyped item shape for the same reason — the recorded sample was empty.Type of Change
Testing
Tested manually. 16 wiring tests cover tool/registry/block alignment, URL and query construction, param remapping, tri-state contract filtering, identifier trimming, API-key scrubbing on 401, and response normalization.
bun run lint,bun run check:audits(29/29), canvas sentences (91/91 operations), and type-check all pass.Checklist