@dylanpiercey/work 0.0.223 → 0.1.0

package/lib/review/markdown.mjs

@@ -85,16 +85,16 @@ function commentBlock(c, n, repos) {
 }
 
 /** The folder a comment's `worktree` names: a section id (`repo@topic`,
- * `repo#12`) is a view of the folder, and older rows stored the id.
+ * `repo#12`) is a view of the folder, and a row may hold the id.
  * @param {string} worktree */
 function folderOf(worktree) {
   return worktree.split(/[@#]/)[0];
 }
 
-/** The section a comment was made in: an older row names it outright; a
- * newer one by the head it was made at — the first section at that head,
- * so a lane still level with the checkout reads as the checkout, which it
- * is.
+/** The section a comment was made in: a row holding a section id names it
+ * outright; one holding the folder, by the head it was made at — the first
+ * section at that head, so a lane still level with the checkout reads as
+ * the checkout, which it is.
  * @param {Comment} c
  * @param {Repo[] | null | undefined} repos */
 function sectionOf(c, repos) {

package/lib/setup/bridge.d.mts

@@ -0,0 +1,10 @@
+export const BASELINE: number;
+export function bridgeSpec(): string;
+export function bridgeCommands(dataDir: string): string[];
+export function needsBridge(file: string): boolean;
+export function bridgeIfNeeded(
+  dataDir: string,
+  env: Record<string, string | undefined>,
+  log?: (line: string) => void,
+): boolean;
+export function bridgeForServer(dataDir: string): boolean;

package/lib/setup/bridge.mjs

@@ -0,0 +1,84 @@
+// The one step between a 0.0.x database and this release: 0.0.223's
+// `work migrate` runs the sweeps that 0.0.x kept for its own upgrades and
+// stamps schema version 1, the baseline src/server/migrations.ts starts
+// from. `work setup` runs it before touching the service, and the server
+// runs it itself when it opens a database that has not had it (installed
+// without setup, or restarted before setup ran), naming the same commands
+// only if it fails. Everything about crossing into 0.1 is here, so it goes
+// whole once no install is left on 0.0.x. Plain node, no dependencies.
+import { spawnSync } from "node:child_process";
+import fs from "node:fs";
+import { createRequire } from "node:module";
+import path from "node:path";
+
+/** The schema version 0.0.223 stamps: step 1 in src/server/migrations.ts. */
+export const BASELINE = 1;
+
+/** The release that brings a database to the baseline. WORK_BRIDGE_SPEC
+ * names another (an `npm pack` tarball of it) to try the bridge before it
+ * is published. */
+export function bridgeSpec() {
+  return process.env.WORK_BRIDGE_SPEC || "@dylanpiercey/work@0.0.223";
+}
+
+/** What to run by hand for the database in `dataDir`: its data dir named
+ * outright, since the shell running it may not have the service's. */
+export function bridgeCommands(dataDir) {
+  const quoted = /^[\w@%+=:,./-]+$/.test(dataDir)
+    ? dataDir
+    : `'${dataDir.replace(/'/g, `'\\''`)}'`;
+  return [
+    `WORK_DATA_DIR=${quoted} npx -y -p ${bridgeSpec()} work migrate`,
+    "work setup",
+  ];
+}
+
+/** Whether the database at `file` predates the baseline: it has tables and
+ * no version. A missing or empty one is simply new. node:sqlite is loaded
+ * here, not at the top: it warns on load under older node, and every other
+ * CLI command would say so. */
+export function needsBridge(file) {
+  if (!fs.existsSync(file)) return false;
+  const { DatabaseSync } = createRequire(import.meta.url)("node:sqlite");
+  const db = new DatabaseSync(file, { readOnly: true });
+  try {
+    const { user_version } = db.prepare("PRAGMA user_version").get();
+    const tables = db
+      .prepare("SELECT 1 FROM sqlite_master WHERE type = 'table'")
+      .get();
+    return user_version < BASELINE && !!tables;
+  } finally {
+    db.close();
+  }
+}
+
+/** Bring the database in `dataDir` to the baseline when it predates it,
+ * with `env` (the service's) so the sweeps see the same data dir and
+ * workspaces root the server does. True when it is there (or never needed
+ * to be). The package's install scripts are skipped: `migrate` needs none,
+ * and a native build that fails would fail the bridge with it. */
+export function bridgeIfNeeded(dataDir, env, log = () => {}) {
+  const file = path.join(dataDir, "work.db");
+  if (!needsBridge(file)) return true;
+  log(
+    `${file} predates 0.1: bringing it forward once with ${bridgeSpec()} (a one-time download)`,
+  );
+  // The npx beside this node, whatever PATH the service carries.
+  const sibling = path.join(path.dirname(process.execPath), "npx");
+  const npx = fs.existsSync(sibling) ? sibling : "npx";
+  // The package named, then its command: npx runs a spec that is a path
+  // (a tarball) as a program otherwise.
+  spawnSync(npx, ["-y", "-p", bridgeSpec(), "work", "migrate"], {
+    stdio: "inherit",
+    env: { ...env, WORK_DATA_DIR: dataDir, npm_config_ignore_scripts: "true" },
+  });
+  return !needsBridge(file);
+}
+
+/** The server's bridge, for db.ts to hand the migrations: with the server's
+ * own environment, before the database is opened, once per start. */
+export function bridgeForServer(dataDir) {
+  return bridgeIfNeeded(dataDir, process.env, (line) =>
+    console.log(`[work] ${line}`),
+  );
+}

package/lib/setup/doctor.d.mts

@@ -45,6 +45,8 @@ export interface Doctor {
   github: GithubHostStatus[];
   /** Whether the server can keep this Linux host awake; null elsewhere. */
   awake: boolean | null;
+  /** git credential helpers that run a gh which is gone (config keys). */
+  staleGhHelpers: string[];
 }
 export function setupPath(base?: string): string;
 export function which(bin: string): string | null;
@@ -55,6 +57,9 @@ export function miseInstalls(): Promise<Record<
   { version: string; pin?: string }
 > | null>;
 export function ghKnownHosts(): string[];
+export const VERSIONED_PATH: RegExp;
+export function stableGh(): string | null;
+export function staleGhHelpers(): Promise<string[]>;
 export function discoverAgents(fresh?: boolean): DoctorAgent[];
 export function doctor(opts?: { hosts?: string[] }): Promise<Doctor>;
 export function formatDoctor(d: Doctor): string;

package/lib/setup/doctor.mjs

@@ -135,6 +135,43 @@ async function readMiseInstalls() {
   return out;
 }
 
+/** A path owned by one release of a version manager — mise/asdf
+ * `installs`, homebrew's `Cellar`, nvm's `versions` — which that tool
+ * deletes on its next update. */
+export const VERSIONED_PATH = /\/(installs|Cellar|versions|toolchains)\//;
+
+/** The gh for git to run as a credential helper: the launcher on PATH
+ * (mise's shim, /usr/bin/gh), when it is not itself one release's. Still an
+ * absolute path, so it works for GUI clients and cron jobs that never
+ * sourced a shell profile, but one that follows upgrades. */
+export function stableGh() {
+  const gh = which("gh");
+  return gh && !VERSIONED_PATH.test(gh) ? gh : null;
+}
+
+/** Credential helpers in ~/.gitconfig that run a gh which is gone, by
+ * config key. `gh auth setup-git` writes the absolute path of the gh that
+ * ran it, and git runs that string literally: once a version manager's
+ * upgrade deletes it, every https push fails with "could not read
+ * Username" while `gh` itself still works. Listed only when there is a
+ * stable gh to point them at, which `work setup` does. */
+export async function staleGhHelpers() {
+  if (!stableGh()) return [];
+  const { stdout } = await run("git", [
+    "config",
+    "--global",
+    "--get-regexp",
+    String.raw`^credential\..*\.helper$`,
+  ]);
+  const keys = new Set();
+  for (const line of stdout.split("\n")) {
+    const sp = line.indexOf(" ");
+    const bin = /^!(\S+) auth git-credential$/.exec(line.slice(sp + 1))?.[1];
+    if (sp > 0 && bin && !fs.existsSync(bin)) keys.add(line.slice(0, sp));
+  }
+  return [...keys];
+}
+
 /** Hosts `gh` knows about (~/.config/gh/hosts.yml top-level keys) — the
  * source of truth for "which GitHub hosts has this machine signed in to",
  * merged with whatever the caller has configured. */
@@ -338,11 +375,12 @@ export async function doctor(opts = {}) {
     ...(opts.hosts || []),
     ...ghKnownHosts(),
   ]);
-  const [versions, github, awake, extensions] = await Promise.all([
+  const [versions, github, awake, extensions, helpers] = await Promise.all([
     miseVersions(),
     Promise.all([...hostSet].map(githubHostStatus)),
     canStayAwake(),
     ghExtensions(),
+    staleGhHelpers(),
   ]);
 
   const tools = TOOLS.map((t) => {
@@ -390,6 +428,7 @@ export async function doctor(opts = {}) {
     agents,
     github,
     awake,
+    staleGhHelpers: helpers,
   };
 }
 
@@ -423,6 +462,11 @@ export function formatDoctor(d) {
       `sleep      ✗ nothing keeps this host awake: systemd-inhibit is missing or could not take a lock, so the host can suspend under work. Allow org.freedesktop.login1.inhibit-block-idle (a polkit rule), or keep the host up: sudo systemctl mask sleep.target suspend.target`,
     );
   }
+  for (const key of d.staleGhHelpers) {
+    lines.push(
+      `git        ✗ ${key} runs a gh that is gone, so https pushes fail: \`work setup\` points it at the gh on PATH`,
+    );
+  }
   lines.push("");
   lines.push(d.ready ? "ready" : `not ready, missing: ${d.missing.join(", ")}`);
   return lines.join("\n");

package/lib/setup/install.d.mts

@@ -21,3 +21,4 @@ export function installAgent(id: string, opts?: RunOpts): Promise<void>;
 export function authGithub(host: string, opts?: RunOpts): Promise<string[]>;
 export function refreshGithub(host: string, opts?: RunOpts): Promise<string[]>;
 export function logoutGithub(host: string, opts?: RunOpts): Promise<string[]>;
+export function repairGhHelpers(keys: string[], opts?: RunOpts): void;

package/lib/setup/install.mjs

@@ -7,7 +7,7 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { agentById, toolById } from "./catalog.mjs";
-import { setupPath, which } from "./doctor.mjs";
+import { setupPath, stableGh, which } from "./doctor.mjs";
 
 /**
  * Run a command, streaming each output line to `log`. stdout/err are
@@ -270,3 +270,23 @@ export async function logoutGithub(host, opts = {}) {
     env: { GH_PROMPT_DISABLED: "1" },
   });
 }
+
+/** Point credential helpers that run a gh which is gone (`keys`, from
+ * doctor's staleGhHelpers) at the stable one. The value pattern edits only
+ * gh's own entry: `gh auth setup-git` writes an empty helper above it to
+ * reset any inherited ones, and that must survive. */
+export function repairGhHelpers(keys, { log = () => {} } = {}) {
+  const gh = stableGh();
+  if (!gh || !keys.length) return;
+  for (const key of keys) {
+    execFileSync("git", [
+      "config",
+      "--global",
+      "--replace-all",
+      key,
+      `!${gh} auth git-credential`,
+      "auth git-credential",
+    ]);
+  }
+  log(`pointed ${keys.length} git credential helper(s) at ${gh}`);
+}

package/lib/setup/listen.mjs

@@ -1,6 +1,6 @@
 // Where work listens (WORK_LISTEN) and where this machine reaches it: one
-// rule for the CLI (the socket unit, its own requests to the server), the
-// server (the hooks it writes for panes and bare caches) and migrate.mjs.
+// rule for the CLI (the socket unit, its own requests to the server) and
+// the server (the hooks it writes for panes and bare caches).
 
 /** Addresses that mean every interface. */
 const EVERY = new Set(["0.0.0.0", "::", "*"]);

package/lib/share/manifest.mjs

@@ -58,7 +58,8 @@ export function decodeRepo(entry) {
     base = at > 0 ? entry.slice(at + 1) : undefined;
   } else if (entry && typeof entry === "object") {
     const o = /** @type {Record<string, unknown>} */ (entry);
-    // Format-1 rows and API shapes spell the name differently.
+    // A workspace's repo rows (`work workspace --json`, pasted as an
+    // invite) and the API's own bodies spell the name differently.
     full = String(o.repo ?? o.full_name ?? o.fullName ?? "");
     dir = typeof o.dir === "string" ? o.dir : undefined;
     if (typeof o.worktree === "string") dir = o.worktree;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dylanpiercey/work",
-  "version": "0.0.223",
+  "version": "0.1.0",
   "type": "module",
   "description": "Web GUI for agentic dev workspaces",
   "license": "MIT",
@@ -15,8 +15,7 @@
     "dist",
     "bin",
     "lib",
-    "install.sh",
-    "scripts/migrate.mjs"
+    "install.sh"
   ],
   "publishConfig": {
     "access": "public"

package/dist/_LF2Fo5WE.js

@@ -1,2 +0,0 @@
-import { h as notifySessionsChanged } from "./_CpANOzpd.js";
-export { notifySessionsChanged };

package/dist/public/assets/_1OL9vXM-.js

@@ -1 +0,0 @@
-import{$input_workspaceChange as e}from"./_unwtcXFQ.js";export{e as _};

package/dist/public/assets/_5f9fMvWJ.js

@@ -1 +0,0 @@
-import"./_xdrkfok5.js";import{C as e,F as t,G as n,I as r,a as i,b as a,i as o}from"./_BGCK5I_O.js";import"./_C9qO_dny.js";import"./_COECwOVb.js";import"./_qJTL2ooe.js";var s=`_gW _d5`,c=`_gX _d6`,l=`<li><span> </span><!><span><!><!></span></li>`,u=` D D l%b D%b%m`,d=e(0),f=r(6,0,e=>d(e,e._.o)),p=f,m=e(0),h=r(5,0,e=>m(e,e._.n)),g=h,_=r(3,1,e=>{o(e.a,`title`,e._.m||void 0),n(e.b,e._.m||``)}),v=e=>{_._(e),i(e.a,c)},y=r(3,0,e=>n(e.c,e._.l||`Starting…`)),b=e=>{y._(e),i(e.a,[`_cc`,c]),i(e.b,`_2`)},x=(e,t)=>i(e.a,[`_dZ`,`_gU`,t&&`_gV`]);function S(e){i(e.b,[`_M`,s]),i(e.e,`_d7`)}var C=(e,t)=>{o(e.b,`title`,t),n(e.c,t)},w=t(3,`<span><span></span> </span>`,` D b `,b,`<span> </span>`,` D `,v),T=a(11,e=>{w(e,+(e.l==null)),y(e)}),E=t(5,`<!><!><!>`,`b%`,g),D=a(13,e=>{E(e,+!e.n),h(e)}),O=t(6,`<!><!><!>`,`b%`,p),k=a(14,e=>{O(e,+!e.o),f(e)}),A=a(12,_);export{D as a,l as c,x as i,u as l,T as n,C as o,A as r,S as s,k as t};

package/dist/public/assets/_5y69OR6M.js

@@ -1 +0,0 @@
-import{$input_instanceChange as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_B0uJCjmk.js

@@ -1 +0,0 @@
-import{$input_onScopeChange as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_B5TpOQCZ.js

@@ -1 +0,0 @@
-import{$input_filesHref as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_B6UdulKa.js

@@ -1 +0,0 @@
-import{$input_agentsChange as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_BFZHNauV.js

@@ -1 +0,0 @@
-import{$input_findOpen as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_BLNEzyb5.js

@@ -1 +0,0 @@
-import{$input_status as e}from"./_unwtcXFQ.js";export{e as _};

package/dist/public/assets/_BPhXWliU.js

@@ -1 +0,0 @@
-import{$input_onSent as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_BQXTv9Cr.js

@@ -1 +0,0 @@
-import{$input_findOpenChange as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_BaXKdIeb.js

@@ -1 +0,0 @@
-import{$input_filesHref as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_BfOOLKo5.js

@@ -1 +0,0 @@
-import{$input_file as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_BhS8bTnr.js

@@ -1 +0,0 @@
-import{C as e,E as t,S as n,T as r,Y as i,w as a}from"./_xdrkfok5.js";var o=r,s=(e=>`/${e}&`)(t);function c(t){e(t.a,i)}var l=(e,t)=>a(e.a,t),u=(e,t)=>n(e.a,t);export{s as a,o as i,l as n,c as r,u as t};

package/dist/public/assets/_BhZ3ZoG0.js

@@ -1 +0,0 @@
-import{$input_lineChange as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_Bi3_ZsOQ.js

@@ -1 +0,0 @@
-import{$input_workspaceName as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_Bj_Lfa8B.js

@@ -1 +0,0 @@
-import{$input_onLoaded as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_BlN_x04q.js

@@ -1 +0,0 @@
-import{$input_findOpenChange as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_Bn5nsnie.js

@@ -1 +0,0 @@
-import{$input_rev as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_C18Torhh.js

@@ -1 +0,0 @@
-import{$input_setup as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_C3P1VHUI.js

@@ -1 +0,0 @@
-import{$input_treeOpen as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_C6xwTKke.js

@@ -1 +0,0 @@
-import{$input_active as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_C6yqlRfl.js

@@ -1 +0,0 @@
-import{$input_findableChange as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_C75D8bH2.js

@@ -1 +0,0 @@
-import{$setup as e,$template as t,$walks as n}from"./_BJRSndB4.js";var r=[t,n,e];export{r as _};

package/dist/public/assets/_C8Vc_vBH.js

@@ -1 +0,0 @@
-import{$input_agents as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_C9eoo0sE.js

@@ -1 +0,0 @@
-import{$input_fileChange as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_CCie36pi.js

@@ -1 +0,0 @@
-import{$input_pageActive as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CDD9N34p.js

@@ -1 +0,0 @@
-import{$input_workspaceId as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CM5GtJb2.js

@@ -1 +0,0 @@
-import{$input_preferredChange as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_COECwOVb.js

@@ -1 +0,0 @@
-import{C as e,E as t,S as n,T as r,dt as i,w as a}from"./_xdrkfok5.js";var o=r,s=(e=>`/${e}&`)(t);function c(t){e(t.a,i)}var l=(e,t)=>a(e.a,t),u=(e,t)=>n(e.a,t),d=`_cf _9 __`;export{o as a,c as i,u as n,s as o,l as r,d as t};

package/dist/public/assets/_CSQE9nMN.js

@@ -1 +0,0 @@
-import{$input_findTick as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CSXMgHLQ.js

@@ -1 +0,0 @@
-import{$input_findable as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_CVRiByCT.js

@@ -1 +0,0 @@
-import{$input_fileChange as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CVSrm4Pl.js

@@ -1 +0,0 @@
-import{$input_findOpen as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_C_L7l9Mg.js

@@ -1 +0,0 @@
-import{$input_preferred as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_CaMrYy1W.js

@@ -1 +0,0 @@
-import{$input_changesHref as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_Ce-E7sJE.js

@@ -1 +0,0 @@
-import{$input_revealNonce as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_Cg3S760D.js

@@ -1 +0,0 @@
-import{$input_focusNonce as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CicyY7KP.js

@@ -1 +0,0 @@
-import{$input_focusFile as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_Ck0YOw_7.js

@@ -1 +0,0 @@
-import{$input_findTickChange as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CmHlTZSs.js

@@ -1 +0,0 @@
-import{$input_onRevChange as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CnECYh66.js

@@ -1 +0,0 @@
-import{$input_workspaceId as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_CnUVvVPd.js

@@ -1 +0,0 @@
-import{$input_sessions as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_CqSgGCGC.js

@@ -1 +0,0 @@
-import{$input_changesHref as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_CzETFsJs.js

@@ -1 +0,0 @@
-import{$setup as e,$template as t,$walks as n}from"./_BnOOD6nr.js";var r=[t,n,e];export{r as _};

package/dist/public/assets/_D-S9H-tX.js

@@ -1 +0,0 @@
-import{C as e,E as t,S as n,T as r,w as i,z as a}from"./_xdrkfok5.js";var o=r,s=(e=>`/${e}&`)(t);function c(t){e(t.a,a)}var l=(e,t)=>i(e.a,t),u=(e,t)=>n(e.a,t);export{s as a,o as i,l as n,c as r,u as t};

package/dist/public/assets/_D5fTwHXK.js

@@ -1 +0,0 @@
-import{$input_sectionChange as e}from"./_Da2Dmt_d.js";export{e as _};

package/dist/public/assets/_DAH8evhA.js

@@ -1 +0,0 @@
-import{$input_line as e}from"./_BJRSndB4.js";export{e as _};

package/dist/public/assets/_DAemmoHI.js

@@ -1 +0,0 @@
-import{$input_workspaceBranch as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_DEMYxWZf.js

@@ -1 +0,0 @@
-import{$input_onOpenFile as e}from"./_BnOOD6nr.js";export{e as _};

package/dist/public/assets/_DHhv0pCA.js

@@ -1 +0,0 @@
-import{$input_onOpenDiff as e}from"./_BJRSndB4.js";export{e as _};