Sync AGENTS.md commands from contributing docs via prek hook#68204
Conversation
|
@RoyLee1224 A few things need addressing before review — see our Pull Request quality criteria. No rush. Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this two-stage triage process so that our maintainers' limited time is spent where it matters most: the conversation with you. |
e29a5a9 to
7596474
Compare
7596474 to
436ef7e
Compare
jason810496
left a comment
There was a problem hiding this comment.
Thanks for the hardening, we just sync in offline call and it LGTM. Thanks.
Backport successfully created: v3-2-testNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
|
related: #62500
Problem
AGENTS.md (468 lines) is loaded in full at session start. Anthropic's own guidance recommends keeping CLAUDE.md under 200 lines — longer files consume more context and reduce adherence. In long multi-turn sessions, agents accumulate enough context to enter the "Dumb Zone": they revert to generic Python patterns and lose Airflow-specific routing (e.g. choosing uv over breeze for a db_test-only case).
A prerequisite to addressing this (via modular skills, selective context loading) is having a single source of truth for the commands that AGENTS.md exposes. Today the Commands section is hand-maintained independently from the contributing docs — they can drift without anyone noticing.
What this PR does
Makes contributing-docs the single source of truth for the AGENTS.md Commands section (#62500).
.. AGENT-SKILL-START/.. AGENT-SKILL-ENDmarkers to three contributing docs (08_static_code_checks.rst, testing/unit_tests.rst, 05_pull_requests.rst)insert_documentation()Future Work
With contributing-docs as the enforced source of truth, subsequent PRs can:
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.