Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 19 additions & 19 deletions js/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,14 @@
"prepare": "cd .. && husky js/.husky"
},
"dependencies": {
"@amplitude/unified": "1.0.5",
"@amplitude/unified": "1.0.15",
"@babel/helpers": "^7.29.2",
"@babel/runtime": "^7.29.2",
"@codemirror/lang-sql": "^6.10.0",
"@codemirror/lang-yaml": "^6.1.3",
"@codemirror/merge": "^6.12.1",
"@codemirror/state": "^6.6.0",
"@codemirror/view": "^6.40.0",
"@codemirror/view": "^6.41.0",
"@dagrejs/dagre": "1.1.8",
"@datarecce/ui": "workspace:*",
"@emotion/react": "^11.14.0",
Expand All @@ -41,15 +41,15 @@
"@jeromefitz/date-fns-tz": "3.0.0-canary.8",
"@mui/material": "^7.3.9",
"@mui/system": "^7.3.9",
"@next/third-parties": "16.2.1",
"@sentry/nextjs": "^10.46.0",
"@sentry/react": "^10.46.0",
"@next/third-parties": "16.2.2",
"@sentry/nextjs": "^10.47.0",
"@sentry/react": "^10.47.0",
"@tailwindcss/postcss": "^4.2.2",
"@tanstack/react-query": "5.95.2",
"@tanstack/react-query": "5.96.1",
"@types/js-cookie": "^3.0.6",
"@uiw/codemirror-theme-github": "^4.25.9",
"@uiw/react-codemirror": "^4.25.9",
"@xyflow/react": "^12.10.1",
"@xyflow/react": "^12.10.2",
"ag-grid-community": "^35.2.0",
"ag-grid-react": "^35.2.0",
"chart.js": "^4.5.1",
Expand All @@ -62,8 +62,8 @@
"html2canvas-pro": "1.6.6",
"import-in-the-middle": "^3.0.0",
"js-cookie": "^3.0.5",
"lodash": "^4.17.23",
"next": "16.2.1",
"lodash": "^4.18.1",
"next": "16.2.2",
"next-themes": "^0.4.6",
"react": "19.2.4",
"react-chartjs-2": "^5.3.1",
Expand All @@ -77,12 +77,12 @@
"simplebar": "^6.3.3",
"simplebar-react": "3.3.2",
"usehooks-ts": "^3.1.1",
"write-excel-file": "^3.0.6",
"write-excel-file": "^3.0.8",
"yaml": "^2.8.3"
},
"devDependencies": {
"@amplitude/analytics-core": "^2.44.0",
"@biomejs/biome": "2.4.9",
"@amplitude/analytics-core": "^2.44.1",
"@biomejs/biome": "2.4.10",
"@testing-library/dom": "^10.4.1",
"@testing-library/jest-dom": "6.9.1",
"@testing-library/react": "16.3.2",
Expand All @@ -97,12 +97,12 @@
"@types/react-syntax-highlighter": "^15.5.13",
"@vitejs/plugin-react": "^5.1.4",
"@vitest/coverage-v8": "^4.1.2",
"baseline-browser-mapping": "^2.10.11",
"baseline-browser-mapping": "^2.10.13",
"fast-check": "^4.6.0",
"globals": "^17.4.0",
"husky": "^9.1.7",
"postcss": "^8.5.8",
"read-excel-file": "^7.0.3",
"read-excel-file": "^8.0.3",
"rimraf": "^6.1.3",
"tailwindcss": "^4.2.2",
"typescript": "^5.9.3",
Expand All @@ -116,24 +116,24 @@
"brace-expansion@>=2.0.0 <=2.0.1": ">=2.0.2",
"js-yaml@<3.14.2": ">=3.14.2",
"glob@>=10.2.0 <10.5.0": ">=10.5.0",
"@tanstack/react-query": "5.95.2",
"@tanstack/react-query": "5.96.1",
"@types/react": "19.2.14",
"@types/react-dom": "19.2.3",
"@vitejs/plugin-react": "^5.1.4",
"@biomejs/biome": "2.4.9",
"@biomejs/biome": "2.4.10",
"@testing-library/jest-dom": "6.9.1",
"@testing-library/react": "16.3.2",
"happy-dom": "^20.8.9",
"@emotion/styled": "^11.14.1",
"@mui/material": "^7.3.9",
"@mui/system": "^7.3.9",
"@xyflow/react": "^12.10.1",
"@xyflow/react": "^12.10.2",
"@dagrejs/dagre": "1.1.8",
"html-to-image": "1.11.13",
"html2canvas-pro": "1.6.6",
"ag-grid-community": "^35.2.0",
"ag-grid-react": "^35.2.0",
"lodash": "^4.17.23",
"lodash": "^4.18.1",
"next-themes": "^0.4.6",
"typescript": "^5.9.3",
"tailwindcss": "^4.2.2",
Expand All @@ -146,7 +146,7 @@
"minimatch@>=9.0.0 <9.0.7": ">=9.0.7",
"minimatch@>=10.0.0 <10.2.3": ">=10.2.3",
"rollup@>=4.0.0 <4.59.0": ">=4.59.0",
"@amplitude/unified": "1.0.5"
"@amplitude/unified": "1.0.15"
},
Comment on lines 146 to 150
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pnpm.overrides forces @amplitude/unified to 1.0.15 across the monorepo, but @datarecce/ui declares a pinned dependency of @amplitude/unified at 1.0.1 (and the dependency contract doc calls it “Pinned. Exact version required”). This means CI/dev is no longer exercising the version that consumers of the published @datarecce/ui package will actually install, which can hide compatibility issues. Consider either updating js/packages/ui/package.json (and DEPENDENCIES.md) to the new pinned version, or removing the override / aligning root versions so the tested version matches the published contract.

Copilot uses AI. Check for mistakes.
"onlyBuiltDependencies": [
"@sentry/cli",
Expand Down
14 changes: 7 additions & 7 deletions js/packages/storybook/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,20 +15,20 @@
"@datarecce/ui": "workspace:*"
},
"devDependencies": {
"@playwright/test": "^1.58.2",
"@storybook/addon-docs": "^10.3.3",
"@storybook/addon-vitest": "^10.3.3",
"@storybook/react": "^10.3.3",
"@storybook/react-vite": "^10.3.3",
"@playwright/test": "^1.59.1",
"@storybook/addon-docs": "^10.3.4",
"@storybook/addon-vitest": "^10.3.4",
"@storybook/react": "^10.3.4",
"@storybook/react-vite": "^10.3.4",
"@testing-library/jest-dom": "^6.9.1",
"@testing-library/react": "^16.3.2",
"@vitejs/plugin-react": "^5.1.4",
"@vitest/browser-playwright": "^4.1.2",
"@vitest/coverage-v8": "^4.1.2",
"chartjs-adapter-date-fns": "^3.0.0",
"msw": "^2.12.14",
"playwright": "^1.58.2",
"storybook": "^10.3.3",
"playwright": "^1.59.1",
"storybook": "^10.3.4",
"vitest": "^4.1.2"
},
"peerDependencies": {
Expand Down
6 changes: 3 additions & 3 deletions js/packages/ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -170,11 +170,11 @@
"devDependencies": {
"@tailwindcss/cli": "^4.2.2",
"@tailwindcss/postcss": "^4.2.2",
"@tsdown/css": "^0.21.5",
"@tsdown/css": "^0.21.7",
"postcss": "^8.5.0",
"read-excel-file": "^7.0.3",
"read-excel-file": "^8.0.3",
"tailwindcss": "^4.2.2",
"tsdown": "^0.21.5",
"tsdown": "^0.21.7",
"typedoc": "^0.28.18",
"typedoc-plugin-markdown": "^4.11.0",
"typescript": "^5.9.0"
Expand Down
11 changes: 6 additions & 5 deletions js/packages/ui/src/utils/csv/__tests__/excel.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,16 @@ async function parseExcelBlob(blob: Blob): Promise<{
columns: string[];
rows: unknown[][];
}> {
// read-excel-file accepts Blob in browser environments
const jsonData = await readXlsxFile(blob);
// read-excel-file v8 returns sheet objects: { sheet, data }[]
const sheets = await readXlsxFile(blob);
const sheetData = sheets[0]?.data ?? [];

if (jsonData.length === 0) {
if (sheetData.length === 0) {
return { columns: [], rows: [] };
}

const columns = (jsonData[0] as unknown[]).map(String);
const rows = jsonData.slice(1);
const columns = (sheetData[0] as unknown[]).map(String);
const rows = sheetData.slice(1) as unknown[][];

return { columns, rows };
}
Expand Down
Loading
Loading