refactor(mixins): add transformation mixins with _transform suffix

- Create goals_transform.py, meetings_transform.py, issues_transform.py
- Create headlines_transform.py, todos_transform.py
- Rename users.py to users_transform.py for consistency
- Update mixins/__init__.py exports

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: franccesco

src/bloomy/operations/mixins/__init__.py

@@ -1,7 +1,17 @@
 """Mixins for shared operation logic."""
 
-from .users import UserOperationsMixin
+from .goals_transform import GoalOperationsMixin
+from .headlines_transform import HeadlineOperationsMixin
+from .issues_transform import IssueOperationsMixin
+from .meetings_transform import MeetingOperationsMixin
+from .todos_transform import TodoOperationsMixin
+from .users_transform import UserOperationsMixin
 
 __all__ = [
+    "GoalOperationsMixin",
+    "HeadlineOperationsMixin",
+    "IssueOperationsMixin",
+    "MeetingOperationsMixin",
+    "TodoOperationsMixin",
     "UserOperationsMixin",
 ]

src/bloomy/operations/mixins/goals_transform.py

@@ -0,0 +1,137 @@
+"""Mixin for shared goal operations logic."""
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, Any
+
+from ...models import (
+    ArchivedGoalInfo,
+    CreatedGoalInfo,
+    GoalInfo,
+    GoalStatus,
+)
+
+if TYPE_CHECKING:
+    from collections.abc import Sequence
+
+
+class GoalOperationsMixin:
+    """Shared logic for goal operations."""
+
+    def _transform_goal_list(self, data: Sequence[dict[str, Any]]) -> list[GoalInfo]:
+        """Transform API response to list of GoalInfo models.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A list of GoalInfo models.
+
+        """
+        return [
+            GoalInfo(
+                id=goal["Id"],
+                user_id=goal["Owner"]["Id"],
+                user_name=goal["Owner"]["Name"],
+                title=goal["Name"],
+                created_at=goal["CreateTime"],
+                due_date=goal["DueDate"],
+                status="Completed" if goal.get("Complete") else "Incomplete",
+                meeting_id=goal["Origins"][0]["Id"] if goal.get("Origins") else None,
+                meeting_title=(
+                    goal["Origins"][0]["Name"] if goal.get("Origins") else None
+                ),
+            )
+            for goal in data
+        ]
+
+    def _transform_archived_goals(
+        self, data: Sequence[dict[str, Any]]
+    ) -> list[ArchivedGoalInfo]:
+        """Transform API response to list of ArchivedGoalInfo models.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A list of ArchivedGoalInfo models.
+
+        """
+        return [
+            ArchivedGoalInfo(
+                id=goal["Id"],
+                title=goal["Name"],
+                created_at=goal["CreateTime"],
+                due_date=goal["DueDate"],
+                status="Complete" if goal.get("Complete") else "Incomplete",
+            )
+            for goal in data
+        ]
+
+    def _transform_created_goal(
+        self, data: dict[str, Any], title: str, meeting_id: int, user_id: int
+    ) -> CreatedGoalInfo:
+        """Transform API response to CreatedGoalInfo model.
+
+        Args:
+            data: The raw API response data.
+            title: The title of the goal.
+            meeting_id: The ID of the meeting associated with the goal.
+            user_id: The ID of the user responsible for the goal.
+
+        Returns:
+            A CreatedGoalInfo model.
+
+        """
+        # Map completion status
+        completion_map = {2: "complete", 1: "on", 0: "off"}
+        status = completion_map.get(data.get("Completion", 0), "off")
+
+        return CreatedGoalInfo(
+            id=data["Id"],
+            user_id=user_id,
+            user_name=data["Owner"]["Name"],
+            title=title,
+            meeting_id=meeting_id,
+            meeting_title=data["Origins"][0]["Name"],
+            status=status,
+            created_at=data["CreateTime"],
+        )
+
+    def _build_goal_update_payload(
+        self,
+        accountable_user: int,
+        title: str | None = None,
+        status: GoalStatus | str | None = None,
+    ) -> dict[str, Any]:
+        """Build payload for goal update operation.
+
+        Args:
+            accountable_user: The ID of the user responsible for the goal.
+            title: The new title of the goal.
+            status: The status value (GoalStatus enum or string).
+
+        Returns:
+            A dictionary containing the update payload.
+
+        Raises:
+            ValueError: If an invalid status value is provided.
+
+        """
+        payload: dict[str, Any] = {"accountableUserId": accountable_user}
+
+        if title is not None:
+            payload["title"] = title
+
+        if status is not None:
+            valid_status = {"on": "OnTrack", "off": "AtRisk", "complete": "Complete"}
+            # Handle both GoalStatus enum and string
+            status_value = status.value if isinstance(status, GoalStatus) else status
+            status_key = status_value.lower()
+            if status_key not in valid_status:
+                raise ValueError(
+                    "Invalid status value. Must be 'on', 'off', or 'complete'."
+                )
+            payload["completion"] = valid_status[status_key]
+
+        return payload

