textbrowser 0.54.8 → 0.54.10

package/CHANGES.md

@@ -1,5 +1,13 @@
 # CHANGES to `textbrowser`
 
+## 0.54.10
+
+- fix: more logging
+
+## 0.54.9
+
+- fix: logging worker issues
+
 ## 0.54.8
 
 - fix: worker issues

package/dist/activateCallback-es.js

@@ -55,10 +55,12 @@ async function activateCallback ({
   //  anyways); also important to avoid conflicts with
   //  already-running versions upon future sw updates
   log('Activate: Callback called');
+  log('Activate: Fetching files manifest', files);
   const r = await fetch(files);
   const {groups} = /** @type {import('./utils/WorkInfo.js').FilesObject} */ (
     await r.json()
   );
+  log('Activate: Manifest loaded; groups:', groups.length);
 
   /* eslint-disable jsdoc/reject-any-type -- Generic */
   /**
@@ -67,6 +69,7 @@ async function activateCallback ({
    */
   const addJSONFetch = (arr, path) => {
     /* eslint-enable jsdoc/reject-any-type -- Generic */
+    log('Activate: Queueing JSON fetch', basePath + path);
     arr.push(
       (async () => (await fetch(basePath + path)).json())()
     );
@@ -89,6 +92,13 @@ async function activateCallback ({
   const metadataFiles = [];
   groups.forEach(
     ({files: fileObjs, metadataBaseDirectory, schemaBaseDirectory}) => {
+      log(
+        'Activate: Processing group',
+        metadataBaseDirectory,
+        schemaBaseDirectory,
+        'files:',
+        fileObjs.length
+      );
       fileObjs.forEach(({file: {$ref: filePath}, metadataFile, schemaFile, name}) => {
         // We don't i18nize the name here
         dataFileNames.push(name);
@@ -98,6 +108,7 @@ async function activateCallback ({
       });
     }
   );
+  log('Activate: Awaiting queued fetches; works:', dataFiles.length);
   const promises = await Promise.all([
     ...dataFiles, ...schemaFiles, ...metadataFiles
   ]);
@@ -109,11 +120,14 @@ async function activateCallback ({
 
   log('Activate: Files fetched');
   const dbName = namespace + '-textbrowser-cache-data';
+  log('Activate: Deleting existing database', dbName);
   indexedDB.deleteDatabase(dbName);
   return new Promise((resolve, reject) => {
+    log('Activate: Opening database', dbName);
     const req = indexedDB.open(dbName);
     // @ts-expect-error Ok
     req.addEventListener('upgradeneeded', ({target: {result: db}}) => {
+      log('Activate: Database upgrade needed', dbName);
       db.onversionchange = () => {
         db.close();
         const err = /** @type {Error & {type: string}} */ (new Error('versionchange'));
@@ -122,6 +136,7 @@ async function activateCallback ({
       };
       dataFileResponses.forEach(({data: tableRows}, i) => {
         const dataFileName = dataFileNames[i];
+        log('Activate: Creating object store', dataFileName, 'rows:', tableRows.length);
         const store = db.createObjectStore('files-to-cache-' + dataFileName);
 
         const schemaFileResponse = schemaFileResponses[i];
@@ -180,6 +195,12 @@ async function activateCallback ({
         });
 
         const uniqueColumnIndexes = [...new Set(columnIndexes)];
+        log(
+          'Activate: Preparing rows for store',
+          dataFileName,
+          'unique indexes:',
+          uniqueColumnIndexes.length
+        );
 
         tableRows.forEach((tableRow, j) => {
           // Todo: Optionally send notice when complete
@@ -203,6 +224,7 @@ async function activateCallback ({
           // log('objRow', objRow);
           store.put(objRow, j);
         });
+        log('Activate: Finished object store population', dataFileName);
       });
     });
 
@@ -218,6 +240,7 @@ async function activateCallback ({
      * @param {Event & {error?: Error}} ev
      */
     const onerr = (ev) => {
+      log('Activate: Database request error event fired');
       const error = /** @type {Error & {type: string}} */ (
         ev.error ?? new Error('dbError')
       );

package/dist/sw-helper.js

@@ -184,7 +184,7 @@ function swHelper (self) {
    * @returns {Promise<void>}
    */
   async function install (time) {
-    post({type: 'beginInstall'});
+    await post({type: 'beginInstall'});
     log(`Install: Trying, attempt ${time}`);
     const now = Date.now();
     const json = await getJSON(/** @type {string} */ (pathToUserJSON));
@@ -237,11 +237,26 @@ function swHelper (self) {
       ...userDataFiles,
       ...stylesheets
     ];
+    log(`Install: Pre-fetching ${urlsToPrefetch.length} assets`);
+    const pendingAssets = new Set();
+    const pendingLogTimer = setInterval(() => {
+      if (pendingAssets.size) {
+        log(
+          `Install: Still pending ${pendingAssets.size} assets: ` +
+            JSON.stringify([...pendingAssets])
+        );
+      }
+    }, 10000);
     // .map((url) => url === 'index.html' ? new Request(url, {cache: 'reload'}) : url)
     try {
-      const cachePromises = urlsToPrefetch.map(async (urlToPrefetch) => {
+      const cachePromises = urlsToPrefetch.map(async (urlToPrefetch, idx) => {
         // This constructs a new URL object using the service worker's script
         //   location as the base for relative URLs.
+        pendingAssets.add(urlToPrefetch);
+        log(
+          `Install: Fetching asset ${idx + 1}/` +
+            `${urlsToPrefetch.length}: ${urlToPrefetch}`
+        );
         const url = new URL(urlToPrefetch, location.href);
         url.search += (url.search ? '&' : '?') + 'cache-bust=' + now;
         const request = new Request(url, {mode: 'no-cors'});
@@ -251,8 +266,12 @@ function swHelper (self) {
             throw new Error('request for ' + urlToPrefetch +
                         ' failed with status ' + response.statusText);
           }
-          return cache.put(urlToPrefetch, response);
+          await cache.put(urlToPrefetch, response);
+          log(`Install: Cached asset ${idx + 1}/${urlsToPrefetch.length}: ${urlToPrefetch}`);
+          pendingAssets.delete(urlToPrefetch);
+          return;
         } catch (error) {
+          pendingAssets.delete(urlToPrefetch);
           logError(
             /** @type {Error} */
             (error),
@@ -271,6 +290,8 @@ function swHelper (self) {
       );
       // Failing gives chance for a new client to re-trigger install?
       throw error;
+    } finally {
+      clearInterval(pendingLogTimer);
     }
 
     // An install update event will not be reported until controlled,
@@ -279,7 +300,7 @@ function swHelper (self) {
 
     // Although we only need one client to which to send
     //   arguments, we want to signal phase complete to all
-    post({type: 'finishedInstall'});
+    await post({type: 'finishedInstall'});
   }
 
   /**
@@ -288,7 +309,12 @@ function swHelper (self) {
    * @returns {Promise<void>}
    */
   async function activate (time) {
-    post({type: 'beginActivate'});
+    console.log('Activate event worker callback entered', time);
+    await post({
+      type: 'log',
+      message: `Activate: Event callback entered (attempt ${time})`
+    });
+    await post({type: 'beginActivate'});
     log(`Activate: Trying, attempt ${time}`);
     const [json, cacheNames] = await Promise.all([
       getJSON(/** @type {string} */ (pathToUserJSON)),
@@ -308,9 +334,12 @@ function swHelper (self) {
     await activateCallback({namespace, files, basePath, log});
     log('Activate: Database changes completed');
 
+    await self.clients.claim();
+    log('Activate: Clients claimed');
+
     log(`Activate: Posting finished message to clients`);
     // Signal phase complete to all clients
-    post({type: 'finishedActivate'});
+    await post({type: 'finishedActivate'});
   }
 
   // @ts-expect-error Ok
@@ -327,6 +356,7 @@ function swHelper (self) {
   });
 
   self.addEventListener('activate', (e) => {
+    console.log('sw activate event fired');
     // Erring is of no present use here:
     //   https://github.com/w3c/ServiceWorker/issues/659#issuecomment-384919053
     e.waitUntil(tryAndRetry(activate, 5 * minutes, 'Error activating'));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "textbrowser",
-  "version": "0.54.8",
+  "version": "0.54.10",
   "description": "Multilinear text browser",
   "type": "module",
   "main": "dist/index-es.min.js",

package/resources/activateCallback.js

@@ -55,10 +55,12 @@ export default async function activateCallback ({
   //  anyways); also important to avoid conflicts with
   //  already-running versions upon future sw updates
   log('Activate: Callback called');
+  log('Activate: Fetching files manifest', files);
   const r = await fetch(files);
   const {groups} = /** @type {import('./utils/WorkInfo.js').FilesObject} */ (
     await r.json()
   );
+  log('Activate: Manifest loaded; groups:', groups.length);
 
   /* eslint-disable jsdoc/reject-any-type -- Generic */
   /**
@@ -67,6 +69,7 @@ export default async function activateCallback ({
    */
   const addJSONFetch = (arr, path) => {
     /* eslint-enable jsdoc/reject-any-type -- Generic */
+    log('Activate: Queueing JSON fetch', basePath + path);
     arr.push(
       (async () => (await fetch(basePath + path)).json())()
     );
@@ -89,6 +92,13 @@ export default async function activateCallback ({
   const metadataFiles = [];
   groups.forEach(
     ({files: fileObjs, metadataBaseDirectory, schemaBaseDirectory}) => {
+      log(
+        'Activate: Processing group',
+        metadataBaseDirectory,
+        schemaBaseDirectory,
+        'files:',
+        fileObjs.length
+      );
       fileObjs.forEach(({file: {$ref: filePath}, metadataFile, schemaFile, name}) => {
         // We don't i18nize the name here
         dataFileNames.push(name);
@@ -98,6 +108,7 @@ export default async function activateCallback ({
       });
     }
   );
+  log('Activate: Awaiting queued fetches; works:', dataFiles.length);
   const promises = await Promise.all([
     ...dataFiles, ...schemaFiles, ...metadataFiles
   ]);
@@ -109,11 +120,14 @@ export default async function activateCallback ({
 
   log('Activate: Files fetched');
   const dbName = namespace + '-textbrowser-cache-data';
+  log('Activate: Deleting existing database', dbName);
   indexedDB.deleteDatabase(dbName);
   return new Promise((resolve, reject) => {
+    log('Activate: Opening database', dbName);
     const req = indexedDB.open(dbName);
     // @ts-expect-error Ok
     req.addEventListener('upgradeneeded', ({target: {result: db}}) => {
+      log('Activate: Database upgrade needed', dbName);
       db.onversionchange = () => {
         db.close();
         const err = /** @type {Error & {type: string}} */ (new Error('versionchange'));
@@ -122,6 +136,7 @@ export default async function activateCallback ({
       };
       dataFileResponses.forEach(({data: tableRows}, i) => {
         const dataFileName = dataFileNames[i];
+        log('Activate: Creating object store', dataFileName, 'rows:', tableRows.length);
         const store = db.createObjectStore('files-to-cache-' + dataFileName);
 
         const schemaFileResponse = schemaFileResponses[i];
@@ -180,6 +195,12 @@ export default async function activateCallback ({
         });
 
         const uniqueColumnIndexes = [...new Set(columnIndexes)];
+        log(
+          'Activate: Preparing rows for store',
+          dataFileName,
+          'unique indexes:',
+          uniqueColumnIndexes.length
+        );
 
         tableRows.forEach((tableRow, j) => {
           // Todo: Optionally send notice when complete
@@ -203,6 +224,7 @@ export default async function activateCallback ({
           // log('objRow', objRow);
           store.put(objRow, j);
         });
+        log('Activate: Finished object store population', dataFileName);
       });
     });
 
@@ -218,6 +240,7 @@ export default async function activateCallback ({
      * @param {Event & {error?: Error}} ev
      */
     const onerr = (ev) => {
+      log('Activate: Database request error event fired');
       const error = /** @type {Error & {type: string}} */ (
         ev.error ?? new Error('dbError')
       );