Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

This PR adds AGENT_CONNECT_ prefix to environment variables in the agent_connect tool to avoid polluting global scope.

Changes:

  • Added AGENT_CONNECT_ prefix to all environment variables in agent_connect tool:
    • HOST_DOMAIN -> AGENT_CONNECT_HOST_DOMAIN
    • HOST_PORT -> AGENT_CONNECT_HOST_PORT
    • HOST_WS_PATH -> AGENT_CONNECT_HOST_WS_PATH
    • DID_DOCUMENT_PATH -> AGENT_CONNECT_DID_DOCUMENT_PATH
    • SSL_CERT_PATH -> AGENT_CONNECT_SSL_CERT_PATH
    • SSL_KEY_PATH -> AGENT_CONNECT_SSL_KEY_PATH
  • Updated error messages to reflect new environment variable names
  • Maintained framework-agnostic implementation

Link to Devin run: https://app.devin.ai/sessions/1cd424d834d04e048e72749570f90f27

devin-ai-integration bot and others added 5 commits December 18, 2024 23:52
Co-Authored-By: root@a10k.co <root@a10k.co>
Co-Authored-By: root@a10k.co <root@a10k.co>
- Port implementation from crewAI-tools
- Make functions framework-agnostic
- Remove unnecessary framework-specific files
- Update config.json with correct dependencies

Co-Authored-By: root@a10k.co <root@a10k.co>
- Add back cta field to indicate Composio's extensive tool offerings
- Update setup instructions to reference __init__.py

Co-Authored-By: root@a10k.co <root@a10k.co>
…NNECT_ prefix

Co-Authored-By: root@a10k.co <root@a10k.co>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.

@devin-ai-integration
Copy link
Contributor Author

Closing in favor of contributing directly to PR #76

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