Follow-up from #280 / #232 review (pre-existing gap, not a regression).
For ordinary comments (kind: "comment"), Python LinearAdapter.parse_message returns thread_id="" and a hardcoded "unknown" author (user_name/full_name), reading comment.userId.
Upstream parseMessage (chat@4.41.1 packages/adapter-linear/src/index.ts:2385-2427) encodes linear:{issueId}:c:{comment.id} and reads the structured comment.user (id, displayName, fullName, type === "bot").
Core does not call parse_message (the comment webhook builds messages in _build_message, comment fetch in _comment_node_to_message), so routing is unaffected; this only matters to consumers calling adapter.parse_message(raw) directly.
Suggested fix: encode the thread id from comment.issueId/comment.id, and read comment.user when present (falling back to userId for the flat webhook shape). Port upstream's ordinary-comment parseMessage test assertions on threadId/author.
Tracked as a known-divergence row in docs/UPSTREAM_SYNC.md ("Linear parse_message for ordinary comments").
Part of #184
Follow-up from #280 / #232 review (pre-existing gap, not a regression).
For ordinary comments (
kind: "comment"), PythonLinearAdapter.parse_messagereturnsthread_id=""and a hardcoded"unknown"author (user_name/full_name), readingcomment.userId.Upstream
parseMessage(chat@4.41.1packages/adapter-linear/src/index.ts:2385-2427) encodeslinear:{issueId}:c:{comment.id}and reads the structuredcomment.user(id,displayName,fullName,type === "bot").Core does not call
parse_message(the comment webhook builds messages in_build_message, comment fetch in_comment_node_to_message), so routing is unaffected; this only matters to consumers callingadapter.parse_message(raw)directly.Suggested fix: encode the thread id from
comment.issueId/comment.id, and readcomment.userwhen present (falling back touserIdfor the flat webhook shape). Port upstream's ordinary-comment parseMessage test assertions on threadId/author.Tracked as a known-divergence row in
docs/UPSTREAM_SYNC.md("Linearparse_messagefor ordinary comments").Part of #184