Skip to content

fix: do not include _Service and _entities in the federated graph#19

Merged
Aenimus merged 1 commit intomainfrom
suvij/eng-3852-do-not-include-_service-in-the-federated-graph
Aug 29, 2023
Merged

fix: do not include _Service and _entities in the federated graph#19
Aenimus merged 1 commit intomainfrom
suvij/eng-3852-do-not-include-_service-in-the-federated-graph

Conversation

@JivusAyrus
Copy link
Copy Markdown
Member

@JivusAyrus JivusAyrus requested a review from Aenimus August 29, 2023 10:33
@linear
Copy link
Copy Markdown

linear Bot commented Aug 29, 2023

@Aenimus Aenimus merged commit 97201ed into main Aug 29, 2023
@ysmolski ysmolski deleted the suvij/eng-3852-do-not-include-_service-in-the-federated-graph branch September 1, 2025 14:24
jensneuse added a commit that referenced this pull request Apr 24, 2026
…view #22)

Add a short up-front section at the top of docs/REQUEST_SCOPED.md that
distinguishes the two per-request L1 caches: coordinate L1 (this
document, keyed by {subgraph}.{@requestScoped key}) vs entity L1
(keyed by {subgraph}.{type}.{@key values}). Reader no longer has to
read the full doc to tell them apart.

#19 was investigated and is effectively refuted — composition/AGENTS.md
has no `[N]` or `][` reference-link syntax for markdownlint MD052 to
fire on, so the bot's missing-reference claim does not match the file.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants