Skip to content

Prevent Design Decision Gate invocation loops #54302

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pelikhan merged 5 commits into from
Aug 20, 2026
+4 −3
Merged

Prevent Design Decision Gate invocation loops #54302

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
79dc3c4
Initial plan
Copilot Aug 20, 2026
2c0148b
Prevent design gate invocation loops
Copilot Aug 20, 2026
15ecb1e
Merge remote-tracking branch 'origin/main' into copilot/copilotfile-d…
Copilot Aug 20, 2026
fd1b1ee
Resolve merge conflict in design decision gate workflow lock
Copilot Aug 20, 2026
7d9ee89
Merge remote-tracking branch 'origin/main' into copilot/copilotfile-d…
Copilot Aug 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/design-decision-gate.lock.yml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 3 additions & 2 deletions .github/workflows/design-decision-gate.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -227,8 +227,9 @@ Stop and emit a safe output **immediately** when any of the following is true:
- Missing `pr.diff` → `mcp__github__get_pull_request_diff` (only if `diff_available` is `true` in the summary; if `false`, the diff exceeds the 300-file API limit — use `pr-files.json` instead and do **not** call the diff API)
- Missing `adr-prefetch-summary.json` → compute manually from PR files and labels
3. Do **not** perform broad exploration. Only fetch extra data if a required field is missing from pre-fetched files.
4. Call exactly one final safe output action (`add-comment`, `push-to-pull-request-branch`, or `noop`) and then stop.
5. If you have enough evidence to decide, stop immediately. Do not gather optional data.
4. Do not use the `Agent` or `Task` tools, delegate work, run `git push`, or investigate safe-output tool availability or permissions. The required safe-output tools are available directly.
5. Call only the minimum final safe outputs, then stop: use `noop` or `add-comment` for every non-draft outcome; when a draft ADR is generated, call `push-to-pull-request-branch` and then `add-comment`.
6. If you have enough evidence to decide, stop immediately. Do not gather optional data.

## Gate Quality Bar

Expand Down
Loading