Skip to content

Add doc-edit skill - #150

Open
loganlipscomb wants to merge 4 commits into
mainfrom
add-doc-edit-skill
Open

Add doc-edit skill#150
loganlipscomb wants to merge 4 commits into
mainfrom
add-doc-edit-skill

Conversation

@loganlipscomb

Copy link
Copy Markdown

Adds the doc-edit skill (NetFoundry writing style guide checklist) so it can be saved and shared across docs repos and team.

Adds the doc-edit skill (NetFoundry writing style guide checklist) so it
can be shared across docs repos.
@docpreview-dovholuknf-test

docpreview-dovholuknf-test Bot commented Aug 13, 2026

Copy link
Copy Markdown

Documentation preview

Project Status Preview Commit Updated (UTC)
netfoundry/docusaurus-shared Failed · 1s 6a6e06c Aug 13, 2026 11:04pm

@vercel

vercel Bot commented Aug 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
nf-theme-sandbox Ignored Ignored Preview Aug 13, 2026 8:13pm
unified-doc-preview Ignored Ignored Preview Aug 13, 2026 8:13pm

Request Review

Comment thread skills/doc-edit/SKILL.md Outdated
Removing this for now, as I don't think its needed context and just takes extra space
Comment thread skills/doc-edit/SKILL.md Outdated
linter-fix:   19 error MD012/no-multiple-blanks Multiple consecutive blank lines [Expected  1; Actual  2]
Comment thread skills/doc-edit/SKILL.md
not just "this reads oddly."
- **Fast-scan checklist:** before signing off on any doc, confirm each of these is true. If any is unclear, jump
to the matching section below for the full rule.
- [ ] Headings use sentence-style casing.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

aren't these already codified in a .md? https://github.com/netfoundry/docusaurus-shared/blob/main/nf-style-guide.md why not just say "use that .md" or something similar?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is a good point. I honestly followed the format I did as it is Anthropic's model for skill creation and the features of AI in my previous role was very siloed, they couldn't go to URLs, so that is partly out of habit. So this is a great point.

I asked Claude after you brought this up, and from my understanding it seems like if we were to link directly to the current .md for the style guide that the agent would use extra tools to get to the page and ingest the info to markdown. The link workflow would use more tokens than the current format that we have, at least from what Claude says. Let me know what you think on that. Maybe we can discuss further in person today. I have some thoughts on a best of both worlds idea, but it might be easier to talk on it rather than type it. :)

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.

3 participants