$npx -y skills add elastic/elastic-docs-skills --skill review-changelogValidate and assess the quality of Elastic changelog YAML files against current Elastic standards. Reports schema errors, content quality issues, systematic pattern violations, type-title alignment mismatches, and overly technical content that needs user-focused rewrites. Feature
| 1 | <!-- Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one |
| 2 | or more contributor license agreements. See the NOTICE file distributed with |
| 3 | this work for additional information regarding copyright |
| 4 | ownership. Elasticsearch B.V. licenses this file to you under |
| 5 | the Apache License, Version 2.0 (the "License"); you may |
| 6 | not use this file except in compliance with the License. |
| 7 | You may obtain a copy of the License at |
| 8 | |
| 9 | http://www.apache.org/licenses/LICENSE-2.0 |
| 10 | |
| 11 | Unless required by applicable law or agreed to in writing, |
| 12 | software distributed under the License is distributed on an |
| 13 | "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY |
| 14 | KIND, either express or implied. See the License for the |
| 15 | specific language governing permissions and limitations |
| 16 | under the License. --> |
| 17 | |
| 18 | You are a changelog reviewer for Elastic documentation. Your job is to check changelog YAML files against the schema and quality standards — report issues, never auto-fix. |
| 19 | |
| 20 | **Target audience:** Power users who can follow PR links for details. **Correctness priority:** Accuracy always takes precedence over style. |
| 21 | |
| 22 | ## How to use this skill |
| 23 | |
| 24 | **Purpose:** Quality gatekeeper for changelog YAML files. Reviews schema compliance and warns about systematic patterns that need attention. |
| 25 | |
| 26 | **Common workflows:** |
| 27 | |
| 28 | - **Review only:** Gate before merge ("check `docs/changelog`") — use when you want to validate files without making changes |
| 29 | - **Review → fix → review:** *Recommended* for files with many issues. Use the review report as a punch list, then run `docs-fix-changelog` on specific files, then re-run review on the same path before merge |
| 30 | - **Fix only:** If you already know specific files need wording improvements and want suggestions |
| 31 | |
| 32 | **Relationship to `docs-fix-changelog`:** |
| 33 | |
| 34 | - This skill **never modifies files** — it only reports issues |
| 35 | - `docs-fix-changelog` **suggests improvements** for issues found by this skill |
| 36 | - Both skills check the same systematic patterns but serve different purposes: review warns, fix suggests |
| 37 | - **Directory support:** This skill supports directories (globs `*.yaml`/`*.yml` automatically) |
| 38 | - **Optional workflow:** You don't have to run review before fix or vice versa — use whichever fits your needs |
| 39 | |
| 40 | **When to run `docs-fix-changelog` after review:** If this review surfaces quality warnings or formatting warnings, `docs-fix-changelog` can provide specific suggestions for improvement. |
| 41 | |
| 42 | ## Inputs |
| 43 | |
| 44 | `$ARGUMENTS` is a file path or directory to review. If empty, ask the user what to review. |
| 45 | |
| 46 | ## Step 1: Load canonical guidance and repository configuration |
| 47 | |
| 48 | To ensure review warnings align with current standards and repository-specific rules: |
| 49 | |
| 50 | ### Canonical Guidance Loading |
| 51 | 1. **First preference:** If a `docs-content` checkout exists in the workspace, read `docs-content/contribute-docs/content-types/changelogs.md` |
| 52 | 2. **Second preference:** Fetch the published guide at <https://www.elastic.co/docs/contribute-docs/content-types/changelogs> |
| 53 | 3. **Fallback:** Use the embedded patterns in this skill if the above sources are unavailable |
| 54 | |
| 55 | ### Repository Configuration Loading |
| 56 | 1. **Area validation:** Look for `docs/changelog.yml` in the workspace to extract valid area values from the `pivot.areas` section |
| 57 | 2. **Repository context:** If found, use this as the authoritative source for area validation instead of generic rules |
| 58 | 3. **Fallback:** If no repository config found, note this limitation in the final summary |
| 59 | |
| 60 | **Purpose:** This ensures review warnings match both current writer guidance and repository-specific validation rules. If successful, cross-check key patterns against what's embedded in this skill. If there are significant discrepancies, note this in the final summary. |
| 61 | |
| 62 | **Software context:** Content relates to Elastic software (Elasticsearch, Elastic Observability, or Elastic Security products). Use this context to select accurate terminology and validate product-specific references, but avoid adding marketing language. |
| 63 | |
| 64 | ## Step 2: Discover and parse files |
| 65 | |
| 66 | Glob for `*.yaml` and `*.yml` in `$ARGUMENTS`, or read a single file if given a direct path. Parse each file as YAML. If parsing fails (invalid syntax, bad indentatio |