fix: removed old types and function definitoins causing build error

Author: tejaskash

src/cli/commands/create/command.tsx

@@ -3,7 +3,7 @@ import type { ModelProvider, SDKFramework, TargetLanguage } from '../../../schem
 import { getErrorMessage } from '../../errors';
 import { COMMAND_DESCRIPTIONS } from '../../tui/copy';
 import { CreateScreen } from '../../tui/screens/create';
-import { createProject, createProjectWithAgent, getDryRunInfo, type ProgressCallback } from './action';
+import { type ProgressCallback, createProject, createProjectWithAgent, getDryRunInfo } from './action';
 import type { CreateOptions } from './types';
 import { validateCreateOptions } from './validate';
 import type { Command } from '@commander-js/extra-typings';

src/cli/operations/agent/generate/schema-mapper.ts

@@ -120,56 +120,50 @@ export function mapGenerateConfigToResources(config: GenerateConfig): GenerateCo
 }
 
 /**
- * Compute the default env var name for a memory.
+ * Compute the memory env var name for a memory resource.
+ * Pattern: MEMORY_{NAME}_ID (matches CDK construct pattern)
  */
 function computeMemoryEnvVarName(memoryName: string): string {
-  return `AGENTCORE_MEMORY_${memoryName.toUpperCase()}`;
+  return `MEMORY_${memoryName.toUpperCase()}_ID`;
 }
 
 /**
  * Maps memory option to memory providers for template rendering.
  */
-function mapMemoryOptionToMemoryProviders(
-  memory: MemoryOption,
-  projectName: string
-): MemoryProviderRenderConfig[] {
+function mapMemoryOptionToMemoryProviders(memory: MemoryOption, projectName: string): MemoryProviderRenderConfig[] {
   if (memory === 'none') {
     return [];
   }
 
   const memoryName = `${projectName}Memory`;
+  const strategies = mapGenerateInputToMemories(memory, projectName)[0]?.strategies ?? [];
+
   return [
     {
       name: memoryName,
       envVarName: computeMemoryEnvVarName(memoryName),
+      strategies: strategies.map(s => s.type),
     },
   ];
 }
 
 /**
- * Compute the memory env var name for a memory resource.
- * Pattern: MEMORY_{NAME}_ID (matches CDK construct pattern)
- */
-function computeMemoryEnvVarName(memoryName: string): string {
-  return `MEMORY_${memoryName.toUpperCase()}_ID`;
-}
-
-/**
- * Maps memory option to memory providers for template rendering.
+ * Maps model provider to identity providers for template rendering.
+ * Bedrock uses IAM, so no identity provider is needed.
  */
-function mapMemoryOptionToMemoryProviders(memory: MemoryOption, projectName: string): MemoryProviderRenderConfig[] {
-  if (memory === 'none') {
+function mapModelProviderToIdentityProviders(
+  modelProvider: ModelProvider,
+  projectName: string
+): IdentityProviderRenderConfig[] {
+  if (modelProvider === 'Bedrock') {
     return [];
   }
 
-  const memoryName = `${projectName}Memory`;
-  const strategies = mapGenerateInputToMemories(memory, projectName)[0]?.strategies ?? [];
-
+  const credentialName = computeCredentialName(projectName, modelProvider);
   return [
     {
-      name: memoryName,
-      envVarName: computeMemoryEnvVarName(memoryName),
-      strategies: strategies.map(s => s.type),
+      name: credentialName,
+      envVarName: computeDefaultCredentialEnvVarName(credentialName),
     },
   ];
 }
@@ -191,6 +185,5 @@ export function mapGenerateConfigToRenderConfig(config: GenerateConfig, actualPr
     hasIdentity: config.modelProvider !== 'Bedrock',
     memoryProviders: mapMemoryOptionToMemoryProviders(config.memory, config.projectName),
     identityProviders: mapModelProviderToIdentityProviders(config.modelProvider, projectNameForCredentials),
-    memoryProviders: mapMemoryOptionToMemoryProviders(config.memory, config.projectName),
   };
 }

src/cli/templates/types.ts

@@ -1,12 +1,5 @@
 import type { MemoryStrategyType, ModelProvider, SDKFramework, TargetLanguage } from '../../schema';
 
-/**
- * Memory provider config for template rendering.
- */
-export interface MemoryProviderRenderConfig {
-  name: string;
-  envVarName: string;
-}
 /**
  * Identity provider info for template rendering.
  */
@@ -36,9 +29,8 @@ export interface AgentRenderConfig {
   modelProvider: ModelProvider;
   hasMemory: boolean;
   hasIdentity: boolean;
+  /** Memory providers for template rendering */
   memoryProviders: MemoryProviderRenderConfig[];
   /** Identity providers for template rendering (maps to credentials in schema) */
   identityProviders: IdentityProviderRenderConfig[];
-  /** Memory providers for template rendering */
-  memoryProviders: MemoryProviderRenderConfig[];
 }

src/cli/tui/App.tsx

@@ -139,7 +139,13 @@ function AppContent() {
   }
 
   if (route.name === 'add') {
-    return <AddFlow isInteractive={true} onExit={() => setRoute({ name: 'help' })} onDeploy={() => setRoute({ name: 'deploy' })} />;
+    return (
+      <AddFlow
+        isInteractive={true}
+        onExit={() => setRoute({ name: 'help' })}
+        onDeploy={() => setRoute({ name: 'deploy' })}
+      />
+    );
   }
 
   if (route.name === 'remove') {

src/cli/tui/screens/create/CreateScreen.tsx

@@ -22,11 +22,7 @@ import { join } from 'path';
 import { useCallback, useEffect } from 'react';
 
 /** Build a text representation of the completion screen for terminal output */
-function buildExitMessage(
-  projectName: string,
-  steps: Step[],
-  agentConfig: AddAgentConfig | null
-): string {
+function buildExitMessage(projectName: string, steps: Step[], agentConfig: AddAgentConfig | null): string {
   const lines: string[] = [];
 
   // Title

src/cli/tui/screens/mcp/AddGatewayFlow.tsx

@@ -43,7 +43,13 @@ const MODE_OPTIONS: SelectableItem[] = [
   { id: 'bind', title: 'Bind existing gateway', description: 'Attach an existing gateway to an agent' },
 ];
 
-export function AddGatewayFlow({ isInteractive = true, availableAgents, onExit, onBack, onDeploy }: AddGatewayFlowProps) {
+export function AddGatewayFlow({
+  isInteractive = true,
+  availableAgents,
+  onExit,
+  onBack,
+  onDeploy,
+}: AddGatewayFlowProps) {
   const { createGateway, reset: resetCreate } = useCreateGateway();
   const { gateways: existingGateways, refresh: refreshGateways } = useExistingGateways();
   const [flow, setFlow] = useState<FlowState>({ name: 'mode-select' });

src/cli/tui/screens/mcp/AddMcpToolFlow.tsx

@@ -35,7 +35,13 @@ const MODE_OPTIONS: SelectableItem[] = [
   { id: 'bind', title: 'Bind existing MCP runtime', description: 'Add an agent to an existing MCP runtime' },
 ];
 
-export function AddMcpToolFlow({ isInteractive = true, existingAgents, onExit, onBack, onDeploy }: AddMcpToolFlowProps) {
+export function AddMcpToolFlow({
+  isInteractive = true,
+  existingAgents,
+  onExit,
+  onBack,
+  onDeploy,
+}: AddMcpToolFlowProps) {
   const { createTool, reset: resetCreate } = useCreateMcpTool();
   const { gateways: existingGateways } = useExistingGateways();
   const { toolNames: existingToolNames } = useExistingToolNames();