docs: move trigger decision matrix to triggers.md, add coverage/digest fallback guidance#42618
Merged
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…riggers.md - Move decision matrix, scenario examples, and noop patterns from section 2 of create-agentic-workflow.md to triggers.md (new Trigger Selection section) — keeps section 2 concise as requested - Add non-technical persona examples (documentation governance, PM digests, compliance reviews) to triggers.md scenario list - Add section 2e to create-agentic-workflow.md with coverage-analysis compact guidance (artifact-first, PR-signals-first, explicit fallback) - Add fallback pattern for incomplete customer-impact metadata to section 2a of create-agentic-workflow.md and to report.md Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Investigate agent persona exploration findings
docs: move trigger decision matrix to triggers.md, add coverage/digest fallback guidance
Jul 1, 2026
pelikhan
approved these changes
Jul 1, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors and extends the gh-aw authoring guidance by moving the trigger decision matrix and scenario examples into a dedicated reference (triggers.md), while adding explicit fallback guidance for digests/reports when metadata or coverage inputs are incomplete.
Changes:
- Added a new “Trigger Selection” reference section (decision matrix, scenario examples, and
nooppatterns) to.github/aw/triggers.md. - Simplified Section 2 in
.github/aw/create-agentic-workflow.mdto point totriggers.mdand added compact guidance for metadata fallbacks and coverage-analysis fallbacks. - Added repo-wide report/digest fallback rules for incomplete metadata to
.github/aw/report.md. - Updated
.github/workflows/agentics-maintenance.ymlcheckout steps to setclean: falsefor sparseactionscheckouts.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/agentics-maintenance.yml | Adjusts actions/checkout inputs for multiple “checkout actions folder” steps. |
| .github/aw/triggers.md | Introduces a trigger-selection decision matrix and expanded scenario + noop examples. |
| .github/aw/report.md | Adds a cross-workflow “incomplete metadata” fallback policy for reports/digests. |
| .github/aw/create-agentic-workflow.md | Trims trigger guidance and adds compact fallback guidance for metadata and coverage analysis. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 4/4 changed files
- Comments generated: 8
- Review effort level: Low
Comment on lines
108
to
111
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
154
to
157
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
242
to
245
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
289
to
292
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
597
to
601
| with: | ||
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
686
to
690
| with: | ||
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Comment on lines
732
to
736
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false | ||
|
|
Comment on lines
822
to
826
| with: | ||
| sparse-checkout: | | ||
| actions | ||
| clean: false | ||
| persist-credentials: false |
Contributor
|
🎉 This pull request is included in a new release. Release: |
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.
Section 2 of
create-agentic-workflow.mdhad grown too large — it embedded the full trigger decision matrix, scenario examples, and noop patterns inline instead of delegating to a reference file.Changes
.github/aw/triggers.md— new## Trigger Selectionsection (before existing patterns):noopexamples moved here fromcreate-agentic-workflow.md.github/aw/create-agentic-workflow.md— section 2 trimmed to essentials:triggers.mdnoopwhen no data is recoverable.github/aw/report.md— new## Fallback for Incomplete Metadatasection with the same digest-fallback rules scoped to all report-generating workflows