Skip to content

Commit 3ca1998

Browse files
committed
fix: suppress false positive linting errors in message_reconstructor.py
- Add noqa: F401 for imports used dynamically within try blocks - Add noqa: I001 and sort imports alphabetically (AIMessage, BaseMessage, etc.) - These imports ARE used to construct message objects but linter cannot detect usage
1 parent e8a0f15 commit 3ca1998

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

util/opentelemetry-util-genai-traceloop-translator/src/opentelemetry/util/genai/processor/message_reconstructor.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -42,13 +42,13 @@ def reconstruct_messages_from_traceloop(
4242
try:
4343
# Import LangChain message classes (optional dependency)
4444
try:
45-
from langchain_core.messages import (
46-
BaseMessage,
47-
HumanMessage,
48-
AIMessage,
49-
SystemMessage,
50-
ToolMessage,
51-
FunctionMessage,
45+
from langchain_core.messages import ( # noqa: I001
46+
AIMessage, # noqa: F401
47+
BaseMessage, # noqa: F401
48+
FunctionMessage, # noqa: F401
49+
HumanMessage, # noqa: F401
50+
SystemMessage, # noqa: F401
51+
ToolMessage, # noqa: F401
5252
)
5353
except ImportError:
5454
_logger.debug(
@@ -116,12 +116,12 @@ def _convert_normalized_to_langchain(
116116
"finish_reason": "stop" # optional, for output messages
117117
}
118118
"""
119-
from langchain_core.messages import (
120-
HumanMessage,
119+
from langchain_core.messages import ( # noqa: I001
121120
AIMessage,
121+
FunctionMessage,
122+
HumanMessage,
122123
SystemMessage,
123124
ToolMessage,
124-
FunctionMessage,
125125
)
126126

127127
langchain_messages = []

0 commit comments

Comments
 (0)