tux-review 0.1.3 → 0.1.5

package/client/tux-review.js

@@ -561,11 +561,24 @@
   // resize/scroll/DOM mutations (rAF-throttled) so markers follow layout
   // changes: splitter drag, carousel transforms, dialogs opening and
   // closing, route-level reflows. Invisible targets hide their pin and
-  // reveal it again when the element becomes visible (SPC 60).
+  // reveal it again when the element becomes visible (SPC 60). The
+  // overlay host is recomputed on every pass, so a pin restored while
+  // its dialog is closed re-parents into the dialog the moment it opens
+  // (and back out when it closes) — otherwise it would stay parked in
+  // the page layer, covered by the dialog backdrop.
   let pinRegistry = [];
   function positionPins() {
+    const layer = root() ? root().querySelector('[data-tux-layer]') : null;
     for (const entry of pinRegistry) {
-      const { pin, target, inTopLayer } = entry;
+      const { pin, target } = entry;
+      const host = overlayHostFor(target);
+      if (pin.parentElement !== host) {
+        host.appendChild(pin);
+        entry.inTopLayer = host !== layer;
+        pin.style.position = entry.inTopLayer ? 'fixed' : '';
+        entry.lastKey = '';
+      }
+      const inTopLayer = entry.inTopLayer;
       const rects = target.getClientRects();
       const rect = target.getBoundingClientRect();
       const visible = rects.length > 0 && rect.width > 0 && rect.height > 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tux-review",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "TUX - TUX UIX review system: attach structured, machine-readable feedback directly to running web UIs (pages, routes, components, elements, states) with a deterministic CLI, canonical JSON and LLM-friendly skills.",
   "license": "MIT",
   "author": {

package/src/cli.js

@@ -7,13 +7,20 @@
  *
  * @module cli
  */
+import { readFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { CliError, EXIT } from './errors.js';
 import { loadConfig } from './config.js';
 import { opShow, opCreate, opUpdate, opDelete, opClear, opExport, opIncorporate, opValidate } from './feedback.js';
 import { opDesignInstall, opDesignCreate, opLiveCreate, opDesignStart, opDesignStatus, opDesignStop } from './design.js';
 import { opLiveInstall, opLiveStart, opLiveStatus, opLiveStop } from './live.js';
 
-export const VERSION = '0.1.0';
+// Single source of truth is package.json — a hardcoded constant here
+// drifted to 0.1.0 after the 0.1.1 release and broke the install test.
+export const VERSION = JSON.parse(
+  readFileSync(join(dirname(fileURLToPath(import.meta.url)), '..', 'package.json'), 'utf8'),
+).version;
 
 const VALUE_FLAGS = new Set([
   'config', 'format', 'type', 'text', 'route', 'page', 'component', 'component-instance', 'instance',

package/src/server.js

@@ -303,8 +303,15 @@ function proxy(state, req, res) {
       const chunks = [];
       up.on('data', (c) => chunks.push(c));
       up.on('end', () => {
-        res.writeHead(up.statusCode, up.headers);
-        res.end(inject(Buffer.concat(chunks).toString('utf8')));
+        // injection lengthens the body — the upstream Content-Length must
+        // not be forwarded or clients truncate/wait on the response
+        const body = inject(Buffer.concat(chunks).toString('utf8'));
+        const headers = { ...up.headers };
+        delete headers['transfer-encoding'];
+        headers['content-length'] = String(Buffer.byteLength(body));
+        headers['cache-control'] = 'no-store';
+        res.writeHead(up.statusCode, headers);
+        res.end(body);
       });
       return;
     }