Skip to content

Require agents to check existing issues and PRs before starting work - #691

Open
thehobbit85 wants to merge 1 commit into
seedds:mainfrom
thehobbit85:codex/agent-pr-issue-check
Open

thehobbit85 wants to merge 1 commit into
seedds:mainfrom
thehobbit85:codex/agent-pr-issue-check

Conversation

@thehobbit85

@thehobbit85 thehobbit85 commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Conflict refresh — 2026-11-10 UTC

Rebased abd8f73b484d91e0d6743112f908438d2f01ef0a onto bbea4c90f136f478e25228bc3d4ab33c77420f2e as 6d7544678058a0842199df9278b98892a5ece1b6. 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_docs passed all six current documentation tests on this exact commit. AGENTS is 1,483 words. Ancestry, range-diff and git diff --check passed. 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; candidate abd8f73b484d91e0d6743112f908438d2f01ef0a. 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_docs passed all 11 tests; git diff --check passed. 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 1c12377c89d77d028634d471342fa065d27dda06 is exactly one commit directly atop 975bb4c14456d145eb3d969814534eecdd996ce9. Its tree matches tested pre-squash 1a903275ae3101ee370492b7fdc5d4205e283191. All 11 documentation tests passed (DM_GAME=010015100B514000 python3 -m unittest -v dm.tools.tests.test_docs). AGENTS.md is 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 main bd98317eecfe832b40d401219c15207a398e78dd. 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:

Commit 9d5fcf02ae35ebb2313133f6fce1301f3b5e484a
Author: seedds <seedds@gmail.com>

Require agents to check existing issues and pull requests

Rebased/corrective commits before this squash:

Commit 1a903275ae3101ee370492b7fdc5d4205e283191
Author: seedds <seedds@gmail.com>

Require agents to check existing issues and pull requests

@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch 11 times, most recently from 89d864e to 1dbafe6 Compare October 31, 2026 22:07
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch 3 times, most recently from 049e8c0 to f3cb539 Compare November 1, 2026 06:23
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch from 6483dd6 to 1a8634f Compare November 1, 2026 12:31
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch 9 times, most recently from 058ae65 to 40233cb Compare November 3, 2026 03:13
@github-actions github-actions Bot added host The Horizon host: SVCs, services, threads, memory (dm/host) docs Documentation labels Nov 3, 2026
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch from 40233cb to 9d5fcf0 Compare November 4, 2026 00:46
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch 2 times, most recently from 1c12377 to abd8f73 Compare November 5, 2026 06:04
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
thehobbit85 force-pushed the codex/agent-pr-issue-check branch from abd8f73 to 6d75446 Compare November 10, 2026 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation host The Horizon host: SVCs, services, threads, memory (dm/host)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants