Revert Document AI Plugin - #125
Open
samyuktaprabhu wants to merge 2 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
This PR is a clean and comprehensive removal of the cds-feature-sap-document-ai module. All references have been consistently removed across build files (pom.xml, package.json), documentation (README.md), integration tests, CI action configuration, the bookshop sample, and coverage reporting. The import order fix in BaseIntegrationTest.java is a minor unrelated cleanup included in the PR. Overall the removal appears complete and well-coordinated across the repository.
PR Bot Information
Version: 1.29.14
- Experiment Variant:
control - LLM:
anthropic--claude-4.6-sonnet - Event Trigger:
pull_request.opened - Correlation ID:
dc551c70-8ce9-11f1-9abe-de81ed32c6a3 - File Content Strategy: Full file content
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.
Revert Document AI Plugin
♻️ Revert: Removes the
cds-feature-sap-document-aiplugin and all related artifacts from the repository, including its source code, tests, sample integrations, and documentation.Changes
cds-feature-sap-document-ai/(entire module): Removed — all Java source files, CDS models, configuration, test classes, and documentation for the Document AI plugin have been deleted.pom.xml: Removedcds-feature-sap-document-aifrom the root module list and dependency management section.cds-starter-ai/pom.xml: Removed the placeholder comment referencingcds-feature-sap-document-aiand its planned inclusion pending multi-tenancy support.cds-starter-ai/README.md: Cleaned up references tocds-feature-sap-document-ai, including the prerequisite for the DIE service binding and the related link.coverage-report/pom.xml: Removedcds-feature-sap-document-aifrom coverage aggregation (class copying and JaCoCo exec file merging)..github/actions/integration-tests/action.yml: Removedcds-feature-sap-document-aifrom the Maven build step for integration test dependencies.integration-tests/spring/pom.xml: Removedcds-feature-sap-document-aidependency from the integration test module.integration-tests/spring/(test classes): Removed all Document AI integration tests (DocumentSubmissionTest,ExtractionLifecycleTest,ExtractionErrorTest,EventEmissionTest,PluginLoadTest, etc.).integration-tests/spring/test-service.cds: Removed thesap-document-aiCDS import.samples/bookshop/: Removed the supplier invoices Fiori app, UI config, CDS service definitions, Java handlers (SupplierInvoiceHandler,DocumentExtractionResultHandler), schema entities (Suppliers,SupplierInvoices), dummy invoice PDF, and related configuration inapplication.yamlandpackage.json.README.md&integration-tests/README.md: Updated descriptions and coverage data source lists to remove all Document AI references.PR Bot Information
Version:
1.29.14anthropic--claude-4.6-sonnetpull_request.openeddc551c70-8ce9-11f1-9abe-de81ed32c6a3