Merge branch 'master' into add-truefoundry-node

Author: venkat1924

.github/workflows/build-unit-test-pr-comment.yml

@@ -8,6 +8,7 @@ permissions:
   pull-requests: read
   contents: read
   actions: write
+  issues: write
 
 jobs:
   validate_and_dispatch:
@@ -91,7 +92,9 @@ jobs:
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           HEAD_SHA: ${{ steps.check_permissions.outputs.headSha }}
+          PR_NUMBER: ${{ steps.check_permissions.outputs.prNumber }}
         run: |
           gh workflow run ci-manual-build-unit-tests.yml \
             --repo "${{ github.repository }}" \
-            -f ref="${HEAD_SHA}"
+            -f ref="${HEAD_SHA}" \
+            -f pr_number="${PR_NUMBER}"

.github/workflows/ci-manual-build-unit-tests.yml

@@ -6,8 +6,44 @@ on:
       ref:
         description: Commit SHA or ref to check out
         required: true
+      pr_number:
+        description: PR number (optional, for check reporting)
+        required: false
+        type: string
+
+permissions:
+  contents: read
+  checks: write
 
 jobs:
+  create-check-run:
+    name: Create Check Run
+    runs-on: ubuntu-latest
+    if: inputs.pr_number != ''
+    outputs:
+      check_run_id: ${{ steps.create.outputs.check_run_id }}
+    steps:
+      - name: Create pending check run on PR
+        id: create
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            const { data: checkRun } = await github.rest.checks.create({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              name: 'Build & Unit Tests - Checks',
+              head_sha: '${{ inputs.ref }}',
+              status: 'in_progress',
+              output: {
+                title: 'Build & Unit Tests - Checks',
+                summary: 'Running build, unit tests, and lint...'
+              }
+            });
+
+            core.setOutput('check_run_id', checkRun.id);
+            console.log(`Created check run ${checkRun.id} on commit ${{ inputs.ref }}`);
+
   install-and-build:
     name: Install & Build
     runs-on: blacksmith-2vcpu-ubuntu-2204
@@ -47,9 +83,50 @@ jobs:
   post-build-unit-tests:
     name: Build & Unit Tests - Checks
     runs-on: ubuntu-latest
-    needs: [install-and-build, unit-tests, lint]
+    needs: [create-check-run, install-and-build, unit-tests, lint]
     if: always()
     steps:
+      - name: Update check run on PR (if triggered from PR comment)
+        if: inputs.pr_number != ''
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            const checkRunId = '${{ needs.create-check-run.outputs.check_run_id }}';
+
+            if (!checkRunId) {
+              console.log('No check run ID found, skipping update');
+              return;
+            }
+
+            const buildResult = '${{ needs.install-and-build.result }}';
+            const testResult = '${{ needs.unit-tests.result }}';
+            const lintResult = '${{ needs.lint.result }}';
+
+            const conclusion = (buildResult === 'success' && testResult === 'success' && lintResult === 'success')
+              ? 'success'
+              : 'failure';
+
+            const summary = `
+            **Build**: ${buildResult}
+            **Unit Tests**: ${testResult}
+            **Lint**: ${lintResult}
+            `;
+
+            await github.rest.checks.update({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              check_run_id: parseInt(checkRunId),
+              status: 'completed',
+              conclusion: conclusion,
+              output: {
+                title: 'Build & Unit Tests - Checks',
+                summary: summary
+              }
+            });
+
+            console.log(`Updated check run ${checkRunId} with conclusion: ${conclusion}`);
+
       - name: Fail if any job failed
         if: needs.install-and-build.result == 'failure' || needs.unit-tests.result == 'failure' || needs.lint.result == 'failure'
         run: exit 1

docker/images/runners/Dockerfile

@@ -20,6 +20,21 @@ RUN pnpm install --prod --no-lockfile --silent
 RUN mv package.json extras.json
 RUN mv package.json.bak package.json
 
