fix: look up assignee agent name when executionAgentNameKey is null (#36)

When an agent completes an issue, the Discord notification shows
"Completed by: Agent" instead of the agent's display name.

Root cause: issue.executionAgentNameKey is not reliably populated
by the Paperclip platform. When it's null, the fallback
payload.agentName ?? "Agent" produces the literal string "Agent",
which the formatter uses verbatim.

Fix: if agentName is still null after the executionAgentNameKey
assignment, look up the assignee agent by ID via ctx.agents.list()
and use its name field (the display name — e.g. "Scribe", "Claira").
This resolves without any API changes on the platform side.

Co-authored-by: Claira <claira@homicidezero.com>

Author: HomicideZero

src/worker.ts

@@ -171,6 +171,14 @@ async function enrichIssueNotificationPayload(
       if (payload.assigneeAgentId == null) payload.assigneeAgentId = issue.assigneeAgentId;
       if (payload.assigneeUserId == null) payload.assigneeUserId = issue.assigneeUserId;
       if (payload.agentName == null) payload.agentName = issue.executionAgentNameKey;
+      // executionAgentNameKey is not always populated — fall back to looking up the
+      // assignee agent's display name so "Completed by" shows "Scribe" not "Agent".
+      if (payload.agentName == null && (payload.assigneeAgentId || issue.assigneeAgentId)) {
+        const agentId = payload.assigneeAgentId ?? issue.assigneeAgentId;
+        const agents = await ctx.agents.list({ companyId });
+        const match = (agents as Array<{ id: string; name: string }>).find((a) => a.id === agentId);
+        if (match?.name) payload.agentName = match.name;
+      }
       if (payload.completedAt == null && issue.completedAt) payload.completedAt = String(issue.completedAt);
       if (payload.updatedAt == null && issue.updatedAt) payload.updatedAt = String(issue.updatedAt);
       if (payload.projectName == null && issue.project?.name) payload.projectName = issue.project.name;