dry run

Author: maparent

apps/roam/src/components/settings/AdminPanel.tsx

@@ -253,11 +253,12 @@ const MigrationTab = (): React.ReactElement => {
   const enabled = getSetting("use-reified-relations");
   const [useMigrationResults, setMigrationResults] = useState<string>("");
   const [useOngoing, setOngoing] = useState<boolean>(false);
+  const [useDryRun, setDryRun] = useState<boolean>(false);
   const doMigrateRelations = async () => {
     setOngoing(true);
     try {
       const before = await countReifiedRelations();
-      const numProcessed = await migrateRelations();
+      const numProcessed = await migrateRelations(useDryRun);
       const after = await countReifiedRelations();
       if (after - before < numProcessed)
         setMigrationResults(
@@ -301,6 +302,15 @@ const MigrationTab = (): React.ReactElement => {
           disabled={!enabled || useOngoing}
           text="Migrate all relations"
         ></Button>
+        <Checkbox
+          className="left-6 inline-block"
+          defaultChecked={useDryRun}
+          onChange={(e) => {
+            const target = e.target as HTMLInputElement;
+            setDryRun(target.checked);
+          }}
+          labelElement={<>Dry run</>}
+        />
       </p>
       {useOngoing ? (
         <Spinner />

apps/roam/src/utils/migrateRelations.ts

@@ -10,7 +10,7 @@ import {
 
 const MIGRATION_PROP_NAME = "relation-migration";
 
-const migrateRelations = async (): Promise<number> => {
+const migrateRelations = async (dryRun = false): Promise<number> => {
   const authorized = getSetting("use-reified-relations");
   if (!authorized) return 0;
   const processed = new Set<string>();
@@ -20,40 +20,42 @@ const migrateRelations = async (): Promise<number> => {
     const key = `${rel.source}:${rel.relUid}:${rel.target}`;
     if (processed.has(key)) continue;
     processed.add(key);
-    const uid = (await createReifiedRelation({
-      sourceUid: rel.source,
-      destinationUid: rel.target,
-      relationBlockUid: rel.relUid,
-    }))!;
-    const sourceProps = getBlockProps(rel.source);
-    const dgDataOrig = sourceProps[DISCOURSE_GRAPH_PROP_NAME];
-    const dgData: Record<string, json> =
-      dgDataOrig !== null &&
-      typeof dgDataOrig === "object" &&
-      !Array.isArray(dgDataOrig)
-        ? dgDataOrig
-        : {};
-    const migrationDataOrig = dgData[MIGRATION_PROP_NAME];
-    let migrationData: Record<string, json> =
-      migrationDataOrig !== null &&
-      typeof migrationDataOrig === "object" &&
-      !Array.isArray(migrationDataOrig)
-        ? migrationDataOrig
-        : {};
-    if (migrationData[uid] !== undefined) {
-      console.debug(`reprocessed ${key}`);
+    if (!dryRun) {
+      const uid = (await createReifiedRelation({
+        sourceUid: rel.source,
+        destinationUid: rel.target,
+        relationBlockUid: rel.relUid,
+      }))!;
+      const sourceProps = getBlockProps(rel.source);
+      const dgDataOrig = sourceProps[DISCOURSE_GRAPH_PROP_NAME];
+      const dgData: Record<string, json> =
+        dgDataOrig !== null &&
+        typeof dgDataOrig === "object" &&
+        !Array.isArray(dgDataOrig)
+          ? dgDataOrig
+          : {};
+      const migrationDataOrig = dgData[MIGRATION_PROP_NAME];
+      let migrationData: Record<string, json> =
+        migrationDataOrig !== null &&
+        typeof migrationDataOrig === "object" &&
+        !Array.isArray(migrationDataOrig)
+          ? migrationDataOrig
+          : {};
+      if (migrationData[uid] !== undefined) {
+        console.debug(`reprocessed ${key}`);
+      }
+      // clean up old migration entries
+      migrationData = Object.fromEntries(
+        Object.entries(migrationData).filter(
+          ([uid]) =>
+            window.roamAlphaAPI.q(`[:find ?p :where [?p :block/uid "${uid}"]]`)
+              .length > 0,
+        ),
+      );
+      migrationData[uid] = new Date().valueOf();
+      dgData[MIGRATION_PROP_NAME] = migrationData;
+      setBlockProps(rel.source, { [DISCOURSE_GRAPH_PROP_NAME]: dgData });
     }
-    // clean up old migration entries
-    migrationData = Object.fromEntries(
-      Object.entries(migrationData).filter(
-        ([uid]) =>
-          window.roamAlphaAPI.q(`[:find ?p :where [?p :block/uid "${uid}"]]`)
-            .length > 0,
-      ),
-    );
-    migrationData[uid] = new Date().valueOf();
-    dgData[MIGRATION_PROP_NAME] = migrationData;
-    setBlockProps(rel.source, { [DISCOURSE_GRAPH_PROP_NAME]: dgData });
     numProcessed++;
   }
   return numProcessed;