Skip to content

chore: Exclude autoexposed entities - #103

Open
danjoa wants to merge 28 commits into
mainfrom
exclude-autoexposed-entities
Open

chore: Exclude autoexposed entities#103
danjoa wants to merge 28 commits into
mainfrom
exclude-autoexposed-entities

Conversation

@danjoa

@danjoa danjoa commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Twin PR to cap-js/mcp#80

P.S. We should move that redundant implementation to a common reuse function later.

@danjoa
danjoa requested review from a team as code owners September 10, 2026 14:36
@hyperspace-pr-bot

Copy link
Copy Markdown

Summary

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


🔧 Refine Entity Filtering Logic in getFilteredEntities

Updates the getFilteredEntities utility function to improve how auto-exposed entities are excluded.

What changed:

  • Simplified the auto-exposure check: previously allowed entities with both @cds.autoexposed and @cds.autoexpose set — now excludes all entities with @cds.autoexposed
  • Added filtering for .drafts suffixed entities
  • Minor style consistency update (single quotes)
  • Reordered filter conditions for clarity

This aligns the filtering behavior with a twin implementation in the cap-js/mcp#80 PR.

Have you...

  • Added relevant entry to the change log?

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

Version: 1.31.20

  • Summary Prompt: Default Prompt
  • Correlation ID: f7808920-ad24-11f1-8294-44bf00ed16fd
  • Output Template: Repository PR Template
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened
  • File Content Strategy: Full file content

@danjoa
danjoa enabled auto-merge (squash) September 10, 2026 14:36
swaldmann
swaldmann previously approved these changes Sep 10, 2026
swaldmann
swaldmann previously approved these changes Sep 11, 2026
swaldmann
swaldmann previously approved these changes Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants