Skip to content

docs: clarify validation cache scope - #674

Merged
jonathan-vella merged 1 commit into
mainfrom
docs/clarify-validation-cache-scope
Aug 27, 2026
Merged

docs: clarify validation cache scope#674
jonathan-vella merged 1 commit into
mainfrom
docs/clarify-validation-cache-scope

Conversation

@jonathan-vella

Copy link
Copy Markdown
Owner

Description

Clarifies that caching applies to the Node validator suite, while external validators also run. Addresses the post-merge Copilot feedback on #673.

Type of Change

  • Documentation update

Token / latency impact (Plan 01 Phase 5)

  • NO

Workflow Used

  • Direct implementation (simple change)

Testing Performed

  • npm run lint:md
  • Markdown link check
  • git diff --check
  • Confirmed the edited row is 83 characters

Pre-Submission Checklist

Copilot AI lite review requested due to automatic review settings August 27, 2026 16:09
@jonathan-vella
jonathan-vella enabled auto-merge (squash) August 27, 2026 16:09

Copilot AI 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.

🟢 Approval recommended

The documentation change is accurate, scoped, and consistent with the underlying validate:all composition.

Pull request overview

This PR updates the validation command reference documentation to better describe the scope of caching in the validate:all aggregate command, aligning the wording with how the Node validation runner shares caches while external validators still run.

Changes:

  • Clarify that validate:all runs cache-aware Node validators in addition to external validators.
File summaries
File Description
site/src/content/docs/reference/validation-reference.md Updates the validate:all table description to clarify cache scope vs external validators.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@jonathan-vella
jonathan-vella merged commit 8369663 into main Aug 27, 2026
11 checks passed
@jonathan-vella
jonathan-vella deleted the docs/clarify-validation-cache-scope branch August 27, 2026 16:12
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.

2 participants