fix(ci): simplify activity item memoization
This commit is contained in:
parent
e1a85786e7
commit
d2ef5965e2
1 changed files with 8 additions and 17 deletions
|
|
@ -668,11 +668,8 @@ export const ActivityItem = memo(
|
||||||
}, [message.timestamp]);
|
}, [message.timestamp]);
|
||||||
|
|
||||||
const structured = parseStructuredAgentMessage(message.text);
|
const structured = parseStructuredAgentMessage(message.text);
|
||||||
const bootstrapDisplay = useMemo(() => getBootstrapPromptDisplay(message), [message]);
|
const bootstrapDisplay = getBootstrapPromptDisplay(message);
|
||||||
const bootstrapAcknowledgement = useMemo(
|
const bootstrapAcknowledgement = getBootstrapAcknowledgementDisplay(message);
|
||||||
() => getBootstrapAcknowledgementDisplay(message),
|
|
||||||
[message]
|
|
||||||
);
|
|
||||||
// Only flag agent messages as rate-limited, not user's own quotes
|
// Only flag agent messages as rate-limited, not user's own quotes
|
||||||
const rateLimited = message.from !== 'user' && isRateLimitMessage(message.text);
|
const rateLimited = message.from !== 'user' && isRateLimitMessage(message.text);
|
||||||
// Highlight messages containing API errors
|
// Highlight messages containing API errors
|
||||||
|
|
@ -681,22 +678,16 @@ export const ActivityItem = memo(
|
||||||
const isAuthError = isApiError && AUTH_ERROR_PATTERNS.some((p) => p.test(message.text));
|
const isAuthError = isApiError && AUTH_ERROR_PATTERNS.some((p) => p.test(message.text));
|
||||||
// Never collapse rate limit messages as noise — they must be visible
|
// Never collapse rate limit messages as noise — they must be visible
|
||||||
const noiseLabel = structured && !rateLimited ? getNoiseLabel(structured) : null;
|
const noiseLabel = structured && !rateLimited ? getNoiseLabel(structured) : null;
|
||||||
const idleSemantic = useMemo(() => classifyIdleNotification(message), [message]);
|
const idleSemantic = classifyIdleNotification(message);
|
||||||
|
|
||||||
const systemLabel = !structured && !rateLimited ? getSystemMessageLabel(message.text) : null;
|
const systemLabel = !structured && !rateLimited ? getSystemMessageLabel(message.text) : null;
|
||||||
const isManaged = collapseMode === 'managed';
|
const isManaged = collapseMode === 'managed';
|
||||||
const isExpanded = isManaged ? !isCollapsed : true;
|
const isExpanded = isManaged ? !isCollapsed : true;
|
||||||
|
|
||||||
const parsedCrossTeamPrefix = useMemo(() => parseCrossTeamPrefix(message.text), [message.text]);
|
const parsedCrossTeamPrefix = parseCrossTeamPrefix(message.text);
|
||||||
const qualifiedRecipient = useMemo(() => parseQualifiedRecipient(message.to), [message.to]);
|
const qualifiedRecipient = parseQualifiedRecipient(message.to);
|
||||||
const crossTeamSentTarget = useMemo(
|
const crossTeamSentTarget = getCrossTeamSentTarget(message.to, teamName, localMemberNames);
|
||||||
() => getCrossTeamSentTarget(message.to, teamName, localMemberNames),
|
const crossTeamSentMemberName = getCrossTeamSentMemberName(message.to);
|
||||||
[message.to, teamName, localMemberNames]
|
|
||||||
);
|
|
||||||
const crossTeamSentMemberName = useMemo(
|
|
||||||
() => getCrossTeamSentMemberName(message.to),
|
|
||||||
[message.to]
|
|
||||||
);
|
|
||||||
const isCrossTeam = message.source === CROSS_TEAM_SOURCE || parsedCrossTeamPrefix !== null;
|
const isCrossTeam = message.source === CROSS_TEAM_SOURCE || parsedCrossTeamPrefix !== null;
|
||||||
const isCrossTeamSent =
|
const isCrossTeamSent =
|
||||||
message.source === CROSS_TEAM_SENT_SOURCE || crossTeamSentTarget !== null;
|
message.source === CROSS_TEAM_SENT_SOURCE || crossTeamSentTarget !== null;
|
||||||
|
|
@ -827,7 +818,7 @@ export const ActivityItem = memo(
|
||||||
slashCommandMeta,
|
slashCommandMeta,
|
||||||
structured,
|
structured,
|
||||||
]);
|
]);
|
||||||
const summaryText = useMemo(() => extractMarkdownPlainText(rawSummary), [rawSummary]);
|
const summaryText = extractMarkdownPlainText(rawSummary);
|
||||||
const commentTaskRef =
|
const commentTaskRef =
|
||||||
message.messageKind === 'task_comment_notification' ? (message.taskRefs?.[0] ?? null) : null;
|
message.messageKind === 'task_comment_notification' ? (message.taskRefs?.[0] ?? null) : null;
|
||||||
const commentTaskDisplayId =
|
const commentTaskDisplayId =
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue