Repository navigation
[delight] UX Analysis Report — 2026-04-26 #28595
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Delight. A newer discussion is available at Discussion #28715. |
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.
Executive Summary
Today's targeted analysis covered:
docs/src/content/docs/reference/triggers.mdvalidate,pr(short descriptions audit)approach-validator.md,architecture-guardian.mdcall_workflow_validation.go,template_validation.goOverall Quality:⚠️ Needs Minor Work
Key Finding: The
approach-validator.mdfailure message exposes a raw{status}placeholder without actionable guidance, creating a confusing experience when the workflow fails. A companion pattern issue exists inarchitecture-guardian.md. The best-practice pattern is already established inarchie.md.Quality Highlights ✅
1. Exemplary Error Messages —
call_workflow_validation.gopkg/workflow/call_workflow_validation.gonotFoundErrmessage is a model of clarity — it tells the user what was searched, where, and provides 3 concrete corrective actions."Checked for: %s.md, %s.lock.yml, %s.yml\n\nTo fix:\n1. Verify the workflow file exists in .github/workflows/\n2. Ensure the filename matches exactly..."2. Best-Practice Failure Message —
archie.md.github/workflows/archie.mdrun-failuremessage avoids the raw{status}placeholder and includes a direct log link:"📐 [{workflow_name}]({run_url}) encountered an issue and could not complete the architecture diagram. Check the [run logs]({run_url}) for details."Improvement Opportunities 💡
High Priority
Opportunity 1: Vague Failure Message —
approach-validator.md.github/workflows/approach-validator.md"❌ [{workflow_name}]({run_url}) {status} during approach validation."{status}placeholder renders as a technical status string (e.g., "failed") without context. The message provides no actionable next step. Compare toarchie.mdwhich links to logs.Medium Priority
Opportunity 2: Generic Documentation Meta Description —
triggers.mddocs/src/content/docs/reference/triggers.mddescription: Triggers in GitHub Agentic Workflowsfrontmatter.mdwhich has:"Complete guide to all available frontmatter configuration options for GitHub Agentic Workflows, including triggers, permissions, AI engines, and workflow settings."— that description adds discoverability value. Additionally, the introductory paragraph (line 8) is terse and does not highlight the gh-aw-specific enhancements that differentiate this from standard GitHub Actions triggers.Files Reviewed
Documentation
docs/src/content/docs/reference/triggers.md— Rating:CLI Commands
gh aw validate— Rating: ✅ Professionalgh aw pr— Rating:Workflow Messages
.github/workflows/approach-validator.md— Rating: ❌ Needs Significant Work (run-failuremessage).github/workflows/archie.md— Rating: ✅ Professional (best-practice reference)Validation Code
pkg/workflow/call_workflow_validation.go— Rating: ✅ Professionalpkg/workflow/template_validation.go— Rating: ✅ ProfessionalMetrics
🎯 Actionable Tasks
Task 1: Fix Failure Message —
approach-validator.mdFile to Modify:
.github/workflows/approach-validator.mdCurrent Experience
Line 49 contains:
The
{status}token renders as a raw technical string. There is no link to logs or actionable guidance.Quality Issue
Design Principle: Actionable Communication
When a workflow fails, users must know where to go next. The current message leaves them without direction.
Before:
After:
Why This Matters
archie.md)Success Criteria
.github/workflows/approach-validator.mdonly{status}placeholder removed({run_url})Scope Constraint
.github/workflows/approach-validator.mdTask 2: Improve Meta Description —
triggers.mdFile to Modify:
docs/src/content/docs/reference/triggers.mdCurrent Experience
Line 2:
This repeats the title verbatim and adds no additional discoverability or context.
Quality Issue
Design Principle: Clarity and Precision
Meta descriptions in Astro Starlight populate both
<meta name="description">and the search index summary. A generic description fails to differentiate this page from a title alone.Before:
After:
Why This Matters
Success Criteria
docs/src/content/docs/reference/triggers.mdonlyScope Constraint
docs/src/content/docs/reference/triggers.mdReferences: §24956162884
All reactions