diff --git a/agent-teams-controller/src/internal/review.js b/agent-teams-controller/src/internal/review.js index b2d86afe..65cac819 100644 --- a/agent-teams-controller/src/internal/review.js +++ b/agent-teams-controller/src/internal/review.js @@ -82,7 +82,7 @@ function requestReview(context, taskId, flags = {}) { to: reviewer, from, text: - `Please review task #${task.displayId || task.id}.\n\n` + + `**Please review** task #${task.displayId || task.id}\n\n` + wrapAgentBlock( `When approved, use MCP tool review_approve:\n` + `{ teamName: "${context.teamName}", taskId: "${task.id}", notifyOwner: true }\n\n` + @@ -140,8 +140,8 @@ function approveReview(context, taskId, flags = {}) { from, text: note && note !== 'Approved' - ? `Task #${task.displayId || task.id} approved.\n\n${note}` - : `Task #${task.displayId || task.id} approved.`, + ? `@${from} **approved** task #${task.displayId || task.id}\n\n${note}` + : `@${from} **approved** task #${task.displayId || task.id}`, summary: `Approved #${task.displayId || task.id}`, source: 'system_notification', ...(leadSessionId ? { leadSessionId } : {}), @@ -192,7 +192,7 @@ function requestChanges(context, taskId, flags = {}) { to: task.owner, from, text: - `Task #${task.displayId || task.id} needs fixes.\n\n${comment}\n\n` + + `@${from} **requested changes** for task #${task.displayId || task.id}\n\n${comment}\n\n` + 'The task has been moved back to pending. When you are ready to resume, review the task context, start it explicitly, implement the fixes, mark it completed, and request review again.', ...(Array.isArray(flags.taskRefs) ? { taskRefs: flags.taskRefs } : {}), summary: `Fix request for #${task.displayId || task.id}`, diff --git a/src/renderer/components/team/dialogs/TaskCommentsSection.tsx b/src/renderer/components/team/dialogs/TaskCommentsSection.tsx index 949f2e12..f5a0d54e 100644 --- a/src/renderer/components/team/dialogs/TaskCommentsSection.tsx +++ b/src/renderer/components/team/dialogs/TaskCommentsSection.tsx @@ -210,7 +210,7 @@ export const TaskCommentsSection = ({
) : ( { diff --git a/src/renderer/components/team/dialogs/TaskDetailDialog.tsx b/src/renderer/components/team/dialogs/TaskDetailDialog.tsx index 1464765f..989f2823 100644 --- a/src/renderer/components/team/dialogs/TaskDetailDialog.tsx +++ b/src/renderer/components/team/dialogs/TaskDetailDialog.tsx @@ -566,6 +566,26 @@ export const TaskDetailDialog = ({ Unassigned )}
+ {currentTask.reviewer || + (currentTask.reviewState && currentTask.reviewState !== 'none') ? ( +
+ + {currentTask.reviewer ? ( + + ) : null} + {currentTask.reviewState && currentTask.reviewState !== 'none' ? ( + + {REVIEW_STATE_DISPLAY[currentTask.reviewState].label} + + ) : null} +
+ ) : null} {currentTask.createdBy ? (
@@ -688,451 +708,462 @@ export const TaskDetailDialog = ({
) : null} - {/* Description */} - } - contentClassName="pl-2.5" - headerClassName="-mx-6 w-[calc(100%+3rem)]" - headerContentClassName="pl-6" - defaultOpen - > - {editingDescription ? ( -
-
- - -
- {descriptionPreview ? ( -
- {descriptionDraft.trim() ? ( - - ) : ( -

Nothing to preview

- )} -
- ) : ( -