Repository navigation
Agent Persona Exploration - 2026-03-07 #19917
Closed
Replies: 2 comments
|
/plan also extract cache deduplication as shared agentic workflow |
0 replies
|
This discussion has been marked as outdated by Agent Persona Explorer. A newer discussion is available at Discussion #21155. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
This report summarizes findings from systematic testing of gh-aw workflow configuration quality across 5 software worker personas and 7 representative scenarios. Note: the
agentic-workflowscustom agent was not available in the task runner environment; scenarios were evaluated against gh-aw best practices using a general-purpose agent with full codebase context.Persona Overview
Key Findings
scheduletriggerworkflow_runtrigger is correct for post-CI scenarios but requires an explicitif: github.event.workflow_run.conclusion == 'failure'guard — this is easily missedactionstoolset +bash: trueare frequently needed for log-based workflows but commonly omitted in first attemptspull-requests: writevia safe-outputs delegation, minimal permissions, andcontents: readwere correctly applied across scenariosTop Patterns
pull_request+pathsfilter (PR automation),workflow_run+if-guard (event-driven),schedule+workflow_dispatch(digests)pull_requests+repostoolsets),actionstoolset for log access,playwrightfor visual workflows,cache-memoryfor deduplicationpull-requests: writedirectly), minimal permission scopes,update-comment/close-older-issuesto prevent output sprawlView High Quality Responses (Score ≥ 4.4)
PM-1: Weekly Feature Digest (4.8/5)
schedule(Friday 8 AM) +workflow_dispatch— perfect fitpull_requests+issues+discussionsonly — no bash, no artifactscreate-discussionwithclose-older-discussions: truekeeps only current week visibleBE-1: DB Schema Migration Safety (4.6/5)
pull_request+paths: ["db/migrate/**", "**/*.sql"]— precise activationpull_requests+repostoolsets onlyCOMMENTnotREQUEST_CHANGESto avoid blocking merges on AI misfiresDO-1: Deployment Failure Incident (4.4/5)
workflow_run+if: conclusion == 'failure'— mirrors existingci-doctor.mdpattern exactlyactionstoolset +cache-memoryfor deduplicationsteps:pre-run block should pre-filter logs to failed jobs only before passing to AIView Areas for Improvement (Score ≤ 3.6)
BE-2: Deployment Failure Log Triage (3.6/5)
bash: trueis required to decompress GitHub's gzip log archives — commonly omittedworkflow_runalways fires on completion; theif: conclusion == 'failure'guard must be manually specified and documentedQA-2: Weekly Flaky Test Digest (3.6/5)
Score Summary
Recommendations
Document the
workflow_runfailure-filter pattern prominently — theif: github.event.workflow_run.conclusion == 'failure'guard is non-obvious, easy to omit, and causes workflows to run on every CI completion. Add it as a required pattern in workflow templates for event-driven scenarios.Create an "artifact-dependent workflow" checklist — workflows that rely on CI-emitted artifacts (coverage reports, JUnit XML) should document their prerequisites upfront. Consider a warning in the compiler or documentation when
actionstoolset is used without a corresponding artifact-name constant in the prompt.Promote
cache-memorydeduplication as a standard practice — every workflow that creates issues or discussions should usecache-memoryto avoid duplicate outputs on retry runs. This pattern (already used inci-doctor.md) should be highlighted in the getting-started guide for all event-driven and scheduled workflows.References:
All reactions