luckiest-co 1.0.34 → 1.0.36

package/.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
       "name": "luckiest",
       "source": "./",
       "description": "Plan, go, finish, plus bundled free Luckiest skills that work in Claude Code, web chat, and Cowork.",
-      "version": "0.1.33",
+      "version": "0.1.35",
       "author": { "name": "Luckiest" }
     }
   ]

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "luckiest",
   "description": "Plan, go, finish. Guided planning, progress dashboards, and your luckiest.co tribe inside Claude Code.",
-  "version": "0.1.33",
+  "version": "0.1.35",
   "author": { "name": "Luckiest" }
 }

package/bin/bump-version.js

@@ -0,0 +1,62 @@
+#!/usr/bin/env node
+
+// Release bump: patch-bumps the npm version and the plugin version, then dates
+// the CHANGELOG (turns "## Unreleased" into the release header, or adds one).
+
+const fs = require('fs');
+const path = require('path');
+
+const green = '\x1b[32m';
+const reset = '\x1b[0m';
+
+const VERSION_RE = /("version":\s*")(\d+)\.(\d+)\.(\d+)(")/;
+
+const patch = (v) => v.replace(/\d+$/, (n) => Number(n) + 1);
+
+/** Bumps all four release files in dir. Returns { npm, plugin }. Writes nothing if the plugin versions disagree. */
+function bumpVersion({ dir = path.join(__dirname, '..'), date = new Date().toLocaleDateString('en-CA') } = {}) {
+  const files = {
+    pkg: path.join(dir, 'package.json'),
+    plugin: path.join(dir, '.claude-plugin/plugin.json'),
+    market: path.join(dir, '.claude-plugin/marketplace.json'),
+    changelog: path.join(dir, 'CHANGELOG.md'),
+  };
+  const text = Object.fromEntries(Object.entries(files).map(([k, f]) => [k, fs.readFileSync(f, 'utf8')]));
+  const current = (k) => {
+    const m = text[k].match(VERSION_RE);
+    if (!m) throw new Error(`No version found in ${files[k]}`);
+    return `${m[2]}.${m[3]}.${m[4]}`;
+  };
+
+  const oldPlugin = current('plugin');
+  if (current('market') !== oldPlugin) {
+    throw new Error(`plugin.json (${oldPlugin}) and marketplace.json (${current('market')}) disagree. Fix them by hand first.`);
+  }
+  const npm = patch(current('pkg'));
+  const plugin = patch(oldPlugin);
+
+  // ponytail: replaces the first "version" in each file, which is the right one in all three today.
+  const write = (k, v) => fs.writeFileSync(files[k], text[k].replace(VERSION_RE, `$1${v}$5`));
+  write('pkg', npm);
+  write('plugin', plugin);
+  write('market', plugin);
+  // An "## Unreleased" section becomes the dated header, notes and all. Otherwise a fresh header goes on top.
+  const header = `## ${plugin} — ${date} (npm luckiest-co ${npm})`;
+  const unreleased = /^## Unreleased$/m;
+  fs.writeFileSync(files.changelog, unreleased.test(text.changelog)
+    ? text.changelog.replace(unreleased, header)
+    : `${header}\n\n${text.changelog}`);
+  return { npm, plugin };
+}
+
+if (require.main === module) {
+  try {
+    const { npm, plugin } = bumpVersion();
+    console.log(`  ${green}Bumped to npm luckiest-co ${npm}, plugin ${plugin}.${reset} Check the notes under the new CHANGELOG.md header.`);
+  } catch (err) {
+    console.error(`  ${err.message}`);
+    process.exit(1);
+  }
+}
+
+module.exports = { bumpVersion };

package/bin/install.js

