@nextclaw/ui 0.27.1 → 0.27.2

package/src/features/chat/features/conversation/hooks/__tests__/use-session-conversation-controller.test.tsx

@@ -143,6 +143,47 @@ function createControllerParams(params: {
   };
 }
 
+describe('useSessionConversationController HTTP message identity', () => {
+  it.each(['send', 'sendSteering', 'sendPresetMessage'] as const)(
+    'supports %s over HTTP without randomUUID and keeps submission identities distinct',
+    async (action) => {
+      const params = createControllerParams({ isRunning: false });
+      const { result } = renderHook(() => useSessionConversationController(params));
+      const getRandomValues = globalThis.crypto.getRandomValues.bind(globalThis.crypto);
+      const now = vi.spyOn(Date, 'now').mockReturnValue(1780000000000);
+      vi.stubGlobal('crypto', { getRandomValues });
+      try {
+        await act(async () => {
+          if (action === 'sendPresetMessage') {
+            await result.current.sendPresetMessage('Update this session title');
+            await result.current.sendPresetMessage('Update this session title again');
+          } else {
+            await result.current[action]();
+            await result.current[action]();
+          }
+        });
+      } finally {
+        now.mockRestore();
+        vi.unstubAllGlobals();
+      }
+
+      expect(params.agent.send).toHaveBeenCalledTimes(2);
+      const envelopes = params.agent.send.mock.calls.map(([envelope]) => envelope);
+      expect(envelopes[0].message.id).not.toBe(envelopes[1].message.id);
+      for (const envelope of envelopes) {
+        expect(envelope.message.id).toMatch(/^user-/);
+        expect(envelope.idempotencyKey).toBe(envelope.message.id);
+        expect(envelope.sessionId).toBe('session-1');
+        if (action !== 'sendPresetMessage') {
+          expect(params.beginSubmission).toHaveBeenCalledWith(envelope, expect.any(Object));
+          expect(params.acceptSubmission).toHaveBeenCalledWith(envelope.message.id);
+        }
+      }
+      if (action === 'sendPresetMessage') expect(params.beginSubmission).not.toHaveBeenCalled();
+    },
+  );
+});
+
 describe('useSessionConversationController backend run queue', () => {
   it('preallocates one stable session identity for the first root message', async () => {
     const send = vi.fn<TestAgentSend>(async (envelope) => {

package/src/features/chat/features/conversation/hooks/use-session-conversation-controller.ts

@@ -191,9 +191,12 @@ function buildSubmissionEnvelope(
   sessionKey: string | null,
   delivery: NcpAgentSendEnvelope['delivery'] = 'queue',
 ): NcpAgentSendEnvelope | null {
+  // getRandomValues is also available on HTTP origins, unlike randomUUID.
+  const messageIdBytes = crypto.getRandomValues(new Uint8Array(16));
+  const messageId = `user-${Array.from(messageIdBytes, (byte) => byte.toString(16).padStart(2, '0')).join('')}`;
   const envelope = buildNcpRequestEnvelope({
     sessionId: sessionKey ?? undefined,
-    messageId: `user-${crypto.randomUUID()}`,
+    messageId,
     text: draft.composerSnapshot.text.trim(),
     attachments: [...draft.composerSnapshot.attachments],
     parts: deriveNcpMessagePartsFromComposer(

package/dist/assets/agents-E4nu-17c.js

@@ -1 +0,0 @@
-import{t as e}from"./agents-CfpN8PQc.js";export{e as AgentsPage};

package/dist/assets/apps-YFlJirpx.js

@@ -1 +0,0 @@
-import{st as e}from"./app-presenter-provider-2DwHRRle.js";export{e as AppsPanel};

package/dist/assets/cron-CH5c55CR.js

@@ -1 +0,0 @@
-import{K as e}from"./app-presenter-provider-2DwHRRle.js";export{e as CronJobResource};

package/dist/assets/doc-browser-ClDUWo9V.js

@@ -1 +0,0 @@
-import{Ct as e}from"./app-presenter-provider-2DwHRRle.js";export{e as DocBrowser};

package/dist/assets/inbox-BjBZIbgx.js

@@ -1 +0,0 @@
-import{z as e}from"./app-presenter-provider-2DwHRRle.js";export{e as InboxPage};

package/dist/assets/mcp-marketplace-page-Dor6H9IC.js

@@ -1 +0,0 @@
-import{ct as e}from"./app-presenter-provider-2DwHRRle.js";export{e as McpMarketplacePage};

package/dist/assets/panel-apps-C-izvtu9.js

@@ -1 +0,0 @@
-import{s as e,v as t}from"./theme-provider-BToFZGZG.js";import"./app-presenter-provider-2DwHRRle.js";export{t as PanelAppHostProvider,e as PanelAppRuntimeSurface};

package/dist/assets/remote-DvCdHjzA.js

@@ -1 +0,0 @@
-import{F as e}from"./app-presenter-provider-2DwHRRle.js";export{e as RemoteAccessPage};