@cueloop/daemon 0.1.0-alpha.38 → 0.1.0-alpha.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/daemon",
-  "version": "0.1.0-alpha.38",
+  "version": "0.1.0-alpha.39",
   "description": "The cueloop session daemon and its client: socket protocol, atomic store, resumable waits",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -28,7 +28,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/schema": "0.1.0-alpha.38",
+    "@cueloop/schema": "0.1.0-alpha.39",
     "valibot": "^1.4.2"
   },
   "engines": {

package/src/api.test.ts

@@ -63,6 +63,29 @@ describe("session lifecycle", () => {
     expect(core.sessionGet(session.id).annotations.length).toBe(0);
   });
 
+  test("annotate with an author registers them in the participant registry", () => {
+    // Arrange
+    const session = core.sessionCreate({ workspace: WS, artifact: PLAN });
+
+    // Act
+    core.sessionAnnotate(
+      session.id,
+      {
+        id: "a1",
+        kind: "comment",
+        anchor: { quote: "carefully", prefix: "the thing ", suffix: "." },
+        body: "A collaborator's note.",
+        author: "SHA256:ana",
+      },
+      "Ana",
+    );
+
+    // Assert
+    const stored = core.sessionGet(session.id);
+    expect(stored.annotations[0]!.author).toBe("SHA256:ana");
+    expect(stored.participants).toEqual([{ id: "SHA256:ana", provider: "ssh", name: "Ana" }]);
+  });
+
   test("resolve produces feedback.md and maps to the agent contract", () => {
     // Arrange
     const session = core.sessionCreate({ workspace: WS, artifact: PLAN });

package/src/api.ts

@@ -11,6 +11,7 @@ import {
   isAddressed,
   isAgentNote,
   parseBlocks,
+  registerParticipant,
   resolveAnchor,
   verdictAllows,
   type Annotation,
@@ -143,7 +144,15 @@ export class DaemonCore {
     });
   }
 
-  sessionAnnotate(id: string, annotation: Omit<Annotation, "createdAt">): ReviewSession {
+  /**
+   * `authorName` registers the annotation's author in the participant registry;
+   * a bare author id (no name) keeps its short-handle fallback instead.
+   */
+  sessionAnnotate(
+    id: string,
+    annotation: Omit<Annotation, "createdAt">,
+    authorName?: string,
+  ): ReviewSession {
     const session = this.mutable(id);
     const existing = session.annotations.findIndex((candidate) => candidate.id === annotation.id);
     const full: Annotation = { ...annotation, createdAt: new Date().toISOString() };
@@ -153,6 +162,12 @@ export class DaemonCore {
         ...full,
         createdAt: session.annotations[existing]!.createdAt,
       };
+    if (annotation.author && authorName)
+      session.participants = registerParticipant(
+        session,
+        annotation.author,
+        authorName,
+      ).participants;
     this.store.upsert(session);
     this.emit("session.updated", id);
     return session;

package/src/client.ts

@@ -38,7 +38,11 @@ export interface SessionClient {
   subscribe(): Promise<void>;
   sessionGet(id: string): Promise<ReviewSession>;
   sessionList(filter?: { status?: "pending" | "resolved" }): Promise<ReviewSession[]>;
-  sessionAnnotate(id: string, annotation: Omit<Annotation, "createdAt">): Promise<ReviewSession>;
+  sessionAnnotate(
+    id: string,
+    annotation: Omit<Annotation, "createdAt">,
+    authorName?: string,
+  ): Promise<ReviewSession>;
   sessionRemoveAnnotation(id: string, annotationId: string): Promise<ReviewSession>;
   sessionSetWorkingCopy(id: string, workingCopy: string | undefined): Promise<ReviewSession>;
   sessionSetViewed(id: string, viewedPaths: string[]): Promise<ReviewSession>;
@@ -188,8 +192,12 @@ export class DaemonClient implements SessionClient {
   sessionWait(id: string, timeoutMs: number): Promise<ReviewSession | null> {
     return this.request("session.wait", { id, timeoutMs }, timeoutMs + 10_000);
   }
-  sessionAnnotate(id: string, annotation: Omit<Annotation, "createdAt">): Promise<ReviewSession> {
-    return this.request("session.annotate", { id, annotation });
+  sessionAnnotate(
+    id: string,
+    annotation: Omit<Annotation, "createdAt">,
+    authorName?: string,
+  ): Promise<ReviewSession> {
+    return this.request("session.annotate", { id, annotation, authorName });
   }
   sessionRemoveAnnotation(id: string, annotationId: string): Promise<ReviewSession> {
     return this.request("session.removeAnnotation", { id, annotationId });

package/src/server.ts

@@ -246,7 +246,7 @@ export class DaemonServer {
       }
       case "session.annotate": {
         const params = parseParams("session.annotate", request.params);
-        return core.sessionAnnotate(params.id, params.annotation);
+        return core.sessionAnnotate(params.id, params.annotation, params.authorName);
       }
       case "session.removeAnnotation": {
         const params = parseParams("session.removeAnnotation", request.params);

package/src/validate.ts

@@ -125,7 +125,11 @@ export const Params = {
       60_000,
     ),
   }),
-  "session.annotate": v.object({ id: SessionId, annotation: AnnotationSchema }),
+  "session.annotate": v.object({
+    id: SessionId,
+    annotation: AnnotationSchema,
+    authorName: v.optional(v.string()),
+  }),
   "session.removeAnnotation": v.object({ id: SessionId, annotationId: NonEmpty }),
   "session.setWorkingCopy": v.object({ id: SessionId, workingCopy: v.optional(v.string()) }),
   "session.setViewed": v.object({ id: SessionId, viewedPaths: v.array(v.string()) }),