chore: remove unused dependencies
Merged
DT
4d ago+36+36+01234567890123456789-51-51-01234567890123456789
Removes two runtime dependencies and updates the pinned SDK.
- Drops
zod-to-json-schema, which is declared but never imported insrc. - Drops
string-width. Its only use was sizing the update-notice box insrc/utils/update-notice-format.ts, which now measures terminal columns withnode:util'sstripVTControlCharactersandIntl.Segmenter. Emoji and CJK count as two columns and combining marks as zero. The existing alignment test (emoji, color codes, a combining accent and CJK) passes unchanged. - Bumps
firecrawlfrom 4.50.0 to 4.51.0.
Installed tree from npm pack plus npm install --omit=dev: 65 packages before, 60 after.
pnpm exec tsc --noEmit and the prettier check pass. The full vitest run has the same 60 failures (alexandria-beta, commands/parse and scrape-shorthand) on main and on this branch in this environment, and none in the update-notice tests.
Summary by cubic
Removes two unused runtime dependencies and updates the pinned firecrawl.
- Drops
zod-to-json-schema, which is declared but never imported insrc. - Drops
string-width; the update-notice box insrc/utils/update-notice-format.tsnow measures terminal columns withnode:util'sstripVTControlCharactersandIntl.Segmenter. Emoji, CJK, and fullwidth characters count as two columns, combining marks as zero, and the alignment test passes unchanged. - Bumps
firecrawlfrom 4.50.0 to 4.51.0.
The full vitest run shows the same 60 pre-existing failures (alexandria-beta, commands/parse, scrape-shorthand) as on main; none are in the update-notice tests.