tux-review 0.1.6 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tux-review",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "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/design.js

@@ -11,6 +11,7 @@ import { canonicalJson } from './canonical.js';
 import { CONFIG_FILE, defaultConfig } from './config.js';
 import { SUPPORTED_FRAMEWORKS, designTemplate } from './templates.js';
 import { startServer, opLiveStatus, opLiveStop } from './live.js';
+import { deploySkills } from './skills.js';
 
 function textReport(lines) {
   return { stdout: lines.join('\n') + '\n', exit: EXIT.ok };
@@ -40,6 +41,7 @@ export function opDesignInstall(opts, spec) {
   if (spec.root) config.design.root = spec.root;
   const root = config.design.root ?? 'requirements';
   writeFileSync(configPath, canonicalJson(config) + '\n', 'utf8');
+  deploySkills(cwd);
   const report = {
     action: 'install',
     kind: 'design',

package/src/live.js

@@ -13,6 +13,7 @@ import { CliError, EXIT } from './errors.js';
 import { canonicalJson } from './canonical.js';
 import { canonicalTimestamp } from './ids.js';
 import { loadStore } from './store.js';
+import { deploySkills } from './skills.js';
 
 const SERVER_STATE_FILE = '.tux/server.json';
 
@@ -59,6 +60,7 @@ export function opLiveInstall(opts) {
     const m = devCommand.match(/--port[= ](\d+)/);
     if (m) devPort = Number(m[1]);
   }
+  deploySkills(cwd);
   const report = {
     action: 'install',
     kind: 'live',
@@ -87,6 +89,7 @@ export async function startServer(opts, spec) {
   if (spec.dryRun) {
     return { stdout: canonicalJson(spec.plan) + '\n', exit: EXIT.ok };
   }
+  deploySkills(opts.cwd);
   mkdirSync(join(opts.cwd, '.tux'), { recursive: true });
   let appPid = null;
   if (spec.cmd) {

package/src/server.js

@@ -287,6 +287,22 @@ function static_(state, res, path) {
   res.end(body);
 }
 
+// Transparent proxy response headers (SPC 34): forward every upstream header
+// except the hop-by-hop / length headers this server manages itself. Must
+// produce the same header set as py/tux/server.py::_proxy.
+function proxyHeaders(upHeaders, { html, bodyLength }) {
+  const headers = {};
+  for (const [k, v] of Object.entries(upHeaders)) {
+    const lk = k.toLowerCase();
+    if (lk === 'connection' || lk === 'transfer-encoding' || lk === 'content-length') continue;
+    if (html && lk === 'cache-control') continue;
+    headers[k] = v;
+  }
+  if (html) headers['cache-control'] = 'no-store';
+  headers['content-length'] = String(bodyLength);
+  return headers;
+}
+
 function proxy(state, req, res) {
   const target = new URL(state.target);
   const options = {
@@ -298,25 +314,17 @@ function proxy(state, req, res) {
     headers: { ...req.headers, host: target.host },
   };
   const upstream = httpRequest(options, (up) => {
-    const type = up.headers['content-type'] ?? '';
-    if (type.startsWith('text/html')) {
-      const chunks = [];
-      up.on('data', (c) => chunks.push(c));
-      up.on('end', () => {
-        // 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;
-    }
-    res.writeHead(up.statusCode, up.headers);
-    up.pipe(res);
+    const isHtml = (up.headers['content-type'] ?? '').startsWith('text/html');
+    const chunks = [];
+    up.on('data', (c) => chunks.push(c));
+    up.on('end', () => {
+      const raw = Buffer.concat(chunks);
+      // Injection lengthens the body, so the upstream Content-Length must be
+      // recomputed; non-HTML passes through as raw bytes (no text round-trip).
+      const out = isHtml ? Buffer.from(inject(raw.toString('utf8')), 'utf8') : raw;
+      res.writeHead(up.statusCode, proxyHeaders(up.headers, { html: isHtml, bodyLength: out.length }));
+      res.end(out);
+    });
   });
   upstream.on('error', (err) => {
     sendJson(res, 502, { error: { code: 'bad_gateway', message: `target unreachable: ${err.message}` } });

package/src/skills.js

@@ -0,0 +1,46 @@
+/**
+ * Canonical skill deployment (SPC sections 28, 32): `tux design install`,
+ * `tux live install` and the start-review commands deploy the packaged
+ * skills verbatim into the project's agent skill directories, so a package
+ * installation plus one CLI invocation leaves the project ready, and a
+ * package update refreshes the deployed copies on the next invocation.
+ *
+ * @module skills
+ */
+import { mkdirSync, readdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { CliError, EXIT } from './errors.js';
+
+/** Agent skill directories relative to the project root (SPC sections 28, 32). */
+export const AGENT_DIRS = ['.kilo', '.claude', '.hermes'];
+
+const packagedSkillsDir = join(dirname(fileURLToPath(import.meta.url)), '..', 'skills');
+
+/**
+ * Deploy every packaged `tux-<name>.md` skill verbatim into
+ * `<root>/<agent>/skills/<name>/SKILL.md` for each agent directory.
+ * Idempotent: deployed copies are rewritten on every run; files not
+ * shipped by this package are never touched. Returns the sorted names.
+ *
+ * @param {string} root project root directory
+ * @returns {string[]} deployed skill names, sorted
+ */
+export function deploySkills(root) {
+  const names = readdirSync(packagedSkillsDir)
+    .filter((f) => /^tux-.+\.md$/.test(f))
+    .sort()
+    .map((f) => f.replace(/\.md$/, ''));
+  try {
+    for (const agent of AGENT_DIRS) {
+      for (const name of names) {
+        const dst = join(root, agent, 'skills', name, 'SKILL.md');
+        mkdirSync(dirname(dst), { recursive: true });
+        writeFileSync(dst, readFileSync(join(packagedSkillsDir, `${name}.md`)));
+      }
+    }
+  } catch (e) {
+    throw new CliError(EXIT.general, `skill deployment failed: ${e.message}`);
+  }
+  return names;
+}