Skip to content

feat(integration-toolkit): add pricing-file import operations - #425

Closed
paulofph wants to merge 1 commit into
mainfrom
feat/erp-imports-integration-toolkit
Closed

feat(integration-toolkit): add pricing-file import operations#425
paulofph wants to merge 1 commit into
mainfrom
feat/erp-imports-integration-toolkit

Conversation

@paulofph

@paulofph paulofph commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

Regenerates @epilot/integration-toolkit-client and @epilot/sdk from the current live spec (https://docs.api.epilot.io/integration-toolkit.yaml) to pick up five operations merged in erp-integration-api MR !145 under /v2/erp/imports:

  • createErpImportPOST /v2/erp/imports, create a pricing-file import job from an uploaded file, starts the validate phase
  • listErpImportsGET /v2/erp/imports, list recent import jobs for the org, cursor-paginated
  • getErpImportGET /v2/erp/imports/{importId}, poll a job's status, validation summary, and progress
  • executeErpImportPOST /v2/erp/imports/{importId}:execute, confirm and run the write phase of a validated (READY) job
  • abortErpImportPOST /v2/erp/imports/{importId}:abort, cooperatively stop a running job

Purely additive — diffed the spec change and confirmed no existing operations were removed or renamed.

Scope of the diff

Only integration-toolkit-related generated files: the client's openapi.json/openapi.d.ts/openapi-runtime.json, and the aggregator's corresponding definitions/types/docs for that one API. Nothing else in either package was touched — integration-toolkit was already a registered API, so this only adds operations to it, not a new subpath.

Not yet released

A changeset is included (@epilot/integration-toolkit-client minor, @epilot/sdk minor), but changeset version / changeset publish has deliberately not been run. This repo's registry auth is live in the environment this PR was prepared in, so that step is left for a maintainer to run as its own explicit action.

Test plan

  • npm run openapi && npm run typegen && npm run build in clients/integration-toolkit-client — clean, openapi.d.ts now contains all five new operations
  • pnpm generate-sdk at the repo root — regenerated only the integration-toolkit entry, confirmed via git status
  • pnpm build in packages/epilot-sdk-v2 — clean
  • pnpm lint in packages/epilot-sdk-v2 — clean, no fixes needed
  • pnpm test in packages/epilot-sdk-v2 — 25 failures, all in operations.test.ts (generic custom-API/override mechanics unrelated to integration-toolkit), all ERR_INVALID_URL from the sandbox's proxy env var interacting with msw's request interceptor. Confirmed pre-existing via git stash + rerun against unmodified main — identical 25 failures, same tests. Left untouched as out of scope for this change.

Generated by Claude Code

Regenerates @epilot/integration-toolkit-client and @epilot/sdk from the
now-current live spec to pick up five operations merged in
erp-integration-api MR !145 under /v2/erp/imports: createErpImport,
listErpImports, getErpImport, executeErpImport, abortErpImport.

Purely additive — no existing operations changed or removed (verified
by diff). Client and aggregator both rebuilt clean; lint clean.

Not yet released: changeset is included but version-packages/
publish-packages has deliberately not been run.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018KNRiHh2NL8KHqudrexZRZ

paulofph commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Closing — superseded by #424 + #426, both already merged, which regenerate the exact same five integration-toolkit operations (createErpImport, listErpImports, getErpImport, executeErpImport, abortErpImport). Confirmed the two are equivalent: main now has 15 matches for those operation names in both packages/epilot-sdk-v2/src/types/integration-toolkit.d.ts and clients/integration-toolkit-client/src/openapi.d.ts, identical to what this branch produced.

This PR was prepared without visibility into #424/#426, which were merged in the same window. No action needed on this one.


Generated by Claude Code


Generated by Claude Code

@paulofph paulofph closed this Aug 5, 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.

2 participants