@skill-map/cli 0.66.0 → 0.68.0

package/dist/cli/tutorial/sm-tutorial/scripts/lib/paths.js

@@ -7,14 +7,22 @@
 export const PROVIDER_TOKEN = '__PROVIDER__';
 
 export function providerDir(provider) {
-  // agent-skills and Antigravity share the open `.agents/skills/` layout.
-  return provider === 'agent-skills' || provider === 'antigravity'
+  // agent-skills, Antigravity and Codex all keep their SKILLS under the
+  // open `.agents/skills/` layout. (Codex additionally has TOML agents under
+  // `.codex/agents/`, supplied by the codex overlay's literal paths, not this
+  // single base dir.)
+  return provider === 'agent-skills' || provider === 'antigravity' || provider === 'codex'
     ? '.agents/skills'
     : '.claude';
 }
 
 export const PROVIDER_KINDS = {
   claude: new Set(['agent', 'command', 'skill', 'markdown']),
+  // Codex authors its agents as TOML under `.codex/agents/` (a different shape
+  // than the base `__PROVIDER__/agents/*.md`), so the base tier lays only its
+  // shared skill + markdown nodes; the codex overlay supplies the TOML agents
+  // and the command-as-skill nodes (Codex has no `command` kind).
+  codex: new Set(['skill', 'markdown']),
   'agent-skills': new Set(['skill', 'markdown']),
   antigravity: new Set(['skill', 'markdown']),
 };
@@ -23,6 +31,44 @@ export function kindsFor(provider) {
   return PROVIDER_KINDS[provider] ?? PROVIDER_KINDS.claude;
 }
 
+/**
+ * Tutorial track for a provider, by the "does this lens have an `agent`
+ * kind?" axis (see `_core.md` §Provider detection):
+ *   - `rich`  (agent + skill + slash + `@`): `claude`, `codex`.
+ *   - `basic` (skill + markdown, connected by markdown references): the
+ *     open-standard family `agent-skills`, `antigravity`.
+ * The book renders the track's parts; the same lens always resolves to
+ * the same track, so a resumed session never re-derives it.
+ */
+export function trackFor(provider) {
+  return provider === 'claude' || provider === 'codex' ? 'rich' : 'basic';
+}
+
+/**
+ * The provider whose fixture overlays a given provider reuses. The
+ * open-standard family (`agent-skills`, `antigravity`) shares one on-disk
+ * shape (`.agents/skills/`, skill + markdown, connected by markdown
+ * references), so `antigravity` reuses the canonical `agent-skills`
+ * overlays rather than duplicating them. Every other provider keys its own.
+ */
+export function overlayKey(provider) {
+  return provider === 'antigravity' ? 'agent-skills' : provider;
+}
+
+/**
+ * Kinds whose edit fragments (the todo-connectors hub bullets, etc.) apply for
+ * a provider, keyed by TRACK, not by the base-tier kinds. A rich provider links
+ * to every node role even when it renders some differently, Codex's agent is a
+ * TOML overlay and its command-node is a skill, but an `@agent` mention and a
+ * `/command` invocation still resolve, so every bullet applies. A basic
+ * provider only has skill + markdown, so the agent / command bullets fold away.
+ */
+export function fragmentKindsFor(provider) {
+  return trackFor(provider) === 'rich'
+    ? new Set(['agent', 'command', 'skill', 'markdown'])
+    : new Set(['skill', 'markdown']);
+}
+
 /**
  * Per-provider kind directories. The token path is always written in
  * the claude shape (`__PROVIDER__/skills/<name>/...`); resolving it is
@@ -31,6 +77,7 @@ export function kindsFor(provider) {
  */
 const KIND_DIRS = {
   claude: { agents: '.claude/agents', commands: '.claude/commands', skills: '.claude/skills' },
+  codex: { skills: '.agents/skills' },
   'agent-skills': { skills: '.agents/skills' },
   antigravity: { skills: '.agents/skills' },
 };
