Merge pull request #70 from Point72/tkp/embed

Clean up comments

Author: timkpaine

chatom/base/message.py

@@ -600,9 +600,7 @@ def forwarded_from_id(self) -> str:
         """
         return self.forwarded_from.id if self.forwarded_from else ""
 
-    # -------------------------------------------------------------------------
     # Response Convenience Methods
-    # -------------------------------------------------------------------------
     # These methods construct new Message instances based on this message.
     # They make it easy to create replies, forwards, or quotes.
 

chatom/format/embed.py

@@ -38,17 +38,13 @@ class FormattedEmbed(BaseModel):
 
     embed: Embed = Field(default_factory=Embed, description="The underlying embed data.")
 
-    # ------------------------------------------------------------------
     # Convenience constructors
-    # ------------------------------------------------------------------
     @classmethod
     def from_embed(cls, embed: Embed) -> "FormattedEmbed":
         """Create a FormattedEmbed from an existing Embed instance."""
         return cls(embed=embed)
 
-    # ------------------------------------------------------------------
     # Text fallback rendering (for content list)
-    # ------------------------------------------------------------------
     def render(self, format: FORMAT = Format.MARKDOWN) -> str:
         """Render the embed as a text fallback.
 
@@ -150,9 +146,7 @@ def _render_footer(self, fmt: Format) -> str:
             return f"_{footer.text}_"
         return footer.text
 
-    # ------------------------------------------------------------------
     # Structured per-backend payloads
-    # ------------------------------------------------------------------
     def to_discord_dict(self) -> Dict[str, Any]:
         """Convert to a Discord embed dict suitable for the API."""
         e = self.embed

chatom/tests/test_format.py

@@ -2482,11 +2482,6 @@ def test_action_row_with_select_symphony(self):
         assert "<select" in result["messageml"]
 
 
-# -------------------------------------------------------------------------
-# Phase 1: Embed / FormattedEmbed tests
-# -------------------------------------------------------------------------
-
-
 class TestFormattedEmbed:
     """Tests for the FormattedEmbed content node."""
 
@@ -2507,8 +2502,6 @@ def _make_embed(self):
             ],
         )
 
-    # -- text fallback rendering --
-
     def test_render_markdown(self):
         fe = FormattedEmbed(embed=self._make_embed())
         rendered = fe.render(Format.MARKDOWN)
@@ -2539,8 +2532,6 @@ def test_render_plaintext(self):
         rendered = fe.render(Format.PLAINTEXT)
         assert "Uptime: 3d 12h" in rendered
 
-    # -- structured payloads --
-
     def test_to_discord_dict(self):
         fe = FormattedEmbed(embed=self._make_embed())
         d = fe.to_discord_dict()
@@ -2581,8 +2572,6 @@ def test_to_telegram_html(self):
         assert "<b>" in html
         assert "Bot Status" in html
 
-    # -- from_embed factory --
-
     def test_from_embed(self):
         from chatom.base.embed import Embed
 

chatom/tests/test_mcp.py

@@ -16,10 +16,6 @@
 )
 from chatom.mcp import build_mcp_server
 
-# ---------------------------------------------------------------------------
-# Minimal mock backend (same pattern as chatom/tests/test_agent.py)
-# ---------------------------------------------------------------------------
-
 
 class _MockBackend:
     """Bare-bones BackendBase stand-in for MCP server tests."""
@@ -133,11 +129,6 @@ async def add_reaction(
         self.reactions.append({"message_id": msg_id, "emoji": emoji, "channel": ch_id})
 
 
-# ---------------------------------------------------------------------------
-# Fixtures
-# ---------------------------------------------------------------------------
-
-
 @pytest.fixture
 def alice() -> User:
     return User(id="U1", name="Alice", display_name="Alice A")
@@ -177,11 +168,6 @@ def mock_backend(alice: User, general: Channel, sample_messages: List[Message])
     )
 
 
-# ---------------------------------------------------------------------------
-# Server builder tests
-# ---------------------------------------------------------------------------
-
-
 class TestBuildMcpServer:
     """Tests for build_mcp_server via the Client (avoids accessing internals)."""
 
@@ -230,11 +216,6 @@ async def test_capability_gating(self) -> None:
             assert "read_channel_history" in tool_names
 
 
-# ---------------------------------------------------------------------------
-# MCP Client integration tests
-# ---------------------------------------------------------------------------
-
-
 class TestMcpClientIntegration:
     """Test tools via FastMCP Client (in-process, no subprocess)."""