fix: add content links and InfluxDB documentation MCP server instructions#6885
Merged
jstirnaman merged 15 commits intomasterfrom Feb 28, 2026
Merged
fix: add content links and InfluxDB documentation MCP server instructions#6885jstirnaman merged 15 commits intomasterfrom
jstirnaman merged 15 commits intomasterfrom
Conversation
Vale linting uses Docker (jdkato/vale:latest) via .ci/vale/vale.sh, not the npm package. Update VS Code setup docs to use system Vale.
Replace local InfluxData MCP server references with the hosted kapa.ai documentation MCP server (https://influxdb-docs.mcp.kapa.ai). Changes: - Update MCP configuration from local stdio server to hosted URL - Replace kapa_query() examples with natural language prompts - Add rate limit info (40/hour, 200/day per Google OAuth user) - Standardize Hugo commands to use yarn (yarn hugo --quiet) - Fix E2E test command (node cypress/support/run-e2e-specs.js) - Use .ci/vale/vale.sh consistently for Vale linting - Remove VS Code-specific instructions for agent focus - Move TODOs to GitHub issue #6853
Contributor
|
jstirnaman
commented
Feb 27, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Adds and updates documentation and tooling guidance for MCP server usage across the docs repo, alongside dependency maintenance to support the updated workflow.
Changes:
- Upgraded various JS dependencies in
yarn.lockand removed@vvago/valefrompackage.json. - Updated shared MCP server documentation links for Cloud Dedicated/Clustered environment variables.
- Updated testing and contributor workflow docs (Vale IDE setup, Hugo via Yarn, hosted documentation MCP server guidance) and expanded allowed CLI tooling in Claude settings.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
yarn.lock |
Dependency upgrades/cleanup; includes a tar resolution mismatch that should be addressed. |
package.json |
Removes @vvago/vale from devDependencies. |
content/shared/influxdb3-admin/mcp-server.md |
Updates environment-variable docs to link to /influxdb3/version/admin/... resources. |
DOCS-TESTING.md |
Updates Vale editor integration guidance; still contains an outdated Vale run command. |
.claude/skills/content-editing/SKILL.md |
Refines content workflow guidance (hosted docs MCP server, yarn hugo usage, Vale wrapper), but contains a path typo and inconsistent/outdated Vale run instructions. |
.claude/settings.json |
Allows additional CLI tools (htmlq, yq) for Claude workflows. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
The vale Compose service was removed in 922e481 but docs still referenced `docker compose run -T vale`, which fails. Vale linting now works as follows: - If vale is installed locally (v3+), .ci/vale/vale.sh uses it directly. Contributors can install via `brew install vale` for faster linting without Docker. - Otherwise, the script falls back to Docker with a pinned image tag (jdkato/vale:v3.13.1) instead of :latest. - If the local version is incompatible (pre-v3), the script warns and falls back to Docker. Docker is no longer required for pre-commit hooks that only run Vale. Also adds a weekly GitHub Actions workflow (check-pinned-deps.yml) that compares pinned dependency versions against upstream releases and opens a PR when updates are available.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
|
@jstirnaman I've opened a new pull request, #6887, to work on those changes. Once the pull request is ready, I'll request review from you. |
Contributor
|
@jstirnaman I've opened a new pull request, #6888, to work on those changes. Once the pull request is ready, I'll request review from you. |
Contributor
|
@jstirnaman I've opened a new pull request, #6889, to work on those changes. Once the pull request is ready, I'll request review from you. |
- vale.sh: make version-detection grep non-fatal so Docker fallback works - check-pinned-deps: add set -euo pipefail, auth header, and null guards - DOCS-CONTRIBUTING: fix broken links to DOCS-TESTING.md - DOCS-TESTING: rename VS Code section heading per review suggestion
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docker compose run -T valereferences (novaleservice incompose.yaml).ci/vale/vale.shto prefer local vale binary, fall back to pinned Docker imageImportant
New contributor requirement: install Vale locally.
Vale style linting no longer requires Docker. Install Vale with
brew install vale(or see vale.sh/docs/install).
The
.ci/vale/vale.shwrapper uses a localvalebinary (v3+) if available,otherwise falls back to Docker with a pinned image tag.