Skip to content

Ai policy and agent instructions#1187

Merged
jstoobysmith merged 7 commits into
leanprover-community:masterfrom
jfindlay:AI-Policy
Jun 17, 2026
Merged

Ai policy and agent instructions#1187
jstoobysmith merged 7 commits into
leanprover-community:masterfrom
jfindlay:AI-Policy

Conversation

@jfindlay

@jfindlay jfindlay commented Jun 15, 2026

Copy link
Copy Markdown
Contributor
  • AI-POLICY.md: human-facing contract for AI-assisted contributions. Covers author obligations, review process rules, and enforcement consequences.
  • AGENTS.md: instructions for AI agents working on Physlib. Covers content rules, proof structure guidance, the pre-finish checklist, PR scope, commit conventions, and the format for PR descriptions.

The two files cross-reference each other and both point to docs/ReviewGuidelines.md.

Reviewer map: There is no Lean code in this PR. Read AI-POLICY.md first (human obligations), then AGENTS.md (agent-facing rules). Check that the two files are mutually consistent and that together they cover the governance surface you expect.

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for this PR, which will now be reviewed. Please
see our review guidelines
if you are not familiar with the process. You should expect a back and forth
with a reviewer before your PR is merged. See also that link for how to
add appropriate labels to your PR. The PR will also go through a number
of automated checks. You can learn more about these here,
including how to run them locally.

If you want to bring attention to this PR, please write a message on this
thread of the Lean Zulip.

@jfindlay jfindlay mentioned this pull request Jun 15, 2026
Comment thread AI-POLICY.md Outdated
jfindlay and others added 3 commits June 16, 2026 09:45
Agent-actionable instructions belong in the conventionally-discovered
AGENTS.md file; human policy and accountability obligations remain in
AI-POLICY.md. This separation ensures agents can find their instructions
without custom discovery logic, while keeping human-facing policy and
enforcement rules separate.

Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Give agents the merge-blocking checks and a definition-of-done so they
self-verify before finishing.

Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Updated the AGENTS.md file to improve clarity and organization of instructions for AI agents contributing to Physlib.
@jfindlay jfindlay changed the title Ai policy Ai policy and agent instructions Jun 16, 2026
@jfindlay jfindlay marked this pull request as ready for review June 16, 2026 14:05
Comment thread AI-POLICY.md Outdated
- Remove duplicate section from `AI-POLICY.md`
- Remove redundant prefix wording from `AI-POLICY.md` and `AGENTS.md`
- Blank space fixups
Comment thread AGENTS.md

- Use `lemma`, not `theorem`, unless the result is well known in the physics literature.
- Never use the `axiom` declaration.
- Never use `sorry`.

@doxtor6 doxtor6 Jun 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Better to add following too:

Never use structure fields of type True, or theorems returning True.
Never use an existential statement ∃ x, ..., True (any number of quantifiers ending in True).
Never use a scope-level variable hypothesis that assumes the conclusion.

Forbid True-typed structure fields and theorem conclusions, existentials
of the form ∃ x, ..., True, and scope-level hypotheses that assume the
conclusion. These declarations typecheck while asserting nothing, which
defeats the human-certification obligation in AI-POLICY.md.

Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>

@jstoobysmith jstoobysmith left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I've approved this and will merge it shortly

@jstoobysmith jstoobysmith added the ready-to-merge This PR is approved and will be merged shortly label Jun 17, 2026
@jstoobysmith jstoobysmith merged commit 851e49a into leanprover-community:master Jun 17, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR is approved and will be merged shortly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants