Repository navigation
chore: remove the leftover ERP Integration API (erp-integration) files - #566
Merged
Merged
Conversation
The ERP Integration API was renamed to the Integration Toolkit API long ago and is served as `epilot integration-toolkit` and `@epilot/sdk/integration-toolkit`. The `erp-integration` files were left behind: the CLI no longer registered the command (`epilot erp-integration` falls through to the usage text) and the SDK registry no longer loads the definitions, but the frozen 0.58.0 spec, its types and docs were still shipped and picked up by docs tooling, advertising operations the current spec marks deprecated without saying so. Every one of its 43 operations exists under the same operationId in the Integration Toolkit API. Also drops the stale README rows and the `erp-integration-client` entry from publish-clients.sh (that client directory no longer exists).
karlrank
added a commit
that referenced
this pull request
Oct 23, 2026
…files chore: delete the leftover erp-integration files (completes #566)
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.
Removes the leftover ERP Integration API (
erp-integration) files. The API was renamed to the Integration Toolkit API and lives on asepilot integration-toolkit/@epilot/sdk/integration-toolkit.Why it's dead code
erp-integrationis not registered inpackages/cli/src/index.ts. On@epilot/cli@0.1.224,epilot erp-integration --helpfalls through to the top-level usage text and the command isn't listed. The CLI generator builds commands fromclients/, and there is noerp-integration-clientthere any more._registry.ts/client-map.tsdon't reference it, and there is no./erp-integrationexport.@epilot/clidefinitions/erp-integration.json) and picked up by docs tooling. They advertise operations the current spec marks deprecated (the four v1 monitoring query endpoints,processErpUpdatesEvents,processErpUpdatesEventsV2) without the flag.operationIdin the Integration Toolkit API.Changes
packages/cli/{definitions,docs}/erp-integration.*,packages/cli/src/commands/apis/erp-integration.tspackages/epilot-sdk-v2/{docs,src/definitions,src/docs,src/types}/erp-integration*erp-integrationrows frompackages/cli-wrapper/README.mdand theerp-integration-cliententry frompublish-clients.sh@epilot/clipatch,@epilot/sdkpatchVerification
packages/cli:vitest160/160,packages/epilot-sdk-v2:vitest362/362, SDKtsupbuild OK,biome check .cleantsc --noEmit: CLI 12 / SDK 2 errors, identical onmain(existing, unrelated:profiles.ts, definition typing)Follow-up outside this repo: how-to-everything still has
13-cli/commands/erp-integration.mdand references in13-cli/cli.md.🤖 Generated with Claude Code