Greptile Summary

This PR fixes a UX issue in the Feishu integration where the raw string "main" was being displayed as a card header title for the default agent — a confusing artifact of the agent ID being used as a fallback when no identity is configured.

The change modifies resolveCardHeader to suppress "main" as a fallback name and adds an early return producing title: "" when both name and emoji are empty. Both the streaming card path (line 272) and the static card path (line 433) are updated to conditionally omit the header when cardHeader.title is falsy, which correctly handles the empty-string sentinel.

Key observations:

• The core logic is sound and correctly handles all four documented test scenarios (default agent, name-only identity, emoji-only identity, non-"main" subagent).
• One minor edge case: when agentId === "main", no name is set, but an emoji IS configured, the title becomes "🤖 " (emoji followed by a trailing space from the template literal `${emoji} ${name}` where name is ""). Trimming the final composed title would resolve this.
• resolveCardNote (line 67) still falls back to the raw agentId (including "main"), so the note footer still shows "Agent: main" for the default agent. This is a pre-existing inconsistency and out of scope for this PR, but may be worth a follow-up.

Confidence Score: 4/5

• Safe to merge; the change is narrowly scoped, functionally correct, and only introduces a minor cosmetic edge case.
• Both streaming and static card code paths were correctly updated. The sentinel value approach (empty string title → omit header) is consistent throughout. The one minor issue — a trailing space in the title when only emoji is set for the "main" agent — is a cosmetic edge case with no runtime impact.
• No files require special attention.

This is a comment left during a code review.
Path: extensions/feishu/src/reply-dispatcher.ts
Line: 54-58

Comment:
**Trailing space in title when only emoji is configured**

When `agentId === "main"`, `identity.name` is empty/unset, but `identity.emoji` is configured, `name` resolves to `""`. The early return is skipped (because `emoji` is truthy), and the title is built as `` `${emoji} ${name}` `` which produces e.g. `"🤖 "` — a trailing space after the emoji. This is a new edge case introduced by this change (previously `name` always fell back to a non-empty `agentId`).

A simple fix is to trim the final composed title:

```suggestion
  if (!name && !emoji) return { title: "", template: identity?.theme ?? "blue" };
  return {
    title: (emoji ? `${emoji} ${name}` : name).trim(),
    template: identity?.theme ?? "blue",
  };
```

How can I resolve this? If you propose a fix, please make it concise.

_{Last reviewed commit: e176af9}

Covered by #71542. Current main now omits the generic main card header for Feishu streaming and static cards while preserving configured names, emoji, and non-main agent fallbacks.

Closing this as superseded. Thanks for catching the default-header polish issue.