src/bloomy/operations/mixins/headlines_transform.py

@@ -0,0 +1,77 @@
+"""Mixin for shared headline operations logic."""
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, Any
+
+from ...models import (
+    HeadlineDetails,
+    HeadlineListItem,
+    MeetingInfo,
+    OwnerDetails,
+)
+
+if TYPE_CHECKING:
+    from collections.abc import Sequence
+
+
+class HeadlineOperationsMixin:
+    """Shared logic for headline operations."""
+
+    def _transform_headline_details(self, data: dict[str, Any]) -> HeadlineDetails:
+        """Transform API response to HeadlineDetails model.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A HeadlineDetails model.
+
+        """
+        return HeadlineDetails(
+            id=data["Id"],
+            title=data["Name"],
+            notes_url=data["DetailsUrl"],
+            meeting_details=MeetingInfo(
+                id=data["OriginId"],
+                title=data["Origin"],
+            ),
+            owner_details=OwnerDetails(
+                id=data["Owner"]["Id"],
+                name=data["Owner"]["Name"],
+            ),
+            archived=data["Archived"],
+            created_at=data["CreateTime"],
+            closed_at=data["CloseTime"],
+        )
+
+    def _transform_headline_list(
+        self, data: Sequence[dict[str, Any]]
+    ) -> list[HeadlineListItem]:
+        """Transform API response to list of HeadlineListItem models.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A list of HeadlineListItem models.
+
+        """
+        return [
+            HeadlineListItem(
+                id=headline["Id"],
+                title=headline["Name"],
+                meeting_details=MeetingInfo(
+                    id=headline["OriginId"],
+                    title=headline["Origin"],
+                ),
+                owner_details=OwnerDetails(
+                    id=headline["Owner"]["Id"],
+                    name=headline["Owner"]["Name"],
+                ),
+                archived=headline["Archived"],
+                created_at=headline["CreateTime"],
+                closed_at=headline["CloseTime"],
+            )
+            for headline in data
+        ]

src/bloomy/operations/mixins/issues_transform.py

@@ -0,0 +1,83 @@
+"""Mixin for shared issue operations logic."""
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, Any
+
+from ...models import (
+    CreatedIssue,
+    IssueDetails,
+    IssueListItem,
+)
+
+if TYPE_CHECKING:
+    from collections.abc import Sequence
+
+
+class IssueOperationsMixin:
+    """Shared logic for issue operations."""
+
+    def _transform_issue_details(self, data: dict[str, Any]) -> IssueDetails:
+        """Transform API response to IssueDetails model.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            An IssueDetails model.
+
+        """
+        return IssueDetails(
+            id=data["Id"],
+            title=data["Name"],
+            notes_url=data["DetailsUrl"],
+            created_at=data["CreateTime"],
+            completed_at=data["CloseTime"],
+            meeting_id=data["OriginId"],
+            meeting_title=data["Origin"],
+            user_id=data["Owner"]["Id"],
+            user_name=data["Owner"]["Name"],
+        )
+
+    def _transform_issue_list(
+        self, data: Sequence[dict[str, Any]]
+    ) -> list[IssueListItem]:
+        """Transform API response to list of IssueListItem models.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A list of IssueListItem models.
+
+        """
+        return [
+            IssueListItem(
+                id=issue["Id"],
+                title=issue["Name"],
+                notes_url=issue["DetailsUrl"],
+                created_at=issue["CreateTime"],
+                meeting_id=issue["OriginId"],
+                meeting_title=issue["Origin"],
+            )
+            for issue in data
+        ]
+
+    def _transform_created_issue(self, data: dict[str, Any]) -> CreatedIssue:
+        """Transform API response to CreatedIssue model.
+
+        Args:
+            data: The raw API response data.
+
+        Returns:
+            A CreatedIssue model.
+
+        """
+        return CreatedIssue(
+            id=data["Id"],
+            meeting_id=data["OriginId"],
+            meeting_title=data["Origin"],
+            title=data["Name"],
+            user_id=data["Owner"]["Id"],
+            notes_url=data["DetailsUrl"],
+        )