Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/memory.md
Original file line numberDiff line numberDiff line change
Expand Up@@ -229,6 +229,13 @@ pushed to a delivery target in your account, enabling event-driven architectures

### Enabling Streaming

Via the interactive wizard:

```bash
agentcore add memory
# Select "Yes" when prompted for streaming, then provide the data stream ARN and content level
```

Via CLI flags:

```bash
Expand Down
5 changes: 3 additions & 2 deletions src/cli/commands/add/validate.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,6 +15,7 @@ import {
getSupportedModelProviders,
matchEnumValue,
} from '../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../shared/arn-utils';
import { parseAndValidateLifecycleOptions } from '../shared/lifecycle-utils';
import { validateVpcOptions } from '../shared/vpc-utils';
import { validateJwtAuthorizerOptions } from './auth-options';
Expand DownExpand Up@@ -697,8 +698,8 @@ export function validateAddMemoryOptions(options: AddMemoryOptions): ValidationR
return { valid: false, error: '--data-stream-arn is required when --delivery-type is set' };
}

if (options.dataStreamArn && !options.dataStreamArn.startsWith('arn:')) {
return { valid: false, error: '--data-stream-arn must be a valid ARN (starts with arn:)' };
if (options.dataStreamArn && !isValidArn(options.dataStreamArn)) {
return { valid: false, error: `--data-stream-arn: ${ARN_VALIDATION_MESSAGE}` };
}

if (
Expand Down
28 changes: 28 additions & 0 deletions src/cli/commands/shared/__tests__/arn-utils.test.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
import { isValidArn } from '../arn-utils';
import { describe, expect, it } from 'vitest';

describe('isValidArn', () => {
it('accepts a valid Kinesis stream ARN', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream/my-stream')).toBe(true);
});

it('accepts a valid Lambda ARN', () => {
expect(isValidArn('arn:aws:lambda:us-east-1:123456789012:function:my-func')).toBe(true);
});

it('rejects a string that does not start with arn:', () => {
expect(isValidArn('not-an-arn')).toBe(false);
});

it('rejects an ARN with too few parts', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012')).toBe(false);
});

it('accepts an ARN with colons in the resource part', () => {
expect(isValidArn('arn:aws:kinesis:us-west-2:123456789012:stream:extra:parts')).toBe(true);
});

it('rejects an empty string', () => {
expect(isValidArn('')).toBe(false);
});
});
11 changes: 11 additions & 0 deletions src/cli/commands/shared/arn-utils.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
const ARN_PART_COUNT = 6;
const ARN_FORMAT = 'arn:partition:service:region:account:resource';

/**
* Check whether a string looks like a valid ARN (starts with `arn:` and has at least 6 colon-separated parts).
*/
export function isValidArn(value: string): boolean {
return value.startsWith('arn:') && value.split(':').length >= ARN_PART_COUNT;
}

export const ARN_VALIDATION_MESSAGE = `Must be a valid ARN (${ARN_FORMAT})`;
3 changes: 3 additions & 0 deletions src/cli/tui/hooks/useCreateMemory.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,6 +8,7 @@ interface CreateMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: { type: string }[];
streaming?: { dataStreamArn: string; contentLevel: string };
}

interface CreateStatus<T> {
Expand All@@ -27,6 +28,8 @@ export function useCreateMemory() {
name: config.name,
expiry: config.eventExpiryDuration,
strategies: strategiesStr || undefined,
dataStreamArn: config.streaming?.dataStreamArn,
contentLevel: config.streaming?.contentLevel,
});
if (!addResult.success) {
throw new Error(addResult.error ?? 'Failed to create memory');
Expand Down
92 changes: 81 additions & 11 deletions src/cli/tui/screens/memory/AddMemoryScreen.tsx
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
import type { MemoryStrategyType } from '../../../../schema';
import { AgentNameSchema } from '../../../../schema';
import { AgentNameSchema, StreamContentLevelSchema } from '../../../../schema';
import { ARN_VALIDATION_MESSAGE, isValidArn } from '../../../commands/shared/arn-utils';
import {
ConfirmReview,
Panel,
Expand All@@ -14,7 +15,7 @@ import { HELP_TEXT } from '../../constants';
import { useListNavigation, useMultiSelectNavigation } from '../../hooks';
import { generateUniqueName } from '../../utils';
import type { AddMemoryConfig } from './types';
import { EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { CONTENT_LEVEL_OPTIONS, EVENT_EXPIRY_OPTIONS, MEMORY_STEP_LABELS, MEMORY_STRATEGY_OPTIONS } from './types';
import { useAddMemoryWizard } from './useAddMemoryWizard';
import React, { useMemo } from 'react';

Expand All@@ -24,6 +25,11 @@ interface AddMemoryScreenProps {
existingMemoryNames: string[];
}

const STREAMING_OPTIONS: SelectableItem[] = [
{ id: 'no', title: 'No', description: 'No streaming' },
{ id: 'yes', title: 'Yes', description: 'Stream memory record events to a delivery target (e.g. Kinesis)' },
];

export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: AddMemoryScreenProps) {
const wizard = useAddMemoryWizard();

Expand All@@ -37,9 +43,17 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
[]
);

const contentLevelItems: SelectableItem[] = useMemo(
() => CONTENT_LEVEL_OPTIONS.map(opt => ({ id: opt.id, title: opt.title, description: opt.description })),
[]
);

const isNameStep = wizard.step === 'name';
const isExpiryStep = wizard.step === 'expiry';
const isStrategiesStep = wizard.step === 'strategies';
const isStreamingStep = wizard.step === 'streaming';
const isStreamArnStep = wizard.step === 'streamArn';
const isContentLevelStep = wizard.step === 'contentLevel';
const isConfirmStep = wizard.step === 'confirm';

const expiryNav = useListNavigation({
Expand All@@ -58,6 +72,20 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
requireSelection: false,
});

const streamingNav = useListNavigation({
items: STREAMING_OPTIONS,
onSelect: item => wizard.setStreamingEnabled(item.id === 'yes'),
onExit: () => wizard.goBack(),
isActive: isStreamingStep,
});

const contentLevelNav = useListNavigation({
items: contentLevelItems,
onSelect: item => wizard.setContentLevel(StreamContentLevelSchema.parse(item.id)),
onExit: () => wizard.goBack(),
isActive: isContentLevelStep,
});

useListNavigation({
items: [{ id: 'confirm', title: 'Confirm' }],
onSelect: () => onComplete(wizard.config),
Expand All@@ -67,16 +95,37 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add

const helpText = isStrategiesStep
? 'Space toggle · Enter confirm · Esc back'
: isExpiryStep
: isExpiryStep || isStreamingStep || isContentLevelStep
? HELP_TEXT.NAVIGATE_SELECT
: isConfirmStep
? HELP_TEXT.CONFIRM_CANCEL
: HELP_TEXT.TEXT_INPUT;

const headerContent = <StepIndicator steps={wizard.steps} currentStep={wizard.step} labels={MEMORY_STEP_LABELS} />;

const confirmFields = useMemo(
() => [
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
...(wizard.config.streaming
? [
{ label: 'Stream ARN', value: wizard.config.streaming.dataStreamArn },
{ label: 'Content Level', value: wizard.config.streaming.contentLevel },
]
: [{ label: 'Streaming', value: 'Disabled' }]),
],
[wizard.config]
);

return (
<Screen title="Add Memory" onExit={onExit} helpText={helpText} headerContent={headerContent}>
<Screen
title="Add Memory"
onExit={onExit}
helpText={helpText}
headerContent={headerContent}
exitEnabled={isNameStep}
>
<Panel>
{isNameStep && (
<TextInput
Expand DownExpand Up@@ -109,15 +158,36 @@ export function AddMemoryScreen({ onComplete, onExit, existingMemoryNames }: Add
/>
)}

{isConfirmStep && (
<ConfirmReview
fields={[
{ label: 'Name', value: wizard.config.name },
{ label: 'Event Expiry', value: `${wizard.config.eventExpiryDuration} days` },
{ label: 'Strategies', value: wizard.config.strategies.map(s => s.type).join(', ') || 'None' },
]}
{isStreamingStep && (
<WizardSelect
title="Enable memory record streaming?"
description="Stream memory record lifecycle events to a delivery target"
items={STREAMING_OPTIONS}
selectedIndex={streamingNav.selectedIndex}
/>
)}

{isStreamArnStep && (
<TextInput
key="streamArn"
prompt="Delivery target ARN (e.g. Kinesis stream)"
initialValue=""
onSubmit={wizard.setStreamArn}
onCancel={() => wizard.goBack()}
customValidation={value => isValidArn(value) || ARN_VALIDATION_MESSAGE}
/>
)}

{isContentLevelStep && (
<WizardSelect
title="Stream content level"
description="What data to include in stream events"
items={contentLevelItems}
selectedIndex={contentLevelNav.selectedIndex}
/>
)}

{isConfirmStep && <ConfirmReview fields={confirmFields} />}
</Panel>
</Screen>
);
Expand Down
22 changes: 20 additions & 2 deletions src/cli/tui/screens/memory/types.ts
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,35 @@
import type { MemoryStrategyType } from '../../../../schema';
import type { MemoryStrategyType, StreamContentLevel } from '../../../../schema';
import { MemoryStrategyTypeSchema } from '../../../../schema';

// ─────────────────────────────────────────────────────────────────────────────
// Memory Flow Types
// ─────────────────────────────────────────────────────────────────────────────

export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'confirm';
export type AddMemoryStep = 'name' | 'expiry' | 'strategies' | 'streaming' | 'streamArn' | 'contentLevel' | 'confirm';

export interface AddMemoryStrategyConfig {
type: MemoryStrategyType;
}

export interface AddMemoryStreamingConfig {
dataStreamArn: string;
contentLevel: StreamContentLevel;
}

export interface AddMemoryConfig {
name: string;
eventExpiryDuration: number;
strategies: AddMemoryStrategyConfig[];
streaming?: AddMemoryStreamingConfig;
}

export const MEMORY_STEP_LABELS: Record<AddMemoryStep, string> = {
name: 'Name',
expiry: 'Expiry',
strategies: 'Strategies',
streaming: 'Streaming',
streamArn: 'Stream ARN',
contentLevel: 'Content Level',
confirm: 'Confirm',
};

Expand DownExpand Up@@ -49,6 +58,15 @@ export const EVENT_EXPIRY_OPTIONS = [
{ id: 365, title: '365 days', description: 'Maximum retention' },
] as const;

export const CONTENT_LEVEL_OPTIONS = [
{ id: 'FULL_CONTENT' as const, title: 'Full content', description: 'Include memory record text in stream events' },
{
id: 'METADATA_ONLY' as const,
title: 'Metadata only',
description: 'Only include metadata (IDs, timestamps, namespaces)',
},
] as const satisfies readonly { id: StreamContentLevel; title: string; description: string }[];

// ─────────────────────────────────────────────────────────────────────────────
// Defaults
// ─────────────────────────────────────────────────────────────────────────────
Expand Down
Loading
Loading