chore: let coding agents run local validation - #76
Merged
Merged
Conversation
The owner now wants coding agents to run formatting, lint, type checking, tests, builds, and Playwright themselves instead of waiting for the owner to validate every change before a push. - AGENTS.md: agents run `pnpm validate` (and `pnpm test:e2e` when behavior can change) and may push and open a draft PR once their own final validation passes; failures are fixed, never skipped; checks that cannot run are named and handed to the owner - Real-provider and paid commands still require an explicit owner ask - README and TESTING describe the shared validation sequence Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016aHdPe3nkZ7bEQRB33qwEm
christopherjnelson
marked this pull request as ready for review
September 29, 2026 18:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Before: AGENTS.md said coding agents must not run formatting, lint, typecheck, tests, builds or Playwright. They had to stop before pushing and wait for the owner to confirm local validation passed.
After: coding agents run
pnpm validatethemselves, pluspnpm test:e2ewhen World Lab or Game API behavior can change. They may push and open a draft PR once their own final validation passes on that commit. Failures are fixed and never skipped. A check that can't run in the agent's environment is named and handed to the owner. Real-provider and paid commands such aspnpm compare:livestill need an explicit owner request.How: the engineering workflow rules in AGENTS.md are rewritten, and the validation sections in README and
docs/TESTING.mdnow describe one shared sequence for the owner and agents.Validation (run locally in the agent session):
pnpm validatepassed (format check, lint, typecheck, tests, build).🤖 Generated with Claude Code
https://claude.ai/code/session_016aHdPe3nkZ7bEQRB33qwEm
Generated by Claude Code