feat: add seven UX-laws lenses (complete the /ux-laws-review roster)#4
Open
vigneshsubbiah16 wants to merge 1 commit into
Open
feat: add seven UX-laws lenses (complete the /ux-laws-review roster)#4vigneshsubbiah16 wants to merge 1 commit into
vigneshsubbiah16 wants to merge 1 commit into
Conversation
…rchitecture, attention, familiarity, gestalt-visual, targeting) Completes the 8-lens roster for /ux-laws-review. With trust-and-honesty (shipped in PR #2) plus these seven, every law from the 30 Laws of UX re-weighted for 2026 AI-native development now lives in exactly one canonical lens, and the parent skill's surface-aware preset routing resolves cleanly for all five surfaces (checkout, onboarding, dashboard, settings, AI chat). S-tier law distribution (10 unanimous-high laws, each in exactly one lens): - speed-and-flow: Doherty Threshold, Flow - cognition: Cognitive Load, Working Memory - decision-architecture: Choice Overload, Occam's Razor - attention: Paradox of the Active User - familiarity: Mental Model - trust-and-honesty: Tesler's Law, Peak-End Rule (existing) Each lens file mirrors the canonical trust-and-honesty.md template: H1 framing > How to use me > Laws covered > What I check > Scoring rubric > Evidence I collect > Output > Gating behavior > Key principles. All eight lens files share the same JSON output contract so the parent skill assembles a unified scorecard at /build Step 4.2 without per-lens schema awareness. Evidence vocabularies adapt per lens: - speed-and-flow: quantitative (INP/LCP p75, waterfalls, progress affordances) - cognition: structural (decision counts, chunk-boundary analysis) - decision-architecture: structural + behavioral (choice counts, default presence) - attention: visual + behavioral (heat-map, power-user-vs-first-timer) - familiarity: conceptual (convention adherence, comparison-vs-leaders) - gestalt-visual: visual (annotated screenshots, grouping violations) - targeting: quantitative + visual (target size, thumb-reach zones) Gating is uniform across all eight lenses: lens avg < 6 = WARN, < 4 = FAIL. S-tier laws on critical-flow surfaces (signup, checkout, payment, AI chat) scoring < 5 escalate to FAIL via the parent's s_tier_flags consumption. Lenses without S-tier laws (gestalt-visual, targeting) cannot FAIL via the S-tier path by design. Cross-references between lenses are italicized read-only rows at the bottom of relevant check tables, never scoring rows; each law is scored exactly once. Two cross-references exist today, both pointing to Tesler's Law in trust-and-honesty (from decision-architecture's Choice Overload row and familiarity's Mental Model row). Reviewed by principal-architect (plan), principal-engineer (implementation), code-simplifier (consistency), elite-pr-reviewer (0 CRITICAL, 1 self-downgraded WARNING, 11 confirming-clean INFOs). The one actionable INFO (Paradox key naming) was addressed before commit. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Summary
/ux-laws-reviewroster. Combined withtrust-and-honesty(shipped in PR feat: add /ux-laws-review parent + trust-and-honesty lens #2), every law in the 30 Laws of UX (re-weighted for 2026 AI-native development) now lives in exactly one canonical lens.trust-and-honesty.mdtemplate; identical JSON output contract across all 8 lenses so the parent skill assembles a unified scorecard at/buildStep 4.2 without per-lens schema awareness.The seven lenses
speed-and-flowcognitiondecision-architectureattentionfamiliaritygestalt-visualtargetingS-tier coverage (10/10, no duplicates)
speed-and-flow→ Doherty Threshold, Flowcognition→ Cognitive Load, Working Memorydecision-architecture→ Choice Overload, Occam's Razorattention→ Paradox of the Active Userfamiliarity→ Mental Modeltrust-and-honesty(already merged) → Tesler's Law, Peak-End RuleDecisions taken (with council weigh-in)
Reviewed by
principal-architect→principal-engineer→code-simplifier→elite-pr-reviewer→principal-architect(final sign-off):*Related law:*cross-references elsewhere. Never re-score a law in a non-canonical lens.targetingas an intentional one-law lens (~91 lines). Parent routing already wires it as a peer.Cross-references (today, both → Tesler's Law)
decision-architecture.mdChoice Overload row → italicized one-liner pointing readers to Tesler's Law in trust-and-honesty (smart defaults are where the irreducible complexity lives architecturally)familiarity.mdMental Model row → italicized one-liner pointing readers to Tesler's Law in trust-and-honesty (pretending the agent's complexity is gone breaks the user's mental model fastest)Surface-preset routing now resolves cleanly
All eight referenced lens files exist on disk after this PR lands.
Review findings addressed
elite-pr-reviewerpass 1: 0 CRITICAL, 1 WARNING (self-downgraded as the pattern was consistent across all 3 multi-S-tier files), 11 INFOs (10 confirming-clean observations + 1 actionable).Action taken: renamed snake_case key
paradox_active_user→paradox_of_the_active_userinattention.mdto match the architect's plan (3 occurrences). All other observations were confirming the implementation matches the plan.principal-architectpost-fix sign-off: READY TO MERGE — 7 files scoped, paradox rename consistent across all 3 usage sites, S-tier coverage 10/10 with no duplication.Out of scope (separate follow-up PRs)
/eng-blocker-counciland/ux-blocker-councilskills — referenced as forward-references incommands/build.mdStep 5 (shipped in PR feat: fan out three reviewers in parallel at /build Step 4 #3); now genuinely the next thing to buildcommands/ux-laws-review.md— already stable, governs lens routingTest plan
Markdown skill library, no automated test framework. Verification by inspection (architect's 7-check plan):
trust-and-honesty.mdbaseline (same conventions, same long-line / bare-fence style)install.sh --forceinstalls all 7 new lenses as namespaced commands (/ux-laws-review:speed-and-flow, etc.)commands/ux-laws-review/, nothing else changed/ux-laws-reviewagainst a real PR after merge to confirm fan-out behavior across all eight lenses🤖 Generated with Claude Code