nixamp 0.7.16 → 0.7.18

package/dist/publish.d.ts

@@ -12,7 +12,14 @@ export interface PublishTarget {
      * gets a 401 in JSON, which is a caller hearing nothing.
      */
     audio?: string;
-    tracks: number;
+    /**
+     * How many tracks there are, asked at every heartbeat rather than once.
+     *
+     * Taken as a number, it was read before the library had finished loading --
+     * the port opens first now -- so a server with five thousand tracks
+     * advertised nought of them for as long as it stayed up.
+     */
+    tracks: () => number;
     nowPlaying: () => string;
     /**
      * The account this stream belongs to, from `nixamp login`.

package/dist/publish.js

@@ -58,7 +58,7 @@ export class Publisher {
                     name: this.target.name,
                     url: this.target.url,
                     ...(this.target.audio ? { audio: this.target.audio } : {}),
-                    tracks: this.target.tracks,
+                    tracks: this.target.tracks(),
                     nowPlaying: this.target.nowPlaying(),
                 }),
             });

package/dist/server.js

@@ -2937,9 +2937,9 @@ export async function serve(argv, version = "0.1.0") {
             name: options.name || hostname(),
             url: listen,
             audio,
-            // Asked of the engine rather than a variable, because the library is
-            // now read after the port opens and may still be arriving.
-            tracks: engine.snapshot(false).trackCount,
+            // Asked at every heartbeat rather than once, because the library is
+            // read after the port opens and is still arriving when this is made.
+            tracks: () => engine.snapshot(false).trackCount,
             // From `nixamp login`. The directory will not list a stream it cannot
             // attribute to somebody, because a listing is now a phone code that
             // costs money to answer.
@@ -2966,7 +2966,18 @@ export async function serve(argv, version = "0.1.0") {
     }
     if (options.publish !== "no" && publishable_) {
         const listen = shareLink(publishable_.url, listenKey, false);
-        const wanted = options.publish === "yes"
+        // Listed unless told otherwise.
+        //
+        // It used to ask, and a daemon has nobody to ask -- so a server started in
+        // the background was never listed, which meant nobody could find it, which
+        // meant it could not be paid for either. A stream that nobody can find is
+        // not a stream. What is published is the listening link, never the one
+        // that drives, and `--no-publish` is there for a server that should not be
+        // public at all.
+        //
+        // Still asked when a person is at the terminal to answer, because that is
+        // somebody who can say no.
+        const wanted = options.publish === "yes" || !process.stdin.isTTY
             ? true
             : await confirm(`\n  List this stream at ${DEFAULT_DIRECTORY}/directory so anyone can find it?\n  It publishes ${listen} — listen only, not the controls.`);
         if (wanted) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nixamp",
-  "version": "0.7.16",
+  "version": "0.7.18",
   "description": "It really whips the terminal's ass. A Winamp-shaped audio player for your terminal.",
   "license": "MIT",
   "type": "module",

package/src/publish.ts

@@ -21,7 +21,14 @@ export interface PublishTarget {
    * gets a 401 in JSON, which is a caller hearing nothing.
    */
   audio?: string;
-  tracks: number;
+  /**
+   * How many tracks there are, asked at every heartbeat rather than once.
+   *
+   * Taken as a number, it was read before the library had finished loading --
+   * the port opens first now -- so a server with five thousand tracks
+   * advertised nought of them for as long as it stayed up.
+   */
+  tracks: () => number;
   nowPlaying: () => string;
   /**
    * The account this stream belongs to, from `nixamp login`.
@@ -92,7 +99,7 @@ export class Publisher {
           name: this.target.name,
           url: this.target.url,
           ...(this.target.audio ? { audio: this.target.audio } : {}),
-          tracks: this.target.tracks,
+          tracks: this.target.tracks(),
           nowPlaying: this.target.nowPlaying(),
         }),
       });

package/src/server.ts

@@ -3394,9 +3394,9 @@ export async function serve(argv: string[], version = "0.1.0"): Promise<void> {
         name: options.name || hostname(),
         url: listen,
         audio,
-        // Asked of the engine rather than a variable, because the library is
-        // now read after the port opens and may still be arriving.
-        tracks: engine.snapshot(false).trackCount,
+        // Asked at every heartbeat rather than once, because the library is
+        // read after the port opens and is still arriving when this is made.
+        tracks: () => engine.snapshot(false).trackCount,
         // From `nixamp login`. The directory will not list a stream it cannot
         // attribute to somebody, because a listing is now a phone code that
         // costs money to answer.
@@ -3423,7 +3423,18 @@ export async function serve(argv: string[], version = "0.1.0"): Promise<void> {
 
   if (options.publish !== "no" && publishable_) {
     const listen = shareLink(publishable_.url, listenKey, false);
-    const wanted = options.publish === "yes"
+    // Listed unless told otherwise.
+    //
+    // It used to ask, and a daemon has nobody to ask -- so a server started in
+    // the background was never listed, which meant nobody could find it, which
+    // meant it could not be paid for either. A stream that nobody can find is
+    // not a stream. What is published is the listening link, never the one
+    // that drives, and `--no-publish` is there for a server that should not be
+    // public at all.
+    //
+    // Still asked when a person is at the terminal to answer, because that is
+    // somebody who can say no.
+    const wanted = options.publish === "yes" || !process.stdin.isTTY
       ? true
       : await confirm(`\n  List this stream at ${DEFAULT_DIRECTORY}/directory so anyone can find it?\n  It publishes ${listen} — listen only, not the controls.`);
 

package/web/dist/sw.js

@@ -1,5 +1,5 @@
 /* nixamp service worker — generated, do not edit */
-const CACHE = "nixamp-1788962400373";
+const CACHE = "nixamp-1788963208633";
 const PRECACHE = [
   "/",
   "/apple-touch-icon.png",