@songsid/agend 2.3.3-beta.8 → 2.3.3

package/dist/fleet-manager.d.ts

@@ -1,6 +1,7 @@
 import { type HumanOrigin } from "./human-delivery-receipts.js";
 import { SettingsExecution } from "./settings-transaction.js";
 import { AppIconStore } from "./web-app-icon.js";
+import { type NamedTestResult } from "./tunnel/named-link-test.js";
 import { OfficeShareService } from "./office-share-service.js";
 import { type OfficeShare } from "./office-share.js";
 import type { LoginCodeOwner } from "./web-login.js";
@@ -1642,6 +1643,21 @@ export declare class FleetManager implements FleetContext, LifecycleContext, Arc
     private publicWebLink;
     /** #1723: the public link's preview tunnel — a manager and lease of its own, never the fleet's one tunnel lane. */
     private getPreviewTunnel;
+    private namedLinkTestRunning;
+    /**
+     * #2906 (the user, 2026-11-01): Settings → Access → Test connection, on the fixed-address values as filled in (the
+     * typed connector token, else the saved one). One at a time; the result is closed step codes only — the token is
+     * never logged or returned. Admin only, never over the public link (settings-api.ts).
+     */
+    testNamedLink(input: {
+        hostname: unknown;
+        port: unknown;
+        protocol: unknown;
+        token?: unknown;
+        current?: () => boolean;
+    }): Promise<NamedTestResult | {
+        busy: true;
+    }>;
     private getTunnelLane;
     private publicOwnerCurrent;
     /** Exact owning General. No primary-adapter or name fallback participates in authorization. */
@@ -2849,6 +2865,21 @@ export declare class FleetManager implements FleetContext, LifecycleContext, Arc
      * would have to make this async and change every caller.
      */
     notifyInstanceTopic(instanceName: string, text: string, extraOpts?: import("./channel/types.js").SendOpts): boolean;
