kld-sdd 2.7.3 → 2.7.8-2

package/skywalk-sdd/index.cjs

@@ -244,7 +244,7 @@ function findActiveStage(projectRoot, criteria) {
       const matchingStarts = [];
       for (const changeDir of changeDirs) {
         const dir = path.join(eventsDir, changeDir);
-        const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort().reverse();
+        const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0)).reverse();
         for (const file of files) {
           const lines = fs.readFileSync(path.join(dir, file), 'utf-8').split('\n').filter(Boolean);
           for (let i = lines.length - 1; i >= 0; i--) {
@@ -357,7 +357,7 @@ function discoverTaskFiles(projectRoot, changeName) {
   return walkFiles(changeDir, filePath => {
     const fileName = path.basename(filePath).toLowerCase();
     return fileName === 'tasks.md' || fileName === 'task.md';
-  }).sort();
+  }).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
 }
 
 function checklistEntriesOutsideCodeFences(lines) {
@@ -587,7 +587,7 @@ function scanTaskCompletionForArchiveDir(projectRoot, changeName, archiveDir) {
   const taskFiles = walkFiles(archiveDir, filePath => {
     const fileName = path.basename(filePath).toLowerCase();
     return fileName === 'tasks.md' || fileName === 'task.md';
-  }).sort();
+  }).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
   return scanTaskCompletionFromFiles(normalizedRoot, changeName, taskFiles);
 }
 
@@ -796,7 +796,7 @@ function collectSpecFiles(changeRoot) {
   if (!changeRoot || !fs.existsSync(changeRoot)) return [];
   const specsDir = path.join(changeRoot, 'specs');
   if (fs.existsSync(specsDir) && fs.statSync(specsDir).isDirectory()) {
-    return walkFiles(specsDir, filePath => path.basename(filePath).toLowerCase() === 'spec.md').sort();
+    return walkFiles(specsDir, filePath => path.basename(filePath).toLowerCase() === 'spec.md').sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
   }
   const rootSpec = path.join(changeRoot, 'spec.md');
   if (fs.existsSync(rootSpec)) return [rootSpec];
@@ -1313,28 +1313,32 @@ function acquireEventWriteLock(dataDir, changeName, options = {}) {
       created_at: nowISO(),
       change: safeChangeName(changeName),
     };
-    let descriptor;
+    const candidatePath = `${lockPath}.${token}.tmp`;
     try {
-      descriptor = fs.openSync(lockPath, 'wx');
-      fs.writeFileSync(descriptor, JSON.stringify(payload), 'utf8');
-      fs.closeSync(descriptor);
+      fs.writeFileSync(candidatePath, JSON.stringify(payload), { encoding: 'utf8', flag: 'wx', mode: 0o600 });
+      // Publish a complete payload atomically. A contender must never observe an
+      // empty lock and remove it while its owner is still initializing it.
+      fs.linkSync(candidatePath, lockPath);
       return { ...payload, path: lockPath };
     } catch (error) {
-      if (descriptor !== undefined) fs.closeSync(descriptor);
       if (error?.code !== 'EEXIST') throw error;
       let existing = null;
       try {
         existing = JSON.parse(fs.readFileSync(lockPath, 'utf8'));
       } catch {}
       const createdAt = Date.parse(existing?.created_at || '');
-      const stale = !Number.isFinite(createdAt)
-        || Date.now() - createdAt > staleMs
-        || !isProcessAlive(Number(existing?.pid));
+      let age = 0;
+      try { age = Date.now() - fs.statSync(lockPath).mtimeMs; } catch {}
+      const stale = existing && Number.isFinite(createdAt)
+        ? !isProcessAlive(Number(existing.pid))
+        : age > staleMs;
       if (stale) {
         try { fs.rmSync(lockPath, { force: true }); } catch {}
         continue;
       }
       Atomics.wait(EVENT_LOCK_WAIT_BUFFER, 0, 0, 10);
+    } finally {
+      try { fs.unlinkSync(candidatePath); } catch {}
     }
   }
   const error = new Error(`EVENT_WRITE_LOCK_TIMEOUT: 等待 change "${changeName}" 的事件写锁超时`);
