Reviewing files that changed from the base of the PR and between df73667 and 122fcf5.

• client/transport/sse.go (1 hunks)
• client/transport/stdio.go (4 hunks)
• client/transport/streamable_http.go (1 hunks)
• mcp/types.go (2 hunks)
• mcp/utils.go (7 hunks)
• server/sse.go (2 hunks)
• server/stdio.go (1 hunks)

client/transport/streamable_http.go (1)

134-134: Good update to match Go documentation conventions.

The capitalization of "SendRequest" in the comment now correctly matches the exported method name, following Go conventions where documentation for exported identifiers should use the same capitalization.

mcp/utils.go (7)

128-129: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

159-160: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

182-183: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

191-192: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

200-201: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

210-211: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

---

385-386: Good documentation improvement.

Adding the function name as a comment header improves readability and follows good documentation practices.

client/transport/sse.go (1)

229-229: Good update to match Go documentation conventions.

The capitalization of "SendRequest" in the comment now correctly matches the exported method name, following Go conventions where documentation for exported identifiers should use the same capitalization.

server/stdio.go (1)

43-45: Good correction to reflect the actual function name.

Updated the comment to correctly reference "WithStdioContextFunc" instead of just "WithContextFunc", ensuring the documentation accurately describes the function it's documenting.

server/sse.go (2)

99-99: Good improvement to comment clarity.

Updated the comment to be more descriptive and consistent with the style of other option function comments.

---

153-154: Good correction to reflect the actual function name.

Updated the comment to correctly reference "WithSSEContextFunc" instead of just "WithContextFunc", ensuring the documentation accurately describes the function it's documenting.

mcp/types.go (2)

14-15: Excellent documentation improvement

These changes improve the godoc output by starting each method comment with the identifier name, which follows Go's documentation best practices. This makes the documentation clearer and more consistent.

Also applies to: 18-19, 22-23, 26-27, 30-31, 34-35, 38-39, 42-43, 46-47, 50-51, 56-57, 60-61

---

135-135: Good change to use pointer receiver for MarshalJSON

Changing from a value receiver to a pointer receiver for the MarshalJSON method is a good practice. This now matches the UnmarshalJSON method which also uses a pointer receiver, creating a consistent pattern. For marshaling methods, pointer receivers are generally preferred as they avoid unnecessary copying of the struct.

client/transport/stdio.go (4)

136-136: Whitespace cleanup

Removed trailing whitespace, improving code cleanliness.

---

151-152: Clarified handler behavior documentation

The updated comment now clearly indicates that only one notification handler can be active at a time, and setting a new one replaces any previous handler. This clarification helps prevent misunderstandings about how the handler registration works.

---

211-211: Capitalized method name in comment

Capitalized "SendRequest" in the method comment, following Go's convention of starting comments for exported identifiers with the identifier name. This improves documentation consistency.

---

264-264: Whitespace cleanup

Removed trailing whitespace, improving code cleanliness.