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
7 changes: 5 additions & 2 deletions js/.husky/pre-commit
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export GIT_WORK_TREE="$REPO_ROOT"
# ALLOW_BIOME_BUMP=1 git commit ...
PINNED_PKG_JSON="js/package.json"
if git diff --cached --name-only --diff-filter=ACM | grep -qx "$PINNED_PKG_JSON"; then
# Pinned: @biomejs/biome → 2.5.7.
# Pinned: @biomejs/biome → 2.5.12.
#
# Rationale: biome 2.4.12 through 2.4.16 have a deterministic Rust stack
# overflow on macOS arm64 when running TS-aware nursery rules
Expand All @@ -26,6 +26,9 @@ if git diff --cached --name-only --diff-filter=ACM | grep -qx "$PINNED_PKG_JSON"
# 2.4.16 locally on 2026-05-28 and CONFIRMED it still crashes — see
# DataRecce/recce-cloud-infra#1382 for the testing notes.
#
# 2.5.12 was verified with the full 705-file source check on macOS arm64
# during the September 2026 Dependabot consolidation, with no stack overflow.
#
# 2.5.7 is ABOVE the affected 2.4.12–2.4.16 range and verified clean on
# macOS arm64 (`biome check --staged` + a 639-file source check exit 0, no
# stack overflow). Bumped 2.5.5 → 2.5.7 in the 2026-08 dependency refresh
Expand All @@ -35,7 +38,7 @@ if git diff --cached --name-only --diff-filter=ACM | grep -qx "$PINNED_PKG_JSON"
# currently enable those nursery rules either. A silent dependabot bump
# still requires deliberate ack: when changing biome, update EXPECTED_BIOME
# in the same commit as the bump (and re-run the macOS arm64 check above).
EXPECTED_BIOME="2.5.7"
EXPECTED_BIOME="2.5.12"
# head -1 hardens against a future pnpm.overrides entry adding a second
# "@biomejs/biome": line. Fail-loud (below) if extraction is empty so a
# package.json reformat (or removed dep) can't silently disable the guard.
Expand Down
2 changes: 1 addition & 1 deletion js/biome.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"$schema": "https://biomejs.dev/schemas/2.5.5/schema.json",
"$schema": "https://biomejs.dev/schemas/2.5.12/schema.json",
"vcs": {
"enabled": true,
"clientKind": "git",
Expand Down
56 changes: 28 additions & 28 deletions js/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,47 +25,47 @@
"prepare": "cd .. && husky js/.husky"
},
"dependencies": {
"@amplitude/unified": "1.1.28",
"@amplitude/unified": "1.1.32",
"@babel/helpers": "^8.0.0",
"@babel/runtime": "^8.0.0",
"@codemirror/lang-sql": "^6.10.0",
"@codemirror/lang-yaml": "^6.1.3",
"@codemirror/merge": "^6.12.2",
"@codemirror/state": "^6.7.1",
"@codemirror/view": "^6.43.8",
"@codemirror/state": "^6.7.4",
"@codemirror/view": "^6.43.11",
"@dagrejs/dagre": "3.1.0",
"@datarecce/ui": "workspace:*",
"@emotion/react": "^11.14.0",
"@emotion/styled": "^11.14.1",
"@fontsource/montserrat": "^5.3.0",
"@hello-pangea/dnd": "^18.0.1",
"@jeromefitz/date-fns-tz": "3.0.0-canary.8",
"@mui/material": "^9.3.1",
"@mui/system": "^9.3.0",
"@next/third-parties": "16.3.0",
"@sentry/nextjs": "^10.70.0",
"@sentry/react": "^10.70.0",
"@mui/material": "^9.4.0",
"@mui/system": "^9.4.0",
"@next/third-parties": "16.3.4",
"@sentry/nextjs": "^10.73.0",
"@sentry/react": "^10.73.0",
"@tailwindcss/postcss": "^4.3.3",
"@tanstack/react-query": "5.101.4",
"@tanstack/react-query": "5.102.8",
"@types/js-cookie": "^3.0.6",
"@uiw/codemirror-theme-github": "^4.25.11",
"@uiw/react-codemirror": "^4.25.11",
"@xyflow/react": "^12.11.2",
"@zumer/snapdom": "^2.24.1",
"@xyflow/react": "^12.11.6",
"@zumer/snapdom": "^2.24.15",
"ag-grid-community": "^36.1.0",
"ag-grid-react": "^36.1.0",
"chart.js": "^4.5.1",
"codemirror": "^6.0.2",
"common-tags": "^1.8.2",
"date-fns": "^4.4.0",
"file-saver": "^2.0.5",
"happy-dom": "^20.11.2",
"import-in-the-middle": "^3.3.3",
"happy-dom": "^20.14.0",
"import-in-the-middle": "^3.4.0",
"js-cookie": "^3.0.8",
"lodash": "^4.18.1",
"next": "16.3.0",
"next": "16.3.4",
"next-themes": "^0.4.6",
"posthog-js": "^1.415.1",
"posthog-js": "^1.428.3",
"react": "19.2.8",
"react-chartjs-2": "^5.3.1",
"react-dom": "19.2.8",
Expand All @@ -82,32 +82,32 @@
"yaml": "^2.9.0"
},
"devDependencies": {
"@amplitude/analytics-core": "^2.54.1",
"@biomejs/biome": "2.5.7",
"@amplitude/analytics-core": "^2.55.0",
"@biomejs/biome": "2.5.12",
"@testing-library/dom": "^10.4.1",
"@testing-library/jest-dom": "7.0.1",
"@testing-library/react": "16.3.2",
"@testing-library/user-event": "14.6.3",
"@testing-library/react": "16.3.3",
"@testing-library/user-event": "14.6.7",
"@types/common-tags": "^1.8.4",
"@types/file-saver": "^2.0.7",
"@types/lodash": "^4.17.25",
"@types/node": "^26.2.0",
"@types/node": "^26.5.0",
"@types/react": "19.2.18",
"@types/react-dom": "19.2.4",
"@types/react-dom": "19.2.7",
"@types/react-syntax-highlighter": "^15.5.13",
"@typescript/native": "npm:typescript@^7.0.2",
"@vitejs/plugin-react": "^6.0.5",
"@vitest/coverage-v8": "^4.1.10",
"baseline-browser-mapping": "^2.11.13",
"@vitejs/plugin-react": "^6.1.1",
"@vitest/coverage-v8": "^4.1.11",
"baseline-browser-mapping": "^2.11.21",
"fast-check": "^4.9.0",
"globals": "^17.9.0",
"globals": "^17.12.0",
"husky": "^9.1.7",
"postcss": "^8.5.26",
"postcss": "^8.5.28",
"read-excel-file": "^9.3.10",
"rimraf": "^6.1.3",
"tailwindcss": "^4.3.3",
"typescript": "npm:@typescript/typescript6@^6.0.2",
"vite": "^8.2.1",
"vitest": "^4.1.10"
"vite": "^8.2.2",
"vitest": "^4.1.11"
}
}
26 changes: 13 additions & 13 deletions js/packages/storybook/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,23 +14,23 @@
},
"dependencies": {
"@datarecce/ui": "workspace:*",
"@tanstack/react-query": "5.101.4"
"@tanstack/react-query": "5.102.8"
},
"devDependencies": {
"@playwright/test": "^1.62.1",
"@storybook/addon-docs": "^10.5.7",
"@storybook/addon-vitest": "^10.5.7",
"@storybook/react": "^10.5.7",
"@storybook/react-vite": "^10.5.7",
"@playwright/test": "^1.63.0",
"@storybook/addon-docs": "^10.6.0",
"@storybook/addon-vitest": "^10.6.0",
"@storybook/react": "^10.6.0",
"@storybook/react-vite": "^10.6.0",
"@testing-library/jest-dom": "7.0.1",
"@testing-library/react": "16.3.2",
"@vitejs/plugin-react": "^6.0.5",
"@vitest/browser-playwright": "^4.1.10",
"@vitest/coverage-v8": "^4.1.10",
"@testing-library/react": "16.3.3",
"@vitejs/plugin-react": "^6.1.1",
"@vitest/browser-playwright": "^4.1.11",
"@vitest/coverage-v8": "^4.1.11",
"msw": "^2.15.0",
"playwright": "^1.62.1",
"storybook": "^10.5.7",
"vitest": "^4.1.10"
"playwright": "^1.63.0",
"storybook": "^10.6.0",
"vitest": "^4.1.11"
},
"peerDependencies": {
"react": "^18.0.0 || ^19.0.0",
Expand Down
28 changes: 14 additions & 14 deletions js/packages/ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -136,24 +136,24 @@
}
},
"dependencies": {
"@amplitude/analytics-core": "^2.54.1",
"@amplitude/unified": "1.1.28",
"@amplitude/analytics-core": "^2.55.0",
"@amplitude/unified": "1.1.32",
"@codemirror/lang-sql": "^6.10.0",
"@codemirror/lang-yaml": "^6.1.0",
"@codemirror/merge": "^6.12.0",
"@codemirror/state": "^6.7.1",
"@codemirror/view": "^6.43.7",
"@codemirror/state": "^6.7.4",
"@codemirror/view": "^6.43.11",
"@dagrejs/dagre": "3.1.0",
"@emotion/react": "^11.14.0",
"@emotion/styled": "^11.14.0",
"@hello-pangea/dnd": "^18.0.0",
"@mui/material": "^9.0.0",
"@mui/system": "^9.0.0",
"@tanstack/react-query": "^5.0.0",
"@mui/material": "^9.4.0",
"@mui/system": "^9.4.0",
"@tanstack/react-query": "^5.102.8",
"@uiw/codemirror-theme-github": "^4.25.11",
"@uiw/react-codemirror": "^4.25.11",
"@xyflow/react": "^12.10.1",
"@zumer/snapdom": "^2.23.2",
"@xyflow/react": "^12.11.6",
"@zumer/snapdom": "^2.24.15",
"ag-grid-community": "^36.0.1",
"ag-grid-react": "^36.0.1",
"chart.js": "^4.5.0",
Expand All @@ -164,7 +164,7 @@
"js-cookie": "^3.0.7",
"lodash": "^4.17.21",
"next-themes": "^0.4.0",
"posthog-js": "^1.410.9",
"posthog-js": "^1.428.3",
"react-chartjs-2": "^5.3.0",
"react-icons": "^5.7.0",
"react-markdown": "^10.1.0",
Expand All @@ -178,14 +178,14 @@
"devDependencies": {
"@tailwindcss/cli": "^4.3.3",
"@tailwindcss/postcss": "^4.3.3",
"@tsdown/css": "^0.22.14",
"@tsdown/css": "^0.23.0",
"@typescript/native": "npm:typescript@^7.0.2",
"postcss": "^8.5.26",
"postcss": "^8.5.28",
"read-excel-file": "^9.3.10",
"tailwindcss": "^4.3.3",
"tsdown": "^0.22.14",
"tsdown": "^0.23.0",
"typedoc": "^0.28.20",
"typedoc-plugin-markdown": "^4.11.0",
"typedoc-plugin-markdown": "^4.13.0",
"typescript": "npm:@typescript/typescript6@^6.0.2"
},
"scripts": {
Expand Down
3 changes: 1 addition & 2 deletions js/packages/ui/src/components/run/RunView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,7 @@ export interface RunViewProps {
*/
RunResultView?: ForwardRefExoticComponent<
// biome-ignore lint/suspicious/noExplicitAny: RunResultView types vary by run type
RunResultViewProps<any> & RefAttributes<any>
>;
RunResultViewProps<any> & RefAttributes<any>>;

/**
* Render prop for custom result rendering.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,13 +94,14 @@ describe("defaultRenderCell - Basic Rendering", () => {
test("renders boolean false without gray styling", () => {
const colDef: ColDefWithMetadata = { field: "active" };
const params = createParams({ active: false }, colDef);
const textColor = "rgb(12 34 56)";

render(<>{defaultRenderCell(params)}</>);
render(<div style={{ color: textColor }}>{defaultRenderCell(params)}</div>);

const element = screen.getByText("false");
expect(element).toBeInTheDocument();
// Boolean false does not get grayOut styling per toRenderedValue implementation
expect(element).toHaveStyle({ color: "inherit" });
// Boolean false inherits its parent's color; only null/empty values turn gray.
expect(element).toHaveStyle({ color: textColor });
});
});

Expand Down
Loading
Loading