From 3b6777e25965fa6682b5f02574bbc7468b0f9208 Mon Sep 17 00:00:00 2001
From: Aria <weglossing@gmail.com>
Date: Tue, 5 Aug 2025 19:05:17 -0500
Subject: [PATCH 1/5] Fix signature of functions in log and logValues and add
 tests

---
 packages/core/pattern.mjs           |  4 +--
 packages/core/test/pattern.test.mjs | 40 ++++++++++++++++++++++++++++-
 2 files changed, 41 insertions(+), 3 deletions(-)

diff --git a/packages/core/pattern.mjs b/packages/core/pattern.mjs
index 4168645b8..d724bb853 100644
--- a/packages/core/pattern.mjs
+++ b/packages/core/pattern.mjs
@@ -852,14 +852,14 @@ export class Pattern {
     );
   }
 
-  log(func = (_, hap) => `[hap] ${hap.showWhole(true)}`, getData = (_, hap) => ({ hap })) {
+  log(func = (hap) => `[hap] ${hap.showWhole(true)}`, getData = (hap) => ({ hap })) {
     return this.onTrigger((...args) => {
       logger(func(...args), undefined, getData(...args));
     }, false);
   }
 
   logValues(func = id) {
-    return this.log((_, hap) => func(hap.value));
+    return this.log((hap) => func(hap.value));
   }
 
   //////////////////////////////////////////////////////////////////////
diff --git a/packages/core/test/pattern.test.mjs b/packages/core/test/pattern.test.mjs
index 93c4168c9..51e0acaef 100644
--- a/packages/core/test/pattern.test.mjs
+++ b/packages/core/test/pattern.test.mjs
@@ -6,7 +6,7 @@ This program is free software: you can redistribute it and/or modify it under th
 
 import Fraction from 'fraction.js';
 
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 
 import {
   TimeSpan,
@@ -55,6 +55,8 @@ import {
   expand,
 } from '../index.mjs';
 
+import { log, logValues } from '../pattern.mjs'
+
 import { steady } from '../signal.mjs';
 
 import { n, s } from '../controls.mjs';
@@ -1306,4 +1308,40 @@ describe('Pattern', () => {
       );
     });
   });
+  describe('log', () => {
+    it('logs to console', () => {
+      const mockConsoleLog = vi.spyOn(console, 'log').mockImplementation(() => {});
+      const pattern = pure('a').log()
+      const haps = pattern.queryArc(0, 1);  // query during first time arc
+
+      // Force a trigger
+      haps.forEach(hap => {
+        hap.context?.onTrigger?.(hap);
+      });
+
+      expect(mockConsoleLog).toHaveBeenCalledWith(
+        '%c[hap] 0/1 → 1/1: a',
+        'background-color: black;color:white;border-radius:15px',
+      );
+      mockConsoleLog.mockRestore();
+    });
+  });
+  describe('logValues', () => {
+    it('logs values to console', () => {
+      const mockConsoleLog = vi.spyOn(console, 'log').mockImplementation(() => {});
+      const pattern = pure('a').note("c#").log()
+      const haps = pattern.queryArc(0, 1);  // query during first time arc
+
+      // Force a trigger
+      haps.forEach(hap => {
+        hap.context?.onTrigger?.(hap);
+      });
+
+      expect(mockConsoleLog).toHaveBeenCalledWith(
+        '%c[hap] 0/1 → 1/1: value:a note:c#',
+        'background-color: black;color:white;border-radius:15px',
+      );
+      mockConsoleLog.mockRestore();
+    });
+  });
 });

From 23e5358ae424bfb851d23688c9e547ce2061ae4d Mon Sep 17 00:00:00 2001
From: Aria <weglossing@gmail.com>
Date: Tue, 5 Aug 2025 20:46:00 -0500
Subject: [PATCH 2/5] Formatting

---
 packages/core/test/pattern.test.mjs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/packages/core/test/pattern.test.mjs b/packages/core/test/pattern.test.mjs
index 51e0acaef..b8e7de504 100644
--- a/packages/core/test/pattern.test.mjs
+++ b/packages/core/test/pattern.test.mjs
@@ -55,7 +55,7 @@ import {
   expand,
 } from '../index.mjs';
 
-import { log, logValues } from '../pattern.mjs'
+import { log, logValues } from '../pattern.mjs';
 
 import { steady } from '../signal.mjs';
 
@@ -1311,11 +1311,11 @@ describe('Pattern', () => {
   describe('log', () => {
     it('logs to console', () => {
       const mockConsoleLog = vi.spyOn(console, 'log').mockImplementation(() => {});
-      const pattern = pure('a').log()
-      const haps = pattern.queryArc(0, 1);  // query during first time arc
+      const pattern = pure('a').log();
+      const haps = pattern.queryArc(0, 1);
 
       // Force a trigger
-      haps.forEach(hap => {
+      haps.forEach((hap) => {
         hap.context?.onTrigger?.(hap);
       });
 
@@ -1329,11 +1329,11 @@ describe('Pattern', () => {
   describe('logValues', () => {
     it('logs values to console', () => {
       const mockConsoleLog = vi.spyOn(console, 'log').mockImplementation(() => {});
-      const pattern = pure('a').note("c#").log()
-      const haps = pattern.queryArc(0, 1);  // query during first time arc
+      const pattern = pure('a').note('c#').log();
+      const haps = pattern.queryArc(0, 1);
 
       // Force a trigger
-      haps.forEach(hap => {
+      haps.forEach((hap) => {
         hap.context?.onTrigger?.(hap);
       });
 

From de15d79edf39fea50159ec990c268c8aa56fcde3 Mon Sep 17 00:00:00 2001
From: Aria <weglossing@gmail.com>
Date: Tue, 5 Aug 2025 22:29:59 -0500
Subject: [PATCH 3/5] Add doscstrings and move stringifyValues into its own
 function for reuse

---
 packages/core/hap.mjs               |  9 ++-------
 packages/core/pattern.mjs           | 19 ++++++++++++++++++-
 packages/core/test/pattern.test.mjs |  4 ++--
 packages/core/util.mjs              | 10 ++++++++++
 4 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/packages/core/hap.mjs b/packages/core/hap.mjs
index a6e3c55ad..5f820d644 100644
--- a/packages/core/hap.mjs
+++ b/packages/core/hap.mjs
@@ -4,6 +4,7 @@ Copyright (C) 2022 Strudel contributors - see <https://codeberg.org/uzu/strudel/
 This program is free software: you can redistribute it and/or modify it under the terms of the GNU Affero General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Affero General Public License for more details. You should have received a copy of the GNU Affero General Public License along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 import Fraction from './fraction.mjs';
+import { stringifyValues } from './util.mjs';
 
 export class Hap {
   /*
@@ -148,13 +149,7 @@ export class Hap {
   }
 
   showWhole(compact = false) {
-    return `${this.whole == undefined ? '~' : this.whole.show()}: ${
-      typeof this.value === 'object'
-        ? compact
-          ? JSON.stringify(this.value).slice(1, -1).replaceAll('"', '').replaceAll(',', ' ')
-          : JSON.stringify(this.value)
-        : this.value
-    }`;
+    return `${this.whole == undefined ? '~' : this.whole.show()}: ${stringifyValues(this.value, compact)}`;
   }
 
   combineContext(b) {
diff --git a/packages/core/pattern.mjs b/packages/core/pattern.mjs
index d724bb853..f64761f4f 100644
--- a/packages/core/pattern.mjs
+++ b/packages/core/pattern.mjs
@@ -21,6 +21,7 @@ import {
   numeralArgs,
   parseNumeral,
   pairs,
+  stringifyValues,
 } from './util.mjs';
 import drawLine from './drawLine.mjs';
 import { logger } from './logger.mjs';
@@ -852,13 +853,29 @@ export class Pattern {
     );
   }
 
+  /**
+   * Writes the content of the current event to the console, which is visible in the side menu
+   * or as the developer console.
+   * @name log
+   * @memberof Pattern
+   * @example
+   * s("bd sd").log()
+   */
   log(func = (hap) => `[hap] ${hap.showWhole(true)}`, getData = (hap) => ({ hap })) {
     return this.onTrigger((...args) => {
       logger(func(...args), undefined, getData(...args));
     }, false);
   }
 
-  logValues(func = id) {
+  /**
+   * A simplified version of `log` which writes all "values" (various configurable parameters)
+   * within the event to the console, which is visible in the side menu or as the developer console.
+   * @name logValues
+   * @memberof Pattern
+   * @example
+   * s("bd sd").gain("0.25 0.5 1").n("2 1 0").logValues()
+   */
+  logValues(func = (value) => `${stringifyValues(value, true)}`) {
     return this.log((hap) => func(hap.value));
   }
 
diff --git a/packages/core/test/pattern.test.mjs b/packages/core/test/pattern.test.mjs
index b8e7de504..b82d39b9d 100644
--- a/packages/core/test/pattern.test.mjs
+++ b/packages/core/test/pattern.test.mjs
@@ -1329,7 +1329,7 @@ describe('Pattern', () => {
   describe('logValues', () => {
     it('logs values to console', () => {
       const mockConsoleLog = vi.spyOn(console, 'log').mockImplementation(() => {});
-      const pattern = pure('a').note('c#').log();
+      const pattern = pure('a').note('c#').logValues();
       const haps = pattern.queryArc(0, 1);
 
       // Force a trigger
@@ -1338,7 +1338,7 @@ describe('Pattern', () => {
       });
 
       expect(mockConsoleLog).toHaveBeenCalledWith(
-        '%c[hap] 0/1 → 1/1: value:a note:c#',
+        '%cvalue:a note:c#',
         'background-color: black;color:white;border-radius:15px',
       );
       mockConsoleLog.mockRestore();
diff --git a/packages/core/util.mjs b/packages/core/util.mjs
index 756fac8e8..2e7c6e026 100644
--- a/packages/core/util.mjs
+++ b/packages/core/util.mjs
@@ -487,3 +487,13 @@ export function getCurrentKeyboardState() {
 //   }
 //   return lcm((x * y) / gcd(x, y), ...z);
 // };
+
+// Takes values -- typically derived from events, i.e. `hap`s -- and renders them
+// into a readable format
+export function stringifyValues(value, compact = false) {
+  return typeof value === 'object'
+    ? compact
+      ? JSON.stringify(value).slice(1, -1).replaceAll('"', '').replaceAll(',', ' ')
+      : JSON.stringify(value)
+    : value;
+}

From f22f89347e9dc8da263a56d30bde6f41187b5506 Mon Sep 17 00:00:00 2001
From: Aria <weglossing@gmail.com>
Date: Tue, 5 Aug 2025 22:40:11 -0500
Subject: [PATCH 4/5] Final docstring cleanup & adding hap tag to logValues for
 consistency

---
 packages/core/pattern.mjs           | 7 +++----
 packages/core/test/pattern.test.mjs | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/packages/core/pattern.mjs b/packages/core/pattern.mjs
index f64761f4f..dbf1b17f7 100644
--- a/packages/core/pattern.mjs
+++ b/packages/core/pattern.mjs
@@ -854,8 +854,7 @@ export class Pattern {
   }
 
   /**
-   * Writes the content of the current event to the console, which is visible in the side menu
-   * or as the developer console.
+   * Writes the content of the current event to the console (visible in the side menu).
    * @name log
    * @memberof Pattern
    * @example
@@ -869,13 +868,13 @@ export class Pattern {
 
   /**
    * A simplified version of `log` which writes all "values" (various configurable parameters)
-   * within the event to the console, which is visible in the side menu or as the developer console.
+   * within the event to the console (visible in the side menu).
    * @name logValues
    * @memberof Pattern
    * @example
    * s("bd sd").gain("0.25 0.5 1").n("2 1 0").logValues()
    */
-  logValues(func = (value) => `${stringifyValues(value, true)}`) {
+  logValues(func = (value) => `[hap] ${stringifyValues(value, true)}`) {
     return this.log((hap) => func(hap.value));
   }
 
diff --git a/packages/core/test/pattern.test.mjs b/packages/core/test/pattern.test.mjs
index b82d39b9d..1df5c8776 100644
--- a/packages/core/test/pattern.test.mjs
+++ b/packages/core/test/pattern.test.mjs
@@ -1338,7 +1338,7 @@ describe('Pattern', () => {
       });
 
       expect(mockConsoleLog).toHaveBeenCalledWith(
-        '%cvalue:a note:c#',
+        '%c[hap] value:a note:c#',
         'background-color: black;color:white;border-radius:15px',
       );
       mockConsoleLog.mockRestore();

From 2ab2cc5fc824c1abb6a8358e5e7a15680c2497db Mon Sep 17 00:00:00 2001
From: Aria <weglossing@gmail.com>
Date: Fri, 15 Aug 2025 13:27:08 -0500
Subject: [PATCH 5/5] Add example tests

---
 test/__snapshots__/examples.test.mjs.snap | 34 +++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/test/__snapshots__/examples.test.mjs.snap b/test/__snapshots__/examples.test.mjs.snap
index eb6c238d5..51dff68a1 100644
--- a/test/__snapshots__/examples.test.mjs.snap
+++ b/test/__snapshots__/examples.test.mjs.snap
@@ -5285,6 +5285,40 @@ exports[`runs examples > example "lock" example index 0 1`] = `
 ]
 `;
 
+exports[`runs examples > example "log" example index 0 1`] = `
+[
+  "[ 0/1 → 1/2 | s:bd ]",
+  "[ 1/2 → 1/1 | s:sd ]",
+  "[ 1/1 → 3/2 | s:bd ]",
+  "[ 3/2 → 2/1 | s:sd ]",
+  "[ 2/1 → 5/2 | s:bd ]",
+  "[ 5/2 → 3/1 | s:sd ]",
+  "[ 3/1 → 7/2 | s:bd ]",
+  "[ 7/2 → 4/1 | s:sd ]",
+]
+`;
+
+exports[`runs examples > example "logValues" example index 0 1`] = `
+[
+  "[ (0/1 → 1/3) ⇝ 1/2 | s:bd gain:0.25 n:2 ]",
+  "[ 0/1 ⇜ (1/3 → 1/2) | s:bd gain:0.5 n:1 ]",
+  "[ (1/2 → 2/3) ⇝ 1/1 | s:sd gain:0.5 n:1 ]",
+  "[ 1/2 ⇜ (2/3 → 1/1) | s:sd gain:1 n:0 ]",
+  "[ (1/1 → 4/3) ⇝ 3/2 | s:bd gain:0.25 n:2 ]",
+  "[ 1/1 ⇜ (4/3 → 3/2) | s:bd gain:0.5 n:1 ]",
+  "[ (3/2 → 5/3) ⇝ 2/1 | s:sd gain:0.5 n:1 ]",
+  "[ 3/2 ⇜ (5/3 → 2/1) | s:sd gain:1 n:0 ]",
+  "[ (2/1 → 7/3) ⇝ 5/2 | s:bd gain:0.25 n:2 ]",
+  "[ 2/1 ⇜ (7/3 → 5/2) | s:bd gain:0.5 n:1 ]",
+  "[ (5/2 → 8/3) ⇝ 3/1 | s:sd gain:0.5 n:1 ]",
+  "[ 5/2 ⇜ (8/3 → 3/1) | s:sd gain:1 n:0 ]",
+  "[ (3/1 → 10/3) ⇝ 7/2 | s:bd gain:0.25 n:2 ]",
+  "[ 3/1 ⇜ (10/3 → 7/2) | s:bd gain:0.5 n:1 ]",
+  "[ (7/2 → 11/3) ⇝ 4/1 | s:sd gain:0.5 n:1 ]",
+  "[ 7/2 ⇜ (11/3 → 4/1) | s:sd gain:1 n:0 ]",
+]
+`;
+
 exports[`runs examples > example "loop" example index 0 1`] = `
 [
   "[ 0/1 → 1/1 | s:casio loop:1 ]",