Align YAML header convention across templates, editor-manual, and DOCS/#41
Align YAML header convention across templates, editor-manual, and DOCS/#41MatMatt wants to merge 3 commits into
Conversation
Resolve conflicts between three sources that defined different YAML header formats. Standardize on the convention actually used by all 25 DOCS/ files on main: no double quotes, category first, toc group present, version in YAML (not forbidden). - editor-manual: update YAML example, move version from forbidden to required, add category/toc fields, remove product-name - PUM/ATBD templates: strip double quotes, add missing fields, remove product-name, clean up placeholder text
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f8289bc11b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| toc: true | ||
| toc-depth: 3 | ||
| toc-title: Content | ||
| version: 1.0 |
There was a problem hiding this comment.
Don't require fields the build strips or overwrites
In the publish path I checked deploy-docs.yml's Build Docs step: build-docs.sh runs strip_unknown_frontmatter.py, which drops toc, toc-depth, and toc-title, and then fill_version.py, whose header says the source version: is overwritten from .llm_cache/versions.json; validate_qmd_files.py also explicitly does not check these fields. Listing them as required here will send editors to maintain metadata that production ignores or replaces, so source changes like toc-title: Content or manual version edits won't affect the deployed document.
Useful? React with 👍 / 👎.
Problem
The YAML header convention was inconsistent across three sources:
_meta/templates/) used double quotes,product-name, aformat:block, placeholderversiontextversionin YAML, requiredproduct-name, omittedcategory/toc/toc-depth/toc-titlecategoryfirst,tocgroup,versionpresentChanges
editor-manual_v1.qmdversionmoved from forbidden to requiredcategory,toc,toc-depth,toc-titledocumentedproduct-nameremovedTemplates (PUM + ATBD)
category/toc/version, removedproduct-nameSmall_Landscape_Features_2021_PUM_v1.qmdDD/MM/YYYYto ISO format to pass frontmatter validationNo other DOCS/ content files touched.