Skip to content

feat: cache hits in expr context docs#265

Closed
SkArchon wants to merge 1 commit intomainfrom
milinda/eng-9179-router-add-cache-metric-hits-to-the-expression-context
Closed

feat: cache hits in expr context docs#265
SkArchon wants to merge 1 commit intomainfrom
milinda/eng-9179-router-add-cache-metric-hits-to-the-expression-context

Conversation

@SkArchon
Copy link
Copy Markdown
Contributor

@SkArchon SkArchon commented Mar 12, 2026

Description

This PR adds docs for the new attributes in the expr context for cache hit statuses. Depends on wundergraph/cosmo#2643

Summary by CodeRabbit

  • Documentation
    • Added five new cache hit indicator fields to Operation object in template expressions: normalizationCacheHit, variablesNormalizationCacheHit, variablesRemappingCacheHit, persistedOperationCacheHit, and planCacheHit.
    • Updated examples demonstrating cache hit field usage.

Checklist

  • I have validated my changes locally using mintlify dev
  • I have validated my changes are working as expected by looking at the preview deployment
  • All links in my documentation work correctly
  • All images render properly
  • Code examples are tested and formatted correctly
  • If I added new pages, they are included in the navigation (docs.json)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 12, 2026

Walkthrough

Documentation update adding five new boolean cache-related fields to the Operation object in template expressions: normalizationCacheHit, variablesNormalizationCacheHit, variablesRemappingCacheHit, persistedOperationCacheHit, and planCacheHit. Example expressions updated to demonstrate planCacheHit usage.

Changes

Cohort / File(s) Summary
Documentation Updates
docs/router/configuration/template-expressions.mdx
Adds five new boolean fields (normalizationCacheHit, variablesNormalizationCacheHit, variablesRemappingCacheHit, persistedOperationCacheHit, planCacheHit) to Operation object reference documentation and updates example expressions to demonstrate planCacheHit usage.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding documentation for cache hit fields in the expression context for the router configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

@SkArchon SkArchon marked this pull request as ready for review March 12, 2026 17:43
@SkArchon SkArchon requested review from a team as code owners March 12, 2026 17:43
@SkArchon SkArchon requested review from StarpTech, devsergiy, thisisnithin and wilsonrivera and removed request for a team March 12, 2026 17:43
@SkArchon SkArchon closed this Mar 15, 2026
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