chore: install ui-ux-pro-max skill, add HTML prototype, and GitHub Pages deployment#8
Merged
Merged
Conversation
Add UI/UX Pro Max design intelligence skill (67 styles, 96 palettes, 57 font pairings, 99 UX rules, 13 stacks) via uipro-cli. Update SKILLS.md directory structure, quick reference, and phase guide. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add clickable HTML prototype (login + dashboard) under prototype/pages/ with index.html entry point. Add GitHub Actions workflow to auto-deploy prototype/ to GitHub Pages on push to main. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Review Summary by QodoAdd UI/UX Pro Max skill, design system generator, and interactive prototype with GitHub Pages deployment
WalkthroughsDescription• Install UI/UX Pro Max design intelligence skill with 67 styles, 96 palettes, 57 font pairings • Add comprehensive design system generator with BM25 search and Master + Overrides pattern • Create interactive HTML prototype (login + dashboard) with accessibility and responsive design • Deploy prototype to GitHub Pages via automated GitHub Actions workflow Diagramflowchart LR
A["UI/UX Pro Max Skill"] --> B["Design System Generator"]
B --> C["BM25 Search Engine"]
B --> D["Master + Overrides Pattern"]
E["HTML Prototype"] --> F["Login Page"]
E --> G["Dashboard Page"]
F --> H["GitHub Pages"]
G --> H
H --> I["Live at GitHub Pages URL"]
File Changes1. .claude/SKILLS.md
|
Code Review by Qodo
1.
|
- design-system/labelsuite/MASTER.md: persisted design system from ui-ux-pro-max (Flat Design, Indigo/Emerald palette, Crimson Pro + Atkinson Hyperlegible typography) - specs/001-sso-login/spec.md: feature spec for Google/GitHub SSO login with logout support (4 user stories, 10 FRs, 6 SCs) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
- Replace Vietnamese example text with English in SKILL.md (English-first rule) - Remove undocumented 'prompt' domain from SKILL.md domain table - Replace broken tasks.html links with aria-disabled="#" in dashboard prototype Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove 'prompt' from search.py docstring and core.py style keywords since it is not a supported --domain value (closes qodo issue #2) - Remove committed __pycache__/*.pyc files from git tracking - Add .gitignore with Python, Node, env, and IDE exclusions (closes qodo issue #4) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Owner
Author
|
All qodo findings have been addressed:
@qodo-merge review |
singyichen
added a commit
that referenced
this pull request
Apr 10, 2026
CLAUDE.md: - Reduce SDD section from 62 lines to ~20 lines; remove pipeline table, status trigger list, and archive bash block (all now live in SKILL.md and specs/STATUS.md) - Keep only hard behavioral constraints: pipeline one-liner, TDD MUST NOT rule, pre-PR analyze gate, module names, design paths, status/archive pointers sdd-workflow SKILL.md: - Add /superpowers:brainstorm as first pipeline step with gate condition - Add STATUS.md update annotations at each pipeline stage - Mark /speckit.analyze as REQUIRED gate (was "optional") - Add archive step at pr-flow stage - Update Spec Directory Structure to show STATUS.md and _archive/ - Add /superpowers:brainstorm to Spec-Kit Commands table - Add new TDD Rule section with Red-Green-Refactor, applies-to scope, anti-rationalisation table, and restart instruction - Expand SDD Rules from 5 to 8: add TDD rule (#3), analyze gate (#4), archive on merge (#8) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
singyichen
added a commit
that referenced
this pull request
Apr 14, 2026
* docs: add spec status index and archive convention
- Add specs/STATUS.md: single source of truth for all 16 feature pipeline
statuses, with status legend, feature table, and archive log
- Add specs/_archive/ directory for completed specs post-merge
- Update CLAUDE.md: add Spec Status Index section with required update
triggers, Spec Archive Convention with bash command, and two new rows
in Workflow Quick Reference table
Addresses weak document indexing and unclear completion status in spec-kit.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: strengthen SDD pipeline with brainstorm, TDD, and analyze gate
- CLAUDE.md: add full SDD pipeline table (8 stages with hard gates) at
the top of the SDD section, replacing vague /sdd-workflow reference
- CLAUDE.md: add TDD Rule section with mandatory language ("MUST NOT
write implementation before failing test") and explicit rejection of
common rationalisations
- CLAUDE.md: update Workflow Quick Reference — new feature row now shows
full stage sequence; add Pre-PR gate row requiring speckit.analyze
zero findings before every PR
- speckit.implement.md: step 4 upgraded from suggestion to REQUIRED gate
with anti-rationalisation list and restart instruction
- speckit.implement.md: add step 7 (mandatory /speckit.analyze gate,
must pass before PR); step 8 updates STATUS.md on completion
Addresses three SDD gaps identified from OpenSpec and Superpowers articles:
missing brainstorm entry point, unforced TDD, and optional analyze gate.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: slim CLAUDE.md SDD section and sync sdd-workflow SKILL.md
CLAUDE.md:
- Reduce SDD section from 62 lines to ~20 lines; remove pipeline table,
status trigger list, and archive bash block (all now live in SKILL.md
and specs/STATUS.md)
- Keep only hard behavioral constraints: pipeline one-liner, TDD MUST NOT
rule, pre-PR analyze gate, module names, design paths, status/archive
pointers
sdd-workflow SKILL.md:
- Add /superpowers:brainstorm as first pipeline step with gate condition
- Add STATUS.md update annotations at each pipeline stage
- Mark /speckit.analyze as REQUIRED gate (was "optional")
- Add archive step at pr-flow stage
- Update Spec Directory Structure to show STATUS.md and _archive/
- Add /superpowers:brainstorm to Spec-Kit Commands table
- Add new TDD Rule section with Red-Green-Refactor, applies-to scope,
anti-rationalisation table, and restart instruction
- Expand SDD Rules from 5 to 8: add TDD rule (#3), analyze gate (#4),
archive on merge (#8)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: add 1→N iteration support to SDD workflow
- spec-template: add Version field, Spec Dependencies section (upstream/downstream), and Changelog section
- constitution: add iteration rule (update vs new spec), semantic versioning policy, and downstream impact requirement
- sdd-workflow SKILL: add Iteration Workflow (1→N) section and Cross-Spec Dependencies section
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: add version, spec dependencies, and changelog to account specs 001-002
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* fix: address PR #28 review findings
- specs/STATUS.md: clear stale branch and update notes for feature 012 (wireframe done via PR #27)
- speckit.implement: remove premature STATUS.md→done update; defer to pr-flow
- constitution.md: bump version to 1.3.0 and update Last Amended to 2026-04-13
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* fix: address PR #28 review finding — fix agent-team ordering in workflow table
Move /agent-team before analyze gate in Workflow Quick Reference; it is an
implementation method, not a post-analyze step.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* fix: address PR #28 review findings
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* docs: rewrite AGENTS.md as standard agent rules file
Replace agent directory catalog with a concise rules-focused format.
Reduces token consumption by ~65% (211 → 80 lines) by removing
redundant tables and replacing them with boundary constraints,
hard rules, and required behaviors for autonomous agents.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* fix: address PR #28 review findings — align SDD pipeline across docs
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Sonnet 4.6 <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
1 participant
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
ui-ux-pro-maxdesign intelligence skill viauipro-cli(67 styles, 96 palettes, 57 font pairings).claude/SKILLS.mdto document the new skill (total: 30 skills)prototype/pages/(login + dashboard with logout)prototype/to GitHub Pages on push tomainPrototype Preview
After merge, the prototype will be live at:
https://singyichen.github.io/label-suite/
Pages:
prototype/index.html→ auto-redirects to loginprototype/pages/login.html— SSO login (Google / GitHub buttons)prototype/pages/dashboard.html— dashboard with logout modalTest Plan
login.htmlopens in browser, both SSO buttons navigate todashboard.htmldashboard.htmlSign out button opens confirmation modallogin.htmlrole=alertbanner🤖 Generated with Claude Code