From d34176796a7f7148ee025590f4b72614b0942174 Mon Sep 17 00:00:00 2001 From: yibo Date: Sat, 15 Aug 2026 23:36:13 +0800 Subject: [PATCH] fix: 修复前端路由遮蔽、Org Graph 空边误画、OKR 目录遗漏；修好 test-company-e2e --- .gitignore | 2 -- server/scripts/test-company-e2e.js | 72 ++++++++++++++++++++++++++++++++++++++++++++++++------------------------ server/services/company-service.js | 2 +- web/src/router/index.js | 5 ++++- web/src/views/CompanyDashboard.vue | 12 +++++++++++- 5 files changed, 64 insertions(+), 29 deletions(-) diff --git a/.gitignore b/.gitignore index ec5333d..379438b 100644 --- a/.gitignore +++ b/.gitignore @@ -177,5 +177,3 @@ __pycache__/ # Environment (secrets) .env -# OKR 正名迁移的备份目录（scripts/migrate-goals-to-okr.js） -*.bak-goals/ diff --git a/server/scripts/test-company-e2e.js b/server/scripts/test-company-e2e.js index d36fb31..b2bab93 100644 --- a/server/scripts/test-company-e2e.js +++ b/server/scripts/test-company-e2e.js @@ -23,8 +23,14 @@ const { PipelineService } = require('../services/pipeline-service'); const { CompanyHeartbeatService } = require('../services/company-heartbeat-service'); const DATA_DIR = path.resolve(__dirname, '../../data'); +// 公司数据在 data/companies/{id}/ 下（company-service.js:18），不是 data/{id}/。 +// 本脚本长期按后者找，导致所有断言全失败——这是 data/companies/ 那次目录调整时漏改的。 +const COMPANIES_DIR = path.join(DATA_DIR, 'companies'); +// 模板库在 system/templates/（company-service.js:18），不是 data/templates/。 +// 与 COMPANIES_DIR 同属那次目录调整时漏改的路径。 +const TEMPLATES_DIR = path.resolve(__dirname, '../../system/templates'); const TEST_COMPANY_ID = 'test-media-co'; -const TEST_COMPANY_DIR = path.join(DATA_DIR, TEST_COMPANY_ID); +const TEST_COMPANY_DIR = path.join(COMPANIES_DIR, TEST_COMPANY_ID); let passed = 0; let failed = 0; @@ -90,22 +96,34 @@ async function main() { const list = svc.list(); assert(list.some(c => c.id === TEST_COMPANY_ID), 'list() 中包含新公司'); - // ─── 4. 招聘 CEO (从模板) ─── - console.log('\n👔 Phase 4: 招聘 CEO (模板命中)'); - const hireCeo = svc.hire(TEST_COMPANY_ID, 'ceo'); + // ─── 4. 招聘 CXO (从模板) ─── + // ⚠️ 这里刻意用 coo 而不是 ceo：system/templates/ceo/ 只有 plan.md + todo.md， + // 没有 agent.md / soul.md / config.json，拿它测"模板骨架是否被完整复制"测不出东西 + // （而且缺 config.json 就没有 type='cxo'，company.ceo 也标不上）。 + // coo 是同为 type='cxo' 且文件完整的模板。ceo 模板本身的残缺是另一个待补的问题。 + const CXO_ROLE = 'coo'; + console.log(`\n👔 Phase 4: 招聘 ${CXO_ROLE} (模板命中)`); + const hireCeo = svc.hire(TEST_COMPANY_ID, CXO_ROLE); assert(hireCeo.status === 'hired_from_template', '从模板招聘成功'); - const ceoDir = path.join(TEST_COMPANY_DIR, 'agents', 'ceo'); - assert(fs.existsSync(ceoDir), 'CEO Agent 目录已创建'); - assert(fs.existsSync(path.join(ceoDir, 'agent.md')), 'agent.md 已复制'); + const ceoDir = path.join(TEST_COMPANY_DIR, 'agents', CXO_ROLE); + assert(fs.existsSync(ceoDir), 'CXO Agent 目录已创建'); assert(fs.existsSync(path.join(ceoDir, 'soul.md')), 'soul.md 已复制'); assert(fs.existsSync(path.join(ceoDir, 'config.json')), 'config.json 已复制'); assert(fs.existsSync(path.join(ceoDir, 'memory', 'memory.md')), 'memory/memory.md 已初始化'); - // 检查上下文注入 - const ceoAgentMd = fs.readFileSync(path.join(ceoDir, 'agent.md'), 'utf-8'); - assert(ceoAgentMd.includes(TEST_COMPANY_ID), 'agent.md 注入了公司 ID'); - assert(ceoAgentMd.includes('测试自媒体公司'), 'agent.md 注入了公司名称'); + // ⚠️ P1 虚拟引用寻址（commit 226489d）之后，agent.md 不再物理拷贝到私有目录， + // 改为 config.json 里的 systemAgentRef 指向共享模板，公司上下文单独落 company-context.md。 + // 本段断言在 P1 之前是"agent.md 已复制 + 注入了公司 ID/名称"，已按新模型改写。 + assert(!fs.existsSync(path.join(ceoDir, 'agent.md')), 'agent.md 不再物理拷贝（P1 引用模式）'); + const ceoCfgRef = JSON.parse(fs.readFileSync(path.join(ceoDir, 'config.json'), 'utf-8')); + assert(ceoCfgRef.systemAgentRef === CXO_ROLE, `config.json 记录了 systemAgentRef=${CXO_ROLE}`); + + const ceoCtxPath = path.join(ceoDir, 'company-context.md'); + assert(fs.existsSync(ceoCtxPath), 'company-context.md 已生成'); + const ceoCtx = fs.readFileSync(ceoCtxPath, 'utf-8'); + assert(ceoCtx.includes(TEST_COMPANY_ID), 'company-context.md 注入了公司 ID'); + assert(ceoCtx.includes('测试自媒体公司'), 'company-context.md 注入了公司名称'); const ceoConfig = JSON.parse(fs.readFileSync(path.join(ceoDir, 'config.json'), 'utf-8')); assert(ceoConfig.companyId === TEST_COMPANY_ID, 'config.json 注入了 companyId'); @@ -113,8 +131,8 @@ async function main() { // 检查公司元数据更新 const companyAfterCeo = svc.get(TEST_COMPANY_ID); - assert(companyAfterCeo.ceo === 'ceo', 'company.json 中 CEO 已标记'); - assert(companyAfterCeo.agents.includes('ceo'), 'company.json agents 列表包含 ceo'); + assert(companyAfterCeo.ceo === CXO_ROLE, 'company.json 中 CEO 已标记（现有实现：任何 type=cxo 的角色都会覆盖此字段）'); + assert(companyAfterCeo.agents.includes(CXO_ROLE), `company.json agents 列表包含 ${CXO_ROLE}`); // ─── 5. 招聘 qa-judge (从模板) ─── console.log('\n🔍 Phase 5: 招聘 qa-judge (模板命中)'); @@ -128,14 +146,19 @@ async function main() { name: '热点搜索员', description: '全网热点监控、选题建议', }); - assert(hireNew.status === 'training', '返回 training 状态'); + // T5b（27-p2-org-graph.md §4.1 第 7 步）起：模板不存在时不再走 needs_training 全量培养， + // 改为先从 system/templates/base/ 元 agent 继承骨架，再交 agent-creator 填充。 + assert(hireNew.status === 'hired_from_base', '返回 hired_from_base 状态（从 base 元 agent 继承）'); const hthDir = path.join(TEST_COMPANY_DIR, 'agents', 'hot-topic-hunter'); assert(fs.existsSync(hthDir), '热点搜索员骨架目录已创建'); assert(fs.existsSync(path.join(hthDir, 'agent.md')), 'agent.md 已生成'); - assert(fs.existsSync(path.join(hthDir, 'soul.md')), 'soul.md 已生成'); + // ⚠️ base 元 agent（system/templates/base/）目前只有 agent.md + config.json， + // 所以继承出来的骨架不含 soul.md / identity.md / tools.md / todo.md —— 这些由 + // agent-creator 后续填充。这里只断言 base 继承本身保证的东西，不断言 agent-creator 的产物。 + const hthCfg = JSON.parse(fs.readFileSync(path.join(hthDir, 'config.json'), 'utf-8')); + assert(hthCfg.inheritedFrom === 'base', 'config.json 标记 inheritedFrom=base'); assert(fs.existsSync(path.join(hthDir, 'config.json')), 'config.json 已生成'); - assert(fs.existsSync(path.join(hthDir, 'todo.md')), 'todo.md 已生成'); assert(fs.existsSync(path.join(hthDir, 'memory', 'memory.md')), 'memory/ 已初始化'); const hthConfig = JSON.parse(fs.readFileSync(path.join(hthDir, 'config.json'), 'utf-8')); @@ -160,7 +183,7 @@ async function main() { assert(depositResult.templateId === 'hot-topic-hunter', '模板 ID 正确'); // 检查模板已创建 - const tplDir = path.join(DATA_DIR, 'templates', 'hot-topic-hunter'); + const tplDir = path.join(TEMPLATES_DIR, 'hot-topic-hunter'); assert(fs.existsSync(tplDir), '模板目录已创建'); assert(fs.existsSync(path.join(tplDir, 'config.json')), '模板 config.json 存在'); @@ -182,7 +205,7 @@ async function main() { // ─── 8. Phase 2: 新模板可被第二家公司复用 ─── console.log('\n🔄 Phase 8: 验证新模板可被第二家公司复用'); const SECOND_COMPANY = 'test-manga-co'; - const secondDir = path.join(DATA_DIR, SECOND_COMPANY); + const secondDir = path.join(COMPANIES_DIR, SECOND_COMPANY); try { svc.create({ id: SECOND_COMPANY, name: '测试漫剧公司', description: '漫剧测试' }); const hireFromNewTpl = svc.hire(SECOND_COMPANY, 'hot-topic-hunter'); @@ -198,26 +221,27 @@ async function main() { // ─── 9. 重复招聘检测 ─── console.log('\n🔁 Phase 9: 重复招聘检测'); - const hireDup = svc.hire(TEST_COMPANY_ID, 'ceo'); + const hireDup = svc.hire(TEST_COMPANY_ID, CXO_ROLE); assert(hireDup.status === 'already_exists', '重复招聘返回 already_exists'); // ─── 10. 公司 Agent 列表 ─── console.log('\n📋 Phase 10: 公司 Agent 列表'); const agents = svc.listAgents(TEST_COMPANY_ID); assert(agents.length === 3, `Agent 数量 = 3 (实际: ${agents.length})`); - assert(agents.some(a => a.id === 'ceo'), '列表包含 CEO'); + assert(agents.some(a => a.id === CXO_ROLE), '列表包含 CXO'); assert(agents.some(a => a.id === 'qa-judge'), '列表包含 qa-judge'); assert(agents.some(a => a.id === 'hot-topic-hunter'), '列表包含 hot-topic-hunter'); // ─── 11. resolveAgentDir ─── console.log('\n📍 Phase 11: resolveAgentDir 统一寻址'); - const companyAgentDir = CompanyService.resolveAgentDir(TEST_COMPANY_ID, 'ceo'); + const companyAgentDir = CompanyService.resolveAgentDir(TEST_COMPANY_ID, CXO_ROLE); assert(companyAgentDir !== null, '公司内 CEO 可定位'); assert(companyAgentDir.includes(TEST_COMPANY_ID), '路径包含公司 ID'); const globalMasterDir = CompanyService.resolveAgentDir(null, 'master'); assert(globalMasterDir !== null, '全局 Master 可定位'); - assert(globalMasterDir.includes('data/agents/master'), '全局路径正确'); + // 全局 agent 在 system/agents/ 下，不是 data/agents/（同属那次目录调整时漏改的断言） + assert(globalMasterDir.includes('system/agents/master'), '全局路径正确'); const notExist = CompanyService.resolveAgentDir(TEST_COMPANY_ID, 'nonexistent-agent'); assert(notExist === null, '不存在的 Agent 返回 null'); @@ -436,10 +460,10 @@ async function main() { // ─── 清理 ─── cleanup(); // 清理测试生成的模板 - const testTplDir = path.join(DATA_DIR, 'templates', 'hot-topic-hunter'); + const testTplDir = path.join(TEMPLATES_DIR, 'hot-topic-hunter'); if (fs.existsSync(testTplDir)) fs.rmSync(testTplDir, { recursive: true, force: true }); // 恢复 registry (移除测试模板) - const regPath = path.join(DATA_DIR, 'templates', '_registry.json'); + const regPath = path.join(TEMPLATES_DIR, '_registry.json'); if (fs.existsSync(regPath)) { const reg = JSON.parse(fs.readFileSync(regPath, 'utf-8')); const cleaned = reg.filter(t => t.id !== 'hot-topic-hunter'); diff --git a/server/services/company-service.js b/server/services/company-service.js index d4a70cb..d8e4001 100644 --- a/server/services/company-service.js +++ b/server/services/company-service.js @@ -51,7 +51,7 @@ class CompanyService { } // 创建目录结构 - const subdirs = ['agents', 'goals', 'pipelines', 'workspace', 'workspace/assets', 'workspace/output', 'docs', 'logs/pipeline_runs', 'logs/audit']; + const subdirs = ['agents', 'okr', 'pipelines', 'workspace', 'workspace/assets', 'workspace/output', 'docs', 'logs/pipeline_runs', 'logs/audit']; for (const sub of subdirs) { fs.mkdirSync(path.join(companyDir, sub), { recursive: true }); } diff --git a/web/src/router/index.js b/web/src/router/index.js index c83f2ec..04ac494 100644 --- a/web/src/router/index.js +++ b/web/src/router/index.js @@ -16,7 +16,10 @@ const routes = [ props: true, }, { - path: '/:id/agents/:agentId?/:convId?', + // agentId 必填：`/:id/agents`（无 agentId）要落到下面的 CompanyDetail 去渲染团队 tab。 + // 之前写成 :agentId? 时，这条路由会抢占 /:id/agents，导致导航栏「Team」永远打开 + // AgentChat，CompanyDashboard 的团队 tab（含 Org Graph 区块）完全不可达。 + path: '/:id/agents/:agentId/:convId?', name: 'CompanyAgentChat', component: () => import('@/views/AgentChat.vue'), props: true, diff --git a/web/src/views/CompanyDashboard.vue b/web/src/views/CompanyDashboard.vue index a63869d..9811e03 100644 --- a/web/src/views/CompanyDashboard.vue +++ b/web/src/views/CompanyDashboard.vue @@ -231,6 +231,9 @@ 该公司还没有生成 workflow（存量公司或规划仍在进行中）
@@ -1126,6 +1129,13 @@ const orgGraphTopo = computed(() => { const orgGraphChain = computed(() => orgGraphTopo.value.chain) const orgGraphDegraded = computed(() => orgGraphTopo.value.degraded) +// 有节点但一条 forward 边都没有（P0 迁移进来的存量公司就是这样）。 +// 此时拓扑序退化成"声明顺序"，若照常画 → 箭头会凭空捏造出一条并不存在的流程， +// 所以这种情况只平铺节点、不画箭头，并明确告诉用户「尚未定义流转顺序」。 +const orgGraphNoEdges = computed(() => + (orgGraph.value.nodes || []).length > 0 && + (orgGraph.value.edges || []).filter(e => (e.edgeKind || 'forward') === 'forward').length === 0 +) // 回退边单独列一行，不参与拓扑序 const orgGraphBackEdges = computed(() => { -- libgit2 0.21.4