feat(netsuite): add Oracle NetSuite integration - #6476
feat(netsuite): add Oracle NetSuite integration#6476BillLeoutsakosvl346 wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview The new Oracle NetSuite block exposes 27 operations: record CRUD/upsert, subresources, forms, select options, attach/detach, actions, transforms, SuiteQL, SuiteAnalytics datasets, metadata discovery, async batch jobs (status/result polling), server time, and governance limits. Operation-specific sub-blocks handle paging, JSON record bodies, batch items, and advanced options; seven templates and five agent skills ship with the block. Docs and catalog surfaces are updated: Reviewed by Cursor Bugbot for commit 61407f5. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR adds an Oracle NetSuite integration backed by SuiteTalk REST Web Services and OAuth 2.0 client credentials.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/tools/netsuite/utils.ts | Implements account normalization, OAuth certificate signing, private-key-bound token caching, request construction, response normalization, and error sanitization. |
| apps/sim/tools/netsuite/utils.test.ts | Covers OAuth signing, token-cache isolation by private-key identity, request behavior, and utility validation. |
| apps/sim/blocks/blocks/netsuite.ts | Defines the 27-operation NetSuite block, its credential and operation inputs, parameter coercion, outputs, templates, and skills. |
| apps/sim/blocks/blocks/netsuite.test.ts | Verifies operation routing, required fields, coercion, metadata, and the configured absolute import alias. |
| apps/sim/tools/netsuite/netsuite.test.ts | Exercises the NetSuite tool configurations and operation-specific HTTP request contracts. |
| apps/sim/tools/registry.ts | Registers the new NetSuite tools for runtime dispatch. |
Sequence Diagram
sequenceDiagram
participant Workflow
participant Block as NetSuite Block
participant Tool as NetSuite Tool
participant Cache as Token Cache
participant OAuth as NetSuite OAuth
participant API as SuiteTalk REST API
Workflow->>Block: Execute selected operation
Block->>Tool: Map and coerce operation parameters
Tool->>Cache: Lookup by account, client, certificate, and key fingerprint
alt Valid cached token
Cache-->>Tool: Bearer token
else Cache miss
Tool->>OAuth: Signed client-credentials assertion
OAuth-->>Tool: Access token
Tool->>Cache: Store bounded token entry
end
Tool->>API: Authenticated SuiteTalk request
API-->>Tool: Status, data, and location
Tool-->>Workflow: Normalized operation result
Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile
|
@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 cdd5240. Configure here.
|
@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 61407f5. Configure here.
Summary
Type of Change
Testing
Checklist