Merge pull request #1104 from objectstack-ai/copilot/fix-action-step-issue-another-one

Author: hotlong

CHANGELOG.md

@@ -13,6 +13,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- **CI Build Fix: Replace dynamic `require()` with static imports** (`@object-ui/plugin-view`): Replaced module-level `require('@object-ui/react')` calls in `index.tsx` and `ObjectView.tsx` with static `import` statements. The dynamic `require()` pattern is not supported by Next.js Turbopack, causing the docs site build to fail during SSR prerendering of the `/docs/components/complex/view-switcher` page with `Error: dynamic usage of require is not supported`. Since `@object-ui/react` is already a declared dependency and other files in the package use static imports from it, replacing the `require()` with static imports is safe and eliminates the SSR compatibility issue.
+
+- **CI Build Fix: Remove unused `...rest` parameter** (`@object-ui/plugin-calendar`): Removed unused `...rest` destructured parameter from `ObjectCalendar` component props (TS6133), which caused the declaration file generation to emit a TypeScript error during the build.
+
 - **CI Build Fix: Unused Parameters in Test Mocks** (`apps/console`): Fixed TypeScript `noUnusedParameters` errors (TS6133) in `ListToolbarActions.test.tsx` where mock component `props` parameters were declared but never used in `@object-ui/plugin-kanban` and `@object-ui/plugin-calendar` mocks, causing the console `tsc` build step to fail in CI.
 
 - **i18n loadLanguage Not Compatible with Spec REST API Response Format** (`apps/console`): Fixed `loadLanguage` in `apps/console/src/main.tsx` to correctly unwrap the `@objectstack/spec` REST API envelope `{ data: { locale, translations } }`. Previously, the function returned the raw JSON response, which meant `useObjectLabel` and `useObjectTranslation` hooks received a wrapped object instead of the flat translation map, causing business object and field labels (e.g., CRM contact/account) to fall back to English. The fix extracts `data.translations` when the spec envelope is detected, while preserving backward compatibility with mock/dev environments that return flat translation objects. Includes 6 unit tests covering spec envelope unwrapping, flat fallback, HTTP errors, network failures, and edge cases.

packages/plugin-calendar/src/ObjectCalendar.tsx

@@ -155,7 +155,6 @@ export const ObjectCalendar: React.FC<ObjectCalendarProps> = ({
   onViewChange,
   onEventDrop,
   locale,
-  ...rest
 }) => {
   // When the parent (e.g. ObjectView) pre-fetches data and passes it via the `data` prop,
   // we must not trigger a second fetch. Detect external data by checking for an array.

packages/plugin-view/src/ObjectView.tsx

@@ -57,20 +57,13 @@ import {
 } from '@object-ui/components';
 import { Plus } from 'lucide-react';
 import { buildExpandFields } from '@object-ui/core';
+import { SchemaRenderer as ImportedSchemaRenderer } from '@object-ui/react';
 import { ViewSwitcher } from './ViewSwitcher';
 
 /**
- * Attempt to import SchemaRenderer from @object-ui/react.
- * Falls back to null if not available.
+ * SchemaRenderer from @object-ui/react, used to render sub-view schemas.
  */
-let SchemaRendererComponent: React.FC<any> | null = null;
-try {
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
-  const mod = require('@object-ui/react');
-  SchemaRendererComponent = mod.SchemaRenderer || null;
-} catch {
-  // @object-ui/react not available
-}
+const SchemaRendererComponent: React.FC<any> = ImportedSchemaRenderer;
 
 export interface ObjectViewProps {
   /**

packages/plugin-view/src/index.tsx

@@ -8,6 +8,7 @@
 
 import React, { useContext } from 'react';
 import { ComponentRegistry } from '@object-ui/core';
+import { SchemaRendererContext as ImportedSchemaRendererContext } from '@object-ui/react';
 import { ObjectView } from './ObjectView';
 import { ViewSwitcher } from './ViewSwitcher';
 import { FilterUI } from './FilterUI';
@@ -25,22 +26,9 @@ export type { SharedViewLinkProps } from './SharedViewLink';
 
 /**
  * SchemaRendererContext is created by @object-ui/react.
- * We import it dynamically to avoid a circular dependency.
  * The context value provides { dataSource }.
- * A fallback context is created so hooks are never called conditionally.
  */
-const FallbackContext = React.createContext<any>(null);
-let SchemaRendererContext: React.Context<any> = FallbackContext;
-try {
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
-  const mod = require('@object-ui/react');
-  // The context is re-exported from @object-ui/react
-  if (mod.SchemaRendererContext) {
-    SchemaRendererContext = mod.SchemaRendererContext;
-  }
-} catch {
-  // @object-ui/react not available — registry-based dataSource only
-}
+const SchemaRendererContext: React.Context<any> = ImportedSchemaRendererContext;
 
 // Register object-view component
 const ObjectViewRenderer: React.FC<{ schema: any }> = ({ schema }) => {