jackherd 1.0.2 → 1.0.3

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 What changed for people who use jackherd, newest first.
 
+## 1.0.3
+
+- **Update through the old command.** `agent-kit update`, run after moving from `@crossdelta/agent-kit`, now installs `jh` globally. Before, it found the bundled copy already current and installed nothing, so `jh` stayed off your path.
+
 ## 1.0.2
 
 - **npm.** The package carries keywords, so a search on npm finds it.

package/herdr-plugin/herdr-plugin.toml

@@ -1,6 +1,6 @@
 id = "crossdelta.jackherd"
 name = "jackherd"
-version = "1.0.2"
+version = "1.0.3"
 min_herdr_version = "0.8.2"
 description = "Show the plans of a board's lanes in a herdr pane"
 platforms = ["linux", "macos"]

package/install.sh

@@ -9,7 +9,7 @@ set -eu
 
 # The default is the version this file ships with. The release bump changes it together with
 # package.json; install.test.ts fails while the two differ. Overridable, but only by a plain release.
-JACKHERD_VERSION="${JACKHERD_VERSION:-1.0.2}"
+JACKHERD_VERSION="${JACKHERD_VERSION:-1.0.3}"
 
 JACKHERD_PACKAGE='jackherd'
 # Test seam: the terminal the wizard asks its questions on.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jackherd",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "keywords": [
     "claude-code",
     "coding-agents",

package/src/update/command.ts

@@ -13,7 +13,7 @@ import { defaultUpdateCheck, lockedVersion } from '../sync/if-changed'
 import { locateKit } from '../sync/kit-source'
 import { readInstalledVersion } from '../version/installed-version'
 import { CLI_NAME } from '../version/release'
-import { detectPackageManager, legacyPackageRoot } from './package-manager'
+import { detectPackageManager, legacyPackageRoot, shellHostedGlobalRoot } from './package-manager'
 import { runUpdate, UPDATE_EXIT, type UpdateAdapters } from './update'
 
 const USAGE = `Usage: ${CLI_NAME} update`
@@ -36,15 +36,18 @@ const runInherited = async (argv: readonly string[], cwd: string): Promise<numbe
 const adaptersFor = async (): Promise<UpdateAdapters> => {
   const { version, packageRoot } = await locateKit()
   const home = homedir()
+  const realRoot = realpathSync(packageRoot)
+  const targetRoot = shellHostedGlobalRoot(realRoot) ?? realRoot
   return {
     installedVersion: version,
-    manager: detectPackageManager(realpathSync(packageRoot)),
+    manager: detectPackageManager(realRoot),
     latestVersion: defaultUpdateCheck().latestVersion,
-    installedVersionOnDisk: () => readInstalledVersion(packageRoot),
-    kitCli: [process.execPath, join(packageRoot, 'src', 'cli.ts')],
+    installedVersionOnDisk: () => readInstalledVersion(targetRoot),
+    kitCli: [process.execPath, join(targetRoot, 'src', 'cli.ts')],
     run: runInherited,
     hasGlobalKit: async () => (await lockedVersion(home)) !== null,
-    hasLegacyPackage: () => existsSync(legacyPackageRoot(realpathSync(packageRoot))),
+    hasLegacyPackage: () => existsSync(legacyPackageRoot(targetRoot)),
+    jhOnPath: () => Bun.which(CLI_NAME) !== null,
     home,
     projectRoot: findProjectRoot(process.cwd(), home, existsSync),
     log: (line) => console.log(line),

package/src/update/package-manager.ts

@@ -25,6 +25,17 @@ export const detectPackageManager = (packageRoot: string): PackageManager | null
   return GLOBAL_LOCATIONS.find(([pattern]) => pattern.test(path))?.[1] ?? null
 }
 
+const SHELL_DEPENDENCY = `/${LEGACY_PACKAGE_NAME}/node_modules/${PACKAGE_NAME}`
+
+/**
+ * Where a global jackherd lies when this one sits nested inside the `@crossdelta/agent-kit` shell, as npm lays
+ * it out (beside the shell, in the same global `node_modules`); null for any other layout, bun's flat one included.
+ */
+export const shellHostedGlobalRoot = (packageRoot: string): string | null => {
+  const path = packageRoot.replaceAll('\\', '/')
+  return path.endsWith(SHELL_DEPENDENCY) ? `${path.slice(0, -SHELL_DEPENDENCY.length)}/${PACKAGE_NAME}` : null
+}
+
 /**
  * The argv, never a shell line, that installs `version` globally without running install scripts: the kit
  * has none and a dependency's would run before the new code is looked at. Throws unless `version` is a plain release.

package/src/update/update.ts

@@ -18,6 +18,8 @@ export type UpdateAdapters = {
   hasGlobalKit: () => Promise<boolean>
   /** True when the package before the rename is installed globally beside this one. */
   hasLegacyPackage: () => boolean
+  /** Whether a `jh` command is on the path; false when only the `@crossdelta/agent-kit` shell carries this jackherd. Asked before and after the install. */
+  jhOnPath: () => boolean
   home: string
   /** The project the working directory belongs to, or null. */
   projectRoot: string | null
@@ -57,10 +59,16 @@ const installAndVerify = async (
   return UPDATE_EXIT.error
 }
 
+const doneLine = (adapters: UpdateAdapters, manager: PackageManager, latest: string, jhWasOnPath: boolean): string => {
+  if (jhWasOnPath) return `${CLI_NAME} updated ${adapters.installedVersion} → ${latest}.`
+  if (adapters.jhOnPath()) return `${CLI_NAME} ${latest} installed globally.`
+  return `${CLI_NAME} ${latest} installed, but no ${CLI_NAME} is on your path yet: add the global bin directory of ${manager} to it.`
+}
+
 /**
  * The update itself: installs the newest release, then runs the syncs with the freshly installed code. Does
- * nothing when no newer release exists. The version it
- * installs is the one it checked, and only a plain release reaches the command line.
+ * nothing when no newer release exists and `jh` is on the path; without `jh` it installs even the running
+ * version. The version it installs is the one it checked, and only a plain release reaches the command line.
  */
 const updateToLatest = async (adapters: UpdateAdapters, manager: PackageManager): Promise<number> => {
   const { installedVersion, log } = adapters
@@ -73,7 +81,8 @@ const updateToLatest = async (adapters: UpdateAdapters, manager: PackageManager)
     log(`${CLI_NAME} update: the npm registry answered no plain release version, nothing was changed.`)
     return UPDATE_EXIT.error
   }
-  if (!isNewerVersion(latest, installedVersion)) {
+  const jhWasOnPath = adapters.jhOnPath()
+  if (jhWasOnPath && !isNewerVersion(latest, installedVersion)) {
     log(`${CLI_NAME} ${installedVersion} is the latest release.`)
     return UPDATE_EXIT.ok
   }
@@ -82,7 +91,7 @@ const updateToLatest = async (adapters: UpdateAdapters, manager: PackageManager)
   for (const { argv, cwd } of await syncSteps(adapters)) {
     if ((await adapters.run(argv, cwd)) !== 0) return failed(log, 'the sync', argv.join(' '))
   }
-  log(`${CLI_NAME} updated ${installedVersion} → ${latest}.`)
+  log(doneLine(adapters, manager, latest, jhWasOnPath))
   return UPDATE_EXIT.ok
 }