@stravica-ai/digital-operator-pa 0.1.46 → 0.1.47

package/dist/engineering-pa-core/docs/commands/onboard.md

@@ -94,8 +94,25 @@ line to run; do not run it for them (their shell rc is not our file to write).
 
 Ask if they want the local workspace started now. On yes, run
 `pnpm exec pa workspace start` in the background and print the URL from
-`operator/config/preferences.json:workspacePort` (default 4200). On no, note
-that `pnpm exec pa workspace start` is the verb whenever they want it.
+`operator/config/preferences.json:workspacePort` (default 4200).
+
+If the start fails because the configured port is already in use (the
+operator has something else on 4200, or the previous session did not
+shut down cleanly), do NOT stop the walkthrough. Instead:
+
+1. Pick a free port at or above 4210 (probe 4210, 4211, 4212 ... in
+   order until one is free).
+2. Retry `pnpm exec pa workspace start --port N` with that port.
+3. On success, update `operator/config/preferences.json` with
+   `"workspacePort": N` so subsequent sessions start on the same port
+   without asking again. Preserve the rest of the file exactly, and
+   round-trip through `JSON.parse` / `JSON.stringify` with a two-space
+   indent to keep the on-disk shape stable.
+4. Tell the operator you moved the workspace off 4200 and named the
+   port that stuck.
+
+On no, note that `pnpm exec pa workspace start` is the verb whenever
+they want it.
 
 ### 6. atlassianOffer
 

package/dist/engineering-pa-scaffold/src/prompts.mjs

@@ -25,10 +25,54 @@ export function validatePaName(value) {
   return { ok: true, value: trimmed };
 }
 
-/** Non-empty, 1-80 chars, trimmed. Allows letters / spaces / punctuation. */
+/**
+ * Strip one wrapping pair of straight or typographic quotes around a
+ * string, if present, then trim. Applied to name-class identity fields so
+ * an operator who types their name with wrapping quotes ("Barry" or
+ * U+201CBarryU+201D) never lands a quoted value in preferences.json — the
+ * greeting renderer is faithful, so a quoted store shows up on the home
+ * screen ("Evening, "Barry"").
+ *
+ * Handled pairs (any single pair around the whole value, whitespace
+ * tolerated on the outside):
+ *   - ASCII double quote      U+0022
+ *   - ASCII single quote      U+0027
+ *   - Left/right double quote U+201C U+201D
+ *   - Left/right single quote U+2018 U+2019
+ *   - Left/right guillemet    U+00AB U+00BB
+ *
+ * Only ONE wrapping pair is stripped; a value that is deliberately quoted
+ * for another reason (e.g. `'''`) still surfaces as data. Whitespace
+ * inside the quotes is preserved by the trim after the strip.
+ *
+ * @param {string} value
+ * @returns {string}
+ */
+export function stripWrappingQuotes(value) {
+  if (typeof value !== 'string') return value;
+  let s = value.trim();
+  const pairs = [
+    ['"', '"'],
+    ["'", "'"],
+    ['“', '”'],
+    ['‘', '’'],
+    ['«', '»'],
+  ];
+  for (const [open, close] of pairs) {
+    if (s.length >= 2 && s.startsWith(open) && s.endsWith(close)) {
+      s = s.slice(open.length, s.length - close.length).trim();
+      break;
+    }
+  }
+  return s;
+}
+
+/** Non-empty, 1-80 chars, trimmed, wrapping quotes stripped. Allows letters / spaces / punctuation. */
 export function validateOperatorName(value) {
   if (typeof value !== 'string') return { ok: false, reason: 'must be a string' };
-  const trimmed = value.trim();
+  // Strip wrapping quotes BEFORE the length check so a padded typographic
+  // pair around a short name never trips the max-length ceiling.
+  const trimmed = stripWrappingQuotes(value);
   if (trimmed.length < 1) return { ok: false, reason: 'must not be empty' };
   if (trimmed.length > 80) return { ok: false, reason: 'must be at most 80 characters' };
   return { ok: true, value: trimmed };
@@ -64,7 +108,7 @@ export async function promptValues(seam, defaults) {
   out.operatorName = await ensureValid(
     seam,
     defaults.operatorName,
-    "Operator name (the human this PA serves, e.g. 'Baz'): ",
+    'Operator name (the human this PA serves, e.g. Baz): ',
     validateOperatorName,
   );
 

package/dist/engineering-pa-workspace/src/install-daemon.mjs

@@ -46,6 +46,7 @@ import { homedir, platform } from 'node:os';
 import { dirname, join, resolve } from 'node:path';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import { readPreferencesSync } from '../../engineering-pa-core/src/preferences.mjs';
+import { findRepoRootSync } from './repo-root.mjs';
 import {
   darwinLogBasenames,
   launchdLabelFor,
@@ -60,25 +61,13 @@ import {
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
-// Resolve the operator repo root. In v2 this package is installed at
-// node_modules/@stravica-ai/engineering-pa-workspace/, so walking up three
-// __dirname levels lands inside node_modules/, not at the operator repo
-// root. Precedence:
-//   1. DAVE_REPO_ROOT or PA_REPO_ROOT env (explicit override)
-//   2. Walk up from process.cwd() looking for .git or pnpm-workspace.yaml
-//   3. Fall back to process.cwd() (best-effort; lets dry-run work in odd
-//      checkouts)
-function findRepoRoot(start) {
-  let d = start;
-  while (d && d !== resolve(d, '..')) {
-    if (existsSync(join(d, '.git')) || existsSync(join(d, 'pnpm-workspace.yaml'))) return d;
-    d = resolve(d, '..');
-  }
-  return null;
-}
-
-const envRoot = process.env.DAVE_REPO_ROOT || process.env.PA_REPO_ROOT;
-const REPO_ROOT = envRoot ? resolve(envRoot) : (findRepoRoot(process.cwd()) || process.cwd());
+// Instance-root resolution routed through the shared helper (see
+// `./repo-root.mjs`). Under the published tarball layout, `__dirname`-relative
+// walks land inside `node_modules/@stravica-ai/digital-operator-pa/dist/` and
+// mis-target every derived path; the shared resolver walks up from cwd for
+// `.git`, `operator/config/preferences.json`, or `pnpm-workspace.yaml`, and
+// honours `DAVE_REPO_ROOT` / `PA_REPO_ROOT` overrides.
+const REPO_ROOT = findRepoRootSync();
 const SERVER_SCRIPT = join(__dirname, 'server.mjs');
 
 /**

package/dist/engineering-pa-workspace/src/repo-root.mjs

@@ -0,0 +1,75 @@
+// repo-root.mjs — shared sync resolver for the operator instance root.
+//
+// Under the published tarball layout, `@stravica-ai/engineering-pa-workspace`
+// lives at `node_modules/@stravica-ai/digital-operator-pa/dist/engineering-pa-workspace/`.
+// A `resolve(__dirname, '..', '..', '..')` walk from `src/` therefore lands
+// inside the `digital-operator-pa` package directory (a pnpm-store hard-link),
+// NOT at the operator repo root — silently misdirecting every file the
+// workspace derives from that path (`.ws.pid`, `.ws.log`, preferences overlay
+// reads, `paName` lookups, the server subprocess cwd).
+//
+// The resolver mirrors the precedence the rest of the workspace package
+// already uses (`server.mjs`, `install-daemon.mjs`):
+//
+//   1. `DAVE_REPO_ROOT` or `PA_REPO_ROOT` env override (tests + operator-set).
+//   2. Walk up from `process.cwd()` looking for a marker that identifies an
+//      operator instance root:
+//        - `.git` (present in every scaffolded instance — pa-init runs `git
+//          init` at scaffold time).
+//        - `operator/config/preferences.json` (the canonical instance marker;
+//          matches instances that lost their `.git` for any reason).
+//        - `pnpm-workspace.yaml` (matches monorepo checkouts, e.g. this
+//          engineering-pa-dev repo itself, so package-local tests still work).
+//   3. Fall back to `process.cwd()` so pre-scaffold and dry-run flows still
+//      have a stable value to render.
+//
+// The `startDir` argument is injectable for tests and callers that need to
+// resolve from a directory other than `process.cwd()`. Prod callers use
+// the default.
+
+import { existsSync } from 'node:fs';
+import { join, resolve } from 'node:path';
+
+const MARKERS = [
+  '.git',
+  join('operator', 'config', 'preferences.json'),
+  'pnpm-workspace.yaml',
+];
+
+/**
+ * Walk up from `start` looking for the first directory that contains any
+ * marker in {@link MARKERS}. Returns the directory path, or null when the
+ * walk reaches the filesystem root without a hit.
+ *
+ * @param {string} start
+ * @returns {string|null}
+ */
+export function walkUpForMarker(start) {
+  let d = resolve(start);
+  while (d && d !== resolve(d, '..')) {
+    for (const marker of MARKERS) {
+      if (existsSync(join(d, marker))) return d;
+    }
+    d = resolve(d, '..');
+  }
+  return null;
+}
+
+/**
+ * Resolve the operator instance root. Deterministic and synchronous; safe
+ * to call at module top-level. See the header comment for the full
+ * precedence rule.
+ *
+ * @param {object} [opts]
+ * @param {string} [opts.startDir=process.cwd()] Directory to start the walk from.
+ * @param {NodeJS.ProcessEnv} [opts.env=process.env] Env source (tests inject).
+ * @returns {string} Absolute path.
+ */
+export function findRepoRootSync(opts = {}) {
+  const env = opts.env || process.env;
+  const startDir = opts.startDir || process.cwd();
+  const envRoot = env.DAVE_REPO_ROOT || env.PA_REPO_ROOT;
+  if (envRoot) return resolve(envRoot);
+  const found = walkUpForMarker(startDir);
+  return found || resolve(startDir);
+}

package/dist/engineering-pa-workspace/src/server.mjs

@@ -33,32 +33,18 @@ import { buildAllowedHosts, buildAllowedOrigins, checkRequestGuards, isGuardedPa
 import { readTeamScopeConfigSync } from '../../engineering-pa-core/src/desk/team-scope.mjs';
 import { readPreferencesSync } from '../../engineering-pa-core/src/preferences.mjs';
 import { startPoller as startStatusMonPoller } from './status-mon/poller.mjs';
+import { findRepoRootSync } from './repo-root.mjs';
 
 const __dirname = fileURLToPath(new URL('.', import.meta.url));
 
 // In v2 the workspace package is installed inside the operator repo at
-// `node_modules/@stravica-ai/engineering-pa-workspace/`. The operator repo root
-// is wherever `.git` lives — discovered by walking up from process.cwd(), or
-// taken from DAVE_REPO_ROOT / PA_REPO_ROOT for tests and ephemeral harnesses.
-// The package's own assets live at `../app/` (one level up from src/).
-function findRepoRootSync(start) {
-  let d = start;
-  let prev;
-  while (d && d !== prev) {
-    try {
-      const s = existsSync(join(d, '.git'));
-      if (s) return d;
-    } catch { /* ignore */ }
-    prev = d;
-    d = resolve(d, '..');
-  }
-  return null;
-}
-
-const envRepoRoot = process.env.DAVE_REPO_ROOT || process.env.PA_REPO_ROOT;
-const REPO_ROOT = envRepoRoot
-  ? resolve(envRepoRoot)
-  : (findRepoRootSync(process.cwd()) || process.cwd());
+// `node_modules/@stravica-ai/engineering-pa-workspace/`. The operator repo
+// root is resolved via the shared helper (`./repo-root.mjs`), which walks
+// up from cwd for `.git`, `operator/config/preferences.json`, or
+// `pnpm-workspace.yaml`, and honours `DAVE_REPO_ROOT` / `PA_REPO_ROOT`
+// overrides. The package's own assets live at `../app/` (one level up
+// from src/).
+const REPO_ROOT = findRepoRootSync();
 const APP_DIR = resolve(__dirname, '..', 'app');
 
 // Silent by default. Set WS_DEBUG=1 to re-enable startup banner, file-change

package/dist/engineering-pa-workspace/src/status-mon/run.mjs

@@ -29,6 +29,7 @@ import { fileURLToPath } from 'node:url';
 
 import { readPreferencesSync } from '../../../engineering-pa-core/src/preferences.mjs';
 import { poll, resolveSources, mergeWithPrevious } from './poll.mjs';
+import { findRepoRootSync } from '../repo-root.mjs';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -61,15 +62,15 @@ function parseArgs(argv) {
 }
 
 function findRepoRoot() {
+  // DAVE_ROOT is the historical status-mon env; keep it as a first-class
+  // override so scheduled runs already setting it are unaffected. Otherwise
+  // route through the shared resolver (`../repo-root.mjs`), which walks up
+  // from cwd for the instance markers and also honours DAVE_REPO_ROOT /
+  // PA_REPO_ROOT. The old fallback (`resolve(__dirname, '..', '..', '..')`)
+  // was broken under the installed tarball layout — see
+  // `w-2026-09-17-dave-016`.
   if (process.env.DAVE_ROOT) return process.env.DAVE_ROOT;
-  let dir = __dirname;
-  while (dir !== '/' && dir !== '.') {
-    if (existsSync(join(dir, '.git'))) return dir;
-    const parent = dirname(dir);
-    if (parent === dir) break;
-    dir = parent;
-  }
-  return resolve(__dirname, '..', '..', '..');
+  return findRepoRootSync();
 }
 
 function readPreviousState(statePath) {

package/dist/engineering-pa-workspace/src/ws.mjs

@@ -28,10 +28,20 @@ import {
   restartCommand,
 } from './supervisor.mjs';
 import { resolvePaName } from './unit-name.mjs';
+import { findRepoRootSync } from './repo-root.mjs';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
-const REPO_ROOT = resolve(__dirname, '..', '..', '..');
+// Instance root — NOT `resolve(__dirname, '..', '..', '..')`. Under the
+// published `@stravica-ai/digital-operator-pa` tarball layout the workspace
+// package lives under `dist/engineering-pa-workspace/`, so a three-parent
+// __dirname walk lands inside the pnpm-store hard-linked package directory
+// rather than at the operator instance root. `findRepoRootSync` walks up
+// from `process.cwd()` for `.git` / `operator/config/preferences.json` /
+// `pnpm-workspace.yaml` and honours `DAVE_REPO_ROOT` / `PA_REPO_ROOT`
+// overrides. See `./repo-root.mjs` for the full precedence rule and
+// `w-2026-09-17-dave-016` for the P1 that caused this to be extracted.
+const REPO_ROOT = findRepoRootSync();
 
 const SERVER_SCRIPT = join(__dirname, 'server.mjs');
 // Env overrides let tests point at a temp directory so the real repo's

package/docs/what-it-can-do.md

@@ -43,7 +43,7 @@ The PA is a Digital Operator that runs inside Claude Code on your laptop: it run
 
 - The dispatch-briefer analyses your request and writes an eight-section brief in fresh context, covering boundaries, outputs and completion criteria.
 - Work can go to a single agent or a task group with a coordinator and workers, or stay in the current conversation.
-- Parallel workers use separate git worktrees in the assistant's own repositories; writes to downstream project repositories run sequentially.
+- Parallel workers use separate git worktrees in the operator's own repositories; writes to downstream project repositories run sequentially.
 - A coordinator hook enforces tool-use policy inside Claude Code.
 
 ## The roles it comes with
@@ -74,12 +74,12 @@ The PA is a Digital Operator that runs inside Claude Code on your laptop: it run
 
 - Solo use is the shipped setup; team lead and member modes are not implemented.
 - The remote library is dormant in the default local mode.
-- HQ additions, including specialised roles, the boot-context hook and distillation-health injection, are not shipped.
+- Specialised roles and the session-boot context injections used on the first-party estate are not part of the shipped package.
 - Connections to services through MCP need your own Claude Code configuration.
 - You supply your voice guide, personal writing rules, principles and profile knowledge; the starter content is structure and placeholders.
 - Sharing repositories or memory across machines needs a remote and coordination that you set up.
 
 ## How to start
 
-Open the getting started guide.  
-Follow it to set up the assistant on your laptop.
+Open [docs/onboarding-solo.md](./onboarding-solo.md).
+Follow it to set up your Digital Operator on your laptop.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stravica-ai/digital-operator-pa",
-  "version": "0.1.46",
+  "version": "0.1.47",
   "description": "Digital Operator PA — single-bundled public package surface (bins + library) for the platform.",
   "type": "module",
   "main": "./index.mjs",
@@ -39,11 +39,7 @@
     "LICENSE"
   ],
   "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/Stravica/digital-operator-pa.git",
-    "directory": "packages/engineering-pa"
-  },
+  "homepage": "https://stravica.ai",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"