@windsland52/maafw-live 0.3.0 → 0.4.0

package/lib/commands/timing.js

@@ -9,11 +9,13 @@
  * 单次采样退化为 P50=P95=该值，与历史单次行为逐字段一致。
  */
 import { EXIT, fail } from '../protocol.js';
-import { computePipelineOverride, mergeOverride } from '../interface/index.js';
+import { computePipelineOverride, appendOverride } from '../interface/index.js';
+import { buildAgentEnv } from '../runtime/pi-env.js';
+import { parseOptionArgs } from '../option-args.js';
 import * as act from '../runtime/actions.js';
 import { withDaemon } from '../runtime/actions.js';
 import { describeSession, ensureSession } from '../runtime/session.js';
-import { CONNECT_OPTIONS, daemonFail, sessionOptions } from './runtime.js';
+import { CONNECT_OPTIONS, daemonFail, argPath, sessionOptions } from './runtime.js';
 /** 最近邻秩百分位：sorted 升序；n=1 时任意 p 返回该值，n 小时偏保守端（P95 取最大） */
 export function percentile(values, p) {
     if (!values.length)
@@ -79,11 +81,14 @@ export const timingCommand = {
         runs: { type: 'string' },
         override: { type: 'string' },
         preset: { type: 'string' },
+        /* 与 run 同一取值通道：timing 测的就是 run 那条路，option 进不去会让
+         * "时长分布"测的是一个根本不该跑的配置（P1-#4）。 */
+        option: { type: 'string', multiple: true },
     },
     async run(ctx) {
-        const o = sessionOptions(ctx.values);
+        const o = sessionOptions(ctx);
         const entry = (typeof ctx.values.entry === 'string' ? ctx.values.entry : ctx.positionals[0]) ?? undefined;
-        const resourceDir = typeof ctx.values['resource-dir'] === 'string' ? ctx.values['resource-dir'] : undefined;
+        const resourceDir = argPath(ctx, typeof ctx.values['resource-dir'] === 'string' ? ctx.values['resource-dir'] : undefined);
         if (!entry)
             return fail('BAD_ARGUMENTS', '缺少入口任务名：--entry <task>', undefined, EXIT.USAGE);
         if (!o.project && !resourceDir) {
@@ -105,8 +110,13 @@ export const timingCommand = {
                 return fail('BAD_ARGUMENTS', '--override 不是合法 JSON', undefined, EXIT.USAGE);
             }
         }
+        /* --option 与 run 同一取值通道（CLI > preset > default_case 由 computePipelineOverride 判） */
+        const parsedOptions = parseOptionArgs(ctx.values.option);
+        if (parsedOptions.error) {
+            return fail('BAD_ARGUMENTS', parsedOptions.error, '例：--option taskOpt=b  --option 章节数.章节号=4', EXIT.USAGE);
+        }
         try {
-            return await withDaemon(async (client) => {
+            return await withDaemon(ctx, async (client) => {
                 const s = await ensureSession(client, o);
                 let args;
                 if (o.project) {
@@ -121,16 +131,24 @@ export const timingCommand = {
                         resourceName: s.plan?.resource?.selected ?? null,
                         taskName: task?.name ?? null,
                         presetName: typeof ctx.values.preset === 'string' ? ctx.values.preset : null,
+                        ...(Object.keys(parsedOptions.cases).length ? { optionCases: parsedOptions.cases } : {}),
+                        ...(Object.keys(parsedOptions.fields).length ? { optionFields: parsedOptions.fields } : {}),
                     });
-                    /* 用户 --override 最后覆盖合成结果：**与 run 共用同一份合并实现**。
-                     * 这里曾经自己展开 fields——字符串会被摊成 {0:'a',1:'b'}、null / 数字静默变空对象，
-                     * 于是同一个 --override 在 timing 与 run 下结果不同。 */
-                    mergeOverride(chain.override, override);
+                    /* 用户 --override 追加到链尾 = 最高优先级：**与 run 共用同一份追加实现**。
+                     * 分层数组交给框架逐层继承；不在这里预合并（层间 type 边界会被摊平毁掉）。 */
+                    appendOverride(chain.override, override);
                     args = {
                         resourceDirs: paths, entry: task?.entry ?? entry,
                         override: chain.override, timeoutMs,
                         agents: s.plan.loaded.agents.map((a) => ({ exec: a.exec, args: a.args })),
                         agentCwd: s.plan.loaded.dir ?? undefined,
+                        /* agent 子进程的环境变量声明按"当前 controller/resource 选择"生成——
+                         * 此前 timing 不带 env，agent 拿到的配置与 run 跑的不是同一套（P1-#4）。 */
+                        agentEnv: buildAgentEnv({
+                            loaded: s.plan.loaded,
+                            controllerName: s.plan?.controllerName ?? null,
+                            resourceName: s.plan?.resource?.selected ?? null,
+                        }),
                     };
                 }
                 else {
@@ -180,7 +198,7 @@ export const timingCommand = {
                 const unsettled = agg.nodeStats.filter((n) => n.nextBeforeStable);
                 const last = samples.at(-1);
                 const human = [
-                    ...describeSession(s),
+                    ...describeSession(s, ctx.flags.verbose),
                     'timing 采样 x' + runs + '（' + agg.successful + ' 成功' + (agg.failed ? ' / ' + agg.failed + ' 失败' : '') + '）：',
                     ...samples.map((smp) => '  #' + smp.index + ' ' + (smp.ok ? '成功' : '失败（不计入建议）') + '  ' + smp.durationMs + 'ms  ' + smp.nodes.length + ' 节点'),
                     '节点时长分布（成功样本，P50 / P95）：',