[sync] Merge pull request #1077 from teableio/fix/T1720-ai-field-prompt-editor-resize

Synced from teableio/teable-ee@2f9c9c1

Author: teable-bot

.gitignore

@@ -29,6 +29,9 @@ node_modules
 /build
 /dist/
 
+# Next.js auto-generated type definitions
+**/next-env.d.ts
+
 # Cache
 *.tsbuildinfo
 **/.eslintcache

apps/nestjs-backend/package.json

@@ -98,7 +98,7 @@
     "dotenv-flow-cli": "1.1.1",
     "es-check": "7.1.1",
     "eslint": "8.57.0",
-    "eslint-config-next": "14.2.14",
+    "eslint-config-next": "15.5.9",
     "get-tsconfig": "4.7.3",
     "istanbul-merge": "2.0.0",
     "npm-run-all2": "6.1.2",
@@ -213,7 +213,7 @@
     "nestjs-i18n": "10.5.1",
     "nestjs-pino": "4.4.1",
     "nestjs-redoc": "2.2.2",
-    "next": "14.2.35",
+    "next": "16.1.3",
     "node-fetch": "2.7.0",
     "node-sql-parser": "5.3.8",
     "nodemailer": "6.9.13",

apps/nestjs-backend/src/features/next/next.service.ts

@@ -4,14 +4,13 @@ import { ConfigService } from '@nestjs/config';
 import { generateQueryId } from '@teable/core';
 import type { IQueryParamsRo, IQueryParamsVo } from '@teable/openapi';
 import createServer from 'next';