@@ -1503,7 +1507,7 @@ function readEventsRaw(dataDir, changeName) {
     if (!fs.existsSync(dir)) return [];
 
     const events = [];
-    const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort();
+    const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
     for (const file of files) {
       const lines = fs.readFileSync(path.join(dir, file), 'utf-8').split('\n').filter(Boolean);
       for (const line of lines) {
@@ -1539,7 +1543,15 @@ function readAllEvents(dataDir) {
 /**
  * 计算单个 change 的四维指标
  */
+function normalizeLifecycleEvents(events) {
+  // Older clients stored run notifications as stage_end with a different event ID.
+  // Preserve the event and its wire kind without counting it as another stage.
+  return events.map(event => event.type === 'stage_end' && event.event_kind === 'run_ended'
+    ? { ...event, type: 'run_end' } : event);
+}
+
 function computeChangeMetrics(changeName, events) {
+  events = normalizeLifecycleEvents(events);
   const changeEvents = events.filter(e => e.change === changeName && !e.orphan);
   if (changeEvents.length === 0) return null;
 
@@ -1868,6 +1880,7 @@ function createReworkBucket(command, capability) {
 }
 
 function summarizeStageExecutions(events) {
+  events = normalizeLifecycleEvents(events);
   const stageEvents = events.filter(e => !e.orphan && (e.type === 'stage_start' || e.type === 'stage_end'));
   const starts = stageEvents
     .filter(e => e.type === 'stage_start')
@@ -3329,6 +3342,7 @@ function computeTelemetryHealthMetrics(events, options = {}) {
 }
 
 function computePdfMvpMetrics(events, options = {}) {
+  events = normalizeLifecycleEvents(events);
   if (options.level === 'capability' || options.capability) {
     return computeCapabilityMetrics(options.capability, events, options);
   }
@@ -4856,10 +4870,11 @@ function tddPairFinding(pair) {
 }
 
 function buildReport(projectRoot, events, options = {}) {
+  events = normalizeLifecycleEvents(events);
   const scopedReport = scopeEventsForChangeReport(events, options.change);
   events = scopedReport.events;
   const archiveEvent = latestByTimestamp(events.filter(e => {
-    return e.command === 'archive' && e.type === 'stage_end';
+    return e.command === 'archive' && e.type === 'stage_end' && e.details?.archive_result;
   }));
   const gitCommitTimeline = resolveGitCommitTimeline(
     events,
@@ -5208,6 +5223,7 @@ function filterEventsByDate(events, dateFrom, dateTo) {
 }
 
 function computeDoctorReport(events, options = {}) {
+  events = normalizeLifecycleEvents(events);
   const scopedEvents = options.change
     ? events.filter(e => e.change === options.change)
     : events;
@@ -5418,12 +5434,12 @@ function cmdStart(args) {
 
 const ONTOLOGY_AUTHORING_STAGES = new Set(['propose', 'spec', 'design', 'task', 'check']);
 
-function syncWorkingOntologyForStage(projectRoot, changeName, command, result) {
+function syncWorkingOntologyForStage(projectRoot, changeName, command, result, verifiedReconciliation = null) {
   if (!ONTOLOGY_AUTHORING_STAGES.has(command) || result === 'failure') return null;
   if (!changeName || changeName === 'general') return null;
   const changeDir = getChangeDir(projectRoot, changeName);
   if (!fs.existsSync(changeDir)) return null;
-  const reconciled = require('./ontology/runtime.cjs').reconcileChange(
+  const reconciled = verifiedReconciliation || require('./ontology/runtime.cjs').reconcileChange(
     projectRoot,
     changeName,
     {
@@ -5475,7 +5491,8 @@ function cmdEnd(args, options = {}) {
     command: args.command,
     agent_type: args.agent || args['agent-type'],
   };
-  const startEvent = eventId
+  // let：下方占位 change 迁移逻辑需要重新赋值 startEvent（const 会在运行时抛 TypeError）
+  let startEvent = eventId
     ? searchEventInDataDir(dataDir, eventId)
     : findActiveStage(projectRoot, activeCriteria);
   if (!eventId && startEvent) {
@@ -5591,7 +5608,8 @@ function cmdEnd(args, options = {}) {
 
   let semanticState = null;
   try {
-    semanticState = syncWorkingOntologyForStage(projectRoot, change, command, result);
+    semanticState = syncWorkingOntologyForStage(projectRoot, change, command, result,
+      command === 'check' ? options.verifiedReconciliation : null);
     if (semanticState) {
       details = {
         ...details,
@@ -5778,7 +5796,7 @@ function cmdEnd(args, options = {}) {
       appendEvent(dataDir, event.change, cleanOptionalFields({
         schema_version: SCHEMA_VERSION,
         event_id: generateEventId(),
-        type: 'stage_end',
+        type: 'run_end',
         event_kind: 'run_ended',
         source: event.source,
         command: 'archive',
@@ -5960,7 +5978,7 @@ function normalizeRepositoryRelativeFiles(files, {
     }
     normalized.push(raw);
   }
-  return [...new Set(normalized)].sort();
+  return [...new Set(normalized)].sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
 }
 
 function normalizeStrictCheckResult(details, result) {
@@ -6412,13 +6430,13 @@ function normalizeStrictTestResult(args, details) {
 
   const taskId = String(args['task-id'] || args.task_id || input.task_id || '').trim();
   const verifiedTaskIds = Array.isArray(input.verified_task_ids)
-    ? [...new Set(input.verified_task_ids.map(value => String(value || '').trim()).filter(Boolean))].sort()
+    ? [...new Set(input.verified_task_ids.map(value => String(value || '').trim()).filter(Boolean))].sort((a, b) => (a < b ? -1 : a > b ? 1 : 0))
     : [];
   if (!taskId && verifiedTaskIds.length === 0) {
     fail('E_STRICT_FIELD_REQUIRED: strict test_result 必须提供 task_id 或 verified_task_ids');
   }
   if (taskId && !verifiedTaskIds.includes(taskId)) verifiedTaskIds.push(taskId);
-  verifiedTaskIds.sort();
+  verifiedTaskIds.sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
 
   const normalized = {
     run_id: requireStrictString(args['run-id'] || args.run_id || input.run_id, 'EVENT_RUN_ID_REQUIRED', 'run_id'),
@@ -6433,8 +6451,10 @@ function normalizeStrictTestResult(args, details) {
     failure_type: failureType,
     expected_failure: expectedFailure,
     evidence_tier: 'strict',
+    ...(input.execution ? { execution: input.execution } : {}),
     valid_red: tddPhase === 'red'
       ? expectedFailure && exitCode !== 0 && ['assertion', 'contract'].includes(failureType)
+        && (!input.execution || (input.counts_known && Number(input.failed) > 0))
       : false,
   };
   if (input.counts_known) {
@@ -6452,6 +6472,7 @@ function isValidCompletionTestResults(testResults) {
   if (!testResults || typeof testResults !== 'object') return false;
   if (!['green', 'refactor', 'regression'].includes(testResults.tdd_phase)) return false;
   if (testResults.exit_code !== 0 || testResults.failure_type !== 'none') return false;
+  if (testResults.execution && (!testResults.counts_known || !(testResults.passed > 0))) return false;
   if (testResults.counts_known === true && Number(testResults.failed) !== 0) return false;
   return true;
 }
@@ -7347,18 +7368,6 @@ function cmdRecord(args) {
   try {
     details = parseJsonOption(args, 'details-json', 'details-file', projectRoot, {});
     detailsFilePath = args['details-file'] || args.details_file || null;
-    // 默认清理 details-file，避免项目根目录被临时 JSON 污染；--keep-details 保留调试用
-    if (detailsFilePath && !args['keep-details'] && !args.keep_details) {
-      try {
-        const fileToRemove = normalizeDetailsFilePath(detailsFilePath, projectRoot);
-        if (fs.existsSync(fileToRemove)) {
-          fs.unlinkSync(fileToRemove);
-        }
-      } catch (cleanupErr) {
-        // 清理失败不阻塞主流程
-        try { console.error(`[telemetry] details-file 清理失败（不阻塞）: ${cleanupErr.message}`); } catch {}
-      }
-    }
   } catch (err) {
     fail(`details JSON 解析失败: ${err.message}`);
   }
@@ -7466,6 +7475,9 @@ function cmdRecord(args) {
     };
   }
   if (strict && type === 'test_result') {
+    if (details?.test_results?.execution) {
+      require('./lib/test-execution.cjs').verifyCapture(details.test_results, projectRoot);
+    }
     const testResults = normalizeStrictTestResult(args, details);
     strictRunId = testResults.run_id;
     strictTaskId = testResults.task_id;
@@ -7559,6 +7571,7 @@ function cmdRecord(args) {
       const taskId = strictTaskId || details?.task_id || null;
       if (taskId) runCheckTaskSync(projectRoot, change, taskId);
     } catch (taskErr) {
+      if (args['require-projection']) throw taskErr;
       try { console.error(`[telemetry] tasks.md 主任务状态自动同步失败（不阻塞）: ${taskErr.message}`); } catch {}
     }
   };
@@ -7635,6 +7648,85 @@ function cmdRecord(args) {
   } catch (error) {
     fail(error.message || String(error));
   }
+  // Keep retry input until validation, persistence and task projection all succeed.
+  if (detailsFilePath && !args['keep-details'] && !args.keep_details) {
+    try { fs.unlinkSync(normalizeDetailsFilePath(detailsFilePath, projectRoot)); }
+    catch (error) {
+      if (error.code !== 'ENOENT') console.error(`[telemetry] details-file 清理失败（不阻塞）: ${error.message}`);
+    }
+  }
+  if (!args.quiet) console.log(JSON.stringify(output, null, 2));
+  return output;
+}
+
+function cmdTestRun(args) {
+  const projectRoot = normalizeProjectRoot(args.project || process.cwd());
+  requireValidChangeName(args.change);
+  requireStrictString(args['session-id'], 'TEST_SESSION_REQUIRED', 'session-id');
+  const completion = args['completion-file']
+    ? JSON.parse(fs.readFileSync(path.resolve(args['completion-file']), 'utf8')) : [];
+  const declared = String(args['task-ids'] || args['task-id'] || '').split(',').map(x => x.trim());
+  if (!Array.isArray(completion) || completion.some(item => !declared.includes(item.task_id)
+      || typeof item.tdd_required !== 'boolean'
+      || (!Array.isArray(item.files) && !item.no_file_change_reason))) {
+    throw new Error('TEST_COMPLETION_INVALID: 完成信息须关联本轮明确任务，并包含实际文件及 tdd_required');
+  }
+  for (const item of completion) {
+    normalizeRepositoryRelativeFiles(item.files || []);
+    runCheckTaskSync(projectRoot, args.change, item.task_id, true);
+  }
+  const captured = require('./lib/test-execution.cjs').captureTest(args, projectRoot, args.change);
+  const record = cmdRecord({ ...args, quiet: true, strict: true, type: 'test_result',
+    command: 'test', project: projectRoot, change: args.change, 'run-id': captured.runId,
+    source: args.source || 'test-run', summary: args.summary || `执行测试：${captured.testResults.command}`,
+    result: captured.testResults.result, details: undefined,
+    'details-file': captured.receipt, 'keep-details': true,
+  });
+  const completed = [];
+  if (isValidCompletionTestResults(captured.testResults)) {
+    for (const item of completion) {
+      const updated = cmdRecord({ ...args, quiet: true, strict: true, type: 'task_update', command: 'apply',
+        'require-projection': true,
+        project: projectRoot, 'run-id': `${captured.runId}:${item.task_id}`, 'task-id': item.task_id,
+        status: 'completed', result: 'success', source: args.source || 'test-run', summary: `${item.task_id} 通过本轮测试`,
+        'details-json': JSON.stringify({ task_update: { ...item, test_event_id: record.event_id } }),
+      });
+      completed.push({ task_id: item.task_id, event_id: updated.event_id, test_event_id: record.event_id });
+    }
+  }
+  const output = { ...record, test_results: captured.testResults, receipt: captured.receipt, completed_tasks: completed };
+  console.log(JSON.stringify(output, null, 2));
+  process.exitCode = captured.testResults.exit_code;
+  return output;
+}
+
+function cmdCheckRecord(args) {
+  const projectRoot = normalizeProjectRoot(args.project || process.cwd());
+  requireValidChangeName(args.change);
+  const preflight = JSON.parse(fs.readFileSync(path.resolve(args['preflight-file']), 'utf8'));
+  const review = JSON.parse(fs.readFileSync(path.resolve(args['review-file']), 'utf8'));
+  const state = require('./ontology/runtime.cjs').reconcileChange(projectRoot, args.change, { profile: preflight.profile || 'auto', markPending: true });
+  if (!preflight.valid || !state.valid || preflight.revision !== state.revision
+      || preflight.review_inputs?.identity_history_hash !== state.state.identity_history_hash
+      || JSON.stringify(preflight.review_inputs?.files) !== JSON.stringify(state.state.files)) {
+    throw new Error('CHECK_REVIEW_STALE: 文档或历史身份已变化，重新预检并评审受影响内容');
+  }
+  const events = readEvents(getDataDir(projectRoot), args.change);
+  const beforeApply = !events.some(e => (e.type === 'stage_start' && ['apply', 'test', 'archive'].includes(e.command)) || ['test_result', 'task_update', 'build_result'].includes(e.type));
+  const previous = events.filter(e => e.type === 'check_result').slice(-1)[0]?.details?.check_results;
+  const assembled = require('./lib/check-review.cjs').assembleReview(review, scanTaskCompletion(projectRoot, args.change), {
+    beforeApply, previousBeforeApply: previous?.fixed_before_apply, revision: preflight.revision,
+  });
+  const record = cmdRecord({ ...args, quiet: true, strict: true, type: 'check_result', command: 'check',
+    project: projectRoot, result: assembled.result, 'run-id': args['run-id'] || generateEventId(),
+    source: args.source || 'opsx-command', summary: args.summary || '按实际评审项生成 Check 结果',
+    'session-id': args['session-id'] || review.reviewer?.parent_session_id || review.reviewer?.review_session_id,
+    'details-json': JSON.stringify(assembled.details),
+  });
+  const ended = args['end-event-id'] ? cmdEnd({ ...args, 'event-id': args['end-event-id'], command: 'check',
+    project: projectRoot, result: assembled.result, summary: args.summary || 'Check 已记录，复核文档状态',
+  }, { silent: true, verifiedReconciliation: state }) : null;
+  const output = { ...record, result: assembled.result, check_results: assembled.details.check_results, stage_end: ended };
   console.log(JSON.stringify(output, null, 2));
   return output;
 }
@@ -7859,7 +7951,7 @@ function searchEventInDataDir(dataDir, eventId) {
 
     for (const changeDir of changeDirs) {
       const dir = path.join(eventsDir, changeDir);
-      const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort().reverse();
+      const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0)).reverse();
       for (const file of files) {
         const lines = fs.readFileSync(path.join(dir, file), 'utf-8').split('\n').filter(Boolean);
         for (const line of lines) {
@@ -7885,7 +7977,7 @@ function readAllEventEndsById(dataDir, eventId, changeName) {
 
   const scanDir = (dir) => {
     if (!fs.existsSync(dir)) return;
-    const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort().reverse();
+    const files = fs.readdirSync(dir).filter(f => f.endsWith('.jsonl')).sort((a, b) => (a < b ? -1 : a > b ? 1 : 0)).reverse();
     for (const file of files) {
       const lines = fs.readFileSync(path.join(dir, file), 'utf-8').split('\n').filter(Boolean);
       for (const line of lines) {
@@ -8174,18 +8266,37 @@ function cmdSemanticScan(args) {
 
 function cmdSemanticReconcile(args, options = {}) {
   const context = semanticCommandContext(args);
+  const started = process.hrtime.bigint();
   const result = require('./ontology/runtime.cjs').reconcileChange(
     context.projectRoot,
     context.changeName,
     { profile: context.profile, markPending: Boolean(options.markPending) },
   );
-  console.log(JSON.stringify(semanticResultPayload(result), null, 2));
+  const payload = semanticResultPayload(result);
+  if (options.includeTasks) {
+    const semanticFinished = process.hrtime.bigint();
+    // A review packet avoids another CLI launch and a parent-agent document scan.
+    // It is fresh input metadata, never a cached approval or a substitute for review.
+    payload.task_completion = scanTaskCompletion(context.projectRoot, context.changeName);
+    payload.review_inputs = {
+      change_dir: path.join(context.projectRoot, 'openspec', 'changes', context.changeName),
+      files: result.state.files,
+      identity_history_hash: result.state.identity_history_hash,
+    };
+    const finished = process.hrtime.bigint();
+    payload.timings_ms = {
+      semantic: Number(semanticFinished - started) / 1e6,
+      tasks: Number(finished - semanticFinished) / 1e6,
+      total: Number(finished - started) / 1e6,
+    };
+  }
+  console.log(JSON.stringify(payload, null, 2));
   if (!result.valid) process.exitCode = 1;
   return result;
 }
 
 function cmdSemanticCheck(args) {
-  return cmdSemanticReconcile(args, { markPending: true });
+  return cmdSemanticReconcile(args, { markPending: true, includeTasks: Boolean(args['include-tasks']) });
 }
 
 function cmdSemanticStatus(args) {
@@ -8404,6 +8515,12 @@ function main() {
     case 'record':
       cmdRecord(flags);
       break;
+    case 'test-run':
+      cmdTestRun(flags);
+      break;
+    case 'check-record':
+      cmdCheckRecord(flags);
+      break;
     case 'metrics':
       cmdMetrics(flags);
       break;
@@ -8476,7 +8593,7 @@ SDD Telemetry CLI - 流程度量采集工具
   node skywalk-sdd/log.cjs semantic-identity --delta-state=modified --entity-id=<uuid> --predecessor-version=<uuid>
   node skywalk-sdd/log.cjs semantic-identity --delta-state=unchanged --entity-id=<uuid> --version-id=<uuid>
   node skywalk-sdd/log.cjs semantic-reconcile --project=<path> --change=<name> [--profile=auto|simple|full|strict]
-  node skywalk-sdd/log.cjs semantic-check --project=<path> --change=<name> [--profile=...]
+  node skywalk-sdd/log.cjs semantic-check --project=<path> --change=<name> [--profile=...] [--include-tasks]
   node skywalk-sdd/log.cjs semantic-status --project=<path> --change=<name>
   node skywalk-sdd/log.cjs semantic-observe --project=<path> --change=<name> [--interval=1500]
 
@@ -8484,6 +8601,8 @@ SDD Telemetry CLI - 流程度量采集工具
   start    记录 SDD 阶段开始，返回 event_id
   end      记录 SDD 阶段结束，关联 event_id
   record   记录 task_update/check_result/test_result 等结构化事件
+  test-run 执行 --command-json 参数数组，采集退出码、测试计数、原始输出与任务关联
+  check-record 从 --review-file 与 --preflight-file 计算并记录 Check 结果
   metrics  查询度量指标（四维分析）
   report   生成只读度量报告（不写入事件）
   doctor   诊断 Telemetry 数据质量
@@ -8494,7 +8613,7 @@ SDD Telemetry CLI - 流程度量采集工具
   semantic-identity 生成或复用实体/版本 ID（8 位十六进制）；无需网络和中央 ID 服务
   semantic-scan 只读解析并校验当前 Change
   semantic-reconcile 全量对账文件与本地工作态本体实例
-  semantic-check 全量对账并在通过时标记 pending
+  semantic-check 全量对账并在通过时标记 pending；--include-tasks 同时返回任务状态、评审输入清单和程序耗时
   semantic-status 读取当前工作态本体实例
   semantic-observe 观察文件变化并同步；不提供 Hook 式控制，Check/Archive 仍会全量对账