luckiest-co 1.0.50 → 1.0.52

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.49",
+      "version": "0.1.51",
       "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.49",
+  "version": "0.1.51",
   "author": { "name": "Luckiest" }
 }

package/bin/sync-plugin-repo.js

@@ -2,7 +2,10 @@
 
 // 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.
+// bumped and published. --dry-run stops before anything is pushed. --merge
+// squash-merges the PR it opened (the release workflow uses this). With GH_TOKEN
+// set (CI), git pushes through `gh auth setup-git`, so the token never lands in
+// a URL or a log.
 
 const { execFileSync } = require('child_process');
 const fs = require('fs');
@@ -18,10 +21,25 @@ 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 gitHas = (key, cwd) => {
+  try {
+    return Boolean(sh('git', ['config', key], cwd));
+  } catch {
+    return false;
+  }
+};
+
+/** Returns { status: 'synced' | 'dry-run' | 'pr', dir, changes, url, merged }. */
+function syncPluginRepo({
+  src = path.join(__dirname, '..'),
+  remote = `https://github.com/${REPO}.git`,
+  dryRun = false,
+  merge = false,
+  gh = (args, cwd) => sh('gh', args, cwd),
+} = {}) {
   const plugin = versionOf(src, '.claude-plugin/plugin.json');
   const npm = versionOf(src, 'package.json');
+  if (process.env.GH_TOKEN && remote.startsWith('https://github.com/')) gh(['auth', 'setup-git']);
   const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'luckiest-plugin-sync-'));
   sh('git', ['clone', '--depth', '1', remote, dir]);
 
