Skip to content

Feat/agentcore adapter#98

Open
rkmaws wants to merge 5 commits into
IABTechLab:mainfrom
rkmaws:feat/agentcore-adapter
Open

Feat/agentcore adapter#98
rkmaws wants to merge 5 commits into
IABTechLab:mainfrom
rkmaws:feat/agentcore-adapter

Conversation

@rkmaws

@rkmaws rkmaws commented Jun 2, 2026

Copy link
Copy Markdown

Implementation of AgentCore runtime deployment for the buyer agent

Ranjith Krishnamoorthy added 4 commits April 25, 2026 00:31
- .gitignore: add AgentCore artifacts (.bedrock_agentcore/, Dockerfile,
  *.db, .unique-id-*)
- pyproject.toml: register agentcore pytest marker
Add Amazon Bedrock AgentCore deployment interface for the buyer agent.
Wraps the existing DealBookingFlow and ChatInterface without modifying
community-maintained agent/crew code — all new files live in
src/ad_buyer/interfaces/agentcore/ and patches/.

Runtime architecture:
- HTTP mode: BedrockAgentCoreApp entrypoint with two routing paths
  - crew: DealBookingFlow with PortfolioCrew (Bedrock LLM) for
    campaign planning and budget allocation across channels
  - chat: existing ChatInterface keyword router
- Background FastAPI server on localhost for DealBookingFlow internals

Key components:
- http_main.py: AgentCore entrypoint, routing, crew invocation
- crew_tools.py: DealBookingFlow wrapper with prompt-to-brief parsing
- patches/crewai_bedrock_fix.py: Bedrock Converse API compatibility
- deploy.sh: Build and deploy via agentcore CLI with CodeBuild
- Campaign briefs: sample briefs for demo scenarios (JSON)

Tests: 52 unit + 3 integration
Docs: agentcore-deployment.md, agentcore.md (architecture), updated index.md
- Memory patch: replace StorageBackend with AgentCoreStorageBackend
- Set _read_only=True to prevent RememberTool injection (Nova Lite
  can't serialize the schema correctly, calls with empty {})
- RecallMemoryTool still active for cross-session context recall
- Memory LLM: Nova Lite (fast/cheap for query analysis)
- Crew LLM: Nova Pro (unchanged)
- Deploy script: Dockerfile workaround for agentcore toolkit
- 25 unit tests for the memory patch
- Remove unused http_entrypoint.py (deploy.sh uses http_main.py)
- Remove dead _format_crew_output and its tests
- Update test docstrings to reference http_main.py
- All 6 review items from issue IABTechLab#99 verified addressed in current code

Closes IABTechLab#99
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.

1 participant