feat(company): target_dir 的 API/UI 配置入口（42 号文档 Phase 3）

39 号文档 P5 落地的是"配了 company.json.target_dir 就生效"这一半， "人怎么把这两个字段填进去"此前只能手工编辑 company.json。复用已存在的 PUT /api/company/:companyId（companyService.update()），没有新建端点。新增 target_dir/target_mode 分支，校验直接调 path-resolver.validateLinkedTarget（39 号文档 P5 抽出来的共享函数，和 graph-engine.js 运行时校验同一份判据）：合法路径写入成功、敏感路径黑名单拒绝、linked-new 非空目录拒绝、linked-existing 不存在拒绝，校验失败路由层转 400；传空字符串清空配置；不传 target_mode 默认 linked-existing，与 graph-engine.js 的 readCompanyTargetConfig 同口径。前端：CompanyDashboard.vue 概览 tab 加一张配置卡片，本地表单状态不直接双向绑定 selectedCompany，保存失败不会冲掉用户刚打的字。新增测试 8 例，npm run web:build 编译通过。**未做真实浏览器点击验证**——服务器上有其它并行会话真实在跑的 agent 进程，这次没有重启服务器（42 号文档 §二记过一次"重启杀掉别人真实进程"的教训），文档里标记了这个已知缺口。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BwG4b7Nq1NLR9eeE3M9Us9

feat(company): target_dir 的 API/UI 配置入口（42 号文档 Phase 3）
39 号文档 P5 落地的是"配了 company.json.target_dir 就生效"这一半， "人怎么把这两个字段填进去"此前只能手工编辑 company.json。复用已存在的 PUT /api/company/:companyId（companyService.update()），没有新建端点。新增 target_dir/target_mode 分支，校验直接调 path-resolver.validateLinkedTarget（39 号文档 P5 抽出来的共享函数，和 graph-engine.js 运行时校验同一份判据）：合法路径写入成功、敏感路径黑名单拒绝、linked-new 非空目录拒绝、linked-existing 不存在拒绝，校验失败路由层转 400；传空字符串清空配置；不传 target_mode 默认 linked-existing，与 graph-engine.js 的 readCompanyTargetConfig 同口径。前端：CompanyDashboard.vue 概览 tab 加一张配置卡片，本地表单状态不直接双向绑定 selectedCompany，保存失败不会冲掉用户刚打的字。新增测试 8 例，npm run web:build 编译通过。**未做真实浏览器点击验证**——服务器上有其它并行会话真实在跑的 agent 进程，这次没有重启服务器（42 号文档 §二记过一次"重启杀掉别人真实进程"的教训），文档里标记了这个已知缺口。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BwG4b7Nq1NLR9eeE3M9Us9
yibo
1 parent 9e26bea2
Showing 5 changed files with 265 additions and 4 deletions
docs/todo/system-design/42-org-work-graph-reliability-followups.md
server/routes/company.js
server/services/company-service.js
server/tests/unit/company-target-dir-config.test.js
web/src/views/CompanyDashboard.vue
 # 42. Org/Work Graph 扳撠  39 ﹝
  
-> ****嚗hase 1-2  撌脣嚗2026-08-30嚗hase 3-4  敺撌乓
+> ****嚗hase 1-3  撌脣嚗2026-08-30嚗hase 3 垢蝻箔甈∠撉砲秩hase 4  敺撌乓
 > ****嚗2026-08-30
 > ****嚗39-boss-agent-task-supervision.md](./39-boss-agent-task-supervision.md)  P1-P5 摰甈∠垢蝡舀遣  銝 goal  `ceo`   銝銝芸揪虜laywright 5/5 撉**1469 銝芸蝏選活瘚撠 3 銝芸瘝∟憸**遢﹝甈∪銝撌仿◇摨葵 phase
 > **霂餉◆**嚗葵 phase 膩甈∠嚗瘚雿辣/銵靘摰
@@ -73,7 +73,15 @@ Phase 4嚗imeline.ndjson  planning/reflection 頧刻蕨  蝏扳
  
 ---
  
-## hase 3嚗target_dir`  API/UI 
+## hase 3嚗target_dir`  API/UI    撌脣嚗2026-08-30嚗
+
+**蝏**嚗
+- 垢憭鈭歇摮 `PUT /api/company/:companyId`嚗companyService.update()`嚗瓷撱箇垢挽霈⊿"憒歇摮撠望撅"餈銝凋update()` 憓 `target_dir`/`target_mode` 嚗撉靚 `path-resolver.validateLinkedTarget`嚗39 ﹝ P5 鈭怠嚗 `graph-engine.js` 餈撉隞賢嚗撉仃韐交嚗楝撅 `try/catch` 頧 400嚗target_dir` 隡征摮泵銝脫征蔭嚗 `target_mode` 暺恕 `linked-existing`嚗 `graph-engine.js`  `readCompanyTargetConfig` 敺server/tests/unit/company-target-dir-config.test.js`嚗8 靘linked-existing`/`linked-new` 楝敺恕 mode楝敺linked-new` 征敶linked-existing` 銝征蔭銝嚗
+- 垢嚗CompanyDashboard.vue` "璁"tab 撘敶獢 + 璅∪ + 靽嚗銵典 `selectedCompany`嚗仃韐交銝砲霈拍◤
+
+**撉撘**嚗npm run web:build` 蝻 + 垢瘚楝撅 mock service嚗**瘝⊥撉**銝摰僎銵頝 agent 餈ps`  3 銝 `claude` 摮霈拙垢42 ﹝ 禮鈭 撌脩扇餈甈"鈭箇"悌嚗活銝**餈銝銝芸歇撩嚗""**嚗活蝒嚗瓷僎銵頝摨砲銵乩甈∠撉
+
+---
  
 ****嚗5 " `company.json.target_dir` 撠梁"餈"鈭箸舅銝芸挾憛怨"餈瓷極蝻敶 JSON 辣
  
@@ -191,8 +191,16 @@ router.get(&#39;/:companyId&#39;, (req, res) =&gt; {
 });
  
 // PUT /api/company/:companyId
+// 42 号文档 Phase 3：body 里带 target_dir 时，companyService.update() 会做真实的路径校验
+// （复用 path-resolver.validateLinkedTarget，敏感路径黑名单 / linked-new 非空 / linked-existing
+// 不存在等都在这一步拦，不用等到真正跑任务分配项目根才发现），校验失败抛出，这里接住转 400。
 router.put('/:companyId', (req, res) => {
-  const company = companyService.update(req.params.companyId, req.body);
+  let company;
+  try {
+    company = companyService.update(req.params.companyId, req.body);
+  } catch (err) {
+    return res.status(400).json({ error: err.message });
+  }
   if (!company) return res.status(404).json({ error: '公司不存在' });
   res.json({ success: true, company });
 });
@@ -357,6 +357,26 @@ class CompanyService {
     }
     if (updates.sop !== undefined) company.sop = updates.sop;
     if (updates.heartbeat !== undefined) company.heartbeat = { ...company.heartbeat, ...updates.heartbeat };
+
+    // 42 号文档 Phase 3：target_dir 的配置入口。校验复用 path-resolver 的 validateLinkedTarget
+    // ——39 号文档 P5 落地时就是从这里抽出来的共享函数，graph-engine.js 的 ensureProjectRoot
+    // 用它做运行时校验，这里在配置阶段就先做一遍，不用等第一次真的分配项目根才发现路径不对。
+    // 校验失败直接抛出（调用方 try/catch 转 400），不静默吞掉、不写进一半的脏状态——
+    // 抛出发生在 _saveCompany() 之前，本次 update 里其它字段也不会被落盘（同一次保存，要么全写要么不写）。
+    if (updates.target_dir !== undefined) {
+      if (updates.target_dir === null || updates.target_dir === '') {
+        company.target_dir = null;
+        company.target_mode = null;
+      } else {
+        // 缺省 'linked-existing'：与 graph-engine.js 的 readCompanyTargetConfig 同一条默认值口径，
+        // 两处判据分叉是这个仓库反复栽过的坑（"两套体系不统一"），不能各写一份。
+        const mode = updates.target_mode === 'linked-new' ? 'linked-new' : 'linked-existing';
+        const { validateLinkedTarget } = require('./devloop/path-resolver');
+        company.target_dir = validateLinkedTarget(updates.target_dir, mode);
+        company.target_mode = mode;
+      }
+    }
+
     company.updatedAt = new Date().toISOString();
  
     this._saveCompany(company);
+/**
+ * Unit Tests — PUT /api/company/:companyId 的 target_dir 配置入口（42 号文档 Phase 3）
+ *
+ * 39 号文档 P5 落地的是"配了 company.json.target_dir 就生效"这一半，"人怎么把这两个字段
+ * 填进去"这一半此前只能手工编辑 company.json。这条测试钉住新入口的校验分支——校验复用
+ * path-resolver.validateLinkedTarget（P5 已经抽出来的共享函数），跟 graph-engine.js 运行时
+ * 用的是同一份判据，不重新写一遍。
+ *
+ * 走真入口：把 router 挂到真 express 上 listen(0) 再 fetch，不 mock company-service。
+ */
+import { describe, it, expect, beforeAll, afterAll } from 'vitest';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import express from 'express';
+
+const COMPANIES_DIR = path.resolve(__dirname, '../../../data/companies');
+
+describe('PUT /api/company/:companyId — target_dir 配置（42 号文档 Phase 3）', () => {
+  let server;
+  let base;
+  const createdCompanies = [];
+  const createdDirs = [];
+
+  beforeAll(async () => {
+    const app = express();
+    app.use(express.json());
+    app.use('/api/company', require('../../routes/company'));
+    await new Promise((resolve) => { server = app.listen(0, resolve); });
+    base = `http://127.0.0.1:${server.address().port}/api/company`;
+  });
+
+  afterAll(() => {
+    if (server) server.close();
+    for (const cid of createdCompanies) {
+      fs.rmSync(path.join(COMPANIES_DIR, cid), { recursive: true, force: true });
+    }
+    for (const d of createdDirs) {
+      try { fs.rmSync(d, { recursive: true, force: true }); } catch { /* ignore */ }
+    }
+  });
+
+  function makeCompany(companyId) {
+    const dir = path.join(COMPANIES_DIR, companyId);
+    fs.mkdirSync(dir, { recursive: true });
+    fs.writeFileSync(path.join(dir, 'company.json'), JSON.stringify({ id: companyId, name: companyId }));
+    createdCompanies.push(companyId);
+    return dir;
+  }
+
+  async function put(companyId, body) {
+    const r = await fetch(`${base}/${companyId}`, {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify(body),
+    });
+    return [r.status, await r.json()];
+  }
+
+  it('linked-existing：合法的已存在空目录，写入成功', async () => {
+    const cid = 'target-dir-cfg-existing';
+    makeCompany(cid);
+    const target = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-existing-'));
+    createdDirs.push(target);
+
+    const [status, body] = await put(cid, { target_dir: target, target_mode: 'linked-existing' });
+    expect(status).toBe(200);
+    expect(body.company.target_dir).toBe(path.resolve(target));
+    expect(body.company.target_mode).toBe('linked-existing');
+
+    // 真的落盘了，不是只回了个内存对象
+    const onDisk = JSON.parse(fs.readFileSync(path.join(COMPANIES_DIR, cid, 'company.json'), 'utf8'));
+    expect(onDisk.target_dir).toBe(path.resolve(target));
+  });
+
+  it('linked-new：目录不存在时自动创建，写入成功', async () => {
+    const cid = 'target-dir-cfg-new';
+    makeCompany(cid);
+    const parent = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-new-parent-'));
+    createdDirs.push(parent);
+    const target = path.join(parent, 'brand-new-project');
+
+    expect(fs.existsSync(target)).toBe(false);
+    const [status, body] = await put(cid, { target_dir: target, target_mode: 'linked-new' });
+    expect(status).toBe(200);
+    expect(body.company.target_mode).toBe('linked-new');
+    expect(fs.existsSync(target)).toBe(true); // validateLinkedTarget 的既有副作用：自动建目录
+  });
+
+  it('不传 target_mode 时默认按 linked-existing 处理（与 graph-engine.js 的默认口径一致）', async () => {
+    const cid = 'target-dir-cfg-default-mode';
+    makeCompany(cid);
+    const target = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-default-'));
+    createdDirs.push(target);
+
+    const [status, body] = await put(cid, { target_dir: target });
+    expect(status).toBe(200);
+    expect(body.company.target_mode).toBe('linked-existing');
+  });
+
+  it('敏感路径命中黑名单 → 400，不写入', async () => {
+    const cid = 'target-dir-cfg-sensitive';
+    makeCompany(cid);
+    const sensitive = path.join(os.homedir(), '.ssh', 'target-dir-cfg-should-not-write');
+
+    const [status, body] = await put(cid, { target_dir: sensitive, target_mode: 'linked-existing' });
+    expect(status).toBe(400);
+    expect(body.error).toContain('sensitive_path');
+
+    const onDisk = JSON.parse(fs.readFileSync(path.join(COMPANIES_DIR, cid, 'company.json'), 'utf8'));
+    expect(onDisk.target_dir).toBeUndefined(); // 校验失败，company.json 没有被改动
+  });
+
+  it('linked-new 但目录已存在且非空 → 400，不写入', async () => {
+    const cid = 'target-dir-cfg-new-nonempty';
+    makeCompany(cid);
+    const target = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-nonempty-'));
+    createdDirs.push(target);
+    fs.writeFileSync(path.join(target, 'already-here.txt'), 'x');
+
+    const [status, body] = await put(cid, { target_dir: target, target_mode: 'linked-new' });
+    expect(status).toBe(400);
+    expect(body.error).toContain('not empty');
+  });
+
+  it('linked-existing 但目录不存在 → 400，不写入', async () => {
+    const cid = 'target-dir-cfg-existing-missing';
+    makeCompany(cid);
+    const parent = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-missing-parent-'));
+    createdDirs.push(parent);
+    const target = path.join(parent, 'does-not-exist');
+
+    const [status, body] = await put(cid, { target_dir: target, target_mode: 'linked-existing' });
+    expect(status).toBe(400);
+    expect(body.error).toContain('does not exist');
+  });
+
+  it('target_dir 传空字符串清掉已有配置', async () => {
+    const cid = 'target-dir-cfg-clear';
+    makeCompany(cid);
+    const target = fs.mkdtempSync(path.join(os.tmpdir(), 'target-dir-cfg-clear-'));
+    createdDirs.push(target);
+
+    const [s1, b1] = await put(cid, { target_dir: target, target_mode: 'linked-existing' });
+    expect(s1).toBe(200);
+    expect(b1.company.target_dir).toBeTruthy();
+
+    const [s2, b2] = await put(cid, { target_dir: '' });
+    expect(s2).toBe(200);
+    expect(b2.company.target_dir).toBeNull();
+    expect(b2.company.target_mode).toBeNull();
+  });
+
+  it('公司不存在 → 404', async () => {
+    const [status] = await put('target-dir-cfg-nope', { target_dir: '/tmp/whatever' });
+    expect(status).toBe(404);
+  });
+});
@@ -123,6 +123,44 @@
             </div>
           </div>
  
+          <!-- Target Dir Card（42 号文档 Phase 3：公司级目标代码目录配置） -->
+          <div class="overview-card">
+            <div class="overview-card-header">
+              <h3>📁 目标代码目录</h3>
+            </div>
+            <p style="font-size:13px; color:var(--text-secondary); margin:0 0 12px;">
+              配了之后，这家公司所有任务的交付物代码都固定落在这个目录，不再逐任务分配临时沙盒
+              （类似 Claude Code 选工作目录）。留空则维持默认的沙盒模式。
+            </p>
+            <div class="overview-info-grid" style="grid-template-columns: 1fr;">
+              <div class="info-item">
+                <span class="info-label">目标目录（绝对路径）</span>
+                <input
+                  v-model="targetDirForm.target_dir"
+                  type="text"
+                  placeholder="/Users/you/projects/my-app"
+                  style="padding:6px 10px; border:1px solid var(--border-color); border-radius:6px; font-size:13px; width:100%; box-sizing:border-box;"
+                />
+              </div>
+              <div class="info-item">
+                <span class="info-label">模式</span>
+                <select v-model="targetDirForm.target_mode" style="padding:6px 10px; border:1px solid var(--border-color); border-radius:6px; font-size:13px;">
+                  <option value="linked-existing">linked-existing（接管一个已存在的目录）</option>
+                  <option value="linked-new">linked-new（新建/空目录）</option>
+                </select>
+              </div>
+            </div>
+            <div class="overview-actions">
+              <button class="btn-sm" @click="saveTargetDir" :disabled="isSavingTargetDir">
+                {{ isSavingTargetDir ? '⏳ 保存中...' : '💾 保存' }}
+              </button>
+              <span v-if="targetDirError" style="color:var(--status-failed); font-size:13px;">{{ targetDirError }}</span>
+              <span v-else-if="selectedCompany?.target_dir" style="color:var(--text-secondary); font-size:13px;">
+                当前生效：{{ selectedCompany.target_dir }}（{{ selectedCompany.target_mode }}）
+              </span>
+            </div>
+          </div>
+
           <!-- Agent Status Table -->
           <div class="overview-card agents-card">
             <div class="overview-card-header">
@@ -739,7 +777,7 @@
 </template>
  
 <script setup>
-import { ref, computed, onMounted, watch, onUnmounted } from 'vue'
+import { ref, reactive, computed, onMounted, watch, onUnmounted } from 'vue'
 import { useRouter, useRoute } from 'vue-router'
 import axios from 'axios'
 import AppNav from '@/components/AppNav.vue'
@@ -756,6 +794,11 @@ const companyStatus = ref(null)
 const isStarting = ref(false)
 const isStopping = ref(false)
 const selectedCompany = ref(null)
+// 42 号文档 Phase 3：target_dir 配置表单本地状态，不直接双向绑到 selectedCompany——
+// 保存失败时输入框不该跟着回滚成服务端的旧值，用户还得看着自己刚打的字重打一遍。
+const targetDirForm = reactive({ target_dir: '', target_mode: 'linked-existing' })
+const isSavingTargetDir = ref(false)
+const targetDirError = ref('')
 const companyAgents = ref([])
 const pipelines = ref([])
 const pipelineRuns = ref([])
@@ -972,6 +1015,14 @@ function goToTaskBoard() {
   router.push(`/${selectedCompany.value.id}/task-board`)
 }
  
+// 42 号文档 Phase 3：公司切换 / 首次加载时，把表单同步成服务端当前值——只在 id 变化时同步，
+// 不watch 整个 selectedCompany，否则用户正在改的输入框会被别的字段刷新（比如心跳状态轮询）打断。
+watch(() => selectedCompany.value?.id, () => {
+  targetDirForm.target_dir = selectedCompany.value?.target_dir || ''
+  targetDirForm.target_mode = selectedCompany.value?.target_mode || 'linked-existing'
+  targetDirError.value = ''
+}, { immediate: true })
+
 watch(() => route?.params, async (params) => {
   if (!params) return
   if (route.name === 'CompanyDetail' && params.id) {
@@ -1154,6 +1205,22 @@ async function stopCompany() {
   isStopping.value = false
 }
  
+async function saveTargetDir() {
+  if (!selectedCompany.value) return
+  isSavingTargetDir.value = true
+  targetDirError.value = ''
+  try {
+    const { data } = await axios.put(`/api/company/${selectedCompany.value.id}`, {
+      target_dir: targetDirForm.target_dir,
+      target_mode: targetDirForm.target_mode,
+    })
+    selectedCompany.value = data.company
+  } catch (e) {
+    targetDirError.value = e.response?.data?.error || e.message
+  }
+  isSavingTargetDir.value = false
+}
+
 async function deleteCompany(companyId) {
   if (!confirm('确认要删除该公司吗？该操作仅会从列表中移除公司声明。')) return
   try {