Skip to content

Commit d6db150

Browse files
committed
Update dagre 1.1.8 => 2.0.4, fix CJS/ESM build and dev issues
1 parent d35216e commit d6db150

5 files changed

Lines changed: 59 additions & 55 deletions

File tree

.changeset/purple-things-count.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'layerchart': patch
3+
---
4+
5+
chore: Update deps (including dagre from 1.x to 2.x)

docs/vite.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export default defineConfig({
1313
// which causes "Failed to fetch dynamically imported module" errors
1414
// Ref: https://github.com/vitest-dev/vitest/issues/5477
1515
optimizeDeps: {
16-
include: ['@layerstack/tailwind', '@layerstack/utils', 'svelte-ux', '@dagrejs/dagre']
16+
include: ['@layerstack/tailwind', '@layerstack/utils', 'svelte-ux']
1717
},
1818
plugins: [
1919
tailwindcss(),
@@ -98,6 +98,6 @@ export default defineConfig({
9898
noExternal: isTest ? undefined : true,
9999
// CJS-only packages that fail with "module is not defined" in Vite's ESM
100100
// module runner. Externalizing lets Node.js load them natively as CJS.
101-
external: ['satori', '@resvg/resvg-js', '@dagrejs/dagre']
101+
external: ['satori', '@resvg/resvg-js']
102102
}
103103
});

packages/layerchart/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
},
6161
"type": "module",
6262
"dependencies": {
63-
"@dagrejs/dagre": "^1.1.8",
63+
"@dagrejs/dagre": "^2.0.4",
6464
"@layerstack/svelte-actions": "1.0.1-next.18",
6565
"@layerstack/svelte-state": "0.1.0-next.23",
6666
"@layerstack/tailwind": "2.0.0-next.21",

packages/layerchart/vite.config.js

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,11 @@ const config = defineConfig({
2525
'd3-quadtree',
2626
'memoize',
2727
'runed',
28-
'@dagrejs/dagre',
2928
],
3029
},
3130
plugins: [sveltekit()],
3231
ssr: {
3332
noExternal: true, // https://github.com/AdrianGonz97/refined-cf-pages-action/issues/26#issuecomment-2878397440
34-
// @dagrejs/dagre is CJS-only; pre-bundle for SSR to convert to ESM
35-
optimizeDeps: {
36-
include: ['@dagrejs/dagre'],
37-
},
3833
},
3934
test: {
4035
projects: [

pnpm-lock.yaml

Lines changed: 51 additions & 47 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)