fix: reflect PR state in channel task card badge

[adamleithp]

Problem

A channel task card always showed "In progress" even after the task was done and a PR was created.

The status badge (TaskStatusBadge in ChannelFeedView.tsx) read only the run status (taskRunStatus / latest_run.status). Cloud run rows routinely linger on in_progress after the agent opens the PR, so the badge got stuck. The PR's real GitHub state was already fetched and rendered by the sidebar TaskIcon (via useTaskPrStatus), but the card badge ignored it.

Fix

The badge now pulls PR state the same way the icon does, and a PR outranks the run status:

PR state	Badge
open	In review (info)
merged	Merged (success)
draft	Draft PR (default)
closed	Closed (destructive)

Precedence: Needs input → live In progress → PR state → run status (Completed / Failed / Local / Draft).

A failed / cancelled run still wins over an open PR — that's a deliberate end state we shouldn't hide.

Card badge and sidebar icon now agree. One file touched, typecheck green.

---

Created with PostHog Code

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[greptile-apps]

_{Reviews (1): Last reviewed commit: "fix: reflect PR state in channel task ca..." | Re-trigger Greptile}

[greptile-apps]

Circular-reference detection here is nearly identical to the logic inside serializeError in errorDetails.ts — both maintain a WeakSet<object> and return "[circular]" for revisited nodes. Per the team's OnceAndOnlyOnce rule, a small shared helper (e.g. makeCircularSafeReplacer) could serve both call sites and keep the two serializers in sync when edge cases are discovered in one of them.

Suggested change

	if (typeof arg === "object" && arg !== null) {
	const seen = new WeakSet<object>();
	try {
	return (
	JSON.stringify(arg, (_key, value: unknown) => {
	if (typeof value === "object" && value !== null) {
	if (seen.has(value)) return "[circular]";
	seen.add(value);
	}
	if (typeof value === "bigint" || typeof value === "function") {
	return String(value);
	}
	return value;
	}) ?? String(arg)
	);
	} catch {
	return String(arg);
	}
	}
	if (typeof arg === "object" && arg !== null) {
	const seen = new WeakSet<object>();
	const replacer = (_key: string, value: unknown): unknown => {
	if (typeof value === "object" && value !== null) {
	if (seen.has(value)) return "[circular]";
	seen.add(value);
	}
	if (typeof value === "bigint" || typeof value === "function") {
	return String(value);
	}
	return value;
	};
	try {
	return JSON.stringify(arg, replacer) ?? String(arg);
	} catch {
	return String(arg);
	}
	}

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

[github-actions]

React Doctor found no issues in the changed files. 🎉

_{Reviewed by React Doctor for commit 7d2f4a0.}