feat: Update output mode logic for Claude client and enhance documentation (#123)

* feat: Update output mode logic for Claude client and enhance documentation

* feat: Add workflow for publishing alpha versions to Verdaccio

* feat: Improve GitHub Actions workflow by adding job names for clarity

* feat: Update codecall configuration and schema to support optional parameters

* fix: Refactor service injection and improve error handling in flow instance

* fix: Adjust context declaration order in flow instance for clarity

* fix: Update ESLint configuration to allow unused variables with underscore prefix

* fix: Enhance error handling by replacing generic errors with InternalMcpError in flow instance

Author: frontegg-david

.github/workflows/publish-alpha.yml

@@ -0,0 +1,81 @@
+name: Publish Alpha to Verdaccio
+
+on:
+  workflow_dispatch:
+    inputs:
+      base_version:
+        description: "Base version (leave empty to use package.json)"
+        required: false
+        type: string
+        default: ""
+
+jobs:
+  publish-alpha:
+    runs-on: ubuntu-latest
+    env:
+      NX_DAEMON: "false"
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node
+        uses: actions/setup-node@v6
+        with:
+          node-version-file: ".nvmrc"
+          cache: "yarn"
+
+      - name: Install dependencies
+        run: yarn install --frozen-lockfile
+
+      - name: Resolve synchronized libs
+        id: libs
+        shell: bash
+        run: |
+          set -euo pipefail
+          PROJECTS=$(npx nx show projects -p tag:versioning:synchronized --type lib --json | node -e "console.log(JSON.parse(require('fs').readFileSync(0,'utf8')).join(','))")
+          echo "projects=$PROJECTS" >> "$GITHUB_OUTPUT"
+          echo "Found libs: $PROJECTS"
+
+      - name: Calculate alpha version
+        id: version
+        shell: bash
+        run: |
+          set -euo pipefail
+          FULL_SHA=$(git rev-parse HEAD)
+          SHA_SUFFIX="${FULL_SHA: -16}"
+          if [ -n "${{ inputs.base_version }}" ]; then
+            BASE="${{ inputs.base_version }}"
+          else
+            BASE=$(node -e "console.log(require('./package.json').version)")
+          fi
+          ALPHA_VERSION="${BASE}-alpha.${SHA_SUFFIX}"
+          echo "version=$ALPHA_VERSION" >> "$GITHUB_OUTPUT"
+          echo "Alpha version: $ALPHA_VERSION"
+
+      - name: Update package versions with Nx
+        run: npx nx release version ${{ steps.version.outputs.version }} --skip-lockfile-update
+
+      - name: Build all packages
+        run: npx nx run-many --targets=build --projects="${{ steps.libs.outputs.projects }}" --parallel
+
+      - name: Configure npm for Verdaccio
+        shell: bash
+        run: |
+          set -euo pipefail
+          REGISTRY_HOST="${VERDACCIO_REGISTRY_URL#https://}"
+          REGISTRY_HOST="${REGISTRY_HOST#http://}"
+          echo "//${REGISTRY_HOST}/:_authToken=${VERDACCIO_TOKEN}" >> ~/.npmrc
+        env:
+          VERDACCIO_REGISTRY_URL: ${{ secrets.VERDACCIO_REGISTRY_URL }}
+          VERDACCIO_TOKEN: ${{ secrets.VERDACCIO_TOKEN }}
+
+      - name: Publish to Verdaccio
+        run: npx nx release publish --registry=${{ secrets.VERDACCIO_REGISTRY_URL }}
+
+      - name: Summary
+        run: |
+          echo "✅ Published alpha version: ${{ steps.version.outputs.version }}"
+          echo "📦 Packages: ${{ steps.libs.outputs.projects }}"
+          echo "🎯 Registry: Verdaccio (private)"

.github/workflows/push.yml

@@ -9,6 +9,7 @@ concurrency:
 jobs:
   # Shared setup job that other jobs can reference
   setup:
+    name: "Setup"
     runs-on: ubuntu-latest
     outputs:
       node-version: ${{ steps.node-version.outputs.version }}
@@ -22,6 +23,7 @@ jobs:
 
   # Lint and format checks (fast, independent)
   lint:
+    name: "Lint & Format Checks"
     needs: setup
     runs-on: ubuntu-latest
     env:
@@ -46,14 +48,14 @@ jobs:
 
       - name: Run linter
         run: npx nx affected -t lint
-        continue-on-error: true
 
       - name: Run prettier
         run: npx prettier --check .
         continue-on-error: true
 
   # Build all libraries
   build:
+    name: "Build Libraries"
     needs: setup
     runs-on: ubuntu-latest
     env:
@@ -90,6 +92,7 @@ jobs:
 
   # Unit tests (depends on build)
   unit-tests:
+    name: "Unit Tests"
     needs: [setup, build]
     runs-on: ubuntu-latest
     env:
@@ -120,72 +123,28 @@ jobs:
       - name: Run unit tests
         run: npx nx affected -t test --passWithNoTests --exclude='demo-e2e-*'
 
-  # E2E tests - split into batches for parallelization
-  e2e-batch-1:
-    needs: [setup, build]
-    runs-on: ubuntu-latest
-    env:
-      NX_DAEMON: "false"
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Setup Node
-        uses: actions/setup-node@v6
-        with:
-          node-version: ${{ needs.setup.outputs.node-version }}
-          cache: "yarn"
-
-      - name: Install dependencies
-        run: yarn install --frozen-lockfile
-
-      - name: Download build artifacts
-        uses: actions/download-artifact@v4
-        with:
-          name: dist
-
-      - name: Set Nx SHAs
-        uses: nrwl/nx-set-shas@v4
-
-      - name: Run E2E tests (batch 1)
-        run: npx nx run-many -t test --projects=demo-e2e-public,demo-e2e-transparent,demo-e2e-openapi,demo-e2e-codecall
-
-  e2e-batch-2:
-    needs: [setup, build]
-    runs-on: ubuntu-latest
-    env:
-      NX_DAEMON: "false"
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Setup Node
-        uses: actions/setup-node@v6
-        with:
-          node-version: ${{ needs.setup.outputs.node-version }}
-          cache: "yarn"
-
-      - name: Install dependencies
-        run: yarn install --frozen-lockfile
-
-      - name: Download build artifacts
-        uses: actions/download-artifact@v4
-        with:
-          name: dist
-
-      - name: Set Nx SHAs
-        uses: nrwl/nx-set-shas@v4
-
-      - name: Run E2E tests (batch 2)
-        run: npx nx run-many -t test --projects=demo-e2e-redis,demo-e2e-providers,demo-e2e-cache,demo-e2e-ui
-
-  e2e-batch-3:
+  # E2E tests - matrix strategy for parallelization
+  e2e-tests:
+    name: "E2E Tests (${{ matrix.project }})"
     needs: [setup, build]
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        project:
+          - demo-e2e-public
+          - demo-e2e-transparent
+          - demo-e2e-openapi
+          - demo-e2e-codecall
+          - demo-e2e-redis
+          - demo-e2e-providers
+          - demo-e2e-cache
+          - demo-e2e-ui
+          - demo-e2e-hooks
+          - demo-e2e-errors
+          - demo-e2e-notifications
+          - demo-e2e-serverless
+          - demo-e2e-multiapp
     env:
       NX_DAEMON: "false"
     steps:
@@ -211,5 +170,5 @@ jobs:
       - name: Set Nx SHAs
         uses: nrwl/nx-set-shas@v4
 
-      - name: Run E2E tests (batch 3)
-        run: npx nx run-many -t test --projects=demo-e2e-hooks,demo-e2e-errors,demo-e2e-notifications,demo-e2e-serverless,demo-e2e-multiapp
+      - name: Run E2E tests (${{ matrix.project }})
+        run: npx nx run ${{ matrix.project }}:test

apps/auth/demo-orchestrated-auth/jest.e2e.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 export default {
   displayName: 'demo-orchestrated-auth-e2e',
   preset: '../../../jest.preset.js',

apps/auth/demo-public/jest.e2e.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 export default {
   displayName: 'demo-public-e2e',
   preset: '../../../jest.preset.js',

apps/auth/demo-transparent-auth/jest.e2e.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 export default {
   displayName: 'demo-transparent-auth-e2e',
   preset: '../../../jest.preset.js',

apps/demo/jest.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 module.exports = {
   displayName: '@frontmcp/demo',
   preset: '../../jest.preset.js',

apps/ui/mdx-demo/jest.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 module.exports = {
   displayName: '@frontmcp/mdx-demo',
   preset: '../../../jest.preset.js',

apps/ui/react-demo/jest.config.ts

@@ -1,4 +1,3 @@
-/* eslint-disable */
 module.exports = {
   displayName: '@frontmcp/react-demo',
   preset: '../../../jest.preset.js',

eslint.config.mjs

@@ -11,11 +11,22 @@ export default [
     files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
     rules: {
       '@typescript-eslint/no-explicit-any': 'warn',
+      '@typescript-eslint/no-empty-function': 'warn',
+      '@typescript-eslint/no-empty-interface': 'warn',
+      '@typescript-eslint/no-empty-object-type': 'warn',
+      '@typescript-eslint/no-unused-vars': [
+        'warn',
+        {
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+          caughtErrorsIgnorePattern: '^_',
+        },
+      ],
       '@nx/enforce-module-boundaries': [
         'error',
         {
           enforceBuildableLibDependency: true,
-          allow: ['^.*/eslint(\\.base)?\\.config\\.[cm]?[jt]s$'],
+          allow: ['^.*/eslint(\\.base)?\\.config\\.[cm]?[jt]s$', '@frontmcp/sdk'],
           depConstraints: [
             {
               sourceTag: '*',
@@ -40,4 +51,14 @@ export default [
       '@typescript-eslint/no-unsafe-function-type': 'off',
     },
   },
+  {
+    // Relax rules for test files
+    files: ['**/*.test.ts', '**/*.spec.ts', '**/__tests__/**/*.ts', '**/__test-utils__/**/*.ts', '**/fixtures/**/*.ts', '**/mocks/**/*.ts', '**/*.mock.ts'],
+    rules: {
+      '@typescript-eslint/no-empty-function': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-unused-vars': 'off',
+      'no-unused-private-class-members': 'off',
+    },
+  },
 ];

libs/adapters/src/openapi/openapi.utils.ts

@@ -137,6 +137,7 @@ export function buildRequest(
         const headerValue = coerceToString(value, mapper.key, 'header');
         // Validate header values for injection attacks
         // Check for: CR, LF, null byte, form feed, vertical tab
+        // eslint-disable-next-line no-control-regex
         if (/[\r\n\x00\f\v]/.test(headerValue)) {
           throw new Error(
             `Invalid header value for '${mapper.key}': contains control characters (possible header injection attack)`,