ework-web 0.10.97 → 0.10.99

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ework-web",
-  "version": "0.10.97",
+  "version": "0.10.99",
   "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/upstream-sync.ts

@@ -6,6 +6,7 @@ import {
   getIssueByUpstreamNumber,
   getCommentByUpstreamId,
   createIssue,
+  ensureUser,
   postComment,
   editIssue,
   updateUpstreamSyncState,
@@ -43,6 +44,12 @@ export interface UpstreamSyncPollResult {
 
 const GITEA_COMMENT_ISSUE_RE = /\/issues\/(\d+)$/;
 
+// GitHub CI/actions identities end in [bot]; their comments must not wake
+// the engine, so they are imported as bot-kind users.
+function fromGithubBot(login: string | undefined): boolean {
+  return typeof login === "string" && /\[bot\]$/i.test(login.trim());
+}
+
 function upstreamIssueNumberFromComment(gc: GiteaComment): number | null {
   const m = (gc.issue_url ?? "").match(GITEA_COMMENT_ISSUE_RE);
   return m ? Number(m[1]) : null;
@@ -103,6 +110,7 @@ export class UpstreamSync {
   }
 
   private async importIssue(gi: GiteaIssue, emit: boolean): Promise<void> {
+    await ensureUser(gi.user?.login ?? "upstream", fromGithubBot(gi.user?.login) ? "bot" : "human");
     await createIssue(
       this.project.id,
       (gi.pull_request ? "[PR] " : "") + (gi.title || `#${gi.number}`),
@@ -143,7 +151,13 @@ export class UpstreamSync {
       if (await getCommentByUpstreamId(gc.id)) continue;
       // write-back comments carry this marker; importing them would duplicate locally
       if ((gc.body ?? "").includes("<!-- ework-mirror -->")) continue;
-      const row = await postComment(local.id, gc.body ?? "", gc.user?.login ?? "upstream", {
+      const gcAuthor = gc.user?.login ?? "upstream";
+      // ensure the kind before postComment's internal user creation runs
+      await ensureUser(gcAuthor, fromGithubBot(gcAuthor) ? "bot" : "human");
+      // invisible provenance marker: ework-mirror skips events carrying it, so
+      // synced-in comments never echo back to the upstream thread
+      const body = (gc.body ?? "") + "\n\n<!-- upstream-sync -->";
+      const row = await postComment(local.id, body, gcAuthor, {
         createdAt: gc.created_at,
         updatedAt: gc.updated_at,
         upstreamCommentId: gc.id,
@@ -221,7 +235,9 @@ export class UpstreamSync {
         if (!upstreamNumber) continue;
         const local = await getIssueByUpstreamNumber(this.project.id, upstreamNumber);
         if (!local) continue;
-        const row = await postComment(local.id, gc.body ?? "", gc.user?.login ?? "upstream", {
+        const gcAuthor = gc.user?.login ?? "upstream";
+        await ensureUser(gcAuthor, fromGithubBot(gcAuthor) ? "bot" : "human");
+        const row = await postComment(local.id, gc.body ?? "", gcAuthor, {
           createdAt: gc.created_at,
           updatedAt: gc.updated_at,
           upstreamCommentId: gc.id,