Skip to content

chore: Fix examples build (failing due to two deckgl versions) - #359

Merged
ilyabo merged 3 commits into
mainfrom
chore--Fix-examples-build
Feb 13, 2026
Merged

chore: Fix examples build (failing due to two deckgl versions)#359
ilyabo merged 3 commits into
mainfrom
chore--Fix-examples-build

Conversation

@ilyabo

@ilyabo ilyabo commented Feb 11, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • CI/CD

    • Build workflow now triggers only for changes in the examples directory and uses a CI-specific examples build step with increased parallelism, improving efficiency and throughput.

  • Chores

    • Updated TypeScript configs across multiple example projects to improve module resolution.
    • Updated Next.js example pages to force dynamic rendering behavior.

@coderabbitai

coderabbitai Bot commented Feb 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Adds a PR path filter to the examples build workflow, increases example build concurrency, adds TypeScript baseUrl/paths for deck.gl in multiple example tsconfigs, and marks two Next.js example pages as dynamic via export const dynamic = 'force-dynamic' (ai-nextjs also switches from a client dynamic import to a direct import).

Changes

Cohort / File(s) Summary
GitHub Workflow
\.github/workflows/build-examples.yml		 Adds on.pull_request.paths: - 'examples/**' and updates the "Build examples" step to run pnpm build:examples:ci.
Package scripts
package.json		 Increases build:examples:ci turbo concurrency from --concurrency=1 to --concurrency=3.
Next.js pages
examples/ai-nextjs/app/page.tsx, examples/nextjs/app/page.tsx		 Adds export const dynamic = 'force-dynamic'. ai-nextjs also replaces a client-side dynamic import/"use client" with a direct AppShell import.
deck.gl example tsconfigs
examples/deckgl-discuss/tsconfig.app.json, examples/deckgl-mosaic/tsconfig.app.json, examples/deckgl/tsconfig.app.json, examples/kepler/tsconfig.app.json		 Adds compilerOptions.baseUrl: "." and a paths mapping for @deck.gl/core/src/* pointing to multiple local/node_modules locations to adjust module/type resolution.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • fix: Avoid deckgl v8 type errors #351: Adds baseUrl and paths mappings for @deck.gl/core/src/* in TypeScript configs to fix deck.gl v8 type resolution (same pattern applied to example tsconfigs here).

Poem

🐰 I hopped through configs, scripts, and flow,
Marked pages dynamic so they always grow,
Paths for deck.gl I neatly wove,
Workflow tuned for examples to go,
A tiny rabbit patch — swift, and apropos. 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (100 files):

⚔️ .github/workflows/build-examples.yml (content)
⚔️ CHANGELOG.md (content)
⚔️ CONTRIBUTING.md (content)
⚔️ README.md (content)
⚔️ docs/.vitepress/config.ts (content)
⚔️ docs/overview.md (content)
⚔️ examples/ai-agent/src/components/MainView.tsx (content)
⚔️ examples/ai-core/src/components/MainView.tsx (content)
⚔️ examples/ai-nextjs/app/page.tsx (content)
⚔️ examples/ai-rag/src/components/MainView.tsx (content)
⚔️ examples/ai/src/components/MainView.tsx (content)
⚔️ examples/deckgl-discuss/tsconfig.app.json (content)
⚔️ examples/deckgl-mosaic/tsconfig.app.json (content)
⚔️ examples/deckgl/tsconfig.app.json (content)
⚔️ examples/kepler/tsconfig.app.json (content)
⚔️ examples/nextjs/app/page.tsx (content)
⚔️ lerna.json (content)
⚔️ package.json (content)
⚔️ packages/ai-config/CHANGELOG.md (content)
⚔️ packages/ai-config/package.json (content)
⚔️ packages/ai-core/CHANGELOG.md (content)
⚔️ packages/ai-core/package.json (content)
⚔️ packages/ai-core/src/AiSlice.ts (content)
⚔️ packages/ai-core/src/chatTransport.ts (content)
⚔️ packages/ai-core/src/components/Chat.tsx (content)
⚔️ packages/ai-core/src/components/QueryControls.tsx (content)
⚔️ packages/ai-core/src/types.ts (content)
⚔️ packages/ai-rag/CHANGELOG.md (content)
⚔️ packages/ai-rag/package.json (content)
⚔️ packages/ai-settings/CHANGELOG.md (content)
⚔️ packages/ai-settings/package.json (content)
⚔️ packages/ai/CHANGELOG.md (content)
⚔️ packages/ai/package.json (content)
⚔️ packages/canvas/CHANGELOG.md (content)
⚔️ packages/canvas/package.json (content)
⚔️ packages/cosmos/CHANGELOG.md (content)
⚔️ packages/cosmos/package.json (content)
⚔️ packages/crdt/CHANGELOG.md (content)
⚔️ packages/crdt/package.json (content)
⚔️ packages/data-table/CHANGELOG.md (content)
⚔️ packages/data-table/package.json (content)
⚔️ packages/discuss/CHANGELOG.md (content)
⚔️ packages/discuss/package.json (content)
⚔️ packages/dropzone/CHANGELOG.md (content)
⚔️ packages/dropzone/package.json (content)
⚔️ packages/duckdb-core/CHANGELOG.md (content)
⚔️ packages/duckdb-core/package.json (content)
⚔️ packages/duckdb-node/CHANGELOG.md (content)
⚔️ packages/duckdb-node/package.json (content)
⚔️ packages/duckdb/CHANGELOG.md (content)
⚔️ packages/duckdb/package.json (content)
⚔️ packages/kepler-config/CHANGELOG.md (content)
⚔️ packages/kepler-config/package.json (content)
⚔️ packages/kepler/CHANGELOG.md (content)
⚔️ packages/kepler/package.json (content)
⚔️ packages/kepler/src/components/KeplerImageExport.tsx (content)
⚔️ packages/layout-config/CHANGELOG.md (content)
⚔️ packages/layout-config/package.json (content)
⚔️ packages/layout/CHANGELOG.md (content)
⚔️ packages/layout/package.json (content)
⚔️ packages/monaco-editor/CHANGELOG.md (content)
⚔️ packages/monaco-editor/package.json (content)
⚔️ packages/mosaic/CHANGELOG.md (content)
⚔️ packages/mosaic/package.json (content)
⚔️ packages/motherduck/CHANGELOG.md (content)
⚔️ packages/motherduck/package.json (content)
⚔️ packages/preset-eslint/CHANGELOG.md (content)
⚔️ packages/preset-eslint/package.json (content)
⚔️ packages/preset-jest/CHANGELOG.md (content)
⚔️ packages/preset-jest/package.json (content)
⚔️ packages/preset-typedoc/CHANGELOG.md (content)
⚔️ packages/preset-typedoc/package.json (content)
⚔️ packages/preset-typescript/CHANGELOG.md (content)
⚔️ packages/preset-typescript/package.json (content)
⚔️ packages/recharts/CHANGELOG.md (content)
⚔️ packages/recharts/package.json (content)
⚔️ packages/room-config/CHANGELOG.md (content)
⚔️ packages/room-config/package.json (content)
⚔️ packages/room-shell/CHANGELOG.md (content)
⚔️ packages/room-shell/package.json (content)
⚔️ packages/room-store/CHANGELOG.md (content)
⚔️ packages/room-store/package.json (content)
⚔️ packages/s3-browser-config/CHANGELOG.md (content)
⚔️ packages/s3-browser-config/package.json (content)
⚔️ packages/s3-browser/CHANGELOG.md (content)
⚔️ packages/s3-browser/package.json (content)
⚔️ packages/s3-utils/CHANGELOG.md (content)
⚔️ packages/s3-utils/package.json (content)
⚔️ packages/schema-tree/CHANGELOG.md (content)
⚔️ packages/schema-tree/package.json (content)
⚔️ packages/sql-editor-config/CHANGELOG.md (content)
⚔️ packages/sql-editor-config/package.json (content)
⚔️ packages/sql-editor/CHANGELOG.md (content)
⚔️ packages/sql-editor/package.json (content)
⚔️ packages/ui/CHANGELOG.md (content)
⚔️ packages/ui/package.json (content)
⚔️ packages/utils/CHANGELOG.md (content)
⚔️ packages/utils/package.json (content)
⚔️ packages/vega/CHANGELOG.md (content)
⚔️ packages/vega/package.json (content)

These conflicts must be resolved before merging into main.		 Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main objective of the PR: fixing the examples build that was failing due to multiple deckgl versions. The changes across multiple files (tsconfig updates, workflow adjustments, and build script modifications) all work toward resolving this specific issue.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore--Fix-examples-build
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch chore--Fix-examples-build
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ilyabo
fix examples build
07072d4 
Signed-off-by: Ilya Boyandin <[email protected]>
@ilyabo
using build:examples:ci
e9ebf2f 
Signed-off-by: Ilya Boyandin <[email protected]>
@ilyabo
ilyabo merged commit db1263e into main Feb 13, 2026
5 checks passed
@ilyabo
ilyabo deleted the chore--Fix-examples-build branch February 13, 2026 16:50
@coderabbitai coderabbitai Bot mentioned this pull request Apr 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

See More

Successfully merging this pull request may close these issues.

1 participant

@ilyabo