Skip to content

Conversation

@aphilippartd
Copy link

Issue #, if available:

Description of changes: Example of implementing a TS strands agent with AgentCore memory for long and short term memory.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link

Latest scan for commit: 788343e | Updated: 2025-12-19 22:50:12 UTC

✅ Security Scan Report (PR Files Only)

Scanned Files

  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/.gitignore
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/README.md
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/images/architecture.png
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/images/create-memory.png
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/package.json
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/src/agent.ts
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/src/database.ts
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/src/index.ts
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/src/memory.ts
  • typescript/01-tutorials/01-fundamentals/07-agent-core-memory/tsconfig.json

Security Scan Results

Critical High Medium Low Info
0 0 0 0 0

Threshold: High

No security issues detected in your changes. Great job!

This scan only covers files changed in this PR.

@cagataycali
Copy link
Member

🤖 PR Review: Agent with AgentCore Memory Tutorial

Check Status
CI Status ✅ SUCCESS
Mergeable ✅ No conflicts
Change Scope +490 lines, 10 files

Analysis:

This is a valuable tutorial demonstrating AgentCore memory integration for TypeScript Strands agents.

Value Add:

  1. Long-term memory - Essential for production agents that need context persistence
  2. Short-term memory - Useful for conversation context within sessions
  3. Clear example - 10 files with 490 lines of well-structured example code

Notes:

  • CI is passing ✅
  • No conflicts with main branch
  • Adds useful sample for the community

Recommendation: This looks ready for maintainer review. Great contribution! 🚀


Review by strands-coder autonomous agent 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants