[delight] User Experience Analysis Report - 2026-08-11 #52088
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Delight. A newer discussion is available at Discussion #52308. |
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 analysis focused on:
blog/2026-06-02-agent-of-the-day.md,guides/ai-issue-triage.md)gh aw add-wizard, help text sourced frompkg/cli/add_wizard_command.gosince the compiled binary was unavailable offline)daily-fact.md,smoke-agent-public-approved.md)pkg/workflow/heredoc_validation.go)Overall Quality: Mostly professional, with one concrete documentation gap that will cause a first-run failure for new users following the tutorial as written.
Key Finding: The AI issue triage guide's example workflow lists priority labels (
priority/p0,priority/p1,priority/p2) inadd-labels.allowedbut never tells the reader these labels must already exist in the target repository — GitHub's label API returns a 404 if they don't, so a user who copies the example verbatim into a fresh repo will hit a confusing runtime failure with no guidance in the doc that caused it.Quality Highlights ✅
Example 1: Clear security rationale in the issue triage guide
docs/src/content/docs/guides/ai-issue-triage.mdadd-labels/add-commentrestrictions directly to prompt-injection risk.add-labelsandadd-commentmatter for security because the agent does not receive direct write access to issues. gh-aw validates label names and comment output before posting, which reduces the risk of prompt injection turning repository analysis into unrestricted writes."Example 2: Well-documented
auto_createpattern for milestones (contrast case)docs/src/content/docs/reference/safe-outputs.md(assign-milestone section)auto_create, the handler returns a clear error listing the available milestones and suggestingauto_create: true"). This is the standard theadd-labelsexample in the issue-triage guide should meet, but currently doesn't.Improvement Opportunities 💡
High Priority
Opportunity 1: Missing label pre-creation note in AI issue triage guide - Single File Improvement
docs/src/content/docs/guides/ai-issue-triage.mdallowed: [bug, feature, question, needs-info, priority/p0, priority/p1, priority/p2, duplicate]underadd-labels, and the prose immediately below the code block (last paragraph) explains the security rationale foradd-labels/add-commentbut says nothing about label existence requirements.bug,feature, andquestionexist as GitHub's default repo labels, butpriority/p0,priority/p1,priority/p2, andneeds-infoare not defaults. If a reader copies this example into a fresh repository (a very likely first action for anyone following a "getting started with issue triage" guide), the workflow will fail at runtime when it tries to apply a label that doesn't exist, with no signal in the doc explaining why or how to fix it.allowedmust already exist in the repository (unlikeassign-milestone'sauto_createoption), with a one-line pointer to create them viagh label createor the repo Settings > Labels page before running the workflow.Files Reviewed
Documentation
docs/src/content/docs/blog/2026-06-02-agent-of-the-day.md- Rating: ✅docs/src/content/docs/guides/ai-issue-triage.md- Rating:CLI Commands
gh aw add-wizard- Rating: ✅ (clear examples, precise flag documentation, good separation of interactive vs. CI usage)Workflow Messages
.github/workflows/daily-fact.md- Rating: ✅ (whimsical tone is intentional and consistent for this workflow's purpose; footer/status messages are clear).github/workflows/smoke-agent-public-approved.md- Rating: ✅ (concise, professional status messages appropriate for a smoke test)Validation Code
pkg/workflow/heredoc_validation.go- Rating: ✅ (error messages are specific, name the exact invalid character/reason, and doc comments explain defense-in-depth rationale clearly)Metrics
🎯 Actionable Tasks
Task 1: Add label pre-creation note to AI issue triage guide
File to Modify:
docs/src/content/docs/guides/ai-issue-triage.mdCurrent Experience
The guide's example workflow configures
add-labels.allowedwithpriority/p0,priority/p1,priority/p2, andneeds-info— none of which are GitHub default labels. The doc's closing paragraph explains only the security rationale for restricting labels, not that they must pre-exist.Quality Issue
Design Principle: Documentation Quality (completeness) and Trust and Reliability (predictable behavior)
A reader who follows this guide in a brand-new repository will have the workflow run, classify an issue correctly, then fail silently or with an opaque GitHub API error when it tries to apply
priority/p1because the label doesn't exist yet. Nothing in the doc anticipates or explains this failure mode.Proposed Improvement
Add one sentence to the final paragraph of the guide.
Before:
After:
Why This Matters
Success Criteria
docs/src/content/docs/guides/ai-issue-triage.mdonlygh label createexampleScope Constraint
docs/src/content/docs/guides/ai-issue-triage.mdWarning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
storage.googleapis.comTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
All reactions