fdeops 3.9.17 → 3.9.18

package/README.md

@@ -7,7 +7,9 @@
 [![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg)](LICENSE)
 [![Node](https://img.shields.io/badge/node-%3E%3D18-brightgreen)](https://nodejs.org)
 
-**Memory + methodology + skills, in one kit.** Skill packs teach your AI agent how to build. None of them remember who the client is, what was decided, or what's safe to ship. FDEOps adds the missing layer: a private fieldbook per engagement (`.fde/`), a field methodology (land → close), and one `@fde` skill that routes it all. Built for Forward Deployed Engineers, and anyone embedded in client work: consultants, agency developers, solutions architects, fractional CTOs. Feels like a second brain; behaves like a defensible record (dated, sourced, yours).
+**Memory + methodology + skills, in one kit.** Skill packs teach your AI agent how to build. None of them remember who the client is, what was decided, or what's safe to ship. FDEOps adds the missing layer: a private fieldbook per engagement (`.fde/`), a field methodology (land → close), and one `@fde` skill that routes it all. 
+
+Built for Forward Deployed Engineers, and anyone embedded in client work: consultants, agency developers, solutions architects, fractional CTOs. Feels like a second brain; behaves like a defensible record (dated, sourced, yours).
 
 ```
   land      discover      plan      build      ship      close

package/bin/fde.js

@@ -1671,9 +1671,92 @@ function collectDoctorIssues(eng) {
       `${dupes.length} duplicate open-risk cluster(s) (e.g. "${sample}${sample.length >= 60 ? '…' : ''}") - consolidate or retire echoes in risks.md`
     )
   }
+  // Failure-path (exception-led operating map): required once past discover.
+  // Land seeds; discover fills; plan+ without a real break→owner row is wallpaper.
+  if (/^(plan|build|ship|close)$/.test(s.phase) && !hasOperatingMapContent(eng)) {
+    issues.push(
+      `phase is ${s.phase} with empty operating map - fill terrain.md ## Operating map (exception-led): break → who notices → workaround → evidence`
+    )
+  }
+  const aliases = findAmbiguousStakeholders(eng)
+  if (aliases.length) {
+    const sample = aliases[0].forms.slice(0, 3).join(' / ')
+    issues.push(
+      `${aliases.length} stakeholder identity cluster(s) (e.g. "${sample}") - same person under different names? consolidate in stakeholders.md`
+    )
+  }
   return issues
 }
 
+// True when ## Operating map has at least one real exception row (not the empty template).
+function hasOperatingMapContent(eng) {
+  const terrain = stripTemplateNoise(readClean(eng, 'terrain.md'))
+  const body = sectionBody(terrain, 'Operating map')
+  if (!body.trim()) return false
+  const table = parseMdTable(body)
+  if (table) {
+    const exIdx = colIndex(table.headers, /exception|break/i)
+    const idx = exIdx !== -1 ? exIdx : 0
+    for (const row of table.rows) {
+      const cell = (row[idx] || '').trim()
+      if (cell && !/^unknown/i.test(cell)) return true
+    }
+  }
+  for (const raw of body.split('\n')) {
+    const t = raw.trim().replace(/^[-*]\s+/, '')
+    if (!t || t.startsWith('#') || t.startsWith('|') || /^\*\*/.test(t)) continue
+    if (t.length >= 8 && !/^unknown/i.test(t)) return true
+  }
+  return false
+}
+
+// Near-duplicate stakeholder forms sharing a first-name key (Denise vs Denise Chen).
+function findAmbiguousStakeholders(eng) {
+  const forms = []
+  const md = readClean(eng, 'stakeholders.md')
+  const table = parseMdTable(md)
+  if (table) {
+    const nameIdx = colIndex(table.headers, /name|who/i)
+    if (nameIdx !== -1) {
+      for (const row of table.rows) {
+        const name = (row[nameIdx] || '').trim()
+        if (!name || name.length < 2) continue
+        forms.push(name)
+      }
+    }
+  }
+  for (const h of parseSignalHistoryEntries(eng)) {
+    const name = displayNameFromSignalText(h.text)
+    if (name && name.length >= 2 && !/^anon:/i.test(name)) forms.push(name)
+  }
+  const byKey = new Map()
+  for (const name of forms) {
+    const key = signalSubjectKey(name)
+    if (!key || key.startsWith('anon:')) continue
+    const norm = name.replace(/\s+/g, ' ').trim().toLowerCase()
+    if (!byKey.has(key)) byKey.set(key, new Set())
+    byKey.get(key).add(norm)
+  }
+  const clusters = []
+  for (const [key, set] of byKey) {
+    if (set.size < 2) continue
+    // Prefer clusters where forms aren't just identical casing - already lowercased.
+    // Require at least one multi-token form vs a shorter form (Denise / Denise Chen).
+    const list = [...set]
+    const hasLong = list.some(f => f.split(/\s+/).length >= 2)
+    const hasShort = list.some(f => f.split(/\s+/).length === 1)
+    if (hasLong && hasShort) {
+      clusters.push({ key, forms: list })
+      continue
+    }
+    // Or two multi-token forms that share first token but differ later (Denise Chen / Denise C.)
+    if (list.length >= 2 && list.every(f => f.split(/\s+/).length >= 2)) {
+      clusters.push({ key, forms: list })
+    }
+  }
+  return clusters
+}
+
 // Strip template comments / italic *(hints)* so doctor does not treat stubs as filled.
 function stripTemplateNoise(md) {
   return String(md || '')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fdeops",
-  "version": "3.9.17",
+  "version": "3.9.18",
   "description": "Field kit for engineers embedded in client work - a real CLI (recon, memory, portfolio), one @fde skill with field judgment on top, and hooks that make it automatic. Claude Code plugin and any agent that loads skills.",
   "bin": {
     "fdeops": "bin/install.js",

package/skills/fde/SKILL.md

@@ -88,7 +88,7 @@ When NOT to interrogate or challenge: unambiguous one-liners, mechanical ops, FD
 | "Draft the sponsor update" / how are we doing | `fde status` then follow `references/status.md` for the narrative |
 | "Log that they went quiet" / trust signal | `fde log contact "…" --signal amber\|green\|red`. If they already named the color ("log that as amber"), that is the confirm — write it. If they only described the situation, playback the color once, then write. |
 | Want the HTML fieldbook | `fde dashboard` |
-| "Clean up the fieldbook" / hygiene / memory feels messy | `fde doctor` - walk issues in plain language; propose fixes; never auto-rewrite without confirm. Contradictions need judgment (brief vs reality) - doctor is structural; you handle meaning. |
+| "Clean up the fieldbook" / hygiene / memory feels messy | `fde doctor` - walk issues in plain language; propose fixes; never auto-rewrite without confirm. Includes structural gaps: empty operating map (plan+), stakeholder name forks (Denise vs Denise Chen), duplicates, ship/close risks. Contradictions need judgment (brief vs reality) - doctor is structural; you handle meaning. |
 | "Scrub this secret / redact that token" (buried line, not just last write) | `fde redact <term>` preview, then `fde redact <term> --apply` after confirm. Undo is last-write only; redact is for buried lines. Remind them to rotate the real credential. |
 
 **The debrief verb.** Highest-frequency loop. When the FDE shares notes or says "debrief": **you** run the smart path (write notes to a temp file if needed). `--smart` writes a propose file via deterministic heuristics (existing prefixes + light keywords); authentic rambling notes often land mostly in context until **you** rewrite lines with type prefixes. Show the proposed routing in plain language. Only `--apply` (or pipe prefixed lines) after they confirm. Never ask them to run the CLI. Detail: `references/debrief.md`.

package/skills/fde/references/discover.md

@@ -92,7 +92,7 @@ The real spec is what people **do** when the system fails - not what the slide d
 - **The hesitation.** When someone says "well, there's also this other thing we do…" - stop them, ask them to finish. The main story is what they're comfortable explaining; the hesitation is the real problem.
 - **"Which part of the codebase do you least want to touch?"** The answer is unanimous and it's the load-bearing wall. Check it against your churn scan - when the human answer and the churn data agree, that's your first map landmark.
 - **Shadow AI.** Someone pasting data into ChatGPT to cope = a real unmet need + an uncontrolled data risk. Note both.
-- **Exception-led operating map.** For each real break (not the slide-deck process): what fails, who notices first, what they do today, and which artifact is trusted in that moment. Prefer exceptions over happy-path swimlanes — the workaround is the operating system. Write rows under `terrain.md` → `## Operating map (exception-led)`. If the section is missing on an older engagement, add it; never regenerate the rest of terrain. When AI is in play, also fill `## Intelligence placement` (deterministic vs LLM judgement vs human approve).
+- **Exception-led operating map.** For each real break (not the slide-deck process): what fails, who notices first, what they do today, and which artifact is trusted in that moment. Prefer exceptions over happy-path swimlanes — the workaround is the operating system. Write rows under `terrain.md` → `## Operating map (exception-led)`. If the section is missing on an older engagement, add it; never regenerate the rest of terrain. When AI is in play, also fill `## Intelligence placement` (deterministic vs LLM judgement vs human approve). **`fde doctor` requires at least one filled exception row before plan/build/ship/close** — empty map after discover is a hygiene fail, not optional polish.
 
 ## Method - part 3: workshop facilitation
 

package/skills/fde/references/stakeholder-radar.md

@@ -28,6 +28,8 @@ The org chart tells you who reports to whom. The stakeholder radar tells you who
 
 **3. The 48-hour rule.** A stakeholder who goes amber has roughly 48 hours before they go red. A stakeholder who goes red is already escalating above you. Respond same-day to amber signals - not with more delivery, with a conversation.
 
+**3b. One name per person.** If the table says "Denise Chen" and Signal history says "Denise" or "D. Chen", trust keys fork and prep lies. Consolidate to one spelling. `fde doctor` flags these identity clusters — treat that as a fix, not a nit.
+
 **4. Detect the invisible escalation.** Three markers:
 - Questions shift from "what are you building" to "when will it be done" - someone above is asking.
 - A meeting gets shortened or cancelled - they're meeting without you.

package/templates/.fde/stakeholders.md

@@ -1,6 +1,7 @@
 # Stakeholders
 
-<!-- Champions, blockers, veto power. Trust signals: quiet = escalate. -->
+<!-- Champions, blockers, veto power. Trust signals: quiet = escalate.
+     One row per real person - if Denise / Denise Chen / D. Chen appear, consolidate. -->
 
 | Name | Role | Stance | Notes |
 |------|------|--------|-------|
@@ -14,5 +15,6 @@
 <!-- `fde log contact --signal <color>` and `fde debrief` write dated tokens here.
      fde status/receipts read the LATEST dated [signal:...] token in this section
      to drive the trust column - do not delete this heading, and if you rewrite
-     this file as an artifact, keep this section's entries intact. -->
+     this file as an artifact, keep this section's entries intact.
+     Prefer the same Name spelling as the table so trust keys don't fork. -->
 

package/templates/.fde/terrain.md

@@ -8,7 +8,8 @@
 
 ## Operating map (exception-led)
 
-<!-- How work actually runs when the happy path fails. Fill in discover; leave blank until heard/seen. -->
+<!-- How work actually runs when the happy path fails. Fill in discover; leave blank until heard/seen.
+     fde doctor requires ≥1 real exception row before plan/build/ship/close. -->
 
 | Exception / break | Who notices first | What they do today (workaround) | System of record then | Blast if wrong | Evidence |
 |-------------------|-------------------|---------------------------------|-----------------------|----------------|----------|