+# Remove `catalog` and `workspace` references from package.json to allow `pnpm add` in extended images
+RUN node -e "const pkg = require('./package.json'); \
+    Object.keys(pkg.dependencies || {}).forEach(k => { \
+        const val = pkg.dependencies[k]; \
+        if (val === 'catalog:' || val.startsWith('catalog:') || val.startsWith('workspace:')) \
+            delete pkg.dependencies[k]; \
+    }); \
+    Object.keys(pkg.devDependencies || {}).forEach(k => { \
+        const val = pkg.devDependencies[k]; \
+        if (val === 'catalog:' || val.startsWith('catalog:') || val.startsWith('workspace:')) \
+            delete pkg.devDependencies[k]; \
+    }); \
+    delete pkg.devDependencies; \
+    require('fs').writeFileSync('./package.json', JSON.stringify(pkg, null, 2));"
+
 # ==============================================================================
 # STAGE 2: Python runner build (@n8n/task-runner-python) with uv
 # Produces a relocatable venv tied to the python version used
@@ -107,7 +122,7 @@ ENV NODE_ENV=production \
     N8N_RELEASE_TYPE=${N8N_RELEASE_TYPE} \
     SHELL=/bin/sh
 
-# Bring `uv` over from python-runner-builder
+# Bring `uv` over from python-runner-builder, to make the image easier to extend
 COPY --from=python-runner-builder /usr/local/bin/uv /usr/local/bin/uv
 
 # Bring node over from node-alpine
@@ -117,6 +132,11 @@ COPY --from=node-alpine /usr/local/bin/node /usr/local/bin/node
 # libc6-compat is required by task-runner-launcher
 RUN apk add --no-cache ca-certificates tini libstdc++ libc6-compat
 
+# Bring corepack and pnpm over, to make the image easier to extend
+COPY --from=node-alpine /usr/local/lib/node_modules /usr/local/lib/node_modules
+RUN ln -s ../lib/node_modules/corepack/dist/corepack.js /usr/local/bin/corepack && \
+    ln -s ../lib/node_modules/corepack/dist/pnpm.js /usr/local/bin/pnpm
+
 RUN addgroup -g 1000 -S runner \
  && adduser  -u 1000 -S -G runner -h /home/runner -D runner
 

jest.config.js

@@ -1,5 +1,6 @@
 const { pathsToModuleNameMapper } = require('ts-jest');
 const { compilerOptions } = require('get-tsconfig').getTsconfig().config;
