Skip to content

fix: added agent ID to all templates json#76

Merged
sawradip merged 1 commit into
mainfrom
sawra/agent-id-to-template
Oct 28, 2025
Merged

fix: added agent ID to all templates json#76
sawradip merged 1 commit into
mainfrom
sawra/agent-id-to-template

Conversation

@sawradip
Copy link
Copy Markdown
Contributor

@sawradip sawradip commented Oct 28, 2025

Summary by CodeRabbit

  • Configuration Updates

    • Added agent_id configuration field across all agent templates.
    • Removed environment variable configurations from OpenAI and Parlant templates.
  • Changes

    • Deprecated LlamaIndex template exports.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 28, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

A new agent_id field with a UUID value is added to multiple runagent.config.json template files across different frameworks. In three templates (OpenAI and Parlant), the env_vars configuration blocks are removed alongside this addition. Additionally, two LlamaIndex template exports are removed from the package's __init__.py.

Changes

Cohort / File(s) Summary
Agent ID Field Addition Across Templates
templates/ag2/default/runagent.config.json, templates/agno/default/runagent.config.json, templates/autogen/default/runagent.config.json, templates/crewai/default/runagent.config.json, templates/default/default/runagent.config.json, templates/langchain/chatbot/runagent.config.json, templates/langchain/default/runagent.config.json, templates/langgraph/*/runagent.config.json, templates/letta/*/runagent.config.json, templates/llamaindex/default/runagent.config.json, templates/n8n/default/runagent.config.json
Added new top-level agent_id field with placeholder UUID value "00000000-0000-0000-0000-000000000000" to JSON configuration across multiple template directories. Trailing commas adjusted to maintain valid JSON structure.
Agent ID Addition with Environment Variable Removal
templates/openai/default/runagent.config.json, templates/openai/email_agent/runagent.config.json, templates/parlant/default/runagent.config.json
Removed env_vars blocks containing API keys and environment configuration, and added new top-level agent_id field. Replaces environment-based configuration with explicit agent identifier.
Template Exports Cleanup
templates/llamaindex/__init__.py
Removed public exports: LlamaIndexAdvancedTemplate and LlamaIndexBasicTemplate.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Homogeneous changes across most files (repetitive agent_id field additions to config files)
  • Consistent pattern makes verification straightforward once understood
  • Minimal variations: three files include env_vars removals; one file involves import removals
  • Areas for extra attention:
    • templates/openai/* and templates/parlant/default/runagent.config.json — verify that removing env_vars does not break intended configuration semantics
    • templates/llamaindex/__init__.py — confirm that removing these exports does not impact public API or downstream dependencies

Possibly related PRs

Poem

🐰 A UUID springs forth, a badge so new,
Each template blooms with identity true.
Old env keys fade, their work is done,
Agent IDs rise—configurations, one by one!
Say hello to order in the config sun! 🌞

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sawra/agent-id-to-template

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1fa6e81 and f5eeaeb.

📒 Files selected for processing (19)
  • templates/ag2/default/runagent.config.json (1 hunks)
  • templates/agno/default/runagent.config.json (1 hunks)
  • templates/autogen/default/runagent.config.json (1 hunks)
  • templates/crewai/default/runagent.config.json (1 hunks)
  • templates/default/default/runagent.config.json (1 hunks)
  • templates/langchain/chatbot/runagent.config.json (1 hunks)
  • templates/langchain/default/runagent.config.json (1 hunks)
  • templates/langgraph/advanced/runagent.config.json (1 hunks)
  • templates/langgraph/default/runagent.config.json (1 hunks)
  • templates/langgraph/problem_solver_advanced/runagent.config.json (1 hunks)
  • templates/letta/advanced/runagent.config.json (1 hunks)
  • templates/letta/default/runagent.config.json (1 hunks)
  • templates/letta/rag/runagent.config.json (1 hunks)
  • templates/llamaindex/__init__.py (0 hunks)
  • templates/llamaindex/default/runagent.config.json (1 hunks)
  • templates/n8n/default/runagent.config.json (1 hunks)
  • templates/openai/default/runagent.config.json (1 hunks)
  • templates/openai/email_agent/runagent.config.json (1 hunks)
  • templates/parlant/default/runagent.config.json (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sawradip sawradip merged commit 5d8b0aa into main Oct 28, 2025
1 check was pending
@coderabbitai coderabbitai Bot mentioned this pull request Dec 15, 2025
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