+    /**
+     * notifyInstanceTopic's routing, handing back the send itself — and which world sent it — for a caller that must
+     * edit or delete the message later (#2897). Null when there is no adapter or no postable target (already logged).
+     */
+    private sendToInstanceTopic;
+    private parkedNoticesStore;
+    private get parkedNotices();
+    /**
+     * A held CLI dialog's notices: the instance-topic one and the General one, each edited in place while the
+     * dialog stays (one live message per place, never a pile), and taken back by resolveParkedNotice.
+     */
+    showParkedNotice(instanceName: string, topicText: string | null, generalText: string | null): void;
+    /** The held dialog is resolved, or the instance stopped or restarted: delete its notices (no-op when none). */
+    resolveParkedNotice(instanceName: string): void;
+    private postParkedNotice;
     /**
      * Post decision buttons whose callback ids are `<prefix><nonce>:<action>`.
      * The entry is registered before the send and rolled back if the send

package/dist/fleet-manager.js

@@ -27,6 +27,7 @@ import { clearPublicLinkOwner, readPublicLinkOwner, writePublicLinkOwner } from
 import { PUBLIC_PREVIEW_FRAME_SRC, PUBLIC_PREVIEW_TUNNEL_DIR, PublicPreview, publicPreviewWanted } from "./public-preview.js";
 import { CloudflaredProvider } from "./tunnel/cloudflared.js";
 import { ensureCloudflared } from "./tunnel/cloudflared-install.js";
+import { namedTestLane, runNamedLinkTest } from "./tunnel/named-link-test.js";
 import { OFFICE_SHARE_TUNNEL_DIR, OfficeShareService } from "./office-share-service.js";
 import { ALL_OFFICES, WHOLE_FLEET } from "./office-share.js";
 import { TunnelPurposeLane } from "./tunnel/purpose-lane.js";
@@ -80,6 +81,7 @@ import { TmuxManager } from "./tmux-manager.js";
 import { AccessManager } from "./channel/access-manager.js";
 import { IpcClient } from "./channel/ipc-bridge.js";
 import { usesChannelAddress } from "./channel/mcp-tools.js";
+import { ParkedNotices, PARKED_NOTICE_LEDGER_FILE } from "./parked-notices.js";
 import { formatFleetTime } from "./tz-utils.js";
 import { createAdapter } from "./channel/factory.js";
 import { isWebChannelEcho, WEB_ECHO_PREFIX } from "./web-channel-echo.js";
@@ -3079,6 +3081,7 @@ export class FleetManager {
         if (this.reloadPending)
             this.scheduleReconcile();
         void this.sweepOrphanedCancelButtons();
+        void this.parkedNotices.sweep();
     }
     /**
      * We are the post-update fleet: the update is over by definition. Clearing it here (rather than in the update
@@ -6583,6 +6586,32 @@ export class FleetManager {
         return this.previewTunnel ??= new ManagedTunnel({ dataDir: join(this.dataDir, PUBLIC_PREVIEW_TUNNEL_DIR),
             log: () => this.logger.warn("Public preview tunnel cleanup requires attention") });
     }
+    namedLinkTestRunning = false;
+    /**
+     * #2906 (the user, 2026-11-01): Settings → Access → Test connection, on the fixed-address values as filled in (the
+     * typed connector token, else the saved one). One at a time; the result is closed step codes only — the token is
+     * never logged or returned. Admin only, never over the public link (settings-api.ts).
+     */
+    async testNamedLink(input) {
+        if (this.namedLinkTestRunning)
+            return { busy: true };
+        this.namedLinkTestRunning = true;
+        try {
+            const typed = typeof input.token === "string" && input.token.trim() ? input.token : null;
+            const result = await runNamedLinkTest({ hostname: input.hostname, port: input.port, protocol: input.protocol, token: typed ?? process.env[PUBLIC_TUNNEL_TOKEN_ENV] ?? null }, {
+                healthPort: this.fleetConfig?.health_port ?? 19280,
+                ensureBinary: async (signal) => (await ensureCloudflared({ dataDir: this.dataDir, pinnedOnly: true, signal })).path,
+                liveMarker: values => this.publicWebLink?.liveNamedMarker(values) ?? null,
+                reserve: namedTestLane(this.getTunnelLane()),
+                current: input.current,
+            });
+            this.logger[result.cleanup ? "warn" : "info"]({ steps: result.steps.map(s => `${s.step}:${s.code}`), typedToken: !!typed, ...(result.cleanup ? { cleanup: result.cleanup } : {}) }, result.cleanup ? "Named link test: the test connector's stop could not be proven; the tunnel lane stays held" : "Named link test");
+            return result;
+        }
+        finally {
+            this.namedLinkTestRunning = false;
+        }
+    }
     getTunnelLane() {
         return this.tunnelPurposeLane ??= new TunnelPurposeLane(new ManagedTunnel({ dataDir: this.dataDir,
             log: () => this.logger.warn("Managed tunnel cleanup requires attention") }));
@@ -6958,7 +6987,8 @@ export class FleetManager {
                 : "\n" + t("dashboard.public_status", status.state, status.remainingSeconds ?? 0)
                     + (status.expiresAt ? t("dashboard.public_until", formatFleetTime(status.expiresAt)) : ""));
         const choices = [{ action: "local", label: t("dashboard.local_signin") },
-            ...(publicLinkSettings(this.fleetConfig?.web).allowed ? [{ action: "public", label: t("dashboard.public_open") }] : []),
+            ...(publicLinkSettings(this.fleetConfig?.web).allowed
+                ? [{ action: "public", label: t(publicLinkNamedHostname(this.fleetConfig?.web) ? "dashboard.public_open_named" : "dashboard.public_open") }] : []),
             ...(status.state !== "closed" ? [{ action: "close", label: t("dashboard.public_close") }] : [])];
         const posted = await this.postNonceButtonPrompt({
             prefix: "dashboard:", alertType: "login", instanceName: "dashboard", adapter, adapterId: owner.adapterId,
@@ -14278,11 +14308,22 @@ export class FleetManager {
      * would have to make this async and change every caller.
      */
     notifyInstanceTopic(instanceName, text, extraOpts) {
+        const send = this.sendToInstanceTopic(instanceName, text, extraOpts);
+        if (!send)
+            return false;
+        send.sent.catch(e => this.logger.warn({ err: e, instanceName }, send.failure));
+        return true;
+    }
+    /**
+     * notifyInstanceTopic's routing, handing back the send itself — and which world sent it — for a caller that must
+     * edit or delete the message later (#2897). Null when there is no adapter or no postable target (already logged).
+     */
+    sendToInstanceTopic(instanceName, text, extraOpts) {
         const adapter = this.getAdapterForInstance(instanceName) ?? this.adapter;
         if (!adapter) {
             // Early startup: adapters are created well after the fleet object exists.
             this.logger.warn({ instanceName }, "No adapter yet — instance topic notification not sent");
-            return false;
+            return null;
         }
         const channelCfg = this.getChannelConfig(this.getInstanceAdapterId(instanceName));
         const groupId = channelCfg?.group_id;
@@ -14291,17 +14332,13 @@ export class FleetManager {
         const worldId = this.getInstanceAdapterId(instanceName);
         if (threadId != null && groupId) {
             const out = this.renderPlaces(worldId, text, { threadId: String(threadId), ...extraOpts });
-            adapter.sendText(String(groupId), out.text, out.opts)
-                .catch(e => this.logger.warn({ err: e, instanceName }, "Failed to send instance topic notification"));
-            return true;
+            return { adapterId: worldId, sent: adapter.sendText(String(groupId), out.text, out.opts), failure: "Failed to send instance topic notification" };
         }
         // Classic instance: find its channelId from the classic manager
         const classicChatId = this.classicChannels?.getChannelIdByInstance(instanceName);
         if (classicChatId) {
             // #1990: a ClassicBot room's readers need not be in the fleet's group, so its notices name places as plain text.
-            adapter.sendText(classicChatId, stripPlaceRefs(text), extraOpts)
-                .catch(e => this.logger.warn({ err: e, instanceName }, "Failed to send classic notification"));
-            return true;
+            return { adapterId: worldId, sent: adapter.sendText(classicChatId, stripPlaceRefs(text), extraOpts), failure: "Failed to send classic notification" };
         }
         // Fallback: the instance has neither a topic nor a classic channel, so post
         // where fleet-wide notices go. Not the bare group id: on Discord that is a
@@ -14309,12 +14346,67 @@ export class FleetManager {
         const target = this.fleetNoticeTarget(worldId);
         if (target) {
             const out = this.renderPlaces(worldId, text, { ...target.opts, ...extraOpts });
-            adapter.sendText(target.chatId, out.text, out.opts)
-                .catch(e => this.logger.warn({ err: e, instanceName }, "Failed to send notification (no topic)"));
-            return true;
+            return { adapterId: worldId, sent: adapter.sendText(target.chatId, out.text, out.opts), failure: "Failed to send notification (no topic)" };
         }
         this.logger.warn({ instanceName }, "No postable target — instance topic notification not sent");
-        return false;
+        return null;
+    }
+    // ── Parked-dialog notices (#2897) ──
+    parkedNoticesStore = null;
+    get parkedNotices() {
+        this.parkedNoticesStore ??= new ParkedNotices({
+            file: join(this.dataDir, PARKED_NOTICE_LEDGER_FILE),
+            logger: this.logger,
+            post: (instanceName, place, text) => this.postParkedNotice(instanceName, place, text),
+            adapterFor: adapterId => this.worlds.get(adapterId)?.adapter
+                ?? (adapterId === this.getPrimaryAdapterId() ? this.adapter ?? undefined : undefined),
+            resolvedText: () => t("fleet.dialog_parked_resolved"),
+            // A ClassicBot room names places as plain text (#1990), as its post did.
+            render: (row, text) => row.place === "topic" && this.classicChannels?.getChannelIdByInstance(row.instanceName) === row.chatId
+                ? stripPlaceRefs(text) : this.renderPlaces(row.adapterId, text).text,
+        });
+        return this.parkedNoticesStore;
+    }
+    /**
+     * A held CLI dialog's notices: the instance-topic one and the General one, each edited in place while the
+     * dialog stays (one live message per place, never a pile), and taken back by resolveParkedNotice.
+     */
+    showParkedNotice(instanceName, topicText, generalText) {
+        if (topicText !== null)
+            void this.parkedNotices.show(instanceName, "topic", topicText);
+        if (generalText !== null)
+            void this.parkedNotices.show(instanceName, "general", generalText);
+    }
+    /** The held dialog is resolved, or the instance stopped or restarted: delete its notices (no-op when none). */
+    resolveParkedNotice(instanceName) {
+        void this.parkedNotices.resolve(instanceName);
+    }
+    async postParkedNotice(instanceName, place, text) {
+        let send;
+        if (place === "topic") {
+            send = this.sendToInstanceTopic(instanceName, text);
+        }
+        else {
+            // Where notifyFleetError posts, without its text throttle: the ledger is what keeps this to one message.
+            const general = Object.entries(this.fleetConfig?.instances ?? {})
+                .find(([, config]) => config.general_topic === true)?.[0];
+            const target = general ? null : this.fleetNoticeTarget();
+            if (general)
+                send = this.sendToInstanceTopic(general, text);
+            else if (this.adapter && target) {
+                const out = this.renderPlaces(this.getPrimaryAdapterId(), text, target.opts);
+                send = { adapterId: this.getPrimaryAdapterId(), sent: this.adapter.sendText(target.chatId, out.text, out.opts) };
+            }
+            else
+                send = null;
+        }
+        if (!send)
+            return null;
+        const sent = await send.sent;
+        const adapterId = send.adapterId ?? this.getPrimaryAdapterId();
+        if (!adapterId)
+            return null;
+        return { adapterId, chatId: sent.chatId, ...(sent.threadId !== undefined ? { threadId: sent.threadId } : {}), messageId: sent.messageId };
     }
     // ── Nonce-armed button prompts (hang / assist / exit / clear) ──
     //
@@ -21215,6 +21307,8 @@ Plus the operational skills (fleet-health, instance-lifecycle, scheduling, sessi
         for (const timer of this.cancelButtonIdleRetireTimers.values())
             clearTimeout(timer);
         this.cancelButtonIdleRetireTimers.clear();
+        // #2897: notices still being retired stay in the ledger; the next fleet's sweep finishes them.
+        this.parkedNoticesStore?.dispose();
         if (this.topicCleanupTimer) {
             clearInterval(this.topicCleanupTimer);
             this.topicCleanupTimer = null;