Skip to content

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

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

thehobbit85 wants to merge 2 commits into
seedds:mainfrom
thehobbit85:codex/agent-pr-issue-check

Conversation

@thehobbit85

@thehobbit85 thehobbit85 commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Require agents to search the target repository's open, closed and merged issues/PRs before investigating or implementing a task. The guide requires reading discussion and actual code, reporting overlap, reusing relevant findings, linking issues and stating search limits. This rule is still absent from main; existing repository rules and guide anchors are preserved.

Current head 40233cb11b62e6fae5edcb532738a43d15864d4c. python3 -m unittest dm.tools.tests.test_docs passed on this revision (11 tests). The feature sources, tests and documentation match the previously validated revision 058ae6549684 byte for byte.

This PR includes main 6136418bcd564f086cf57cb0a59e8f997d36445e. The required search for existing issues and PRs is still absent from main’s agent guide. The branch retains the separately tested AudioOut wakeup regression.

Local validation on 058ae6549684. python3 -m unittest dm.tools.tests.test_docs dm.tools.tests.test_usage — 17 tests passed.

The native ASan/UBSan audio fixture and the repository’s full threaded TSan target passed.

The notification-draining regression is commit 40233cb11b62. Shared AudioOut test

GitHub checks for this revision

Earlier CI repair evidence

On the earlier documentation commit cb262c62, the sanitizer job reclaimed all six audio buffers in order but expected six notifications and counted five. #1008 fixes that inherited test assumption by draining buffers after each notification.

The full host ThreadSanitizer target passed on a92de64f with the existing sanitizer settings. The executable sources and CI configuration match those validated in #1008. The remaining differences are the original contributor-guide edits in AGENTS.md and docs/agents/06-dev-loop.md.

@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 7 times, most recently from 8ce2f6e to cb262c6 Compare November 2, 2026 17:43
@thehobbit85
thehobbit85 force-pushed the codex/agent-pr-issue-check branch 2 times, most recently from 058ae65 to 40233cb Compare November 3, 2026 03:13
@github-actions github-actions Bot added the host The Horizon host: SVCs, services, threads, memory (dm/host) label Nov 3, 2026
@github-actions github-actions Bot added the docs Documentation label Nov 3, 2026
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