Skip to content

Feat/monorepo react sdk#7

Merged
felipefdl merged 27 commits intomasterfrom
feat/monorepo-react-sdk
Apr 11, 2026
Merged

Feat/monorepo react sdk#7
felipefdl merged 27 commits intomasterfrom
feat/monorepo-react-sdk

Conversation

@vitorfdl
Copy link
Copy Markdown
Member

No description provided.

Remove 7 packages (eslint, prettier, eslint-config-prettier,
eslint-plugin-import, eslint-plugin-prettier, @typescript-eslint/eslint-plugin,
@typescript-eslint/parser) and their config files (.eslintrc.js, .eslintignore,
.prettierrc).

Add @biomejs/biome with biome.json configured to match existing formatting
preferences (double quotes, 120 line width, trailing commas es5, 2-space indent).

Fix pre-existing lint issues caught by Biome:
- Replace `| void` with `| undefined` in union return types (noConfusingVoidType)
- Replace `.map()` with `.forEach()` where return value is unused
- Sort imports alphabetically
Clean tsconfig.json: remove suppressImplicitAnyIndexErrors (deprecated in
TS 5.0), remove all commented-out options, fix typeRoots (was using glob
instead of directory), add isolatedModules, update module to ES2020 and
lib to ES2022.

Fix type errors surfaced by removing suppressImplicitAnyIndexErrors:
- Change pool from Array to Record<string, PoolCallback | null>
- Add proper type assertions at pool call sites where TEventData is
  passed as TData/TError (pre-existing type mismatch in the protocol)
Remove shortid and @types/shortid dependencies. Add local generateId()
helper that uses crypto.randomUUID() with a fallback for insecure contexts.

Update test mocks to use Object.defineProperty on globalThis.crypto
instead of vi.mock("shortid"). Remove duplicate sendMessage test block.
Remove @vitest/coverage-c8 and c8 (deprecated). Add @vitest/coverage-v8.
Upgrade jsdom from 20 to 25. Update vite.config.ts with v8 coverage
provider configuration.

Fix un-awaited .resolves assertions that would fail in future Vitest
versions (add async/await to promise-based test cases).
Fix hardcoded ts-loader path in webpack.config.js to use bare module
name, which resolves correctly in both npm and pnpm monorepo layouts.

Add .gitignore entries for pnpm (.pnpm-store), Turborepo (.turbo),
and Biome (.biome) in preparation for monorepo restructuring.
Minor/patch upgrades:
- clean-css-cli: 5.6.1 -> 5.6.3
- less: 4.1.3 -> 4.6.4
- ts-loader: 9.4.1 -> 9.5.4
- webpack: 5.74.0 -> 5.105.4

Major upgrades:
- typescript: 5.7.3 -> 5.9.3 (strictInference now enabled under strict)
- webpack-cli: 4.10.0 -> 7.0.2
- vitest: 3.2.4 -> 4.1.0 (AST-based v8 coverage remapping)
- @vitest/coverage-v8: 3.2.4 -> 4.1.0
- jsdom: 25.0.1 -> 29.0.0

New dependency:
- vite: 6.4.1 (required as peer dep by Vitest 4)

Fix TS 5.9 type error: replace `global.console` with `globalThis.console`
in test file (global is not defined in DOM lib).
Comment thread packages/js/src/custom-widget.ts Fixed
Comment thread packages/js/src/custom-widget.ts Fixed
vitorfdl and others added 5 commits March 19, 2026 08:32
Consolidate build configuration with shared vite.config.ts, update
dependencies across all packages, migrate to pnpm workspaces, and
refactor JS package for 2.0.
…ealtime sorting

- Move auto-fill logic to React hooks (useSendData, useEditData) instead of core
- Fix deleteData to send string payload "id:device" matching dashboard expectations
- Add runAnalysis to core, JS SDK, and React hook (useRunAnalysis)
- Fix mergeStrategy to remove deleted records and sort by time desc
- Add customPreferences and preferences to TUserInformation
- Support bridge recreation for React StrictMode remount
- Add tests for runAnalysis, delete records, and user preferences
@vitorfdl vitorfdl self-assigned this Apr 9, 2026
GitHub Pages deploy workflow for backward-compatible dist URLs
feat: monorepo v2 refactor with unified build config
@felipefdl felipefdl merged commit ad01ca3 into master Apr 11, 2026
1 of 3 checks passed
@felipefdl felipefdl deleted the feat/monorepo-react-sdk branch April 11, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants