chore: code review comments

Signed-off-by: Tomas Weiss <tomas.weiss2@gmail.com>

Author: tomkis

apps/agentstack-cli/src/agentstack_cli/commands/agent.py

@@ -1209,9 +1209,9 @@ async def list_feedback(
             rating_icon = "✓" if item.rating == 1 else "✗"
             agent_name = item.agent_name or str(item.provider_id)[:8]
             task_id_short = str(item.task_id)[:8]
-            comment = (item.comment or "")[:50]
-            if len(item.comment or "") > 50:
-                comment += "..."
+            comment = item.comment or ""
+            if len(comment) > 50:
+                comment = comment[:50] + "..."
             tags = ", ".join(item.comment_tags or []) if item.comment_tags else "-"
             created_at = item.created_at.strftime("%Y-%m-%d")
 

apps/agentstack-server/src/agentstack_server/api/routes/user_feedback.py

@@ -54,21 +54,7 @@ async def list_user_feedback(
         after_cursor=after_cursor,
     )
     return ListUserFeedbackResponse(
-        items=[
-            UserFeedbackResponse(
-                id=feedback.id,
-                provider_id=feedback.provider_id,
-                task_id=feedback.task_id,
-                context_id=feedback.context_id,
-                rating=feedback.rating,
-                message=feedback.message,
-                comment=feedback.comment,
-                comment_tags=feedback.comment_tags,
-                created_at=feedback.created_at,
-                agent_name=feedback.agent_name,
-            )
-            for feedback in feedback_list
-        ],
+        items=[UserFeedbackResponse.model_validate(dict(feedback)) for feedback in feedback_list],
         total_count=total,
         has_more=has_more,
     )

apps/agentstack-server/src/agentstack_server/infrastructure/persistence/repositories/user_feedback.py

@@ -83,22 +83,6 @@ async def list(
             order="desc",
         )
 
-        feedback_list = [
-            UserFeedback(
-                id=row.id,
-                provider_id=row.provider_id,
-                task_id=row.task_id,
-                context_id=row.context_id,
-                trace_id=row.trace_id,
-                rating=row.rating,
-                message=row.message,
-                comment=row.comment,
-                comment_tags=row.comment_tags,
-                created_at=row.created_at,
-                created_by=row.created_by,
-                agent_name=row.agent_name,
-            )
-            for row in result.items
-        ]
+        feedback_list = [UserFeedback.model_validate(dict(row._mapping)) for row in result.items]
 
         return feedback_list, result.total_count, result.has_more

docs/development/agent-integration/mcp.mdx

@@ -5,7 +5,7 @@ description: "Learn how Agent Stack helps with authentication when using MCP."
 
 ## What are Connectors?
 
-A connector is an MCP endpoint exposed by the Agent Stack that forwards MCP requests to a remote MCP server, acting as a proxy and handling authenticationon.
+A connector is an MCP endpoint exposed by the Agent Stack that forwards MCP requests to a remote MCP server, acting as a proxy and handling authentication.
 
 It allows users to set up a connection to a third-party service (like GitHub, Box etc.) and then exposes a local MCP server that can be consumed by agents. This architecture eliminates the need for user authentication in your agent code. The connector handles authentication with the third-party service on behalf of the authenticated user, and the MCP interface is properly scoped to that user's permissions and data.