-import type { NextServer } from 'next/dist/server/next';
 import { CacheService } from '../../cache/cache.service';
 import type { ICacheStore } from '../../cache/types';
 
 @Injectable()
 export class NextService implements OnModuleInit, OnModuleDestroy {
   private logger = new Logger(NextService.name);
-  public server!: NextServer;
+  public server!: ReturnType<typeof createServer>;
   constructor(
     private configService: ConfigService,
     private readonly cacheService: CacheService<ICacheStore>

apps/nestjs-backend/src/index.ts

@@ -14,7 +14,23 @@ async function main() {
 
 main();
 
+// Force exit after timeout if app.close() hangs during development
+// enableShutdownHooks() in bootstrap.ts handles graceful shutdown,
+// but some modules may not release resources properly
 if (module.hot) {
+  const forceExitTimeout = 5000; // 5 seconds
+
+  const forceExit = (signal: string) => {
+    console.log(`Received ${signal}, forcing exit in ${forceExitTimeout}ms if not closed...`);
+    setTimeout(() => {
+      console.log('Force exiting due to timeout...');
+      process.exit(0);
+    }, forceExitTimeout).unref();
+  };
+
+  process.on('SIGINT', () => forceExit('SIGINT'));
+  process.on('SIGTERM', () => forceExit('SIGTERM'));
+
   module.hot.accept((err: Error) => {
     if (err) {
       console.error('[HMR] Update failed, restarting...', err);

apps/nestjs-backend/test/base-export-sentry.e2e-spec.ts

@@ -44,12 +44,9 @@ describe('Base export sentry reporting (e2e)', () => {
 
   it('captures export failures in sentry even when running asynchronously', async () => {
     const exportError = new Error('mock export failure');
-    type ExportServiceTestTarget = BaseExportService & {
-      captureExportError: (err: unknown, context: Record<string, unknown>) => void;
-      processExportBaseZip: (...args: unknown[]) => Promise<unknown>;
-      notifyExportResult: (...args: unknown[]) => Promise<unknown>;
-    };
-    const exportService = baseExportService as ExportServiceTestTarget;
+    // Cast to `any` to access private methods for testing purposes
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    const exportService = baseExportService as any;
 
     const captureErrorSpy = vi.spyOn(exportService, 'captureExportError');
     const processSpy = vi

apps/nextjs-app/next-env.d.ts

@@ -1,11 +1,11 @@
 // @ts-check
 
-const { readFileSync } = require('node:fs');
-const path = require('node:path');
+const { readFileSync } = require('fs');
+const path = require('path');
 const { createSecureHeaders } = require('next-secure-headers');
 const pc = require('picocolors');
 
-const workspaceRoot = path.resolve(__dirname, '..', '..');
+const workspaceRoot = path.resolve(__dirname, '..', '..', '..');
 /**
  * Once supported replace by node / eslint / ts and out of experimental, replace by
  * `import packageJson from './package.json' assert { type: 'json' };`
@@ -122,7 +122,18 @@ const nextConfig = {
       : '',
   reactStrictMode: true,
   productionBrowserSourceMaps: NEXT_BUILD_ENV_SOURCEMAPS === true,
-  optimizeFonts: true,
+  // optimizeFonts is enabled by default in Next.js 16
+
+  // Transpile packages that use React to ensure single React instance
+  transpilePackages: [
+    'streamdown',
+    'd3-interpolate',
+    'd3-color',
+    // Fix Turbopack "unexpected export *" warnings for CommonJS modules
+    '@dnd-kit/core',
+    '@dnd-kit/sortable',
+    '@dnd-kit/utilities',
+  ],
 
   httpAgentOptions: {
     // @link https://nextjs.org/blog/next-11-1#builds--data-fetching
@@ -136,20 +147,14 @@ const nextConfig = {
 
   // @link https://nextjs.org/docs/advanced-features/compiler#minification
   // @link discussion: https://github.com/vercel/next.js/discussions/30237
-  // Sometimes buggy so enable/disable when debugging.
-  swcMinify: true,
+  // swcMinify is enabled by default in Next.js 15+
 
   compiler: {
     // emotion: true,
   },
 
-  sentry: {
-    hideSourceMaps: true,
-    // To disable the automatic instrumentation of API route handlers and server-side data fetching functions
-    // In other words, disable if you prefer to explicitly handle sentry per api routes (ie: wrapApiHandlerWithSentry)
-    // https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#configure-server-side-auto-instrumentation
-    autoInstrumentServerFunctions: false,
-  },
+  // Note: sentry configuration moved to withSentryConfig wrapper
+  // See: https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
   // @link https://nextjs.org/docs/basic-features/image-optimization
   images: {
@@ -187,6 +192,10 @@ const nextConfig = {
     ? { output: 'standalone', outputFileTracing: true }
     : {}),
 
+  // Server-only packages that should not be bundled for the browser
+  // @link https://nextjs.org/docs/app/api-reference/config/next-config-js/serverExternalPackages
+  serverExternalPackages: ['next-i18next', 'i18next-fs-backend'],
+
   experimental: {
     // @link https://nextjs.org/docs/advanced-features/output-file-tracing#caveats
     ...(NEXT_BUILD_ENV_OUTPUT === 'standalone' ? { outputFileTracingRoot: workspaceRoot } : {}),
@@ -204,15 +213,29 @@ const nextConfig = {
     // appDir: true,
   },
 
+  // Turbopack configuration (Next.js 16 default bundler)
+  turbopack: {
+    // Set workspace root to fix multiple lockfile detection
+    root: workspaceRoot,
+    rules: {
+      '*.svg': {
+        loaders: ['@svgr/webpack'],
+        as: '*.js',
+      },
+    },
+    resolveAlias: {
+      // Required: next-i18next and i18next-fs-backend require 'fs' at top level
+      fs: './turbopack-empty-stub.js',
+    },
+  },
+
   typescript: {
     ignoreBuildErrors: !NEXT_BUILD_ENV_TYPECHECK,
     tsconfigPath: NEXT_BUILD_ENV_TSCONFIG,
   },
 
-  eslint: {
-    ignoreDuringBuilds: !NEXT_BUILD_ENV_LINT,
-    // dirs: [`${__dirname}/src`],
-  },
+  // Note: eslint configuration is no longer supported in next.config.js
+  // Use ESLint CLI directly: npx eslint .
 
   // @link https://nextjs.org/docs/api-reference/next.config.js/rewrites
   async rewrites() {
@@ -266,6 +289,21 @@ const nextConfig = {
       config.resolve.fallback = { ...config.resolve.fallback, fs: false };
     }
 
+    // Handle node: protocol imports for server-side (e.g., node:fs from i18next-fs-backend)
+    // This tells webpack to treat node: protocol imports as external commonjs modules
+    if (isServer) {
+      config.externals = config.externals || [];
+      config.externals.push(
+        (/** @type {{ request: string }} */ { request }, /** @type {Function} */ callback) => {
+          if (request.startsWith('node:')) {
+            // Convert node:fs -> fs, node:path -> path, etc.
+            return callback(null, 'commonjs ' + request.slice(5));
+          }
+          callback();
+        }
+      );
+    }
+
     // https://docs.sentry.io/platforms/javascript/guides/nextjs/configuration/tree-shaking/
     config.plugins.push(
       new webpack.DefinePlugin({
@@ -304,6 +342,8 @@ const nextConfig = {
     APP_NAME: packageJson.name ?? 'not-in-package.json',
     APP_VERSION: packageJson.version ?? 'not-in-package.json',
     BUILD_TIME: new Date().toISOString(),
+    // Note: Sentry debug/tracing variables are handled via webpack DefinePlugin
+    // and cannot be set via Next.js env config (reserved key format)
   },
 };
 
@@ -313,26 +353,22 @@ if (NEXT_BUILD_ENV_SENTRY_ENABLED === true) {
   try {
     // https://docs.sentry.io/platforms/javascript/guides/nextjs/
     const { withSentryConfig } = require('@sentry/nextjs');
-    // @ts-ignore because sentry does not match nextjs current definitions
     config = withSentryConfig(config, {
-      // Additional config options for the Sentry Webpack plugin. Keep in mind that
-      // the following options are set automatically, and overriding them is not
-      // recommended:
-      //   release, url, org, project, authToken, configFile, stripPrefix,
-      //   urlPrefix, include, ignore
-      // For all available options, see:
-      // https://github.com/getsentry/sentry-webpack-plugin#options.
-      // silent: isProd, // Suppresses all logs
-      dryRun: NEXT_BUILD_ENV_SENTRY_UPLOAD_DRY_RUN === true,
-      silent: NEXT_BUILD_ENV_SENTRY_DEBUG === false,
+      // Sentry 10.x options
+      // https://docs.sentry.io/platforms/javascript/guides/nextjs/configuration/build/
+      bundleSizeOptimizations: {
+        excludeDebugStatements: !NEXT_BUILD_ENV_SENTRY_DEBUG,
+        excludeTracing: !NEXT_BUILD_ENV_SENTRY_TRACING,
+      },
+      sourcemaps: {
+        disable: NEXT_BUILD_ENV_SENTRY_UPLOAD_DRY_RUN,
+      },
+      silent: !NEXT_BUILD_ENV_SENTRY_DEBUG,
     });
     console.log(`- ${pc.green('info')} Sentry enabled for this build`);
   } catch {
     console.log(`- ${pc.red('error')} Could not enable sentry, import failed`);
   }
-} else {
-  const { sentry, ...rest } = config;
-  config = rest;
 }
 
 if (tmModules.length > 0) {

apps/nextjs-app/next.config.js

@@ -44,8 +44,8 @@
     "flamegraph-home": "npx 0x --output-dir './.debug/flamegraph/{pid}.0x' --on-port 'autocannon http://localhost:$PORT --duration 20' -- node ../../node_modules/.bin/next start"
   },
   "devDependencies": {
-    "@next/bundle-analyzer": "14.2.14",
-    "@next/env": "14.2.14",
+    "@next/bundle-analyzer": "16.1.1",
+    "@next/env": "16.1.1",
     "@playwright/test": "1.42.1",
     "@size-limit/file": "11.1.2",
     "@svgr/webpack": "8.1.0",
@@ -59,11 +59,11 @@
     "@types/lodash": "4.17.0",
     "@types/node": "22.18.0",
     "@types/nprogress": "0.2.3",
-    "@types/react": "18.2.69",
-    "@types/react-dom": "18.2.22",
+    "@types/react": "18.3.18",
+    "@types/react-dom": "18.3.5",
     "@types/react-grid-layout": "1.3.5",
     "@types/react-syntax-highlighter": "15.5.11",
-    "@types/react-test-renderer": "18.0.7",
+    "@types/react-test-renderer": "18.3.1",
     "@types/sharedb": "3.3.10",
     "@types/streamsaver": "2.0.5",
     "@vitejs/plugin-react-swc": "3.6.0",
@@ -74,7 +74,7 @@
     "dotenv-flow-cli": "1.1.1",
     "es-check": "7.1.1",
     "eslint": "8.57.0",
-    "eslint-config-next": "14.2.14",
+    "eslint-config-next": "15.5.9",
     "get-tsconfig": "4.7.3",
     "happy-dom": "15.11.6",
     "npm-run-all2": "6.1.2",
@@ -118,8 +118,8 @@
     "@hookform/resolvers": "3.3.4",
     "@nem035/gpt-3-encoder": "1.1.7",
     "@radix-ui/react-icons": "1.3.0",
-    "@sentry/nextjs": "7.108.0",
-    "@sentry/react": "7.108.0",
+    "@sentry/nextjs": "10.33.0",
+    "@sentry/react": "10.33.0",
     "@tailwindcss/container-queries": "0.1.1",
     "@tanstack/react-query": "5.90.16",
     "@tanstack/react-table": "8.11.7",
@@ -153,7 +153,7 @@
     "lodash": "4.17.21",
     "lru-cache": "10.2.0",
     "lucide-react": "0.363.0",
-    "next": "14.2.35",
+    "next": "16.1.3",
     "next-i18next": "15.2.0",
     "next-secure-headers": "2.2.0",
     "next-seo": "6.5.0",

apps/nextjs-app/package.json

@@ -1,7 +1,7 @@
 import { forwardRef, useImperativeHandle } from 'react';
 import { AutomationPage } from '../Pages';
 
-interface WorkFlowPanelRef {
+export interface WorkFlowPanelRef {
   getWorkflow?: () => unknown | undefined;
   checkCanActive?: () => {
     canActive: boolean;

apps/nextjs-app/src/features/app/automation/workflow-panel/WorkFlowPanel.tsx

@@ -17,10 +17,11 @@ import { XIcon } from 'lucide-react';
 import { forwardRef, lazy, Suspense, useImperativeHandle, useRef, useState } from 'react';
 import { useTranslation } from 'react-i18next';
 import { tableConfig } from '@/features/i18n/table.config';
+import type { WorkFlowPanelRef } from '@overridable/WorkFlowPanel';
 import { useWorkFlowPanelStore } from './useWorkFlowPaneStore';
 
 const WorkFlowPanelLazy = lazy(() =>
-  import('./WorkFlowPanel').then((module) => ({
+  import('@overridable/WorkFlowPanel').then((module) => ({
     default: module.WorkFlowPanel,
   }))
 );
@@ -71,15 +72,6 @@ const AlertCloseWorkflowDialog = forwardRef<AlertCloseDialogRef, AlertCloseDialo
 
 AlertCloseWorkflowDialog.displayName = 'AlertCloseWorkflowDialog';
 
-interface WorkFlowPanelRef {
-  getWorkflow?: () => unknown | undefined;
-  checkCanActive?: () => {
-    canActive: boolean;
-    message: string;
-  };
-  activeWorkflow?: () => Promise<void>;
-}
-
 export const WorkFlowPanelModal = () => {
   const { baseId = '', workflowId = '', closeModal, open } = useWorkFlowPanelStore();
   const isHydrated = useIsHydrated();