Skip to content

fix: repair malformed CSV rows, sync CLI assets, fix uninstall type error - #512

Open
alyami-ouja wants to merge 1 commit into
nextlevelbuilder:mainfrom
alyami-ouja:fix/csv-integrity-and-cli-sync
Open

alyami-ouja wants to merge 1 commit into
nextlevelbuilder:mainfrom
alyami-ouja:fix/csv-integrity-and-cli-sync

Conversation

@alyami-ouja

Copy link
Copy Markdown

Summary

Malformed CSV rows (search returned wrong or missing data):

  • typography.csv: two records were merged into one line, so the "Bauhaus Geometric" pairing was unsearchable. Split into its own row (antigravity now supports skill #74).
  • styles.csv row 77 (Neo Brutalism Mobile): a missing closing quote shifted every column after CSS/Technical Keywords.
  • stacks/laravel.csv rows 24, 45: code fields containing commas weren't quoted.
  • stacks/astro.csv row 35, stacks/angular.csv row 28: each row was missing a column. Fields were realigned and the missing value filled in.

Out-of-date CLI assets: cli/assets/ lagged behind src/ui-ux-pro-max/ (e.g. search.py --help listed only 3 stacks). Re-synced per the CLAUDE.md sync rules.

TypeScript error: cli/src/commands/uninstall.ts:104 passed AIType (which includes 'all') to removeSkillDir. Added a narrowing guard. No behavior change.

draft.csv/design.csv were left as-is: they're prose reference docs that the engine doesn't read.

Verification

  • Every CSV the engine reads now parses with consistent column counts
  • Search works in all 7 domains and all 16 stacks, from both src/ and cli/assets/, and --design-system runs
  • tsc --noEmit passes; esbuild bundle succeeds

🤖 Generated with Claude Code

…rror

- typography.csv: split merged row so "Bauhaus Geometric" (nextlevelbuilder#74) is searchable
- styles.csv: close missing quote in row 77 that shifted trailing columns
- laravel.csv: quote comma-containing code fields (rows 24, 45)
- astro.csv / angular.csv: restore missing column (rows 35, 28)
- cli/assets: sync data, scripts, templates from src/ (was stale)
- cli uninstall: narrow out 'all' when iterating detected platforms

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant