Skip to content

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Change agent lookup endpoint from GET to POST method

Summary

This PR changes the HTTP request method from GET to POST for the agent lookup endpoint in the Python SDK. This aligns with the backend API modification made in vlm-lab repository (PR #978) where the /agent/lookup endpoint was changed from GET to POST method.

Files Changed:

  • vlmrun/client/agent.py: Updated the get() method to use POST instead of GET for the agent lookup request

Impact: This is a breaking change that affects the client.agent.get() method calls, but the SDK interface remains unchanged from the user perspective.

Review & Testing Checklist for Human

  • Verify backend API is ready: Confirm that vlm-lab PR #978 has been deployed and the backend /agent/lookup endpoint now accepts POST requests
  • Test real API calls: Run integration tests or manual verification that client.agent.get(name="..."), client.agent.get(id="..."), and client.agent.get(prompt="...") work correctly against the live API
  • Check deployment timing: Ensure this SDK change is deployed after (or simultaneously with) the backend API change to avoid breaking existing functionality

Notes

This change aligns the Python SDK with the backend API modification
made in vlm-lab repository (PR #978) where the /lookup endpoint was
changed from GET to POST method.

Co-Authored-By: [email protected] <[email protected]>
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. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@spillai spillai merged commit 6e80f48 into main Aug 28, 2025
4 checks passed
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