@mmerterden/multi-agent-pipeline 20.17.2 → 20.19.0

package/pipeline/scripts/release-notes.mjs

@@ -0,0 +1,167 @@
+#!/usr/bin/env node
+/**
+ * release-notes.mjs  -  what an update brought, grouped so the user can be told.
+ *
+ * `/multi-agent:update` runs this from the release it just unpacked, against that
+ * release's CHANGELOG: every version after the one that was installed, up to the
+ * one now installed, with its bullets grouped by kind (added, changed, fixed,
+ * removed, ...). A jump across many releases is reported whole, not cut at a line
+ * count. The same is done for each stack plugin the plugin refresh moved, read
+ * from that plugin's CHANGELOG in the marketplace checkout.
+ *
+ * The output is data for the update step, which summarizes it for the user in
+ * their language. It never fails an update: a missing or unreadable CHANGELOG is
+ * empty notes and exit 0.
+ *
+ * Usage:
+ *   release-notes.mjs --changelog <CHANGELOG.md> --from <old> --to <new>
+ *                     [--plugins-dir <marketplace>/plugins --plugin-refresh-log <file>]
+ *
+ * stdout: JSON `{ pipeline: Notes, plugins: [{ name, from, to, notes: Notes }] }`
+ * where Notes is `{ from, to, versions: [{ version, date, sections }], counts }`.
+ */
+
+import { existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+import { pathToFileURL } from "node:url";
+
+import { runMain } from "../lib/fatal.mjs";
+
+const RELEASE_RE = /^## \[(\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?)\](?:\s*-\s*(.+))?\s*$/;
+const SECTION_RE = /^### (.+?)\s*$/;
+
+function cmp(a, b) {
+  const [pa, qa = ""] = a.split("-");
+  const [pb, qb = ""] = b.split("-");
+  const x = pa.split(".").map(Number);
+  const y = pb.split(".").map(Number);
+  for (let i = 0; i < 3; i++) if (x[i] !== y[i]) return x[i] - y[i];
+  if (qa === qb) return 0;
+  if (!qa) return 1;
+  if (!qb) return -1;
+  return qa < qb ? -1 : 1;
+}
+
+/**
+ * Released versions in a Keep a Changelog file, newest first as written.
+ * `## [Unreleased]` is skipped; a bullet's wrapped lines are joined.
+ *
+ * @param {string} text
+ * @returns {{version: string, date: string|null, sections: Record<string, string[]>}[]}
+ */
+export function parseChangelog(text) {
+  const releases = [];
+  let release = null;
+  let section = null;
+  let bullet = null;
+  const flush = () => {
+    if (bullet !== null && release && section) release.sections[section].push(bullet.trim());
+    bullet = null;
+  };
+  for (const line of String(text || "").split("\n")) {
+    if (line.startsWith("## ")) {
+      flush();
+      const m = RELEASE_RE.exec(line);
+      release = m ? { version: m[1], date: m[2]?.trim() || null, sections: {} } : null;
+      if (release) releases.push(release);
+      section = null;
+      continue;
+    }
+    if (!release) continue;
+    const s = SECTION_RE.exec(line);
+    if (s) {
+      flush();
+      section = s[1].toLowerCase();
+      release.sections[section] ||= [];
+      continue;
+    }
+    if (!section) continue;
+    if (/^[-*] /.test(line)) {
+      flush();
+      bullet = line.slice(2);
+    } else if (bullet !== null && /^\s+\S/.test(line)) {
+      bullet += " " + line.trim();
+    } else if (!line.trim()) {
+      flush();
+    }
+  }
+  flush();
+  return releases;
+}
+
+/**
+ * Releases after `from` up to and including `to`, newest first, with a count of
+ * bullets per section across all of them.
+ *
+ * @param {string} text
+ * @param {{from: string, to: string}} range
+ */
+export function releaseNotes(text, { from, to }) {
+  const versions = parseChangelog(text)
+    .filter((r) => cmp(r.version, from) > 0 && cmp(r.version, to) <= 0)
+    .sort((a, b) => cmp(b.version, a.version));
+  const counts = {};
+  for (const v of versions) {
+    for (const [k, items] of Object.entries(v.sections))
+      counts[k] = (counts[k] || 0) + items.length;
+  }
+  return { from, to, versions, counts };
+}
+
+/**
+ * Plugins the refresh moved, from the CLI's own lines
+ * (`Plugin "<name>" updated from <a> to <b> for scope ...`). A plugin updated in
+ * several scopes is reported once, from the oldest version any of them had.
+ *
+ * @param {string} output
+ * @returns {{name: string, from: string, to: string}[]}
+ */
+export function pluginUpdates(output) {
+  const byName = new Map();
+  const re = /Plugin "([^"]+)" updated from (\S+) to (\S+)/g;
+  for (const m of String(output || "").matchAll(re)) {
+    const [, name, from, to] = m;
+    const seen = byName.get(name);
+    if (!seen) byName.set(name, { name, from, to });
+    else {
+      if (cmp(from, seen.from) < 0) seen.from = from;
+      if (cmp(to, seen.to) > 0) seen.to = to;
+    }
+  }
+  return [...byName.values()];
+}
+
+function read(path) {
+  try {
+    return existsSync(path) ? readFileSync(path, "utf8") : "";
+  } catch {
+    return "";
+  }
+}
+
+function arg(argv, name) {
+  const i = argv.indexOf(name);
+  return i >= 0 ? argv[i + 1] : undefined;
+}
+
+export function main(argv = process.argv.slice(2)) {
+  const from = arg(argv, "--from") || "0.0.0";
+  const to = arg(argv, "--to") || from;
+  const pipeline = releaseNotes(read(arg(argv, "--changelog") || ""), { from, to });
+
+  const pluginsDir = arg(argv, "--plugins-dir");
+  const refreshLog = arg(argv, "--plugin-refresh-log");
+  const plugins = [];
+  if (pluginsDir && refreshLog) {
+    for (const u of pluginUpdates(read(refreshLog))) {
+      const notes = releaseNotes(read(join(pluginsDir, u.name, "CHANGELOG.md")), u);
+      plugins.push({ ...u, notes });
+    }
+  }
+  process.stdout.write(JSON.stringify({ pipeline, plugins }, null, 2) + "\n");
+  return 0;
+}
+
+if (import.meta.url === pathToFileURL(process.argv[1] || "").href) {
+  runMain("release-notes", () => main());
+}

package/pipeline/scripts/uninstall.mjs

@@ -53,6 +53,12 @@ import { createHash } from "crypto";
 import { join, dirname } from "path";
 import { homedir } from "os";
 import { pathToFileURL } from "url";
+import {
+  CLAUDE_MARKETPLACE_DIR,
+  XCODE_MARKETPLACE,
+  XCODE_PLUGIN_ID,
+  XCODE_SKILLS_MANIFEST,
+} from "../lib/xcode-plugin.mjs";
 import { createInterface } from "readline";
 
 const HOME = process.env.HOME;
@@ -425,6 +431,48 @@ function deregisterClaudeMcpServer() {
   deregisterMcpServer("claude", existsSync(localBin) ? localBin : undefined, ["--scope", "user"]);
 }
 
+/**
+ * Apple's Xcode plugin is linked into Claude Code from a local directory
+ * marketplace the installer keeps at the installed Xcode build. Without the
+ * pipeline nothing refreshes that copy, so it goes with the pipeline; Xcode
+ * itself still offers the same skills.
+ */
+function unlinkClaudeXcodePlugin(home) {
+  const mpDir = join(home, CLAUDE_MARKETPLACE_DIR);
+  if (!existsSync(mpDir)) return;
+  const localBin = join(homedir(), ".local", "bin", "claude");
+  const cli = existsSync(localBin) ? localBin : "claude";
+  const steps = [
+    ["plugin", "uninstall", XCODE_PLUGIN_ID, "--scope", "user"],
+    ["plugin", "marketplace", "remove", XCODE_MARKETPLACE],
+  ];
+  if (dryRun) {
+    for (const args of steps) report("would run", `claude ${args.join(" ")}`);
+    report("would remove", mpDir);
+    return;
+  }
+  const failed = [];
+  for (const args of steps) {
+    try {
+      execFileSync(cli, args, { stdio: "pipe", timeout: 20_000 });
+    } catch (e) {
+      const out = `${e?.stderr ?? ""}${e?.stdout ?? ""}`;
+      if (!/not (found|installed)|no such|does not exist|unknown/i.test(out)) failed.push(args);
+    }
+  }
+  // Claude Code's settings still point at the directory until the CLI drops
+  // them; deleting it first would leave a marketplace that fails to load at
+  // every start.
+  if (failed.length) {
+    console.log("    kept: Apple's Xcode skills link - claude could not remove it; run:");
+    for (const args of failed) console.log(`      claude ${args.join(" ")}`);
+    console.log(`      then delete ${mpDir}`);
+    return;
+  }
+  rmIfExists(mpDir);
+  console.log("    removed: Apple's Xcode skills link (xcode-apple marketplace)");
+}
+
 /**
  * Copilot registers the same server, so leaving it behind would keep a stale entry
  * pointing at a package the user just removed.
@@ -589,7 +637,14 @@ function rmMatchingFiles(parent, predicate) {
 const PLUGIN_SKILLS_MANIFEST = ".plugin-skills-manifest.json";
 
 function rmPluginDeliveredSkills(skillsDir) {
-  const manifestPath = join(skillsDir, PLUGIN_SKILLS_MANIFEST);
+  return (
+    rmManifestListedSkills(skillsDir, PLUGIN_SKILLS_MANIFEST) +
+    rmManifestListedSkills(skillsDir, XCODE_SKILLS_MANIFEST)
+  );
+}
+
+function rmManifestListedSkills(skillsDir, manifestName) {
+  const manifestPath = join(skillsDir, manifestName);
   if (!existsSync(manifestPath)) return 0;
   let names;
   try {
@@ -855,6 +910,7 @@ export async function main() {
     console.log("  [Claude Code] Removing...");
     deregisterClaudeMcpServer();
     const CLAUDE = join(HOME, ".claude");
+    unlinkClaudeXcodePlugin(HOME);
     rmCommandsPreservingLocalOnly(join(CLAUDE, "commands", "multi-agent"));
     rmIfExists(join(CLAUDE, "scripts"));
     // Pipeline-managed trees the installer lays down alongside scripts/.

package/pipeline/skills/.skill-manifest.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": "1.0.0",
-  "generatedAt": "2026-10-08T12:53:56Z",
+  "generatedAt": "2026-10-09T05:08:19Z",
   "skillCount": 228,
   "entries": [
     {
@@ -569,7 +569,7 @@
     },
     {
       "path": "shared/external/ios-iphone-duo/SKILL.md",
-      "sha256": "2da65582a8d86957ded84345e60095584a26942f9ded8f61e901f3d04fb66c37"
+      "sha256": "934df39d42cac41473d4c7605891c4b15aca2d185099c21b36a3c1abbbea8359"
     },
     {
       "path": "shared/external/ios-localization/SKILL.md",

package/pipeline/skills/shared/external/ios-iphone-duo/SKILL.md

@@ -19,6 +19,13 @@ APIs where the fold or the vertical bars need them.
 in the iOS 27.1 SDK and may change before release. Check availability before
 shipping code that depends on it, and gate it with `if #available`.
 
+**Apple's skill first.** Xcode 27.1 and later ship Apple's `app-resizability`
+skill, which the pipeline links from this Mac's Xcode (Claude Code:
+`xcode-integration:app-resizability`). When it is available, run it first for
+the code scan and the API modernization; use this skill for what it adds: the
+fold and poses, reserved regions, arrangements, vertical bars and the design
+review. Without Xcode 27.1, this skill covers the whole job.
+
 ## One rule behind every fix
 
 Do not ask "is this iPhone Duo?". Ask "how much space do I have, and what is
@@ -54,6 +61,8 @@ Each hit is a candidate defect with a known replacement:
 | Symmetric insets (`insets.left * 2`) | Side-mounted bars make insets asymmetric | Each edge's inset on its own |
 | Hand-built `UIToolbar`, `UINavigationBar`, `UITabBar` | Only container-provided bars move to the side | Toolbar items on navigation containers |
 | Full-bleed fill media | Wider displays crop the subject | Choose fill or fit from size, set a focal point |
+| App-delegate-only life cycle | From iOS 27 an app built with the latest SDK must adopt scenes or it fails to launch | Scene manifest and scene delegate, `UIWindow(windowScene:)` |
+| `topLayoutGuide`, keyboard frames, stored insets | Deprecated or computed for a full screen | `safeAreaLayoutGuide`, `keyboardLayoutGuide`, insets read at layout |
 
 ## Step 3: let containers adapt
 

package/pipeline/skills/shared/external/ios-iphone-duo/evals/evals.json

@@ -45,6 +45,21 @@
         "Uses ToolbarOverflowMenu or the system overflow instead of a custom menu",
         "Treats disabling vertical bars as a rare, stable per-screen choice"
       ]
+    },
+    {
+      "id": 3,
+      "name": "scene-lifecycle-and-legacy-guides",
+      "prompt": "Our UIKit app still does all its foreground and background work in AppDelegate (applicationDidBecomeActive and friends), creates its window with UIWindow(frame: UIScreen.main.bounds), pins the editor to topLayoutGuide, and moves the input bar up by reading keyboardFrameEndUserInfoKey. We are moving to the iOS 27 SDK and want iPhone Duo support. What breaks and what do we change?",
+      "expected_output": "States that from iOS 27 an app built with the latest SDK must adopt the scene-based life cycle or it fails to launch, and that foreground and background arrive per scene. Moves window creation to a scene delegate with UIWindow(windowScene:), the four lifecycle callbacks to their scene equivalents, keeps one-time work in didFinishLaunching, and treats multiple windows as optional. Replaces topLayoutGuide with safeAreaLayoutGuide and keyboard-frame math with keyboardLayoutGuide, and reads insets at layout time. Mentions running Apple's app-resizability skill first when Xcode provides it.",
+      "files": [],
+      "expectations": [
+        "Says an app built with the iOS 27 SDK without the scene life cycle fails to launch",
+        "Moves window creation to a scene delegate using UIWindow(windowScene:)",
+        "Maps the app delegate lifecycle callbacks to scene callbacks and keeps one-time work at launch",
+        "Replaces topLayoutGuide with safeAreaLayoutGuide",
+        "Replaces keyboard frame math with keyboardLayoutGuide",
+        "Points to Apple's app-resizability skill as the first route when Xcode 27.1+ provides it"
+      ]
     }
   ]
 }

package/pipeline/skills/shared/external/ios-iphone-duo/references/fixed-assumptions.md

@@ -15,7 +15,8 @@ the iPhone Duo simulator. Exclude vendored and generated code from the count.
 7. Insets
 8. Bars
 9. Media
-10. Recording the result
+10. Life cycle, legacy guides and the keyboard
+11. Recording the result
 
 ## 1. The search
 
@@ -33,9 +34,18 @@ rg -n --type swift \
   -e '(width|height)\s*(==|>=|<=|>|<)\s*(320|375|390|393|402|414|428|430|440)\b' \
   -e 'UIToolbar\(|UINavigationBar\(|UITabBar\(' \
   -e 'scaleAspectFill|contentMode:\s*\.fill' \
+  -e 'topLayoutGuide|bottomLayoutGuide' \
+  -e 'keyboardWillShowNotification|keyboardFrameEndUserInfoKey' \
+  -e 'applicationDidBecomeActive|applicationWillResignActive|applicationDidEnterBackground|applicationWillEnterForeground' \
   .
 ```
 
+The scene manifest lives in Info.plist, which the Swift search does not read:
+
+```bash
+rg -l 'UIApplicationSceneManifest' -g '*.plist' . || echo "no scene manifest"
+```
+
 Add Objective-C (`--type objc`) and storyboards (`-g '*.storyboard'` for
 fixed-size constraints) when the project has them.
 
@@ -138,7 +148,54 @@ Replace: choose fill or fit from the current size or aspect ratio, crop around
 a focal point, and extend backgrounds under bars with
 `backgroundExtensionEffect()` rather than stretching the image.
 
-## 10. Recording the result
+## 10. Life cycle, legacy guides and the keyboard
+
+**Scene life cycle.** Hits: the four `application...` foreground and background
+callbacks doing real work in the app delegate, no `UIApplicationSceneManifest`
+in Info.plist, no `UIWindowSceneDelegate`.
+
+Why: from iOS 27, an app built with the latest SDK that has not adopted the
+scene-based life cycle fails to launch. The system also delivers foreground and
+background per scene, so app-level callbacks miss what happens to one window
+of several.
+
+Replace: a scene manifest and a scene delegate that creates the window with
+`UIWindow(windowScene:)`, and the per-window work moved to the scene callbacks
+(`sceneDidBecomeActive`, `sceneWillResignActive`, `sceneDidEnterBackground`,
+`sceneWillEnterForeground`). Work that runs once per process stays in
+`application(_:didFinishLaunchingWithOptions:)`. Supporting several windows is
+a separate, optional step.
+
+**Legacy layout guides.** Hits: `topLayoutGuide`, `bottomLayoutGuide`, and
+constants standing in for the status bar or navigation bar height.
+
+Why: those guides are deprecated and know nothing about side insets; a fixed
+bar height is wrong as soon as a bar moves or the status bar changes.
+
+Replace: `safeAreaLayoutGuide` and layout margins.
+
+**The keyboard.** Hits: frames computed from `keyboardFrameEndUserInfoKey` in
+a `keyboardWillShowNotification` handler.
+
+Why: a screen-space keyboard frame needs converting into the window by hand,
+and frame arithmetic misses the cases a resized window creates.
+
+Replace: constraints to `view.keyboardLayoutGuide` (iOS 15); SwiftUI already
+avoids the keyboard through the safe area.
+
+**Stored insets.** Hits: a safe-area inset copied into a property at load
+time.
+
+Why: insets change with pose, bar placement and Split View position.
+
+Replace: read them where layout happens, and react in
+`viewSafeAreaInsetsDidChange()` / `safeAreaInsetsDidChange()`.
+
+**Before replacing, classify.** An idiom or orientation check is a defect
+only when it stands in for available space. A check for a genuine capability of
+the device, or an orientation lock for a camera or game, stays.
+
+## 11. Recording the result
 
 For each hit, record `file:line`, the category, keep or fix, and for fixes the
 replacement. Separate "confirmed on the simulator" from "found in code only".