Claude/modern website naoivr#359
Conversation
Adds design intelligence entries for mission-driven and impact-focused product categories inspired by the awesome-social-enterprise ecosystem: - Social Enterprise Platform (nextlevelbuilder#162): B-Corp/hybrid mission-driven orgs, Accessible & Ethical + Organic Biophilic, Impact Metrics Dashboard, earth green + trust blue + amber palette - Impact Investing & ESG Platform (nextlevelbuilder#163): Sustainable finance/ESG, Data-Dense + Swiss Modernism 2.0, ESG Portfolio Dashboard, deep navy + sustainability green + teal palette - Cooperative & Community Ownership (nextlevelbuilder#164): Worker co-ops/credit unions, Accessible & Ethical + Flat Design, Member Analytics Dashboard, solidarity teal + warm amber + grassroots green palette - Microfinance & Community Lending (nextlevelbuilder#165): Financial inclusion/microcredit, Trust & Authority + Accessible & Ethical, Loan Portfolio Dashboard, trust blue + growth green + hope gold palette Synced to cli/assets/data/ per project sync rules. https://claude.ai/code/session_01EgN1neLEnUa1gPPVwBbR6f
|
Summary: I’m deferring this PR in cron-safe review because it is too large and mixes product data changes with a generated website/docs payload.nnDecision: deferred / needs narrowingnnEvidence:n- Scope is 8 files with +1246/-163, including duplicated 515-line generated pages in both docs/index.html and index.html.n- The PR has no description explaining the product/website change, expected generated artifact flow, or why docs and root HTML both need to be committed.n- It also changes CSV data files, so this is not just a static-site refresh.nnNext step: please narrow this to one intent and add a PR description: either data update, website generation output, or docs-site refresh. If generated HTML is intended, please document the generation command/source of truth so maintainers can reproduce it. |
mrgoonie
added
agent:github-maintain
mrgoonie
left a comment
mrgoonie
left a comment
There was a problem hiding this comment.
Thanks for the contribution. I can’t approve this in its current shape.nnImportant blockers:nn1. The PR adds two full static website entrypoints (index.html and docs/index.html, ~1k generated lines) that are not tied to the repo’s skill packaging/runtime contract. This repo’s source of truth is the skill data/templates under src/ui-ux-pro-max/ plus mirrored cli/assets/ when needed. Adding a broad standalone marketing site here creates a maintenance surface without a clear user-facing package path or linked issue.nn2. The CSV sync is inconsistent. src/ui-ux-pro-max/data/products.csv gets only the four new product rows, while cli/assets/data/products.csv is rewritten wholesale (roughly the whole file), creating noisy/generated churn and making future asset-sync validation risky. Please keep canonical src/ and cli/assets/ mirrors minimal and byte/row aligned.nnSuggested narrower path: split this into a focused data PR that only adds the four social-impact product/color rows to both canonical and mirrored CSVs, with no static site files unless there is a linked issue/spec explaining how the site is shipped and maintained.
|
Recommend closing, @mrgoonie. Placeholder/auto-generated PR (Claude/modern website naoivr), a demo-website branch unrelated to the skill. |
|
Closing this as out of scope / not reviewable for this repo. The PR branch appears to be an unrelated generated demo website: it adds duplicated root/docs HTML pages, rewrites product CSV data, has an empty PR body, and is currently conflicted (mergeStateStatus=DIRTY). A contributor also flagged it as a placeholder/auto-generated demo branch.nnIf there is a real repo change intended here, please open a new narrow PR with one source-of-truth change, a clear description, the generation command for any produced website/docs output, and verification evidence. |
4 participants
No description provided.