greprag 5.85.1 → 5.85.2

package/dist/commands/load.js

@@ -88,9 +88,11 @@ const LIBRARY = {
         files: ['skill/templates/delivery.md'],
         purpose: 'Why the delivery rules read the way they do: what the goal already authorizes, who counts as a peer (a chip does not), the merge-lock landing sequence, and where a profile says to deploy from.',
     },
+    // adr: adr/skill-ownership-git.md — GrepRAG holds no skill authoring doctrine;
+    // this name survives only as a pointer for launchers and docs that still call it.
     'skill-change': {
         files: ['skill/templates/skill-change.md'],
-        purpose: 'How to change a skill after a run: edit its git folder and commit (GrepRAG only mirrors it), Convention A/B shapes, and when to propose instead.',
+        purpose: 'Pointer: skill authoring and change rules live in /skill-instructions; git owns skills.',
     },
     mechanic: {
         files: ['skill/mechanic/SKILL.md', 'skill/mechanic/docs/skill-fix-conventions.md'],

package/dist/native-skill-mirror.js

@@ -349,6 +349,7 @@ Required action chains:
 - **Use this skill:** run \`${loadCommand}\` before acting.
 - **Change this skill:** edit ${gitSkillFolder(skill)} with normal file edits, commit, then run \`${syncCommand}\` (session start does it too) and verify with \`${loadCommand}\`. Uncommitted edits are never served. History and rollback are git log / git revert on that folder.
 - **Create another skill:** create \`~/.claude/skills/<new-name>/SKILL.md\` (dotclaude repo, global) or \`<repo>/.claude/skills/<new-name>/SKILL.md\` (project), commit, then run \`${syncCommand}\`.
+- **How to write or change a skill:** follow \`/skill-instructions\` (\`~/.claude/skills/skill-instructions/\`).
 
 Authority: ${skillId}
 Source binding: ${sourceBindingState(skill)}

package/dist/opencode-plugin.bundle.js

@@ -4413,6 +4413,7 @@ Required action chains:
 - **Use this skill:** run \`${loadCommand}\` before acting.
 - **Change this skill:** edit ${gitSkillFolder(skill)} with normal file edits, commit, then run \`${syncCommand}\` (session start does it too) and verify with \`${loadCommand}\`. Uncommitted edits are never served. History and rollback are git log / git revert on that folder.
 - **Create another skill:** create \`~/.claude/skills/<new-name>/SKILL.md\` (dotclaude repo, global) or \`<repo>/.claude/skills/<new-name>/SKILL.md\` (project), commit, then run \`${syncCommand}\`.
+- **How to write or change a skill:** follow \`/skill-instructions\` (\`~/.claude/skills/skill-instructions/\`).
 
 Authority: ${skillId}
 Source binding: ${sourceBindingState(skill)}

package/dist/skill-staleness.js

@@ -358,7 +358,7 @@ function formatSkillStaleness(report, options = {}) {
         lines.push('1. Pause trusting this skill; read the changed watched source files/diffs first.');
         lines.push('2. Classify relevance: low relevance or small delta -> use repo source as truth; no chip.');
         lines.push('3. If this task depends on the stale area and the delta is material, spawn a skill-refresh chip.');
-        lines.push('4. The refresh chip edits the skill\'s git folder (~/.claude/skills/<name>/ in dotclaude, or <repo>/.claude/skills/<name>/) and commits; the mirror follows.');
+        lines.push('4. The refresh chip edits the skill\'s git folder (~/.claude/skills/<name>/ in dotclaude, or <repo>/.claude/skills/<name>/) and commits, following /skill-instructions; the mirror follows.');
     }
     if (report.projects.some(p => p.dirtyPaths.length > 0)) {
         lines.push('Baseline rule: do not stamp a new baseline while watched local edits are dirty.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "greprag",
-  "version": "5.85.1",
+  "version": "5.85.2",
   "description": "Private GrepRAG client for existing Claude Code, Codex, OpenCode, and Grok Build tenants.",
   "main": "dist/index.js",
   "bin": {

package/skill/mechanic/docs/skill-fix-conventions.md

@@ -1,20 +1,13 @@
 # Skill-Fix Conventions
 
-**Status:** SHIPPED leg of the skill-tuning reflex (`docs/mechanic-skill-tuning.md`). The portable recipe set the reflex applies when a skill exhibits friction. Ships **with greprag** (bundled into the mechanic skill package, mirrored to `~/.claude/skills/mechanic/docs/`) so the fixer works for any user — it does **not** depend on `/skill-optimize` being installed. `/skill-optimize` remains the richer interactive tool; this is the minimal portable subset the reflex carries.
+The six recipes the skill-tuning reflex applies when a skill shows friction (`docs/mechanic-skill-tuning.md` in the greprag repo). How a skill is written and changed (where it lives, the loud-trigger form, what to cut, the commit check) is owned by `/skill-instructions` (`~/.claude/skills/skill-instructions/`); load it before editing. This doc holds only the mechanic's friction-to-edit map.
 
 ## What this is for
 
-The skill-tuning reflex injects a directive when a skill under break-in loads. The old directive said "watch your execution, emit a SELF-TUNING block, propose a fix" — vague (it pointed at the *run*, not the *skill*) and propose-only (no edit ever landed). This doc replaces that with **six locatable skill defects**, each mapped to one concrete edit. The reflex's loop is: **detect a defined friction → locate it in the skill's SKILL.md → apply the fix per the matching recipe → show the diff for one accept/revert.**
+The loop: **detect a defined friction → locate it in the skill's SKILL.md → apply the fix per the matching recipe → show the diff for one accept/revert.**
 
 A friction that doesn't match one of the six below is **not auto-fixable inline**. Search Memory for the exact skill/friction first; apply a clear overcome and continue. If Memory has no answer, the overcome fails, or the friction repeats, spawn one doctrine fix chip (`greprag fix spawn --type doctrine "<skill friction unit>"`). Use `greprag fix log` only for deliberate audit/design notes. Do not invent edits outside these recipes.
 
-## The two edit primitives (skill-optimize vocabulary)
-
-Every fix below resolves to one of two inline forms:
-
-- **Convention A — one-liner + link.** Reference material (facts, recipes, paths). Inline: one sentence naming what the linked doc holds + the path.
-- **Convention B — loud trigger + inline rule + link.** A proactive-fire rule with no external trigger. Three parts: (1) interrogative intent-catch — `ABOUT TO <do X>?` names the *moment*, not the topic; (2) hard stop — `STOP —`; (3) the inline rule, complete enough to comply without opening the link. A flat `IMPORTANT: do Y` reads as wallpaper and gets skipped — the interrogative + hard stop is what catches the model mid-generation.
-
 ## The six frictions → six fixes
 
 Each: **signature** (how it shows up in the run) · **location** (where it lives in SKILL.md) · **fix** (the exact edit).
@@ -32,12 +25,12 @@ Each: **signature** (how it shows up in the run) · **location** (where it lives
 ### 3. flattened-relay
 - **Signature:** the skill made you summarize/paraphrase content that should pass through verbatim (or forced verbatim where a summary was wanted).
 - **Location:** the output-shape / relay rule.
-- **Fix:** add a Convention-B rule pinning relay fidelity — `ABOUT TO SUMMARIZE <X>? STOP — relay it verbatim; <why>.`
+- **Fix:** add a loud trigger pinning relay fidelity — `ABOUT TO SUMMARIZE <X>? STOP — relay it verbatim; <why>.`
 
 ### 4. wallpaper-rule
 - **Signature:** a real rule in the skill got skipped because it's a flat `IMPORTANT:`/prose imperative the model read as decoration.
 - **Location:** the rule's existing inline form.
-- **Fix:** rewrite that rule in place to Convention B (interrogative intent-catch + `STOP —` + the inline rule). Same content, fingerprint that catches the model in the act. Don't add a new rule — convert the existing one.
+- **Fix:** rewrite that rule in place as a loud trigger. Same content; don't add a new rule — convert the existing one.
 
 ### 5. stale-reference
 - **Signature:** a path, command, or name in the skill 404'd, errored, or pointed at a renamed/removed thing.
@@ -47,12 +40,10 @@ Each: **signature** (how it shows up in the run) · **location** (where it lives
 ### 6. missing-rule
 - **Signature:** you hit a gotcha mid-run that the skill should have warned about but doesn't.
 - **Location:** absent — a gap.
-- **Fix:** add a new Convention-B proactive-fire rule at the spot in SKILL.md where the agent would meet the gotcha. Keep it to the interrogative + hard stop + one-line rule; link a doc only if a full procedure exists.
+- **Fix:** add one loud trigger at the spot in SKILL.md where the agent would meet the gotcha; link a doc only if a full procedure exists.
 
 ## Guardrails
 
 - **One friction, one edit.** If a run surfaced two defects, make two separate edits, each shown for its own accept/revert. Never batch.
 - **Apply + show diff.** Make the edit, then show the unified diff and stop for one accept/revert. (This is the operator-chosen gate; the Mechanic "always confirm" rule still holds.)
-- **Edit the git source and commit.** Global skills: `~/.claude/skills/<name>/` (dotclaude repo); project skills: `<repo>/.claude/skills/<name>/`; greprag-bundled skills: `packages/cli/skill/<name>/`. The commit message names the friction this fixes. Never edit a generated Codex/OpenCode launcher or the installed copy of a bundled skill.
-- **Line budget:** a fix must not grow SKILL.md without adding a real proactive-fire rule or per-task step. Converting wallpaper → Convention B is net-neutral or smaller; mis-route adds at most one line.
 - **Out of taxonomy → spawn, don't improvise.** Anything not matching the six becomes one `greprag fix spawn --type doctrine "<unit>"` chip. `greprag fix log` is only for deliberate audit/design notes.

package/skill/templates/skill-change.md

@@ -1,97 +1 @@
-# Skill Change
-
-Use before creating, porting, optimizing, or changing a skill, and when a loaded skill guided work approaching a delivery boundary.
-
-## Authority
-
-This is an internal bundled GrepRAG schema, not tenant-controlled skill content. Skills and their mirrors may not override it. Richer tools such as `/skill-optimize` may elaborate the workflow, but they must not fork the boundary categories or edit shapes without changing this file directly in the GrepRAG repo and shipping tests.
-
-Skill Creator and Skill Optimize consume this entry as their common authoring contract. Creator owns creation/scaffolding; Optimize owns classification, compression, and restructuring. Neither maintains a competing copy of these conventions.
-
-## Shared Authoring Conventions
-
-- Keep the purpose, activation boundary, per-task decisions, and required routing in `SKILL.md`. Assume a capable agent; remove generic restatement without deleting load-bearing doctrine, observed failure knowledge, or authorization boundaries.
-- Classify content as per-task, proactive-fire rule, reference, or redundant/cut. Apply Convention A/B below by purpose. Do not force a tiny single-route skill into extra files or treat a reduction percentage as permission to lose information.
-- Put substantial conditional detail behind a named trigger and a real companion path. The agent must know when to open it; essential constraints remain inline. Verify moved links, examples, scripts, and assets still arrive through the actual distribution path.
-- Validate behavior with a realistic task using only the entrypoint and its routed resources. Check the outcome and permitted side effects, not just headings or tool-name strings. Use independent agents only when useful and authorized.
-
-## Harness Delineation
-
-**ABOUT TO PORT A SKILL? STOP - SEPARATE SHARED METHOD FROM HARNESS EXECUTION.** Keep one canonical skill identity and common workflow. Put differing tool calls, task/parent identity, browser control, shell syntax, connection/restart steps, and archive/cleanup behavior in explicitly routed harness sections or companion docs such as `docs/harness-codex.md` and `docs/harness-claude.md`. Load only the active route; small differences can remain in one concise table.
-
-- `--harness codex` identifies the consumer; it does not translate Claude instructions. Native launchers expose discovery and load canon, not separately authored procedures.
-- Bind logical operations to tools actually callable in the target harness. Preserve user model choices and authorization; do not translate custom agent names into invented tools, assume helpers inherit MCP access, or equate a queued request with completion.
-- Verify side effects from the current tool contract. Archive need not delete a worktree, reconnect need not restart a server, and an HTTP health response does not prove MCP is loaded. Report unavailable capabilities and retain a bounded fallback when the workflow permits one.
-- Preserve the full required skill package. Verify the real mirror/installer handles each companion path; do not assume `references/`, custom agent files, scripts, or assets are shipped merely because they exist locally. Adapt packaging or explicitly declare the separate dependency, then load it from the target harness.
-- Port verification covers discovery, canonical payload and companion loading, needed tool schemas, and at least one relevant execution/read-only connection check. Distinguish those checks from unrun publishing, deletion, image generation, or production tests.
-
-**ABOUT TO RESTORE A SKILL? STOP - A GENERATED LAUNCHER IS NOT THE PROCEDURE.** If a SKILL.md contains a generated-adapter header or only points back to its own `greprag load`, recover the actual procedure from git history or the mirror's archived revisions and commit it to the git folder. Do not recursively load it or commit another launcher as the repair.
-
-## Boundary Rule
-
-**ABOUT TO CHANGE A SKILL? STOP - LOAD THIS RULE FIRST.** Skill edits are learning-capture, not cleanup. Edit only when the run exposed a durable rule, missing progressive-disclosure link, stale reference, trigger bug, or wrong handoff. If the change alters the skill's method or risk posture, propose it instead of silently landing it.
-
-Explicitly requested creation, porting, or redesign is authorized work. The proposal rule applies to changes beyond that scope; do not ask again for the same approved change.
-
-**ABOUT TO REFRESH A STALE SKILL? STOP - THIS IS A DECISION GATE, NOT AUTO-EDIT PERMISSION.** First read the changed watched source files/diffs and classify relevance. Low relevance or small deltas use repo source as truth and do not need a chip. Spawn a `skill-refresh` chip only when the stale area directly affects the task and the delta is material.
-
-## Where To Edit
-
-Git owns every skill; GrepRAG only mirrors the committed files and refuses every write to the mirror.
-
-- Global skill: edit `~/.claude/skills/<name>/` (the dotclaude git repo) and commit.
-- Project skill: edit `<repo>/.claude/skills/<name>/` and commit in that repo.
-- Bundled GrepRAG skill: edit `packages/cli/skill/<name>/` in the greprag repo, never the installed artifact.
-- Codex/OpenCode `SKILL.md` files are generated launchers: never edit them. Edit the git folder they name.
-- Unknown source: stop and identify the git folder before editing.
-- **ABOUT TO CALL A SKILL CHANGE DONE? STOP - COMMIT IT.** Uncommitted edits are never mirrored; the commit message carries the learning the edit captures, not "update skill". The next session start (or `greprag skill mirror sync`) copies the commit into the mirror for every harness.
-- History and rollback: `git log`, `git diff` and `git revert` on the skill folder. `greprag skill history <skill>` and `greprag skill diff <skill> <revA> [<revB>]` still read the mirror's archived revisions.
-
-## Skill-Refresh Chips
-
-A `skill-refresh` chip must:
-
-- load the skill with `greprag load <skill>`
-- read the changed watched source files/diffs before trusting stale skill claims
-- decide whether the right result is a skill edit or only a new source stamp (`greprag skill source stamp`, which writes the git SKILL.md)
-- edit the skill's git folder and commit, never a generated launcher
-- report checks and whether release/deploy is needed
-
-## Edit Shapes
-
-- **Convention A - one-liner + link.** Use for reference material: facts, recipes, paths, command syntax. Inline sentence names what the linked doc contains and its path.
-- **Convention B - loud trigger + inline rule + optional link.** Use for proactive-fire rules. Shape: `ABOUT TO <do X>? STOP - <complete rule>.` Catch the intent at the moment of action, state the complete constraint inline, and link only the longer procedure. A topic heading or bare `IMPORTANT` label does not carry this convention.
-
-## Auto-Land vs Propose
-
-Auto-land only narrow, low-risk edits:
-
-- add a missing trigger phrase or `NOT for` boundary
-- repoint a verified stale path/name/command
-- add a missing progressive-disclosure link
-- convert skipped wallpaper prose into Convention B without changing meaning
-- add one gotcha rule that directly follows from the run
-
-Propose instead:
-
-- method changes
-- broader scope changes
-- new autonomy/destructive behavior
-- ambiguous doctrine
-- anything that would need user taste, strategy, or risk judgment
-
-## Defect Map
-
-- Mis-route: update `description` / `Use when` / `NOT for`.
-- Wrong altitude: add a handoff or out-of-scope line naming the delegate.
-- Flattened relay: add a Convention-B relay-fidelity rule.
-- Wallpaper rule: rewrite the existing rule to Convention B.
-- Stale reference: verify and repoint; remove only if wholly dead.
-- Missing rule: add one Convention-B gotcha rule at the point the agent meets it.
-
-## Guardrails
-
-- One learning, one edit.
-- Do not grow `SKILL.md` unless adding a real per-task step or proactive-fire rule.
-- Show the diff before finalizing if the operator has not already authorized the exact edit.
-- Out of taxonomy: `greprag fix spawn "<skill friction unit>"`.
+Skill authoring and change rules live in /skill-instructions (~/.claude/skills/skill-instructions). Git owns skills.

package/dist/claude-lifecycle-cutover.js

@@ -1,171 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.CLAUDE_LIFECYCLE_SKILLS = void 0;
-exports.applyClaudeLifecycleCutover = applyClaudeLifecycleCutover;
-exports.syncClaudeLifecycleCutover = syncClaudeLifecycleCutover;
-const fs = __importStar(require("fs"));
-const os = __importStar(require("os"));
-const path = __importStar(require("path"));
-const native_skill_mirror_1 = require("./native-skill-mirror");
-const skill_mirror_files_1 = require("./skill-mirror-files");
-exports.CLAUDE_LIFECYCLE_SKILLS = ['commit', 'deploy', 'release'];
-function adapterFor(skill) {
-    const files = (0, native_skill_mirror_1.buildNativeSkillAdapter)('claude-code', skill, { allowClaudeCode: true });
-    if (!files || files.length !== 1 || files[0].path !== 'SKILL.md') {
-        throw new Error(`Claude lifecycle adapter is invalid for ${skill.skillName}.`);
-    }
-    return files;
-}
-function safeTimestamp(now) {
-    return now.toISOString().replace(/[:.]/g, '-');
-}
-function requiredSkills(skills, names) {
-    const byName = new Map(skills
-        .filter(skill => skill.scope !== 'project')
-        .map(skill => [skill.skillName, skill]));
-    return names.map(name => {
-        const skill = byName.get(name);
-        if (!skill)
-            throw new Error(`Canonical Claude lifecycle skill missing: ${name}.`);
-        return skill;
-    });
-}
-/**
- * Explicit, narrow Claude cutover. Only SKILL.md becomes mirror-managed;
- * companion scripts and docs remain untouched in the native package.
- *
- * `names` selects which mirrored skills to materialize; it defaults to the
- * lifecycle triple. Naming skills explicitly is how a Claude Code operator
- * closes an adapter gap without arming a fleet-wide sweep.
- */
-function applyClaudeLifecycleCutover(params) {
-    const homeDir = params.homeDir || os.homedir();
-    const root = (0, native_skill_mirror_1.nativeSkillRoot)('claude-code', homeDir);
-    const state = (0, native_skill_mirror_1.readState)(root);
-    const names = params.names?.length ? params.names : exports.CLAUDE_LIFECYCLE_SKILLS;
-    const prepared = requiredSkills(params.skills, names).map(skill => {
-        const target = path.join(root, skill.skillName);
-        if ((0, skill_mirror_files_1.isSkillFolderOwnedElsewhere)(target)) {
-            throw new Error(`Claude skill folder is owned elsewhere (a link or another installer's package): ${skill.skillName}; not written.`);
-        }
-        const skillMd = path.join(target, 'SKILL.md');
-        const adapterFiles = adapterFor(skill);
-        const nextHash = (0, skill_mirror_files_1.computeSkillFileHash)(adapterFiles);
-        const managed = state.skills[skill.skillName];
-        let currentContent;
-        if (fs.existsSync(skillMd))
-            currentContent = fs.readFileSync(skillMd, 'utf8');
-        if (managed) {
-            const currentFiles = currentContent === undefined
-                ? null
-                : [{ path: 'SKILL.md', content: currentContent }];
-            if (!currentFiles || (0, skill_mirror_files_1.computeSkillFileHash)(currentFiles) !== managed.hash) {
-                throw new Error(`Claude lifecycle adapter diverged: ${skill.skillName}.`);
-            }
-        }
-        else if (fs.existsSync(target) && currentContent === undefined) {
-            throw new Error(`Claude lifecycle package has no SKILL.md: ${skill.skillName}.`);
-        }
-        return { skill, target, skillMd, adapterFiles, nextHash, managed, currentContent };
-    });
-    const replacements = prepared.filter(item => !item.managed && item.currentContent !== undefined);
-    let backupDir;
-    if (replacements.length > 0) {
-        backupDir = path.join(homeDir, '.greprag', 'backups', 'claude-lifecycle-cutover', safeTimestamp(params.now || new Date()));
-        for (const item of replacements) {
-            const destination = path.join(backupDir, item.skill.skillName, 'SKILL.md');
-            fs.mkdirSync(path.dirname(destination), { recursive: true });
-            fs.writeFileSync(destination, item.currentContent, 'utf8');
-        }
-    }
-    const result = {
-        installed: 0, updated: 0, current: 0, blocked: 0, backupDir,
-    };
-    for (const item of prepared) {
-        if (item.managed && item.managed.hash === item.nextHash) {
-            result.current++;
-            continue;
-        }
-        fs.mkdirSync(item.target, { recursive: true });
-        fs.writeFileSync(item.skillMd, item.adapterFiles[0].content, 'utf8');
-        state.skills[item.skill.skillName] = { hash: item.nextHash, files: ['SKILL.md'] };
-        if (item.managed)
-            result.updated++;
-        else
-            result.installed++;
-    }
-    (0, native_skill_mirror_1.writeState)(root, state);
-    return result;
-}
-async function syncClaudeLifecycleCutover(params) {
-    const names = params.names?.length ? [...new Set(params.names)] : [...exports.CLAUDE_LIFECYCLE_SKILLS];
-    const list = await fetch(`${params.apiUrl}/v1/skillmirror`, {
-        headers: { Authorization: `Bearer ${params.apiKey}` },
-    });
-    if (!list.ok)
-        throw new Error(`skill mirror list failed: HTTP ${list.status}`);
-    const data = await list.json();
-    const wanted = new Set(names);
-    const rows = (data.skills || []).filter(row => wanted.has(row.skillName) && row.scope !== 'project');
-    if (rows.length !== names.length) {
-        const missing = names.filter(name => !rows.some(row => row.skillName === name));
-        throw new Error(`Not in the global skill mirror: ${missing.join(', ')}; native files preserved.`);
-    }
-    const fetched = await Promise.all(rows.map(async (row) => {
-        const query = new URLSearchParams({ skillId: row.skillId || `global:${row.skillName}` });
-        const response = await fetch(`${params.apiUrl}/v1/skillmirror/${encodeURIComponent(row.skillName)}?${query.toString()}`, { headers: { Authorization: `Bearer ${params.apiKey}` } });
-        if (!response.ok)
-            return null;
-        const body = await response.json();
-        if (!Array.isArray(body.files))
-            return null;
-        return {
-            ...row,
-            scope: 'global',
-            files: body.files,
-            contentHash: body.contentHash || row.contentHash,
-            revision: body.revision ?? row.revision,
-        };
-    }));
-    if (fetched.some(skill => skill === null)) {
-        throw new Error('Claude lifecycle snapshot incomplete; native files preserved.');
-    }
-    return applyClaudeLifecycleCutover({
-        skills: fetched,
-        homeDir: params.homeDir,
-        names,
-    });
-}

package/dist/native-skill-adoption.js

@@ -1,217 +0,0 @@
-"use strict";
-/** Adopt unmanaged native skills into the GrepRAG skill mirror. */
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.planNativeSkillAdoption = planNativeSkillAdoption;
-exports.installAdoptedNativeSkill = installAdoptedNativeSkill;
-const fs = __importStar(require("node:fs"));
-const os = __importStar(require("node:os"));
-const path = __importStar(require("node:path"));
-const native_skill_mirror_1 = require("./native-skill-mirror");
-const skill_mirror_files_1 = require("./skill-mirror-files");
-const NATIVE_SKILL_NAME_RE = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
-function listSkillDirs(root) {
-    try {
-        return fs.readdirSync(root, { withFileTypes: true })
-            .filter(entry => entry.isDirectory() && fs.existsSync(path.join(root, entry.name, 'SKILL.md')))
-            .map(entry => entry.name)
-            .sort((a, b) => a.localeCompare(b));
-    }
-    catch {
-        return [];
-    }
-}
-function countSkillFiles(root) {
-    let count = 0;
-    const walk = (dir, depth) => {
-        if (depth > 7 || count >= 1500)
-            return;
-        let entries;
-        try {
-            entries = fs.readdirSync(dir, { withFileTypes: true });
-        }
-        catch {
-            return;
-        }
-        for (const entry of entries) {
-            if (count >= 1500)
-                return;
-            const full = path.join(dir, entry.name);
-            if (entry.isDirectory())
-                walk(full, depth + 1);
-            else if (entry.isFile() && entry.name.toLowerCase() === 'skill.md')
-                count++;
-        }
-    };
-    walk(root, 0);
-    return count;
-}
-function adoptionRoots(params) {
-    const roots = [{ root: (0, native_skill_mirror_1.nativeSkillRoot)(params.platform, params.homeDir), scope: 'global' }];
-    const projectRoot = params.projectDir && params.projectId
-        ? (0, native_skill_mirror_1.writerOwnedProjectSkillRoot)(params.platform, params.projectDir, params.homeDir)
-        : null;
-    if (projectRoot) {
-        roots.push({
-            root: projectRoot,
-            scope: 'project',
-            projectId: params.projectId,
-            projectName: params.projectName,
-        });
-    }
-    return roots;
-}
-function ignoredDiscoveryRoots(params) {
-    const roots = [];
-    if (params.platform === 'codex') {
-        roots.push([path.join(params.homeDir, '.agents', 'skills'), 'discovered by Codex but not writer-owned'], [path.join(params.homeDir, '.codex', 'plugins', 'cache'), 'plugin cache is read-only']);
-        if (params.projectDir)
-            roots.push([path.join(params.projectDir, '.codex', 'skills'), 'discovered by Codex but not writer-owned']);
-    }
-    const owned = new Set(adoptionRoots({
-        platform: params.platform,
-        homeDir: params.homeDir,
-        projectDir: params.projectDir,
-        projectId: '00000000-0000-4000-8000-000000000000',
-    }).map(root => path.resolve(root.root).toLowerCase()));
-    return roots
-        .filter(([root]) => !owned.has(path.resolve(root).toLowerCase()))
-        .map(([root, reason]) => ({ root, reason, skillCount: countSkillFiles(root) }))
-        .filter(root => root.skillCount > 0);
-}
-function planNativeSkillAdoption(params) {
-    const homeDir = params.homeDir || os.homedir();
-    const projectDir = params.projectDir ? path.resolve(params.projectDir) : undefined;
-    const plan = {
-        candidates: [],
-        managed: [],
-        blocked: [],
-        skipped: [],
-        ignoredRoots: ignoredDiscoveryRoots({ platform: params.platform, homeDir, projectDir }),
-    };
-    if (!(0, native_skill_mirror_1.nativeSkillMirrorEnabled)(params.platform))
-        return plan;
-    for (const root of adoptionRoots({
-        platform: params.platform,
-        homeDir,
-        projectDir,
-        projectId: params.projectId,
-        projectName: params.projectName,
-    })) {
-        const state = (0, native_skill_mirror_1.readState)(root.root);
-        for (const skillName of listSkillDirs(root.root)) {
-            const target = (0, native_skill_mirror_1.targetPath)(root.root, skillName);
-            const managed = state.skills[skillName];
-            if (managed) {
-                const currentFiles = (0, native_skill_mirror_1.readManagedFiles)(target, managed);
-                const currentHash = currentFiles ? (0, skill_mirror_files_1.computeSkillFileHash)(currentFiles) : '';
-                const entry = { ...root, skillName, reason: currentHash === managed.hash ? 'managed/current' : 'managed/diverged' };
-                if (currentHash === managed.hash)
-                    plan.managed.push(entry);
-                else
-                    plan.blocked.push(entry);
-                continue;
-            }
-            if (!NATIVE_SKILL_NAME_RE.test(skillName)) {
-                plan.skipped.push({ ...root, skillName, reason: 'platform-invalid name' });
-                continue;
-            }
-            const files = params.collectFiles(target);
-            if (!files) {
-                plan.skipped.push({ ...root, skillName, reason: 'invalid or oversized markdown payload' });
-                continue;
-            }
-            // A directory greprag already wrote an adapter into is not an unmanaged
-            // native skill, whatever the local state file says (it is per-root and
-            // per-machine, so a fresh checkout or a lost state file makes every
-            // adapter look adoptable). Adopting one would upload the launcher as
-            // canon and erase the real body. Same rule as the shadow upload path.
-            // adr: adr/skill-adapter-never-canon.md
-            const main = files.find(file => file.path === 'SKILL.md');
-            if (main && (0, skill_mirror_files_1.isGeneratedSkillAdapter)(main.content)) {
-                plan.skipped.push({ ...root, skillName, reason: 'generated adapter is never a mirror source' });
-                continue;
-            }
-            plan.candidates.push({ ...root, skillName, files, sourceHash: (0, skill_mirror_files_1.computeSkillFileHash)(files) });
-        }
-    }
-    const globalNames = new Set([
-        ...plan.candidates.filter(candidate => candidate.scope === 'global').map(candidate => candidate.skillName.toLowerCase()),
-        ...plan.managed.filter(managed => managed.scope === 'global').map(managed => managed.skillName.toLowerCase()),
-    ]);
-    for (let i = plan.candidates.length - 1; i >= 0; i--) {
-        const candidate = plan.candidates[i];
-        if (candidate.scope !== 'project' || !globalNames.has(candidate.skillName.toLowerCase()))
-            continue;
-        plan.blocked.push({ ...candidate, reason: 'project/global collision requires explicit replaces metadata' });
-        plan.candidates.splice(i, 1);
-    }
-    return plan;
-}
-function installAdoptedNativeSkill(params) {
-    if (!(0, native_skill_mirror_1.nativeSkillMirrorEnabled)(params.platform))
-        return 'skipped';
-    if (!NATIVE_SKILL_NAME_RE.test(params.skill.skillName))
-        return 'skipped';
-    const target = (0, native_skill_mirror_1.targetPath)(params.root, params.skill.skillName);
-    const state = (0, native_skill_mirror_1.readState)(params.root);
-    if (state.skills[params.skill.skillName])
-        return 'blocked';
-    const currentFiles = params.collectFiles(target);
-    if (!currentFiles || (0, skill_mirror_files_1.computeSkillFileHash)(currentFiles) !== params.expectedSourceHash)
-        return 'blocked';
-    const adapterFiles = (0, native_skill_mirror_1.buildNativeSkillAdapter)(params.platform, params.skill);
-    if (!adapterFiles)
-        return 'skipped';
-    // Adoption replaces only the files it just moved into canon. The mirror
-    // carries SKILL.md and docs/**/*.md; everything else in the package
-    // (scripts/, references/, assets/, any non-markdown file) exists nowhere but
-    // this folder, and wiping the folder destroyed exactly those. Write the
-    // adapter, then remove the adopted files it did not overwrite, the same rule
-    // a managed adapter update follows. adr: adr/skill-bundle-shape.md
-    for (const file of adapterFiles) {
-        const destination = (0, native_skill_mirror_1.filePath)(target, file.path);
-        fs.mkdirSync(path.dirname(destination), { recursive: true });
-        fs.writeFileSync(destination, file.content, 'utf8');
-    }
-    const adapterPaths = new Set(adapterFiles.map(file => file.path));
-    (0, native_skill_mirror_1.removeTrackedFiles)(target, currentFiles.map(file => file.path).filter(relative => !adapterPaths.has(relative)));
-    state.skills[params.skill.skillName] = {
-        hash: (0, skill_mirror_files_1.computeSkillFileHash)(adapterFiles),
-        files: adapterFiles.map(file => file.path).sort(),
-    };
-    (0, native_skill_mirror_1.writeState)(params.root, state);
-    return 'installed';
-}