Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

[AGENT-179] Call the agent from the corrent endpoint#294

Merged
pec1985 merged 6 commits intomainfrom
AGENT-179
May 16, 2025
Merged

[AGENT-179] Call the agent from the corrent endpoint#294
pec1985 merged 6 commits intomainfrom
AGENT-179

Conversation

@pec1985
Copy link
Copy Markdown
Contributor

@pec1985 pec1985 commented May 15, 2025

Comment thread internal/agent/agent.go Outdated
Comment thread cmd/agent.go Outdated
@pec1985 pec1985 requested review from jhaynie and potofpie and removed request for potofpie May 15, 2025 21:48
Copy link
Copy Markdown
Member

@jhaynie jhaynie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nits

Comment thread cmd/agent.go Outdated

if len(selectedAgent.Types) == 0 {
// error out
logger.Fatal("Agent %s has no running types (webhook or api)", selectedAgent.Name)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is fine but ideally use the tui.ShowError

Comment thread cmd/agent.go Outdated
if local {
port, _ := dev.FindAvailablePort(theproject)
endpoint = fmt.Sprintf("http://localhost:%d/agent_%s", port, agentID)
endpoint = fmt.Sprintf("http://localhost:%d/%s", port, agentID)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets use 127.0.0.1

@pec1985 pec1985 merged commit 0d1fd62 into main May 16, 2025
14 checks passed
@pec1985 pec1985 deleted the AGENT-179 branch May 16, 2025 08:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants