@longtable/provider-codex 0.1.38 → 0.1.40

package/dist/skills.d.ts

@@ -9,8 +9,9 @@ export interface InstalledCodexSkill {
     path: string;
     description: string;
 }
+export type LongTableSkillSurface = "compact" | "full";
 export declare function resolveCodexSkillsDir(customDir?: string): string;
-export declare function buildCodexSkillSpecs(roles: RoleDefinition[]): CodexSkillSpec[];
-export declare function installCodexSkills(roles: RoleDefinition[], customDir?: string): Promise<InstalledCodexSkill[]>;
+export declare function buildCodexSkillSpecs(roles: RoleDefinition[], surface?: LongTableSkillSurface): CodexSkillSpec[];
+export declare function installCodexSkills(roles: RoleDefinition[], customDir?: string, surface?: LongTableSkillSurface): Promise<InstalledCodexSkill[]>;
 export declare function removeCodexSkills(roles: RoleDefinition[], customDir?: string): Promise<string[]>;
-export declare function listInstalledCodexSkills(roles: RoleDefinition[], customDir?: string): Promise<InstalledCodexSkill[]>;
+export declare function listInstalledCodexSkills(roles: RoleDefinition[], customDir?: string, surface?: LongTableSkillSurface): Promise<InstalledCodexSkill[]>;

package/dist/skills.js

