What the developer wrote
Authored by Roman Zeyde
docs(core): add `ThpContext` docstring
[no changelog]
What changed, and why it matters
This commit only adds a plain-language comment (docstring) explaining what the ThpContext class does. No code behavior was changed, so it cannot introduce or fix a security issue on its own.
No security action needed; this is a documentation-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds a class-level docstring to ThpContext in core/src/trezor/wire/thp/interface_context.py describing its role in receiving THP data from multiple wire interfaces and handling low-level single-packet THP messages. There are no functional code changes.
Changed components
core/src/trezor/wire/thp/interface_context.pyInspect captured patch +5 / −0
diff --git a/core/src/trezor/wire/thp/interface_context.py b/core/src/trezor/wire/thp/interface_context.py
index 328f27dfc..78968620e 100644
--- a/core/src/trezor/wire/thp/interface_context.py
+++ b/core/src/trezor/wire/thp/interface_context.py
@@ -43,6 +43,11 @@ def _timeout_after(ms: int) -> Generator[sleep, int, NoReturn]:
class ThpContext:
+ """
+ This class handles THP receiving from multiple wire interfaces.
+ It also handles and responds to low-level single packet THP messages, creating new channels if needed.
+ """
+
def __init__(self, *ifaces: WireInterface) -> None:
max_packet_len = max(iface.RX_PACKET_LEN for iface in ifaces)
self._packet_buf = bytearray(max_packet_len)
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.