jazz-tools 0.18.37 → 0.18.38

package/dist/inspector/register-custom-element.js

@@ -1,5 +1,5 @@
 // src/inspector/register-custom-element.ts
 if (typeof window !== "undefined" && process.env.NODE_ENV === "development") {
-  import("./custom-element-SUVJ7CPN.js");
+  import("./custom-element-QESCMFY7.js");
 }
 //# sourceMappingURL=register-custom-element.js.map

package/dist/inspector/viewer/history-view.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"history-view.d.ts","sourceRoot":"","sources":["../../../src/inspector/viewer/history-view.tsx"],"names":[],"mappings":"AAAA,OAAO,EAKL,SAAS,EAET,UAAU,EAEX,MAAM,QAAQ,CAAC;AA0BhB,wBAAgB,WAAW,CAAC,EAC1B,OAAO,EACP,IAAI,GACL,EAAE;IACD,OAAO,EAAE,UAAU,CAAC;IACpB,IAAI,EAAE,SAAS,CAAC;CACjB,2CAyEA"}
+{"version":3,"file":"history-view.d.ts","sourceRoot":"","sources":["../../../src/inspector/viewer/history-view.tsx"],"names":[],"mappings":"AAAA,OAAO,EAKL,SAAS,EAET,UAAU,EAEX,MAAM,QAAQ,CAAC;AA2BhB,wBAAgB,WAAW,CAAC,EAC1B,OAAO,EACP,IAAI,GACL,EAAE;IACD,OAAO,EAAE,UAAU,CAAC;IACpB,IAAI,EAAE,SAAS,CAAC;CACjB,2CAoFA"}

package/package.json

@@ -187,7 +187,7 @@
   },
   "type": "module",
   "license": "MIT",
