fix(skills): update config resolution in SKILL.md files migrated from workflow.md

The 5 skills whose workflow.md was absorbed into SKILL.md by PR #2308
still had the old config.yaml loading instruction. Updated them to use
resolve_config.py like all other skills.

src/bmm-skills/1-analysis/bmad-prfaq/bmad-manifest.json

@@ -7,8 +7,8 @@
       "description": "Produces battle-tested PRFAQ document and optional LLM distillate for PRD input.",
       "supports-headless": true,
       "phase-name": "1-analysis",
-      "preceded-by": ["brainstorming", "perform-research"],
-      "followed-by": ["create-prd"],
+      "after": ["brainstorming", "perform-research"],
+      "before": ["create-prd"],
       "is-required": false,
       "output-location": "{planning_artifacts}"
     }

src/bmm-skills/1-analysis/bmad-product-brief/bmad-manifest.json

@@ -8,8 +8,8 @@
       "description": "Produces executive product brief and optional LLM distillate for PRD input.",
       "supports-headless": true,
       "phase-name": "1-analysis",
-      "preceded-by": ["brainstorming", "perform-research"],
-      "followed-by": ["create-prd"],
+      "after": ["brainstorming", "perform-research"],
+      "before": ["create-prd"],
       "is-required": true,
       "output-location": "{planning_artifacts}"
     }

src/bmm-skills/module-help.csv

@@ -1,4 +1,4 @@
-module,skill,display-name,menu-code,description,action,args,phase,preceded-by,followed-by,required,output-location,outputs
+module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs
 BMad Method,_meta,,,,,,,,,false,https://docs.bmad-method.org/llms.txt,
 BMad Method,bmad-document-project,Document Project,DP,Analyze an existing project to produce useful documentation.,,,anytime,,,false,project-knowledge,*
 BMad Method,bmad-generate-project-context,Generate Project Context,GPC,Scan existing codebase to generate a lean LLM-optimized project-context.md. Essential for brownfield projects.,,,anytime,,,false,output_folder,project context

src/core-skills/bmad-distillator/resources/distillate-format-reference.md

@@ -139,7 +139,7 @@ parts: 1
 
 ## Solution Architecture
 - Plugins: skill bundles with Anthropic plugin standard as base format + bmad-manifest.json extending for BMAD-specific metadata (installer options, capabilities, help integration, phase ordering, dependencies)
-- Existing manifest example: `{"module-code":"bmm","replaces-skill":"bmad-create-product-brief","capabilities":[{"name":"create-brief","menu-code":"CB","supports-headless":true,"phase-name":"1-analysis","preceded-by":["brainstorming"],"followed-by":["create-prd"],"is-required":true}]}`
+- Existing manifest example: `{"module-code":"bmm","replaces-skill":"bmad-create-product-brief","capabilities":[{"name":"create-brief","menu-code":"CB","supports-headless":true,"phase-name":"1-analysis","after":["brainstorming"],"before":["create-prd"],"is-required":true}]}`
 - Vercel skills CLI handles platform translation; integration pattern (wrap/fork/call) is PRD decision
 - bmad-setup: global skill scanning installed bmad-manifest.json files, registering capabilities, configuring project settings; always included as base skill in every bundle (solves bootstrapping)
 - bmad-update: plugin update path without full reinstall; technical approach (diff/replace/preserve customizations) is PRD decision

src/core-skills/bmad-help/SKILL.md

@@ -33,16 +33,16 @@ When this skill completes, the user should:
 The catalog uses this format:
 
 ```
-module,skill,display-name,menu-code,description,action,args,phase,preceded-by,followed-by,required,output-location,outputs
+module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs
 ```
 
 **Phases** determine the high-level flow:
 - `anytime` — available regardless of workflow state
 - Numbered phases (`1-analysis`, `2-planning`, etc.) flow in order; naming varies by module
 
-**Sequencing** determines recommended ordering within and across phases (these are soft suggestions, not hard gates — see `required` for gating):
-- `preceded-by` — skills that should ideally complete before this one
-- `followed-by` — skills that should ideally run after this one
+**Dependencies** determine ordering within and across phases:
+- `after` — skills that should ideally complete before this one
+- `before` — skills that should run after this one
 - Format: `skill-name` for single-action skills, `skill-name:action` for multi-action skills
 
 **Required gates**:

src/core-skills/module-help.csv

@@ -1,4 +1,4 @@
-module,skill,display-name,menu-code,description,action,args,phase,preceded-by,followed-by,required,output-location,outputs
+module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs
 Core,_meta,,,,,,,,,false,https://docs.bmad-method.org/llms.txt,
 Core,bmad-brainstorming,Brainstorming,BSP,Use early in ideation or when stuck generating ideas.,,,anytime,,,false,{output_folder}/brainstorming,brainstorming session
 Core,bmad-party-mode,Party Mode,PM,Orchestrate multi-agent discussions when you need multiple perspectives or want agents to collaborate.,,,anytime,,,false,,

test/test-installation-components.js

@@ -285,10 +285,6 @@ async function runTests() {
     const opencodeInstaller = platformCodes.platforms.opencode?.installer;
 
     assert(opencodeInstaller?.target_dir === '.agents/skills', 'OpenCode target_dir uses native skills path');
-    assert(
-      opencodeInstaller?.commands_target_dir === '.opencode/commands',
-      'OpenCode commands_target_dir is configured for /<skill> slash commands',
-    );
 
     const tempProjectDir = await fs.mkdtemp(path.join(os.tmpdir(), 'bmad-opencode-test-'));
     const installedBmadDir = await createTestBmadFixture();
@@ -305,55 +301,6 @@ async function runTests() {
     const skillFile = path.join(tempProjectDir, '.agents', 'skills', 'bmad-master', 'SKILL.md');
     assert(await fs.pathExists(skillFile), 'OpenCode install writes SKILL.md directory output');
 
-    // Command pointer assertions: a /<canonicalId> slash command should exist
-    // for each installed skill so users can invoke skills directly without
-    // going through the /skills menu.
-    const commandFile = path.join(tempProjectDir, '.opencode', 'commands', 'bmad-master.md');
-    assert(await fs.pathExists(commandFile), 'OpenCode install writes per-skill command pointer file');
-
-    const commandContent = await fs.readFile(commandFile, 'utf8');
-    assert(commandContent.includes('@skills/bmad-master'), 'Command pointer body references the skill via @skills/<canonicalId>');
-    assert(commandContent.includes('description:'), 'Command pointer carries a description in YAML frontmatter');
-
-    // Idempotency: re-running install must not duplicate or rewrite pointers.
-    const result2 = await ideManager.setup('opencode', tempProjectDir, installedBmadDir, {
-      silent: true,
-      selectedModules: ['bmm'],
-    });
-    assert(result2.success === true, 'Second OpenCode install succeeds (idempotent)');
-    assert(await fs.pathExists(commandFile), 'Command pointer survives a second install pass');
-
-    // Description-update propagation: when the manifest description changes
-    // and the on-disk pointer still matches the generator pattern, refresh
-    // the file so users see the updated description.
-    const csvPath = path.join(installedBmadDir, '_config', 'skill-manifest.csv');
-    const updatedCsv =
-      'canonicalId,name,description,module,path\n' +
-      '"bmad-master","bmad-master","UPDATED description for the test agent","core","_bmad/core/bmad-master/SKILL.md"\n';
-    await fs.writeFile(csvPath, updatedCsv);
-    const result3 = await ideManager.setup('opencode', tempProjectDir, installedBmadDir, {
-      silent: true,
-      selectedModules: ['bmm'],
-    });
-    assert(result3.success === true, 'Third OpenCode install succeeds after description update');
-    const refreshed = await fs.readFile(commandFile, 'utf8');
-    assert(refreshed.includes('UPDATED description'), 'Generator-shaped pointer is refreshed when manifest description changes');
-
-    // Hand-edit preservation across the production install flow. The
-    // installer passes previousSkillIds — without the cleanup-side spare,
-    // hand edits would be wiped here.
-    const SENTINEL = 'HAND_EDITED_BY_USER_SHOULD_SURVIVE';
-    const handEditedBody = `---\ndescription: my custom description\n---\n\n${SENTINEL}\n`;
-    await fs.writeFile(commandFile, handEditedBody);
-    const result4 = await ideManager.setup('opencode', tempProjectDir, installedBmadDir, {
-      silent: true,
-      selectedModules: ['bmm'],
-      previousSkillIds: new Set(['bmad-master']),
-    });
-    assert(result4.success === true, 'Fourth OpenCode install succeeds with hand-edited pointer present');
-    const afterReinstall = await fs.readFile(commandFile, 'utf8');
-    assert(afterReinstall.includes(SENTINEL), 'Hand-edited pointer survives a routine reinstall (cleanup spares active-manifest IDs)');
-
     await fs.remove(tempProjectDir);
     await fs.remove(path.dirname(installedBmadDir));
   } catch (error) {
@@ -557,83 +504,10 @@ async function runTests() {
     const copilotInstaller = platformCodes17.platforms['github-copilot']?.installer;
 
     assert(copilotInstaller?.target_dir === '.agents/skills', 'GitHub Copilot target_dir uses native skills path');
-    assert(
-      copilotInstaller?.commands_target_dir === '.github/agents',
-      'GitHub Copilot commands_target_dir is configured for the Custom Agents picker',
-    );
-    assert(copilotInstaller?.commands_extension === '.agent.md', 'GitHub Copilot uses .agent.md extension for Custom Agents files');
-    assert(
-      typeof copilotInstaller?.commands_body_template === 'string' && copilotInstaller.commands_body_template.includes('{canonicalId}'),
-      'GitHub Copilot defines a commands_body_template with {canonicalId} placeholder',
-    );
-    assert(
-      copilotInstaller?.commands_filter === 'agents-only',
-      'GitHub Copilot filters Custom Agents picker to persona agents only (agents-only)',
-    );
 
     const tempProjectDir17 = await fs.mkdtemp(path.join(os.tmpdir(), 'bmad-copilot-test-'));
     const installedBmadDir17 = await createTestBmadFixture();
 
-    // Extend the fixture to exercise the agents-only filter, which detects
-    // persona agents by the `[agent]` section in each skill's source
-    // customize.toml. Five skill types covered:
-    //
-    //   1. Persona agent — has customize.toml with [agent]      → INCLUDED
-    //   2. Persona with non-conventional id — also has [agent]   → INCLUDED
-    //      (verifies the filter doesn't depend on `-agent-` naming)
-    //   3. Meta-skill whose id contains `-agent-` but isn't a
-    //      persona — has customize.toml with [workflow]          → EXCLUDED
-    //      (mirrors `bmad-agent-builder` in the real manifest)
-    //   4. Workflow skill — no customize.toml at all             → EXCLUDED
-    //   5. `bmad-help` — meta-help skill with no customize.toml;
-    //      every persona agent's activation already advertises it,
-    //      so it's correctly excluded from the picker as redundant    → EXCLUDED
-    const fixtureCsvPath17 = path.join(installedBmadDir17, '_config', 'skill-manifest.csv');
-    await fs.writeFile(
-      fixtureCsvPath17,
-      [
-        'canonicalId,name,description,module,path',
-        '"bmad-master","bmad-master","Workflow with no customize.toml — should NOT appear in Copilot agents picker","core","_bmad/core/bmad-master/SKILL.md"',
-        '"bmad-agent-fixture","bmad-agent-fixture","Persona agent — customize.toml has [agent], SHOULD appear","core","_bmad/core/bmad-agent-fixture/SKILL.md"',
-        '"bmad-tea","bmad-tea","Non-conventional id but [agent] in customize.toml — SHOULD appear","core","_bmad/core/bmad-tea/SKILL.md"',
-        '"bmad-agent-builder","bmad-agent-builder","Skill-builder workflow — id contains -agent- but customize.toml has [workflow] — should NOT appear","core","_bmad/core/bmad-agent-builder/SKILL.md"',
-        '"bmad-help","bmad-help","Meta-help skill — no customize.toml; SHOULD NOT appear in agents picker (toml-driven filter)","core","_bmad/core/bmad-help/SKILL.md"',
-        '',
-      ].join('\n'),
-    );
-
-    // Materialise the source skill directories so the agents-only filter
-    // can read their customize.toml. The bmad-master and bmad-agent-builder
-    // SKILL.md files were already populated by createTestBmadFixture (they
-    // share the bmad-master target_dir layout); only the customize.toml
-    // and the new agent fixtures need to be created here.
-    for (const id of ['bmad-agent-fixture', 'bmad-tea', 'bmad-agent-builder', 'bmad-help']) {
-      const dir17 = path.join(installedBmadDir17, 'core', id);
-      await fs.ensureDir(dir17);
-      await fs.writeFile(
-        path.join(dir17, 'SKILL.md'),
-        ['---', `name: ${id}`, `description: fixture for ${id}`, '---', '', `Body of ${id}.`].join('\n'),
-      );
-    }
-    // Note: bmad-help intentionally has NO customize.toml — it exercises
-    // the toml-driven filter's exclusion path (a skill with no
-    // customize.toml is correctly kept out of the Copilot agents picker).
-    // [agent] customize.toml for the two persona fixtures.
-    await fs.writeFile(
-      path.join(installedBmadDir17, 'core', 'bmad-agent-fixture', 'customize.toml'),
-      ['[agent]', 'name = "Fixture Agent"', 'title = "Test Persona"', ''].join('\n'),
-    );
-    await fs.writeFile(
-      path.join(installedBmadDir17, 'core', 'bmad-tea', 'customize.toml'),
-      ['[agent]', 'name = "Murat"', 'title = "Test Architect"', ''].join('\n'),
-    );
-    // [workflow] customize.toml for the meta-skill — its id contains `-agent-`
-    // but it is NOT a persona (mirrors bmad-agent-builder in production).
-    await fs.writeFile(
-      path.join(installedBmadDir17, 'core', 'bmad-agent-builder', 'customize.toml'),
-      ['[workflow]', '', '# Meta-skill that builds agents but is not itself a persona.', ''].join('\n'),
-    );
-
     const copilotInstructionsPath17 = path.join(tempProjectDir17, '.github', 'copilot-instructions.md');
     await fs.ensureDir(path.dirname(copilotInstructionsPath17));
     await fs.writeFile(
@@ -669,56 +543,6 @@ async function runTests() {
       'GitHub Copilot setup preserves user content in copilot-instructions.md',
     );
 
-    // Custom Agents picker integration: persona agents (those with [agent]
-    // in their source customize.toml) get .agent.md files in
-    // .github/agents/. Workflows and meta-skills with [workflow] (or no
-    // customize.toml at all) do NOT — the agents-only filter keeps the
-    // picker uncluttered and the signal is naming-independent.
-    const agentsDir17 = path.join(tempProjectDir17, '.github', 'agents');
-    const agentFileForPersona17 = path.join(agentsDir17, 'bmad-agent-fixture.agent.md');
-    const agentFileForTea17 = path.join(agentsDir17, 'bmad-tea.agent.md');
-    const agentFileForWorkflow17 = path.join(agentsDir17, 'bmad-master.agent.md');
-    const agentFileForMetaSkill17 = path.join(agentsDir17, 'bmad-agent-builder.agent.md');
-    const agentFileForBmadHelp17 = path.join(agentsDir17, 'bmad-help.agent.md');
-
-    assert(
-      await fs.pathExists(agentFileForPersona17),
-      'Persona agent ([agent] in customize.toml) gets a .agent.md file in .github/agents/',
-    );
-    assert(await fs.pathExists(agentFileForTea17), 'Non-conventional id with [agent] in customize.toml is included (no allowlist needed)');
-    assert(!(await fs.pathExists(agentFileForWorkflow17)), 'Workflow skill (no customize.toml) is FILTERED OUT of .github/agents/');
-    assert(
-      !(await fs.pathExists(agentFileForBmadHelp17)),
-      'bmad-help is excluded from Copilot agents picker (no customize.toml; allowlist removed per maintainer feedback)',
-    );
-    assert(
-      !(await fs.pathExists(agentFileForMetaSkill17)),
-      'Meta-skill with -agent- in id but [workflow] in customize.toml is FILTERED OUT (signal is behavior, not naming)',
-    );
-
-    // Body content of the persona agent file: frontmatter description +
-    // LOAD pattern referencing the skill's SKILL.md path under target_dir.
-    const personaAgentContent17 = await fs.readFile(agentFileForPersona17, 'utf8');
-    assert(
-      personaAgentContent17.includes('description:'),
-      'Copilot agent pointer carries a description in YAML frontmatter (drives the agents picker label)',
-    );
-    assert(
-      personaAgentContent17.includes('{project-root}/.agents/skills/bmad-agent-fixture/SKILL.md'),
-      'Copilot agent pointer body resolves to the skill via LOAD {project-root}/<target_dir>/<id>/SKILL.md',
-    );
-
-    // Idempotency: re-running setup must not duplicate or rewrite the agent
-    // pointer when the source manifest is unchanged, AND must not start
-    // emitting workflow-skill agent files.
-    const result17b = await ideManager17.setup('github-copilot', tempProjectDir17, installedBmadDir17, {
-      silent: true,
-      selectedModules: ['bmm'],
-    });
-    assert(result17b.success === true, 'Second GitHub Copilot install succeeds (idempotent)');
-    assert(await fs.pathExists(agentFileForPersona17), 'Persona agent pointer survives a second install pass');
-    assert(!(await fs.pathExists(agentFileForWorkflow17)), 'Workflow skill remains filtered out of agents picker on second install');
-
     await fs.remove(tempProjectDir17);
     await fs.remove(path.dirname(installedBmadDir17));
   } catch (error) {
@@ -2913,113 +2737,6 @@ async function runTests() {
 
   console.log('');
 
-  // ============================================================
-  // Test Suite 40c: OpenCode command pointers in multi-IDE batches
-  // ============================================================
-  // Regression: when OpenCode is the *peer* in a setupBatch sharing
-  // .agents/skills (e.g. with openhands), the skill write is dedup-skipped
-  // but the per-IDE .opencode/commands/ pointers must still be generated.
-  // Symmetrically, partial uninstall while a peer remains must still clean
-  // up OpenCode's own command pointers.
-  console.log(`${colors.yellow}Test Suite 40c: OpenCode command pointers in shared-target batches${colors.reset}\n`);
-
-  try {
-    clearCache();
-    const platformCodes40c = await loadPlatformCodes();
-    const opencodeTarget40c = platformCodes40c.platforms.opencode?.installer?.target_dir;
-    const openhandsTarget40c = platformCodes40c.platforms.openhands?.installer?.target_dir;
-    assert(
-      opencodeTarget40c === '.agents/skills' && openhandsTarget40c === '.agents/skills',
-      'OpenCode and OpenHands share .agents/skills target_dir',
-    );
-
-    // Order A: opencode first → opencode is the writer.
-    const projA = await fs.mkdtemp(path.join(os.tmpdir(), 'bmad-opencode-batch-a-'));
-    const bmadA = await createTestBmadFixture();
-    const mgrA = new IdeManager();
-    await mgrA.ensureInitialized();
-    const resultsA = await mgrA.setupBatch(['opencode', 'openhands'], projA, bmadA, {
-      silent: true,
-      selectedModules: ['core'],
-    });
-    const cmdA = path.join(projA, '.opencode', 'commands', 'bmad-master.md');
-    assert(
-      resultsA.every((r) => r.success === true),
-      'opencode-first batch: all platforms succeed',
-    );
-    assert(await fs.pathExists(cmdA), 'opencode-first batch: command pointer is created');
-
-    // Order B: openhands first → opencode is the peer (skipTarget=true).
-    // Without the fix, the early-return would bypass installCommandPointers.
-    const projB = await fs.mkdtemp(path.join(os.tmpdir(), 'bmad-opencode-batch-b-'));
-    const bmadB = await createTestBmadFixture();
-    const mgrB = new IdeManager();
-    await mgrB.ensureInitialized();
-    const resultsB = await mgrB.setupBatch(['openhands', 'opencode'], projB, bmadB, {
-      silent: true,
-      selectedModules: ['core'],
-    });
-    const cmdB = path.join(projB, '.opencode', 'commands', 'bmad-master.md');
-    const opencodeResultB = resultsB.find((r) => r.ide === 'opencode');
-    assert(
-      resultsB.every((r) => r.success === true),
-      'openhands-first batch: all platforms succeed',
-    );
-    assert(
-      opencodeResultB?.handlerResult?.results?.sharedTargetHandledByPeer === true,
-      'openhands-first batch: opencode is marked sharedTargetHandledByPeer (skill write deduped)',
-    );
-    assert(await fs.pathExists(cmdB), 'openhands-first batch: command pointer is generated even when skill write is deduped');
-
-    // Cleanup symmetry: uninstall opencode while openhands remains.
-    // Uses an in-project bmadDir so the cleanup path can compute removalSet
-    // from the manifest (the production layout). The cross-temp-dir fixture
-    // above can't exercise this — same constraint Test Suite 40 documents.
-    const projC = await fs.mkdtemp(path.join(os.tmpdir(), 'bmad-opencode-batch-c-'));
-    const bmadC = path.join(projC, '_bmad');
-    await fs.ensureDir(path.join(bmadC, '_config'));
-    await fs.writeFile(
-      path.join(bmadC, '_config', 'skill-manifest.csv'),
-      'canonicalId,name,description,module,path\n' +
-        '"bmad-master","bmad-master","Minimal test agent fixture","core","_bmad/core/bmad-master/SKILL.md"\n',
-    );
-    const skillC = path.join(bmadC, 'core', 'bmad-master');
-    await fs.ensureDir(skillC);
-    await fs.writeFile(
-      path.join(skillC, 'SKILL.md'),
-      ['---', 'name: bmad-master', 'description: Minimal test agent fixture', '---', '', 'You are a test agent.'].join('\n'),
-    );
-
-    const mgrC = new IdeManager();
-    await mgrC.ensureInitialized();
-    await mgrC.setupBatch(['openhands', 'opencode'], projC, bmadC, {
-      silent: true,
-      selectedModules: ['core'],
-    });
-    const cmdC = path.join(projC, '.opencode', 'commands', 'bmad-master.md');
-    assert(await fs.pathExists(cmdC), 'in-project fixture: pointer is generated for opencode peer');
-
-    const cleanupResultsC = await mgrC.cleanupByList(projC, ['opencode'], {
-      silent: true,
-      remainingIdes: ['openhands'],
-    });
-    assert(cleanupResultsC[0].success !== false, 'opencode partial-uninstall reports success');
-    const sharedSurvivesC = await fs.pathExists(path.join(projC, '.agents', 'skills', 'bmad-master', 'SKILL.md'));
-    assert(sharedSurvivesC, 'shared .agents/skills/ survives partial uninstall (peer still uses it)');
-    assert(!(await fs.pathExists(cmdC)), 'opencode command pointer is removed on partial uninstall even when peer remains');
-
-    await fs.remove(projA).catch(() => {});
-    await fs.remove(path.dirname(bmadA)).catch(() => {});
-    await fs.remove(projB).catch(() => {});
-    await fs.remove(path.dirname(bmadB)).catch(() => {});
-    await fs.remove(projC).catch(() => {});
-  } catch (error) {
-    console.log(`${colors.red}Test Suite 40c setup failed: ${error.message}${colors.reset}`);
-    failed++;
-  }
-
-  console.log('');
-
   // ============================================================
   // Test Suite 41: Custom-module skill ownership (non-bmad prefix)
   // ============================================================

tools/installer/core/installer.js

@@ -12,7 +12,6 @@ const { BMAD_FOLDER_NAME } = require('../ide/shared/path-utils');
 const { InstallPaths } = require('./install-paths');
 const { ExternalModuleManager } = require('../modules/external-manager');
 const { resolveModuleVersion } = require('../modules/version-resolver');
-const { MODULE_HELP_CSV_HEADER } = require('../modules/module-help-schema');
 
 const { ExistingInstall } = require('./existing-install');
 const { warnPreNativeSkillsLegacy } = require('./legacy-warnings');
@@ -943,7 +942,7 @@ class Installer {
    */
   async mergeModuleHelpCatalogs(bmadDir, _agentEntries = []) {
     const allRows = [];
-    const headerRow = MODULE_HELP_CSV_HEADER;
+    const headerRow = 'module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs';
     const COLUMN_COUNT = 13;
     const PHASE_INDEX = 7;
 
@@ -976,19 +975,9 @@ class Installer {
           const content = await fs.readFile(helpFilePath, 'utf8');
           const lines = content.split('\n').filter((line) => line.trim() && !line.startsWith('#'));
 
-          let headerWarned = false;
           for (const line of lines) {
-            // Header row: warn on drift from canonical schema, then skip.
-            // Data rows are loaded positionally regardless, so the warning
-            // is advisory — the maintainer should rename their columns.
+            // Skip header row
             if (line.startsWith('module,')) {
-              if (!headerWarned && line.trim() !== headerRow) {
-                await prompts.log.warn(
-                  `  ${moduleName}/module-help.csv header does not match canonical schema. ` +
-                    `Expected: ${headerRow} | Found: ${line.trim()} | Data loaded positionally.`,
-                );
-                headerWarned = true;
-              }
               continue;
             }
 

tools/installer/ide/_config-driven.js

@@ -6,125 +6,6 @@ const csv = require('csv-parse/sync');
 const { BMAD_FOLDER_NAME } = require('./shared/path-utils');
 const { getInstalledCanonicalIds, isBmadOwnedEntry } = require('./shared/installed-skills');
 
-// Reserved OpenCode slash commands. A skill whose canonicalId collides with
-// one of these is skipped during command-pointer generation so it doesn't
-// shadow a built-in.
-const RESERVED_OPENCODE_COMMANDS = new Set([
-  'review',
-  'commit',
-  'init',
-  'help',
-  'skills',
-  'fast',
-  'compact',
-  'clear',
-  'undo',
-  'redo',
-  'edit',
-  'editor',
-  'exit',
-  'quit',
-  'theme',
-  'config',
-  'model',
-  'session',
-]);
-
-// Wrap a description for safe insertion into single-line YAML frontmatter.
-// Leaves plain values untouched; double-quotes (and escapes) anything that
-// could break YAML parsing or span multiple lines.
-function yamlSafeSingleLine(value) {
-  const collapsed = String(value)
-    .replaceAll(/[\r\n]+/g, ' ')
-    .trim();
-  const needsQuoting = /[:#'"\\]/.test(collapsed) || /^[!&*?|>%@`[{]/.test(collapsed);
-  if (!needsQuoting) return collapsed;
-  const escaped = collapsed.replaceAll('\\', '\\\\').replaceAll('"', String.raw`\"`);
-  return `"${escaped}"`;
-}
-
-// Validate that a canonicalId is a safe basename — no path separators, no
-// parent-dir traversal, no leading dots, only the character set we expect.
-// Defense-in-depth: the manifest is trusted today, but the value flows
-// directly into a file path and a malformed entry should not write outside
-// the commands directory.
-function isSafeCanonicalId(value) {
-  return typeof value === 'string' && /^[a-zA-Z0-9][a-zA-Z0-9_.-]*$/.test(value) && !value.includes('..');
-}
-
-// Default body template for command pointer files. Used when a platform's
-// installer config doesn't override `commands_body_template`. Matches
-// OpenCode's native `@skills/<id>` skill-reference syntax.
-const DEFAULT_COMMANDS_BODY_TEMPLATE = '@skills/{canonicalId}';
-
-// Is this skill a persona agent (vs. a workflow/tool/standalone skill)?
-// Used by platforms that surface only persona agents (e.g. Copilot's Custom
-// Agents picker). Signal: the skill's source `customize.toml` has an
-// `[agent]` section. This is the actual configuration source of truth —
-// every BMAD persona is configured via [agent] in its customize.toml,
-// every workflow uses [workflow], every standalone skill has no
-// customize.toml at all. Verified against the full installed manifest:
-// catches exactly the 20 description-confirmed personas across BMM, CIS,
-// GDS, WDS, TEA, and correctly excludes meta-skills like
-// `bmad-agent-builder` (a skill-builder workflow whose canonical id
-// contains `-agent-` but which has no [agent] section because it isn't a
-// persona itself).
-//
-// Reading the source toml — at install time the source skill directory
-// (resolved from manifest record.path) still exists; cleanup runs later
-// in the install flow.
-async function isAgentSkill(record, bmadDir) {
-  if (!record?.path || !bmadDir) return false;
-  const bmadFolderName = path.basename(bmadDir);
-  const bmadPrefix = bmadFolderName + '/';
-  const relativePath = record.path.startsWith(bmadPrefix) ? record.path.slice(bmadPrefix.length) : record.path;
-  const tomlPath = path.join(bmadDir, path.dirname(relativePath), 'customize.toml');
-  if (!(await fs.pathExists(tomlPath))) return false;
-  try {
-    const content = await fs.readFile(tomlPath, 'utf8');
-    return /^\[agent\]/m.test(content);
-  } catch {
-    return false;
-  }
-}
-
-// Resolve placeholders in a body template. Supported placeholders:
-//   {canonicalId}   — the skill's canonical id
-//   {target_dir}    — the platform's skill install directory (e.g. .agents/skills)
-//   {project-root}  — left as a literal placeholder for the model/tool to expand
-//                     at runtime; consistent with PR #1769's templates.
-function expandBodyTemplate(template, { canonicalId, targetDir }) {
-  return template.replaceAll('{canonicalId}', canonicalId).replaceAll('{target_dir}', targetDir);
-}
-
-// The exact body the installer would generate for a given description and
-// canonicalId, given the platform's body template. Centralised so both the
-// write and the freshness-check paths agree on the canonical form.
-function buildCommandPointerBody(description, canonicalId, { template, targetDir }) {
-  const bodyText = expandBodyTemplate(template, { canonicalId, targetDir });
-  return `---\ndescription: ${yamlSafeSingleLine(description)}\n---\n\n${bodyText}\n`;
-}
-
-// Heuristic: does an existing pointer file look like our generator's output
-// (and therefore safe to refresh) versus a user-modified file (which we
-// preserve)? We check the body shape rather than full equality so that
-// description-only edits in the manifest can propagate without trampling
-// hand edits to the body.
-function looksLikeGeneratorOutput(content, canonicalId, { template, targetDir }) {
-  if (typeof content !== 'string') return false;
-  const trimmed = content.trim();
-  const expectedTail = expandBodyTemplate(template, { canonicalId, targetDir }).trim();
-  // Must end with the exact body our generator writes (post-expansion).
-  if (!trimmed.endsWith(expectedTail)) return false;
-  // Must start with frontmatter containing exactly one description: line.
-  const fmMatch = trimmed.match(/^---\n([\S\s]*?)\n---\n/);
-  if (!fmMatch) return false;
-  const fmLines = fmMatch[1].split('\n').filter((l) => l.length > 0);
-  if (fmLines.length !== 1) return false;
-  if (!fmLines[0].startsWith('description:')) return false;
-  return true;
-}
-
 /**
  * Config-driven IDE setup handler
  *
@@ -216,15 +97,9 @@ class ConfigDrivenIdeSetup {
     }
 
     // When a peer platform in the same install batch owns this target_dir,
-    // skip the skill write — the peer has already populated it. Command
-    // pointers, however, write to a separate per-IDE directory and must
-    // still be generated for this IDE; they are not deduped across peers.
+    // skip the skill write — the peer has already populated it.
     if (options.skipTarget) {
-      const results = { skills: 0, sharedTargetHandledByPeer: true };
-      if (this.installerConfig.commands_target_dir) {
-        results.commands = await this.installCommandPointers(projectDir, bmadDir, this.installerConfig, options);
-      }
-      return { success: true, results };
+      return { success: true, results: { skills: 0, sharedTargetHandledByPeer: true } };
     }
 
     if (this.installerConfig.target_dir) {
@@ -253,157 +128,11 @@ class ConfigDrivenIdeSetup {
     results.skills = await this.installVerbatimSkills(projectDir, bmadDir, targetPath, config);
     results.skillDirectories = this.skillWriteTracker.size;
 
-    if (config.commands_target_dir) {
-      results.commands = await this.installCommandPointers(projectDir, bmadDir, config, options);
-    }
-
     await this.printSummary(results, target_dir, options);
     this.skillWriteTracker = null;
     return { success: true, results };
   }
 
-  /**
-   * Generate per-skill command pointer files for IDEs that surface commands
-   * separately from skills (e.g. OpenCode's `.opencode/commands/<name>.md`).
-   *
-   * Each pointer is a tiny markdown file whose body is `@skills/<canonicalId>`
-   * so invoking `/<canonicalId>` routes the user straight to the skill instead
-   * of forcing them through a `/skills` menu.
-   *
-   * Skips:
-   *  - Names that collide with reserved built-in slash commands.
-   *  - canonicalIds that aren't safe basename-only identifiers (defense
-   *    against path traversal even though the manifest is currently trusted).
-   *  - Existing files whose body looks user-modified (preserves hand edits);
-   *    pointer files matching the generator pattern get overwritten so that
-   *    description changes in skill-manifest.csv propagate on re-install.
-   *
-   * Per-file write failures are recorded and reported but do not abort the
-   * rest of the install — pointer files are a non-essential adjunct to the
-   * skill copy that already succeeded.
-   *
-   * @param {string} projectDir
-   * @param {string} bmadDir
-   * @param {Object} config - Installer config; reads commands_target_dir.
-   * @param {Object} options - Setup options. forceCommands overwrites existing
-   *   files unconditionally (including hand-modified ones).
-   * @returns {Promise<Object>} { created, updated, skippedExisting, skippedCollision, skippedInvalidId, writeFailures, fallbackDescription }
-   */
-  async installCommandPointers(projectDir, bmadDir, config, options = {}) {
-    const result = {
-      created: 0,
-      updated: 0,
-      skippedExisting: 0,
-      skippedCollision: 0,
-      skippedInvalidId: 0,
-      skippedFiltered: 0,
-      writeFailures: 0,
-      fallbackDescription: 0,
-    };
-
-    const csvPath = path.join(bmadDir, '_config', 'skill-manifest.csv');
-    if (!(await fs.pathExists(csvPath))) return result;
-
-    const commandsPath = path.join(projectDir, config.commands_target_dir);
-    await fs.ensureDir(commandsPath);
-
-    // Per-platform pointer-file shape, all overrideable in platform-codes.yaml.
-    const extension = config.commands_extension || '.md';
-    const template = config.commands_body_template || DEFAULT_COMMANDS_BODY_TEMPLATE;
-    const targetDir = config.target_dir;
-    const filter = config.commands_filter || null;
-
-    const csvContent = await fs.readFile(csvPath, 'utf8');
-    const records = csv.parse(csvContent, { columns: true, skip_empty_lines: true });
-
-    for (const record of records) {
-      const canonicalId = record.canonicalId;
-      if (!canonicalId) continue;
-
-      // Defensive basename validation. canonicalId comes from a trusted
-      // manifest today, but the value flows directly into a file path —
-      // reject anything that could escape commands_target_dir.
-      if (!isSafeCanonicalId(canonicalId)) {
-        result.skippedInvalidId++;
-        continue;
-      }
-
-      // Optional per-platform filter: surfaces that should only show
-      // persona agents (e.g. Copilot's Custom Agents picker) skip
-      // workflow/tool skills here so the picker isn't cluttered with
-      // 90+ unrelated entries.
-      if (filter === 'agents-only' && !(await isAgentSkill(record, bmadDir))) {
-        result.skippedFiltered++;
-        continue;
-      }
-
-      // Reserved-name guard is OpenCode-specific. Other adapters that opt
-      // into commands_target_dir later should declare their own reserved
-      // set rather than inheriting OpenCode's.
-      if (this.name === 'opencode' && RESERVED_OPENCODE_COMMANDS.has(canonicalId)) {
-        result.skippedCollision++;
-        continue;
-      }
-
-      let description = (record.description || '').trim();
-      if (!description) {
-        description = `Run the ${canonicalId} skill`;
-        result.fallbackDescription++;
-      }
-
-      const body = buildCommandPointerBody(description, canonicalId, { template, targetDir });
-      const commandFile = path.join(commandsPath, `${canonicalId}${extension}`);
-
-      // If a pointer file already exists, decide whether to overwrite based
-      // on whether it looks like generator output (description-only diff) or
-      // a user-modified file. forceCommands overrides this protection.
-      if (!options.forceCommands && (await fs.pathExists(commandFile))) {
-        let existing;
-        try {
-          existing = await fs.readFile(commandFile, 'utf8');
-        } catch {
-          // Treat unreadable as user-owned and skip — safer than overwriting.
-          result.skippedExisting++;
-          continue;
-        }
-
-        if (existing === body) {
-          // No-op idempotent re-run.
-          result.skippedExisting++;
-          continue;
-        }
-        if (looksLikeGeneratorOutput(existing, canonicalId, { template, targetDir })) {
-          // Description (or other generated bit) has changed; refresh in place.
-          try {
-            await fs.writeFile(commandFile, body, 'utf8');
-            result.updated++;
-          } catch (error) {
-            result.writeFailures++;
-            if (!options.silent) {
-              await prompts.log.warn(`Failed to update command pointer ${canonicalId}${extension}: ${error.message}`);
-            }
-          }
-          continue;
-        }
-        // Hand-modified pointer — preserve it.
-        result.skippedExisting++;
-        continue;
-      }
-
-      try {
-        await fs.writeFile(commandFile, body, 'utf8');
-        result.created++;
-      } catch (error) {
-        result.writeFailures++;
-        if (!options.silent) {
-          await prompts.log.warn(`Failed to write command pointer ${canonicalId}${extension}: ${error.message}`);
-        }
-      }
-    }
-
-    return result;
-  }
-
   /**
    * Install verbatim native SKILL.md directories from skill-manifest.csv.
    * Copies the entire source directory as-is into the IDE skill directory.
@@ -478,18 +207,6 @@ class ConfigDrivenIdeSetup {
     if (count > 0) {
       await prompts.log.success(`${this.name} configured: ${count} skills → ${targetDir}`);
     }
-    const cmd = results.commands;
-    if (cmd && (cmd.created > 0 || cmd.updated > 0) && this.installerConfig?.commands_target_dir) {
-      const total = cmd.created + cmd.updated;
-      const detail = cmd.updated > 0 ? `${cmd.created} new, ${cmd.updated} refreshed` : `${total}`;
-      await prompts.log.success(`${this.name} commands: ${detail} → ${this.installerConfig.commands_target_dir}`);
-      if (cmd.skippedCollision > 0) {
-        await prompts.log.message(`  (${cmd.skippedCollision} skipped — name collides with reserved slash command)`);
-      }
-      if (cmd.writeFailures > 0) {
-        await prompts.log.warn(`  (${cmd.writeFailures} pointer writes failed — see warnings above)`);
-      }
-    }
   }
 
   /**
@@ -530,36 +247,6 @@ class ConfigDrivenIdeSetup {
       await this.cleanupRovoDevPrompts(projectDir, options);
     }
 
-    // Clean generated command pointer files in commands_target_dir.
-    // Mirrors target_dir cleanup so uninstalls and skill removals don't
-    // leave dangling /<canonicalId> commands pointing at missing skills.
-    // Runs regardless of skipTarget — command pointers live in a per-IDE
-    // directory and are not deduped across peers, so a peer-owned shared
-    // skills directory does not protect this IDE's command pointers from
-    // cleanup. The "currently active" set is passed so install-flow cleanup
-    // (where removalSet contains skills that will be re-added moments later)
-    // doesn't trample hand-edited pointers; install-flow cleanup will only
-    // delete pointers for skills that are not in the new manifest.
-    if (this.installerConfig?.commands_target_dir) {
-      // In the install/update flow (signal: previousSkillIds was passed),
-      // spare pointers whose canonicalId is still in the manifest so hand
-      // edits survive a routine reinstall. In the uninstall flow (no
-      // previousSkillIds — full uninstall or per-IDE removal via
-      // cleanupByList), don't spare anything; the IDE itself is going away,
-      // so its pointers should go with it.
-      const isInstallFlow = options.previousSkillIds && options.previousSkillIds.size > 0;
-      const activeSkillIds = isInstallFlow ? await this._readActiveSkillIds(resolvedBmadDir) : new Set();
-      const extension = this.installerConfig.commands_extension || '.md';
-      await this.cleanupCommandPointers(
-        projectDir,
-        this.installerConfig.commands_target_dir,
-        options,
-        removalSet,
-        activeSkillIds,
-        extension,
-      );
-    }
-
     // Skip target_dir cleanup when a peer platform owns this directory
     // (set during dedup'd install or when uninstalling one of several
     // platforms that share the same target_dir).
@@ -659,97 +346,6 @@ class ConfigDrivenIdeSetup {
     }
   }
 
-  /**
-   * Cleanup generated command pointer files for entries in removalSet.
-   * Symmetric counterpart to installCommandPointers — removes
-   * `<canonicalId><extension>` files whose canonicalId is in the set. Removes
-   * the commands directory entirely if it ends up empty.
-   * @param {string} projectDir
-   * @param {string} commandsTargetDir - Relative dir (e.g. .opencode/commands)
-   * @param {Object} options
-   * @param {Set<string>} removalSet - canonicalIds whose pointer files to remove
-   * @param {Set<string>} [activeSkillIds] - canonicalIds present in the
-   *   current manifest. Pointers for IDs in this set are spared so an
-   *   install-flow cleanup (where removalSet === previousSkillIds and the
-   *   same skills are about to be re-installed) doesn't wipe hand-edited
-   *   pointer files. Pass an empty set or omit to delete every match in
-   *   removalSet (uninstall flow).
-   * @param {string} [extension] - Pointer file extension (default '.md');
-   *   matches the platform's commands_extension config value so cleanup
-   *   correctly identifies pointer files for IDEs whose convention isn't .md
-   *   (e.g. Copilot's `.agent.md`).
-   */
-  async cleanupCommandPointers(
-    projectDir,
-    commandsTargetDir,
-    options = {},
-    removalSet = new Set(),
-    activeSkillIds = new Set(),
-    extension = '.md',
-  ) {
-    if (!removalSet || removalSet.size === 0) return;
-
-    const commandsPath = path.join(projectDir, commandsTargetDir);
-    if (!(await fs.pathExists(commandsPath))) return;
-
-    let entries;
-    try {
-      entries = await fs.readdir(commandsPath);
-    } catch {
-      return;
-    }
-
-    for (const entry of entries) {
-      if (!entry.endsWith(extension)) continue;
-      const canonicalId = entry.slice(0, -extension.length);
-      if (!removalSet.has(canonicalId)) continue;
-      // Spare pointers for skills that are still in the manifest; the
-      // install pass will refresh them in place if their content has gone
-      // stale, while preserving hand edits.
-      if (activeSkillIds.has(canonicalId)) continue;
-      try {
-        await fs.remove(path.join(commandsPath, entry));
-      } catch {
-        // Skip files we can't remove.
-      }
-    }
-
-    // Remove the commands directory if we emptied it.
-    try {
-      const remaining = await fs.readdir(commandsPath);
-      if (remaining.length === 0) {
-        await fs.remove(commandsPath);
-      }
-    } catch {
-      // Directory may already be gone.
-    }
-  }
-
-  /**
-   * Read the canonicalIds currently present in the skill-manifest.csv.
-   * Used by cleanup to distinguish "re-install of an existing skill"
-   * (preserve pointer) from "skill truly being removed" (delete pointer).
-   * @param {string|null} bmadDir
-   * @returns {Promise<Set<string>>}
-   */
-  async _readActiveSkillIds(bmadDir) {
-    const ids = new Set();
-    if (!bmadDir) return ids;
-    const csvPath = path.join(bmadDir, '_config', 'skill-manifest.csv');
-    if (!(await fs.pathExists(csvPath))) return ids;
-    try {
-      const content = await fs.readFile(csvPath, 'utf8');
-      const records = csv.parse(content, { columns: true, skip_empty_lines: true });
-      for (const record of records) {
-        if (record.canonicalId) ids.add(record.canonicalId);
-      }
-    } catch {
-      // Manifest unreadable — return an empty set so cleanup falls back to
-      // the conservative "delete what removalSet says" behavior.
-    }
-    return ids;
-  }
-
   /**
    * Cleanup a specific target directory.
    * When removalSet is provided, only removes entries in that set.

tools/installer/ide/platform-codes.yaml

@@ -132,21 +132,6 @@ platforms:
     installer:
       target_dir: .agents/skills
       global_target_dir: ~/.agents/skills
-      commands_target_dir: .github/agents
-      commands_extension: .agent.md
-      commands_body_template: "LOAD the FULL {project-root}/{target_dir}/{canonicalId}/SKILL.md, READ its entire contents and follow its directions exactly!"
-      # The Custom Agents picker should only show persona agents (not
-      # workflows/tools). Detected by reading each skill's source
-      # `customize.toml` and checking for an `[agent]` section — that's
-      # the actual configuration source of truth: every BMAD persona is
-      # configured under `[agent]`, every workflow under `[workflow]`,
-      # every standalone skill has no customize.toml. This signal is
-      # naming-independent, so personas like `bmad-tea` (which doesn't
-      # follow the `-agent-` convention) are still included, and
-      # meta-skills like `bmad-agent-builder` (which contains `-agent-`
-      # but is a skill-builder workflow, not a persona) are correctly
-      # excluded.
-      commands_filter: agents-only
 
   goose:
     name: "Block Goose"
@@ -237,7 +222,6 @@ platforms:
     installer:
       target_dir: .agents/skills
       global_target_dir: ~/.agents/skills
-      commands_target_dir: .opencode/commands
 
   openhands:
     name: "OpenHands"

tools/installer/modules/module-help-schema.js

@@ -1,13 +0,0 @@
-/**
- * Canonical schema for per-module `module-help.csv` files.
- *
- * Both the merger (`Installer.mergeModuleHelpCatalogs`) and the synthesizer
- * (`PluginResolver._buildSynthesizedHelpCsv`) emit this exact header. The
- * merger compares each per-module file's header against this string and
- * warns on drift, so any rename here must be matched in external module
- * authors' CSVs (or accepted as a positional fall-through with a warning).
- */
-const MODULE_HELP_CSV_HEADER =
-  'module,skill,display-name,menu-code,description,action,args,phase,preceded-by,followed-by,required,output-location,outputs';
-
-module.exports = { MODULE_HELP_CSV_HEADER };

tools/installer/modules/plugin-resolver.js

@@ -1,7 +1,6 @@
 const fs = require('../fs-native');
 const path = require('node:path');
 const yaml = require('yaml');
-const { MODULE_HELP_CSV_HEADER } = require('./module-help-schema');
 
 /**
  * Resolves how to install a plugin from marketplace.json by analyzing
@@ -339,7 +338,8 @@ class PluginResolver {
    * @returns {string} CSV content
    */
   _buildSynthesizedHelpCsv(moduleName, skillInfos) {
-    const rows = [MODULE_HELP_CSV_HEADER];
+    const header = 'module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs';
+    const rows = [header];
 
     for (const info of skillInfos) {
       const displayName = this._formatDisplayName(info.name || info.dirName);