@@ -224,6 +224,58 @@ function placeSkills(extractDir, skillsRoot, fallbackName) {
 }
 
 async function syncSkills() {
+  try {
+    await syncOwnedSkills();
+  } finally {
+    // Self-heal the luckiest-x + luckiest:luckiest-x duplicate: the marketplace
+    // plugin already registers its skills, so a copy in ~/.claude/skills (from
+    // older installers or from this sync) lists every one of them twice.
+    const globalDir = expandTilde(explicitConfigDir) || expandTilde(process.env.CLAUDE_CONFIG_DIR) || path.join(os.homedir(), '.claude');
+    const removed = dropPluginDuplicates(path.join(globalDir, 'skills'), pluginSkillNames(globalDir));
+    if (removed.length) {
+      console.log(`  ${green}✓${reset} Removed ${removed.length} duplicate skill(s) from skills/ (plugin marketplace already provides them)`);
+    }
+  }
+}
+
+/**
+ * Skill folder names shipped by the enabled marketplace plugin, read from its
+ * installPath. Empty when the plugin is absent or disabled.
+ */
+function pluginSkillNames(globalClaudeDir) {
+  const names = new Set();
+  if (!hasMarketplacePlugin(globalClaudeDir)) return names;
+  try {
+    const manifest = JSON.parse(
+      fs.readFileSync(path.join(globalClaudeDir, 'plugins', 'installed_plugins.json'), 'utf8')
+    );
+    for (const [key, value] of Object.entries(manifest.plugins || {})) {
+      if (!key.startsWith('luckiest@')) continue;
+      for (const item of Array.isArray(value) ? value : [value]) {
+        const dir = item && item.installPath && path.join(item.installPath, 'skills');
+        if (dir && fs.existsSync(dir)) fs.readdirSync(dir).forEach((n) => names.add(n));
+      }
+    }
+  } catch {
+    // Unreadable manifest: remove nothing.
+  }
+  return names;
+}
+
+/** Remove skills/<name> for every name the plugin ships. Returns removed names. */
+function dropPluginDuplicates(skillsRoot, names) {
+  const removed = [];
+  for (const name of names) {
+    const dest = path.join(skillsRoot, name);
+    if (fs.existsSync(path.join(dest, 'SKILL.md'))) {
+      fs.rmSync(dest, { recursive: true, force: true });
+      removed.push(name);
+    }
+  }
+  return removed;
+}
+
+async function syncOwnedSkills() {
   // Self-heal the /luckiest:luckiest:* duplicate: older installers copied
   // commands into ~/.claude/commands/luckiest even when the marketplace
   // plugin already registers them. Sync runs every session, so clean it here.
@@ -428,8 +480,12 @@ function install(isGlobal) {
   }
 
   // Copy references/, templates/, .claude-plugin/ into the target root
+  // skills/ is skipped when the marketplace plugin ships them, for the same
+  // reason as commands/ above (luckiest-x and luckiest:luckiest-x).
+  const pluginPresent = hasMarketplacePlugin(defaultGlobalDir);
   const topLevelDirs = ['references', 'templates', 'skills', '.claude-plugin', 'hooks'];
   for (const dir of topLevelDirs) {
+    if (dir === 'skills' && pluginPresent) continue;
     const dirSrc = path.join(src, dir);
     const dirDest = path.join(claudeDir, dir);
     if (fs.existsSync(dirSrc)) {
@@ -718,4 +774,4 @@ if (require.main === module) {
   });
 }
 
-module.exports = { placeSkills };
+module.exports = { placeSkills, dropPluginDuplicates };

package/bin/sync-plugin-repo.js

@@ -0,0 +1,57 @@
+#!/usr/bin/env node
+
+// Plugin repo sync: mirrors this package into the public marketplace repo
+// (luckiestco/luckiest-plugin) and opens a PR there. Run after the release is
+// bumped and published. --dry-run stops before anything is pushed.
+
+const { execFileSync } = require('child_process');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+
+const green = '\x1b[32m';
+const yellow = '\x1b[33m';
+const reset = '\x1b[0m';
+
+const REPO = process.env.LUCKIEST_PLUGIN_REPO || 'luckiestco/luckiest-plugin';
+
+const sh = (cmd, args, cwd) => execFileSync(cmd, args, { cwd, encoding: 'utf8', stdio: ['ignore', 'pipe', 'pipe'] }).trimEnd();
+const versionOf = (dir, file) => JSON.parse(fs.readFileSync(path.join(dir, file), 'utf8')).version;
+
+/** Returns { status: 'synced' | 'dry-run' | 'pr', dir, changes, url }. */
+function syncPluginRepo({ src = path.join(__dirname, '..'), remote = `https://github.com/${REPO}.git`, dryRun = false } = {}) {
+  const plugin = versionOf(src, '.claude-plugin/plugin.json');
+  const npm = versionOf(src, 'package.json');
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'luckiest-plugin-sync-'));
+  sh('git', ['clone', '--depth', '1', remote, dir]);
+
+  if (versionOf(dir, '.claude-plugin/plugin.json') === plugin) return { status: 'synced', dir, changes: [] };
+
+  // --checksum: a same-size edit with a matching mtime is otherwise skipped (a version bump is same-size).
+  sh('rsync', ['-a', '--checksum', '--delete', '--exclude', '.git', '--exclude', '.DS_Store', '--exclude', '__pycache__', `${src}/`, `${dir}/`]);
+  const changes = sh('git', ['status', '--porcelain'], dir).split('\n').filter(Boolean);
+  if (dryRun) return { status: 'dry-run', dir, changes };
+
+  const branch = `sync-${plugin}`;
+  const title = `Sync plugin ${plugin} (npm luckiest-co ${npm})`;
+  sh('git', ['switch', '-c', branch], dir);
+  sh('git', ['add', '-A'], dir);
+  sh('git', ['commit', '-m', title], dir);
+  sh('git', ['push', '-u', 'origin', branch], dir);
+  const url = sh('gh', ['pr', 'create', '--repo', REPO, '--head', branch, '--title', title, '--body', `Mirrors packages/luckiest-plugin at plugin ${plugin}.`], dir);
+  return { status: 'pr', dir, changes, url };
+}
+
+if (require.main === module) {
+  try {
+    const { status, dir, changes, url } = syncPluginRepo({ dryRun: process.argv.includes('--dry-run') });
+    if (status === 'synced') console.log(`  ${green}${REPO} already has this plugin version.${reset}`);
+    if (status === 'dry-run') console.log(`  ${yellow}Dry run, nothing pushed.${reset} ${changes.length} changed files in ${dir}:\n${changes.map((c) => `    ${c}`).join('\n')}`);
+    if (status === 'pr') console.log(`  ${green}Opened ${url}${reset}`);
+  } catch (err) {
+    console.error(`  ${(err.stderr || err.message).toString().trim()}`);
+    process.exit(1);
+  }
+}
+
+module.exports = { syncPluginRepo };

package/package.json

@@ -1,10 +1,15 @@
 {
   "name": "luckiest-co",
-  "version": "1.0.34",
+  "version": "1.0.36",
   "description": "Luckiest for Claude Code: plan, go, finish. Your luckiest.co skills and tribe, inside Claude.",
   "bin": {
     "luckiest-co": "./bin/install.js"
   },
+  "scripts": {
+    "prepublishOnly": "node bin/check-version.js",
+    "bump": "node bin/bump-version.js",
+    "sync-plugin": "node bin/sync-plugin-repo.js"
+  },
   "files": [
     "bin",
     "commands",
@@ -15,7 +20,8 @@
     ".claude-plugin",
     "!**/__pycache__",
     "!**/*.pyc",
-    "!bin/*.test.js"
+    "!bin/*.test.js",
+    "!bin/check-version.js"
   ],
   "license": "SEE LICENSE IN LICENSE",
   "engines": {

package/skills/luckiest-charms/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-charms
+user-invocable: false
 description: "Your Luckiest charms balance and recent related activity. Use when the user says /luckiest charms, luckiest charms, or asks about their charms."
 ---
 

package/skills/luckiest-finish/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-finish
+user-invocable: false
 description: "Wrap up a completed Luckiest plan: recap what shipped, close it out, award charms. Use when the user says /luckiest finish, luckiest finish, or all plan tasks are done."
 ---
 

package/skills/luckiest-go/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-go
+user-invocable: false
 description: "Run the active Luckiest plan, one task at a time, checked as it goes. Use when the user says /luckiest go, luckiest go, or wants to continue their staged Luckiest plan."
 ---
 

package/skills/luckiest-helpers/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-helpers
+user-invocable: false
 description: "See who in your Luckiest tribe needs a hand right now, and claim a request to help. Use when the user says /luckiest helpers, luckiest helpers, or asks who needs help."
 ---
 

package/skills/luckiest-home/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-home
+user-invocable: false
 description: "The Luckiest community dashboard: wishes and charms, plan progress, tribe pulse, and who needs help. Use when the user says /luckiest home, luckiest home, or asks for their Luckiest dashboard."
 ---
 

package/skills/luckiest-leaderboard/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-leaderboard
+user-invocable: false
 description: "See how the Luckiest tribe ranks: top scores, one bar chart. Use when the user says /luckiest leaderboard, luckiest leaderboard, or asks for tribe rankings."
 ---
 

package/skills/luckiest-merge/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-merge
+user-invocable: false
 description: "Merge two overlapping Luckiest skills into one, after the user reviews the diff. Use when the user says /luckiest merge, luckiest merge, or wants to combine overlapping skills."
 ---
 

package/skills/luckiest-next/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-next
+user-invocable: false
 description: "Do the suggested next step: runs the last Next line from this chat, or picks up the plan where it stands. Use when the user says /next, /luckiest next, luckiest next, or next step."
 ---
 

package/skills/luckiest-overlaps/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-overlaps
+user-invocable: false
 description: "Find installed Luckiest skills that overlap with your own, and resolve them. Use when the user says /luckiest overlaps, luckiest overlaps, or asks about duplicate skills."
 ---
 

package/skills/luckiest-plan/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-plan
+user-invocable: false
 description: "Use when the user says /luckiest plan or luckiest plan, or wants to plan a feature, a launch, or their next piece of work before building it, even if they do not say plan (for example \"what should I build next\" or \"break this down into tasks\"). Looks at the project first, drafts 3 to 7 tasks with a suggested skill, model, and runnable check each, saves the context for /luckiest go, and stages the plan after one approval question."
 ---
 

package/skills/luckiest-skills/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-skills
+user-invocable: false
 description: "Your Luckiest skills: list what is installed, pull new purchases. Use when the user says /luckiest skills, luckiest skills, or asks what Luckiest skills they have."
 ---
 

package/skills/luckiest-start/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-start
+user-invocable: false
 description: "A short interview about your project that writes your Brand Brief. Use when the user says /luckiest start, luckiest start, or wants to set up Luckiest for a project."
 ---
 

package/skills/luckiest-status/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-status
+user-invocable: false
 description: "Where you are in your Luckiest plan: progress, tasks, and your one next step. Use when the user says /luckiest status, luckiest status, or asks where their plan stands."
 ---
 

package/skills/luckiest-updates/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-updates
+user-invocable: false
 description: "Check for updates to your installed Luckiest skills and plugin. Use when the user says /luckiest updates, luckiest updates, or asks if their Luckiest install is current."
 ---
 

package/skills/luckiest-vouch/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-vouch
+user-invocable: false
 description: "Request or approve a warm intro to someone outside your Luckiest tribe. Use when the user says /luckiest vouch, luckiest vouch, or asks for an intro."
 ---
 

package/skills/luckiest-wishes/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: luckiest-wishes
+user-invocable: false
 description: "Your Luckiest wishes balance and recent related activity. Use when the user says /luckiest wishes, luckiest wishes, or asks about their wishes."
 ---