Repository navigation
Require agents to check existing issues and PRs before starting work - #691
Open
thehobbit85 wants to merge 1 commit into
Open
thehobbit85 wants to merge 1 commit into
thehobbit85 wants to merge 1 commit into
Conversation
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
11 times, most recently
from
October 31, 2026 22:07
89d864e to
1dbafe6
Compare
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
3 times, most recently
from
November 1, 2026 06:23
049e8c0 to
f3cb539
Compare
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
from
November 1, 2026 12:31
6483dd6 to
1a8634f
Compare
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
9 times, most recently
from
November 3, 2026 03:13
058ae65 to
40233cb
Compare
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
from
November 4, 2026 00:46
40233cb to
9d5fcf0
Compare
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
2 times, most recently
from
November 5, 2026 06:04
1c12377 to
abd8f73
Compare
Search upstream open/closed issues and open/merged/closed pull requests before investigation or implementation. Read matched implementations, report overlap and remaining gaps, and link the resulting PR to existing reports. Keep all current repository rules and guide limits.
thehobbit85
force-pushed
the
codex/agent-pr-issue-check
branch
from
November 10, 2026 02:42
abd8f73 to
6d75446
Compare
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
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.
Conflict refresh — 2026-11-10 UTC
Rebased
abd8f73b484d91e0d6743112f908438d2f01ef0aontobbea4c90f136f478e25228bc3d4ab33c77420f2eas6d7544678058a0842199df9278b98892a5ece1b6. Pinned main still lacks the pre-work search of upstream issues and pull requests. The instructions retain the brief rule in AGENTS and the detailed developer-loop checklist, including checking the actual implementation and linking relevant issues. The conflict resolution keeps main's text-only Switch 2 list, its Mac gameplay requirement and its games-table rule.DM_GAME=010015100B514000 /opt/homebrew/bin/python3 -B -m unittest -v dm.tools.tests.test_docspassed all six current documentation tests on this exact commit. AGENTS is 1,483 words. Ancestry, range-diff andgit diff --checkpassed. The diff contains only AGENTS and the developer-loop guide. It has no runtime, game-profile or shader-chain impact. Remote CI on this replacement head is pending verification.Historical proposal, validation and original commit context
The previous description follows in full. Its results apply to the revisions it records.
Rebased the existing feature commit onto main
f8a5618d5f505a4c0a0bcac7c413872d3a53c009; candidateabd8f73b484d91e0d6743112f908438d2f01ef0a. Main still lacks the required pre-work search of upstream issues and open/merged/closed PRs. The author instructions and developer-loop checklist remain distinct, with all current main rules, Front-ends policy and validation requirements preserved.The rebase was clean; the feature patch is unchanged. Fresh exact-candidate validation:
DM_GAME=010015100B514000 /opt/homebrew/bin/python3 -B -m unittest -v dm.tools.tests.test_docspassed all 11 tests;git diff --checkpassed. AGENTS has 1486 whitespace-separated words, within the enforced limit. This is documentation only, with no runtime/chain/profile impact; no gameplay or build campaign is applicable. Prior validation and all original commit context remain historical below.Historical PR description and original commit context (preserved)
Require an upstream issue/PR search before task-specific investigation or implementation, covering open, closed and merged work. Read actual matches, report existing implementations and remaining gaps, avoid duplicates and preserve issue links in resulting PRs. The developer loop documents the search and its limits.
Current
1c12377c89d77d028634d471342fa065d27dda06is exactly one commit directly atop975bb4c14456d145eb3d969814534eecdd996ce9. Its tree matches tested pre-squash1a903275ae3101ee370492b7fdc5d4205e283191. All 11 documentation tests passed (DM_GAME=010015100B514000 python3 -m unittest -v dm.tools.tests.test_docs).AGENTS.mdis 1,486 words (within1,500); current rules, including the exact final identity/Front-ends:instruction, remain present. Complete main/range diffs, links, generated pages,git diff --check, ancestry and one-commit/same-tree proof pass. The net change is instructions and the developer-loop guide only: no runtime, test or baseline behavior changes, and no game validation is applicable.Ownership was taken over only after verifying the existing sole canonical branch matched the published head/upstream, its checkout had no staged/unstaged/untracked work and the fresh active-handle check was clear; this is Agent2's verified takeover, not a new owner-release claim. No source work or operational journals are introduced by the PR.
Earlier description and historical validation
Require agents to inspect existing issues and PRs before investigation or implementation, read the discussion and code, reuse relevant findings, and disclose search limits.
Current revision
9d5fcf02ae35ebb2313133f6fce1301f3b5e484a, rebased onto upstream mainbd98317eecfe832b40d401219c15207a398e78dd. Range-diff and the remaining diff were reviewed; main is an ancestor.Fresh focused validation on this revision: 17 tests passed. The AudioOut test prerequisite is now incorporated upstream and drops out of this branch. The remaining diff contains only the issue/PR search instructions in AGENTS.md and the developer loop guide; no audio code differs from main.
Remote CI for this replacement head is pending verification by the CI agent; earlier successful checks do not validate it. Current checks.
The earlier audio CI failure and sanitizer evidence remain documented in #1008. That incorporated prerequisite is no longer part of this PR’s diff. Documentation and usage checks passed on this revision; no gameplay or device test applies to these guide changes.
Original commit context (historical, preserved through squash)
These messages preserve the original rationale, attribution, validation limits and interface notes. Any references to earlier source or test results remain historical; the current validation section and tree proof above identify the integrated revision.
Original published PR commits:
Rebased/corrective commits before this squash: