create-obsidian-arrow 0.1.0 → 0.1.8

package/template/scripts/component-hash.mjs

@@ -0,0 +1,123 @@
+#!/usr/bin/env node
+/**
+ * component-hash — sandbox -> plugin porting parity check.
+ *
+ * The sandbox is the source of truth for a component; the plugin gets a
+ * near-verbatim copy. This hashes the canonical *portable body* (imports +
+ * whitespace normalized out; see lib/canonical-source.mjs) so the hook flags
+ * only meaningful drift, not benign import/mount deltas.
+ *
+ * Usage:
+ *   node scripts/component-hash.mjs <file>
+ *       print the canonical hash of one component.
+ *
+ *   node scripts/component-hash.mjs --verify <sandbox-file> <plugin-file>
+ *       compare two files; exit 0 (PARITY OK) or 1 (DRIFT).
+ *
+ *   node scripts/component-hash.mjs --check <manifest.json> [--update]
+ *       check every entry; exit 1 on any drift. --update re-records the baseline
+ *       from the source of truth (sandbox if present, else the plugin copy).
+ *
+ * Manifest shape (paths resolved relative to the manifest file):
+ *   { "entries": [
+ *       { "plugin": "src/chat/arrow/Foo.ts",
+ *         "sandbox": "../obsidian-arrow-sandbox/src/components/Foo.ts",  // optional
+ *         "hash": "<sha256>" }                                          // optional fallback
+ *   ] }
+ *
+ * Pure Node, no dependencies. Runs at the dev/commit boundary (husky/CI).
+ */
+import fs from "node:fs";
+import path from "node:path";
+import { hashSource } from "./lib/canonical-source.mjs";
+
+function fail(message) {
+	console.error(`component-hash: ${message}`);
+	process.exit(2);
+}
+
+function hashFile(file) {
+	try {
+		return hashSource(fs.readFileSync(file, "utf8"));
+	} catch (error) {
+		fail(`cannot read ${file} (${error.code ?? error.message})`);
+		return ""; // unreachable; fail() exits
+	}
+}
+
+const argv = process.argv.slice(2);
+
+if (argv.length === 0) {
+	fail("usage: component-hash <file> | --verify <a> <b> | --check <manifest> [--update]");
+}
+
+if (argv[0] === "--verify") {
+	const [, a, b] = argv;
+	if (!a || !b) {
+		fail("--verify needs <sandbox-file> <plugin-file>");
+	}
+	const ha = hashFile(a);
+	const hb = hashFile(b);
+	if (ha === hb) {
+		console.log(`PARITY OK  ${path.basename(b)}  ${ha.slice(0, 12)}`);
+		process.exit(0);
+	}
+	console.error(`DRIFT  ${b}\n  sandbox ${ha.slice(0, 12)}  !=  plugin ${hb.slice(0, 12)}`);
+	console.error("  -> edit the sandbox component and re-port (don't hand-edit the copy).");
+	process.exit(1);
+}
+
+if (argv[0] === "--check") {
+	const manifestPath = argv[1];
+	const update = argv.includes("--update");
+	if (!manifestPath) {
+		fail("--check needs <manifest.json>");
+	}
+	const manifestDir = path.dirname(path.resolve(manifestPath));
+	const manifest = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
+	const entries = manifest.entries ?? [];
+	const resolve = (p) => path.resolve(manifestDir, p);
+
+	const drifts = [];
+	for (const entry of entries) {
+		const pluginHash = hashFile(resolve(entry.plugin));
+		const hasSandbox = entry.sandbox && fs.existsSync(resolve(entry.sandbox));
+		const sourceHash = hasSandbox ? hashFile(resolve(entry.sandbox)) : undefined;
+		const expected = sourceHash ?? entry.hash;
+
+		if (update) {
+			entry.hash = sourceHash ?? pluginHash;
+			continue;
+		}
+		if (expected === undefined) {
+			drifts.push(`${entry.plugin}: no reference (add a sandbox path or a recorded hash)`);
+		} else if (pluginHash !== expected) {
+			const ref = hasSandbox ? "sandbox" : "recorded";
+			drifts.push(
+				`${entry.plugin}: DRIFT (plugin ${pluginHash.slice(0, 12)} != ${ref} ${expected.slice(0, 12)})`
+			);
+		}
+	}
+
+	if (update) {
+		fs.writeFileSync(manifestPath, `${JSON.stringify(manifest, null, "\t")}\n`);
+		console.log(
+			`Re-recorded ${entries.length} baseline hash(es) in ${path.basename(manifestPath)}.`
+		);
+		process.exit(0);
+	}
+
+	if (drifts.length > 0) {
+		console.error(`Port parity: ${drifts.length} drift(s).`);
+		for (const d of drifts) {
+			console.error(`  ${d}`);
+		}
+		console.error("  -> edit the sandbox component and re-port, or --update to re-bless.");
+		process.exit(1);
+	}
+	console.log(`Port parity OK (${entries.length} component(s)).`);
+	process.exit(0);
+}
+
+// Default: print the hash of one file.
+console.log(hashFile(argv[0]));

package/template/scripts/install-skills.mjs

@@ -1,47 +1,155 @@
 #!/usr/bin/env node
 /**
- * Install the skills bundled in this repo (skills/*) into the developer's agent
- * via the `skills` CLI (https://github.com/vercel-labs/skills). This repo acts
- * as a local marketplace: `skills add .` discovers skills/<name>/SKILL.md and
- * lets the user pick which to install through the interactive TUI.
+ * Install / update the skills bundled in this repo (skills/*) via the `skills`
+ * CLI (https://github.com/vercel-labs/skills). This repo is a local skills
+ * marketplace: the source `.` resolves to skills/<name>/SKILL.md.
  *
- * Wired as `postinstall`, so it runs after `pnpm install`. To keep that safe it
- * only launches the interactive TUI when attached to a real terminal — in CI or
- * any non-interactive context it prints a hint and exits 0 instead of hanging.
- * Run it on demand with `pnpm skills:install` (which passes --force).
+ * Modes (auto-detected):
+ *   - Interactive terminal       → opens the picker TUI (choose what to install).
+ *   - Non-interactive / CI / -y  → installs ALL bundled skills, no prompts.
+ *   - --update                   → updates an already-installed setup in place.
  *
- *   SKIP_SKILLS_INSTALL=1   opt out entirely
- *   pnpm skills:install      force the interactive install
+ * Wiring:
+ *   - `postinstall` (auto on `pnpm install`): only acts in an interactive
+ *     terminal; in CI/non-TTY it prints how to install and exits 0 (never hangs).
+ *   - `pnpm skills:install`        → picker (TUI) on a terminal, else installs all.
+ *   - `pnpm skills:install --yes`  → always non-interactive: install all skills.
+ *   - `pnpm skills:update`         → update installed skills to the latest.
+ *
+ * Flags / env:
+ *   --update / -u           update installed skills (runs `skills update -y`)
+ *   --yes / -y              non-interactive install of all bundled skills
+ *   --agent <name> | --agent=<name>   install for one agent (e.g. claude-code)
+ *                          instead of all detected agents
+ *   --project-dir=<path>    install into another project root (e.g. the outer repo
+ *                          a scaffold is nested in); project-scoped there
+ *   --global / -g           install at user level (~/.<agent>/…), available
+ *                          everywhere; some agents don't support global
+ *   SKILLS_AGENT / SKILLS_PROJECT_DIR / SKILLS_GLOBAL   env forms (for the auto
+ *                          `postinstall` step, which can't take CLI args, and CI)
+ *   --dry-run / SKILLS_DRY_RUN=1   print the command instead of running it
+ *   SKIP_SKILLS_INSTALL=1   opt out of the postinstall auto-step
  */
 import { spawnSync } from "node:child_process";
+import fs from "node:fs";
+import path from "node:path";
 import process from "node:process";
 
-const forced = process.argv.includes("--force");
+const BUNDLED =
+	"obsidian-arrow-sandbox, arrow-js-obsidian-templates, arrow-js-obsidian-patterns, arrow-js-obsidian-porting";
+
+const has = (flag) => process.argv.includes(flag);
+// Accepts both `--flag value` and `--flag=value`.
+const flagValue = (flag) => {
+	const eq = process.argv.find((a) => a.startsWith(`${flag}=`));
+	if (eq) {
+		return eq.slice(flag.length + 1);
+	}
+	const i = process.argv.indexOf(flag);
+	return i >= 0 ? process.argv[i + 1] : undefined;
+};
+
+/** Nearest ancestor *above* `dir` that is a git repo, or null. Used to warn
+ * when this project is nested inside another repo (skills install per-project,
+ * relative to cwd — not the outer root). */
+function outerRepoAbove(dir) {
+	let current = path.dirname(path.resolve(dir));
+	const fsRoot = path.parse(current).root;
+	while (current && current !== fsRoot) {
+		if (fs.existsSync(path.join(current, ".git"))) {
+			return current;
+		}
+		current = path.dirname(current);
+	}
+	return null;
+}
+
+const forced = has("--force"); // set by `pnpm skills:install`
+const update = has("--update") || has("-u");
+const yes = has("--yes") || has("-y");
+const global = has("--global") || has("-g") || process.env.SKILLS_GLOBAL === "1";
+const agent = flagValue("--agent") || process.env.SKILLS_AGENT;
+const projectDir = flagValue("--project-dir") || process.env.SKILLS_PROJECT_DIR;
+const dryRun = has("--dry-run") || process.env.SKILLS_DRY_RUN === "1";
 const isCI = Boolean(process.env.CI);
 const interactive = Boolean(process.stdout.isTTY && process.stdin.isTTY);
 const optedOut = process.env.SKIP_SKILLS_INSTALL === "1";
 
