don't count a jupyter notebook as opened until cells appear -- i.e., make the metric we're tracking much harsher to improve user experience; making opening notebooks faster

Author: williamstein

src/packages/frontend/jupyter/browser-actions.ts

@@ -153,11 +153,7 @@ export class JupyterActions extends JupyterActions0 {
       }
     });
 
-    // Put an entry in the project log once the jupyter notebook gets opened.
-    // NOTE: Obviously, the project does NOT need to put entries in the log.
-    this.syncdb.once("change", () =>
-      this.redux?.getProjectActions(this.project_id).log_opened_time(this.path),
-    );
+    this.initOpenLog();
 
     // project doesn't care about cursors, but browser clients do:
     this.syncdb.on("cursor_activity", this.syncdb_cursor_activity);
@@ -176,6 +172,23 @@ export class JupyterActions extends JupyterActions0 {
     }
   }
 
+  initOpenLog = () => {
+    // Put an entry in the project log once the jupyter notebook gets opened and
+    // shows cells.
+    const reportOpened = () => {
+      if (this._state == "closed") {
+        return;
+      }
+      if (this.syncdb.get_one({ type: "cell" }) != null) {
+        this.redux
+          ?.getProjectActions(this.project_id)
+          .log_opened_time(this.path);
+        this.syncdb.removeListener("change", reportOpened);
+      }
+    };
+    this.syncdb.on("change", reportOpened);
+  };
+
   initUsageInfo = async () => {
     while (this._state != "closed") {
       try {

src/packages/sync/editor/generic/sync-doc.ts

@@ -1027,14 +1027,11 @@ export class SyncDoc extends EventEmitter {
     this.assert_table_is_ready("syncstring");
     this.dbg("set_initialized")({ error, read_only, size });
     const init = { time: this.client.server_time(), size, error };
-    for (let i = 0; i < 3; i++) {
-      await this.set_syncstring_table({
-        init,
-        read_only,
-        last_active: this.client.server_time(),
-      });
-      await delay(1000);
-    }
+    await this.set_syncstring_table({
+      init,
+      read_only,
+      last_active: this.client.server_time(),
+    });
   };
 
   /* List of logical timestamps of the versions of this string in the sync
@@ -1463,7 +1460,7 @@ export class SyncDoc extends EventEmitter {
     log("update interest");
     this.initInterestLoop();
 
-    log("ensure syncstring exists in database (if not using NATS)");
+    log("ensure syncstring exists");
     this.assert_not_closed("initAll -- before ensuring syncstring exists");
     await this.ensure_syncstring_exists_in_db();
 

src/packages/util/smc-version.js

@@ -1,2 +1,2 @@
 /* autogenerated by the update_version script */
-exports.version=1752418521;
+exports.version=1752527312;