« get me outta code hell

data: clean up Thing.composite.from debug messaging - hsmusic-wiki - HSMusic - static wiki software cataloguing collaborative creation
about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
author(quasar) nebula <qznebula@protonmail.com>2023-08-30 15:33:46 -0300
committer(quasar) nebula <qznebula@protonmail.com>2023-09-05 21:02:53 -0300
commit895712f5a0381c41557c6d306d6697019368bb7b (patch)
tree4eee69a5b6688b278bcd193446148733c67d5b04 /src
parent29580733b79872333f3f9e45d50d987218d334ea (diff)
data: clean up Thing.composite.from debug messaging
* print annotation next to every log message, instead of just
  the begin/end messages

* add Thing.composite.debug() to conveniently wrap one property
  access

* don't output (and don't access) track album in inspect.custom
  when depth < 0
Diffstat (limited to 'src')
-rw-r--r--src/data/things/thing.js69
-rw-r--r--src/data/things/track.js40
2 files changed, 60 insertions, 49 deletions
diff --git a/src/data/things/thing.js b/src/data/things/thing.js
index 78ff4c81..4fd6a26a 100644
--- a/src/data/things/thing.js
+++ b/src/data/things/thing.js
@@ -719,11 +719,18 @@ export default class Thing extends CacheableObject {
     from(firstArg, secondArg) {
       const debug = fn => {
         if (Thing.composite.from.debug === true) {
+          const label =
+            (annotation
+              ? color.dim(`[composite: ${annotation}]`)
+              : color.dim(`[composite]`));
           const result = fn();
           if (Array.isArray(result)) {
-            console.log(`[composite]`, ...result);
+            console.log(label, ...result.map(value =>
+              (typeof value === 'object'
+                ? inspect(value, {depth: 0, colors: true, compact: true, breakLength: Infinity})
+                : value)));
           } else {
-            console.log(`[composite]`, result);
+            console.log(label, result);
           }
         }
       };
@@ -919,7 +926,7 @@ export default class Thing extends CacheableObject {
           let valueSoFar = value;         // Set only for {update: true} compositions
           let exportDependencies = null;  // Set only for {compose: true} compositions
 
-          debug(() => color.bright(`begin composition (annotation: ${annotation})`));
+          debug(() => color.bright(`begin composition`));
 
           stepLoop: for (let i = 0; i < exposeSteps.length; i++) {
             const step = exposeSteps[i];
@@ -932,9 +939,9 @@ export default class Thing extends CacheableObject {
             const filteredDependencies = _filterDependencies(dependencies, step);
             const {continuation, continuationStorage} = _prepareContinuation(transform);
 
-            debug(() =>
-              `step #${i+1} - ${transform ? 'transform' : 'compute'} ` +
-              `with dependencies: ${inspect(filteredDependencies, {depth: 0})}`);
+            debug(() => [
+              `step #${i+1} - ${transform ? 'transform' : 'compute'}`,
+              `with dependencies:`, filteredDependencies]);
 
             const result =
               (transform
@@ -946,18 +953,15 @@ export default class Thing extends CacheableObject {
                 throw new TypeError(`Use continuation.exit() or continuation.raise() in {compose: true} compositions`);
               }
 
-              debug(() => `step #${i+1} - early-exit (inferred)`);
-              debug(() => `early-exit: ${inspect(result, {compact: true})}`);
-              debug(() => color.bright(`end composition (annotation: ${annotation})`));
-
+              debug(() => [`step #${i+1} - early-exit (inferred) ->`, result]);
+              debug(() => color.bright(`end composition`));
               return result;
             }
 
             switch (continuationStorage.returnedWith) {
               case 'exit':
-                debug(() => `step #${i+1} - result: early-exit (explicit)`);
-                debug(() => `early-exit: ${inspect(continuationStorage.providedValue, {compact: true})}`);
-                debug(() => color.bright(`end composition (annotation: ${annotation})`));
+                debug(() => [`step #${i+1} - result: early-exit (explicit) ->`, continuationStorage.providedValue]);
+                debug(() => color.bright(`end composition`));
                 return continuationStorage.providedValue;
 
               case 'raise':
@@ -986,7 +990,7 @@ export default class Thing extends CacheableObject {
 
           if (exportDependencies) {
             debug(() => [`raise dependencies:`, exportDependencies]);
-            debug(() => color.bright(`end composition (annotation: ${annotation})`));
+            debug(() => color.bright(`end composition`));
             return continuationIfApplicable(exportDependencies);
           }
 
@@ -998,9 +1002,9 @@ export default class Thing extends CacheableObject {
             valueSoFar !== noTransformSymbol &&
             base.expose.transform;
 
-          debug(() =>
-            `base - ${transform ? 'transform' : 'compute'} ` +
-            `with dependencies: ${inspect(filteredDependencies, {depth: 0})}`);
+          debug(() => [
+            `base - ${transform ? 'transform' : 'compute'}`,
+            `with dependencies:`, filteredDependencies]);
 
           if (base.flags.compose) {
             const {continuation, continuationStorage} = _prepareContinuation(transform);
@@ -1020,15 +1024,15 @@ export default class Thing extends CacheableObject {
 
               case 'exit':
                 debug(() => `base - result: early-exit (explicit)`);
-                debug(() => `early-exit: ${inspect(continuationStorage.providedValue, {compact: true})}`);
-                debug(() => color.bright(`end composition (annotation: ${annotation})`));
+                debug(() => [`early-exit:`, continuationStorage.providedValue]);
+                debug(() => color.bright(`end composition`));
                 return continuationStorage.providedValue;
 
               case 'raise':
                 exportDependencies = _assignDependencies(continuationStorage.providedDependencies, base);
                 debug(() => `base - result: raise`);
-                debug(() => `raise dependencies: ${inspect(exportDependencies, {compact: true})}`);
-                debug(() => color.bright(`end composition (annotation: ${annotation})`));
+                debug(() => [`raise dependencies:`, exportDependencies]);
+                debug(() => color.bright(`end composition`));
                 return continuationIfApplicable(exportDependencies);
             }
           } else {
@@ -1037,8 +1041,8 @@ export default class Thing extends CacheableObject {
                 ? base.expose.transform(valueSoFar, filteredDependencies)
                 : base.expose.compute(filteredDependencies));
 
-            debug(() => `base - non-compose (final) result: ${inspect(result, {compact: true})}`);
-            debug(() => color.bright(`end composition (annotation: ${annotation})`));
+            debug(() => [`base - non-compose (final) result:`, result]);
+            debug(() => color.bright(`end composition`));
 
             return result;
           }
@@ -1067,6 +1071,25 @@ export default class Thing extends CacheableObject {
       return constructedDescriptor;
     },
 
+    // Evaluates a function with composite debugging enabled, turns debugging
+    // off again, and returns the result of the function. This is mostly syntax
+    // sugar, but also helps avoid unit tests avoid accidentally printing debug
+    // info for a bunch of unrelated composites (due to property enumeration
+    // when displaying an unexpected result). Use as so:
+    //
+    //   Without debugging:
+    //     t.same(thing.someProp, value)
+    //
+    //   With debugging:
+    //     t.same(Thing.composite.debug(() => thing.someProp), value)
+    //
+    debug(fn) {
+      Thing.composite.from.debug = true;
+      const value = fn();
+      Thing.composite.from.debug = false;
+      return value;
+    },
+
     // -- Compositional steps for compositions to nest --
 
     // Provides dependencies exactly as they are (or null if not defined) to the
diff --git a/src/data/things/track.js b/src/data/things/track.js
index dc1f5f2a..8ddf3624 100644
--- a/src/data/things/track.js
+++ b/src/data/things/track.js
@@ -604,38 +604,26 @@ export class Track extends Thing {
       ]),
   };
 
-  [inspect.custom]() {
-    const base = Thing.prototype[inspect.custom].apply(this);
+  [inspect.custom](depth) {
+    const parts = [];
 
-    const rereleasePart =
-      (this.originalReleaseTrackByRef
-        ? `${color.yellow('[rerelease]')} `
-        : ``);
+    parts.push(Thing.prototype[inspect.custom].apply(this));
 
-    const {album, dataSourceAlbum} = this;
+    if (this.originalReleaseTrackByRef) {
+      parts.unshift(`${color.yellow('[rerelease]')} `);
+    }
 
-    const albumName =
-      (album
-        ? album.name
-        : dataSourceAlbum?.name);
-
-    const albumIndex =
-      albumName &&
-        (album
-          ? album.tracks.indexOf(this)
-          : dataSourceAlbum.tracks.indexOf(this));
-
-    const trackNum =
-      albumName &&
+    let album;
+    if (depth >= 0 && (album = this.album ?? this.dataSourceAlbum)) {
+      const albumName = album.name;
+      const albumIndex = album.tracks.indexOf(this);
+      const trackNum =
         (albumIndex === -1
           ? '#?'
           : `#${albumIndex + 1}`);
+      parts.push(` (${color.yellow(trackNum)} in ${color.green(albumName)})`);
+    }
 
-    const albumPart =
-      albumName
-        ? ` (${color.yellow(trackNum)} in ${color.green(albumName)})`
-        : ``;
-
-    return rereleasePart + base + albumPart;
+    return parts.join('');
   }
 }