Skip to content

Conversation

@bpmct
Copy link
Member

@bpmct bpmct commented Dec 7, 2025

Adds a style guide documenting PR description patterns observed in the Coder repository. This guide is intended for AI agents to reference when creating PRs, ensuring consistency with project conventions.

The guide covers title format (Conventional Commits), description structure (default concise vs. complex structured), what to include (links, performance context, warnings), and what to avoid (test plans, benefits sections). Includes examples from recent merged PRs demonstrating each pattern.

Placed in .claude/docs/ alongside other agent-specific documentation (WORKFLOWS.md, ARCHITECTURE.md, etc.) rather than in the main contributing docs, as this is primarily for automated tooling reference.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

Adds a style guide documenting PR description patterns observed in the
Coder repository. This guide is intended for AI agents to reference when
creating PRs, ensuring consistency with project conventions.

The guide covers title format (Conventional Commits), description
structure (default concise vs. complex structured), what to include
(links, performance context, warnings), and what to avoid (test plans,
benefits sections). Includes examples from recent merged PRs
demonstrating each pattern.

Placed in .claude/docs/ alongside other agent-specific documentation
(WORKFLOWS.md, ARCHITECTURE.md, etc.) rather than in the main
contributing docs, as this is primarily for automated tooling reference.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@bpmct bpmct force-pushed the docs/pr-description-style-guide branch from 08203b2 to 2bc9bec Compare December 7, 2025 22:34
Updates the PR style guide to instruct agents to always create draft PRs
unless explicitly told otherwise. This allows users to review changes
before the PR enters the review queue.

Also adds the PR_STYLE_GUIDE.md to the auto-imported docs list in
CLAUDE.md so agents will naturally reference it when creating PRs.

Changes:
- Added "Creating PRs as Draft" section with --draft flag usage
- Updated key principles to prioritize draft PRs
- Added PR_STYLE_GUIDE.md reference to CLAUDE.md imports

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bpmct added a commit that referenced this pull request Dec 7, 2025
Add comprehensive documentation style guide in .claude/docs/ directory
following the format established in PR #21148 for PR_STYLE_GUIDE.md.

This guide captures observed patterns from existing Coder documentation
including:

- Research requirements before writing (code verification, permissions,
  thresholds)
- Document structure and organization patterns
- Image usage and screenshot guidelines
- Writing style and terminology conventions
- Code examples and formatting standards
- Documentation manifest requirements
- Accuracy standards for specific numbers, permissions, and API endpoints
- Proactive documentation approach for upcoming features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bpmct added a commit that referenced this pull request Dec 7, 2025
Add comprehensive documentation style guide in .claude/docs/ directory
following the format established in PR #21148 for PR_STYLE_GUIDE.md.

This guide captures observed patterns from existing Coder documentation
including:

- Research requirements before writing (code verification, permissions,
  thresholds)
- Document structure and organization patterns
- Image usage and screenshot guidelines
- Writing style and terminology conventions
- Code examples and formatting standards
- Documentation manifest requirements
- Accuracy standards for specific numbers, permissions, and API endpoints
- Proactive documentation approach for upcoming features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bpmct added a commit that referenced this pull request Dec 7, 2025
Add comprehensive documentation style guide in .claude/docs/ directory
following the format established in PR #21148 for PR_STYLE_GUIDE.md.

This guide captures observed patterns from existing Coder documentation
including:

- Research requirements before writing (code verification, permissions,
  thresholds)
- Document structure and organization patterns
- Image usage and screenshot guidelines
- Writing style and terminology conventions
- Code examples and formatting standards
- Documentation manifest requirements
- Accuracy standards for specific numbers, permissions, and API endpoints
- Proactive documentation approach for upcoming features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bpmct added a commit that referenced this pull request Dec 7, 2025
Add comprehensive documentation style guide in .claude/docs/ directory
following the format established in PR #21148 for PR_STYLE_GUIDE.md.

This guide captures observed patterns from existing Coder documentation
including:

- Research requirements before writing (code verification, permissions,
  thresholds)
- Document structure and organization patterns
- Image usage and screenshot guidelines
- Writing style and terminology conventions
- Code examples and formatting standards
- Documentation manifest requirements
- Accuracy standards for specific numbers, permissions, and API endpoints
- Proactive documentation approach for upcoming features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bpmct added a commit that referenced this pull request Dec 7, 2025
Add comprehensive documentation style guide in .claude/docs/ directory
following the format established in PR #21148 for PR_STYLE_GUIDE.md.

This guide captures observed patterns from existing Coder documentation
including:

- Research requirements before writing (code verification, permissions,
  thresholds)
- Document structure and organization patterns
- Image usage and screenshot guidelines
- Writing style and terminology conventions
- Code examples and formatting standards
- Documentation manifest requirements
- Accuracy standards for specific numbers, permissions, and API endpoints
- Proactive documentation approach for upcoming features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ibetitsmike
Copy link
Contributor

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Addresses feedback to avoid including specific PR links that would
bloat the context. Replaced PR number references with generic
descriptive labels (e.g., 'bugfix', 'dependency update', 'API
documentation fix').

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Copy link
Member Author

bpmct commented Dec 8, 2025

Removed all PR number references (e.g., 'PR #21100', 'PR #21080') to avoid context bloat. Replaced with generic descriptive labels like 'Example (bugfix):', 'Example (major documentation rewrite):' etc.

Copy link
Contributor

@ibetitsmike ibetitsmike left a comment

Choose a reason for hiding this comment

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

Some small nitpicks to sort out.

- Remove CONTRIBUTING.md link to avoid context bloat
- Clarify dependency update example with generic placeholder
- Fix grammar: 'where' -> 'were'

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@bpmct bpmct enabled auto-merge (squash) December 8, 2025 19:47
@bpmct bpmct disabled auto-merge December 8, 2025 19:47
Rewrite docs correction example to be clearer and more concise.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@bpmct bpmct enabled auto-merge (squash) December 8, 2025 19:48
@bpmct bpmct merged commit 493f771 into main Dec 8, 2025
29 checks passed
@bpmct bpmct deleted the docs/pr-description-style-guide branch December 8, 2025 19:55
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants