Skip to content

Create SKILL.MD for documentation review process#8087

Open
danielmarbach wants to merge 8 commits intomasterfrom
docs-review
Open

Create SKILL.MD for documentation review process#8087
danielmarbach wants to merge 8 commits intomasterfrom
docs-review

Conversation

@danielmarbach
Copy link
Contributor

Add guidelines for reviewing Particular Platform documentation, including language standards, style consistency, and technical accuracy.

Add guidelines for reviewing Particular Platform documentation, including language standards, style consistency, and technical accuracy.
Added instructions for local validation of documentation changes using Particular.DocsTool.
@danielmarbach
Copy link
Contributor Author

@dvdstelt I think you have also experimented with skills a lot. Anything missing?

Updated installation instructions for Particular.DocsTool and added troubleshooting tips.
Also add a section about naming consistency
@danielmarbach danielmarbach marked this pull request as ready for review February 23, 2026 10:04
@danielmarbach
Copy link
Contributor Author

@dvdstelt done

@danielmarbach
Copy link
Contributor Author

I would be possible to integrate this with docs flows like this one https://github.com/githubnext/agentics/blob/main/workflows/update-docs.md

Added instructions for updating the reviewed date in documentation.
@mauroservienti
Copy link
Member

I would be possible to integrate this with docs flows like this one https://github.com/githubnext/agentics/blob/main/workflows/update-docs.md

That would be awesome

danielmarbach and others added 2 commits February 23, 2026 20:07
Co-authored-by: David Boike <david.boike@gmail.com>
Added guidelines for code sample requirements, emphasizing the use of snippet projects and proper referencing.
- Outdated references to deprecated features

---

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing that concerns me is that we could end up with the agent making change for change's sake and introducing a lot of noise into the repository. Changing language that isn't quite right in its view, but that a person would think was absolutely fine, for example. This problem could compound over time as the LLM model is updated.

I haven't used agent skills before (so this could be incorrect), but I posted my comment above into Claude, and this is what it suggested:

Suggested change
## Conservative Change Philosophy
**The cost of an unnecessary change is higher than the cost of a missed improvement.**
Documentation lives in a version-controlled repository. Every change creates noise: diff churn, review burden, and the risk of introducing new errors while fixing perceived ones. This review process is optimized to catch real problems, not to impose stylistic preferences.
### The Core Rule
Only suggest or apply a change if it can be justified by a specific, named rule in this skill file. If a change cannot be pinned to a written rule, do not make it. Style preferences, gut feelings, and "this could be clearer" instincts that aren't grounded in a documented standard are not sufficient justification.
### When in Doubt, Leave It Alone
If existing text is technically correct and a reasonable professional reader would find it acceptable, do not flag it. The question is not "could this be better?" but "is this wrong by a defined standard?"
### Consistency Beats Correctness
If a phrasing, capitalization style, or structural pattern is used consistently throughout the existing documentation, do not flag it — even if you would have written it differently. Consistency within the corpus is more valuable than conforming to an external preference. Before flagging anything stylistic, check whether the same pattern appears elsewhere in the docs. If it does, leave it alone.
---

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The addition makes sense. It's worth noting that Copilot will either raise suggestions during a review or open a PR; it doesn't push directly. Still, some noise remains.

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.

5 participants