Skip to content

source-commercetools: new capture connector - #5032

Merged
Alex-Bair merged 1 commit into
mainfrom
bair/source-commercetools
Aug 10, 2026
Merged

source-commercetools: new capture connector#5032
Alex-Bair merged 1 commit into
mainfrom
bair/source-commercetools

Conversation

@Alex-Bair

@Alex-Bair Alex-Bair commented Aug 8, 2026

Copy link
Copy Markdown
Member

Regression?

No.

Description:

This PR introduces a new, minimal connector source-commercetools that captures orders, customers and payments from commercetools' HTTP API.

Notes for reviewers:

This capture connector was created with the create-capture-connector skill with development credentials for the commercetools API. Those dev credentials expire after the 60 day free trial expires though, so we can't use them long-term for discover and capture snapshot tests.

Checklist:

  • If this change is user-visible, the affected connector's CHANGELOG.md and documentation have been updated (see CONTRIBUTING.md). The Docs / CHANGELOG check CI job reviews this automatically; apply the docs-check-skip label to dismiss a false positive.

@Alex-Bair Alex-Bair linked an issue Aug 8, 2026 that may be closed by this pull request
@Alex-Bair
Alex-Bair force-pushed the bair/source-commercetools branch 2 times, most recently from 7f10776 to 57b7644 Compare August 8, 2026 14:19
@Alex-Bair
Alex-Bair marked this pull request as ready for review August 8, 2026 14:21
@Alex-Bair
Alex-Bair requested a review from a team August 8, 2026 14:21
nicolaslazo
nicolaslazo previously approved these changes Aug 10, 2026

@nicolaslazo nicolaslazo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great! No comments

Alex-Bair added a commit to estuary/flow that referenced this pull request Aug 10, 2026
Alex-Bair added a commit to estuary/flow that referenced this pull request Aug 10, 2026
This commit introduces a new connector `source-commercetools` that
captures orders, customers and payments from commercetools' HTTP API.

Streams:
  orders, customers, payments    backfill + incremental

All three share one implementation. Every commercetools Query endpoint
returns the same `PagedQueryResponse` envelope and gives its resources the
same `id`/`lastModifiedAt` pair, so the streams differ only by path.

commercetools caps `offset` at 10,000, which rules out offset paging on
any real `Project`, so paging is keyset throughout. The incremental sweep
sorts by `(lastModifiedAt, id)` and resumes on a compound predicate past
the last document read. The backfill walks `id` ascending.

This connector was built without commercetools credentials. Endpoint
shapes come from commercetools' published API reference.

There is deliberately no test_capture, since it would need a real
account. Discovery's permission check is skipped for the
`TEST_CONFIG_PROJECT_KEY` sentinel -- a value no Project can hold, as
commercetools keys cannot contain a dot -- so spec and discover both run
offline.
@Alex-Bair

Copy link
Copy Markdown
Member Author

Post-approval, I made a couple edits to the docs based on feedback I received in estuary/flow#3332.

@Alex-Bair
Alex-Bair requested a review from nicolaslazo August 10, 2026 16:28

@nicolaslazo nicolaslazo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@Alex-Bair
Alex-Bair merged commit 6dbb792 into main Aug 10, 2026
132 of 207 checks passed
@Alex-Bair
Alex-Bair deleted the bair/source-commercetools branch August 10, 2026 16:34
Alex-Bair added a commit to estuary/flow that referenced this pull request Aug 10, 2026
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.

source-commercetools: new connector

2 participants