-  "version": "0.18.37",
+  "version": "0.18.38",
   "dependencies": {
     "@manuscripts/prosemirror-recreate-steps": "^0.1.4",
     "@scure/base": "1.2.1",
@@ -204,9 +204,9 @@
     "prosemirror-transform": "^1.9.0",
     "use-sync-external-store": "^1.5.0",
     "zod": "4.1.11",
-    "cojson": "0.18.37",
-    "cojson-storage-indexeddb": "0.18.37",
-    "cojson-transport-ws": "0.18.37"
+    "cojson": "0.18.38",
+    "cojson-storage-indexeddb": "0.18.38",
+    "cojson-transport-ws": "0.18.38"
   },
   "devDependencies": {
     "@scure/bip39": "^1.3.0",

package/src/inspector/tests/viewer/history-view.test.tsx

@@ -1,5 +1,5 @@
 // @vitest-environment happy-dom
-import { afterEach, beforeAll, describe, expect, it } from "vitest";
+import { afterEach, assert, beforeAll, describe, expect, it } from "vitest";
 import { createJazzTestAccount, setupJazzTestSync } from "jazz-tools/testing";
 import { co, z } from "jazz-tools";
 import {
@@ -45,7 +45,7 @@ describe("HistoryView", async () => {
       .create({ foo: "bar" }, account);
 
     render(
-      <HistoryView coValue={value.$jazz.raw} node={value.$jazz.localNode} />,
+      <HistoryView coValue={value.$jazz.raw} node={account.$jazz.localNode} />,
     );
 
     expect(
@@ -69,7 +69,10 @@ describe("HistoryView", async () => {
       value.$jazz.delete("certified");
 
       render(
-        <HistoryView coValue={value.$jazz.raw} node={value.$jazz.localNode} />,
+        <HistoryView
+          coValue={value.$jazz.raw}
+          node={account.$jazz.localNode}
+        />,
       );
 
       const history = [
@@ -82,8 +85,6 @@ describe("HistoryView", async () => {
         'Property "certified" has been deleted',
       ].toReversed(); // Default sort is descending
 
-      expect(screen.getAllByRole("row")).toHaveLength(history.length + 2);
-
       await waitFor(() => {
         expect(screen.getAllByRole("row")[2]?.textContent).toContain(
           account.$jazz.id,
@@ -92,6 +93,56 @@ describe("HistoryView", async () => {
 
       expect(extractActions()).toEqual(history);
     });
+
+    it("should render invalid changes", async () => {
+      const account2 = await createJazzTestAccount();
+      const group = co.group().create(account);
+      group.addMember(account2, "reader");
+
+      const Schema = co.map({
+        pet: z.string(),
+        age: z.number(),
+        certified: z.boolean().optional(),
+      });
+
+      const value = Schema.create(
+        { pet: "dog", age: 10, certified: false },
+        group,
+      );
+
+      const valueOnAccount2 = await Schema.load(value.$jazz.id, {
+        loadAs: account2,
+      });
+      assert(valueOnAccount2);
+
+      // This is invalid, since account2 is a reader
+      valueOnAccount2.$jazz.set("pet", "cat");
+
+      render(
+        <HistoryView
+          coValue={valueOnAccount2.$jazz.raw}
+          node={account2.$jazz.localNode}
+        />,
+      );
+
+      const history = [
+        'Property "pet" has been set to "dog"',
+        'Property "age" has been set to 10',
+        'Property "certified" has been set to false',
+
+        // Account2 can't write to the value
+        'Property "pet" has been set to "cat"Invalid transaction: Transactor has no write permissions',
+      ].toReversed(); // Default sort is descending
+
+      await waitFor(() => {
+        expect(screen.getAllByRole("row")[2]?.textContent).toContain(
+          account2.$jazz.id,
+        );
+      });
+
+      expect(extractActions()).toEqual(history);
+    });
+
     it("should render co.map changes with json", async () => {
       const d = new Date();
       const value = co
@@ -147,7 +198,10 @@ describe("HistoryView", async () => {
       value.$jazz.shift();
 
       render(
-        <HistoryView coValue={value.$jazz.raw} node={value.$jazz.localNode} />,
+        <HistoryView
+          coValue={value.$jazz.raw}
+          node={account.$jazz.localNode}
+        />,
       );
 
       const history = [
@@ -194,7 +248,10 @@ describe("HistoryView", async () => {
       value.$jazz.shift();
 
       render(
-        <HistoryView coValue={value.$jazz.raw} node={value.$jazz.localNode} />,
+        <HistoryView
+          coValue={value.$jazz.raw}
+          node={account.$jazz.localNode}
+        />,
       );
 
       const history = [
@@ -223,8 +280,11 @@ describe("HistoryView", async () => {
       const group3 = co.group().create(account);
       group3.addMember(group, "inherit");
 
-      const { container } = render(
-        <HistoryView coValue={group.$jazz.raw} node={group.$jazz.localNode} />,
+      render(
+        <HistoryView
+          coValue={group.$jazz.raw}
+          node={account.$jazz.localNode}
+        />,
       );
 
       const history = [

package/src/inspector/viewer/history-view.tsx

@@ -31,6 +31,7 @@ type HistoryEntry = {
   action: string;
   timestamp: Date;
   isValid: boolean;
+  validationErrorMessage: string | undefined;
 };
 
 export function HistoryView({
@@ -85,7 +86,18 @@ export function HistoryView({
     {
       id: "action",
       header: "Action",
-      accessor: (row) => row.action,
+      accessor: (row) => {
+        if (row.isValid) return row.action;
+
+        return (
+          <>
+            {row.action}
+            <span style={{ color: "red", display: "block" }}>
+              Invalid transaction: {row.validationErrorMessage}
+            </span>
+          </>
+        );
+      },
       sortable: false,
       filterable: true,
       sortFn: (a, b) => a.action.localeCompare(b.action),
@@ -131,10 +143,6 @@ function getTransactionChanges(
         readKey,
       ) ?? []
     );
-
-    // const decryptedString = coValue.core.verified.sessions.get(tx.txID.sessionID)?.impl.decryptNextTransactionChangesJson(tx.txID.txIndex, readKey);
-
-    // return decryptedString ? [decryptedString] : [];
   }
 
   return tx.changes ?? (tx.tx as any).changes ?? [];
@@ -150,6 +158,7 @@ function getHistory(coValue: RawCoValue): HistoryEntry[] {
       action: mapTransactionToAction(change, coValue),
       timestamp: new Date(tx.currentMadeAt),
       isValid: tx.isValid,
+      validationErrorMessage: tx.validationErrorMessage,
     }));
   });
 }