@antoneeo/mkt-agentic-sdlc-skill 0.2.1 → 0.3.1

package/scripts/init.js

@@ -1,147 +1,207 @@
-#!/usr/bin/env node
-
-const fs = require('fs');
-const path = require('path');
-const { execSync } = require('child_process');
-const { SKILL_SOURCE, CLIENTS, clientDetected, loadTemplates, templateFor } = require('./lib');
-
-const cwd = process.cwd();
-
-// 1. Directory layout (canonical mkt_docs structure)
-const directories = [
-  'mkt_docs',
-  'mkt_docs/vision',
-  'mkt_docs/research',
-  'mkt_docs/strategy',
-  'mkt_docs/tactics',
-  'mkt_docs/deliverables',
-  'mkt_docs/audit',
-  'mkt_docs/spikes',
-].map((d) => path.join(cwd, d));
-
-// 2. Project protocol (thin pointer — the operating contract is the skill).
-// Deliberately short: duplicating the skill's rules here made them drift.
-const protocolContent = `# Marketing Agentic SDLC — Project Protocol (pointer)
-
-This project follows the Marketing Agentic SDLC Evidence-First process. The full
-operating contract is the \`mkt-agentic-sdlc\` skill (installed in your agent's
-skills directory); this file is only the minimal always-on pointer.
-
-## Rule Zero — Triage every request
-- E1 Quick: single question or asset feedback (positioning opinion, copy critique). Answer directly; numbers still need a source.
-- E2 Campaign: single-campaign or single-channel plan. Light path: mini-brief + targeted research + campaign document.
-- E3 Full Plan: complete marketing plan. Full SOSTAC workflow via the skill: Discovery -> Research -> Situation -> Objectives -> Strategy -> Tactics -> Action -> Control -> Packaging.
-- Research Spike: time-boxed market question; outcome in \`mkt_docs/spikes/RESEARCH_[topic].md\`.
-- When in doubt, pick the higher level. Declare the chosen level when starting.
-
-## The three non-negotiables
-1. Evidence ledger: every market number carries an [EV-nn] reference (FACT / BENCHMARK / ASSUMPTION). No invented numbers, ever.
-2. Ask only what the user owns: facts come from the user, analysis comes from research. Never ask the user to do the analysis.
-3. Internal consistency: budget sums, funnel math and the objective->tactic->KPI chain are validated mechanically before delivery.
-
-## Where things live
-- Vision (gate for E3): \`mkt_docs/vision/MKT_VISION.md\` — \`Status: DRAFT\` informs, \`Status: APPROVED\` binds.
-- Evidence: \`mkt_docs/research/evidence_ledger.md\` (the single source for every number).
-- Strategy artifacts: \`mkt_docs/strategy/\`; tactical artifacts: \`mkt_docs/tactics/\`; final deliverables: \`mkt_docs/deliverables/\`.
-- Must-reads: \`mkt_docs/README.md\`; full generated manifest: \`mkt_docs/INDEX.md\`.
-- If devPNT is available for this project, its M-VISION / plans / governed artifacts take over (Hybrid mode — see the skill).
-
-## Closure gate
-Deliverables ship only after \`python <skill_dir>/scripts/mkt_check.py check\` is
-CLEAN (where the validator is adopted) and the CMO review passed.
-
-If the mkt-agentic-sdlc skill is not available in this client, ask the user to install it:
-\`npm i -g @antoneeo/mkt-agentic-sdlc-skill\` (the skill installs automatically;
-if npm blocks scripts, run \`npx mkt-sdlc-install-skill\`).
-`;
-
-console.log('🚀 Initializing Marketing Agentic SDLC workflow...');
-
-// 3. Load templates from the single source (skill's templates.md)
-let sections;
-try {
-  sections = loadTemplates();
-} catch (err) {
-  console.error(`❌ Cannot load templates: ${err.message}`);
-  process.exit(1);
-}
-
-let seedFiles;
-try {
-  seedFiles = [
-    ['mkt_docs/README.md', templateFor(sections, 'mkt_docs/README.md')],
-    ['mkt_docs/vision/MKT_VISION.md', templateFor(sections, 'MKT_VISION.md')],
-    ['mkt_docs/vision/principles.md', templateFor(sections, 'principles.md')],
-    ['mkt_docs/research/evidence_ledger.md', templateFor(sections, 'evidence_ledger.md')],
-    // NOTE: INDEX.md is NOT seeded — it is generated by `mkt_check.py index`
-    // and would immediately fail validate.
-  ];
-} catch (err) {
-  console.error(`❌ ${err.message}`);
-  process.exit(1);
-}
-
-// 4. Create directories
-directories.forEach((dir) => {
-  if (!fs.existsSync(dir)) {
-    fs.mkdirSync(dir, { recursive: true });
-    console.log(`📁 Created directory: ${path.relative(cwd, dir)}`);
-  }
-});
-
-// 5. Write seed files (never overwrite)
-const writeIfNotExists = (relPath, content, description) => {
-  const filePath = path.join(cwd, relPath);
-  if (!fs.existsSync(filePath)) {
-    fs.writeFileSync(filePath, content, 'utf8');
-    console.log(`📄 Created file: ${relPath}${description ? ` (${description})` : ''}`);
-    return true;
-  }
-  console.log(`⏭️  Skipped: ${relPath} already exists.`);
-  return false;
-};
-
-seedFiles.forEach(([relPath, content]) => writeIfNotExists(relPath, content));
-
-// 6. Client discovery and protocol pointers
-console.log('\n--- Environment Analysis ---');
-
-const protocolFiles = {
-  claude: 'CLAUDE.md',
-  gemini: 'GEMINI.md',
-  codex: 'AGENTS.md',
-  antigravity: 'AGENTS.md', // Antigravity CLI reads AGENTS.md; single protocolContent reused.
-};
-
-for (const client of CLIENTS) {
-  if (clientDetected(client)) {
-    console.log(`✅ ${client.label} detected.`);
-    writeIfNotExists(protocolFiles[client.key], protocolContent, `${client.label} protocol pointer`);
-  }
-}
-
-// Cursor/Windsurf (always recommended)
-writeIfNotExists('.cursorrules', protocolContent, 'Cursor/Windsurf rules');
-
-// 7. Generate mkt_docs/INDEX.md so the very first `validate` is already clean.
-// The manifest is generated, never seeded: delegate to the validator if Python is available.
-const validator = path.join(SKILL_SOURCE, 'scripts', 'mkt_check.py');
-let indexed = false;
-for (const py of ['python', 'python3', 'py']) {
-  try {
-    execSync(`${py} "${validator}" index --root "${cwd}"`, { stdio: 'ignore' });
-    console.log('📇 Generated mkt_docs/INDEX.md (document manifest).');
-    indexed = true;
-    break;
-  } catch (e) { /* try the next interpreter */ }
-}
-if (!indexed) {
-  console.log('ℹ️  Python not found: generate the manifest later with '
-    + '"python <skill_dir>/scripts/mkt_check.py index" (validate reports it until then).');
-}
-
-console.log('\n✅ Setup completed successfully!');
-console.log('💡 Next steps:');
-console.log('   1. Make sure the mkt-agentic-sdlc skill is installed (mkt-sdlc-install-skill).');
-console.log('   2. Restart/open the project in your AI client so it reads the protocol pointer.');
-console.log('   3. Start an E3 engagement: the skill runs Discovery (its questions) before anything else.');
+#!/usr/bin/env node
+
+const fs = require('fs');
+const path = require('path');
+const { execSync } = require('child_process');
+const { SKILL_SOURCE, INSTALLED_SKILL_NAME, SELF_LENS, SIBLING_LENSES, CLIENTS, clientDetected, skillTarget, loadTemplates, templateFor } = require('./lib');
+
+const cwd = process.cwd();
+
+// 1. Directory layout (canonical mkt_docs structure)
+const directories = [
+  'mkt_docs',
+  'mkt_docs/vision',
+  'mkt_docs/research',
+  'mkt_docs/strategy',
+  'mkt_docs/tactics',
+  'mkt_docs/deliverables',
+  'mkt_docs/audit',
+  'mkt_docs/spikes',
+].map((d) => path.join(cwd, d));
+
+// 2. Project protocol (thin pointer — the operating contract is the skill).
+// Deliberately short: duplicating the skill's rules here made them drift.
+const protocolContent = `# Marketing Agentic SDLC — Project Protocol (pointer)
+
+This project follows the Marketing Agentic SDLC Evidence-First process. The full
+operating contract is the \`mkt-agentic-sdlc\` skill (installed in your agent's
+skills directory); this file is only the minimal always-on pointer.
+
+## Rule Zero — Triage every request
+- E1 Quick: single question or asset feedback (positioning opinion, copy critique). Answer directly; numbers still need a source.
+- E2 Campaign: single-campaign or single-channel plan. Light path: mini-brief + targeted research + campaign document.
+- E3 Full Plan: complete marketing plan. Full SOSTAC workflow via the skill: Discovery -> Research -> Situation -> Objectives -> Strategy -> Tactics -> Action -> Control -> Packaging.
+- Research Spike: time-boxed market question; outcome in \`mkt_docs/spikes/RESEARCH_[topic].md\`.
+- When in doubt, pick the higher level. Declare the chosen level when starting.
+
+## The three non-negotiables
+1. Evidence ledger: every market number carries an [EV-nn] reference (FACT / BENCHMARK / ASSUMPTION). No invented numbers, ever.
+2. Ask only what the user owns: facts come from the user, analysis comes from research. Never ask the user to do the analysis.
+3. Internal consistency: budget sums, funnel math and the objective->tactic->KPI chain are validated mechanically before delivery.
+
+## Where things live
+- Vision (gate for E3): \`mkt_docs/vision/MKT_VISION.md\` — \`Status: DRAFT\` informs, \`Status: APPROVED\` binds.
+- Evidence: \`mkt_docs/research/evidence_ledger.md\` (the single source for every number).
+- Strategy artifacts: \`mkt_docs/strategy/\`; tactical artifacts: \`mkt_docs/tactics/\`; final deliverables: \`mkt_docs/deliverables/\`.
+- Must-reads: \`mkt_docs/README.md\`; full generated manifest: \`mkt_docs/INDEX.md\`.
+- If devPNT is available for this project, its M-VISION / plans / governed artifacts take over (Hybrid mode — see the skill).
+
+## Closure gate
+Deliverables ship only after \`python <skill_dir>/scripts/mkt_check.py check\` is
+CLEAN (where the validator is adopted) and the CMO review passed.
+
+If the mkt-agentic-sdlc skill is not available in this client, ask the user to install it:
+\`npm i -g @antoneeo/mkt-agentic-sdlc-skill\` (the skill installs automatically;
+if npm blocks scripts, run \`npx mkt-sdlc-install-skill\`).
+`;
+
+console.log('🚀 Initializing Marketing Agentic SDLC workflow...');
+
+// 3. Load templates from the single source (skill's templates.md)
+let sections;
+try {
+  sections = loadTemplates();
+} catch (err) {
+  console.error(`❌ Cannot load templates: ${err.message}`);
+  process.exit(1);
+}
+
+let seedFiles;
+try {
+  seedFiles = [
+    ['mkt_docs/README.md', templateFor(sections, 'mkt_docs/README.md')],
+    ['mkt_docs/vision/MKT_VISION.md', templateFor(sections, 'MKT_VISION.md')],
+    ['mkt_docs/vision/principles.md', templateFor(sections, 'principles.md')],
+    ['mkt_docs/research/evidence_ledger.md', templateFor(sections, 'evidence_ledger.md')],
+    // NOTE: INDEX.md is NOT seeded — it is generated by `mkt_check.py index`
+    // and would immediately fail validate.
+  ];
+} catch (err) {
+  console.error(`❌ ${err.message}`);
+  process.exit(1);
+}
+
+// 4. Create directories
+directories.forEach((dir) => {
+  if (!fs.existsSync(dir)) {
+    fs.mkdirSync(dir, { recursive: true });
+    console.log(`📁 Created directory: ${path.relative(cwd, dir)}`);
+  }
+});
+
+// 5. Write seed files (never overwrite)
+const writeIfNotExists = (relPath, content, description) => {
+  const filePath = path.join(cwd, relPath);
+  if (!fs.existsSync(filePath)) {
+    fs.writeFileSync(filePath, content, 'utf8');
+    console.log(`📄 Created file: ${relPath}${description ? ` (${description})` : ''}`);
+    return true;
+  }
+  console.log(`⏭️  Skipped: ${relPath} already exists.`);
+  return false;
+};
+
+seedFiles.forEach(([relPath, content]) => writeIfNotExists(relPath, content));
+
+// 6. Client discovery and protocol pointers
+console.log('\n--- Environment Analysis ---');
+
+const protocolFiles = {
+  claude: 'CLAUDE.md',
+  gemini: 'GEMINI.md',
+  codex: 'AGENTS.md',
+  antigravity: 'AGENTS.md', // Antigravity CLI reads AGENTS.md; single protocolContent reused.
+};
+
+// A protocol pointer that ALREADY existed was written by someone else — typically a
+// sibling lens's init, which seeded the project with its own triage ladder. This init
+// must not overwrite it (create-only is the T1 guarantee), so the second ladder is
+// written aside and merged by hand.
+let protocolPreexisting = false;
+
+for (const client of CLIENTS) {
+  if (clientDetected(client)) {
+    console.log(`✅ ${client.label} detected.`);
+    const created = writeIfNotExists(protocolFiles[client.key], protocolContent, `${client.label} protocol pointer`);
+    if (!created) protocolPreexisting = true;
+  }
+}
+
+// Cursor/Windsurf (always recommended)
+writeIfNotExists('.cursorrules', protocolContent, 'Cursor/Windsurf rules');
+
+// 6b. Sibling lenses: additive only. Never edits a user-authored root file.
+function installedSiblingLenses() {
+  const found = new Map();
+  for (const client of CLIENTS) {
+    if (!clientDetected(client)) continue;
+    const skillsDir = path.dirname(skillTarget(client));
+    for (const [dirName, lens] of Object.entries(SIBLING_LENSES)) {
+      if (fs.existsSync(path.join(skillsDir, dirName))) found.set(dirName, lens);
+    }
+  }
+  return found;
+}
+
+const siblings = installedSiblingLenses();
+if (siblings.size > 0) {
+  const list = [...siblings].map(([dir, lens]) => `- \`${dir}\` — the **${lens}** lens`).join('\n');
+  console.log(`\n🔀 Sibling lens detected: ${[...siblings.keys()].join(', ')}.`);
+  const wrote = writeIfNotExists('AGENTIC_MULTI_LENS.md', `# Multi-lens project — routing note (additive)
+
+This project has more than one lens of the Agentic SDLC family installed:
+
+${list}
+- \`${INSTALLED_SKILL_NAME}\` — the **${SELF_LENS}** lens
+
+One \`ai_docs/\` tree, one project default (\`default_domain:\` in \`ai_docs/README.md\`),
+one lens per unit of work. Before acting on any L2, L3 or Spike, run the domain router
+in the skill's \`routing.md\`: it decides which lens's method and validation rules govern
+that unit. L1 never reaches it. Never refer to a document whose meaning differs by lens
+("threat model", "vision", \`principles.md\`, \`handoff.md\`) by its bare name.
+
+${protocolPreexisting ? `**Merge step owed.** The always-on protocol pointer of this project (\`CLAUDE.md\` /
+\`GEMINI.md\` / \`AGENTS.md\` / \`.cursorrules\`) was written by another lens's init and
+carries ITS triage ladder. This init did not overwrite it. Add the marketing lens's ladder
+to that file by hand — the pointer below — so both are always loaded.
+
+## Rule Zero — Triage every request (marketing lens)
+- E1 Question: one answer from the existing plan, no document written.
+- E2 Campaign: a single campaign or channel, on an approved strategy.
+- E3 Full engagement: the nine-phase SOSTAC workflow, evidence ledger and all
+  three validator gates.
+` : `The always-on protocol pointer for the marketing lens was created by this init. When you
+install another lens over this project, its init will leave its own ladder here for you
+to merge.
+`}
+This file is NOT auto-loaded by any client: it is a note for you, deliberately additive.
+Delete it once the merge is done.
+`, 'multi-lens routing note');
+  if (wrote && protocolPreexisting) {
+    console.log('   ⚠️  A protocol pointer already existed and was NOT overwritten.');
+    console.log(`       Merge the ${SELF_LENS}-lens ladder from AGENTIC_MULTI_LENS.md into it by hand.`);
+  }
+}
+
+// 7. Generate mkt_docs/INDEX.md so the very first `validate` is already clean.
+// The manifest is generated, never seeded: delegate to the validator if Python is available.
+const validator = path.join(SKILL_SOURCE, 'scripts', 'mkt_check.py');
+let indexed = false;
+for (const py of ['python', 'python3', 'py']) {
+  try {
+    execSync(`${py} "${validator}" index --root "${cwd}"`, { stdio: 'ignore' });
+    console.log('📇 Generated mkt_docs/INDEX.md (document manifest).');
+    indexed = true;
+    break;
+  } catch (e) { /* try the next interpreter */ }
+}
+if (!indexed) {
+  console.log('ℹ️  Python not found: generate the manifest later with '
+    + '"python <skill_dir>/scripts/mkt_check.py index" (validate reports it until then).');
+}
+
+console.log('\n✅ Setup completed successfully!');
+console.log('💡 Next steps:');
+console.log('   1. Make sure the mkt-agentic-sdlc skill is installed (mkt-sdlc-install-skill).');
+console.log('   2. Restart/open the project in your AI client so it reads the protocol pointer.');
+console.log('   3. Start an E3 engagement: the skill runs Discovery (its questions) before anything else.');

package/scripts/lib.js

@@ -10,7 +10,65 @@ const { execSync } = require('child_process');
 const PACKAGE_ROOT = path.resolve(__dirname, '..');
 const SKILL_SOURCE = path.join(PACKAGE_ROOT, 'skills', 'mkt-agentic-sdlc');
 const TEMPLATES_PATH = path.join(SKILL_SOURCE, 'templates.md');
-const SKILL_DIR_NAME = 'mkt-agentic-sdlc';
+// Kept for backward compatibility with this package's own scripts; the
+// authoritative value is INSTALLED_SKILL_NAME, derived from SKILL.md below.
+// The directory name each client loads the skill from. Derived from the manifest,
+// never hard-coded by a consumer: three distributions share these scripts, and a
+// literal here is how a copy-fork starts installing under its sibling's name.
+const INSTALLED_SKILL_NAME = (() => {
+  const m = fs.readFileSync(path.join(SKILL_SOURCE, 'SKILL.md'), 'utf8').match(/^name:\s*(\S+)/m);
+  if (!m) throw new Error(`SKILL.md carries no 'name:' field: ${SKILL_SOURCE}`);
+  return m[1];
+})();
+
+// The family's lens table, read from the shared `routing.md` rather than restated
+// here. Same reason as INSTALLED_SKILL_NAME above, and the same failure it prevents:
+// BOTH the row for this lens and the rows for its siblings used to be literals copied
+// between distributions, so kb and mkt wrote a multi-lens note announcing themselves
+// as the code lens. routing.md is one of the byte-identical shared files, so this
+// lookup cannot drift between distributions.
+// Lazy on purpose: preuninstall.js needs INSTALLED_SKILL_NAME and nothing else, and
+// must keep working on an installation damaged badly enough to have lost routing.md.
+function lensTable() {
+  const routing = path.join(SKILL_SOURCE, 'routing.md');
+  if (!fs.existsSync(routing)) throw new Error(`routing.md missing from the skill: ${SKILL_SOURCE}`);
+  // Parsed as a table, not matched with a regex: a mis-escaped pattern matches the
+  // empty string and yields undefined instead of throwing, which is how this lookup
+  // failed the first time it was written.
+  const table = new Map();
+  for (const line of fs.readFileSync(routing, 'utf8').split(/\r?\n/)) {
+    const cells = line.split('|').map((cell) => cell.trim());
+    if (cells.length < 4) continue;
+    const [, lens, skill] = cells;
+    if (!/^[a-z]+$/.test(lens) || !/^`[a-z0-9-]+`$/.test(skill)) continue;
+    const name = skill.slice(1, -1);
+    // A duplicate row would otherwise be won silently by whichever came first, and a
+    // wrong routing.md propagates byte-identically to every distribution.
+    if (table.has(name)) throw new Error(`routing.md lists '${name}' more than once`);
+    table.set(name, lens);
+  }
+  if (!table.size) throw new Error(`routing.md carries no lens table: ${routing}`);
+  return table;
+}
+
+function selfLens() {
+  const lens = lensTable().get(INSTALLED_SKILL_NAME);
+  if (!lens) throw new Error(`routing.md has no lens row for '${INSTALLED_SKILL_NAME}'`);
+  return lens;
+}
+
+// Sibling lenses of the same family: one shared core, one docs tree, a different
+// fidelity discipline each. Keyed by the installed skill directory name.
+function siblingLenses() {
+  const table = lensTable();
+  selfLens();                       // this lens must be in the table too
+  table.delete(INSTALLED_SKILL_NAME);
+  return Object.fromEntries(table);
+}
+
+// One entry per supported AI client. `home` may be overridden by an env var
+// (Claude Desktop / portable installs); presence of the home dir counts as
+// detection even when the CLI is not on PATH.
 
 // One entry per supported AI client. `home` may be overridden by an env var
 // (Claude Desktop / portable installs); presence of the home dir counts as
@@ -88,7 +146,7 @@ function skillTarget(client) {
   // (split on '/' to keep cross-platform path.join correctness). Entries
   // without it resolve to <home>/skills/mkt-agentic-sdlc exactly as before.
   const subdir = client.skillsSubdir ? client.skillsSubdir.split('/') : ['skills'];
-  return path.join(client.home, ...subdir, SKILL_DIR_NAME);
+  return path.join(client.home, ...subdir, INSTALLED_SKILL_NAME);
 }
 
 function copyRecursive(src, dest) {
@@ -156,10 +214,10 @@ function templateFor(sections, needle, index = 0) {
 }
 
 module.exports = {
+  INSTALLED_SKILL_NAME,
   PACKAGE_ROOT,
   SKILL_SOURCE,
   TEMPLATES_PATH,
-  SKILL_DIR_NAME,
   CLIENTS,
   commandExists,
   clientDetected,
@@ -168,3 +226,8 @@ module.exports = {
   loadTemplates,
   templateFor,
 };
+
+// Lazy: reading routing.md is deferred to the consumer that actually asks.
+Object.defineProperty(module.exports, 'SELF_LENS', { enumerable: true, get: selfLens });
+Object.defineProperty(module.exports, 'SIBLING_LENSES', { enumerable: true, get: siblingLenses });
+

package/scripts/postinstall.js

@@ -1,42 +1,42 @@
-#!/usr/bin/env node
-
-const fs = require('fs');
-const { SKILL_SOURCE, CLIENTS, clientDetected, skillTarget, copyRecursive } = require('./lib');
-
-function installSkill(client) {
-  if (!fs.existsSync(SKILL_SOURCE)) {
-    console.log(`⚠️  Skill source not found at ${SKILL_SOURCE}; skipping ${client.label} install.`);
-    return false;
-  }
-  const target = skillTarget(client);
-  try {
-    fs.mkdirSync(target, { recursive: true });
-    copyRecursive(SKILL_SOURCE, target);
-    console.log(`📦 Installed ${client.label} skill at: ${target}`);
-    console.log(`   ${client.reload}`);
-    return true;
-  } catch (err) {
-    console.log(`⚠️  Failed to install ${client.label} skill: ${err.message}`);
-    console.log(`   Manual install: copy "${SKILL_SOURCE}" to "${target}".`);
-    return false;
-  }
-}
-
-console.log('\n--- Marketing Agentic SDLC Skill Discovery ---');
-
-let detected = false;
-for (const client of CLIENTS) {
-  if (clientDetected(client)) {
-    console.log(`✅ Detected: ${client.label}`);
-    detected = true;
-    installSkill(client);
-  }
-}
-
-if (!detected) {
-  console.log('ℹ️  No specific AI CLI detected globally, but you can still use the skill.');
-}
-
-console.log('\nTo initialize a project with the marketing protocol, run:');
-console.log('👉 npx mkt-sdlc-init');
-console.log('------------------------------------\n');
+#!/usr/bin/env node
+
+const fs = require('fs');
+const { SKILL_SOURCE, CLIENTS, clientDetected, skillTarget, copyRecursive } = require('./lib');
+
+function installSkill(client) {
+  if (!fs.existsSync(SKILL_SOURCE)) {
+    console.log(`⚠️  Skill source not found at ${SKILL_SOURCE}; skipping ${client.label} install.`);
+    return false;
+  }
+  const target = skillTarget(client);
+  try {
+    fs.mkdirSync(target, { recursive: true });
+    copyRecursive(SKILL_SOURCE, target);
+    console.log(`📦 Installed ${client.label} skill at: ${target}`);
+    console.log(`   ${client.reload}`);
+    return true;
+  } catch (err) {
+    console.log(`⚠️  Failed to install ${client.label} skill: ${err.message}`);
+    console.log(`   Manual install: copy "${SKILL_SOURCE}" to "${target}".`);
+    return false;
+  }
+}
+
+console.log('\n--- Marketing Agentic SDLC Skill Discovery ---');
+
+let detected = false;
+for (const client of CLIENTS) {
+  if (clientDetected(client)) {
+    console.log(`✅ Detected: ${client.label}`);
+    detected = true;
+    installSkill(client);
+  }
+}
+
+if (!detected) {
+  console.log('ℹ️  No specific AI CLI detected globally, but you can still use the skill.');
+}
+
+console.log('\nTo initialize a project with the marketing protocol, run:');
+console.log('👉 npx mkt-sdlc-init');
+console.log('------------------------------------\n');

package/scripts/preuninstall.js

@@ -1,17 +1,17 @@
-#!/usr/bin/env node
-
-const fs = require('fs');
-const { CLIENTS, skillTarget } = require('./lib');
-
-function removeSkill(client) {
-  const target = skillTarget(client);
-  if (!fs.existsSync(target)) return;
-  try {
-    fs.rmSync(target, { recursive: true, force: true });
-    console.log(`🧹 Removed ${client.label} skill at: ${target}`);
-  } catch (err) {
-    console.log(`⚠️  Could not remove ${target}: ${err.message}`);
-  }
-}
-
-CLIENTS.forEach(removeSkill);
+#!/usr/bin/env node
+
+const fs = require('fs');
+const { CLIENTS, skillTarget } = require('./lib');
+
+function removeSkill(client) {
+  const target = skillTarget(client);
+  if (!fs.existsSync(target)) return;
+  try {
+    fs.rmSync(target, { recursive: true, force: true });
+    console.log(`🧹 Removed ${client.label} skill at: ${target}`);
+  } catch (err) {
+    console.log(`⚠️  Could not remove ${target}: ${err.message}`);
+  }
+}
+
+CLIENTS.forEach(removeSkill);