Skip to content

docs: add detailed agent instructions for testing and spellchecking#1048

Open
jkowall wants to merge 1 commit intojaegertracing:mainfrom
jkowall:add-agent-instructions
Open

docs: add detailed agent instructions for testing and spellchecking#1048
jkowall wants to merge 1 commit intojaegertracing:mainfrom
jkowall:add-agent-instructions

Conversation

@jkowall
Copy link
Copy Markdown
Collaborator

@jkowall jkowall commented Dec 26, 2025

Summary

This PR enhances the AGENTS.md file with more comprehensive guidance for AI agents working on the documentation.

Changes

  • Testing and Verification: Added step-by-step instructions for verifying changes locally:

    • Setup with npm install
    • Running the site locally with make develop
    • Building the site with npm run build
    • Running all quality checks before PR submission
  • Spellchecking Guidance: Added detailed information about:

    • The cspell configuration in .cspell.yml
    • Dictionary files in .cspell/ directory
    • How to add words to project-words.txt (sorted alphabetically)
    • How to add names to project-names-src.txt
    • Verifying spellcheck passes
  • Documentation Versioning: Added guidance on:

    • Where to make changes (_dev/ vs versioned directories)
    • When to backport fixes to existing versions
  • Human-Centric Guidelines Reference: Added note pointing to CONTRIBUTING.md for DCO signing, PR etiquette, and community interaction guidelines.

  • Troubleshooting: Added section for spellcheck failures.

Testing

  • Verified locally that the markdown renders correctly

Copilot AI review requested due to automatic review settings December 26, 2025 13:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances documentation for AI agents and expands the frontend UI configuration reference. It adds comprehensive testing and spellchecking instructions to AGENTS.md while documenting several previously undocumented UI configuration options.

Key changes:

  • Added step-by-step testing and verification workflow to AGENTS.md
  • Documented spellcheck configuration and dictionary management for AI agents
  • Expanded frontend-ui.md with detailed configuration options for themes, critical path, tag prefixes, trace graphs, and UI controls

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
AGENTS.md Adds comprehensive testing/verification steps, spellchecking guidance, and documentation versioning instructions for AI agents
content/docs/v2/_dev/deployment/frontend-ui.md Documents new UI configuration options including themes, critical path visualization, top tag prefixes, trace graph settings, and UI control toggles
.cspell/project-words.txt Adds "Emscripten" (GraphViz module technology) to the custom dictionary in correct alphabetical order

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
@jkowall jkowall force-pushed the add-agent-instructions branch 2 times, most recently from 0ce1ee4 to 99d9638 Compare December 26, 2025 16:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md Outdated
Add concise verification checklist and spellcheck dictionary guidance
for AI agents, referencing CONTRIBUTING.md for full procedures instead
of duplicating them.

Signed-off-by: Jonah Kowall <jkowall@kowall.net>
@jkowall jkowall force-pushed the add-agent-instructions branch from 99d9638 to 7dd5767 Compare April 6, 2026 12:55
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