@troylar/flightdeck 2.11.1 → 2.12.0

package/CHANGELOG.md

@@ -2,6 +2,29 @@
 
 <!-- Generated by build.py from src/releases.json. Edit that file, not this one. -->
 
+## 2.12.0 · Nov 1, 2026
+
+AI assistant files, from the app.
+
+### New
+
+- **See and update AI assistant files without a command line** (Settings). Each file says where it stands. Update follows flightdeck install's rules: a copy is kept first, a file your team changed is never replaced, and one that changes while updating is left alone and named. A save in the instant between the last check and the write can still be overwritten, so close editors on these files first. Update never changes the app file itself. *Where:* Settings › AI assistant files
+- **Show and Copy any file, and merge requests** (Settings). Show and Copy give any file as this version ships it, for an AI assistant that can't open the folder. A file waiting to merge also has Copy merge request, and I merged it, which puts the waiting copy away beside the replaced files. *Where:* Settings › AI assistant files
+- **Keep skills in another folder** (Settings). In another folder saves and updates the skills in a folder your AI assistant reads, with their own record there, like flightdeck skills --to. *Where:* Settings › AI assistant files
+
+### Improved
+
+- **Folder health judges these files by the installer's rules** (Folder health). Folder health and flightdeck install now agree on each file. Its AI assistants items open AI assistant files. *Where:* Settings › Folder health
+
+## 2.11.2 · Oct 31, 2026
+
+Duplicated tabs and Conditions.
+
+### Fixed
+
+- **A duplicated tab always gets its own identity** (Runs). A tab duplicated on a slow computer takes its own identity even when the first window answers late, and follows the recording in the first window. *Where:* Duplicating a browser tab with a run open
+- **Conditions moves as soon as a newer draft arrives** (Editor). When a newer draft gives the selected step an Only when, Conditions comes out of More options right away. *Where:* The builder
+
 ## 2.11.1 · Oct 31, 2026
 
 Odd names in a folder, and notes that share an ID.

package/README.md

@@ -25,7 +25,7 @@
 ## Quick start
 
 1. `npx @troylar/flightdeck open` opens the app in your browser (needs Node 18 or later).
