improvement(brex): fill validate-integration gaps against live API docs#5362
Conversation
- add wandConfig AI-autofill to expense/spend-limit filter subBlocks - surface is_ppro_enabled on get/list transfer tools - surface start_date/end_date/authorization_settings on list_spend_limits to match get_spend_limit
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The Brex block now has Transfer get and list tools and shared types now expose Reviewed by Cursor Bugbot for commit 0c06301. Configure here. |
Greptile SummaryThis PR fills gaps in the Brex integration by surfacing previously missing API fields and adding AI-autofill (
Confidence Score: 5/5All changes are strictly additive — new optional fields in output schemas, new wandConfig on filter subBlocks, and updated TypeScript interfaces — with no modifications to existing fields, request logic, or transformResponse mappings beyond get_transfer's new null-safe assignment. Every new field follows the established coding patterns: snake_case in raw-passthrough tools (list_transfers, list_spend_limits) and camelCase with explicit null-fallback in the single-item mapped tool (get_transfer). The wandConfig prompts are constrained and self-consistent with the existing timestamp wandConfig pattern. TypeScript interface updates correctly use | null and are reflected in the corresponding response interfaces. No fields removed, no endpoint changes, no breaking surface changes. No files require special attention. All five changed files are internally consistent and compatible with each other. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Block["brex.ts Block (UI)"]
B1["userIds subBlock - wandConfig added"]
B2["statuses subBlock - wandConfig added"]
B3["paymentStatuses subBlock - wandConfig added"]
B4["memberUserIds subBlock - wandConfig added"]
end
subgraph Tools["Tool Layer"]
T1["get_transfer.ts - isPproEnabled mapped camelCase"]
T2["list_transfers.ts - is_ppro_enabled output schema"]
T3["list_spend_limits.ts - start_date, end_date, authorization_settings"]
end
subgraph Types["types.ts"]
TY1["BrexTransfer + is_ppro_enabled"]
TY2["BrexSpendLimit + start_date, end_date, authorization_settings"]
TY3["BrexGetTransferResponse + isPproEnabled"]
end
subgraph API["Brex REST API"]
API1["GET /v1/transfers/:id"]
API2["GET /v1/transfers"]
API3["GET /v2/spend_limits"]
end
B1 --> T2
B4 --> T3
T1 --> TY3
T2 --> TY1
T3 --> TY2
T1 --> API1
T2 --> API2
T3 --> API3
%%{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 TD
subgraph Block["brex.ts Block (UI)"]
B1["userIds subBlock - wandConfig added"]
B2["statuses subBlock - wandConfig added"]
B3["paymentStatuses subBlock - wandConfig added"]
B4["memberUserIds subBlock - wandConfig added"]
end
subgraph Tools["Tool Layer"]
T1["get_transfer.ts - isPproEnabled mapped camelCase"]
T2["list_transfers.ts - is_ppro_enabled output schema"]
T3["list_spend_limits.ts - start_date, end_date, authorization_settings"]
end
subgraph Types["types.ts"]
TY1["BrexTransfer + is_ppro_enabled"]
TY2["BrexSpendLimit + start_date, end_date, authorization_settings"]
TY3["BrexGetTransferResponse + isPproEnabled"]
end
subgraph API["Brex REST API"]
API1["GET /v1/transfers/:id"]
API2["GET /v1/transfers"]
API3["GET /v2/spend_limits"]
end
B1 --> T2
B4 --> T3
T1 --> TY3
T2 --> TY1
T3 --> TY2
T1 --> API1
T2 --> API2
T3 --> API3
Reviews (1): Last reviewed commit: "improvement(brex): fill validate-integra..." | Re-trigger Greptile |
Summary
is_ppro_enabledon the get/list transfer tools — confirmed present in Brex's live Transfer schema but previously missing from our outputstart_date/end_date/authorization_settingsonlist_spend_limitsto match whatget_spend_limitalready exposes for the same resourceType of Change
Testing
Tested manually.
bun run lintandbun run type-checkpass clean.Checklist