ework-web 0.10.67 → 0.10.69

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ework-web",
-  "version": "0.10.67",
+  "version": "0.10.69",
   "type": "module",
   "description": "ework-web — standalone multi-project issue tracker. Local SQLite-backed, no external API dependency. Bun + TypeScript + SSR HTML.",
   "license": "MIT",

package/src/webhooks.ts

@@ -28,6 +28,7 @@ import {
   resolveModel,
   listLabelsForIssue,
   getUserByLogin,
+  type UserRow,
   type IssueRow,
   type ProjectRow,
   type CommentRow,
@@ -229,6 +230,9 @@ interface PayloadUser {
   html_url: string;
   is_admin: boolean;
   language: string;
+  kind?: string;
+  display_name?: string | null;
+  is_active?: boolean;
 }
 
 interface PayloadLabel {
@@ -348,6 +352,18 @@ function buildUser(login: string, origin: string): PayloadUser {
   };
 }
 
+function buildUserFromRow(user: UserRow, origin: string): PayloadUser {
+  return {
+    ...buildUser(user.login, origin),
+    full_name: user.display_name ?? user.login,
+    email: user.email ?? `${user.login}@${new URL(origin).host ?? "localhost"}.noreply.ework`,
+    is_admin: user.is_admin === 1,
+    kind: user.kind,
+    display_name: user.display_name,
+    is_active: user.is_active === 1,
+  };
+}
+
 function buildRepository(project: ProjectRow, origin: string, model?: string): PayloadRepository {
   const fullName = `${project.owner}/${project.name}`;
   const htmlUrl = `${origin}/${encodeURIComponent(project.owner)}/${encodeURIComponent(project.name)}`;
@@ -639,11 +655,6 @@ export async function emitIssueEvent(
     const issue = await getIssueById(issueId);
     if (!issue) { log.warn(`emitIssueEvent: issue ${issueId} not found — silent skip prevented`); return; }
 
-    // Web is a dumb message bus for comments — always fan out.
-    // BUT for issue_opened, dispatch state is a subscription-level switch:
-    // if dispatch is off (global/project/issue), don't emit at all.
-    // This is the "webhook switch" — downstream never sees the event.
-    // We still include dispatch_off in payload for defense-in-depth.
     const cfg = await getConfigAll();
     const scopeKey = `${project.owner}/${project.name}`;
     const globalOff = cfg["dispatchEnabled"] === "false";
@@ -651,11 +662,6 @@ export async function emitIssueEvent(
     const issueOff = issue.ai_status === "dispatch_off" || issue.ai_status === "halted";
     const dispatchOff = globalOff || projectOff || issueOff;
 
-    if (action === "opened" && dispatchOff) {
-      log.info(`webhook: issue_opened suppressed (dispatch off: global=${globalOff} project=${projectOff} issue=${issueOff}) for ${scopeKey}#${issue.number}`);
-      return;
-    }
-
     const commentCount = await countCommentsSafe(issueId);
     // Resolve model: project override > global default. Empty string = no
     // override (daemon omits --model, lets opencode pick). globalDefault
@@ -666,6 +672,11 @@ export async function emitIssueEvent(
     const payload = buildIssuePayload(issue, project, commentCount, action, origin, model, labels);
     (payload as IssueEventPayload).event_id = randomUUID();
     (payload as IssueEventPayload).dispatch_off = dispatchOff;
+    const authorUser = await getUserByLogin(issue.author);
+    if (authorUser) {
+      payload.sender = buildUserFromRow(authorUser, origin);
+      payload.issue.user = buildUserFromRow(authorUser, origin);
+    }
     const rawBody = JSON.stringify(payload);
     await fanOut(projectId, "issues", rawBody);
   } catch (e) {
@@ -700,7 +711,13 @@ export async function emitCommentEvent(
     const labels = await toPayloadLabels(issueId);
     const payload = buildCommentPayload(issue, comment, project, commentCount, origin, model, labels);
     (payload as CommentEventPayload).event_id = randomUUID();
-    (payload as CommentEventPayload).comment.author_kind = authorUser?.kind ?? "human";
+    if (authorUser) {
+      payload.sender = buildUserFromRow(authorUser, origin);
+      payload.comment.user = buildUserFromRow(authorUser, origin);
+      (payload as CommentEventPayload).comment.author_kind = authorUser.kind;
+    } else {
+      (payload as CommentEventPayload).comment.author_kind = "human";
+    }
     const rawBody = JSON.stringify(payload);
     await fanOut(projectId, "issue_comment", rawBody);
   } catch (e) {
@@ -728,8 +745,9 @@ export async function emitStatusChanged(
     const issue = await getIssueById(issueId);
     if (!issue) { log.warn(`emitStatusChanged: issue ${issueId} not found (project=${projectId}) — silent skip prevented`); return; }
     const commentCount = await countCommentsSafe(issueId);
+    const actorUser = await getUserByLogin(actor);
+    const sender = actorUser ? buildUserFromRow(actorUser, origin) : buildUser(actor, origin);
     const repo = buildRepository(project, origin);
-    const sender = buildUser(actor, origin);
     const issuePayload = buildIssue(issue, project, commentCount, origin);
     const payload: StatusChangedPayload = {
       event_id: randomUUID(),