+
{item.actor.displayName}
diff --git a/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx b/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
index 8a8ff5713..3011868cb 100644
--- a/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
+++ b/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
@@ -81,7 +81,7 @@ const TeamInboxDetailLayout: React.FC = ({
{children ? (
{children}
) : null}
-
+
diff --git a/src/modules/MainApp/TeamInbox/components/TeamInboxList.tsx b/src/modules/MainApp/TeamInbox/components/TeamInboxList.tsx
index 53ea98f58..0d346f1cb 100644
--- a/src/modules/MainApp/TeamInbox/components/TeamInboxList.tsx
+++ b/src/modules/MainApp/TeamInbox/components/TeamInboxList.tsx
@@ -1,4 +1,4 @@
-import { AtSign, CheckCheck, ClipboardList, Inbox } from "lucide-react";
+import { AtSign, ClipboardList, Inbox } from "lucide-react";
import React, { useCallback, useMemo, useRef } from "react";
import { useTranslation } from "react-i18next";
@@ -12,9 +12,6 @@ import TabPill, { type TabPillItem } from "@src/components/TabPill";
import {
ListPanelScrollArea,
ListPanelTabPillRow,
- PANEL_HEADER_TOKENS,
- PanelHeader,
- PanelRefreshButton,
Placeholder,
} from "@src/modules/shared/layouts/blocks";
@@ -31,15 +28,11 @@ export interface TeamInboxListProps {
filter: TeamInboxFilter;
items: readonly TeamInboxItem[];
selectedItemId: string | null;
- totalUnread: number;
unreadCounts: TeamInboxUnreadCounts;
query: string;
- loading: boolean;
onQueryChange: (query: string) => void;
onFilterChange: (filter: TeamInboxFilter) => void;
onSelectItem: (item: TeamInboxItem) => void;
- onRefresh?: () => void;
- onMarkAllRead?: () => void;
hasMore?: boolean;
loadingMore?: boolean;
onLoadMore?: () => void;
@@ -61,15 +54,11 @@ const TeamInboxList: React.FC
= ({
filter,
items,
selectedItemId,
- totalUnread,
unreadCounts,
query,
- loading,
onQueryChange,
onFilterChange,
onSelectItem,
- onRefresh,
- onMarkAllRead,
hasMore = false,
loadingMore = false,
onLoadMore,
@@ -82,11 +71,7 @@ const TeamInboxList: React.FC = ({
items.findIndex((item) => getTeamInboxItemKey(item) === selectedItemId),
[items, selectedItemId]
);
- const groups = useMemo(
- () => groupTeamInboxItemsByRecency(items, Date.now()),
- [items]
- );
- const activeFilterUnread = unreadCounts[filter];
+ const groups = useMemo(() => groupTeamInboxItemsByRecency(items), [items]);
const filterTabs = useMemo(
() => [
{
@@ -162,41 +147,6 @@ const TeamInboxList: React.FC = ({
className="flex h-full min-h-0 flex-col"
aria-label={t("teamInbox.listLabel")}
>
- 0
- ? t("teamInbox.unreadCount", { count: totalUnread })
- : t("teamInbox.allRead")
- }
- variant="list"
- actions={
- <>
- {activeFilterUnread > 0 && onMarkAllRead ? (
-
- }
- title={t("inbox.markAllAsRead")}
- aria-label={t("inbox.markAllAsRead")}
- onClick={onMarkAllRead}
- />
- ) : null}
- {onRefresh ? (
-
- ) : null}
-
- }
- />
-
(
({ item, itemKey, selected, onSelect }, ref) => {
const { t } = useTranslation();
const isMention = item.kind === "comment_mention";
+ const isGitHubIssue =
+ item.kind === "assigned_work_item" &&
+ isGitHubIssueStatus(item.payload.status);
const title = isMention ? item.target.sessionTitle : item.payload.title;
const summary = useMemo(() => {
if (item.kind === "comment_mention") return item.payload.commentBody;
@@ -61,10 +66,22 @@ const TeamInboxRow = forwardRef(
onClick={() => onSelect(item)}
>
- {isMention ? : }
+ {isMention ? (
+
+ ) : isGitHubIssue ? (
+
+ ) : (
+
+ )}
@@ -83,7 +100,7 @@ const TeamInboxRow = forwardRef(
{relativeTime}
-
+
{summary}
diff --git a/src/modules/MainApp/TeamInbox/domain/index.ts b/src/modules/MainApp/TeamInbox/domain/index.ts
index 6c1dd973e..516b2ffde 100644
--- a/src/modules/MainApp/TeamInbox/domain/index.ts
+++ b/src/modules/MainApp/TeamInbox/domain/index.ts
@@ -18,6 +18,7 @@ export type {
} from "./selectors";
export {
humanizeToken,
+ isGitHubIssueStatus,
workItemPriorityLabelKey,
workItemStatusLabelKey,
} from "./labels";
diff --git a/src/modules/MainApp/TeamInbox/domain/labels.ts b/src/modules/MainApp/TeamInbox/domain/labels.ts
index f49ddb3a8..38ce70ad7 100644
--- a/src/modules/MainApp/TeamInbox/domain/labels.ts
+++ b/src/modules/MainApp/TeamInbox/domain/labels.ts
@@ -1,3 +1,20 @@
+import { WORK_ITEM_STATUS } from "@src/types/core/workItem";
+
+/**
+ * GitHub-backed work items are identified by their status vocabulary: the sync
+ * adapter writes `open` / `closed` where local items use `todo` / `done`.
+ *
+ * Checked against the shared `WORK_ITEM_STATUS` constant rather than importing
+ * ProjectManager's equivalent helper, for the same isolation reason described
+ * on the label keys below.
+ */
+export function isGitHubIssueStatus(status: string): boolean {
+ return (
+ status === WORK_ITEM_STATUS.GITHUB_OPEN ||
+ status === WORK_ITEM_STATUS.GITHUB_CLOSED
+ );
+}
+
/**
* Turns a raw enum token from the work-item read model (e.g. `in_progress`,
* `HIGH`, `in-review`) into a human sentence-cased label (`In progress`,
diff --git a/src/modules/MainApp/TeamInbox/domain/selectors.ts b/src/modules/MainApp/TeamInbox/domain/selectors.ts
index 1d517dc25..cdda63dd7 100644
--- a/src/modules/MainApp/TeamInbox/domain/selectors.ts
+++ b/src/modules/MainApp/TeamInbox/domain/selectors.ts
@@ -153,7 +153,12 @@ const SESSION_BUCKET_TO_RECENCY: Record<
*/
export function groupTeamInboxItemsByRecency(
items: readonly TeamInboxItem[],
- nowMs: number
+ /**
+ * Defaults to the current time. Kept out of the calling component so the
+ * bucket boundaries are not derived from an impure call during render;
+ * tests and any caller needing determinism pass an explicit value.
+ */
+ nowMs: number = Date.now()
): TeamInboxRecencyGroup[] {
const ranges = getSessionDateBucketRanges(new Date(nowMs));
diff --git a/src/modules/shared/layouts/blocks/InfoCard.tsx b/src/modules/shared/layouts/blocks/InfoCard.tsx
index 141d14989..8d60fe28d 100644
--- a/src/modules/shared/layouts/blocks/InfoCard.tsx
+++ b/src/modules/shared/layouts/blocks/InfoCard.tsx
@@ -22,20 +22,31 @@ export interface InfoCardProps {
/** Extra content rendered above the card (e.g. badges) */
header?: React.ReactNode;
className?: string;
+ /**
+ * `surface` (default) paints the filled card. `plain` drops the fill and
+ * padding so the host panel's own background shows through.
+ */
+ variant?: "surface" | "plain";
}
const InfoCard: React.FC = ({
rows,
header,
className = "",
+ variant = "surface",
}) => {
const visibleRows = rows.filter((row) => !row.hidden);
if (visibleRows.length === 0 && !header) return null;
+ const container =
+ variant === "plain"
+ ? INFO_CARD_TOKENS.containerPlain
+ : INFO_CARD_TOKENS.container;
+
return (
{header}
-
+
{visibleRows.map((row) => (
From 2668e24d6e77446576bc2dda0194c211b2ddc9c9 Mon Sep 17 00:00:00 2001
From: Harry19081 <20519290+Harry19081@users.noreply.github.com>
Date: Sat, 25 Jul 2026 00:50:30 +0800
Subject: [PATCH 4/5] fix(team-inbox): derive work-item body loading state
The effect reset state synchronously on every selection change, cascading an
extra render (react-hooks/set-state-in-effect). Store the resolved body with
the key it was fetched for and derive `loading` by comparing that key against
the current target, so the reset falls out of the render pass.
Pre-commit hook ran. Total eslint: 2, total circular: 0
---
.../TeamInbox/useTeamInboxWorkItemBody.ts | 26 ++++++++++++-------
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git a/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts b/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
index 374fa03c7..c97fe7356 100644
--- a/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
+++ b/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
@@ -24,14 +24,20 @@ export function useTeamInboxWorkItemBody(
target: WorkItemTarget
): TeamInboxWorkItemBodyState {
const { projectId, workItemId } = target;
- const [state, setState] = useState
({
- body: null,
- loading: true,
- });
+ const requestKey = `${projectId}:${workItemId}`;
+ /**
+ * Holds the resolved body together with the key it was fetched for. Loading
+ * is derived by comparing that key against the current target rather than
+ * reset by a synchronous setState in the effect, which would cascade an
+ * extra render on every selection change.
+ */
+ const [resolved, setResolved] = useState<{
+ key: string;
+ body: string | null;
+ } | null>(null);
useEffect(() => {
let cancelled = false;
- setState({ body: null, loading: true });
const request = projectId
? projectApi.readWorkItem(projectId, workItemId)
@@ -41,18 +47,20 @@ export function useTeamInboxWorkItemBody(
.then((workItem) => {
if (cancelled) return;
const body = workItem.body.trim();
- setState({ body: body.length > 0 ? body : null, loading: false });
+ setResolved({ key: requestKey, body: body.length > 0 ? body : null });
})
.catch((error: unknown) => {
if (cancelled) return;
log.warn("Failed to load Team Inbox Work Item body", error);
- setState({ body: null, loading: false });
+ setResolved({ key: requestKey, body: null });
});
return () => {
cancelled = true;
};
- }, [projectId, workItemId]);
+ }, [projectId, workItemId, requestKey]);
- return state;
+ return resolved?.key === requestKey
+ ? { body: resolved.body, loading: false }
+ : { body: null, loading: true };
}
From eac30eff85ed289544b21d916e53152d10c68272 Mon Sep 17 00:00:00 2001
From: Harry19081 <20519290+Harry19081@users.noreply.github.com>
Date: Sat, 25 Jul 2026 01:04:47 +0800
Subject: [PATCH 5/5] refactor(team-inbox): reuse work-item components in
detail
The assigned detail rebuilt a reduced view of a work item from the list
payload: a Markdown body plus a four-row InfoCard. It now composes the same
pair the chat panel's work-item pane uses, so the two surfaces stay in step.
- Replace useTeamInboxWorkItemBody with useTeamInboxWorkItem, which resolves
the full item instead of just its body and persists property edits through
updateWorkItemPartial. Standalone items stay read-only; writing those needs
the full frontmatter round-trip only the owning pane assembles.
- Render WorkItemContent for the body and WorkItemProperties inside a
PropertiesRailFrame. The inbox pane is narrower than the work-item pane, so
the rail appears only past a 720px container width.
- Extract toWorkItemPartialUpdate out of WorkItemPanelView into a shared
module so both surfaces build the same wire payload rather than diverging.
- Give TeamInboxDetailLayout a `fill` content mode and make `metadata`
optional; comment mentions keep the padded scroll column and InfoCard.
Pre-commit hook ran. Total eslint: 2, total circular: 0
---
.../ChatPanel/panels/WorkItemPanelView.tsx | 66 +----------
.../components/AssignedWorkItemDetail.tsx | 104 +++++++++++------
.../components/TeamInboxDetailLayout.tsx | 33 ++++--
.../MainApp/TeamInbox/useTeamInboxWorkItem.ts | 107 ++++++++++++++++++
.../TeamInbox/useTeamInboxWorkItemBody.ts | 66 -----------
.../WorkItems/workItemPartialUpdate.ts | 73 ++++++++++++
6 files changed, 276 insertions(+), 173 deletions(-)
create mode 100644 src/modules/MainApp/TeamInbox/useTeamInboxWorkItem.ts
delete mode 100644 src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
create mode 100644 src/modules/ProjectManager/WorkItems/workItemPartialUpdate.ts
diff --git a/src/engines/ChatPanel/panels/WorkItemPanelView.tsx b/src/engines/ChatPanel/panels/WorkItemPanelView.tsx
index f1522a065..73a1d61a5 100644
--- a/src/engines/ChatPanel/panels/WorkItemPanelView.tsx
+++ b/src/engines/ChatPanel/panels/WorkItemPanelView.tsx
@@ -7,7 +7,6 @@ import { useTranslation } from "react-i18next";
import { STORY_SYNC_ADAPTER } from "@src/api/http/integrations/syncConnections";
import {
type WorkItemFrontmatter,
- type WorkItemPartialUpdate,
enrichedWorkItemToUI,
projectApi,
standaloneWorkItemDataToEnriched,
@@ -26,6 +25,7 @@ import {
} from "@src/modules/ProjectManager/WorkItems/components";
import { WorkItemDetailHeaderBreadcrumb } from "@src/modules/ProjectManager/WorkItems/components/WorkItemDetail/WorkItemDetailHeader";
import { useWorkItemOrchestrator } from "@src/modules/ProjectManager/WorkItems/hooks";
+import { toWorkItemPartialUpdate } from "@src/modules/ProjectManager/WorkItems/workItemPartialUpdate";
import { PropertiesRailFrame } from "@src/modules/ProjectManager/shared";
import { WorkstationToolbarTooltip } from "@src/modules/WorkStation/shared";
import { VerticalResizeHandle } from "@src/scaffold/Resize";
@@ -104,70 +104,6 @@ function applyWorkItemPatch(
};
}
-function toWorkItemPartialUpdate(
- updates: Partial
-): WorkItemPartialUpdate {
- const payload: WorkItemPartialUpdate = {};
-
- if (updates.name !== undefined) payload.title = updates.name;
- if (updates.spec !== undefined) payload.body = updates.spec;
- if (updates.workItemStatus !== undefined) {
- payload.status = updates.workItemStatus;
- }
- if (updates.priority !== undefined) payload.priority = updates.priority;
- if (updates.project?.id) payload.project = updates.project.id;
- if (updates.star !== undefined) payload.starred = updates.star;
- if ("assignee" in updates) payload.assignee = updates.assignee?.id ?? null;
- if ("assigneeType" in updates) {
- payload.assigneeType = updates.assigneeType ?? null;
- }
- if ("labels" in updates) {
- payload.labels = updates.labels?.map((label) => label.id) ?? [];
- }
- if ("milestone" in updates) {
- payload.milestone = updates.milestone?.id ?? null;
- }
- if ("startDate" in updates) payload.startDate = updates.startDate ?? null;
- if ("endDate" in updates) payload.targetDate = updates.endDate ?? null;
- if ("target_date" in updates) {
- payload.targetDate = updates.target_date ?? null;
- }
- if (updates.todos !== undefined) {
- payload.todos = updates.todos.map((todo) => ({
- id: todo.id,
- content: todo.content,
- status: todo.status,
- }));
- }
- if (updates.comments !== undefined) {
- payload.comments = updates.comments.map((comment) => ({
- id: comment.id,
- author: comment.author,
- content: comment.content,
- created_at: comment.created_at,
- }));
- }
- if (updates.linkedSessions !== undefined) {
- payload.linkedSessions = updates.linkedSessions;
- }
- if (updates.orchestratorConfig !== undefined) {
- payload.orchestratorConfig = updates.orchestratorConfig;
- }
- if (updates.orchestratorState !== undefined) {
- payload.orchestratorState = updates.orchestratorState;
- }
- if (updates.schedule !== undefined) payload.schedule = updates.schedule;
- if (updates.executionLock !== undefined) {
- payload.executionLock = updates.executionLock;
- }
- if (updates.closeOut !== undefined) payload.closeOut = updates.closeOut;
- if (updates.workProducts !== undefined) {
- payload.workProducts = updates.workProducts;
- }
-
- return payload;
-}
-
export const WorkItemPanelView: React.FC = ({
selectedWorkItem,
onUpdateWorkItem,
diff --git a/src/modules/MainApp/TeamInbox/components/AssignedWorkItemDetail.tsx b/src/modules/MainApp/TeamInbox/components/AssignedWorkItemDetail.tsx
index bdbb9ff70..63219ee5e 100644
--- a/src/modules/MainApp/TeamInbox/components/AssignedWorkItemDetail.tsx
+++ b/src/modules/MainApp/TeamInbox/components/AssignedWorkItemDetail.tsx
@@ -2,18 +2,24 @@ import { ClipboardList, ExternalLink } from "lucide-react";
import React from "react";
import { useTranslation } from "react-i18next";
-import Markdown from "@src/components/MarkDown";
+import {
+ WorkItemContent,
+ WorkItemProperties,
+} from "@src/modules/ProjectManager/WorkItems/components";
+import { PropertiesRailFrame } from "@src/modules/ProjectManager/shared";
+import { Placeholder } from "@src/modules/shared/layouts/blocks";
import {
type AssignedWorkItem,
type TeamInboxNavigationIntent,
- humanizeToken,
- workItemPriorityLabelKey,
- workItemStatusLabelKey,
+ isGitHubIssueStatus,
} from "../domain";
-import { useTeamInboxWorkItemBody } from "../useTeamInboxWorkItemBody";
+import { useTeamInboxWorkItem } from "../useTeamInboxWorkItem";
import TeamInboxDetailLayout from "./TeamInboxDetailLayout";
+/** Matches the work-item pane's info rail so both surfaces read identically. */
+const PROPERTIES_RAIL_WIDTH = 240;
+
export interface AssignedWorkItemDetailProps {
item: AssignedWorkItem;
onNavigate?: (intent: TeamInboxNavigationIntent) => void;
@@ -28,20 +34,17 @@ const AssignedWorkItemDetail: React.FC = ({
onMarkUnread,
}) => {
const { t } = useTranslation();
- const { body } = useTeamInboxWorkItemBody(item.target);
- const excerpt = item.payload.summary ?? null;
- const statusLabel = t(workItemStatusLabelKey(item.payload.status), {
- defaultValue: humanizeToken(item.payload.status),
- });
- const priorityLabel = t(workItemPriorityLabelKey(item.payload.priority), {
- defaultValue: humanizeToken(item.payload.priority),
- });
+ const { workItem, loading, updateWorkItem } = useTeamInboxWorkItem(
+ item.target
+ );
+ const isGitHubIssue = isGitHubIssueStatus(item.payload.status);
return (
= ({
})
: undefined
}
- metadata={[
- { label: t("teamInbox.fields.status"), value: statusLabel },
- { label: t("teamInbox.fields.priority"), value: priorityLabel },
- {
- label: t("teamInbox.fields.assignee"),
- value: item.payload.assigneeName ?? item.payload.assigneeMemberId,
- },
- {
- label: t("teamInbox.fields.workItemId"),
- value: item.target.workItemId,
- },
- ]}
>
- {body ? (
-
-
+ {loading ? (
+
+ ) : workItem ? (
+
+
+
+ onNavigate({
+ kind: "open_session_comment",
+ sessionId,
+ commentId: "",
+ threadId: "",
+ })
+ : undefined
+ }
+ />
+
+ {/* The inbox detail is narrower than the work-item pane, so the rail
+ only appears once there is room for it beside the content. */}
+
- ) : excerpt ? (
-
- {excerpt}
-
- ) : null}
+ ) : (
+
+ )}
);
};
diff --git a/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx b/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
index 3011868cb..beae9bd40 100644
--- a/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
+++ b/src/modules/MainApp/TeamInbox/components/TeamInboxDetailLayout.tsx
@@ -16,7 +16,15 @@ export interface TeamInboxDetailLayoutProps {
title: string;
subtitle: string;
icon: LucideIcon;
- metadata: InfoCardRow[];
+ /** Key-value rows rendered under the body. Omitted when the body owns its
+ * own property surface (see `contentLayout: "fill"`). */
+ metadata?: InfoCardRow[];
+ /**
+ * `scroll` (default) puts the body in a padded, centred scroll column.
+ * `fill` hands it the remaining height untouched, for bodies that manage
+ * their own scrolling and side rails.
+ */
+ contentLayout?: "scroll" | "fill";
unread: boolean;
markReadLabel: string;
markUnreadLabel?: string;
@@ -33,6 +41,7 @@ const TeamInboxDetailLayout: React.FC
= ({
subtitle,
icon,
metadata,
+ contentLayout = "scroll",
unread,
markReadLabel,
markUnreadLabel,
@@ -76,14 +85,22 @@ const TeamInboxDetailLayout: React.FC = ({
}
/>
-
-
- {children ? (
-
{children}
- ) : null}
-
+ {contentLayout === "fill" ? (
+
+ {children}
-
+ ) : (
+
+
+ {children ? (
+
{children}
+ ) : null}
+ {metadata && metadata.length > 0 ? (
+
+ ) : null}
+
+
+ )}
{onOpen ? (
) => void;
+}
+
+/**
+ * Loads the full Work Item behind an assigned inbox row so the detail pane can
+ * render the same `WorkItemContent` / `WorkItemProperties` pair the work-item
+ * pane uses, instead of a reduced summary of the list payload.
+ *
+ * The read is demand-driven (one per selection, no polling) and stale responses
+ * are discarded when the selection changes.
+ */
+export function useTeamInboxWorkItem(
+ target: WorkItemTarget
+): TeamInboxWorkItemState {
+ const { projectId, workItemId } = target;
+ const requestKey = `${projectId}:${workItemId}`;
+ /**
+ * Holds the resolved item together with the key it was fetched for. Loading
+ * is derived by comparing that key against the current target rather than
+ * reset by a synchronous setState in the effect.
+ */
+ const [resolved, setResolved] = useState<{
+ key: string;
+ workItem: WorkItem | null;
+ } | null>(null);
+
+ useEffect(() => {
+ let cancelled = false;
+
+ const request = projectId
+ ? projectApi.readWorkItem(projectId, workItemId)
+ : projectApi.readStandaloneWorkItem(workItemId);
+
+ void request
+ .then((data) => {
+ if (cancelled) return;
+ setResolved({
+ key: requestKey,
+ workItem: enrichedWorkItemToUI(
+ standaloneWorkItemDataToEnriched(data)
+ ),
+ });
+ })
+ .catch((error: unknown) => {
+ if (cancelled) return;
+ log.warn("Failed to load Team Inbox Work Item", error);
+ setResolved({ key: requestKey, workItem: null });
+ });
+
+ return () => {
+ cancelled = true;
+ };
+ }, [projectId, workItemId, requestKey]);
+
+ const updateWorkItem = useCallback(
+ (updates: Partial) => {
+ // Standalone items are written back through a full frontmatter
+ // round-trip that only the owning pane assembles, so the inbox edits
+ // project-scoped items and leaves standalone ones read-only.
+ if (!projectId) return;
+
+ const payload = toWorkItemPartialUpdate(updates);
+ if (Object.keys(payload).length === 0) return;
+
+ void projectApi
+ .updateWorkItemPartial(projectId, workItemId, payload)
+ .then((updated) => {
+ setResolved({
+ key: requestKey,
+ workItem: enrichedWorkItemToUI(updated),
+ });
+ })
+ .catch((error: unknown) => {
+ log.warn("Failed to update Team Inbox Work Item", error);
+ });
+ },
+ [projectId, workItemId, requestKey]
+ );
+
+ const isResolved = resolved?.key === requestKey;
+
+ return {
+ workItem: isResolved ? resolved.workItem : null,
+ loading: !isResolved,
+ updateWorkItem,
+ };
+}
diff --git a/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts b/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
deleted file mode 100644
index c97fe7356..000000000
--- a/src/modules/MainApp/TeamInbox/useTeamInboxWorkItemBody.ts
+++ /dev/null
@@ -1,66 +0,0 @@
-import { useEffect, useState } from "react";
-
-import { projectApi } from "@src/api/http/project";
-import { createLogger } from "@src/hooks/logger";
-
-import type { WorkItemTarget } from "./domain";
-
-const log = createLogger("TeamInboxWorkItemBody");
-
-export interface TeamInboxWorkItemBodyState {
- /** Full Markdown body once resolved, or null while loading / empty / failed. */
- body: string | null;
- loading: boolean;
-}
-
-/**
- * Lazily loads the full Work Item body for the selected assigned inbox item so
- * the detail preview can render the real content instead of the short list
- * excerpt. The fetch reuses the same project store adapters as navigation and is
- * demand-driven (one read per selection, no polling); stale responses are
- * discarded when the selection changes.
- */
-export function useTeamInboxWorkItemBody(
- target: WorkItemTarget
-): TeamInboxWorkItemBodyState {
- const { projectId, workItemId } = target;
- const requestKey = `${projectId}:${workItemId}`;
- /**
- * Holds the resolved body together with the key it was fetched for. Loading
- * is derived by comparing that key against the current target rather than
- * reset by a synchronous setState in the effect, which would cascade an
- * extra render on every selection change.
- */
- const [resolved, setResolved] = useState<{
- key: string;
- body: string | null;
- } | null>(null);
-
- useEffect(() => {
- let cancelled = false;
-
- const request = projectId
- ? projectApi.readWorkItem(projectId, workItemId)
- : projectApi.readStandaloneWorkItem(workItemId);
-
- void request
- .then((workItem) => {
- if (cancelled) return;
- const body = workItem.body.trim();
- setResolved({ key: requestKey, body: body.length > 0 ? body : null });
- })
- .catch((error: unknown) => {
- if (cancelled) return;
- log.warn("Failed to load Team Inbox Work Item body", error);
- setResolved({ key: requestKey, body: null });
- });
-
- return () => {
- cancelled = true;
- };
- }, [projectId, workItemId, requestKey]);
-
- return resolved?.key === requestKey
- ? { body: resolved.body, loading: false }
- : { body: null, loading: true };
-}
diff --git a/src/modules/ProjectManager/WorkItems/workItemPartialUpdate.ts b/src/modules/ProjectManager/WorkItems/workItemPartialUpdate.ts
new file mode 100644
index 000000000..a64c97a37
--- /dev/null
+++ b/src/modules/ProjectManager/WorkItems/workItemPartialUpdate.ts
@@ -0,0 +1,73 @@
+import type { WorkItemPartialUpdate } from "@src/api/http/project";
+import type { WorkItem } from "@src/types/core/workItem";
+
+/**
+ * Map a UI-shaped partial work-item update onto the wire payload the project
+ * store accepts.
+ *
+ * Shared so every surface that edits a work item — the chat panel's work-item
+ * pane and the Team Inbox detail — persists the same fields the same way.
+ */
+export function toWorkItemPartialUpdate(
+ updates: Partial
+): WorkItemPartialUpdate {
+ const payload: WorkItemPartialUpdate = {};
+
+ if (updates.name !== undefined) payload.title = updates.name;
+ if (updates.spec !== undefined) payload.body = updates.spec;
+ if (updates.workItemStatus !== undefined) {
+ payload.status = updates.workItemStatus;
+ }
+ if (updates.priority !== undefined) payload.priority = updates.priority;
+ if (updates.project?.id) payload.project = updates.project.id;
+ if (updates.star !== undefined) payload.starred = updates.star;
+ if ("assignee" in updates) payload.assignee = updates.assignee?.id ?? null;
+ if ("assigneeType" in updates) {
+ payload.assigneeType = updates.assigneeType ?? null;
+ }
+ if ("labels" in updates) {
+ payload.labels = updates.labels?.map((label) => label.id) ?? [];
+ }
+ if ("milestone" in updates) {
+ payload.milestone = updates.milestone?.id ?? null;
+ }
+ if ("startDate" in updates) payload.startDate = updates.startDate ?? null;
+ if ("endDate" in updates) payload.targetDate = updates.endDate ?? null;
+ if ("target_date" in updates) {
+ payload.targetDate = updates.target_date ?? null;
+ }
+ if (updates.todos !== undefined) {
+ payload.todos = updates.todos.map((todo) => ({
+ id: todo.id,
+ content: todo.content,
+ status: todo.status,
+ }));
+ }
+ if (updates.comments !== undefined) {
+ payload.comments = updates.comments.map((comment) => ({
+ id: comment.id,
+ author: comment.author,
+ content: comment.content,
+ created_at: comment.created_at,
+ }));
+ }
+ if (updates.linkedSessions !== undefined) {
+ payload.linkedSessions = updates.linkedSessions;
+ }
+ if (updates.orchestratorConfig !== undefined) {
+ payload.orchestratorConfig = updates.orchestratorConfig;
+ }
+ if (updates.orchestratorState !== undefined) {
+ payload.orchestratorState = updates.orchestratorState;
+ }
+ if (updates.schedule !== undefined) payload.schedule = updates.schedule;
+ if (updates.executionLock !== undefined) {
+ payload.executionLock = updates.executionLock;
+ }
+ if (updates.closeOut !== undefined) payload.closeOut = updates.closeOut;
+ if (updates.workProducts !== undefined) {
+ payload.workProducts = updates.workProducts;
+ }
+
+ return payload;
+}