Skip to content

docs(overlay): document the new: intent directive#914

Open
SoundMatt wants to merge 1 commit into
COVESA:masterfrom
SoundMatt:docs/overlay-new-directive
Open

docs(overlay): document the new: intent directive#914
SoundMatt wants to merge 1 commit into
COVESA:masterfrom
SoundMatt:docs/overlay-new-directive

Conversation

@SoundMatt

Copy link
Copy Markdown
Contributor

Summary

Companion documentation for COVESA/vss-tools#527, which introduces the opt-in new: attribute for overlay files.

  • new: false — asserts the node already exists in the base spec (modify intent); errors on typos and upstream removals
  • new: true — asserts the node does not exist in the base spec (create intent); errors on future naming collisions

New section added between "Node content in Overlays" and "Deleting nodes" in docs-gen/content/extensions/overlay.md.

Test plan

  • Rendered locally with Hugo or reviewed as raw Markdown
  • Links checked — no internal links added, examples are self-contained YAML snippets

@erikbosch

Copy link
Copy Markdown
Collaborator

@SoundMatt - minor conflict after I merged the other PR, could you resolve

@SoundMatt SoundMatt force-pushed the docs/overlay-new-directive branch from 7ef393e to 85153f2 Compare June 15, 2026 13:45
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com>
@SoundMatt SoundMatt force-pushed the docs/overlay-new-directive branch from 85153f2 to f7da336 Compare June 16, 2026 15:10
@SoundMatt

Copy link
Copy Markdown
Contributor Author

Updated to match the naming decision from the maintainer meeting: the attribute has been renamed from new: true/false to overlay: add/edit.

  • overlay: edit (was new: false) — asserts the node exists; errors if absent
  • overlay: add (was new: true) — asserts the node is new; errors if already present

The companion vss-tools#527 PR has been updated with the same rename.

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