Skip to content

Conversation

hyprh
Copy link
Contributor

@hyprh hyprh commented Oct 21, 2025

@hyprh hyprh changed the title ageng: resolve agent card by GetAgentCard agent: resolve agent card by GetAgentCard Oct 21, 2025
@codecov
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.12094%. Comparing base (90234fc) to head (0a9d287).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
agent/a2aagent/a2a_agent.go 75.60976% 6 Missing and 4 partials ⚠️
agent/a2aagent/a2a_agent_option.go 0.00000% 4 Missing ⚠️
server/a2a/server.go 95.00000% 1 Missing ⚠️
Additional details and impacted files
@@                 Coverage Diff                 @@
##                main        #511         +/-   ##
===================================================
- Coverage   78.13849%   78.12094%   -0.01755%     
===================================================
  Files            195         196          +1     
  Lines          29879       29983        +104     
===================================================
+ Hits           23347       23423         +76     
- Misses          5398        5420         +22     
- Partials        1134        1140          +6     
Flag Coverage Δ
unittests 78.12094% <80.00000%> (-0.01755%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hyprh hyprh changed the title agent: resolve agent card by GetAgentCard ageng: resolve agent card by a2a.Client Oct 21, 2025
@sandyskies
Copy link
Contributor

Ageng-》
Agent

@hyprh hyprh changed the title ageng: resolve agent card by a2a.Client agent: resolve agent card by a2a.Client Oct 21, 2025
@hyprh
Copy link
Contributor Author

hyprh commented Oct 21, 2025

Ageng-》 Agent

done,可以先过一下这个 https://github.com/trpc-group/trpc-a2a-go/pull/96,需要先合并这个

}

if options.host == "" {
return nil, errors.New("host is required")
Copy link
Contributor

Choose a reason for hiding this comment

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

如果已经有 agent card,这里对 host 的检查是否过于严格?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

return nil, fmt.Errorf("failed to resolve agent card: %w", err)
}
agent.agentCard = agentCard
// Determine the agent URL and normalize it
Copy link
Contributor

Choose a reason for hiding this comment

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

好像没有 normalize。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

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.

3 participants