Skip to content

docs(agents): point at canonical framework conventions, drop dup ecosystem section#152

Merged
ANcpLua merged 1 commit into
mainfrom
claude/agents-canonical-pointer
May 13, 2026
Merged

docs(agents): point at canonical framework conventions, drop dup ecosystem section#152
ANcpLua merged 1 commit into
mainfrom
claude/agents-canonical-pointer

Conversation

@ANcpLua
Copy link
Copy Markdown
Owner

@ANcpLua ANcpLua commented May 13, 2026

Summary

Adds a "Framework conventions" pointer at the top of AGENTS.md that
links to the canonical doc at
ANcpLua/renovate-config,
and removes the duplicated ## ANcpLua Ecosystem section (with its
### Branch protection, ### Dependency graph, ### Release flow
sub-sections) that is now sourced from the canonical doc.

Repo-specific content — including the German ## Cross-Repo Awareness
deep-ops notes — stays here untouched.

Why

Before this change, the same framework conventions were duplicated in
four AGENTS.md files. When the AUTOMERGE_APP_ID GitHub App was
deleted on 2026-05-12 as an antipattern, none of the four AGENTS.md
files captured the reasoning, and the next session had to rediscover
why three of four repos' auto-merge had silently broken. The canonical
README (landed in renovate-config PR #26) is the prophylactic against
that re-litigation; this PR just wires this repo into it.

Test plan

  • Diff is mechanical: pointer block added after the description,
    ## ANcpLua Ecosystem section removed
  • CI passes

🤖 Generated with Claude Code


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

…ystem section

Adds a "Framework conventions" pointer at the top of AGENTS.md linking
to ANcpLua/renovate-config (the canonical home for branch protection,
auto-merge, CodeRabbit posture, release flow, dependency graph, and
cross-repo bootstrap rules across all four framework repos).

Removes the "## ANcpLua Ecosystem" section (with its "### Branch
protection", "### Dependency graph", "### Release flow" sub-sections)
since it now lives in the canonical doc — duplication is what caused
the recent re-litigation about the deleted AUTOMERGE_APP_ID GitHub App.

The German "## Cross-Repo Awareness" deep-ops notes stay (repo-specific
phrasing not fully captured in the canonical README's English version).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 13, 2026 03:38
@ANcpLua ANcpLua enabled auto-merge (squash) May 13, 2026 03:38
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 13, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • AGENTS.md is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: ec50bbc9-a2d8-41b8-b80a-af5bd8babef5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@github-actions
Copy link
Copy Markdown

@coderabbitai autofix

@claude
Copy link
Copy Markdown

claude Bot commented May 13, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 13, 2026

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Autofix skipped. No unresolved CodeRabbit review comments with fix instructions found.

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 this repo’s AGENTS.md to defer shared “framework conventions” guidance to the canonical documentation in ANcpLua/renovate-config, reducing duplication and keeping repo-specific operational notes in place.

Changes:

  • Added a “Framework conventions” section at the top linking to the canonical cross-repo conventions doc.
  • Removed the duplicated ## ANcpLua Ecosystem section (including branch protection, dependency graph, and release flow details).

Comment thread AGENTS.md
@@ -1,6 +1,14 @@
# AGENTS.md — ANcpLua.NET.Sdk

Opinionated MSBuild SDK providing standardized defaults, policy enforcement, and analyzer injection for .NET projects. `CLAUDE.md` symlinks here.
@ANcpLua
Copy link
Copy Markdown
Owner Author

ANcpLua commented May 13, 2026

@coderabbitai autofix

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 13, 2026

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Autofix skipped. No unresolved CodeRabbit review comments with fix instructions found.

@ANcpLua
Copy link
Copy Markdown
Owner Author

ANcpLua commented May 13, 2026

pr-heal: providers tried — CodeRabbit autofix=true, Claude=failure. None resolved the block (state was BLOCKED). Next sweep in 15 min; if this is recurring, the Codex/Copilot fallback tiers are tracked in pr-heal.yml.

@ANcpLua ANcpLua merged commit af8eecf into main May 13, 2026
16 checks passed
@ANcpLua ANcpLua deleted the claude/agents-canonical-pointer branch May 13, 2026 03:50
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