Skip to content

doc: adaption - #88

Open
Schmarvinius wants to merge 4 commits into
mainfrom
doc/adaption-to-new-strcture
Open

doc: adaption#88
Schmarvinius wants to merge 4 commits into
mainfrom
doc/adaption-to-new-strcture

Conversation

@Schmarvinius

@Schmarvinius Schmarvinius commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Documentation Improvements and Clarifications

Documentation

📝 Improved and corrected documentation across multiple README files, clarifying prerequisites, AI Core binding setup, the AICore CDS service internals, the programmatic API, and integration test profiles.

Changes

  • README.md: Added Maven 3.6.3+ as a prerequisite, removed the outdated Node.js global install requirement, clarified the hermetic build note, updated the integration-test command from mvn test -pl integration-tests/spring -am to mvn verify, and added a reference to the integration-tests README.

  • cds-feature-ai-core/README.md: Clarified that the AICore CDS service is internal (@protocol: 'none') and not exposed via OData. Updated the AI Core binding section to replace bullet-list binding methods with a focused CAP CLI example. Expanded the entity table to reflect UPDATE operations and the stop bound action. Replaced the old AICoreService programmatic usage section with a fully working step-by-step RemoteService-based code snippet. Added a new Public API table documenting ResourceGroupContext, DeploymentIdContext, InferenceClientContext, and ModelDeploymentSpec. Removed the outdated ## Programmatic Usage section that referenced the removed AICoreService facade.

  • cds-feature-recommendations/README.md: Replaced the single model-deduplication note with a two-option guide (Option A: Java-internal only; Option B: OData exposure). Clarified the @cds.odata.valuelist annotation derivation. Fixed a configuration YAML key from requires: to ai:. Added supported CDS HANA types to the field type table. Corrected the mock client name from MockAIClient to MockRecommendationClient. Minor whitespace/trailing-space fixes.

  • integration-tests/README.md: Updated the "skip integration tests" command from the non-existent skip-integration-tests profile to the correct -P-with-integration-tests deactivation syntax. Updated the profiles table accordingly. Lowered cds-feature-recommendations coverage thresholds to 0% with an explanatory note about tightening them once APIs stabilise.

  • 🔄 Regenerate and Update Summary
PR Bot Information

Version: 1.26.0

  • Event Trigger: pull_request.opened
  • Output Template: Default Template
  • Correlation ID: b52a3f81-a40d-4c47-84c7-8ebaa3f13837
  • Summary Prompt: Default Prompt
  • LLM: anthropic--claude-4.6-sonnet
  • File Content Strategy: Full file content

@Schmarvinius
Schmarvinius requested a review from a team as a code owner June 19, 2026 06:59

@hyperspace-pr-bot hyperspace-pr-bot 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.

The PR contains one concrete correctness issue worth addressing: the cds-feature-ai-core/README.md programmatic API example imports RptModelSpec from cds-feature-recommendations, a separate module not bundled with cds-feature-ai-core alone — readers following the example without that dependency on the classpath will get a compile error. The other changes are clear documentation improvements with no other substantive issues.

PR Bot Information

Version: 1.26.0

  • Event Trigger: pull_request.opened
  • LLM: anthropic--claude-4.6-sonnet
  • File Content Strategy: Full file content
  • Correlation ID: b52a3f81-a40d-4c47-84c7-8ebaa3f13837

Comment thread cds-feature-ai-core/README.md Outdated
Comment thread integration-tests/README.md
@Schmarvinius Schmarvinius linked an issue Jul 6, 2026 that may be closed by this pull request
14 tasks
@lisajulia
lisajulia force-pushed the doc/adaption-to-new-strcture branch from 4485dd6 to 66f118a Compare August 18, 2026 19:19
@lisajulia
lisajulia force-pushed the doc/adaption-to-new-strcture branch from 66f118a to 84da043 Compare August 18, 2026 19:26
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.

docs/test: model annotations, opt-in enhancements, READMEs

1 participant