refactor: extract svelteplot library into packages/svelteplot (monorepo) by gka · Pull Request #551 · svelteplot/svelteplot

gka · 2026-04-24T20:55:23Z

Summary

Moves src/lib/ → packages/svelteplot/src/lib/ (the published library), keeping src/lib/helpers/data.ts in the docs app (docs-only helper)
Moves src/tests/ → packages/svelteplot/src/tests/ (tests belong with the library they test)
Adds packages/svelteplot/package.json with library metadata, runtime deps, and build config
Adds packages/svelteplot/vite.config.js with a regex-based alias so tests resolve svelteplot/* subpath imports from source
Updates root package.json: removes library fields, adds svelteplot: workspace:* dependency, delegates pnpm test to the package
Updates vite.config.js and svelte.config.js aliases to point at packages/svelteplot/src/lib — dev HMR continues to work without a build step
Fixes root .gitignore patterns (/.svelte-kit → .svelte-kit, /dist → dist) to cover subdirectories
Removes duplicate htmlTooltip.test.svelte files that caused a case-sensitivity collision on macOS

Test plan

All 96 test files / 835 tests pass (pnpm test)
Dev server starts and docs render correctly (pnpm dev)
pnpm prepack in packages/svelteplot builds dist/ correctly

🤖 Generated with Claude Code

- Move src/lib/ → packages/svelteplot/src/lib/ (library source) - Move src/tests/ → packages/svelteplot/src/tests/ (library tests) - Keep src/lib/helpers/data.ts in docs app (docs-only helper) - Add packages/svelteplot/package.json with library metadata and deps - Add packages/svelteplot/vite.config.js for running tests in package context - Add packages/svelteplot/svelte.config.js and tsconfig.json - Move scripts/fix-js-imports.js to packages/svelteplot/scripts/ - Update root package.json: remove library fields, add svelteplot workspace dep - Update vite.config.js and svelte.config.js aliases to point at packages/svelteplot/src/lib - Update root pnpm test script to delegate to package Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The capital-T and lowercase-t variants are the same file on case-insensitive filesystems. Keep only the lowercase version which has the correct imports. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Remove leading slashes from .svelte-kit and dist so they match in packages/svelteplot/ as well as the repo root. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

netlify · 2026-04-24T20:55:28Z

✅ Deploy Preview for svelteplot ready!

Name	Link
🔨 Latest commit	`fde4b37`
🔍 Latest deploy log	https://app.netlify.com/projects/svelteplot/deploys/69ebec30f6c593000895d4f3
😎 Deploy Preview	https://deploy-preview-551--svelteplot.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bc2612a4a6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

d3-array, d3-geo, d3-random, topojson-client and topojson types were moved to packages/svelteplot but are also needed by docs source files that import these packages directly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Run version bump and npm publish from packages/svelteplot/ since that is now the publishable package. Read version from packages/svelteplot/package.json. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

github-actions · 2026-04-24T21:20:31Z

📦 Preview package for this PR is published!

Version: 0.14.2-pr-551.0

Install it with:

npm install svelteplot@pr-551
# or install the specific version
npm install svelteplot@0.14.2-pr-551.0

… package root Removes the SvelteKit-only src/lib/ nesting convention from the library package — library source now lives directly in packages/svelteplot/src/, and tests move from src/tests/ to packages/svelteplot/tests/. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

github-actions · 2026-04-24T21:47:16Z

📦 Preview package for this PR is published!

Version: 0.14.2-pr-551.1

Install it with:

npm install svelteplot@pr-551
# or install the specific version
npm install svelteplot@0.14.2-pr-551.1

github-actions · 2026-04-24T21:53:40Z

📦 Preview package for this PR is published!

Version: 0.14.2-pr-551.2

Install it with:

npm install svelteplot@pr-551
# or install the specific version
npm install svelteplot@0.14.2-pr-551.2

The REPL environment can't resolve package-name imports within the package itself — they need to be relative. Also removes leftover debug console.log. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

github-actions · 2026-04-24T22:20:22Z

📦 Preview package for this PR is published!

Version: 0.14.2-pr-551.3

Install it with:

npm install svelteplot@pr-551
# or install the specific version
npm install svelteplot@0.14.2-pr-551.3

…558) Adds a `CONTRIBUTING.md` at the repo root following the convention used by Vega-Lite, Svelte, and shadcn/ui — single root file covering project layout, adding a new mark, adding a new transform, and per-PR expectations. Also fixes `scripts/generate-api.js`, which had stale `src/lib/...` paths from before the monorepo refactor (#551). `pnpm docs:api:marks` (referenced from `CONTRIBUTING.md`) won't work without this fix. Both pieces were originally bundled with #552 but they're project-wide tooling, not part of the hexbin feature, so splitting them out so they can land independently. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

gka and others added 3 commits April 24, 2026 16:53

fix: remove duplicate htmlTooltip test files (case collision on macOS)

8976ad1

The capital-T and lowercase-t variants are the same file on case-insensitive filesystems. Keep only the lowercase version which has the correct imports. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: broaden .gitignore patterns to cover packages/ subdirectories

bc2612a

Remove leading slashes from .svelte-kit and dist so they match in packages/svelteplot/ as well as the repo root. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

gka had a problem deploying to npm April 24, 2026 20:56 — with GitHub Actions Failure

chatgpt-codex-connector Bot reviewed Apr 24, 2026

View reviewed changes

Comment thread svelte.config.js Outdated

reinstall missing packages, fix import

13c90c9

gka had a problem deploying to npm April 24, 2026 21:02 — with GitHub Actions Failure

allow imports

014bf80

gka had a problem deploying to npm April 24, 2026 21:06 — with GitHub Actions Failure

gka added 2 commits April 24, 2026 23:07

fix: add missing dev dep

0d34b94

remove generic agents.md

96d8db5

gka had a problem deploying to npm April 24, 2026 21:09 — with GitHub Actions Failure

gka had a problem deploying to npm April 24, 2026 21:14 — with GitHub Actions Failure

fix(ci): update npm-prerelease workflow for monorepo structure

97181af

Run version bump and npm publish from packages/svelteplot/ since that is now the publishable package. Read version from packages/svelteplot/package.json. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

gka temporarily deployed to npm April 24, 2026 21:19 — with GitHub Actions Inactive

gka temporarily deployed to npm April 24, 2026 21:46 — with GitHub Actions Inactive

fix imports

3b27fd2

gka temporarily deployed to npm April 24, 2026 21:53 — with GitHub Actions Inactive

fix(build): rewrite svelteplot/* self-imports to relative paths in dist

fde4b37

The REPL environment can't resolve package-name imports within the package itself — they need to be relative. Also removes leftover debug console.log. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

gka temporarily deployed to npm April 24, 2026 22:19 — with GitHub Actions Inactive

gka merged commit 3a1d520 into main Apr 24, 2026
8 checks passed

gka deleted the chore/monorepo branch April 24, 2026 22:22

gka mentioned this pull request Apr 24, 2026

feat: add hexbin transform and Hexgrid mark #552

Open

5 tasks

RobertFrenken mentioned this pull request Apr 27, 2026

chore: add CONTRIBUTING.md and fix monorepo paths in generate-api.js #558

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

refactor: extract svelteplot library into packages/svelteplot (monorepo)#551

refactor: extract svelteplot library into packages/svelteplot (monorepo)#551
gka merged 12 commits into
mainfrom
chore/monorepo

gka commented Apr 24, 2026

Uh oh!

netlify Bot commented Apr 24, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

gka commented Apr 24, 2026

Summary

Test plan

Uh oh!

netlify Bot commented Apr 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for svelteplot ready!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

netlify Bot commented Apr 24, 2026 •

edited

Loading