create-claude-cabinet 0.6.1 → 0.6.3

package/README.md

@@ -30,7 +30,7 @@ curl -fsSL https://raw.githubusercontent.com/orenmagid/claude-cabinet/main/insta
 ```
 
 That's it. If you don't have git or Node.js, it installs them.
-No choices to make — it sets up everything.
+No choices to make — you get everything.
 
 Then open [Claude Code](https://claude.ai/code) in the same folder and
 say `/onboard`. It'll interview you about your project and set everything
@@ -41,8 +41,8 @@ step-by-step walkthrough.
 
 ### For developers
 
-If you have Node.js installed and want interactive module selection,
-database setup, or the full install:
+If you have Node.js installed and want to choose which modules to
+install, or want the lean option (skips work tracking and compliance):
 
 ```bash
 npx create-claude-cabinet
@@ -55,42 +55,93 @@ it's done, open Claude Code and run `/onboard`.
 ## What You Get
 
 ### The Session Loop (always installed)
-- **`/orient`** — session briefing. Reads project state, checks health,
-  surfaces what needs attention. Every session starts informed.
-- **`/debrief`** — session close. Marks work done, records lessons,
-  updates state. Every session closes the loop so the next one starts
-  clean.
-
-### The Cabinet (opt-in)
-20 expert cabinet members who audit your project from their domain:
-**speed-freak** watches performance, **boundary-man** catches edge cases,
-**record-keeper** flags when docs drift from code, **workflow-cop**
+
+This is the foundation. You run these commands — they don't happen
+automatically.
+
+- **`/orient`** — open every session with this. Claude reads project
+  state, checks health, surfaces what needs attention, and briefs you
+  so you never start blind. Think of it as the morning briefing before
+  the cabinet gets to work.
+- **`/debrief`** — close every session with this. Claude marks work
+  done, records lessons, updates state, and prepares the briefing for
+  next time. Without debrief, the next orient starts with stale
+  information. The loop is what gives Claude memory across sessions.
+
+**The habit matters.** Orient and debrief take 30 seconds each. Skip
+them and sessions start from zero — Claude forgets what happened,
+repeats mistakes, and you spend the first 10 minutes re-explaining
+context. Keep the loop and each session picks up where the last one
+left off.
+
+### The Cabinet (included in lean)
+
+20 expert cabinet members who each own a domain and stay in their lane.
+**Speed-freak** watches performance. **Boundary-man** catches edge cases.
+**Record-keeper** flags when docs drift from code. **Workflow-cop**
 evaluates whether your process actually works. Each member has a
-portfolio, stays in it, and produces structured findings you can
-triage. Convene the whole cabinet or just the committee you need.
+portfolio, produces structured findings, and knows when to speak up
+and when to stay quiet.
+
+You convene the cabinet with **`/audit`** — run it occasionally (every
+few sessions, or before a release) to get a full review from every
+relevant member. You don't need to audit every session. The cabinet
+waits until called.
+
+Members are organized into **committees** — groups by concern, so you
+can convene just the experts you need. Security review? Convene the
+security committee. Performance concerns? Just the speed committee.
+
+### Planning + Execution (included in lean)
+
+Don't just start building — brief the cabinet first.
+
+- **`/plan`** — describe what you want to build. Claude drafts a plan,
+  then the relevant cabinet members critique it before a single line is
+  written. The security member notices the missing auth check. The
+  data integrity member catches the NULL handling gap. You approve the
+  plan, and it carries enough detail for any future session to execute
+  without re-exploring.
+- **`/execute`** — pick up an approved plan and build it step by step.
+  Cabinet members watch at each checkpoint. The plan tells Claude what
+  to do; execute makes sure it gets done right.
 
-### Planning + Execution (opt-in)
-- **`/plan`** — structured planning with cabinet critique. Before you
-  build, the relevant members weigh in on your approach.
-- **`/execute`** — step-through execution with checkpoints. Cabinet
-  members watch at each stage.
+### Work Tracking (full install)
 
-### Work Tracking (opt-in)
 Local SQLite database for actions, projects, and status tracking. Claude
 reads and writes it directly — no external service needed. Skip this if
 you already use GitHub Issues, Linear, or something else.
 
-### Compliance Stack (opt-in)
+### Compliance Stack (full install)
+
 Scoped instructions in `.claude/rules/` that load by file path. An
 enforcement pipeline that promotes recurring feedback into deterministic
 hooks — things that keep going wrong become things that can't go wrong.
 
-### Lifecycle (opt-in)
-- **`/onboard`** — conversational project interview, re-runnable as the
-  project matures.
-- **`/seed`** — detects new tech in your project, recruits cabinet
-  members with the right expertise.
-- **`/cc-upgrade`** — conversational merge when Claude Cabinet updates.
+### Lifecycle (included in lean)
+
+- **`/onboard`** — the cabinet's first briefing. Claude interviews you
+  about your project and prepares everything the members need to do
+  their jobs. Re-run it as the project matures — the interview adapts.
+- **`/seed`** — recruit new members. Claude detects new tech in your
+  project and proposes expert members to cover it. Your cabinet grows
+  with your project.
+- **`/cc-upgrade`** — when Claude Cabinet publishes updates, this skill
+  runs the installer for the mechanical parts and walks you through
+  what changed conversationally. Intelligence is the merge strategy.
+
+## Your Workflow
+
+The day-to-day rhythm:
+
+1. **Start a session** → `/orient` (get briefed)
+2. **Do your work** → talk to Claude, use `/plan` for anything non-trivial
+3. **Build it** → `/execute` to implement approved plans with cabinet oversight
+4. **Check quality** → `/audit` occasionally for a full cabinet review
+5. **Close the session** → `/debrief` (close the loop)
+
+Steps 1 and 5 are the minimum. Everything in between is yours to use as
+needed. The more you use, the more the cabinet learns about your project.
 
 ## How It Works
 

package/lib/cli.js

@@ -105,9 +105,10 @@ function detectProjectState(dir) {
   const entries = fs.readdirSync(dir);
   const signals = entries.filter(e => PROJECT_SIGNALS.includes(e));
   const hasClaude = entries.includes('.claude');
-  const hasPibrc = fs.existsSync(path.join(dir, '.ccrc.json'));
+  const hasCcrc = fs.existsSync(path.join(dir, '.ccrc.json'));
+  const hasCorrc = fs.existsSync(path.join(dir, '.corrc.json'));
 
-  if (hasPibrc) return 'existing-install';
+  if (hasCcrc || hasCorrc) return 'existing-install';
   if (signals.length > 0) return 'existing-project';
   // Allow a few dotfiles (e.g. .git) without calling it a project
   if (entries.filter(e => !e.startsWith('.')).length === 0) return 'empty';

package/lib/copy.js

@@ -64,7 +64,9 @@ async function walkAndCopy(srcRoot, destRoot, currentSrc, results, dryRun, skipC
             results.manifest[relPath] = incomingHash;
           } else {
             results.skipped.push(relPath);
-            results.manifest[relPath] = incomingHash;
+            // Record the hash of what's actually on disk, not the template —
+            // otherwise the manifest lies about file content after a skip.
+            results.manifest[relPath] = hashContent(existing);
           }
           continue;
         }

package/lib/metadata.js

@@ -2,6 +2,7 @@ const fs = require('fs');
 const path = require('path');
 
 const METADATA_FILE = '.ccrc.json';
+const LEGACY_METADATA_FILE = '.corrc.json';
 
 function metadataPath(projectDir) {
   return path.join(projectDir, METADATA_FILE);
@@ -9,8 +10,11 @@ function metadataPath(projectDir) {
 
 function read(projectDir) {
   const file = metadataPath(projectDir);
-  if (!fs.existsSync(file)) return null;
-  return JSON.parse(fs.readFileSync(file, 'utf8'));
+  if (fs.existsSync(file)) return JSON.parse(fs.readFileSync(file, 'utf8'));
+  // Fall back to legacy manifest from pre-v0.6.0 installs
+  const legacyFile = path.join(projectDir, LEGACY_METADATA_FILE);
+  if (fs.existsSync(legacyFile)) return JSON.parse(fs.readFileSync(legacyFile, 'utf8'));
+  return null;
 }
 
 function write(projectDir, data) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-claude-cabinet",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "Claude Cabinet — opinionated process scaffolding for Claude Code projects",
   "bin": {
     "create-claude-cabinet": "bin/create-claude-cabinet.js"

package/templates/cabinet/output-contract.md

@@ -84,7 +84,7 @@ these with `"type": "positive"`:
 {
   "id": "{cabinet-member}-p{NNNN}",
   "type": "positive",
-  "perspective": "{cabinet-member-name}",
+  "cabinet-member": "{cabinet-member-name}",
   "severity": "info",
   "title": "Healthy subsystem confirmation",
   "description": "What was checked and found healthy",
@@ -127,7 +127,7 @@ Return valid JSON matching `scripts/finding-schema.json`.
     {
       "id": "{cabinet-member}-{NNNN}",
       "type": "finding",
-      "perspective": "{cabinet-member-name}",
+      "cabinet-member": "{cabinet-member-name}",
       "severity": "critical|warn|info|idea",
       "title": "Short description (max 120 chars)",
       "description": "Full explanation",
@@ -138,7 +138,7 @@ Return valid JSON matching `scripts/finding-schema.json`.
     }
   ],
   "meta": {
-    "perspective": "{cabinet-member-name}",
+    "cabinet-member": "{cabinet-member-name}",
     "timestamp": "ISO-8601"
   }
 }

package/templates/scripts/finding-schema.json

@@ -7,14 +7,14 @@
       "type": "array",
       "items": {
         "type": "object",
-        "required": ["id", "perspective", "severity", "title", "description", "autoFixable"],
+        "required": ["id", "cabinet-member", "severity", "title", "description", "autoFixable"],
         "properties": {
           "id": {
             "type": "string",
             "description": "Unique finding ID: {cabinet-member}-{NNNN}",
             "pattern": "^[a-z-]+-\\d{4}$"
           },
-          "perspective": {
+          "cabinet-member": {
             "type": "string",
             "description": "Cabinet member name matching the directory name in skills/cabinet-*/",
             "pattern": "^[a-z][a-z0-9-]+$"
@@ -81,9 +81,9 @@
     },
     "meta": {
       "type": "object",
-      "required": ["perspective", "timestamp"],
+      "required": ["cabinet-member", "timestamp"],
       "properties": {
-        "perspective": { "type": "string" },
+        "cabinet-member": { "type": "string" },
         "timestamp": { "type": "string", "format": "date-time" },
         "commitHash": { "type": "string" },
         "durationSeconds": { "type": "number" },

package/templates/scripts/load-triage-history.js

@@ -47,12 +47,12 @@ function tryDatabase() {
     const db = new Database(DB_PATH, { readonly: true });
 
     const rejected = db.prepare(`
-      SELECT id, perspective, title FROM audit_findings
+      SELECT id, cabinet_member, title FROM audit_findings
       WHERE triage_status = 'rejected'
     `).all();
 
     const deferred = db.prepare(`
-      SELECT id, perspective, title FROM audit_findings
+      SELECT id, cabinet_member, title FROM audit_findings
       WHERE triage_status = 'deferred'
     `).all();
 
@@ -60,12 +60,12 @@ function tryDatabase() {
 
     result.rejectedIds = rejected.map(r => r.id);
     result.rejectedFingerprints = rejected.map(r => ({
-      perspective: r.perspective,
+      'cabinet-member': r.cabinet_member,
       title: r.title,
     }));
     result.deferredIds = deferred.map(r => r.id);
     result.deferredFingerprints = deferred.map(r => ({
-      perspective: r.perspective,
+      'cabinet-member': r.cabinet_member,
       title: r.title,
     }));
 
@@ -116,7 +116,7 @@ function tryFilesystem() {
           const verdicts = data.verdicts || data;
 
           for (const v of (Array.isArray(verdicts) ? verdicts : [])) {
-            const fp = { perspective: v.perspective, title: v.title };
+            const fp = { 'cabinet-member': v['cabinet-member'] || v.perspective, title: v.title };
 
             if (v.verdict === 'reject' || v.status === 'rejected') {
               if (v.id && !result.rejectedIds.includes(v.id)) {

package/templates/scripts/merge-findings.js

@@ -54,7 +54,7 @@ if (files.length === 0) {
 
 const allFindings = [];
 const seenIds = new Set();
-const perspectiveCounts = {};
+const memberCounts = {};
 const severityCounts = { critical: 0, warn: 0, info: 0, idea: 0 };
 let positiveCount = 0;
 
@@ -62,7 +62,7 @@ for (const file of files) {
   try {
     const data = JSON.parse(readFileSync(join(runDir, file), 'utf-8'));
     const findings = data.findings || [];
-    const perspective = data.meta?.perspective || basename(file, '.json');
+    const member = data.meta?.['cabinet-member'] || basename(file, '.json');
 
     for (const f of findings) {
       if (seenIds.has(f.id)) continue;
@@ -74,11 +74,11 @@ for (const file of files) {
         positiveCount++;
       } else {
         severityCounts[f.severity] = (severityCounts[f.severity] || 0) + 1;
-        perspectiveCounts[perspective] = (perspectiveCounts[perspective] || 0) + 1;
+        memberCounts[member] = (memberCounts[member] || 0) + 1;
       }
     }
 
-    console.log(`  ${perspective}: ${findings.length} findings`);
+    console.log(`  ${member}: ${findings.length} findings`);
   } catch (err) {
     console.error(`  Error reading ${file}: ${err.message}`);
   }
@@ -96,14 +96,14 @@ const summary = {
     runId,
     timestamp,
     trigger: 'manual',
-    perspectives: Object.keys(perspectiveCounts),
+    members: Object.keys(memberCounts),
     counts: {
       total: allFindings.length,
       findings: allFindings.length - positiveCount,
       positive: positiveCount,
       ...severityCounts,
     },
-    byPerspective: perspectiveCounts,
+    byMember: memberCounts,
   },
 };
 

package/templates/scripts/pib-db-schema.sql

@@ -48,7 +48,7 @@ CREATE TABLE IF NOT EXISTS audit_runs (
 CREATE TABLE IF NOT EXISTS audit_findings (
   id                  TEXT PRIMARY KEY,
   run_id              TEXT NOT NULL REFERENCES audit_runs(id),
-  perspective         TEXT NOT NULL,
+  cabinet_member      TEXT NOT NULL,  -- renamed from 'perspective' in v0.7; migration: ALTER TABLE audit_findings RENAME COLUMN perspective TO cabinet_member
   severity            TEXT NOT NULL CHECK(severity IN ('critical','warn','info','idea')),
   title               TEXT NOT NULL,
   description         TEXT,

package/templates/scripts/pib-db.js

@@ -227,7 +227,7 @@ function ingestFindings(runDir) {
 
   const insert = d.prepare(`
     INSERT OR REPLACE INTO audit_findings
-      (id, run_id, perspective, severity, title, description, assumption,
+      (id, run_id, cabinet_member, severity, title, description, assumption,
        evidence, question, file, line, suggested_fix, auto_fixable, type)
     VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
   `);
@@ -235,7 +235,7 @@ function ingestFindings(runDir) {
   let count = 0;
   for (const f of (data.findings || [])) {
     insert.run(
-      f.id, runId, f.perspective, f.severity, f.title,
+      f.id, runId, f['cabinet-member'], f.severity, f.title,
       f.description || null, f.assumption || null, f.evidence || null,
       f.question || null, f.file || null, f.line || null,
       f.suggestedFix || null, f.autoFixable ? 1 : 0, f.type || 'finding'
@@ -262,20 +262,20 @@ function triageHistory() {
   const d = getDb();
 
   const rejected = d.prepare(`
-    SELECT id, perspective, title FROM audit_findings
+    SELECT id, cabinet_member, title FROM audit_findings
     WHERE triage_status = 'rejected'
   `).all();
 
   const deferred = d.prepare(`
-    SELECT id, perspective, title FROM audit_findings
+    SELECT id, cabinet_member, title FROM audit_findings
     WHERE triage_status = 'deferred'
   `).all();
 
   const result = {
     rejectedIds: rejected.map(r => r.id),
-    rejectedFingerprints: rejected.map(r => ({ perspective: r.perspective, title: r.title })),
+    rejectedFingerprints: rejected.map(r => ({ 'cabinet-member': r.cabinet_member, title: r.title })),
     deferredIds: deferred.map(r => r.id),
-    deferredFingerprints: deferred.map(r => ({ perspective: r.perspective, title: r.title })),
+    deferredFingerprints: deferred.map(r => ({ 'cabinet-member': r.cabinet_member, title: r.title })),
   };
   console.log(JSON.stringify(result, null, 2));
   return result;

package/templates/scripts/triage-ui.html

@@ -30,13 +30,13 @@
     .run-info { color: #909296; font-size: 12px; }
 
     /* Perspective groups */
-    .perspective-group {
+    .member-group {
       margin-bottom: 16px;
       border: 1px solid #373a40;
       border-radius: 8px;
       overflow: hidden;
     }
-    .perspective-header {
+    .member-header {
       display: flex;
       align-items: center;
       justify-content: space-between;
@@ -45,21 +45,21 @@
       cursor: pointer;
       user-select: none;
     }
-    .perspective-header:hover { background: #2c2e33; }
-    .perspective-name {
+    .member-header:hover { background: #2c2e33; }
+    .member-name {
       font-weight: 600;
       color: #e9ecef;
       font-size: 14px;
     }
-    .perspective-stats {
+    .member-stats {
       display: flex;
       gap: 8px;
       align-items: center;
       font-size: 12px;
       color: #909296;
     }
-    .perspective-body { padding: 0; }
-    .perspective-body.collapsed { display: none; }
+    .member-body { padding: 0; }
+    .member-body.collapsed { display: none; }
 
     /* Finding rows */
     .finding {
@@ -136,7 +136,7 @@
     .feedback-input::placeholder { color: #5c5f66; }
 
     /* Perspective-level controls */
-    .perspective-controls {
+    .member-controls {
       padding: 8px 14px;
       background: #2c2e33;
       border-top: 1px solid #373a40;
@@ -144,14 +144,14 @@
       flex-direction: column;
       gap: 6px;
     }
-    .perspective-controls-row {
+    .member-controls-row {
       display: flex;
       gap: 8px;
       align-items: center;
       font-size: 12px;
     }
-    .perspective-controls-row label { color: #909296; flex-shrink: 0; }
-    .perspective-input {
+    .member-controls-row label { color: #909296; flex-shrink: 0; }
+    .member-input {
       flex: 1;
       padding: 4px 8px;
       border: 1px solid #373a40;
@@ -160,8 +160,8 @@
       color: #c1c2c5;
       font-size: 12px;
     }
-    .perspective-input::placeholder { color: #5c5f66; }
-    .perspective-q-btn {
+    .member-input::placeholder { color: #5c5f66; }
+    .member-q-btn {
       padding: 3px 10px;
       border: 1px solid #1c7ed6;
       border-radius: 4px;
@@ -172,8 +172,8 @@
       font-weight: 500;
       flex-shrink: 0;
     }
-    .perspective-q-btn:hover { background: #1c7ed622; }
-    .perspective-q-btn.active { background: #1c7ed6; color: #fff; }
+    .member-q-btn:hover { background: #1c7ed622; }
+    .member-q-btn.active { background: #1c7ed6; color: #fff; }
 
     /* Bulk actions */
     .bulk-bar {
@@ -284,7 +284,7 @@
     // ── State ──
     let findings = [];
     const verdicts = {};  // { findingId: { verdict, feedback } }
-    const perspectiveNotes = {};  // { perspective: { comment, question } }
+    const memberNotes = {};  // { member: { comment, question } }
 
     // ── Public API for Claude (javascript_tool) ──
     window.loadFindings = function(data) {
@@ -304,12 +304,12 @@
         total: findings.length,
         triaged: Object.keys(verdicts).length,
         submitted: !!window._submitted,
-        perspectiveNotes: Object.fromEntries(
-          Object.entries(perspectiveNotes).filter(([_, v]) => v.comment || v.question)
+        memberNotes: Object.fromEntries(
+          Object.entries(memberNotes).filter(([_, v]) => v.comment || v.question)
         ),
         verdicts: findings.map(f => ({
           id: f.id,
-          perspective: f.perspective,
+          'cabinet-member': f['cabinet-member'],
           severity: f.severity,
           title: f.title,
           verdict: verdicts[f.id]?.verdict || null,
@@ -347,32 +347,33 @@
       document.getElementById('loading').style.display = 'none';
       document.getElementById('bottom-bar').style.display = 'flex';
 
-      // Group by perspective
+      // Group by cabinet member
       const groups = {};
       findings.forEach(f => {
-        if (!groups[f.perspective]) groups[f.perspective] = [];
-        groups[f.perspective].push(f);
+        const m = f['cabinet-member'];
+        if (!groups[m]) groups[m] = [];
+        groups[m].push(f);
       });
 
       const root = document.getElementById('findings-root');
       root.innerHTML = '';
 
-      // Sort perspectives alphabetically
-      const perspectives = Object.keys(groups).sort();
+      // Sort members alphabetically
+      const members = Object.keys(groups).sort();
 
-      perspectives.forEach(perspective => {
-        const items = groups[perspective];
+      members.forEach(member => {
+        const items = groups[member];
         const triaged = items.filter(f => verdicts[f.id]?.verdict).length;
 
         const group = document.createElement('div');
-        group.className = 'perspective-group';
+        group.className = 'member-group';
 
         // Header
         const header = document.createElement('div');
-        header.className = 'perspective-header';
+        header.className = 'member-header';
         header.innerHTML = `
-          <span class="perspective-name">${perspective} (${items.length})</span>
-          <span class="perspective-stats">
+          <span class="member-name">${member} (${items.length})</span>
+          <span class="member-stats">
             ${triaged}/${items.length} triaged
           </span>
         `;
@@ -383,7 +384,7 @@
 
         // Body
         const body = document.createElement('div');
-        body.className = 'perspective-body';
+        body.className = 'member-body';
 
         items.forEach(f => {
           const row = document.createElement('div');
@@ -413,21 +414,21 @@
         bulk.className = 'bulk-bar';
         bulk.innerHTML = `
           <label>Bulk:</label>
-          <button class="bulk-btn" data-p="${perspective}" data-v="fix">All Fix</button>
-          <button class="bulk-btn" data-p="${perspective}" data-v="defer">All Defer</button>
-          <button class="bulk-btn" data-p="${perspective}" data-v="reject">All Reject</button>
+          <button class="bulk-btn" data-p="${member}" data-v="fix">All Fix</button>
+          <button class="bulk-btn" data-p="${member}" data-v="defer">All Defer</button>
+          <button class="bulk-btn" data-p="${member}" data-v="reject">All Reject</button>
         `;
         body.appendChild(bulk);
 
-        // Perspective-level controls
-        const pn = perspectiveNotes[perspective] || { comment: '', question: false };
+        // Member-level controls
+        const pn = memberNotes[member] || { comment: '', question: false };
         const pControls = document.createElement('div');
-        pControls.className = 'perspective-controls';
+        pControls.className = 'member-controls';
         pControls.innerHTML = `
-          <div class="perspective-controls-row">
-            <button class="perspective-q-btn ${pn.question ? 'active' : ''}" data-perspective="${perspective}">?</button>
-            <input class="perspective-input" data-perspective="${perspective}"
-              placeholder="${pn.question ? 'your question about this perspective...' : 'comment on this perspective...'}"
+          <div class="member-controls-row">
+            <button class="member-q-btn ${pn.question ? 'active' : ''}" data-member="${member}">?</button>
+            <input class="member-input" data-member="${member}"
+              placeholder="${pn.question ? 'your question about this member...' : 'comment on this member...'}"
               value="${esc(pn.comment || '')}">
           </div>
         `;
@@ -479,9 +480,9 @@
       }
       // Bulk buttons
       if (e.target.classList.contains('bulk-btn')) {
-        const perspective = e.target.dataset.p;
+        const member = e.target.dataset.p;
         const v = e.target.dataset.v;
-        findings.filter(f => f.perspective === perspective).forEach(f => {
+        findings.filter(f => f['cabinet-member'] === member).forEach(f => {
           if (!verdicts[f.id]) verdicts[f.id] = { verdict: '', feedback: '' };
           verdicts[f.id].verdict = v;
         });
@@ -490,10 +491,10 @@
     });
 
     document.addEventListener('click', e => {
-      if (e.target.classList.contains('perspective-q-btn')) {
-        const p = e.target.dataset.perspective;
-        if (!perspectiveNotes[p]) perspectiveNotes[p] = { comment: '', question: false };
-        perspectiveNotes[p].question = !perspectiveNotes[p].question;
+      if (e.target.classList.contains('member-q-btn')) {
+        const p = e.target.dataset.member;
+        if (!memberNotes[p]) memberNotes[p] = { comment: '', question: false };
+        memberNotes[p].question = !memberNotes[p].question;
         render();
       }
     });
@@ -504,10 +505,10 @@
         if (!verdicts[id]) verdicts[id] = { verdict: '', feedback: '' };
         verdicts[id].feedback = e.target.value;
       }
-      if (e.target.classList.contains('perspective-input')) {
-        const p = e.target.dataset.perspective;
-        if (!perspectiveNotes[p]) perspectiveNotes[p] = { comment: '', question: false };
-        perspectiveNotes[p].comment = e.target.value;
+      if (e.target.classList.contains('member-input')) {
+        const p = e.target.dataset.member;
+        if (!memberNotes[p]) memberNotes[p] = { comment: '', question: false };
+        memberNotes[p].comment = e.target.value;
       }
     });