@@ -2,10 +2,20 @@ import { existsSync } from "node:fs";
 import { mkdir, readdir, rm, writeFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { join, resolve } from "node:path";
+const COMPACT_ROLE_SKILL_NAMES = {
+    methods_critic: "longtable-methods",
+    measurement_auditor: "longtable-measure",
+    theory_critic: "longtable-theory",
+    reviewer: "longtable-reviewer",
+    voice_keeper: "longtable-voice"
+};
 export function resolveCodexSkillsDir(customDir) {
     return customDir ? resolve(customDir) : join(homedir(), ".codex", "skills");
 }
-function skillNameForRole(role) {
+function skillNameForRole(role, surface) {
+    if (surface === "compact" && COMPACT_ROLE_SKILL_NAMES[role.key]) {
+        return COMPACT_ROLE_SKILL_NAMES[role.key];
+    }
     return `longtable-${role.key.replaceAll("_", "-")}`;
 }
 function yamlString(value) {
@@ -23,8 +33,8 @@ function renderSkillFile(spec) {
         ...spec.body
     ].join("\n");
 }
-function baseSkillSpecs() {
-    return [
+function baseSkillSpecs(surface = "compact") {
+    const base = [
         {
             name: "longtable",
             description: "Use for LongTable research conversations, lt explore/review/panel requests, researcher checkpoints, and role routing.",
@@ -44,6 +54,7 @@ function baseSkillSpecs() {
                 "- `use the LongTable methods critic on this design`",
                 "- `$longtable: deploy a research team to review this measurement plan, show the main disagreements, and ask me what decision should be recorded before you revise it.`",
                 "- `$longtable: use editor, reviewer, methods, measurement, and voice perspectives to evaluate this manuscript section. Do not collapse disagreement too early.`",
+                "- `$longtable-methods`, `$longtable-measure`, `$longtable-theory`, `$longtable-reviewer`, or `$longtable-voice` when the researcher explicitly wants that shortcut.",
                 "- `$longtable-interview` to create or continue the first research-start interview.",
                 "",
                 "## Routing Rules",
@@ -53,6 +64,7 @@ function baseSkillSpecs() {
                 "- `lt panel` creates a visible multi-role review with synthesis, role opinions, conflict summary, and a decision prompt.",
                 "- `$longtable-interview` runs the provider-native LongTable start interview. It replaces the old CLI start questionnaire as the primary research-start surface.",
                 "- Natural references to methods, measurement, theory, reviewer, editor, ethics, venue, or voice should foreground the matching LongTable role.",
+                "- The compact visible shortcut set is methods, measure, theory, reviewer, and voice. Other roles remain available through this router when the request calls for them.",
                 "- When research responsibility is about to shift, surface a Researcher Checkpoint before closure.",
                 "- When changing LongTable product language, README positioning, or checkpoint policy, surface a Meta-Decision Checkpoint first.",
                 "",
@@ -160,7 +172,9 @@ function baseSkillSpecs() {
                 "",
                 "If MCP tools are unavailable, continue the natural-language interview in Codex, but state that durable LongTable state could not be written. Do not pretend a checkpoint or First Research Shape was recorded."
             ]
-        },
+        }
+    ];
+    const fullOnly = [
         {
             name: "longtable-panel",
             description: "Use when LongTable should run a panel or team-style review with visible role disagreement.",
@@ -227,6 +241,7 @@ function baseSkillSpecs() {
             ]
         }
     ];
+    return surface === "full" ? [...base, ...fullOnly] : base;
 }
 function mustAskQuestionsForRole(role) {
     const common = [
@@ -261,10 +276,10 @@ function mustAskQuestionsForRole(role) {
     };
     return [...(byRole[role.key] ?? []), ...common];
 }
-function roleSkillSpec(role) {
+function roleSkillSpec(role, surface = "compact") {
     const label = role.label;
     return {
-        name: skillNameForRole(role),
+        name: skillNameForRole(role, surface),
         description: `Use the LongTable ${label} role: ${role.shortDescription}`,
         body: [
             "## Purpose",
@@ -311,14 +326,32 @@ function roleSkillSpec(role) {
         ]
     };
 }
-export function buildCodexSkillSpecs(roles) {
-    return [...baseSkillSpecs(), ...roles.map(roleSkillSpec)];
+function compactRoles(roles) {
+    return roles.filter((role) => Object.hasOwn(COMPACT_ROLE_SKILL_NAMES, role.key));
 }
-export async function installCodexSkills(roles, customDir) {
+function allCodexSkillSpecs(roles) {
+    const byName = new Map();
+    for (const spec of [...buildCodexSkillSpecs(roles, "compact"), ...buildCodexSkillSpecs(roles, "full")]) {
+        byName.set(spec.name, spec);
+    }
+    return [...byName.values()];
+}
+export function buildCodexSkillSpecs(roles, surface = "compact") {
+    const roleSpecs = (surface === "compact" ? compactRoles(roles) : roles).map((role) => roleSkillSpec(role, surface));
+    return [...baseSkillSpecs(surface), ...roleSpecs];
+}
+export async function installCodexSkills(roles, customDir, surface = "compact") {
     const skillsDir = resolveCodexSkillsDir(customDir);
     await mkdir(skillsDir, { recursive: true });
+    const specs = buildCodexSkillSpecs(roles, surface);
+    const selectedNames = new Set(specs.map((spec) => spec.name));
+    for (const spec of allCodexSkillSpecs(roles)) {
+        if (!selectedNames.has(spec.name)) {
+            await rm(join(skillsDir, spec.name), { recursive: true, force: true });
+        }
+    }
     const installed = [];
-    for (const spec of buildCodexSkillSpecs(roles)) {
+    for (const spec of specs) {
         const skillDir = join(skillsDir, spec.name);
         await mkdir(skillDir, { recursive: true });
         const path = join(skillDir, "SKILL.md");
@@ -334,7 +367,7 @@ export async function installCodexSkills(roles, customDir) {
 export async function removeCodexSkills(roles, customDir) {
     const skillsDir = resolveCodexSkillsDir(customDir);
     const removed = [];
-    for (const spec of buildCodexSkillSpecs(roles)) {
+    for (const spec of allCodexSkillSpecs(roles)) {
         const skillDir = join(skillsDir, spec.name);
         if (existsSync(skillDir)) {
             await rm(skillDir, { recursive: true, force: true });
@@ -343,13 +376,13 @@ export async function removeCodexSkills(roles, customDir) {
     }
     return removed;
 }
-export async function listInstalledCodexSkills(roles, customDir) {
+export async function listInstalledCodexSkills(roles, customDir, surface = "compact") {
     const skillsDir = resolveCodexSkillsDir(customDir);
     if (!existsSync(skillsDir)) {
         return [];
     }
     const entries = new Set(await readdir(skillsDir));
-    return buildCodexSkillSpecs(roles)
+    return buildCodexSkillSpecs(roles, surface)
         .filter((spec) => entries.has(spec.name) && existsSync(join(skillsDir, spec.name, "SKILL.md")))
         .map((spec) => ({
         name: spec.name,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@longtable/provider-codex",
-  "version": "0.1.38",
+  "version": "0.1.40",
   "private": false,
   "description": "Codex adapter surface for LongTable",
   "type": "module",
@@ -15,9 +15,9 @@
     "typecheck": "tsc -p tsconfig.json --noEmit"
   },
   "dependencies": {
-    "@longtable/checkpoints": "0.1.38",
-    "@longtable/core": "0.1.38",
-    "@longtable/setup": "0.1.38"
+    "@longtable/checkpoints": "0.1.40",
+    "@longtable/core": "0.1.40",
+    "@longtable/setup": "0.1.40"
   },
   "devDependencies": {
     "@types/node": "^22.10.1",