Add AGENTS.md plumbing and correct CentOS support in the doc - #170
Merged
Conversation
Adopt the AGENTS.md standard's supporting plumbing (the module already ships an AGENTS.md). No manifest/behavior changes. - .gitignore: replace the lone /CLAUDE.md line with the canonical AI coding-assistant block (Claude Code, Copilot, Cursor, Windsurf, Gemini, Cline). AGENTS.md is the committed standard; each assistant's own config is ignored. - .pdkignore: exclude /AGENTS.md from built module packages. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jul 7, 2026
metadata.json supports CentOS 9/10 only (not EL8) and declares the OpenVox runtime; the intro line overclaimed CentOS EL8. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hcaballero2
changed the title
There was a problem hiding this comment.
Pull request overview
Updates AI-assistant support plumbing and corrects the repository’s documented platform/runtime support without changing Puppet manifests or module runtime behavior.
Changes:
- Update
AGENTS.mdto matchmetadata.json(CentOS 9/10 support and OpenVox 8 runtime requirement). - Expand
.gitignorewith a canonical block of AI-assistant generated/local configuration ignores (keepingAGENTS.mdas the committed standard). - Exclude
AGENTS.mdfrompdk buildartifacts via.pdkignore.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
AGENTS.md |
Corrects supported OS/runtime wording to match metadata.json. |
.pdkignore |
Excludes AGENTS.md from built module packages. |
.gitignore |
Adds a consolidated ignore block for AI-assistant local/generated config files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
michael-riddle
approved these changes
Jul 14, 2026
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.
Two AGENTS.md-related changes — no manifest or behavior changes:
AI-assistant plumbing — the module already ships an
AGENTS.md; this adds the supporting ignore rules./CLAUDE.mdline with the canonical AI coding-assistant block (Claude Code, Copilot, Cursor, Windsurf, Gemini, Cline).AGENTS.mdis the committed standard;CLAUDE.mdis a local, uncommitted symlink to it./AGENTS.mdfrom built module packages.AGENTS.md accuracy fix — the intro line overclaimed CentOS EL8;
metadata.jsonsupports CentOS 9/10 only, and declares the OpenVox runtime.Split out of the original combined PR #168; test hardening lives in #171 (rspec) and #172 (acceptance).
🤖 Generated with Claude Code