Feat: nexchange plugin#214
Open
j0ntz wants to merge 1 commit into
Open
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 59abfa0. Configure here.
d9a599c to
361b2a6
Compare
361b2a6 to
5591c9e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
Api Key should be shared privately.
Description
Recreated from #212 by external contributor @MMrj9 so that Bugbot (which skips external authors) can run. Git author attribution on the cherry-picked commit is preserved for Miguel Morujão. Original PR should be closed once this one merges.
Original description follows:
nexchangepartner plugin atsrc/partners/nexchange.ts.src/queryEngine.ts./audits/edge/orders) with support for:x-api-keyAuthorization: ApiKey <key>auth headerdateFromincremental sync using persistedlatestIsoDateand lookbacktest/nexchange.test.tsfor transaction mapping and auth header behavior.https://app.asana.com/1/9976422036640/project/1200382638405084/task/1213650284151456?focus=true
Note
Medium Risk
Introduces a new external API ingestion path and enables it in the query engine, which could affect sync completeness/duplication and error handling behavior for that partner. Core engine changes are limited to registration, but the new pagination/date logic needs verification against real API responses.
Overview
Adds a new Nexchange partner integration that fetches
/audits/edge/orders, supportsx-api-keyand/or legacyAuthorizationauth headers, paginates via cursor with offset fallback, and performs incremental sync using a persistedlatestIsoDatewith a 5-day lookback.Registers
nexchangeinsrc/queryEngine.tsso it runs alongside existing partners, and addstest/nexchange.test.tsto validateStandardTxmapping (status/date/amount normalization) and header construction.Reviewed by Cursor Bugbot for commit 5591c9e. Bugbot is set up for automated code reviews on this repo. Configure here.