Chore/20250723#195
Conversation
|
WalkthroughThis update upgrades the pnpm package manager version from 10.12.4 to 10.13.1 across all relevant GitHub Actions workflows and Changes
Estimated code review effort1 (~7 minutes) Possibly related PRs
Suggested labels
Poem
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #195 +/- ##
============================================
Coverage 93.55% 93.55%
============================================
Files 74 74
Lines 9043 9043
Branches 1141 1141
============================================
Hits 8460 8460
Misses 581 581
Partials 2 2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
fdm-app/package.json (2)
32-35: Duplicated workspace packages inflate lockfile
@svenvw/fdm-{calculator,core,data}are declared in bothdependenciesanddevDependencies. This forces pnpm to write two virtual links for the same workspace package and lengthens install time. Removing them fromdevDependencieskeeps runtime behaviour unchanged.Also applies to: 85-87
107-107: Remember to update local tooling after changingpackageManager
Bumping topnpm@10.13.1is fine, but advise contributors to runcorepack prepare pnpm@10.13.1 --activatelocally to avoid lockfile churn.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (11)
.github/workflows/deploy-docs-test.yml(1 hunks).github/workflows/deploy-docs.yml(1 hunks).github/workflows/release.yml(1 hunks).github/workflows/tests.yml(3 hunks)fdm-app/package.json(2 hunks)fdm-calculator/package.json(2 hunks)fdm-core/package.json(3 hunks)fdm-data/package.json(1 hunks)fdm-docs/package.json(3 hunks)package.json(2 hunks)pnpm-workspace.yaml(1 hunks)
🧠 Learnings (9)
📓 Common learnings
Learnt from: SvenVw
PR: SvenVw/fdm#147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax `"typescript": "catalog:"` is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: SvenVw/fdm#147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax `"typescript": "catalog:"` is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: SvenVw/fdm#134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
.github/workflows/deploy-docs.yml (2)
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
pnpm-workspace.yaml (11)
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The schema defined in fdm-core/src/db/schema-authn.ts follows better-auth's structure and requirements. While the schema is defined in the application code, modifications to it should maintain compatibility with better-auth's expectations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The organization schema in fdm-core/src/db/schema-authn.ts is managed by better-auth, and modifications to field constraints (like making the slug field non-nullable) should maintain compatibility with better-auth's expectations, even if application code assumes non-null values.
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #95
File: fdm-core/src/catalogues.ts:134-170
Timestamp: 2025-03-06T15:23:48.352Z
Learning: When writing tests for fdm-core, avoid using Vitest's vi mocking utilities and prefer manual JavaScript mocks.
Learnt from: SvenVw
PR: #95
File: fdm-core/src/cultivation.ts:67-73
Timestamp: 2025-03-06T14:58:48.603Z
Learning: When writing unit tests for the fdm project, avoid using Vitest's mocking utilities (vi) as it has caused problems in the past not related to the actual code. Instead, use simple object literals with methods that throw errors to test error handling.
fdm-data/package.json (3)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
fdm-calculator/package.json (8)
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:34:20.850Z
Learning: The updateField function in fdm-core has optional parameters that don't need to be passed as undefined. Only fdm and b_id are required.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
fdm-core/package.json (14)
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:34:20.850Z
Learning: The updateField function in fdm-core has optional parameters that don't need to be passed as undefined. Only fdm and b_id are required.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #88
File: fdm-core/src/cultivation.ts:246-246
Timestamp: 2025-03-04T11:09:08.169Z
Learning: In the FDM codebase, the fdm parameter should be documented as "The FDM instance providing the connection to the database. The instance can be created with {@link createFdmServer}." in JSDoc comments.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts will be updated in commit b17fac1 to include id and email fields, which are necessary for subsequent role updates and user removal operations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The organization schema in fdm-core/src/db/schema-authn.ts is managed by better-auth, and modifications to field constraints (like making the slug field non-nullable) should maintain compatibility with better-auth's expectations, even if application code assumes non-null values.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts needed to include id and email fields for users, which was fixed in commit b17fac1.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The schema defined in fdm-core/src/db/schema-authn.ts follows better-auth's structure and requirements. While the schema is defined in the application code, modifications to it should maintain compatibility with better-auth's expectations.
fdm-app/package.json (18)
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #45
File: fdm-app/app/routes/farm.$b_id_farm.settings._index.tsx:1-1
Timestamp: 2025-01-14T16:06:21.832Z
Learning: In the fdm project, redirect and other routing utilities should be imported from react-router instead of react-router-dom.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #45
File: fdm-app/app/routes/farm.$b_id_farm._index.tsx:1-1
Timestamp: 2025-01-14T16:06:24.294Z
Learning: In the fdm-app codebase, the redirect function should be imported from react-router, not react-router-dom.
Learnt from: SvenVw
PR: #124
File: fdm-app/app/components/custom/farm/farm-title.tsx:3-3
Timestamp: 2025-04-18T13:49:17.029Z
Learning: In the fdm project, NavLink and other routing components can be imported from either "react-router" or "react-router-dom" as react-router-dom is included in react-router.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts will be updated in commit b17fac1 to include id and email fields, which are necessary for subsequent role updates and user removal operations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts needed to include id and email fields for users, which was fixed in commit b17fac1.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/fertilizer-applications/form.tsx:6-6
Timestamp: 2025-05-09T14:41:43.484Z
Learning: The project uses React Router v7 which exports a Form component directly from the "react-router" package, making importing from "@remix-run/react" unnecessary.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/fertilizer-applications/form.tsx:6-6
Timestamp: 2025-05-09T14:41:43.484Z
Learning: The project uses React Router v7 which exports a Form component directly from the "react-router" package, not from "@remix-run/react".
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/combobox.tsx:34-37
Timestamp: 2025-05-09T14:53:44.578Z
Learning: In the context of this React Router v7 project, it's important to follow the pattern of importing only the types (like UseFormReturn) from "react-hook-form" while importing the Form component from "react-router" to avoid naming conflicts.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/combobox.tsx:34-37
Timestamp: 2025-05-09T14:58:10.465Z
Learning: When updating React components that use both react-hook-form and React Router v7, it's important to only import types (like UseFormReturn, FieldValues) from react-hook-form to avoid naming conflicts with React Router's Form component. Use import type { ... } from 'react-hook-form' syntax to ensure only types are imported.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #16
File: fdm-app/app/routes/app.addfarm.$b_id_farm.cultivations.$b_lu_catalogue.fertilizers.tsx:1-1
Timestamp: 2024-12-16T10:56:07.561Z
Learning: The project uses react-router v7, and the data function is exported and used for error handling in loaders and actions.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
fdm-docs/package.json (3)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
package.json (4)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
🧰 Additional context used
🧠 Learnings (9)
📓 Common learnings
Learnt from: SvenVw
PR: SvenVw/fdm#147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax `"typescript": "catalog:"` is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: SvenVw/fdm#147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax `"typescript": "catalog:"` is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: SvenVw/fdm#134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
.github/workflows/deploy-docs.yml (2)
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
pnpm-workspace.yaml (11)
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The schema defined in fdm-core/src/db/schema-authn.ts follows better-auth's structure and requirements. While the schema is defined in the application code, modifications to it should maintain compatibility with better-auth's expectations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The organization schema in fdm-core/src/db/schema-authn.ts is managed by better-auth, and modifications to field constraints (like making the slug field non-nullable) should maintain compatibility with better-auth's expectations, even if application code assumes non-null values.
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #95
File: fdm-core/src/catalogues.ts:134-170
Timestamp: 2025-03-06T15:23:48.352Z
Learning: When writing tests for fdm-core, avoid using Vitest's vi mocking utilities and prefer manual JavaScript mocks.
Learnt from: SvenVw
PR: #95
File: fdm-core/src/cultivation.ts:67-73
Timestamp: 2025-03-06T14:58:48.603Z
Learning: When writing unit tests for the fdm project, avoid using Vitest's mocking utilities (vi) as it has caused problems in the past not related to the actual code. Instead, use simple object literals with methods that throw errors to test error handling.
fdm-data/package.json (3)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
fdm-calculator/package.json (8)
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:34:20.850Z
Learning: The updateField function in fdm-core has optional parameters that don't need to be passed as undefined. Only fdm and b_id are required.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
fdm-core/package.json (14)
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:34:20.850Z
Learning: The updateField function in fdm-core has optional parameters that don't need to be passed as undefined. Only fdm and b_id are required.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #88
File: fdm-core/src/cultivation.ts:246-246
Timestamp: 2025-03-04T11:09:08.169Z
Learning: In the FDM codebase, the fdm parameter should be documented as "The FDM instance providing the connection to the database. The instance can be created with {@link createFdmServer}." in JSDoc comments.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts will be updated in commit b17fac1 to include id and email fields, which are necessary for subsequent role updates and user removal operations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The organization schema in fdm-core/src/db/schema-authn.ts is managed by better-auth, and modifications to field constraints (like making the slug field non-nullable) should maintain compatibility with better-auth's expectations, even if application code assumes non-null values.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts needed to include id and email fields for users, which was fixed in commit b17fac1.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/db/schema-authn.ts:70-76
Timestamp: 2025-04-18T14:20:40.975Z
Learning: The schema defined in fdm-core/src/db/schema-authn.ts follows better-auth's structure and requirements. While the schema is defined in the application code, modifications to it should maintain compatibility with better-auth's expectations.
fdm-app/package.json (18)
Learnt from: SvenVw
PR: #20
File: fdm-app/tsconfig.json:8-9
Timestamp: 2024-12-11T12:09:35.540Z
Learning: In the fdm-app/tsconfig.json file, the include path .react-router/types/**/* refers to a build-time generated directory which is intentionally not included in the repository.
Learnt from: SvenVw
PR: #45
File: fdm-app/app/routes/farm.$b_id_farm.settings._index.tsx:1-1
Timestamp: 2025-01-14T16:06:21.832Z
Learning: In the fdm project, redirect and other routing utilities should be imported from react-router instead of react-router-dom.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #45
File: fdm-app/app/routes/farm.$b_id_farm._index.tsx:1-1
Timestamp: 2025-01-14T16:06:24.294Z
Learning: In the fdm-app codebase, the redirect function should be imported from react-router, not react-router-dom.
Learnt from: SvenVw
PR: #124
File: fdm-app/app/components/custom/farm/farm-title.tsx:3-3
Timestamp: 2025-04-18T13:49:17.029Z
Learning: In the fdm project, NavLink and other routing components can be imported from either "react-router" or "react-router-dom" as react-router-dom is included in react-router.
Learnt from: SvenVw
PR: #16
File: fdm-app/package.json:40-40
Timestamp: 2024-12-16T10:48:27.598Z
Learning: In the fdm-app/package.json file, remix-toast and sonner are not duplicate dependencies. remix-toast manages toast generation and handling within the Remix framework, while sonner provides the actual implementation and UI of the toast notifications. They serve different purposes and are intended to be used together in the project.
Learnt from: SvenVw
PR: #67
File: fdm-app/app/routes/farm.create.$b_id_farm.fields.$b_id.tsx:601-610
Timestamp: 2025-01-31T15:05:14.310Z
Learning: The updateField function in fdm-core has optional parameters after fdm and b_id. The TypeScript definitions might show 8 required parameters due to a potential version mismatch.
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts will be updated in commit b17fac1 to include id and email fields, which are necessary for subsequent role updates and user removal operations.
Learnt from: SvenVw
PR: #124
File: fdm-core/src/organization.ts:300-339
Timestamp: 2025-04-18T14:51:48.033Z
Learning: The getUsersInOrganization function in fdm-core/src/organization.ts needed to include id and email fields for users, which was fixed in commit b17fac1.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/fertilizer-applications/form.tsx:6-6
Timestamp: 2025-05-09T14:41:43.484Z
Learning: The project uses React Router v7 which exports a Form component directly from the "react-router" package, making importing from "@remix-run/react" unnecessary.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/fertilizer-applications/form.tsx:6-6
Timestamp: 2025-05-09T14:41:43.484Z
Learning: The project uses React Router v7 which exports a Form component directly from the "react-router" package, not from "@remix-run/react".
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/combobox.tsx:34-37
Timestamp: 2025-05-09T14:53:44.578Z
Learning: In the context of this React Router v7 project, it's important to follow the pattern of importing only the types (like UseFormReturn) from "react-hook-form" while importing the Form component from "react-router" to avoid naming conflicts.
Learnt from: SvenVw
PR: #138
File: fdm-app/app/components/custom/combobox.tsx:34-37
Timestamp: 2025-05-09T14:58:10.465Z
Learning: When updating React components that use both react-hook-form and React Router v7, it's important to only import types (like UseFormReturn, FieldValues) from react-hook-form to avoid naming conflicts with React Router's Form component. Use import type { ... } from 'react-hook-form' syntax to ensure only types are imported.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
Learnt from: SvenVw
PR: #16
File: fdm-app/app/routes/app.addfarm.$b_id_farm.cultivations.$b_lu_catalogue.fertilizers.tsx:1-1
Timestamp: 2024-12-16T10:56:07.561Z
Learning: The project uses react-router v7, and the data function is exported and used for error handling in loaders and actions.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
fdm-docs/package.json (3)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo. The catalog: syntax without a package name is the shorthand for referencing the default catalog.
package.json (4)
Learnt from: SvenVw
PR: #134
File: fdm-calculator/package.json:44-46
Timestamp: 2025-05-26T09:39:31.123Z
Learning: When analyzing npm package versions in package.json files, the specified versions are often already the latest stable versions, especially in active projects. Always verify with npm registry before suggesting version updates.
Learnt from: SvenVw
PR: #6
File: fdm-app/vite.config.ts:5-9
Timestamp: 2024-11-25T12:42:32.783Z
Learning: In the fdm-app project, SvenVw is preparing for migration to Remix v3 and may include type declarations or configurations for v3 features in advance, such as in vite.config.ts.
Learnt from: SvenVw
PR: #147
File: fdm-docs/package.json:39-39
Timestamp: 2025-05-28T07:57:19.217Z
Learning: In pnpm workspaces, the syntax "typescript": "catalog:" is valid and correct. It references the version defined in the workspace's catalog section in pnpm-workspace.yaml, allowing consistent dependency versions across packages in a monorepo.
Learnt from: SvenVw
PR: #49
File: fdm-data/rollup.config.js:7-17
Timestamp: 2025-01-24T11:46:49.990Z
Learning: When suggesting external dependencies in Rollup configuration, only include packages that are actually listed in the package's dependencies or peerDependencies.
🔇 Additional comments (22)
.github/workflows/deploy-docs.yml (1)
39-41: pnpm patch‐level bump looks goodVersion alignment to 10.13.1 keeps CI in sync with workspace
packageManagerfields—no further action required.fdm-data/package.json (1)
60-60: All workspaces use pnpm@10.13.1
I’ve checked everypackage.jsonin the repo and confirmed each declares"packageManager": "pnpm@10.13.1". No updates are needed..github/workflows/release.yml (1)
37-41: Workflow now installs pnpm 10.13.1 — step name still accuratePatch bump is harmless; no additional cache key changes needed.
.github/workflows/deploy-docs-test.yml (1)
35-38: CI matrix aligned to latest pnpm patchUpgrade mirrors production deploy workflow; 👍.
.github/workflows/tests.yml (1)
53-57: Consistent pnpm upgrade across all test jobsAll three jobs now pin 10.13.1; caches remain valid because the major version hasn’t changed.
Also applies to: 146-150, 226-230
fdm-calculator/package.json (2)
45-45: Confirm compatibility of decimal.js ^10.6.0 before merge
Minor bumps indecimal.jsoccasionally add new ES features (e.g.,BigIntinterop) that can break older bundlers. Please run the calculator test-suite and tree-shake build to make sure no regressions slip in.
64-64: pnpm 10.13.1 bump looks good
Matches the workspace & CI updates – no further action.fdm-docs/package.json (3)
27-27: lucide-react ^0.525.0 may require React ≥18.3
The icon package has introduced breaking prop changes in a few recent releases. Make sure the docs still render and that any tree-shaken imports compile without type errors.
38-38: typedoc-plugin-markdown 4.7.1 ↔ typedoc catalog ^0.28.7
This plugin release notes mention full support for TypeDoc 0.28, so the versions are aligned – good catch.
56-56: pnpm bump consistent with workspace
No issues spotted.fdm-core/package.json (3)
54-55: @types/node ^24.1.0 may widen the lib set
Node 24 typings include built-inworker_threads,fs/promisesupdates, etc. Verifytsconfig.libstill matches the target runtime (Node 18 is LTS in CI).
67-67: Patch bump of @electric-sql/pglite looks safe
No breaking API changes reported between 0.3.4 → 0.3.5.
76-76: Workspace pnpm version aligned
✅pnpm-workspace.yaml (1)
9-26: Large catalog sweep – run a full workspace build before release
Several major bumps (vite6→7,vitest2→3) can surface subtle TS/ESM issues. A one-shotpnpm -r run build && pnpm -r run testlocally will save CI iterations.package.json (2)
28-31: Biome 2.1.x introduces stricter lint rules
If the repo CI usesbiome check ., expect new diagnostics (notably around unused-vars). Consider pinning the version or runningbiome check --writeonce to auto-fix.
32-32: Root packageManager version bump acknowledged
Consistent with sub-packages & CI.fdm-app/package.json (6)
25-26: Heads-up: @react-email/components 0.3 moves to ESM-only
Version0.3.xships pure ESM. Verify that all build tools (Vite, jest, vitest, etc.) are ESM-aware, otherwise runtimeERR_REQUIRE_ESMwill surface.
27-32: Router family bumped to 7.7 – rerun typegen & e2e routes
All@react-router/*andreact-router(-dom)packages are now^7.7.0. Although this is a minor, it includes the newcreateStaticRouterAPI and a few breaking TS tweaks. Make sure:
•react-router typegencompletes without errors.
• Tree-shaken imports (redirect,Form, etc.) still resolve from barereact-router.
• CI e2e navigation tests pass.Also applies to: 65-66, 83-84
54-56: Verify PostHog dual-runtime alignment
posthog-js@1.257.2andposthog-node@5.6.0are now one major apart. Confirm that the server-side.capture()properties you use still exist in v5 while the client bundle keeps its separate peer dependencies (in React 19).
57-58: proj4 now ships its own types – drop @types/proj4
Good catch upgrading to^2.19.7; this version includes bundled.d.ts. Keeping a stale@types/proj4elsewhere in the mono-repo would cause the “duplicate identifier” error. Nothing to change here.
73-74: Potential peer range mismatch forsonner@2
sonner@2listsreact@>=18 <20. You’re already onreact@19.1.0. Until sonner expands the peer range,npm installwill emit warnings and pnpm may hoist duplicates. Consider pinningsonner@2.0.6-nextor opening an upstream issue.
78-79: zod patch is safe
Upgrade to3.25.76only fixes.nullable()typings, no runtime impact.
Summary by CodeRabbit