@@ -36,18 +54,24 @@ function syncPluginRepo({ src = path.join(__dirname, '..'), remote = `https://gi
   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);
+  // CI runners have no git identity; commit as the release bot there.
+  const who = gitHas('user.name', dir) && gitHas('user.email', dir) ? [] : ['-c', 'user.name=luckiest-release', '-c', 'user.email=release@luckiest.co'];
+  sh('git', [...who, '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 };
+  const url = gh(['pr', 'create', '--repo', REPO, '--head', branch, '--title', title, '--body', `Mirrors packages/luckiest-plugin at plugin ${plugin}.`], dir);
+  if (merge) gh(['pr', 'merge', url, '--repo', REPO, '--squash', '--delete-branch'], dir);
+  return { status: 'pr', dir, changes, url, merged: merge };
 }
 
 if (require.main === module) {
   try {
-    const { status, dir, changes, url } = syncPluginRepo({ dryRun: process.argv.includes('--dry-run') });
+    const { status, dir, changes, url, merged } = syncPluginRepo({
+      dryRun: process.argv.includes('--dry-run'),
+      merge: process.argv.includes('--merge'),
+    });
     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}`);
+    if (status === 'pr') console.log(`  ${green}${merged ? 'Opened and merged' : 'Opened'} ${url}${reset}`);
   } catch (err) {
     console.error(`  ${(err.stderr || err.message).toString().trim()}`);
     process.exit(1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "luckiest-co",
-  "version": "1.0.50",
+  "version": "1.0.52",
   "description": "Luckiest for Claude Code: plan, go, finish. Your luckiest.co skills and tribe, inside Claude.",
   "bin": {
     "luckiest-co": "./bin/install.js"
@@ -21,7 +21,8 @@
     "!**/__pycache__",
     "!**/*.pyc",
     "!bin/*.test.js",
-    "!bin/check-version.js"
+    "!bin/check-version.js",
+    "!bin/release-check.js"
   ],
   "license": "SEE LICENSE IN LICENSE",
   "engines": {

package/skills/video-watcher/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog: luckiest-video-watcher
 
+## 1.1.2 (2026-10-23)
+
+- Auto caption mode picks English when YouTube lists several original
+  auto-caption tracks (for example bn-orig and en-orig), then the video's
+  declared language, then the first. It used to take the first alphabetically,
+  so an English talk came back with Bangla captions. New test:
+  `scripts/test_download.py`.
+
 ## 1.1.1 (2026-10-10)
 
 - Security: the WhisperX setup's uv installer is pinned to 0.11.16 and saved to a

package/skills/video-watcher/SKILL.md

@@ -19,7 +19,7 @@ argument-hint: "<video-url | path/to/video.mp4> [question] [--study]"
 license: MIT. See ATTRIBUTION.md
 allowed-tools: Bash, Read, Write, AskUserQuestion
 metadata:
-  version: "1.1.1"
+  version: "1.1.2"
   listing_id: luckiest-video-watcher
   author: luckiest
 ---
@@ -39,11 +39,11 @@ run of a session, when setup fails, or when a run needs a detail not covered her
 ## Staying current
 
 On activation, call the Luckiest MCP `check_updates` tool with
-`{ listingId: "luckiest-video-watcher", installedSemver: "1.1.1" }`. If
+`{ listingId: "luckiest-video-watcher", installedSemver: "1.1.2" }`. If
 `upToDate: false`, surface the `notice` once and continue. Never block on it.
 
 When done, call `report_usage` once with
-`{ listing_id: "luckiest-video-watcher", skill_version: "1.1.1", matched: true, success: <true|false> }`.
+`{ listing_id: "luckiest-video-watcher", skill_version: "1.1.2", matched: true, success: <true|false> }`.
 Metadata only. Skip silently if unavailable.
 
 ## Standing rules

package/skills/video-watcher/scripts/download.py

@@ -98,7 +98,12 @@ def select_caption(info: dict, language: str = 'auto') -> dict | None:
     manual = {k: v for k, v in (info.get('subtitles') or {}).items() if v and k != 'live_chat'}
     automatic = {k: v for k, v in (info.get('automatic_captions') or {}).items() if v and k != 'live_chat'}
     youtube = str(info.get('extractor_key') or info.get('extractor') or '').lower().startswith('youtube')
-    originals = sorted(k for k in automatic if youtube and k.endswith('-orig'))
+    # Several originals (e.g. bn-orig and en-orig): English first, then the declared language, then alphabetical.
+    declared = str(info.get('language') or '').split('-')[0]
+    def original_rank(key):
+        base = key.removesuffix('-orig').split('-')[0]
+        return base != 'en', base != declared, key
+    originals = sorted((k for k in automatic if youtube and k.endswith('-orig')), key=original_rank)
     original = originals[0][:-5] if len(originals) == 1 else None
     target = language if language != 'auto' else original or info.get('language')
     def matches(key):

package/skills/video-watcher/scripts/test_download.py

@@ -0,0 +1,30 @@
+"""Run: python3 -m unittest scripts/test_download.py (from the skill directory)."""
+import os
+import sys
+import unittest
+
+sys.path.insert(0, os.path.dirname(__file__))
+import download  # noqa: E402
+
+
+def info(language, *keys):
+    return {'extractor_key': 'Youtube', 'language': language,
+            'automatic_captions': {k: [{'ext': 'vtt'}] for k in keys}}
+
+class SelectCaption(unittest.TestCase):
+    def test_several_originals_prefer_english(self):
+        for language in ('en', 'bn', None):
+            track = download.select_caption(info(language, 'bn-orig', 'en-orig', 'bn', 'en'))
+            self.assertEqual(track['key'], 'en-orig', language)
+
+    def test_several_originals_without_english_use_declared_language(self):
+        track = download.select_caption(info('hi', 'bn-orig', 'hi-orig', 'bn', 'hi'))
+        self.assertEqual(track['key'], 'hi-orig')
+
+    def test_single_original_unchanged(self):
+        track = download.select_caption(info(None, 'bn-orig', 'bn', 'en'))
+        self.assertEqual((track['key'], track['provenance']), ('bn-orig', 'original'))
+
+
+if __name__ == '__main__':
+    unittest.main()