Skip to content

Import org-wide AGENTS.md, remove duplicated standards#92

Merged
don-petry merged 2 commits intomainfrom
chore/import-org-agents-md
Mar 28, 2026
Merged

Import org-wide AGENTS.md, remove duplicated standards#92
don-petry merged 2 commits intomainfrom
chore/import-org-agents-md

Conversation

@don-petry
Copy link
Copy Markdown
Collaborator

@don-petry don-petry commented Mar 28, 2026

Summary

  • Imports shared development standards from petry-projects/.github/AGENTS.md
  • Removes duplicated sections (TDD, CI, PR reviews, security, agent guidance)
  • Keeps project-specific content (repo layout, GAS testing pattern, commands, conventions)

Depends on

Test plan

  • Verify no project-specific content was removed
  • Verify all duplicated generic content was removed

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Unified development guidelines by referencing centralized organization standards.
    • Removed redundant guidance sections to reduce duplication across documentation files.
    • Streamlined testing and code style guidance for improved clarity.

References shared standards from petry-projects/.github for TDD,
CI gates, PR reviews, security, and agent guidance. Keeps only
project-specific content in this repo's AGENTS.md.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 28, 2026 13:53
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 50150dff-87e7-4ab8-8b7e-00dcae4526a7

📥 Commits

Reviewing files that changed from the base of the PR and between 5386727 and 23623d0.

📒 Files selected for processing (2)
  • AGENTS.md
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Documentation restructuring that consolidates development standards guidance by referencing organization-wide shared standards from a common repository. Removed local guidance sections from AGENTS.md and added cross-reference in CLAUDE.md.

Changes

Cohort / File(s) Summary
Documentation Standards Consolidation
AGENTS.md, CLAUDE.md
Added references to centralized organization-wide AGENTS.md from petry-projects/.github repository. Removed redundant local guidance sections from AGENTS.md (Why AGENTS.md, Agent operation guidance, Pull request reviews, Security & secrets), consolidated Tests & CI section, and trimmed Code style & commits guidance to avoid duplication with shared standards.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • Addressing PR comments #80: Introduces CLAUDE.md content that is now being updated with organization-wide standards reference in this PR.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Import org-wide AGENTS.md, remove duplicated standards' directly and clearly describes the main changes: importing shared standards and removing duplicates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/import-org-agents-md

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

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

Updates repository agent documentation to inherit organization-wide standards from petry-projects/.github, reducing duplicated guidance while keeping repo-specific conventions in place.

Changes:

  • Adds references/imports to org-wide AGENTS.md standards.
  • Removes duplicated “generic” sections from this repo’s AGENTS.md, keeping project-specific guidance.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
CLAUDE.md Adds an org-wide standards import/reference at the top of the Claude context file.
AGENTS.md Adds an “Organization standards” note and removes duplicated generic guidance sections.

Comment thread CLAUDE.md
Comment thread AGENTS.md Outdated
- Add readable fallback note below Claude Code @import in CLAUDE.md
- Update 'single source of truth' wording to reflect inheritance model

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@don-petry
Copy link
Copy Markdown
Collaborator Author

Re: the @https://... line in CLAUDE.md — this is intentional. The @ prefix is a Claude Code import directive that tells Claude Code to fetch and inline the referenced file as context. It's not meant to render as standard Markdown.

That said, the readability concern is valid for non-Claude readers. I've added a standard Markdown blockquote link below the import line so the file is useful for everyone:

Org-wide standards: This project inherits shared development standards from petry-projects/.github/AGENTS.md.

Also updated the "single source of truth" wording in AGENTS.md per the suggestion. Both fixes are in 23623d0.

@sonarqubecloud
Copy link
Copy Markdown

@don-petry don-petry merged commit 1428a1e into main Mar 28, 2026
13 checks passed
@don-petry don-petry deleted the chore/import-org-agents-md branch March 28, 2026 14:48
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.

2 participants