noteconnection 1.6.8 → 1.8.0

package/dist/src/copy.assets.contract.test.js

@@ -0,0 +1,130 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+const fs = __importStar(require("fs"));
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+class TempDir {
+    constructor(prefix) {
+        this.path = fs.mkdtempSync(path.join(fs.realpathSync(os.tmpdir()), `${prefix}-`));
+    }
+    child(relative) {
+        return path.join(this.path, relative);
+    }
+    mkdir(relative) {
+        const target = this.child(relative);
+        fs.mkdirSync(target, { recursive: true });
+        return target;
+    }
+    file(relative, content) {
+        const target = this.child(relative);
+        fs.mkdirSync(path.dirname(target), { recursive: true });
+        fs.writeFileSync(target, content, 'utf8');
+        return target;
+    }
+    cleanup() {
+        fs.rmSync(this.path, { recursive: true, force: true });
+    }
+}
+describe('copy-assets runtime-first contract', () => {
+    const repoRoot = path.resolve(__dirname, '..');
+    const copyAssetsPath = path.join(repoRoot, 'scripts', 'copy-assets.js');
+    let temp;
+    let srcDir;
+    let destDir;
+    let copyProjectAssets;
+    beforeAll(() => {
+        ({ copyProjectAssets } = require(copyAssetsPath));
+    });
+    beforeEach(() => {
+        temp = new TempDir('noteconnection-copy-assets');
+        srcDir = temp.mkdir('src');
+        destDir = temp.mkdir('dest');
+    });
+    afterEach(() => {
+        temp.cleanup();
+    });
+    test('excludes generated graph payloads by default', () => {
+        temp.file(path.join('src', 'app.js'), 'console.log("app");');
+        temp.file(path.join('src', 'data.js'), 'const graphData = {"nodes":[{"id":"A"}],"edges":[]}');
+        temp.file(path.join('src', 'graph_data.json'), '{"nodes":[{"id":"A"}],"edges":[]}');
+        copyProjectAssets({
+            src: srcDir,
+            dest: destDir,
+            logger: console
+        });
+        expect(fs.existsSync(path.join(destDir, 'app.js'))).toBe(true);
+        expect(fs.existsSync(path.join(destDir, 'data.js'))).toBe(false);
+        expect(fs.existsSync(path.join(destDir, 'graph_data.json'))).toBe(false);
+    });
+    test('copies generated graph payloads only when explicitly requested', () => {
+        temp.file(path.join('src', 'data.js'), 'const graphData = {"nodes":[{"id":"A"}],"edges":[]}');
+        temp.file(path.join('src', 'graph_data.json'), '{"nodes":[{"id":"A"}],"edges":[]}');
+        copyProjectAssets({
+            src: srcDir,
+            dest: destDir,
+            includeGeneratedGraphAssets: true,
+            logger: console
+        });
+        expect(fs.existsSync(path.join(destDir, 'data.js'))).toBe(true);
+        expect(fs.existsSync(path.join(destDir, 'graph_data.json'))).toBe(true);
+    });
+    test('full mode remains valid when repository head no longer ships bundled graph payload files', () => {
+        temp.file(path.join('src', 'app.js'), 'console.log("app");');
+        expect(() => copyProjectAssets({
+            src: srcDir,
+            dest: destDir,
+            includeGeneratedGraphAssets: true,
+            logger: console
+        })).not.toThrow();
+        expect(fs.existsSync(path.join(destDir, 'app.js'))).toBe(true);
+        expect(fs.existsSync(path.join(destDir, 'data.js'))).toBe(false);
+        expect(fs.existsSync(path.join(destDir, 'graph_data.json'))).toBe(false);
+    });
+    test('does not copy LFS pointer placeholders even in explicit full mode', () => {
+        temp.file(path.join('src', 'data.js'), [
+            'version https://git-lfs.github.com/spec/v1',
+            'oid sha256:abc123',
+            'size 177419264'
+        ].join('\n'));
+        copyProjectAssets({
+            src: srcDir,
+            dest: destDir,
+            includeGeneratedGraphAssets: true,
+            logger: console
+        });
+        expect(fs.existsSync(path.join(destDir, 'data.js'))).toBe(false);
+    });
+});

package/dist/src/core/PathBridge.js

@@ -185,8 +185,8 @@ const ALLOWED_BACKGROUND_FILE_EXTENSIONS = ['.exr', '.hdr'];
 const CONFIG_TARGET_ID_MAX_LENGTH = 512;
 const CONFIG_SHORTCUT_MAX_LENGTH = 64;
 const CONFIG_BACKGROUND_MAX_LENGTH = 128;
-const CONFIG_BRIGHTNESS_MIN = 0.01;
-const CONFIG_BRIGHTNESS_MAX = 10.0;
+const CONFIG_BRIGHTNESS_MIN = 0.0;
+const CONFIG_BRIGHTNESS_MAX = 0.1;
 const CONFIG_READER_MEDIA_SCALE_MIN = 0.1;
 const CONFIG_READER_MEDIA_SCALE_MAX = 3.0;
 const CONFIG_CONTROL_CHARACTER_PATTERN = /[\u0000-\u001f\u007f]/;
@@ -831,10 +831,13 @@ class PathBridge {
         this.unauthorizedClientTimers = new Map();
         this.outboundQueueState = new Map();
         this.nextMermaidRenderRequestId = 1;
+        this.readySettled = false;
         const resolvedOptions = typeof options === 'number'
             ? { port: options }
             : (options || {});
-        this.port = resolvedOptions.port || 9876;
+        this.port = Number.isFinite(Number(resolvedOptions.port))
+            ? Number(resolvedOptions.port)
+            : 9876;
         this.host = resolvedOptions.host || '127.0.0.1';
         this.authToken = typeof resolvedOptions.authToken === 'string' ? resolvedOptions.authToken.trim() : '';
         this.wss = new ws_1.WebSocketServer({
@@ -842,6 +845,22 @@ class PathBridge {
             host: this.host,
             maxPayload: MAX_INBOUND_MESSAGE_BYTES,
         });
+        this.readyPromise = new Promise((resolve, reject) => {
+            this.wss.once('listening', () => {
+                this.readySettled = true;
+                const address = this.wss.address();
+                if (address && typeof address === 'object') {
+                    this.port = Number(address.port || this.port);
+                }
+                resolve();
+            });
+            this.wss.once('error', (error) => {
+                if (!this.readySettled) {
+                    this.readySettled = true;
+                    reject(error);
+                }
+            });
+        });
         console.log(`[PathBridge] WebSocket Server started on ws://${this.host}:${this.port}`);
         console.log(`[PathBridge] Inbound frame limit ${BRIDGE_INBOUND_LIMIT_CONFIG.selectedMessageMb} MiB ` +
             `(recommended=${BRIDGE_INBOUND_LIMIT_CONFIG.recommendedMessageMb} MiB, ` +
@@ -1591,5 +1610,11 @@ class PathBridge {
         this.clients.clear();
         this.clientMeta.clear();
     }
+    waitUntilReady() {
+        return this.readyPromise;
+    }
+    getPort() {
+        return this.port;
+    }
 }
 exports.PathBridge = PathBridge;