fix(agno): fix aresponse missing await and double wrapped() calls

[minimAluminiumalism]

Description

Fix issues in agno instrumentation wrapper:

1. aresponse(): Add missing await on early return path.
2. response_stream() & aresponse_stream(): Remove redundant wrapped() call that caused double API invocations

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

instrumentation-loongsuite/loongsuite-instrumentation-agno/tests/test_wrapper.py

• Test A

Does This PR Require a Core Repo Change?

• Yes. - Link to PR:
• No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

• Followed the style guidelines of this project
• Changelogs have been updated
• Unit tests have been added
• Documentation has been updated

[ralf0131]

Thanks for the contribution! Please fix the failing CI checks.

[Copilot]

Pull request overview

Fixes correctness issues in the Agno instrumentation wrapper where async responses could be returned without being awaited and streaming wrappers could invoke the wrapped callable twice.

Changes:

• Ensure AgnoModelWrapper.aresponse() awaits the wrapped coroutine on the early-return path.
• Remove redundant/double wrapped() invocation in response_stream() and aresponse_stream(), and adjust error handling.
• Add unit tests covering the above behaviors; ignore local pyrightconfig.json.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 6 comments.

File	Description
instrumentation-loongsuite/loongsuite-instrumentation-agno/src/opentelemetry/instrumentation/agno/_wrapper.py	Fixes missing await and removes double invocation in streaming wrappers.
instrumentation-loongsuite/loongsuite-instrumentation-agno/tests/test_wrapper.py	Adds regression tests for aresponse() awaiting and single-call streaming behavior.
.gitignore	Ignores local Pyright config file.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

response_stream wraps both user streaming and instrumentation finalization in the same try/except. As written, an internal failure during response attribute extraction or finish_tracing will be caught and re-raised to the caller, which differs from the established pattern in this module (non-stream and other stream wrappers log + suppress finalization errors to avoid breaking user code). Consider splitting the logic so exceptions from wrapped/iteration propagate, but extractor/finalization exceptions are logged/suppressed while still ending the span.

[Copilot]

This test monkey-patches AgnoModelWrapper.response_stream globally and restores it only on the success path. If the test fails/raises before restoration, it can leak the patched method into other tests. Use try/finally around the body or pytest’s monkeypatch fixture to guarantee cleanup.