@ericrisco/rsc 0.1.16 → 0.1.17

package/README.md

@@ -273,7 +273,7 @@ duplicated. The wizard asks which ones; `--target a,b` does it non-interactively
 
 | Target | Skill destination (→ `.rsc/skills/<id>/`) | Always-on detector |
 | --- | --- | --- |
-| `claude` | `.claude/skills/rsc/<id>/` → symlink | SessionStart hook in `.claude/settings.json` |
+| `claude` | `.claude/skills/<id>/` → symlink (copy on Windows) | SessionStart hook in `.claude/settings.json` |
 | `codex` | `.codex/rsc/<id>/` → symlink | block in `AGENTS.md` |
 | `copilot` | `.github/rsc/<id>/` → symlink | block in `.github/copilot-instructions.md` |
 | `cursor` | `.cursor/rules/<id>.mdc` (converted) | always-apply rule |

package/manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.1.16",
+  "version": "0.1.17",
   "counts": {
     "skills": 231
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ericrisco/rsc",
-  "version": "0.1.16",
+  "version": "0.1.17",
   "description": "Eric Risco's agent-skills catalog as a granular, self-recommending CLI installer.",
   "type": "module",
   "bin": {

package/skills/author-skill/references/rsc-conventions.md

@@ -49,10 +49,14 @@ profiles: [core, full]           # optional: named-profile membership
 
 ## Invocation
 
-There are no bundles and no `/<bundle>:<id>` namespacing. A skill installs under
-the target's rsc namespace (e.g. `~/.claude/skills/rsc/<id>/`) and is invoked by
-its `name`. The `suggest` detector is always installed (the floor) and proposes
-installing any skill a task needs via `npx @ericrisco/rsc add <id>`.
+There are no bundles and no `/<bundle>:<id>` namespacing. Most targets install a
+skill under their own rsc folder (e.g. `.codex/rsc/<id>/`), reached from that
+assistant's instructions file. **Claude is the exception:** Claude Code only
+discovers project skills one level under `.claude/skills/`, so rsc skills install
+**flat** at `.claude/skills/<id>/SKILL.md` (a nested `.claude/skills/rsc/<id>/`
+is never discovered). A skill is invoked by its `name`. The `suggest` detector is
+always installed (the floor) and proposes installing any skill a task needs via
+`npx @ericrisco/rsc add <id>`.
 
 ## Wiring steps for a new skill
 

package/skills/harness/references/wiki-protocol.md

@@ -388,7 +388,8 @@ generalizes the SDD session-summary convention to *any* work.
 
 - **Hook — `PreCompact` / `SessionEnd`** (Claude Code): fires right before context
   is lost or the session ends. The hook only *reminds*; the agent writes the
-  worklog (Karpathy: the LLM writes). Wired by `targets/` → `.rsc/worklog-checkpoint.sh`.
+  worklog (Karpathy: the LLM writes). Wired by `targets/` → `.rsc/worklog-checkpoint.mjs`
+  (run via `node`, so it works on Windows too).
 - **Explicit milestone**: after a commit or a shipped feature, capture a worklog.
 - **Daily curation automation**: distills any pending worklog raw on a timer (see
   Continuous Improvement and `daily-curation-automation.md`).

package/targets/claude.js

@@ -1,27 +1,34 @@
-import { readFileSync, writeFileSync, existsSync, mkdirSync, copyFileSync, chmodSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync, copyFileSync, rmSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { linkOrCopy } from './index.js';
 
+const HERE = dirname(fileURLToPath(import.meta.url));
+
 export function writeSkill(id, fromDir, toPath) {
+  // Migrate away from the legacy nested layout (.claude/skills/rsc/<id>) that
+  // Claude Code never discovered — it only reads .claude/skills/<name>/SKILL.md.
+  // toPath is now .claude/skills/<id>; drop the stale rsc/ sibling if present.
+  const legacy = join(dirname(toPath), 'rsc');
+  if (existsSync(legacy)) rmSync(legacy, { recursive: true, force: true });
   return linkOrCopy(fromDir, toPath);
 }
 
-// SessionStart runs a project-local session-start.sh: it cats suggest's always-on
-// body (preserving prior behavior) and appends an onboarding banner when the
-// workspace has no harness profile yet. We materialize the script next to the
-// shared base and point the hook at it. Any prior rsc SessionStart entry (the old
-// `cat …/suggest/SKILL.md` form or a previous script form) is dropped before we
-// add the current one — idempotent, and it migrates legacy hooks in place. Other
-// (non-rsc) SessionStart hooks are preserved.
+// SessionStart runs a project-local session-start.mjs via `node`: it prints
+// suggest's always-on body, an onboarding banner when the workspace has no harness
+// profile yet, and an auto-ingest nudge when the inbox has un-ingested material. We
+// invoke with `node` (not bash) so the hook runs on Windows too. We materialize the
+// script next to the shared base and point the hook at it. Any prior rsc SessionStart
+// entry (the old `cat …/suggest/SKILL.md` form, or a previous `.sh`/`.mjs` script
+// form) is dropped before we add the current one — idempotent, migrating legacy and
+// bash-era hooks in place. Other (non-rsc) SessionStart hooks are preserved.
 export function wireHook(paths) {
-  const scriptDest = join(paths.projectRoot, '.rsc', 'session-start.sh');
+  const scriptDest = join(paths.projectRoot, '.rsc', 'session-start.mjs');
   mkdirSync(dirname(scriptDest), { recursive: true });
-  copyFileSync(join(dirname(fileURLToPath(import.meta.url)), 'session-start.sh'), scriptDest);
-  chmodSync(scriptDest, 0o755);
+  copyFileSync(join(HERE, 'session-start.mjs'), scriptDest);
 
   const suggestMd = `${paths.skillDir('suggest')}/SKILL.md`;
-  const cmd = `bash "${scriptDest}" "${suggestMd}" "${paths.projectRoot}"`;
+  const cmd = `node "${scriptDest}" "${suggestMd}" "${paths.projectRoot}"`;
 
   const file = paths.hookTarget;
   const settings = existsSync(file) ? JSON.parse(readFileSync(file, 'utf8')) : {};
@@ -29,23 +36,23 @@ export function wireHook(paths) {
   settings.hooks.SessionStart ||= [];
   settings.hooks.SessionStart = settings.hooks.SessionStart.filter((e) => {
     const s = JSON.stringify(e);
-    return !s.includes('skills/rsc/suggest') && !s.includes('.rsc/session-start.sh');
+    // drop legacy cat-form and any prior session-start script (.sh from the bash era, or .mjs)
+    return !s.includes('skills/rsc/suggest') && !s.includes('.rsc/session-start.');
   });
   settings.hooks.SessionStart.push({ hooks: [{ type: 'command', command: cmd }] });
 
   // Worklog checkpoint: PreCompact + SessionEnd run a project-local
-  // worklog-checkpoint.sh that reminds the agent to capture what we did this
-  // session into 02-DOCS/raw/worklog/ (the work-driven on-ramp). Silent when the
-  // workspace has no harness wiki. Registered idempotently on both events, with
-  // any prior rsc worklog-checkpoint entry dropped first.
-  const wlDest = join(paths.projectRoot, '.rsc', 'worklog-checkpoint.sh');
-  copyFileSync(join(dirname(fileURLToPath(import.meta.url)), 'worklog-checkpoint.sh'), wlDest);
-  chmodSync(wlDest, 0o755);
-  const wlCmd = `bash "${wlDest}" "${paths.projectRoot}"`;
+  // worklog-checkpoint.mjs via `node` that reminds the agent to capture what we did
+  // this session into 02-DOCS/raw/worklog/ (the work-driven on-ramp). Silent when
+  // the workspace has no harness wiki. Registered idempotently on both events, with
+  // any prior rsc worklog-checkpoint entry (.sh or .mjs) dropped first.
+  const wlDest = join(paths.projectRoot, '.rsc', 'worklog-checkpoint.mjs');
+  copyFileSync(join(HERE, 'worklog-checkpoint.mjs'), wlDest);
+  const wlCmd = `node "${wlDest}" "${paths.projectRoot}"`;
   for (const event of ['PreCompact', 'SessionEnd']) {
     settings.hooks[event] ||= [];
     settings.hooks[event] = settings.hooks[event].filter(
-      (e) => !JSON.stringify(e).includes('.rsc/worklog-checkpoint.sh'),
+      (e) => !JSON.stringify(e).includes('.rsc/worklog-checkpoint.'),
     );
     settings.hooks[event].push({ hooks: [{ type: 'command', command: wlCmd }] });
   }

package/targets/index.js

@@ -11,16 +11,22 @@ export function baseDir(id, cwd = process.cwd()) {
   return join(cwd, '.rsc', 'skills', id);
 }
 
-// Point an assistant's skill folder at the shared base via a relative symlink.
-// Falls back to a real copy when the filesystem rejects symlinks (e.g. Windows
-// without privileges). Idempotent: replaces any existing link/dir at toPath.
+// Point an assistant's skill folder at the shared base. On macOS/Linux a relative
+// symlink avoids duplication. On Windows we copy real files: relative `dir`
+// symlinks require Developer Mode/admin and are not reliably followed by skill
+// discovery, so correctness wins over de-duplication. Idempotent: replaces any
+// existing link/dir at toPath.
 export function linkOrCopy(fromDir, toPath) {
   mkdirSync(dirname(toPath), { recursive: true });
   try { lstatSync(toPath); rmSync(toPath, { recursive: true, force: true }); } catch { /* nothing there */ }
-  try {
-    symlinkSync(relative(dirname(toPath), fromDir), toPath, 'dir');
-  } catch {
+  if (process.platform === 'win32') {
     cpSync(fromDir, toPath, { recursive: true });
+  } else {
+    try {
+      symlinkSync(relative(dirname(toPath), fromDir), toPath, 'dir');
+    } catch {
+      cpSync(fromDir, toPath, { recursive: true });
+    }
   }
   return [toPath];
 }
@@ -31,7 +37,7 @@ export function linkOrCopy(fromDir, toPath) {
 // is a single converted file, not a linked directory.
 const SPEC = {
   // JSON-hook + linked skill dirs
-  claude: { root: '.claude/skills/rsc', hook: '.claude/settings.json', adapter: 'claude' },
+  claude: { root: '.claude/skills', hook: '.claude/settings.json', adapter: 'claude' },
   // Converted .mdc rules
   cursor: { root: '.cursor/rules', hook: '.cursor/rules/rsc-suggest.mdc', adapter: 'cursor', skillExt: '.mdc' },
   // AGENTS.md family — all read the same root AGENTS.md

package/targets/session-start.mjs

@@ -0,0 +1,49 @@
+#!/usr/bin/env node
+// rsc SessionStart payload (claude). Wired by targets/claude.js as `node ...` so
+// it runs on every platform including Windows (no bash dependency).
+//   argv[2] = absolute path to suggest's SKILL.md   argv[3] = absolute project root
+// Always emits suggest's always-on body; appends an onboarding banner when the
+// workspace has no harness profile yet, and an auto-ingest nudge when there is
+// un-ingested material waiting in the inbox.
+import { readFileSync, existsSync, readdirSync } from 'node:fs';
+import { join } from 'node:path';
+
+const suggestMd = process.argv[2];
+const root = process.argv[3] || process.cwd();
+
+try { process.stdout.write(readFileSync(suggestMd, 'utf8')); } catch { /* missing → emit nothing */ }
+
+const profile = join(root, '02-DOCS', 'wiki', 'harness', 'user-profile.md');
+const optout = join(root, '.rsc', '.no-harness');
+if (!existsSync(profile) && !existsSync(optout)) {
+  process.stdout.write(`
+===== rsc onboarding =====
+Fresh setup: 02-DOCS/wiki/harness/user-profile.md is missing.
+ACTION: invoke \`init\` now (first contact: technical level + accompaniment dial) before the task.
+If the user does not want a harness here: create .rsc/.no-harness
+==========================
+`);
+}
+
+// Auto-Ingest nudge: when a harness wiki exists and the inbox holds a real file
+// (anything other than README.md / dotfiles / the _processed archive), tell the
+// agent to run the Auto-Ingest Sweep. The hook only reminds; the agent does the
+// scan + ingest. Cheap signal here; the thorough workspace scan lives in the sweep.
+const inbox = join(root, '02-DOCS', 'inbox');
+if (existsSync(join(root, '02-DOCS', 'wiki')) && existsSync(inbox)) {
+  let pending = false;
+  try {
+    pending = readdirSync(inbox, { withFileTypes: true })
+      .some((e) => e.isFile() && e.name !== 'README.md' && !e.name.startsWith('.'));
+  } catch { /* unreadable → no nudge */ }
+  if (pending) {
+    process.stdout.write(`
+===== rsc auto-ingest =====
+Un-ingested material is waiting in 02-DOCS/inbox/.
+ACTION: run the Auto-Ingest Sweep now — ingest inbox/, then scan the workspace
+(minus .rscignore) for un-ingested documents, recording them in wiki/.ingested.json.
+Originals are copied, never moved; deleting an emptied folder needs explicit consent.
+===========================
+`);
+  }
+}

package/targets/worklog-checkpoint.mjs

@@ -0,0 +1,26 @@
+#!/usr/bin/env node
+// rsc Worklog checkpoint (claude). Wired by targets/claude.js onto PreCompact and
+// SessionEnd as `node ...` so it runs on every platform including Windows.
+//   argv[2] = absolute project root
+// Reminds the agent to run a Worklog Sweep (capture what we did this session into
+// 02-DOCS/raw/worklog/). The hook only reminds; the agent writes the worklog.
+// Silent when this workspace has no harness wiki yet.
+import { existsSync } from 'node:fs';
+import { join } from 'node:path';
+
+const root = process.argv[2] || process.cwd();
+
+// No harness wiki here → nothing to document into. Stay silent.
+if (!existsSync(join(root, '02-DOCS', 'wiki'))) process.exit(0);
+
+process.stdout.write(`
+===== rsc worklog checkpoint =====
+If this session did meaningful work (files changed, a decision made, a commit),
+run a WORKLOG SWEEP before context is lost:
+  1. Write 02-DOCS/raw/worklog/<YYYY-MM-DD>-<slug>.md using the harness
+     wiki-worklog-template.md (what we did · why · files touched · outcome · next).
+  2. Compile it into wiki/ (update existing articles first; wikilinks + Related);
+     append significant decisions to 02-DOCS/wiki/harness/decisions.md.
+Skip entirely if this was a pure read/answer turn with no changes.
+==================================
+`);

package/targets/session-start.sh

@@ -1,43 +0,0 @@
-#!/usr/bin/env bash
-# rsc SessionStart payload (claude). Wired by targets/claude.js.
-#   $1 = absolute path to suggest's SKILL.md   $2 = absolute project root
-# Always emits suggest's always-on body; appends an onboarding banner when the
-# workspace has no harness profile yet, and an auto-ingest nudge when there is
-# un-ingested material waiting in the inbox.
-set -u
-
-cat "$1" 2>/dev/null
-
-profile="$2/02-DOCS/wiki/harness/user-profile.md"
-optout="$2/.rsc/.no-harness"
-
-if [ ! -f "$profile" ] && [ ! -f "$optout" ]; then
-  cat <<'BANNER'
-
-===== rsc onboarding =====
-Fresh setup: 02-DOCS/wiki/harness/user-profile.md is missing.
-ACTION: invoke `init` now (first contact: technical level + accompaniment dial) before the task.
-If the user does not want a harness here: create .rsc/.no-harness
-==========================
-BANNER
-fi
-
-# Auto-Ingest nudge: when a harness wiki exists and the inbox holds a real file
-# (anything other than README.md / dotfiles / the _processed archive), tell the
-# agent to run the Auto-Ingest Sweep. The hook only reminds; the agent does the
-# scan + ingest. Cheap signal here; the thorough workspace scan lives in the sweep.
-inbox="$2/02-DOCS/inbox"
-if [ -d "$2/02-DOCS/wiki" ] && [ -d "$inbox" ]; then
-  pending=$(find "$inbox" -maxdepth 1 -type f ! -name 'README.md' ! -name '.*' 2>/dev/null | head -1)
-  if [ -n "$pending" ]; then
-    cat <<'BANNER'
-
-===== rsc auto-ingest =====
-Un-ingested material is waiting in 02-DOCS/inbox/.
-ACTION: run the Auto-Ingest Sweep now — ingest inbox/, then scan the workspace
-(minus .rscignore) for un-ingested documents, recording them in wiki/.ingested.json.
-Originals are copied, never moved; deleting an emptied folder needs explicit consent.
-===========================
-BANNER
-  fi
-fi

package/targets/worklog-checkpoint.sh

@@ -1,27 +0,0 @@
-#!/usr/bin/env bash
-# rsc Worklog checkpoint payload (claude). Wired by targets/claude.js onto the
-# PreCompact and SessionEnd hooks.
-#   $1 = absolute project root
-# Reminds the agent to run a Worklog Sweep (capture what we did this session into
-# 02-DOCS/raw/worklog/ so the harness can compile it into the wiki). The hook only
-# REMINDS — the agent writes the worklog (Karpathy: the LLM writes the wiki).
-# Silent when this workspace has no harness wiki yet (nothing to document into).
-set -u
-
-root="${1:-$PWD}"
-
-# No harness wiki here → nothing to do. Stay silent.
-[ -d "$root/02-DOCS/wiki" ] || exit 0
-
-cat <<'BANNER'
-
-===== rsc worklog checkpoint =====
-If this session did meaningful work (files changed, a decision made, a commit),
-run a WORKLOG SWEEP before context is lost:
-  1. Write 02-DOCS/raw/worklog/<YYYY-MM-DD>-<slug>.md using the harness
-     wiki-worklog-template.md (what we did · why · files touched · outcome · next).
-  2. Compile it into wiki/ (update existing articles first; wikilinks + Related);
-     append significant decisions to 02-DOCS/wiki/harness/decisions.md.
-Skip entirely if this was a pure read/answer turn with no changes.
-==================================
-BANNER