Skip to content

Feat/law industry rewrite - #491

Open
litoondev wants to merge 6 commits into
nextlevelbuilder:mainfrom
litoondev:feat/law-industry-rewrite
Open

litoondev wants to merge 6 commits into
nextlevelbuilder:mainfrom
litoondev:feat/law-industry-rewrite

Conversation

@litoondev

Copy link
Copy Markdown

What does this PR change?

Why?

Checklist

  • Changes were made in src/ui-ux-pro-max/ (source of truth), not directly in .claude/ or .factory/
  • Ran npm run sync:assets && npm run check:assets in cli/ if data/scripts/templates changed
  • Added or updated tests if behavior changed (.claude/skills/*/scripts/tests/, cli/tests/e2e/)
  • Commit messages follow Conventional Commits (feat:, fix:, docs:, etc.)
  • This PR targets a feature branch, not pushed directly to main
litoondev and others added 6 commits May 17, 2026 14:55
Complete A-to-Z rewrite of all data, templates, docs, and metadata
to serve law firms, attorneys, and legal-tech products exclusively.

- products.csv: 82 legal product types (personal injury, criminal defense,
  family, immigration, estate, corporate, IP, tax, real estate, specialty
  practices, firm types, legal-tech SaaS, marketplaces)
- colors.csv: 82 professional legal palettes (navy/gold, charcoal/burgundy,
  oxford/silver, forest/camel, midnight/teal, etc.)
- typography.csv: curated to 42 legal-appropriate font pairings; dropped
  gaming, retro, wedding, crypto, kids — kept multilingual sets for
  immigration/legal-aid (Vietnamese, Japanese, Korean, Chinese, Arabic,
  Thai, Hebrew)
- landing.csv: 22 law-specific patterns (Free Case Evaluation, Funnel,
  Practice Area Deep Dive, Multilingual Immigration, Case Results, etc.)
- styles.csv: 84 rows retargeted to legal UI contexts
- ui-reasoning.csv: 160 rows rewritten for legal UI (intake forms, client
  portal, e-discovery, matter dashboard, attorney bio, etc.)
- charts.csv: 25 legal scenarios (billable hours, docket deadlines,
  settlement amounts, AR aging, e-discovery review, etc.)
- icons.csv: 105 rows with 16 legal-specific icons added (Scales, Gavel,
  Scroll, Signature, SealCheck, Stamp, etc.)
- ux-guidelines.csv, app-interface.csv, react-performance.csv: all
  retargeted to legal product scenarios
- skill.json, plugin.json, marketplace.json: v3.0.0, legal keywords/desc
- 19 platform template JSONs: legal-focused frontmatter descriptions
- CLAUDE.md, README.md, SKILL.md: updated counts and legal examples
- cli/assets/: synced from src/ui-ux-pro-max/

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Bring in gallery, projects, scripts, stack, .github workflows,
community docs, multilingual READMEs, and config files that were
present on the remote but absent from local working tree.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@clark-cant

Copy link
Copy Markdown
Contributor

This PR is significantly oversized for cron-safe review (+17,706 additions across 212 files) and has merge conflicts (DIRTY state).

Scope concerns:

  • Includes an entire separate Gsap-Skill/ directory (its own plugin structure, examples, skills) that appears to be a standalone project, not a modification of ui-ux-pro-max-skill
  • Adds 4 full README translations (vi, zh, ko, id) at ~630-660 lines each
  • Rewrites workflow files, issue/PR templates, and project config files
  • Adds a law-firm-homepage-structure skill that is industry-specific and unrelated to the core skill
  • Modifies CSV data files, Python scripts, and JSON templates across the CLI

Required actions before this can be reviewed:

  1. Resolve merge conflicts with main
  2. Split into focused, independent PRs:
    • Industry-specific skill addition (law-firm) as its own PR
    • Gsap-Skill as a separate repository or plugin PR (it's a full standalone project)
    • Translation additions one language per PR or as a single i18n PR
    • Workflow/config changes as a separate housekeeping PR
  3. Fill in the PR template — currently all checklist items are unchecked and the description fields are empty

This PR cannot be processed until scoped down and rebased.

Posted by github-maintain cron

@clark-cant clark-cant added agent:github-maintain Processed by github-maintain automation maintain:deferred Deferred by maintain workflow pr:oversized Too large for cron-safe review budget labels Sep 6, 2026
@clark-cant

Copy link
Copy Markdown
Contributor

This PR is being deferred due to its scope and size.

Scope assessment:

  • 17,706 additions across 212 files
  • Includes an entire Gsap-Skill subdirectory (separate skill, not part of this repo's core)
  • Adds a gallery/ Next.js app, stack/ design-audit tool, and projects/ example directory
  • Includes 5 README translations (id, ko, vi, zh)
  • Modifies CLI data files, templates, and workflows

Why deferred:
This PR bundles multiple independent concerns that should be separate PRs for review and merge:

  1. Law-firm industry content (the stated purpose)
  2. GSAP skill as a separate subdirectory
  3. Gallery app as a separate tool
  4. Stack design-audit tool
  5. README translations
  6. CLI data and template updates

Next steps for the author:
Please split this into focused PRs:

  • PR for law-firm industry content only
  • PR for GSAP skill (if it belongs in this repo at all — may need maintainer discussion)
  • PR for gallery app
  • PR for stack tool
  • PR for README translations
  • PR for CLI data/template updates

Each PR should be reviewable in isolation. This one exceeds the cron-safe review budget and requires maintainer discussion on scope and architecture.

Labels added: maintain:deferred, pr:oversized

Posted by github-maintain cron

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:github-maintain Processed by github-maintain automation maintain:deferred Deferred by maintain workflow pr:oversized Too large for cron-safe review budget

2 participants