Merge branch 'main' into feat/transaction-search-docs-and-deploy-note

Author: jesseturner21

.github/workflows/slack-open-prs-notification.yml

@@ -0,0 +1,51 @@
+name: Slack Open PRs Notification
+
+on:
+  schedule:
+    - cron: '0 13 * * *' # 8:00 AM EST (13:00 UTC)
+  workflow_dispatch:
+
+permissions:
+  pull-requests: read
+
+jobs:
+  notify-slack:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Get open PRs
+        id: open-prs
+        uses: actions/github-script@v8
+        with:
+          script: |
+            const { data: prs } = await github.rest.pulls.list({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              state: 'open',
+            });
+
+            const count = prs.length;
+
+            // Format each PR with plain text and bare URL (Slack auto-links URLs)
+            const prList = prs.map(pr =>
+              `• #${pr.number} - ${pr.title} (by ${pr.user.login})\n  ${pr.html_url}`
+            ).join('\n');
+
+            core.setOutput('count', count);
+
+            // Use GITHUB_OUTPUT delimiter for multiline support
+            const fs = require('fs');
+            fs.appendFileSync(
+              process.env.GITHUB_OUTPUT,
+              `pr_list<<PRLIST_EOF\n${prList}\nPRLIST_EOF\n`
+            );
+
+      - name: Send open PRs summary to Slack
+        uses: slackapi/slack-github-action@v2.1.1
+        with:
+          webhook: ${{ secrets.SLACK_OPEN_PRS_WEBHOOK_URL }}
+          webhook-type: webhook-trigger
+          payload: |
+            pr_count: "${{ steps.open-prs.outputs.count }}"
+            pr_list: ${{ toJSON(steps.open-prs.outputs.pr_list) }}
+            repository: "${{ github.repository }}"
+            repository_url: "https://github.com/${{ github.repository }}/pulls"

package-lock.json

@@ -106,13 +106,13 @@
     "aws-cdk-lib": "^2.240.0",
     "constructs": "^10.4.4",
     "esbuild": "^0.27.2",
-    "eslint": "^9.39.3",
+    "eslint": "^10.0.3",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-react-refresh": "^0.4.26",
+    "eslint-plugin-react-refresh": "^0.5.2",
     "eslint-plugin-security": "^4.0.0",
     "husky": "^9.1.7",
     "ink-testing-library": "^4.0.0",

package.json

@@ -35,6 +35,7 @@ export interface AddGatewayOptions {
   agentClientId?: string;
   agentClientSecret?: string;
   agents?: string;
+  semanticSearch?: boolean;
   json?: boolean;
 }
 

src/cli/commands/add/types.ts

@@ -26,6 +26,7 @@ export interface AddGatewayOptions {
   agentClientId?: string;
   agentClientSecret?: string;
   agents?: string;
+  enableSemanticSearch?: boolean;
 }
 
 /**
@@ -156,6 +157,7 @@ export class GatewayPrimitive extends BasePrimitive<AddGatewayOptions, Removable
       .option('--agent-client-id <id>', 'Agent OAuth client ID')
       .option('--agent-client-secret <secret>', 'Agent OAuth client secret')
       .option('--agents <agents>', 'Comma-separated agent names')
+      .option('--no-semantic-search', 'Disable semantic search for tool discovery')
       .option('--json', 'Output as JSON')
       .action(async (rawOptions: Record<string, string | boolean | undefined>) => {
         const cliOptions = rawOptions as unknown as CLIAddGatewayOptions;
@@ -186,6 +188,7 @@ export class GatewayPrimitive extends BasePrimitive<AddGatewayOptions, Removable
             agentClientId: cliOptions.agentClientId,
             agentClientSecret: cliOptions.agentClientSecret,
             agents: cliOptions.agents,
+            enableSemanticSearch: cliOptions.semanticSearch !== false,
           });
 
           if (cliOptions.json) {
@@ -282,6 +285,7 @@ export class GatewayPrimitive extends BasePrimitive<AddGatewayOptions, Removable
       description: options.description ?? `Gateway for ${options.name}`,
       authorizerType: options.authorizerType,
       jwtConfig: undefined,
+      enableSemanticSearch: options.enableSemanticSearch ?? true,
     };
 
     if (options.authorizerType === 'CUSTOM_JWT' && options.discoveryUrl) {
@@ -348,6 +352,7 @@ export class GatewayPrimitive extends BasePrimitive<AddGatewayOptions, Removable
       targets: movedTargets,
       authorizerType: config.authorizerType,
       authorizerConfiguration: this.buildAuthorizerConfiguration(config),
+      enableSemanticSearch: config.enableSemanticSearch,
     };
 
     mcpSpec.agentCoreGateways.push(gateway);

src/cli/primitives/GatewayPrimitive.ts

@@ -1,7 +1,7 @@
 import { useMultiSelectNavigation } from '../useMultiSelectNavigation.js';
 import { Text } from 'ink';
 import { render } from 'ink-testing-library';
-import React from 'react';
+import React, { useImperativeHandle } from 'react';
 import { afterEach, describe, expect, it, vi } from 'vitest';
 
 const UP_ARROW = '\x1B[A';
@@ -34,13 +34,15 @@ function Harness({
   isActive,
   textInputActive,
   requireSelection,
+  initialSelectedIds,
 }: {
   testItems?: Item[];
   onConfirm?: (ids: string[]) => void;
   onExit?: () => void;
   isActive?: boolean;
   textInputActive?: boolean;
   requireSelection?: boolean;
+  initialSelectedIds?: string[];
 }) {
   const { cursorIndex, selectedIds } = useMultiSelectNavigation({
     items: testItems,
@@ -50,6 +52,7 @@ function Harness({
     isActive,
     textInputActive,
     requireSelection,
+    initialSelectedIds,
   });
   return (
     <Text>
@@ -58,6 +61,27 @@ function Harness({
   );
 }
 
+interface ResetHarnessHandle {
+  reset: () => void;
+}
+
+const ResetHarness = React.forwardRef<ResetHarnessHandle, { initialSelectedIds?: string[] }>(
+  ({ initialSelectedIds }, ref) => {
+    const { cursorIndex, selectedIds, reset } = useMultiSelectNavigation({
+      items,
+      getId,
+      initialSelectedIds,
+    });
+    useImperativeHandle(ref, () => ({ reset }));
+    return (
+      <Text>
+        cursor:{cursorIndex} selected:{Array.from(selectedIds).sort().join(',')}
+      </Text>
+    );
+  }
+);
+ResetHarness.displayName = 'ResetHarness';
+
 describe('useMultiSelectNavigation', () => {
   it('starts with cursorIndex=0 and empty selectedIds', () => {
     const { lastFrame } = render(<Harness />);
@@ -217,4 +241,43 @@ describe('useMultiSelectNavigation', () => {
     await delay();
     expect(onExit).not.toHaveBeenCalled();
   });
+
+  it('initialSelectedIds pre-selects items', () => {
+    const { lastFrame } = render(<Harness initialSelectedIds={['1', '3']} />);
+    expect(lastFrame()).toContain('cursor:0');
+    expect(lastFrame()).toContain('selected:1,3');
+  });
+
+  it('initialSelectedIds items can be toggled off', async () => {
+    const { lastFrame, stdin } = render(<Harness initialSelectedIds={['1']} />);
+    await delay();
+    expect(lastFrame()).toContain('selected:1');
+
+    // Cursor is at 0 (item id '1'), press Space to toggle it off
+    stdin.write(SPACE);
+    await delay();
+    expect(lastFrame()).not.toMatch(/selected:\S/);
+  });
+
+  it('reset restores initialSelectedIds', async () => {
+    const ref = React.createRef<ResetHarnessHandle>();
+    const { lastFrame, stdin } = render(<ResetHarness ref={ref} initialSelectedIds={['2']} />);
+    await delay();
+    expect(lastFrame()).toContain('selected:2');
+
+    // Move cursor to item '2' (index 1) and toggle it off
+    stdin.write(DOWN_ARROW);
+    await delay();
+    stdin.write(SPACE);
+    await delay();
+    expect(lastFrame()).not.toMatch(/selected:\S/);
+
+    // Trigger reset to restore initialSelectedIds
+    React.act(() => {
+      ref.current!.reset();
+    });
+    await delay();
+    expect(lastFrame()).toContain('selected:2');
+    expect(lastFrame()).toContain('cursor:0');
+  });
 });

src/cli/tui/hooks/__tests__/useMultiSelectNavigation.test.tsx

@@ -28,6 +28,7 @@ export function useCreateGateway() {
         allowedScopes: config.jwtConfig?.allowedScopes?.join(','),
         agentClientId: config.jwtConfig?.agentClientId,
         agentClientSecret: config.jwtConfig?.agentClientSecret,
+        enableSemanticSearch: config.enableSemanticSearch,
       });
       if (!addResult.success) {
         throw new Error(addResult.error ?? 'Failed to create gateway');

src/cli/tui/hooks/useCreateMcp.ts

@@ -16,6 +16,8 @@ interface UseMultiSelectNavigationOptions<T> {
   textInputActive?: boolean;
   /** Whether to require at least one selection before confirm (default: false) */
   requireSelection?: boolean;
+  /** Initial set of selected item IDs (default: empty set) */
+  initialSelectedIds?: string[];
 }
 
 interface UseMultiSelectNavigationResult {
@@ -56,9 +58,10 @@ export function useMultiSelectNavigation<T>({
   isActive = true,
   textInputActive = false,
   requireSelection = false,
+  initialSelectedIds,
 }: UseMultiSelectNavigationOptions<T>): UseMultiSelectNavigationResult {
   const [cursorIndex, setCursorIndex] = useState(0);
-  const [selectedIds, setSelectedIds] = useState<Set<string>>(new Set());
+  const [selectedIds, setSelectedIds] = useState<Set<string>>(() => new Set(initialSelectedIds ?? []));
 
   const toggleSelection = useCallback(() => {
     const item = items[cursorIndex];
@@ -77,8 +80,8 @@ export function useMultiSelectNavigation<T>({
 
   const reset = useCallback(() => {
     setCursorIndex(0);
-    setSelectedIds(new Set());
-  }, []);
+    setSelectedIds(new Set(initialSelectedIds ?? []));
+  }, [initialSelectedIds]);
 
   useInput(
     (input, key) => {

src/cli/tui/hooks/useMultiSelectNavigation.ts

@@ -15,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
 import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
 import { generateUniqueName } from '../../utils';
 import type { AddGatewayConfig } from './types';
-import { AUTHORIZER_TYPE_OPTIONS, GATEWAY_STEP_LABELS } from './types';
+import { AUTHORIZER_TYPE_OPTIONS, GATEWAY_STEP_LABELS, SEMANTIC_SEARCH_ITEM_ID } from './types';
 import { useAddGatewayWizard } from './useAddGatewayWizard';
 import { Box, Text } from 'ink';
 import React, { useMemo, useState } from 'react';
@@ -27,6 +27,8 @@ interface AddGatewayScreenProps {
   unassignedTargets: string[];
 }
 
+const INITIAL_ADVANCED_SELECTED = [SEMANTIC_SEARCH_ITEM_ID];
+
 export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassignedTargets }: AddGatewayScreenProps) {
   const wizard = useAddGatewayWizard(unassignedTargets.length);
 
@@ -48,10 +50,16 @@ export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassig
     []
   );
 
+  const advancedConfigItems: SelectableItem[] = useMemo(
+    () => [{ id: SEMANTIC_SEARCH_ITEM_ID, title: 'Semantic Search' }],
+    []
+  );
+
   const isNameStep = wizard.step === 'name';
   const isAuthorizerStep = wizard.step === 'authorizer';
   const isJwtConfigStep = wizard.step === 'jwt-config';
   const isIncludeTargetsStep = wizard.step === 'include-targets';
+  const isAdvancedConfigStep = wizard.step === 'advanced-config';
   const isConfirmStep = wizard.step === 'confirm';
 
   const authorizerNav = useListNavigation({
@@ -70,6 +78,17 @@ export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassig
     requireSelection: false,
   });
 
+  const advancedNav = useMultiSelectNavigation({
+    items: advancedConfigItems,
+    getId: item => item.id,
+    initialSelectedIds: INITIAL_ADVANCED_SELECTED,
+    onConfirm: selectedIds =>
+      wizard.setAdvancedConfig({ enableSemanticSearch: selectedIds.includes(SEMANTIC_SEARCH_ITEM_ID) }),
+    onExit: () => wizard.goBack(),
+    isActive: isAdvancedConfigStep,
+    requireSelection: false,
+  });
+
   useListNavigation({
     items: [{ id: 'confirm', title: 'Confirm' }],
     onSelect: () => onComplete(wizard.config),
@@ -136,13 +155,14 @@ export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassig
     }
   };
 
-  const helpText = isIncludeTargetsStep
-    ? 'Space toggle · Enter confirm · Esc back'
-    : isConfirmStep
-      ? HELP_TEXT.CONFIRM_CANCEL
-      : isAuthorizerStep
-        ? HELP_TEXT.NAVIGATE_SELECT
-        : HELP_TEXT.TEXT_INPUT;
+  const helpText =
+    isIncludeTargetsStep || isAdvancedConfigStep
+      ? 'Space toggle · Enter confirm · Esc back'
+      : isConfirmStep
+        ? HELP_TEXT.CONFIRM_CANCEL
+        : isAuthorizerStep
+          ? HELP_TEXT.NAVIGATE_SELECT
+          : HELP_TEXT.TEXT_INPUT;
 
   const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={GATEWAY_STEP_LABELS} />;
 
@@ -202,6 +222,30 @@ export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassig
             <Text dimColor>No unassigned targets available. Press Enter to continue.</Text>
           ))}
 
+        {isAdvancedConfigStep && (
+          <Box flexDirection="column">
+            <Text bold>Advanced Configuration</Text>
+            <Text dimColor>Toggle options with Space, press Enter to continue</Text>
+            <Box marginTop={1} flexDirection="column">
+              {advancedConfigItems.map((item, idx) => {
+                const isCursor = idx === advancedNav.cursorIndex;
+                const isChecked = advancedNav.selectedIds.has(item.id);
+                const checkbox = isChecked ? '[✓]' : '[ ]';
+                return (
+                  <Box key={item.id}>
+                    <Text wrap="truncate">
+                      <Text color={isCursor ? 'cyan' : undefined}>{isCursor ? '❯' : ' '} </Text>
+                      <Text color={isChecked ? 'green' : undefined}>{checkbox} </Text>
+                      <Text color={isCursor ? 'cyan' : undefined}>{item.title}</Text>
+                    </Text>
+                    <Text dimColor> {isChecked ? 'Enabled' : 'Disabled'}</Text>
+                  </Box>
+                );
+              })}
+            </Box>
+          </Box>
+        )}
+
         {isConfirmStep && (
           <ConfirmReview
             fields={[
@@ -228,6 +272,7 @@ export function AddGatewayScreen({ onComplete, onExit, existingGateways, unassig
                     ? wizard.config.selectedTargets.join(', ')
                     : '(none)',
               },
+              { label: 'Semantic Search', value: wizard.config.enableSemanticSearch ? 'Enabled' : 'Disabled' },
             ]}
           />
         )}