fix: list all 15 stacks in plugin.json description#329
fix: list all 15 stacks in plugin.json description#329hangaraulbogdan-arch wants to merge 1 commit into
Conversation
mrgoonie
left a comment
mrgoonie
left a comment
There was a problem hiding this comment.
Summary: This is a small metadata-only PR, but it still leaves the plugin description out of sync with the actual stack assets.
Risk level: Low
Mandatory gates:
- Duplicate/prior implementation: clear; no merged duplicate found for the same plugin description update.
- Project standards: checked against the existing repo asset layout and
.claude-plugin/plugin.json. - Strategic necessity: clear value; accurate plugin metadata helps discovery, but the metadata needs to match source assets.
Findings:
- Important:
src/ui-ux-pro-max/data/stacks/currently contains 16 stack CSVs, includingthreejs.csv, but the updated plugin description still says15 stacksand does not list Three.js. The PR fixes Angular, Laravel, and Nuxt UI, but leaves the count/list inconsistent with the source-of-truth stack data.
Verdict: REQUEST_CHANGES
mrgoonie
added
agent:github-maintain
…lder#289) (nextlevelbuilder#391) The marketplace-facing manifests understated the catalog. Actual data: 84 styles (styles.csv), 73 font pairings (typography.csv), and 17 stacks (data/stacks/*.csv), but plugin.json, marketplace.json, and skill.json all claimed 67 styles / 57 font pairings / 15 stacks. Update the three manifests to 84 / 73 / 17, and extend the stack enumeration to the full 17 (adding Angular, Laravel, JavaFX, Three.js, and Nuxt UI). Palettes (161) and charts (25) were already correct. Resolves the count-drift finding in nextlevelbuilder#289 (supersedes the stuck nextlevelbuilder#282 and nextlevelbuilder#329, which only partially corrected the numbers).
clark-cant
left a comment
clark-cant
left a comment
There was a problem hiding this comment.
Cron maintainer re-review: PR #329 is a small metadata fix (+1/-1 in plugin.json), but the requested changes from 2026-06-21 remain unaddressed.
Exact fix needed: Update the description to say 16 stacks (not 15) and add Three.js to the stack list. The source-of-truth at src/ui-ux-pro-max/data/stacks/ currently contains 16 CSVs including threejs.csv.
Current status: mergeStateStatus=DIRTY (merge conflicts). Please rebase onto current main and push the updated count.
After fix: This should be a quick merge — single-line metadata change, low risk.
If no update within 14 days, this PR may be closed as stale.
Posted by github-maintain cron at 2026-06-27T01:25:00Z
3 participants
The description claimed 15 stacks but only listed 12. Added the missing Angular, Laravel, and Nuxt UI.