Set built JBrowse CLI version to the @jbrowse/core version

Author: Sebastian Benjamin

jbrowse/resources/external/fetch-jbrowse-bundle.mjs

@@ -1,12 +1,31 @@
 import { execSync } from 'node:child_process';
-import { existsSync, mkdirSync, copyFileSync, rmSync } from 'node:fs';
+import { existsSync, mkdirSync, copyFileSync, rmSync, readFileSync } from 'node:fs';
 import { join, resolve } from 'node:path';
 
 const ROOT = resolve('.');
 const BUILD = join(ROOT, 'buildCli');
 const OUTDIR = join(ROOT, 'resources', 'external');
 const OUTFILE = join(OUTDIR, 'jbrowse.js');
 
+function getResolvedCoreVersion() {
+  // Prefer lockfile, fallback to node_modules
+  const lockPath = join(ROOT, 'package-lock.json');
+  if (existsSync(lockPath)) {
+    const lock = JSON.parse(readFileSync(lockPath, 'utf8'));
+    const v =
+      lock?.packages?.['node_modules/@jbrowse/core']?.version ||
+      lock?.dependencies?.['@jbrowse/core']?.version;
+    if (v) return v;
+  }
+
+  const corePkg = join(ROOT, 'node_modules', '@jbrowse', 'core', 'package.json');
+  if (existsSync(corePkg)) {
+    return JSON.parse(readFileSync(corePkg, 'utf8')).version;
+  }
+
+  throw new Error('Could not determine resolved @jbrowse/core version (no lockfile entry and no node_modules).');
+}
+
 async function extractTgz(tgzPath, cwd) {
   try {
     const mod = await import('tar').catch(() => null);
@@ -20,32 +39,46 @@ async function extractTgz(tgzPath, cwd) {
   execSync(`tar -xzf "${tgzPath}" -C "${cwd}"`, { stdio: 'inherit', shell: true });
 }
 
+function findCliEntrypoint(buildDir) {
+  const candidates = [
+    join(buildDir, 'package', 'bundle', 'index.js'),
+    join(buildDir, 'package', 'lib', 'index.js')
+  ];
+
+  for (const p of candidates) {
+    if (existsSync(p)) return p;
+  }
+
+  throw new Error(
+    `Could not find @jbrowse/cli entrypoint. Tried:\n` +
+    candidates.map(c => `  - ${c}`).join('\n')
+  );
+}
+
 async function main() {
   if (existsSync(BUILD)) rmSync(BUILD, { recursive: true, force: true });
   mkdirSync(BUILD, { recursive: true });
   mkdirSync(OUTDIR, { recursive: true });
 
-  console.log('Packing @jbrowse/cli (latest)…');
-  const out = execSync('npm pack @jbrowse/cli', {
+  const coreVersion = getResolvedCoreVersion();
+  const cliSpec = process.env.JBROWSE_CLI_VERSION || coreVersion;
+
+  console.log(`Packing @jbrowse/cli@${cliSpec} (core resolved: ${coreVersion})…`);
+  const out = execSync(`npm pack @jbrowse/cli@${cliSpec}`, {
     cwd: BUILD,
     stdio: ['ignore', 'pipe', 'inherit'],
     shell: true,
-  })
-    .toString()
-    .trim();
+  }).toString().trim();
 
   const tgz = join(BUILD, out);
   console.log(`Downloaded: ${tgz}`);
 
   console.log('Extracting tarball…');
   await extractTgz(tgz, BUILD);
 
-  const bundled = join(BUILD, 'package', 'bundle', 'index.js');
-  if (!existsSync(bundled)) {
-    throw new Error(`bundle/index.js not found at ${bundled}`);
-  }
+  const entry = findCliEntrypoint(BUILD);
 
-  copyFileSync(bundled, OUTFILE);
+  copyFileSync(entry, OUTFILE);
   console.log(`Copied bundle to ${OUTFILE}`);
 
   rmSync(BUILD, { recursive: true, force: true });