Skip to content

Add architecture files for cds-feature-ai-core, cds-feature-recommendations#111

Open
lisajulia wants to merge 1 commit into
mainfrom
add-architecture-files
Open

Add architecture files for cds-feature-ai-core, cds-feature-recommendations#111
lisajulia wants to merge 1 commit into
mainfrom
add-architecture-files

Conversation

@lisajulia

@lisajulia lisajulia commented Jul 6, 2026

Copy link
Copy Markdown
Contributor
  • add cds-feature-ai-core/architecture
  • add cds-feature-recommendations/architecture

@lisajulia lisajulia requested a review from a team as a code owner July 6, 2026 11:40
@hyperspace-insights

Copy link
Copy Markdown
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Add Architecture Documents and Improve READMEs

Documentation

📚 Added detailed architecture documentation for cds-feature-ai-core and cds-feature-recommendations, introduced a new cds-starter-ai README, and updated several existing READMEs with clarifications and additional context.

Changes

  • README.md: Updated project description to include SAP Document AI; clarified starter module scope; improved prerequisites section with per-plugin service binding info; updated fallback behavior wording; fixed integration-tests link; added a new GitHub Actions section documenting CI checks, static analysis tools, JaCoCo aggregation, and the release process.

  • cds-feature-ai-core/docs/architecture.md (new): Comprehensive architecture document covering purpose, dependencies (SAP AI SDK, Caffeine, Resilience4j, CAP MTX), CDS model, public API, key infrastructure classes, multi-tenancy, key flows (tenant subscribe/unsubscribe, inference client resolution with Mermaid diagram), tests, and quality tools.

  • cds-feature-ai-core/README.md: Expanded the sample section with a more specific description of the AdminService.Books demo scenario.

  • cds-feature-recommendations/docs/architecture.md (new): Architecture document covering purpose, dependencies (cds-feature-ai-core, Node.js CDS plugin, ExtensibilityService), key infrastructure classes, recommendation pipeline (Mermaid flowchart), no-prediction-cache invalidation flow, tests, and quality tools.

  • cds-feature-recommendations/README.md: Renamed "How It Works" to "Features"; added opt-out annotation example (@UI.RecommendationState: 0).

  • cds-starter-ai/README.md (new): New README documenting the cds-starter-ai convenience bundle — what it includes, Maven/npm setup, prerequisites, and note on cds-feature-sap-document-ai exclusion pending MT support.

  • cds-feature-sap-document-ai/README.md: Added Multi-Tenancy section to the table of contents and body; added a new Known Limitations section summarizing MT, annotation-based triggering, and OData API gaps.

  • integration-tests/README.md: Updated JaCoCo threshold description to clarify that SonarQube (not JaCoCo check goals) enforces coverage; replaced the per-module threshold table accordingly; added cds-feature-sap-document-ai to the coverage data sources list.


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.26.14

  • Output Template: Default Template
  • Correlation ID: 7f159260-792f-11f1-90a0-17a3f9ca90e6
  • Event Trigger: pull_request.opened
  • File Content Strategy: Full file content
  • LLM: anthropic--claude-4.6-sonnet
  • Summary Prompt: Default Prompt

@hyperspace-insights hyperspace-insights Bot 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.

Two issues found: a broken relative link in cds-feature-recommendations/docs/architecture.md that points to a non-existent path, and a double-article typo ("an the") in cds-starter-ai/README.md. Both are straightforward to fix with the suggested corrections.

PR Bot Information

Version: 1.26.14

  • LLM: anthropic--claude-4.6-sonnet
  • File Content Strategy: Full file content
  • Event Trigger: pull_request.opened
  • Correlation ID: 7f159260-792f-11f1-90a0-17a3f9ca90e6

Comment thread cds-feature-recommendations/docs/architecture.md Outdated
Comment thread cds-starter-ai/README.md Outdated
@lisajulia lisajulia changed the title Add architecture files Add architecture files for cds-feature-ai-core, cds-featur-recommendations, add readme for cds-starter-ai, minor changes to other readme files Jul 6, 2026
@lisajulia lisajulia force-pushed the add-architecture-files branch from dcdb08e to 138c964 Compare July 6, 2026 11:43
@lisajulia lisajulia force-pushed the add-architecture-files branch from 138c964 to a355097 Compare July 6, 2026 11:47
@lisajulia lisajulia changed the title Add architecture files for cds-feature-ai-core, cds-featur-recommendations, add readme for cds-starter-ai, minor changes to other readme files Add architecture files for cds-feature-ai-core, cds-featur-recommendations Jul 6, 2026
@lisajulia lisajulia removed the request for review from samyuktaprabhu July 6, 2026 11:48
@lisajulia lisajulia changed the base branch from chore/import-sap-document-ai to main July 6, 2026 11:50
@lisajulia lisajulia changed the title Add architecture files for cds-feature-ai-core, cds-featur-recommendations Add architecture files for cds-feature-ai-core, cds-feature-recommendations Jul 6, 2026
@lisajulia lisajulia requested a review from agoerler July 6, 2026 12:13
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.

1 participant