+const { resolve } = require('path');
 
 /** @type {import('ts-jest').TsJestGlobalOptions} */
 const tsJestOptions = {
@@ -42,11 +43,14 @@ const config = {
 	},
 	transformIgnorePatterns: [`/node_modules/(?!${esmDependenciesPattern})/`],
 	// This resolve the path mappings from the tsconfig relative to each jest.config.js
-	moduleNameMapper: compilerOptions?.paths
-		? pathsToModuleNameMapper(compilerOptions.paths, {
-				prefix: `<rootDir>${compilerOptions.baseUrl ? `/${compilerOptions.baseUrl.replace(/^\.\//, '')}` : ''}`,
-			})
-		: {},
+	moduleNameMapper: {
+		'^@n8n/utils$': resolve(__dirname, 'packages/@n8n/utils/dist/index.cjs'),
+		...(compilerOptions?.paths
+			? pathsToModuleNameMapper(compilerOptions.paths, {
+					prefix: `<rootDir>${compilerOptions.baseUrl ? `/${compilerOptions.baseUrl.replace(/^\.\//, '')}` : ''}`,
+				})
+			: {}),
+	},
 	setupFilesAfterEnv: ['jest-expect-message'],
 	collectCoverage: isCoverageEnabled,
 	coverageReporters: ['text-summary', 'lcov', 'html-spa'],

package.json

@@ -96,6 +96,7 @@
       "axios": "1.12.0",
       "chokidar": "4.0.3",
       "esbuild": "^0.25.0",
+      "expr-eval@2.0.2": "npm:expr-eval-fork@3.0.0",
       "multer": "^2.0.2",
       "prebuild-install": "7.1.3",
       "pug": "^3.0.3",

packages/@n8n/ai-workflow-builder.ee/package.json

@@ -5,6 +5,7 @@
     "clean": "rimraf dist .turbo",
     "typecheck": "tsc --noEmit",
     "build": "tsc -p ./tsconfig.build.json && tsc-alias -p tsconfig.build.json",
+    "build:with-cli": "pnpm build && pnpm --filter=n8n build",
     "format": "biome format --write src",
     "format:check": "biome ci src",
     "test": "jest",

packages/@n8n/ai-workflow-builder.ee/src/tools/engines/node-search-engine.ts

@@ -1,4 +1,4 @@
-import { sublimeSearch } from '@n8n/utils/dist/search/sublimeSearch';
+import { sublimeSearch } from '@n8n/utils';
 import type { INodeTypeDescription, NodeConnectionType } from 'n8n-workflow';
 import { NodeConnectionTypes } from 'n8n-workflow';
 

packages/@n8n/api-types/src/chat-hub.ts

@@ -1,4 +1,10 @@
-import type { StructuredChunk } from 'n8n-workflow';
+import {
+	type StructuredChunk,
+	type JINA_AI_TOOL_NODE_TYPE,
+	type SEAR_XNG_TOOL_NODE_TYPE,
+	type INode,
+	INodeSchema,
+} from 'n8n-workflow';
 import { z } from 'zod';
 import { Z } from 'zod-class';
 
@@ -28,6 +34,8 @@ export const PROVIDER_CREDENTIAL_TYPE_MAP: Record<
 	google: 'googlePalmApi',
 };
 
+export type ChatHubAgentTool = typeof JINA_AI_TOOL_NODE_TYPE | typeof SEAR_XNG_TOOL_NODE_TYPE;
+
 /**
  * Chat Hub conversation model configuration
  */
@@ -107,6 +115,7 @@ export const emptyChatModelsResponse: ChatModelsResponse = {
 	anthropic: { models: [] },
 	google: { models: [] },
 	n8n: { models: [] },
+	// eslint-disable-next-line @typescript-eslint/naming-convention
 	'custom-agent': { models: [] },
 };
 
@@ -122,6 +131,7 @@ export class ChatHubSendMessageRequest extends Z.class({
 			name: z.string(),
 		}),
 	),
+	tools: z.array(INodeSchema),
 }) {}
 
 export class ChatHubRegenerateMessageRequest extends Z.class({
@@ -153,6 +163,7 @@ export class ChatHubUpdateConversationRequest extends Z.class({
 	model: z.string().max(64).optional(),
 	workflowId: z.string().max(36).optional(),
 	agentId: z.string().uuid().optional(),
+	tools: z.array(INodeSchema).optional(),
 }) {}
 
 export type ChatHubMessageType = 'human' | 'ai' | 'system' | 'tool' | 'generic';
@@ -174,6 +185,7 @@ export interface ChatHubSessionDto {
 	agentName: string | null;
 	createdAt: string;
 	updatedAt: string;
+	tools: INode[];
 }
 
 export interface ChatHubMessageDto {
@@ -216,6 +228,7 @@ export interface ChatHubAgentDto {
 	credentialId: string | null;
 	provider: ChatHubLLMProvider;
 	model: string;
+	tools: INode[];
 	createdAt: string;
 	updatedAt: string;
 }
@@ -227,6 +240,7 @@ export class ChatHubCreateAgentRequest extends Z.class({
 	credentialId: z.string(),
 	provider: chatHubProviderSchema.exclude(['n8n', 'custom-agent']),
 	model: z.string().max(64),
+	tools: z.array(INodeSchema),
 }) {}
 
 export class ChatHubUpdateAgentRequest extends Z.class({
@@ -236,6 +250,7 @@ export class ChatHubUpdateAgentRequest extends Z.class({
 	credentialId: z.string().optional(),
 	provider: chatHubProviderSchema.optional(),
 	model: z.string().max(64).optional(),
+	tools: z.array(INodeSchema).optional(),
 }) {}
 
 export interface EnrichedStructuredChunk extends StructuredChunk {

packages/@n8n/api-types/src/dto/data-table/list-data-table-content-query.dto.ts

@@ -79,4 +79,5 @@ export class ListDataTableContentQueryDto extends Z.class({
 	skip: paginationSchema.skip.optional(),
 	filter: filterValidator.optional(),
 	sortBy: sortByValidator.optional(),
+	search: z.string().optional(),
 }) {}

packages/@n8n/api-types/src/dto/oidc/config.dto.ts

@@ -10,4 +10,5 @@ export class OidcConfigDto extends Z.class({
 		.enum(['none', 'login', 'consent', 'select_account', 'create'])
 		.optional()
 		.default('select_account'),
+	authenticationContextClassReference: z.array(z.string()).default([]),
 }) {}