@@ -64,5 +111,28 @@ export function kindForPath(tokenRelPath) {
   if (tokenRelPath.startsWith(`${PROVIDER_TOKEN}/skills/`)) return 'skill';
   return 'markdown';
 }
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="af99c74c-ee9a-5bf3-8daf-f0b9320a3e61")}catch(e){}}();
-//# debugId=af99c74c-ee9a-5bf3-8daf-f0b9320a3e61
+
+/**
+ * Logical, lens-agnostic node id for a token-form path. The SAME
+ * conceptual node renders in a different kind per lens (a `content-editor`
+ * is an `agent` on claude but a `skill` on agent-skills), so a `--only`
+ * filter or an edit target written in the claude shape must still match
+ * the agent-skills overlay. Agents / commands use the file stem; skills
+ * use the skill directory name; everything else (markdown, notes, docs)
+ * keeps its relpath verbatim. So both `__PROVIDER__/agents/content-editor.md`
+ * and `__PROVIDER__/skills/content-editor/SKILL.md` resolve to `content-editor`.
+ */
+export function nodeIdForTokenPath(tokenRelPath) {
+  const flat = tokenRelPath.match(/^__PROVIDER__\/(?:agents|commands)\/(.+)\.md$/);
+  if (flat) return flat[1];
+  const skill = tokenRelPath.match(/^__PROVIDER__\/skills\/([^/]+)\//);
+  if (skill) return skill[1];
+  // Codex renders an agent as a literal `.codex/agents/<name>.toml`; map it
+  // to the same id as the claude-shaped `__PROVIDER__/agents/<name>.md` so a
+  // `--only` filter (or the skipped-node dedup) matches across the two shapes.
+  const codexAgent = tokenRelPath.match(/^\.codex\/agents\/(.+)\.toml$/);
+  if (codexAgent) return codexAgent[1];
+  return tokenRelPath;
+}
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="9a867c5a-0c61-500e-a583-7d7dd6c5ac92")}catch(e){}}();
+//# debugId=9a867c5a-0c61-500e-a583-7d7dd6c5ac92

package/dist/cli/tutorial/sm-tutorial/scripts/state.js

@@ -31,7 +31,7 @@ import { rmSync, rmdirSync, readdirSync } from 'node:fs';
 import { parseArgs } from './lib/args.js';
 import { exists, readJson, writeJson, succeed, die } from './lib/io.js';
 import { loadManifest, findPart } from './lib/manifest.js';
-import { providerDir } from './lib/paths.js';
+import { providerDir, trackFor } from './lib/paths.js';
 import { loadFixturesManifest, resolveFootprint } from './lib/fixtures-manifest.js';
 
 const STATE_FILE = 'tutorial-state.json';
@@ -68,13 +68,18 @@ const VERBS = {
     if (exists(p) && !args.flags.force) {
       die('exists', `${STATE_FILE} already exists; pass --force to overwrite.`);
     }
+    const provider = args.flags.provider ?? 'claude';
     const state = {
       tutorial: {
         version: STATE_VERSION,
         started_at: now(),
         cwd: args.flags.cwd ?? process.cwd(),
         sm_version: args.flags['sm-version'] ?? null,
-        provider: args.flags.provider ?? 'claude',
+        provider,
+        // Derived from the provider (`_core.md` §Provider detection): the
+        // book renders this track's parts. `rich` = claude/codex,
+        // `basic` = the open-standard family (agent-skills/antigravity).
+        track: trackFor(provider),
         lang: args.flags.lang ?? 'en',
       },
       tester: { level: 2 },
@@ -148,8 +153,13 @@ const VERBS = {
   status() {
     const state = loadState();
     const manifest = loadManifest();
+    // Show only the active track's parts (plus `both`). The rich and basic
+    // campaigns share titles and order, so a session sees exactly one book,
+    // the track resolved at pre-flight (see `_core.md` §Routing + menu).
+    const track = state.tutorial?.track ?? 'rich';
     const parts = manifest.parts
       .filter((p) => p.status === 'active' || state.parts[p.id])
+      .filter((p) => !p.track || p.track === 'both' || p.track === track)
       .map((p) => {
         const tracked = state.parts[p.id];
         return {
@@ -221,8 +231,14 @@ function computeWipePaths(state) {
   const paths = new Set(['tutorial-state.json', 'findings.md', '.skillmapignore', '.skill-map']);
   const addFootprint = (name) => resolveFootprint(manifest, name, provider).forEach((p) => paths.add(p));
 
-  if (has('fundamentals')) addFootprint('prologue');
-  if (has('project-kickoff') || has('connect-harness') || has('daily-loop')) addFootprint('portfolio');
+  // Each footprint covers both tracks: the rich and basic prologues /
+  // campaigns lay the same fixtures (the basic one under the open-standard
+  // provider), so either part's presence means that fixture is on disk.
+  if (has('fundamentals') || has('basic-fundamentals')) addFootprint('prologue');
+  if (
+    has('project-kickoff') || has('connect-harness') || has('daily-loop')
+    || has('basic-kickoff') || has('basic-connect') || has('basic-daily')
+  ) addFootprint('portfolio');
   if (has('extend')) addFootprint('master');
   // `cli` seeds the prologue demo fixture plus its external-ref demo.
   if (has('cli')) { addFootprint('prologue'); addFootprint('cli-external'); }
@@ -258,5 +274,5 @@ function main() {
 }
 
 main();
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="547a8c8d-f962-571c-ae72-8edbb1fc756c")}catch(e){}}();
-//# debugId=547a8c8d-f962-571c-ae72-8edbb1fc756c
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="803fceb9-f545-5f51-8f46-4db3cba6abcf")}catch(e){}}();
+//# debugId=803fceb9-f545-5f51-8f46-4db3cba6abcf