-if (!forced && (isCI || optedOut || !interactive)) {
-	console.log(
-		"[skills] Skipping interactive skill install (non-interactive/CI/opt-out).\n         Run `pnpm skills:install` to install the bundled skills via the npx skills TUI."
-	);
-	process.exit(0);
+// `skills add` installs project-scope relative to cwd. To install into another
+// root (e.g. the outer repo a scaffold is nested in), run the CLI there while
+// sourcing the skills from THIS folder by absolute path.
+const skillsSource = projectDir ? path.resolve(".") : ".";
+const targetCwd = projectDir ? path.resolve(projectDir) : process.cwd();
+
+if (projectDir && !fs.existsSync(targetCwd)) {
+	console.error(`[skills] --project-dir not found: ${targetCwd}`);
+	process.exit(1);
 }
 
-console.log("[skills] Launching `npx skills add .` — pick the skills to install in the TUI…\n");
+function run(args, cwd = process.cwd()) {
+	const where = cwd === process.cwd() ? "" : ` (in ${cwd})`;
+	const pretty = ["npx", "skills", ...args].join(" ");
+	if (dryRun) {
+		console.log(`[skills] (dry-run) would run: ${pretty}${where}`);
+		process.exit(0);
+	}
+	console.log(`[skills] ${pretty}${where}`);
+	const result = spawnSync("npx", ["--yes", "skills", ...args], {
+		cwd,
+		stdio: "inherit",
+		shell: process.platform === "win32",
+	});
+	if (result.error) {
+		console.error(
+			`[skills] could not run npx skills (${result.error.message}). Install manually: npx skills add . --all --yes`
+		);
+		process.exit(0); // never fail an install over an optional convenience step
+	}
+	process.exit(result.status ?? 0);
+}
 
-// `--yes` only auto-confirms npx fetching the `skills` package; the skills CLI's
-// own selection TUI still shows (we deliberately do not pass skills' `-y`).
-const result = spawnSync("npx", ["--yes", "skills", "add", "."], {
-	stdio: "inherit",
-	shell: process.platform === "win32",
-});
+if (optedOut && !update && !forced) {
+	console.log("[skills] SKIP_SKILLS_INSTALL=1 — skipping skill install.");
+	process.exit(0);
+}
+
+// Update an existing setup to the latest (works in any context).
+if (update) {
+	console.log("[skills] Updating installed skills to the latest…");
+	run(["update", "-y"]);
+}
 
-if (result.error) {
-	console.error(
-		`[skills] Could not run npx skills (${result.error.message}).\n         Install manually: npx skills add .`
+// Conservative postinstall: only auto-run when a human is at the terminal so
+// `pnpm install` in CI never hangs or installs things unprompted.
+if (!forced && (isCI || !interactive)) {
+	console.log(
+		`[skills] Bundled skills: ${BUNDLED}\n         Install:  pnpm skills:install        (interactive picker on a terminal)\n                   pnpm skills:install --yes  (non-interactive — installs all)\n         Update:   pnpm skills:update`
 	);
-	process.exit(0); // never fail an install over an optional convenience step
+	process.exit(0);
+}
+
+// Non-interactive (CI/agent/no TTY, or --yes, or an agent/global target): install
+// ALL bundled skills with no prompts. Target one agent if asked, else all agents.
+if (!interactive || yes || agent || global || projectDir) {
+	console.log(`[skills] Installing all bundled skills non-interactively: ${BUNDLED}`);
+	const outer = global || projectDir ? null : outerRepoAbove(process.cwd());
+	if (outer) {
+		console.log(
+			`[skills] note: this folder is nested inside ${outer}. Skills install here, scoped to THIS project. To install at the outer repo instead, re-run with --project-dir=${outer} (or --global for user-level).`
+		);
+	}
+	// `--all` is shorthand for `-s * -a * -y`; to target one agent we spell out
+	// all-skills + that agent explicitly. `skillsSource` is absolute when
+	// --project-dir redirects cwd elsewhere.
+	const args = agent
+		? ["add", skillsSource, "-s", "*", "-a", agent, "-y"]
+		: ["add", skillsSource, "--all"];
+	if (global) {
+		args.push("--global");
+	}
+	run(args, targetCwd);
 }
 
-process.exit(result.status ?? 0);
+// Interactive terminal: let the user pick in the TUI.
+console.log(`[skills] Opening the skills picker. Bundled: ${BUNDLED}`);
+run(["add", "."]);

package/template/scripts/lib/canonical-source.mjs

@@ -0,0 +1,63 @@
+import { createHash } from "node:crypto";
+
+/**
+ * Canonical source hashing for sandbox -> plugin porting parity.
+ *
+ * The sandbox is the source of truth for a component; the plugin gets a
+ * near-verbatim copy. This hashes the component's *portable body* so the only
+ * thing that can change the hash is meaningful drift in the component — not the
+ * bits that legitimately differ on the way into a plugin.
+ *
+ * Canonical form (define it so both sides agree by construction):
+ *   1. Drop `import …;` statements (single- or multi-line). Imports are the main
+ *      legit delta: the sandbox imports stub data / no obsidian; the plugin
+ *      imports the obsidian API / real data wiring. The component body is what
+ *      must match.
+ *   2. Normalize line endings to \n, strip trailing whitespace per line, collapse
+ *      runs of blank lines to one, trim leading/trailing blank lines.
+ *   3. SHA-256 the result.
+ *
+ * Note: comments are intentionally kept (they're part of the component; the
+ * sandbox is the source of truth for them too). To keep parity clean, write
+ * portable components that take data via props/getters so the body is identical
+ * across sandbox and plugin and only the mount site differs.
+ */
+
+// Matches a whole `import …;` statement. `[^;]*` crosses newlines (the negated
+// class matches \n) so multi-line imports are captured up to the first semicolon.
+const IMPORT_STATEMENT = /^[ \t]*import\b[^;]*;[ \t]*$/gm;
+
+export function canonicalizeSource(text) {
+	const withoutImports = text.replace(IMPORT_STATEMENT, "");
+	const lines = withoutImports
+		.replace(/\r\n/g, "\n")
+		.split("\n")
+		.map((line) => line.replace(/[ \t]+$/, ""));
+
+	// Collapse runs of blank lines to a single blank line.
+	const collapsed = [];
+	let lastBlank = false;
+	for (const line of lines) {
+		const blank = line === "";
+		if (blank && lastBlank) {
+			continue;
+		}
+		collapsed.push(line);
+		lastBlank = blank;
+	}
+
+	// Trim leading/trailing blank lines.
+	while (collapsed.length > 0 && collapsed[0] === "") {
+		collapsed.shift();
+	}
+	while (collapsed.length > 0 && collapsed[collapsed.length - 1] === "") {
+		collapsed.pop();
+	}
+
+	return `${collapsed.join("\n")}\n`;
+}
+
+/** SHA-256 hex of the canonical source. */
+export function hashSource(text) {
+	return createHash("sha256").update(canonicalizeSource(text), "utf8").digest("hex");
+}

package/template/skills/arrow-js-obsidian-porting/SKILL.md

@@ -0,0 +1,92 @@
+---
+name: arrow-js-obsidian-porting
+description: Use when porting an Arrow component from the obsidian-arrow-sandbox into the real Obsidian plugin, or wiring a check that the plugin copy hasn't drifted from the sandbox source. Covers the content-addressed porting-parity tool (scripts/component-hash.mjs), the canonical-form rules, a port-parity manifest, and a husky/CI pre-commit hook. The sandbox is the source of truth; the plugin copy must match.
+---
+
+# Porting parity (sandbox → Obsidian plugin)
+
+The sandbox is the **source of truth** for a component; the plugin gets a
+near-verbatim copy. The risk is **drift** — someone hand-edits the plugin copy,
+or a port silently diverges. This makes components **content-addressed**: a
+canonical hash of the component's portable body must match on both sides, the
+same idea as "never hand-edit generated artifacts."
+
+## Write portable components
+
+So the body is byte-identical across sandbox and plugin and only the *mount site*
+differs:
+
+- Take data via **props / getters**, not hard-wired sources. The sandbox passes a
+  stub (`loadStatus()`); the plugin passes the real source (rpc) — but that wiring
+  lives at the **call site** (`main.ts` / `ItemView.onOpen()`), not in the
+  component body.
+- Use Obsidian classes + `var(--…)` tokens (already the rule). Keep
+  `obsidian`-API calls (`setIcon`, …) out of the component body where possible.
+
+## The tool: `scripts/component-hash.mjs`
+
+Dependency-free Node; canonicalizer in `scripts/lib/canonical-source.mjs`.
+
+```sh
+node scripts/component-hash.mjs <file>                       # print canonical hash
+node scripts/component-hash.mjs --verify <sandbox> <plugin>  # PARITY OK | DRIFT (exit 0/1)
+node scripts/component-hash.mjs --check port-parity.json     # check all; exit 1 on drift
+node scripts/component-hash.mjs --check port-parity.json --update   # re-bless baselines
+```
+
+**Canonical form** (define it so both sides agree by construction):
+- `import …;` statements are stripped (single- and multi-line) — imports are the
+  main legit delta (stub vs real data, the `obsidian` API).
+- Line endings → `\n`, trailing whitespace trimmed, blank-line runs collapsed,
+  leading/trailing blanks trimmed.
+- Comments are **kept** (they're part of the component; the sandbox owns them too).
+- SHA-256 of the result.
+
+So two copies that differ only in imports/formatting hash equal; any change to
+the body/template/comments shows as drift.
+
+## Wiring the check in the plugin
+
+The hook lives in the **plugin** repo (that's where ported copies drift). Copy the
+two dependency-free files in (`scripts/lib/canonical-source.mjs`,
+`scripts/component-hash.mjs`) and add a manifest:
+
+```jsonc
+// port-parity.json — paths resolve relative to THIS file
+{
+  "entries": [
+    {
+      "plugin": "src/chat/arrow/SettingsPanel.ts",
+      "sandbox": "../../arrow-ui/obsidian-arrow-sandbox/src/components/SettingsPanel.ts", // optional
+      "hash": "<sha256>"                                                                  // fallback when sandbox isn't checked out
+    }
+  ]
+}
+```
+
+- If the `sandbox` path exists (sibling checkout), the check compares **plugin
+  copy ↔ live sandbox source** — catches drift in both directions.
+- If it doesn't, it falls back to the recorded `hash` (catches hand-edits to the
+  plugin copy). Record/refresh it with `--update` after an intentional re-port.
+
+husky `pre-commit` (and CI):
+
+```sh
+node scripts/component-hash.mjs --check port-parity.json
+```
+
+On drift the commit fails: *edit the sandbox component and re-port* (don't
+hand-edit the copy); use `--update` only to intentionally re-bless.
+
+## What this does NOT do (by design)
+
+- **No runtime component-DOM hashing in Obsidian.** Source parity already
+  guarantees same code → same Arrow DOM, and the sandbox vs real Obsidian DOM
+  *intentionally* differ (icon stub vs `setIcon`, `MarkdownRenderer` nodes, live
+  theme), so a runtime markup hash would be tautological *and* full of false
+  mismatches. The real runtime risk is visual/CSS — verify that by **loading in
+  Obsidian and looking**, not with a hash.
+- **Planned (not built): a styling-freshness check.** A small helper that hashes
+  the *live Obsidian token set* and compares it to the sandbox's pulled `app.css`
+  snapshot, to flag "re-run `pnpm pull-css`." That's the one runtime hash worth
+  having — environment parity, not component parity. Add it when needed.

package/template/skills/arrow-js-obsidian-templates/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: arrow-js-obsidian-templates
-description: Use when writing @arrow-js/core (v1.0.6) html`` templates — the reactive-vs-static rule, full-value attribute binding (returning false removes the attribute), .property and @event binding, keyed lists, async component(fn, { fallback }) wrapped in boundary(), and the footguns: no literal HTML comments inside templates and no partial attribute values (both throw "Invalid HTML position" at render), plus @event handlers must type the param as Event not a narrowed subtype like MouseEvent (TS2345).
+description: Use when writing @arrow-js/core (v1.0.6) html template literals — reactive vs static (wrap reactive reads in arrow functions), full-value attribute binding where returning false removes the attribute, .property and @event binding, keyed lists, and async component(fn, { fallback }) via boundary(); plus the footguns — no literal HTML comments and no partial attribute values (both throw Invalid HTML position at render), and @event handlers must type the param as Event not a narrowed subtype like MouseEvent (TS2345).
 ---
 
 # Arrow.js Templates (v1.0.6)

package/template/skills/obsidian-arrow-maintenance/SKILL.md

@@ -0,0 +1,69 @@
+---
+name: obsidian-arrow-maintenance
+description: Use when updating or maintaining an EXISTING scaffolded obsidian-arrow project — refresh the managed tooling with create-obsidian-arrow update, update installed agent skills with pnpm skills:update, fix skills scoping when the project is nested inside another repo (--project-dir / --global), and re-pull Obsidian styling. Only tooling/skills/docs are refreshed; your src is preserved.
+---
+
+# Maintaining an existing Obsidian Arrow project
+
+How to bring an already-scaffolded project up to date. The scaffolder is
+create-only (it refuses a non-empty dir), so updates split into three tracks:
+**tooling files**, **agent skills**, and **styling**. None of these touch your
+`src/`.
+
+## 1. Refresh the tooling (scripts, skills files, docs, CI, config)
+
+```sh
+npx create-obsidian-arrow update            # in the project root (or: update <dir>)
+npx create-obsidian-arrow update --dry-run  # preview what would change first
+```
+
+Refreshes the **managed** files from the latest template — `scripts/`, `skills/`,
+`docs/`, `.github/`, `.husky/`, `biome.json`, `AGENTS.md`, `CLAUDE.md` — and
+**merges** `package.json` scripts + any missing deps. It **never** touches `src/`,
+`public/`, `index.html`, `vite.config.ts`, `tsconfig.json`, or `.gitignore`. After
+it runs: `pnpm install && pnpm check`.
+
+## 2. Update the installed agent skills
+
+```sh
+pnpm skills:update    # = npx skills update -y  (updates installed skills in place)
+```
+
+Or reinstall the latest straight from the published repo (works from anywhere,
+even if the project predates the skill scripts):
+
+```sh
+npx skills add kylebrodeur/obsidian-arrow-sandbox --all --yes
+```
+
+## 3. Nested inside another repo? Fix skills scoping
+
+The `skills` CLI installs project-scope **relative to cwd**. If this project sits
+inside a larger repo and your agent runs from the **outer** repo, skills installed
+here won't be found. Install them where the agent looks:
+
+```sh
+pnpm skills:install --yes --project-dir=<outer-repo>   # project-scoped at the outer root
+pnpm skills:install --yes --global                     # or user-level (all projects)
+```
+
+(`SKILLS_PROJECT_DIR` / `SKILLS_GLOBAL` env forms drive the auto `postinstall`
+step, which takes no CLI args.)
+
+## 4. Re-pull Obsidian styling after Obsidian updates
+
+`public/app.css` is a local snapshot (git-ignored). If Obsidian updated or the
+sandbox renders stale, refresh it:
+
+```sh
+pnpm pull-css
+```
+
+## 5. Re-check porting parity
+
+After updating, re-run the porting-parity check so plugin copies still match the
+sandbox source (see the **arrow-js-obsidian-porting** skill):
+
+```sh
+node scripts/component-hash.mjs --check port-parity.json
+```

package/template/skills/obsidian-arrow-sandbox/SKILL.md

@@ -34,6 +34,19 @@ elsewhere pass `--path <obsidian.asar|app.css>` or set `OBSIDIAN_ASAR=<path>`.
 `public/app.css` is **git-ignored** (Obsidian's proprietary CSS — not
 redistributed), so run `pnpm pull-css` once before `pnpm dev`.
 
+## Install the bundled skills
+
+```sh
+pnpm skills:install --yes   # non-interactive: install ALL bundled skills (agents/CI)
+pnpm skills:install         # interactive picker on a terminal
+pnpm skills:update          # update an already-installed setup
+```
+
+Scope flags: `--agent <name>`, `--project-dir=<path>` (install into another repo
+root — use this when the project is **nested** inside a larger repo), `--global`.
+To update an existing project's tooling later, see the **obsidian-arrow-maintenance**
+skill (`npx create-obsidian-arrow update`).
+
 ## Build a component
 
 Add `src/components/MyThing.ts` exporting an Arrow `component()`, then mount it

package/template/test/component-hash.test.mjs

@@ -0,0 +1,48 @@
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { canonicalizeSource, hashSource } from "../scripts/lib/canonical-source.mjs";
+
+/**
+ * Porting-parity canonicalization. The hash must ignore the bits that
+ * legitimately differ when a component is ported sandbox -> plugin (imports,
+ * whitespace) and flag everything else (the component body/template).
+ */
+
+const SANDBOX = `import { component, html } from "@arrow-js/core";
+import { loadStatus } from "../data/loadStatus";
+
+export const Card = component(() => html\`<div class="setting-item">\${() => loadStatus()}</div>\`);
+`;
+
+// Same body, plugin-flavored imports (obsidian + real data wiring).
+const PLUGIN = `import { component, html } from "@arrow-js/core";
+import { loadStatus } from "./rpc";
+import { setIcon } from "obsidian";
+
+export const Card = component(() => html\`<div class="setting-item">\${() => loadStatus()}</div>\`);
+`;
+
+test("imports are stripped — same body hashes equal across sandbox and plugin", () => {
+	assert.equal(hashSource(SANDBOX), hashSource(PLUGIN));
+});
+
+test("a body change is detected as drift", () => {
+	const drifted = PLUGIN.replace("setting-item", "setting-item is-hacked");
+	assert.notEqual(hashSource(SANDBOX), hashSource(drifted));
+});
+
+test("whitespace and blank-line differences are normalized away", () => {
+	const messy = `${SANDBOX.replace(/\n/g, "\n\n")}   \n`;
+	assert.equal(hashSource(SANDBOX), hashSource(messy));
+});
+
+test("multi-line imports are stripped", () => {
+	const multiline = `import {\n\tcomponent,\n\thtml,\n} from "@arrow-js/core";\n\nexport const X = 1;\n`;
+	const singleline = `import { component, html } from "@arrow-js/core";\n\nexport const X = 1;\n`;
+	assert.equal(hashSource(multiline), hashSource(singleline));
+	assert.match(canonicalizeSource(multiline), /^export const X = 1;\n$/);
+});
+
+test("hash is stable sha256 hex", () => {
+	assert.match(hashSource(SANDBOX), /^[0-9a-f]{64}$/);
+});

package/template/test/skills-frontmatter.test.mjs

@@ -0,0 +1,57 @@
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import path from "node:path";
+import { test } from "node:test";
+import { fileURLToPath } from "node:url";
+
+/**
+ * The `skills` CLI parses each SKILL.md's YAML frontmatter; if it can't, the
+ * skill is silently skipped (not installed). An unquoted plain-scalar
+ * `description:` that contains `: ` (read as a nested key) or an unescaped
+ * double-quote breaks the parse — that's a real footgun (it skipped
+ * arrow-js-obsidian-templates until caught). These guards keep descriptions
+ * installable.
+ */
+const root = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..");
+const skillsDir = path.join(root, "skills");
+
+function skillFiles() {
+	if (!fs.existsSync(skillsDir)) {
+		return [];
+	}
+	return fs
+		.readdirSync(skillsDir, { withFileTypes: true })
+		.filter((entry) => entry.isDirectory())
+		.map((entry) => path.join(skillsDir, entry.name, "SKILL.md"))
+		.filter((file) => fs.existsSync(file));
+}
+
+function frontmatter(text) {
+	const match = text.match(/^---\n([\s\S]*?)\n---/);
+	return match ? match[1] : "";
+}
+
+test("every skill has name + description frontmatter", () => {
+	for (const file of skillFiles()) {
+		const fm = frontmatter(fs.readFileSync(file, "utf8"));
+		assert.match(fm, /^name:\s+\S/m, `${path.relative(root, file)}: missing name`);
+		assert.match(fm, /^description:\s+\S/m, `${path.relative(root, file)}: missing description`);
+	}
+});
+
+test("skill descriptions are YAML-safe so the skills CLI installs them", () => {
+	const offenders = [];
+	for (const file of skillFiles()) {
+		const fm = frontmatter(fs.readFileSync(file, "utf8"));
+		const desc = (fm.match(/^description:\s+(.*)$/m) ?? [])[1] ?? "";
+		const quoted = /^["']/.test(desc);
+		if (!quoted && (/: /.test(desc) || desc.includes('"'))) {
+			offenders.push(path.relative(root, file));
+		}
+	}
+	assert.deepEqual(
+		offenders,
+		[],
+		"unquoted description must not contain ': ' or a double-quote (breaks YAML; the skills CLI skips it)"
+	);
+});