-2. No Node? Save [flightdeck.html](https://unpkg.com/@troylar/flightdeck@2.6.1/dist/flightdeck.html) from the npm package as a file (right-click, **Save link as…**) and open that file in Edge or Chrome.
+2. No Node? Save [flightdeck.html](https://unpkg.com/@troylar/flightdeck@2.12.0/dist/flightdeck.html) from the npm package as a file (right-click, **Save link as…**) and open that file in Edge or Chrome.
 3. Choose **Look around the examples** to try it, or **Start your team** to set up your team's folder.
 
 <p align="center"><a href="https://github.com/troylar/flightdeck/blob/main/docs/images/run-light.png" title="Open in GitHub's image viewer"><picture><source media="(prefers-color-scheme: dark)" srcset="https://raw.githubusercontent.com/troylar/flightdeck/main/docs/images/run-dark.png"><img src="https://raw.githubusercontent.com/troylar/flightdeck/main/docs/images/run-light.png" alt="A release in progress on the run screen: 7/15 answered in the header with Details, Screen share, Receipt and More; a strip saying you're recording, with Pause and Hand off and the run's details; Phases with each gate's state; the checklist with Next up pinned at its top; and the selected step, whose text shows the run's tag (Start the production deploy job v2026.10.16), with the values to copy, Record the result, Done, Done differently and Blocked" width="1000"></picture></a></p>
@@ -373,7 +373,7 @@ Thousands of runs open and refresh without stalling. Long lists show a page at a
 
 ## Skills for your AI assistant
 
-`flightdeck install` puts these in the team folder's `.flightdeck/skills/`, with an `AGENTS.md` that any assistant reads. They're plain `SKILL.md` files, so any assistant that supports skills can use them. Open the team folder with your assistant and ask for one by name:
+**Settings → AI assistant files** in the app (or `flightdeck install` in a terminal) puts these in the team folder's `.flightdeck/skills/`, with an `AGENTS.md` that any assistant reads. They're plain `SKILL.md` files, so any assistant that supports skills can use them. Open the team folder with your assistant and ask for one by name:
 
 | Skill | Does |
 |---|---|
@@ -388,6 +388,20 @@ Thousands of runs open and refresh without stalling. Long lists show a page at a
 
 In `AGENTS.md`, only the part between the `flightdeck:begin` and `flightdeck:end` markers is Flightdeck's; your Team notes below it are never touched. A file your team changed is left alone on update, and the new default waits in `.flightdeck/updates/<version>/` with a prompt for your assistant to merge it.
 
+### AI assistant files, from the app
+
+**Settings → AI assistant files** installs and updates `AGENTS.md` and the skills without a command line. Each file shows where it stands:
+
+| Status | Means |
+|---|---|
+| Up to date | The same as this version ships |
+| Update ready | An older default nobody changed; **Update** replaces it, after keeping a copy |
+| Not there yet, or Deleted, can be added again | **Update** adds it |
+| You changed it | Never replaced |
+| Waiting to merge | Your team changed it and the default changed too; the new default waits in `.flightdeck/updates/<version>/` |
+
+**Show** and **Copy** give any file as this version ships it, for an assistant that can't open the folder. **Copy merge request** copies the prompt for your assistant, and **I merged it** puts the waiting copy away beside the replaced files once it's merged. **In another folder** does the same for an assistant that reads skills from its own folder (`flightdeck skills --to`). The app and the command share the same rules for these files and the same record (`.flightdeck/installed.json`), so they never disagree; the command also updates `flightdeck.html` itself, which the app leaves alone. Folder health's **AI assistants** items open this screen.
+
 ## Safe by design
 
 <p align="center"><a href="https://github.com/troylar/flightdeck/blob/main/docs/images/safe-light.png" title="Open in GitHub's image viewer"><picture><source media="(prefers-color-scheme: dark)" srcset="https://raw.githubusercontent.com/troylar/flightdeck/main/docs/images/safe-dark.png"><img src="https://raw.githubusercontent.com/troylar/flightdeck/main/docs/images/safe-light.png" alt="Safe by design: nothing leaves your machine; no server or accounts; saves that check first; built for OneDrive; a careful command; accessible." width="1000"></picture></a></p>
@@ -402,7 +416,7 @@ Roles prevent ordinary mistakes, not attacks: anyone who can open the team folde
 
 ## Updating and when something goes wrong
 
-Run `flightdeck install <folder>` again with the new version (or `npx @troylar/flightdeck@<version> install <folder>`). It updates what's still a Flightdeck default and keeps what your team changed. [Updating Flightdeck](https://github.com/troylar/flightdeck/blob/main/docs/updating.md) walks through backups, mixed versions and [rolling back](https://github.com/troylar/flightdeck/blob/main/docs/updating.md#rolling-back).
+Run `flightdeck install <folder>` again with the new version (or `npx @troylar/flightdeck@<version> install <folder>`). It updates what's still a Flightdeck default and keeps what your team changed. Once the new `flightdeck.html` is in the folder, **Settings → AI assistant files** can update `AGENTS.md` and the skills from the app the same way. [Updating Flightdeck](https://github.com/troylar/flightdeck/blob/main/docs/updating.md) walks through backups, mixed versions and [rolling back](https://github.com/troylar/flightdeck/blob/main/docs/updating.md#rolling-back).
 
 When the app shows a warning, **Settings → When something goes wrong** explains it in the app; the same guide is [online](https://github.com/troylar/flightdeck/blob/main/docs/recovery.md), covering conflict copies, lost folder access, OneDrive copies and restoring from a backup.
 

package/bin/flightdeck.js

@@ -186,27 +186,40 @@ function appStatus(dir, file, html, mine, force) {
    A copy still waiting in .flightdeck/updates/<any version>/ keeps a file at merge until someone deletes the copy.
    With --force, kept and merge files are replaced too, after a copy is saved. */
 function classify(t, files, base, force) {
-  return files.map((f) => {
+  const items = files.map((f) => {
     const parts = f.rel.split('/'), now = readInside(t.dir, parts), want = hash(managed(f.key, f.text));
-    const item = { key: f.key, rel: f.rel, parts, text: f.text, now, want, staged: staged(t, f.rel) };
-    if (now === null) return Object.assign(item, { status: t.record.files[f.key] ? 'missing' : 'new' });
-    const have = hash(managed(f.key, now));
-    /* Still waiting on a merge. When this version's default is new to the person (not already staged, not what the file
-       already says), it's staged too, so an older pending merge never hides newer instructions. */
-    if (item.staged.length && !force) return Object.assign(item, { status: 'merge', waiting: item.staged.map((x) => x.version).join(', '),
-      stage: have !== want && !item.staged.some((x) => x.version === t.version || hash(managed(f.key, x.text)) === want) });
-    if (have === want) return Object.assign(item, { status: 'current' });
-    if ((base[f.baseKey || baselineKey(f.key)] || {})[have]) return Object.assign(item, { status: 'update' });
-    if (force) return Object.assign(item, { status: 'forced' });
-    if (t.record.files[f.key] === want) return Object.assign(item, { status: 'kept' });
-    return Object.assign(item, { status: 'merge' });
+    return { key: f.key, rel: f.rel, baseKey: f.baseKey, parts, text: f.text, now, want, have: now === null ? null : hash(managed(f.key, now)), staged: staged(t, f.rel) };
+  });
+  const why = decide(items.map((i) => ({ key: i.key, baseKey: i.baseKey, want: i.want, have: i.have, staged: i.staged.map((x) => ({ version: x.version, hash: hash(managed(i.key, x.text)) })) })), t.record, base, t.version, force);
+  return items.map((i, n) => Object.assign(i, why[n], { key: i.key }));
+}
+/* The decision alone, from hashes: the same as PF.contract.classify (test/assistant-files.test.js checks they agree), plus
+   forced, which only --force gives. */
+function decide(items, record, base, version, force) {
+  const rf = (record && record.files) || {};
+  return items.map((it) => {
+    const out = { key: it.key, status: '' }, st = it.staged || [];
+    if (it.have === null || it.have === undefined) { out.status = rf[it.key] ? 'missing' : 'new'; return out; }
+    if (st.length && !force) {
+      out.status = 'merge'; out.waiting = st.map((x) => x.version).join(', ');
+      out.stage = it.have !== it.want && !st.some((x) => x.version === version || x.hash === it.want);
+      return out;
+    }
+    if (it.have === it.want) out.status = 'current';
+    else if ((base[it.baseKey || baselineKey(it.key)] || {})[it.have]) out.status = 'update';
+    else if (force) out.status = 'forced';
+    else if (rf[it.key] === it.want) out.status = 'kept';
+    else out.status = 'merge';
+    return out;
   });
 }
 /* The versions under the updates folder, and which of them hold a copy of rel still waiting to be merged. */
+/* Oldest first, dev last: the order install and the app both list waiting versions in. */
+function byVersion(a, b) { return a === b ? 0 : a === 'dev' ? 1 : b === 'dev' ? -1 : cmp(a, b); }
 function versionsStaged(t) {
   if (t.versions) return t.versions;
   const home = folders(t.dir, t.updates, false);
-  t.versions = home ? fs.readdirSync(home).filter((v) => /^(\d+\.\d+\.\d+|dev)$/.test(v) && folders(t.dir, t.updates.concat(v), false)) : [];
+  t.versions = home ? fs.readdirSync(home).filter((v) => /^(\d+\.\d+\.\d+|dev)$/.test(v) && folders(t.dir, t.updates.concat(v), false)).sort(byVersion) : [];
   return t.versions;
 }
 function staged(t, rel) {
@@ -230,12 +243,19 @@ function installedNewer(r) {
 /* The folder's record, checked field by field. Anyone can edit it, so a record that isn't exactly what this command writes is
    ignored (and said so), and only entries for files this version installs, with a well-formed hash, are read. */
 function readRecord(t, keys) {
-  const text = readInside(t.dir, t.recordAt), none = { files: {} };
-  if (text === null) return none;
+  t.recordText = readInside(t.dir, t.recordAt);
+  const r = parseRecord(t.recordText, keys);
+  if (r.newer) fail(path.join(t.dir, ...t.recordAt) + ' ' + r.newer + ' Nothing was changed. Use the newer Flightdeck package.');
+  return r;
+}
+/* The record's text as install reads it: the same as PF.contract.readRecord (test/assistant-files.test.js checks they agree). */
+function parseRecord(text, keys) {
+  const none = { files: {} };
+  if (text === null || text === undefined) return none;
   let r = null;
-  try { r = JSON.parse(text); } catch (e) { /* reported below */ }
+  try { r = JSON.parse(text); } catch (e) { /* ignored below */ }
   const newer = installedNewer(r);
-  if (newer) fail(path.join(t.dir, ...t.recordAt) + ' ' + newer + ' Nothing was changed. Use the newer Flightdeck package.');
+  if (newer) return { newer };
   const ok = r && typeof r === 'object' && !Array.isArray(r) && r.format === 'flightdeck-installed/1' && typeof r.version === 'string' && /^(\d+\.\d+\.\d+|dev)$/.test(r.version)
     && typeof r.at === 'string' && (r.by === 'cli' || r.by === 'app') && r.files && typeof r.files === 'object' && !Array.isArray(r.files)
     && Object.keys(r.files).every((k) => typeof r.files[k] === 'string' && /^[0-9a-f]{64}$/.test(r.files[k]));
@@ -248,19 +268,21 @@ function readRecord(t, keys) {
 function stamp(d) { const p = (n) => String(n).padStart(2, '0'); return d.getFullYear() + '-' + p(d.getMonth() + 1) + '-' + p(d.getDate()) + ' ' + p(d.getHours()) + p(d.getMinutes()); }
 function fineStamp(d) { const p = (n, w) => String(n).padStart(w || 2, '0'); return stamp(d).replace(' ', '-') + p(d.getSeconds()) + '-' + p(d.getMilliseconds(), 3); }
 
-function mergeReadme(version, items, t) {
-  const here = t.updates.concat(version).join('/');
+function mergeReadme(version, items, t) { return mergeReadmeText(version, items, t.updates, t.home); }
+/* The same text as PF.contract.mergeReadme (test/assistant-files.test.js checks they agree). */
+function mergeReadmeText(version, items, updates, home) {
+  const here = updates.concat(version).join('/');
   return `# Flightdeck ${version}: files to merge
 
 Flightdeck ${version} changed these files, and your team changed them too, so the update left your versions in place:
 
 ${items.map((i) => '- `' + i.rel + '`' + (i.key === AGENTS ? ' (only the part between the flightdeck markers)' : '')).join('\n')}
 
-The new default of each file is in this folder (\`${here}/\`), at the same path it has in ${t.home}.
+The new default of each file is in this folder (\`${here}/\`), at the same path it has in ${home}.
 
 ## Ask your AI assistant to merge them
 
-Open your AI assistant in ${t.home}, and give it this:
+Open your AI assistant in ${home}, and give it this:
 
 > Flightdeck ${version} put new default versions of some files in \`${here}/\`; its README.md lists them. For each one, compare the new default with our file at the same path. Keep our own changes, and bring in what the new default adds or changes. In AGENTS.md, change only the part between \`<!-- flightdeck:begin -->\` and \`<!-- flightdeck:end -->\`; anything that is ours belongs under Team notes, below the end marker. Show me each change before you save it. When a file is merged, delete its copy here. When only README.md is left, delete this folder.
 
@@ -268,14 +290,61 @@ Then run the same flightdeck command with \`--check\`: exit code 0 means nothing
 `;
 }
 
+/* What an install would write has to be something the app can read back (PF.contract.MAX_BYTES): over the limit, nothing is
+   touched. The same check and words as Settings, AI assistant files (afRoom). */
+const CONTRACT_MAX = 8 * 1048576;
+function roomFor(t, items) {
+  const why = (name, n, then) => name + ' would be too big for Flightdeck to read back (' + (Math.ceil(n / 1048576 * 100) / 100) + ' MB, over ' + (CONTRACT_MAX / 1048576) + ' MB). Nothing was changed. ' + then;
+  for (const i of items) {
+    if (['current', 'kept', 'merge'].includes(i.status)) continue;
+    const n = Buffer.byteLength(merged(i), 'utf8');
+    if (n > CONTRACT_MAX) fail(why(i.rel, n, 'Shorten what your team added to it, then try again.'));
+  }
+  const rec = nextRecord(t.record, items, t.version);
+  if (!rec) return;
+  const size = Buffer.byteLength(JSON.stringify({ format: 'flightdeck-installed/1', version: t.version, at: new Date().toISOString(), by: 'cli', files: rec }, null, 2) + '\n', 'utf8');
+  if (size > CONTRACT_MAX) fail(why(t.recordAt.join('/'), size, 'Remove the entries for files Flightdeck no longer ships, then try again.'));
+}
+/* Something in the way of where a file goes: the same words as PF.contract.outcomeWords. Every destination is looked at before
+   anything is written, so a predictable refusal never leaves an update half done. */
+function blockedWhy(rel, want) { return rel + (want === 'file' ? ' is a folder, where Flightdeck needs a file.' : ' is a file, where Flightdeck needs a folder.') + ' Nothing was changed. Move it, then try again.'; }
+function clearFor(dir, parts) {
+  let cur = dir;
+  for (let i = 0; i < parts.length; i++) {
+    cur = path.join(cur, parts[i]);
+    const st = lstatOrNull(cur), rel = parts.slice(0, i + 1).join('/');
+    if (!st) return;
+    if (st.isSymbolicLink()) fail('refusing to write through a link: ' + cur);
+    if (i === parts.length - 1 ? !st.isFile() : !st.isDirectory()) fail(blockedWhy(rel, i === parts.length - 1 ? 'file' : 'folder'));
+  }
+}
+function destinationsClear(t, items, more) {
+  const dests = (more || []).slice();
+  for (const i of items) {
+    if (['current', 'kept'].includes(i.status)) continue;
+    if (i.status === 'merge') {
+      if (i.waiting && !i.stage) continue;
+      dests.push(t.updates.concat(t.version, i.rel.split('/')), t.updates.concat(t.version, 'README.md'), t.updates.concat(t.version, 'replaced', 'x'));
+      continue;
+    }
+    dests.push(i.parts);
+    if (i.now !== null && i.now !== merged(i)) dests.push(t.updates.concat(t.version, 'replaced', 'x'));
+  }
+  if (nextRecord(t.record, items, t.version)) dests.push(t.recordAt);
+  dests.forEach((parts) => clearFor(t.dir, parts));
+}
 /* install and skills: one routine for the team folder, and for skills --to <dir>. */
 function contract(t, files, opts) {
   const base = baselines(opts.html), now = new Date();
+  if (!opts.check || opts.room) clearFor(t.dir, t.updates.concat(t.version, 'x'));
   inside(t.dir, t.updates.concat(t.version, 'replaced'));
   t.record = readRecord(t, files.map((f) => f.key));
   t.changed = [];
   const items = classify(t, files, base, opts.force);
+  if (!opts.check || opts.room) { roomFor(t, items); destinationsClear(t, items, opts.more); }
   if (opts.check) return items;
+  /* The merge list as it was before anything is written: the only text it may replace. */
+  const readmeAt = t.updates.concat(t.version, 'README.md'), readmeWas = items.some((i) => i.status === 'merge' && (!i.waiting || i.stage)) ? readInside(t.dir, readmeAt) : null;
   for (const i of items) {
     if (['current', 'kept'].includes(i.status) || (i.status === 'merge' && i.waiting && !i.stage)) {
       if (opts.force) i.staged.forEach((x) => fileAway(t, x));
@@ -293,13 +362,34 @@ function contract(t, files, opts) {
     }
   }
   const merges = items.filter((i) => i.status === 'merge' && !i.changed);
-  if (merges.some((i) => !i.waiting || i.stage)) safeWrite(t.dir, t.updates.concat(t.version, 'README.md'), mergeReadme(t.version, merges.filter((i) => !i.waiting || i.stage || i.staged.some((x) => x.version === t.version)), t));
-  const rec = Object.assign({}, t.record.all || {});
-  items.forEach((i) => { if (!i.changed) rec[i.key] = i.want; });
-  if (!t.record.ignored && t.record.version === t.version && JSON.stringify(rec) === JSON.stringify(t.record.all)) return items;
-  safeWrite(t.dir, t.recordAt, JSON.stringify({ format: 'flightdeck-installed/1', version: t.version, at: now.toISOString(), by: 'cli', files: rec }, null, 2) + '\n');
+  if (merges.some((i) => !i.waiting || i.stage)) {
+    const next = mergeReadme(t.version, merges.filter((i) => !i.waiting || i.stage || i.staged.some((x) => x.version === t.version)), t);
+    try {
+      if (readmeWas !== null && readmeWas !== next) keepCopy(t, ['merge-README.md'], readmeWas);
+      safeWrite(t.dir, readmeAt, next, readmeWas);
+    } catch (e) {
+      if (!(e instanceof Changed)) throw e;
+      t.changed.push(readmeAt.join('/'));
+    }
+  }
+  const rec = nextRecord(t.record, items, t.version);
+  if (!rec) return items;
+  try {
+    safeWrite(t.dir, t.recordAt, JSON.stringify({ format: 'flightdeck-installed/1', version: t.version, at: now.toISOString(), by: 'cli', files: rec }, null, 2) + '\n', t.recordText);
+    t.recordWritten = true;
+  } catch (e) {
+    if (!(e instanceof Changed)) throw e;
+    t.changed.push(t.recordAt.join('/'));
+  }
   return items;
 }
+/* The record's files after an update, or null when it would be written unchanged: the same as PF.contract.nextRecord. */
+function nextRecord(record, items, version) {
+  const rec = Object.assign({}, record.all || {});
+  items.forEach((i) => { if (!i.changed) rec[i.key] = i.want; });
+  if (!record.ignored && record.version === version && JSON.stringify(rec) === JSON.stringify(record.all)) return null;
+  return rec;
+}
 /* Every file install replaces, the app included, is first copied to a folder of its own under
    .flightdeck/updates/<version>/replaced/, made new for this run (never one an earlier run used), and the copy is read back.
    No checked copy, no replacement. */
@@ -330,8 +420,8 @@ function report(items, check, out) {
 }
 /* What install says after the file list: an ignored record, and files left alone because they changed while it ran. */
 function notes(t, out) {
-  if (t.record && t.record.ignored) out.push('Ignored ' + t.recordAt.join('/') + ': it isn\'t a record this version of Flightdeck wrote, so it wasn\'t trusted. A new one was written.');
-  if (t.changed && t.changed.length) out.push('Left as they are, because they changed while install was running: ' + t.changed.join(', ') + '. Run the command again.');
+  if (t.record && t.record.ignored) out.push('Ignored ' + t.recordAt.join('/') + ': it isn\'t a record this version of Flightdeck wrote, so it wasn\'t trusted.' + (t.recordWritten ? ' A new one was written.' : ''));
+  if (t.changed && t.changed.length) out.push('Left as they are, because they changed while updating: ' + t.changed.join(', ') + '. Those weren\'t replaced; check them again.');
 }
 function exitFor(statuses) {
   if (statuses.includes('merge')) return EXIT.merge;
@@ -1724,7 +1814,7 @@ function main(argv) {
       return { out: out.join('\n'), code };
     }
     /* Everything is checked before anything is written, so a refusal (a link, say) leaves the folder as it was. */
-    if (!appOnly) contract(t, files, { html, force, check: true });
+    if (!appOnly) contract(t, files, { html, force, check: true, room: true, more: [[NAME, appS], [OLD, oldS]].filter(([file, st]) => st.status !== 'current' && (file !== OLD || keepOld)).map(([file]) => [file]) });
     t.changed = [];
     [[NAME, appS], [OLD, oldS]].forEach(([file, st]) => {
       if (st.status === 'current' || (file === OLD && !keepOld)) return;
@@ -1808,4 +1898,4 @@ if (require.main === module) {
     if (r && typeof r.then === 'function') r.then(done, failed); else done(r);
   } catch (e) { failed(e); }
 }
-module.exports = { _hooks, main, installedNewer, versionOf, cmp, skillFiles, norm, region, managed, merged, hash, baselineKey, examples, readAdapter, loadEngine, readData, requestData, writeRequest, reportData, writeOnce, EXIT };
+module.exports = { _hooks, main, installedNewer, parseRecord, decide, mergeReadmeText, nextRecord, baselinesOf: baselines, versionOf, cmp, skillFiles, norm, region, managed, merged, hash, baselineKey, examples, readAdapter, loadEngine, readData, requestData, writeRequest, reportData, writeOnce, EXIT };