@zhushanwen/pi-subagent-workflow 8.6.0 → 8.7.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +9 -13
- package/relay/relay.mjs +1 -1
- package/src/host/__tests__/pi-host.test.ts +214 -0
- package/src/host/pi-host.ts +109 -0
- package/src/index.ts +89 -32
- package/src/injectors/__tests__/engine-awareness.test.ts +292 -0
- package/src/injectors/__tests__/engine-section-stability.test.ts +341 -0
- package/src/injectors/__tests__/subagent-list-injector.test.ts +5 -5
- package/src/injectors/__tests__/workflow-list-injector.test.ts +5 -5
- package/src/injectors/engine-awareness.ts +123 -0
- package/src/injectors/model-list-injector.ts +1 -1
- package/src/injectors/subagent-list-injector.ts +8 -8
- package/src/injectors/workflow-list-injector.ts +8 -8
- package/src/interface/__tests__/detectors.test.ts +3 -3
- package/src/interface/__tests__/subagent-tool-path-guard.test.ts +2 -2
- package/src/interface/__tests__/tool-workflow-script-generate.test.ts +2 -2
- package/src/interface/__tests__/tool-workflow-throw-paths.test.ts +1 -1
- package/src/interface/bg-notify-render.ts +4 -4
- package/src/interface/commands.ts +3 -3
- package/src/interface/format.ts +2 -2
- package/src/interface/helpers.ts +1 -1
- package/src/interface/list-component.ts +4 -4
- package/src/interface/list-shared.ts +2 -2
- package/src/interface/list-view.ts +2 -2
- package/src/interface/subagent-actions.ts +263 -11
- package/src/interface/subagent-tool-schema.ts +20 -13
- package/src/interface/subagent-tool.ts +11 -6
- package/src/interface/subagents.ts +3 -3
- package/src/interface/tool-render.ts +34 -2
- package/src/interface/tool-workflow-script.ts +4 -4
- package/src/interface/tool-workflow.ts +6 -6
- package/src/interface/views/WorkflowsView.ts +5 -5
- package/src/interface/views/__tests__/WorkflowsView-signature.test.ts +3 -3
- package/src/interface/views/__tests__/detail-content-session-file.test.ts +2 -2
- package/src/interface/views/detail-content.ts +4 -4
- package/src/interface/views/format.ts +4 -4
- package/src/{orchestration/jsonl-run-store.ts → jsonl-run-store.ts} +11 -11
- package/src/execution/__tests__/__fixtures__/notifier-golden-snapshots.json +0 -53
- package/src/execution/__tests__/__fixtures__/truncline.snapshot.json +0 -1
- package/src/execution/__tests__/agent-registry.test.ts +0 -363
- package/src/execution/__tests__/agent-result-mapper.test.ts +0 -150
- package/src/execution/__tests__/alive-store.test.ts +0 -147
- package/src/execution/__tests__/ask-user-transit-e2e.test.ts +0 -490
- package/src/execution/__tests__/before-agent-start-injection.test.ts +0 -132
- package/src/execution/__tests__/bg-notify-render.test.ts +0 -329
- package/src/execution/__tests__/channel-registry-handshake.test.ts +0 -243
- package/src/execution/__tests__/chat-engine-routing.test.ts +0 -601
- package/src/execution/__tests__/chatmode-first-round-closure-service.test.ts +0 -365
- package/src/execution/__tests__/chatmode-first-round-closure-spawn.test.ts +0 -190
- package/src/execution/__tests__/chatmode-round-notify-real-chain.test.ts +0 -215
- package/src/execution/__tests__/concurrency-pool.test.ts +0 -250
- package/src/execution/__tests__/config.test.ts +0 -110
- package/src/execution/__tests__/conversation-wiring.test.ts +0 -198
- package/src/execution/__tests__/crash-recovery.test.ts +0 -320
- package/src/execution/__tests__/delivery-methods.test.ts +0 -422
- package/src/execution/__tests__/dialog-queue.test.ts +0 -299
- package/src/execution/__tests__/epipe-fallback.test.ts +0 -241
- package/src/execution/__tests__/execute-and-await-worktree.test.ts +0 -243
- package/src/execution/__tests__/execute-nesting.test.ts +0 -323
- package/src/execution/__tests__/execute-options-mapper.test.ts +0 -200
- package/src/execution/__tests__/execution-record.test.ts +0 -1394
- package/src/execution/__tests__/explicit-agent-ref-guard.test.ts +0 -171
- package/src/execution/__tests__/finalize-record.test.ts +0 -368
- package/src/execution/__tests__/finalized-marker.test.ts +0 -82
- package/src/execution/__tests__/format-schema-instruction.test.ts +0 -169
- package/src/execution/__tests__/format.test.ts +0 -458
- package/src/execution/__tests__/gc-timer.test.ts +0 -184
- package/src/execution/__tests__/get-record-for-action-restart.test.ts +0 -254
- package/src/execution/__tests__/gui-mode-dispatch.test.ts +0 -59
- package/src/execution/__tests__/helpers/mock-extension-api.ts +0 -30
- package/src/execution/__tests__/helpers/spawn-mock.ts +0 -242
- package/src/execution/__tests__/host-mode.test.ts +0 -87
- package/src/execution/__tests__/index-session-start-identity.test.ts +0 -371
- package/src/execution/__tests__/index-session-start.test.ts +0 -672
- package/src/execution/__tests__/lifecycle-manager-lock.test.ts +0 -211
- package/src/execution/__tests__/lifecycle-manager.test.ts +0 -383
- package/src/execution/__tests__/lifecycle-predicates.test.ts +0 -116
- package/src/execution/__tests__/list-component.test.ts +0 -401
- package/src/execution/__tests__/list-fields.test.ts +0 -140
- package/src/execution/__tests__/manifest-parentid.test.ts +0 -117
- package/src/execution/__tests__/model-resolver.test.ts +0 -465
- package/src/execution/__tests__/nested-visibility-env-propagation.test.ts +0 -287
- package/src/execution/__tests__/nested-visibility.test.ts +0 -325
- package/src/execution/__tests__/notifier-flush.test.ts +0 -612
- package/src/execution/__tests__/notifier-golden-snapshot.test.ts +0 -261
- package/src/execution/__tests__/notify-ledger.test.ts +0 -826
- package/src/execution/__tests__/one-shot-upgrade.test.ts +0 -205
- package/src/execution/__tests__/output-collector.test.ts +0 -358
- package/src/execution/__tests__/parent-child-matrix.test.ts +0 -336
- package/src/execution/__tests__/path-encoding.test.ts +0 -104
- package/src/execution/__tests__/pi-invocation.test.ts +0 -134
- package/src/execution/__tests__/record-store.test.ts +0 -1057
- package/src/execution/__tests__/records-cwd-isolation.test.ts +0 -91
- package/src/execution/__tests__/recursive-visibility-baseline.test.ts +0 -339
- package/src/execution/__tests__/recursive-visibility-env.test.ts +0 -273
- package/src/execution/__tests__/relay-agent.test.ts +0 -448
- package/src/execution/__tests__/relay-env.test.ts +0 -42
- package/src/execution/__tests__/resource-policy.test.ts +0 -109
- package/src/execution/__tests__/rpc-mode.test.ts +0 -89
- package/src/execution/__tests__/run-and-finalize-chatmode.test.ts +0 -267
- package/src/execution/__tests__/run-spawn-chatmode-settled.test.ts +0 -253
- package/src/execution/__tests__/run-spawn-edges.test.ts +0 -687
- package/src/execution/__tests__/run-spawn-integration.test.ts +0 -933
- package/src/execution/__tests__/run-spawn-resume.test.ts +0 -322
- package/src/execution/__tests__/run-spawn-rpc-mode.test.ts +0 -196
- package/src/execution/__tests__/run-spawn-stdout-callback-throw.test.ts +0 -199
- package/src/execution/__tests__/sdk-contract.test.ts +0 -273
- package/src/execution/__tests__/session-context-resolver.test.ts +0 -167
- package/src/execution/__tests__/session-file-gc.test.ts +0 -293
- package/src/execution/__tests__/session-pending.test.ts +0 -197
- package/src/execution/__tests__/session-reconstructor.test.ts +0 -379
- package/src/execution/__tests__/session-runner-epipe.test.ts +0 -178
- package/src/execution/__tests__/session-runner-schema-env.test.ts +0 -350
- package/src/execution/__tests__/session-start-reaper.test.ts +0 -233
- package/src/execution/__tests__/spawn-args.test.ts +0 -445
- package/src/execution/__tests__/spawn-event-adapter-rpc.test.ts +0 -189
- package/src/execution/__tests__/spawn-event-adapter.test.ts +0 -167
- package/src/execution/__tests__/spawn-worktree-guidance.test.ts +0 -207
- package/src/execution/__tests__/spawned-children.test.ts +0 -92
- package/src/execution/__tests__/start-sync-model-guard.test.ts +0 -150
- package/src/execution/__tests__/startup-config-declaration.test.ts +0 -35
- package/src/execution/__tests__/status-refactor.test.ts +0 -345
- package/src/execution/__tests__/stdin-writer.test.ts +0 -463
- package/src/execution/__tests__/stream-sink-retirement.test.ts +0 -261
- package/src/execution/__tests__/subagent-service-abort.test.ts +0 -60
- package/src/execution/__tests__/subagent-service-message-close.test.ts +0 -629
- package/src/execution/__tests__/subagent-service-parent-guard.test.ts +0 -180
- package/src/execution/__tests__/subagent-service.test.ts +0 -788
- package/src/execution/__tests__/subprocess-agent-runner-routing.test.ts +0 -310
- package/src/execution/__tests__/subprocess-agent-runner.test.ts +0 -599
- package/src/execution/__tests__/temp-prompt.test.ts +0 -53
- package/src/execution/__tests__/timeout-integration.test.ts +0 -615
- package/src/execution/__tests__/tombstone-store.test.ts +0 -73
- package/src/execution/__tests__/tool-action.test.ts +0 -476
- package/src/execution/__tests__/truncline-snapshot.test.ts +0 -81
- package/src/execution/__tests__/turn-limiter-semantics.test.ts +0 -194
- package/src/execution/__tests__/turn-limiter.test.ts +0 -65
- package/src/execution/__tests__/ui-channels.test.ts +0 -187
- package/src/execution/__tests__/ui-interaction-model.test.ts +0 -67
- package/src/execution/__tests__/ui-request-handler-factory.test.ts +0 -370
- package/src/execution/__tests__/ui-request-handler.test.ts +0 -204
- package/src/execution/__tests__/ui-request-observability.test.ts +0 -114
- package/src/execution/__tests__/ui-request-queue.test.ts +0 -133
- package/src/execution/__tests__/worktree-manager.test.ts +0 -633
- package/src/execution/__tests__/worktree-pid-registration.integration.test.ts +0 -229
- package/src/execution/__tests__/worktree-reconcile.integration.test.ts +0 -181
- package/src/execution/__tests__/worktree-registry.test.ts +0 -199
- package/src/execution/agent-registry.ts +0 -199
- package/src/execution/agent-result-mapper.ts +0 -90
- package/src/execution/alive-store.ts +0 -92
- package/src/execution/argv-mirror.ts +0 -113
- package/src/execution/best-effort.ts +0 -38
- package/src/execution/channel-registry-access.ts +0 -144
- package/src/execution/concurrency-pool.ts +0 -116
- package/src/execution/config.ts +0 -96
- package/src/execution/dialog-queue.ts +0 -330
- package/src/execution/engine/__tests__/common/data-dir.test.ts +0 -53
- package/src/execution/engine/__tests__/common/errors.test.ts +0 -132
- package/src/execution/engine/__tests__/common/event-journal.test.ts +0 -177
- package/src/execution/engine/__tests__/common/kill-chain.test.ts +0 -192
- package/src/execution/engine/__tests__/common/nesting-guard.test.ts +0 -81
- package/src/execution/engine/__tests__/common/persona-router.test.ts +0 -123
- package/src/execution/engine/__tests__/common/pool-manager.test.ts +0 -154
- package/src/execution/engine/__tests__/common/schema-emulation.test.ts +0 -128
- package/src/execution/engine/__tests__/conformance/__fixtures__/pi-golden-events.json +0 -28
- package/src/execution/engine/__tests__/conformance/agent-event-invariants.ts +0 -141
- package/src/execution/engine/__tests__/conformance/contract.abort.test.ts +0 -109
- package/src/execution/engine/__tests__/conformance/contract.agent-events.test.ts +0 -101
- package/src/execution/engine/__tests__/conformance/contract.probe.test.ts +0 -77
- package/src/execution/engine/__tests__/conformance/contract.read-degradation.test.ts +0 -104
- package/src/execution/engine/__tests__/conformance/contract.relay.test.ts +0 -342
- package/src/execution/engine/__tests__/conformance/engine-conformance.live.test.ts +0 -201
- package/src/execution/engine/__tests__/conformance/golden-replay.pi.test.ts +0 -76
- package/src/execution/engine/__tests__/conformance/golden-replay.zcode.test.ts +0 -79
- package/src/execution/engine/__tests__/engine-discovery.test.ts +0 -87
- package/src/execution/engine/__tests__/engines-declaration.test.ts +0 -36
- package/src/execution/engine/__tests__/model-prompt.test.ts +0 -85
- package/src/execution/engine/__tests__/paths.test.ts +0 -39
- package/src/execution/engine/__tests__/registry.test.ts +0 -120
- package/src/execution/engine/__tests__/routing.test.ts +0 -231
- package/src/execution/engine/common/data-dir.ts +0 -62
- package/src/execution/engine/common/errors.ts +0 -183
- package/src/execution/engine/common/event-journal.ts +0 -254
- package/src/execution/engine/common/journal-replay.ts +0 -62
- package/src/execution/engine/common/kill-chain.ts +0 -221
- package/src/execution/engine/common/nesting-guard.ts +0 -50
- package/src/execution/engine/common/persona-router.ts +0 -108
- package/src/execution/engine/common/pool-manager.ts +0 -226
- package/src/execution/engine/common/schema-emulation.ts +0 -189
- package/src/execution/engine/common/session-view-projection.ts +0 -51
- package/src/execution/engine/engine-discovery.ts +0 -65
- package/src/execution/engine/engines/pi/__tests__/pi-engine.test.ts +0 -469
- package/src/execution/engine/engines/pi/__tests__/reader.test.ts +0 -155
- package/src/execution/engine/engines/pi/__tests__/task-spec-mapper.test.ts +0 -164
- package/src/execution/engine/engines/pi/pi-engine.ts +0 -415
- package/src/execution/engine/engines/pi/reader.ts +0 -48
- package/src/execution/engine/engines/pi/registration.ts +0 -35
- package/src/execution/engine/engines/pi/task-spec-mapper.ts +0 -100
- package/src/execution/engine/engines/zcode/__tests__/__fixtures__/zcode-golden-spawn.json +0 -39
- package/src/execution/engine/engines/zcode/__tests__/launcher.test.ts +0 -150
- package/src/execution/engine/engines/zcode/__tests__/parser.test.ts +0 -246
- package/src/execution/engine/engines/zcode/__tests__/preparer.test.ts +0 -228
- package/src/execution/engine/engines/zcode/__tests__/reader.test.ts +0 -210
- package/src/execution/engine/engines/zcode/__tests__/registration.test.ts +0 -64
- package/src/execution/engine/engines/zcode/__tests__/zcode-engine.live.test.ts +0 -127
- package/src/execution/engine/engines/zcode/__tests__/zcode-engine.test.ts +0 -580
- package/src/execution/engine/engines/zcode/constants.ts +0 -43
- package/src/execution/engine/engines/zcode/golden-sample.ts +0 -39
- package/src/execution/engine/engines/zcode/launcher.ts +0 -161
- package/src/execution/engine/engines/zcode/parser.ts +0 -436
- package/src/execution/engine/engines/zcode/preparer.ts +0 -363
- package/src/execution/engine/engines/zcode/reader.ts +0 -381
- package/src/execution/engine/engines/zcode/registration.ts +0 -37
- package/src/execution/engine/engines/zcode/zcode-engine.ts +0 -658
- package/src/execution/engine/host-task-spec.ts +0 -47
- package/src/execution/engine/model-prompt.ts +0 -59
- package/src/execution/engine/paths.ts +0 -42
- package/src/execution/engine/port.ts +0 -153
- package/src/execution/engine/registry.ts +0 -123
- package/src/execution/engine/routing.ts +0 -218
- package/src/execution/engine/types.ts +0 -309
- package/src/execution/execute-options-mapper.ts +0 -112
- package/src/execution/execution-record.ts +0 -965
- package/src/execution/finalize-record.ts +0 -251
- package/src/execution/finalized-marker.ts +0 -51
- package/src/execution/get-state-handshake.ts +0 -104
- package/src/execution/host-mode.ts +0 -56
- package/src/execution/idle-gc.ts +0 -47
- package/src/execution/lifecycle-manager.ts +0 -513
- package/src/execution/lifecycle-predicates.ts +0 -65
- package/src/execution/manifest-store.ts +0 -256
- package/src/execution/model-config-service.ts +0 -219
- package/src/execution/model-resolver.ts +0 -248
- package/src/execution/notifier.ts +0 -330
- package/src/execution/notify-ledger.ts +0 -580
- package/src/execution/output-collector.ts +0 -228
- package/src/execution/path-encoding.ts +0 -60
- package/src/execution/pi-invocation.ts +0 -120
- package/src/execution/record-entry.ts +0 -132
- package/src/execution/record-store.ts +0 -1234
- package/src/execution/relay-env.ts +0 -37
- package/src/execution/session-context-resolver.ts +0 -64
- package/src/execution/session-file-gc.ts +0 -120
- package/src/execution/session-pending.ts +0 -164
- package/src/execution/session-reconstructor.ts +0 -678
- package/src/execution/session-runner.ts +0 -1781
- package/src/execution/sessions-index.ts +0 -304
- package/src/execution/spawn-event-adapter.ts +0 -363
- package/src/execution/stdin-writer.ts +0 -198
- package/src/execution/stream-sink.ts +0 -126
- package/src/execution/subagent-service.ts +0 -2141
- package/src/execution/subprocess-agent-runner.ts +0 -317
- package/src/execution/temp-prompt.ts +0 -62
- package/src/execution/tombstone-store.ts +0 -72
- package/src/execution/turn-limiter.ts +0 -102
- package/src/execution/types.ts +0 -973
- package/src/execution/ui-channels.ts +0 -216
- package/src/execution/ui-interaction-model.ts +0 -48
- package/src/execution/ui-request-handler-factory.ts +0 -212
- package/src/execution/ui-request-observability.ts +0 -81
- package/src/execution/ui-request-queue.ts +0 -184
- package/src/execution/worktree-manager.ts +0 -688
- package/src/execution/worktree-registry.ts +0 -222
- package/src/orchestration/__tests__/__fixtures__/worker-template.snapshot.txt +0 -349
- package/src/orchestration/__tests__/agent-call-catch-fallback.test.ts +0 -202
- package/src/orchestration/__tests__/agent-call-stream.test.ts +0 -152
- package/src/orchestration/__tests__/args-validator.test.ts +0 -143
- package/src/orchestration/__tests__/config-loader.test.ts +0 -503
- package/src/orchestration/__tests__/error-recovery-handlers.test.ts +0 -809
- package/src/orchestration/__tests__/error-recovery-postmessage-defense.test.ts +0 -405
- package/src/orchestration/__tests__/error-recovery-serialize-failed-result.test.ts +0 -56
- package/src/orchestration/__tests__/error-recovery-workflow-call.test.ts +0 -166
- package/src/orchestration/__tests__/execute-agent-call.test.ts +0 -451
- package/src/orchestration/__tests__/jsonl-run-store-corrupt-entry.test.ts +0 -150
- package/src/orchestration/__tests__/jsonl-run-store-loadall-sources.test.ts +0 -171
- package/src/orchestration/__tests__/jsonl-run-store-retention.test.ts +0 -202
- package/src/orchestration/__tests__/jsonl-run-store-session-file.test.ts +0 -1030
- package/src/orchestration/__tests__/launcher-nested-workflow.test.ts +0 -600
- package/src/orchestration/__tests__/lifecycle-runid-injection.test.ts +0 -96
- package/src/orchestration/__tests__/lifecycle.test.ts +0 -659
- package/src/orchestration/__tests__/non-cloneable-return-e2e.test.ts +0 -95
- package/src/orchestration/__tests__/review-fix-loop-e2e.test.ts +0 -2025
- package/src/orchestration/__tests__/script-lint.test.ts +0 -831
- package/src/orchestration/__tests__/skill-discovery.test.ts +0 -201
- package/src/orchestration/__tests__/test-mocks.ts +0 -197
- package/src/orchestration/__tests__/worker-exit-without-result.test.ts +0 -368
- package/src/orchestration/__tests__/worker-host.test.ts +0 -120
- package/src/orchestration/__tests__/worker-returnmeta-passthrough.test.ts +0 -164
- package/src/orchestration/__tests__/worker-script-builder-runtime.test.ts +0 -563
- package/src/orchestration/__tests__/worker-script-builder.test.ts +0 -309
- package/src/orchestration/__tests__/worker-script-template-snapshot.test.ts +0 -129
- package/src/orchestration/__tests__/workflow-nesting-e2e.test.ts +0 -317
- package/src/orchestration/__tests__/workflow-script-lint-memo.test.ts +0 -110
- package/src/orchestration/__tests__/workflows-e2e.test.ts +0 -537
- package/src/orchestration/agent-opts-resolver.ts +0 -174
- package/src/orchestration/args-validator.ts +0 -127
- package/src/orchestration/config-loader.ts +0 -315
- package/src/orchestration/error-recovery.ts +0 -1018
- package/src/orchestration/execute-agent-call.ts +0 -256
- package/src/orchestration/launcher.ts +0 -455
- package/src/orchestration/lifecycle.ts +0 -399
- package/src/orchestration/models/__tests__/budget.test.ts +0 -307
- package/src/orchestration/models/__tests__/trace.test.ts +0 -408
- package/src/orchestration/models/agent-call.ts +0 -83
- package/src/orchestration/models/budget.ts +0 -118
- package/src/orchestration/models/ports.ts +0 -164
- package/src/orchestration/models/run-runtime.ts +0 -104
- package/src/orchestration/models/run-spec.ts +0 -83
- package/src/orchestration/models/run-state.ts +0 -44
- package/src/orchestration/models/trace.ts +0 -183
- package/src/orchestration/models/types.ts +0 -301
- package/src/orchestration/models/workflow-run.ts +0 -254
- package/src/orchestration/models/workflow-script-registry.ts +0 -35
- package/src/orchestration/models/workflow-script.ts +0 -117
- package/src/orchestration/script-lint.ts +0 -766
- package/src/orchestration/skill-discovery.ts +0 -115
- package/src/orchestration/worker-handle.ts +0 -115
- package/src/orchestration/worker-host.ts +0 -98
- package/src/orchestration/worker-script-builder.ts +0 -421
- package/src/orchestration/workflow-files.ts +0 -85
- package/src/orchestration/workflow-script-registry-impl.ts +0 -133
- package/src/shared/__tests__/agent-ref.test.ts +0 -34
- package/src/shared/__tests__/meta-parser.test.ts +0 -304
- package/src/shared/__tests__/model-ref.test.ts +0 -306
- package/src/shared/__tests__/resource-discovery-manifest-cache.test.ts +0 -280
- package/src/shared/__tests__/resource-discovery.test.ts +0 -736
- package/src/shared/__tests__/resource-meta.test.ts +0 -51
- package/src/shared/__tests__/schema-jsonify.test.ts +0 -81
- package/src/shared/__tests__/timer-delay.test.ts +0 -61
- package/src/shared/agent-event.ts +0 -13
- package/src/shared/agent-ref.ts +0 -57
- package/src/shared/meta-parser.ts +0 -261
- package/src/shared/model-ref.ts +0 -286
- package/src/shared/resource-discovery.ts +0 -811
- package/src/shared/resource-meta.ts +0 -65
- package/src/shared/schema-env.ts +0 -44
- package/src/shared/schema-jsonify.ts +0 -58
- package/src/shared/timer-delay.ts +0 -54
- package/src/shared/xml-injection.ts +0 -35
- package/workflows/README.md +0 -81
- package/workflows/_shared/agent-refs.cjs +0 -40
- package/workflows/chain.js +0 -131
- package/workflows/map-reduce.js +0 -174
- package/workflows/parallel.js +0 -148
- package/workflows/review-fix-loop-utils.cjs +0 -1309
- package/workflows/review-fix-loop.js +0 -1404
- package/workflows/scatter-gather.js +0 -178
|
@@ -1,167 +0,0 @@
|
|
|
1
|
-
// src/__tests__/spawn-event-adapter.test.ts
|
|
2
|
-
import * as fs from "node:fs";
|
|
3
|
-
import * as os from "node:os";
|
|
4
|
-
import * as path from "node:path";
|
|
5
|
-
|
|
6
|
-
import { describe, expect, it } from "vitest";
|
|
7
|
-
|
|
8
|
-
import {
|
|
9
|
-
deriveSessionFilePath,
|
|
10
|
-
findSessionFileByHeaderId,
|
|
11
|
-
parseSpawnLine,
|
|
12
|
-
} from "../spawn-event-adapter.ts";
|
|
13
|
-
|
|
14
|
-
describe("parseSpawnLine", () => {
|
|
15
|
-
describe("空白行", () => {
|
|
16
|
-
it("空字符串返回 null", () => {
|
|
17
|
-
expect(parseSpawnLine("")).toBeNull();
|
|
18
|
-
});
|
|
19
|
-
|
|
20
|
-
it("纯空白返回 null", () => {
|
|
21
|
-
expect(parseSpawnLine(" ")).toBeNull();
|
|
22
|
-
expect(parseSpawnLine("\t\t")).toBeNull();
|
|
23
|
-
});
|
|
24
|
-
});
|
|
25
|
-
|
|
26
|
-
describe("header 行", () => {
|
|
27
|
-
it("type=session + id 识别为 header", () => {
|
|
28
|
-
const line = JSON.stringify({
|
|
29
|
-
type: "session",
|
|
30
|
-
id: "abc-123",
|
|
31
|
-
timestamp: "2026-07-03T12:00:00.000Z",
|
|
32
|
-
cwd: "/home/user/project",
|
|
33
|
-
});
|
|
34
|
-
const result = parseSpawnLine(line);
|
|
35
|
-
expect(result?.kind).toBe("header");
|
|
36
|
-
if (result?.kind === "header") {
|
|
37
|
-
expect(result.header.id).toBe("abc-123");
|
|
38
|
-
expect(result.header.cwd).toBe("/home/user/project");
|
|
39
|
-
}
|
|
40
|
-
});
|
|
41
|
-
|
|
42
|
-
it("header 含 parentSession + version", () => {
|
|
43
|
-
const line = JSON.stringify({
|
|
44
|
-
type: "session",
|
|
45
|
-
id: "child-456",
|
|
46
|
-
timestamp: "2026-07-03T12:00:00.000Z",
|
|
47
|
-
cwd: "/home/user/project",
|
|
48
|
-
parentSession: "parent-789",
|
|
49
|
-
version: 2,
|
|
50
|
-
});
|
|
51
|
-
const result = parseSpawnLine(line);
|
|
52
|
-
expect(result?.kind).toBe("header");
|
|
53
|
-
if (result?.kind === "header") {
|
|
54
|
-
expect(result.header.parentSession).toBe("parent-789");
|
|
55
|
-
expect(result.header.version).toBe(2);
|
|
56
|
-
}
|
|
57
|
-
});
|
|
58
|
-
});
|
|
59
|
-
|
|
60
|
-
describe("事件行", () => {
|
|
61
|
-
it("tool_execution_start 识别为 event", () => {
|
|
62
|
-
const line = JSON.stringify({
|
|
63
|
-
type: "tool_execution_start",
|
|
64
|
-
toolName: "bash",
|
|
65
|
-
toolCallId: "call-1",
|
|
66
|
-
args: { command: "ls" },
|
|
67
|
-
});
|
|
68
|
-
const result = parseSpawnLine(line);
|
|
69
|
-
expect(result?.kind).toBe("event");
|
|
70
|
-
if (result?.kind === "event") {
|
|
71
|
-
expect(result.event.type).toBe("tool_execution_start");
|
|
72
|
-
expect(result.event.toolName).toBe("bash");
|
|
73
|
-
}
|
|
74
|
-
});
|
|
75
|
-
|
|
76
|
-
it("message_end 识别为 event", () => {
|
|
77
|
-
const line = JSON.stringify({
|
|
78
|
-
type: "message_end",
|
|
79
|
-
message: {
|
|
80
|
-
role: "assistant",
|
|
81
|
-
usage: { input: 100, output: 50 },
|
|
82
|
-
},
|
|
83
|
-
});
|
|
84
|
-
const result = parseSpawnLine(line);
|
|
85
|
-
expect(result?.kind).toBe("event");
|
|
86
|
-
if (result?.kind === "event") {
|
|
87
|
-
expect(result.event.type).toBe("message_end");
|
|
88
|
-
}
|
|
89
|
-
});
|
|
90
|
-
|
|
91
|
-
it("turn_end 识别为 event", () => {
|
|
92
|
-
const result = parseSpawnLine(JSON.stringify({ type: "turn_end" }));
|
|
93
|
-
expect(result?.kind).toBe("event");
|
|
94
|
-
});
|
|
95
|
-
|
|
96
|
-
it("未知 type 仍识别为 event(schema 校验由调用方)", () => {
|
|
97
|
-
const result = parseSpawnLine(JSON.stringify({ type: "some_future_event" }));
|
|
98
|
-
expect(result?.kind).toBe("event");
|
|
99
|
-
});
|
|
100
|
-
});
|
|
101
|
-
|
|
102
|
-
describe("invalid 行", () => {
|
|
103
|
-
it("非 JSON 返回 invalid", () => {
|
|
104
|
-
const result = parseSpawnLine("not json at all");
|
|
105
|
-
expect(result?.kind).toBe("invalid");
|
|
106
|
-
if (result?.kind === "invalid") {
|
|
107
|
-
expect(result.raw).toBe("not json at all");
|
|
108
|
-
expect(result.error).toBeTruthy();
|
|
109
|
-
}
|
|
110
|
-
});
|
|
111
|
-
|
|
112
|
-
it("JSON 但无 type 字段返回 invalid", () => {
|
|
113
|
-
const result = parseSpawnLine(JSON.stringify({ foo: "bar" }));
|
|
114
|
-
expect(result?.kind).toBe("invalid");
|
|
115
|
-
if (result?.kind === "invalid") {
|
|
116
|
-
expect(result.error).toContain("type");
|
|
117
|
-
}
|
|
118
|
-
});
|
|
119
|
-
|
|
120
|
-
it("JSON 但 type 非 string 返回 invalid", () => {
|
|
121
|
-
const result = parseSpawnLine(JSON.stringify({ type: 123 }));
|
|
122
|
-
expect(result?.kind).toBe("invalid");
|
|
123
|
-
});
|
|
124
|
-
|
|
125
|
-
it("JSON null 返回 invalid", () => {
|
|
126
|
-
const result = parseSpawnLine("null");
|
|
127
|
-
expect(result?.kind).toBe("invalid");
|
|
128
|
-
});
|
|
129
|
-
});
|
|
130
|
-
});
|
|
131
|
-
|
|
132
|
-
describe("deriveSessionFilePath", () => {
|
|
133
|
-
it("拼接 sessionDir + fileTimestamp(冒号点转连字符) + id", () => {
|
|
134
|
-
const header = {
|
|
135
|
-
type: "session" as const,
|
|
136
|
-
id: "abc-123",
|
|
137
|
-
timestamp: "2026-07-03T12:00:00.000Z",
|
|
138
|
-
cwd: "/proj",
|
|
139
|
-
};
|
|
140
|
-
const path = deriveSessionFilePath(header, "/sessions/dir");
|
|
141
|
-
// fileTimestamp = timestamp.replace(/[:.]/g, "-") = "2026-07-03T12-00-00-000Z"
|
|
142
|
-
expect(path).toBe("/sessions/dir/2026-07-03T12-00-00-000Z_abc-123.jsonl");
|
|
143
|
-
});
|
|
144
|
-
});
|
|
145
|
-
|
|
146
|
-
describe("findSessionFileByHeaderId", () => {
|
|
147
|
-
it("sessionId 后缀匹配返回实际文件路径", () => {
|
|
148
|
-
const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "spawn-adapter-test-"));
|
|
149
|
-
const expectedFile = path.join(tmpDir, "2026-07-03T12-00-00-000Z_sid-456.jsonl");
|
|
150
|
-
fs.writeFileSync(expectedFile, "{}");
|
|
151
|
-
const result = findSessionFileByHeaderId(tmpDir, "sid-456");
|
|
152
|
-
expect(result).toBe(expectedFile);
|
|
153
|
-
fs.rmSync(tmpDir, { recursive: true });
|
|
154
|
-
});
|
|
155
|
-
|
|
156
|
-
it("无匹配返回 undefined", () => {
|
|
157
|
-
const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "spawn-adapter-test-"));
|
|
158
|
-
const result = findSessionFileByHeaderId(tmpDir, "nonexistent");
|
|
159
|
-
expect(result).toBeUndefined();
|
|
160
|
-
fs.rmSync(tmpDir, { recursive: true });
|
|
161
|
-
});
|
|
162
|
-
|
|
163
|
-
it("目录不存在返回 undefined(不抛错)", () => {
|
|
164
|
-
const result = findSessionFileByHeaderId("/nonexistent/path/xyz", "sid");
|
|
165
|
-
expect(result).toBeUndefined();
|
|
166
|
-
});
|
|
167
|
-
});
|
|
@@ -1,207 +0,0 @@
|
|
|
1
|
-
// src/execution/__tests__/spawn-worktree-guidance.test.ts
|
|
2
|
-
//
|
|
3
|
-
// worktree 模式认知纠正提示注入测试。
|
|
4
|
-
//
|
|
5
|
-
// 背景:worktree checkout 放 os.tmpdir()(路径形似临时沙箱),子 agent 无 worktree 语义
|
|
6
|
-
// 提示时误判 cwd 为"空隔离目录",主动 cd 别处放弃隔离(wave-agent 事故 session 019ff64c)。
|
|
7
|
-
// 修复在 runSpawn 的 appendParts 注入 WORKTREE_GUIDANCE_PROMPT。
|
|
8
|
-
//
|
|
9
|
-
// 验证:
|
|
10
|
-
// - worktree 模式(opts.worktree 传入)→ appendSystemPrompt content 含 WORKTREE_GUIDANCE_PROMPT
|
|
11
|
-
// - 非 worktree 模式 → content 不含 worktree 提示
|
|
12
|
-
//
|
|
13
|
-
// 测试策略:mock writePromptToTempFile 捕获拼接后的 appendParts content(vi.hoisted 防
|
|
14
|
-
// vi.mock hoisting 引用未初始化变量),断言含/不含 worktree 提示标识。
|
|
15
|
-
|
|
16
|
-
import type { PassThrough } from "node:stream";
|
|
17
|
-
|
|
18
|
-
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
|
19
|
-
|
|
20
|
-
// 捕获 writePromptToTempFile 的 content 参数(appendParts.join("\n\n"))。
|
|
21
|
-
// vi.hoisted 保证变量在 vi.mock factory hoisting 前已声明可用。
|
|
22
|
-
const captured = vi.hoisted(() => ({ content: "" }));
|
|
23
|
-
|
|
24
|
-
// ── mock modules(与 session-runner-schema-env.test.ts 同模式)──
|
|
25
|
-
|
|
26
|
-
vi.mock("node:child_process", async () => {
|
|
27
|
-
const { EventEmitter } = await import("node:events");
|
|
28
|
-
const { PassThrough } = await import("node:stream");
|
|
29
|
-
|
|
30
|
-
class FakeChild extends EventEmitter {
|
|
31
|
-
pid = 12345;
|
|
32
|
-
stdout = new PassThrough();
|
|
33
|
-
stdin = new PassThrough(); // runSpawn 注册 stdin EPIPE handler(session-runner),FakeChild 必须提供
|
|
34
|
-
stderr = new PassThrough();
|
|
35
|
-
killed = false;
|
|
36
|
-
killSignal: string | undefined;
|
|
37
|
-
kill(sig?: string): boolean {
|
|
38
|
-
this.killed = true;
|
|
39
|
-
this.killSignal = sig;
|
|
40
|
-
return true;
|
|
41
|
-
}
|
|
42
|
-
}
|
|
43
|
-
|
|
44
|
-
return {
|
|
45
|
-
spawn: vi.fn(() => new FakeChild()),
|
|
46
|
-
execFileSync: vi.fn(() => ""),
|
|
47
|
-
};
|
|
48
|
-
});
|
|
49
|
-
|
|
50
|
-
vi.mock("node:fs", async () => {
|
|
51
|
-
const actual = await import("node:fs");
|
|
52
|
-
return {
|
|
53
|
-
default: {
|
|
54
|
-
...actual,
|
|
55
|
-
mkdirSync: vi.fn(),
|
|
56
|
-
existsSync: vi.fn(() => false),
|
|
57
|
-
appendFileSync: vi.fn(),
|
|
58
|
-
writeFileSync: vi.fn(),
|
|
59
|
-
readdirSync: vi.fn(() => []),
|
|
60
|
-
},
|
|
61
|
-
mkdirSync: vi.fn(),
|
|
62
|
-
existsSync: vi.fn(() => false),
|
|
63
|
-
appendFileSync: vi.fn(),
|
|
64
|
-
writeFileSync: vi.fn(),
|
|
65
|
-
readdirSync: vi.fn(() => []),
|
|
66
|
-
promises: actual.promises,
|
|
67
|
-
};
|
|
68
|
-
});
|
|
69
|
-
|
|
70
|
-
vi.mock("../alive-store.ts", () => ({
|
|
71
|
-
writeAliveMarker: vi.fn(),
|
|
72
|
-
}));
|
|
73
|
-
|
|
74
|
-
vi.mock("../temp-prompt.ts", () => ({
|
|
75
|
-
writePromptToTempFile: vi.fn(async (agent: string, content: string) => {
|
|
76
|
-
captured.content = content;
|
|
77
|
-
const safeName = agent.replace(/[^\w.-]+/g, "_");
|
|
78
|
-
return { dir: `/tmp/fake-${safeName}`, filePath: `/tmp/fake-${safeName}/prompt.md` };
|
|
79
|
-
}),
|
|
80
|
-
cleanupTempPrompt: vi.fn(async () => {}),
|
|
81
|
-
}));
|
|
82
|
-
|
|
83
|
-
import { spawn } from "node:child_process";
|
|
84
|
-
|
|
85
|
-
import { createRecord } from "../execution-record.ts";
|
|
86
|
-
import { runSpawn, type RunOptions, type SessionRunnerContext } from "../session-runner.ts";
|
|
87
|
-
|
|
88
|
-
const mockSpawn = vi.mocked(spawn);
|
|
89
|
-
|
|
90
|
-
interface FakeChild {
|
|
91
|
-
pid: number;
|
|
92
|
-
stdout: PassThrough;
|
|
93
|
-
stderr: PassThrough;
|
|
94
|
-
killed: boolean;
|
|
95
|
-
killSignal: string | undefined;
|
|
96
|
-
kill(sig?: string): boolean;
|
|
97
|
-
emit(event: string, ...args: unknown[]): boolean;
|
|
98
|
-
}
|
|
99
|
-
|
|
100
|
-
function getLastSpawnedChild(): FakeChild {
|
|
101
|
-
const result = mockSpawn.mock.results.at(-1);
|
|
102
|
-
if (!result) throw new Error("spawn was not called yet");
|
|
103
|
-
return result.value as FakeChild;
|
|
104
|
-
}
|
|
105
|
-
|
|
106
|
-
async function waitForSpawn(timeoutMs = 1000): Promise<void> {
|
|
107
|
-
const start = Date.now();
|
|
108
|
-
while (mockSpawn.mock.results.length === 0) {
|
|
109
|
-
if (Date.now() - start > timeoutMs) {
|
|
110
|
-
throw new Error(`spawn was not called within ${timeoutMs}ms`);
|
|
111
|
-
}
|
|
112
|
-
await new Promise((r) => setTimeout(r, 5));
|
|
113
|
-
}
|
|
114
|
-
}
|
|
115
|
-
|
|
116
|
-
function makeRecord() {
|
|
117
|
-
return createRecord("wt-guidance-1", {
|
|
118
|
-
agent: "general-purpose",
|
|
119
|
-
model: "test/model",
|
|
120
|
-
mode: "sync",
|
|
121
|
-
task: "test task",
|
|
122
|
-
startedAt: Date.now(),
|
|
123
|
-
rootSessionId: "s1",
|
|
124
|
-
parentRecordId: undefined,
|
|
125
|
-
depth: 0,
|
|
126
|
-
});
|
|
127
|
-
}
|
|
128
|
-
|
|
129
|
-
function makeRunOpts(overrides: Partial<RunOptions> = {}): RunOptions {
|
|
130
|
-
return {
|
|
131
|
-
resolved: { model: { provider: "test", id: "model" }, thinkingLevel: undefined },
|
|
132
|
-
agentConfig: undefined,
|
|
133
|
-
appendSystemPrompt: undefined,
|
|
134
|
-
skillPath: undefined,
|
|
135
|
-
schema: undefined,
|
|
136
|
-
maxTurns: undefined,
|
|
137
|
-
graceTurns: undefined,
|
|
138
|
-
signal: undefined,
|
|
139
|
-
onEvent: undefined,
|
|
140
|
-
...overrides,
|
|
141
|
-
};
|
|
142
|
-
}
|
|
143
|
-
|
|
144
|
-
function makeCtx(overrides: Partial<SessionRunnerContext> = {}): SessionRunnerContext {
|
|
145
|
-
return {
|
|
146
|
-
cwd: "/fake/cwd",
|
|
147
|
-
agentDir: "/fake/agent",
|
|
148
|
-
skillDirs: [],
|
|
149
|
-
mainCwd: "/fake/cwd",
|
|
150
|
-
sessionRootId: "root-session-test",
|
|
151
|
-
rootCwd: "/fake/cwd",
|
|
152
|
-
...overrides,
|
|
153
|
-
};
|
|
154
|
-
}
|
|
155
|
-
|
|
156
|
-
describe("worktree guidance prompt injection", () => {
|
|
157
|
-
beforeEach(() => {
|
|
158
|
-
vi.clearAllMocks();
|
|
159
|
-
captured.content = "";
|
|
160
|
-
});
|
|
161
|
-
|
|
162
|
-
afterEach(() => {
|
|
163
|
-
vi.restoreAllMocks();
|
|
164
|
-
});
|
|
165
|
-
|
|
166
|
-
it("worktree 模式 → appendSystemPrompt content 含 worktree 认知纠正提示", async () => {
|
|
167
|
-
const record = makeRecord();
|
|
168
|
-
const opts = makeRunOpts({
|
|
169
|
-
worktree: {
|
|
170
|
-
path: "/tmp/pi-subagents/--fake--/pi-sub-sa-wt-guidance-1",
|
|
171
|
-
branch: "pi-sub-sa-wt-guidance-1",
|
|
172
|
-
baseCommit: "abc123",
|
|
173
|
-
mainCwd: "/fake/cwd",
|
|
174
|
-
},
|
|
175
|
-
});
|
|
176
|
-
const ctx = makeCtx();
|
|
177
|
-
|
|
178
|
-
const resultPromise = runSpawn(record, "test task", opts, ctx);
|
|
179
|
-
await waitForSpawn();
|
|
180
|
-
|
|
181
|
-
// content 含 WORKTREE_GUIDANCE_PROMPT 的标题
|
|
182
|
-
expect(captured.content).toContain("Git Worktree");
|
|
183
|
-
// 含关键纠正信息:cwd 含完整项目代码(非临时沙箱)
|
|
184
|
-
expect(captured.content).toContain("complete project source code");
|
|
185
|
-
// 含防 cd 别处的行为指引
|
|
186
|
-
expect(captured.content).toContain("Do NOT");
|
|
187
|
-
|
|
188
|
-
const child = getLastSpawnedChild();
|
|
189
|
-
child.emit("close", 0);
|
|
190
|
-
await resultPromise;
|
|
191
|
-
});
|
|
192
|
-
|
|
193
|
-
it("非 worktree 模式 → content 不含 worktree 认知提示", async () => {
|
|
194
|
-
const record = makeRecord();
|
|
195
|
-
const opts = makeRunOpts(); // 不传 worktree
|
|
196
|
-
const ctx = makeCtx();
|
|
197
|
-
|
|
198
|
-
const resultPromise = runSpawn(record, "test task", opts, ctx);
|
|
199
|
-
await waitForSpawn();
|
|
200
|
-
|
|
201
|
-
expect(captured.content).not.toContain("Git Worktree");
|
|
202
|
-
|
|
203
|
-
const child = getLastSpawnedChild();
|
|
204
|
-
child.emit("close", 0);
|
|
205
|
-
await resultPromise;
|
|
206
|
-
});
|
|
207
|
-
});
|
|
@@ -1,92 +0,0 @@
|
|
|
1
|
-
// src/execution/__tests__/spawned-children.test.ts
|
|
2
|
-
//
|
|
3
|
-
// spawnedChildren Map + getChildByRecord + killAllSpawnedChildren 单元测试(M2-B1)。
|
|
4
|
-
//
|
|
5
|
-
// M2-B1 把 spawnedChildren 从 Set<ChildProcess> 改为 Map<recordId, ChildProcess>,
|
|
6
|
-
// 建立 record→child 映射(busy 投递定位活进程用,设计决策 6)。本文件测 Map API 核心行为:
|
|
7
|
-
// - getChildByRecord:set 后能查到;delete 后查不到;未注册返回 undefined
|
|
8
|
-
// - killAllSpawnedChildren:遍历 .values() kill + clear;跳过 child.killed=true;单 child kill 抛错不阻断
|
|
9
|
-
//
|
|
10
|
-
// 直接操作模块级 spawnedChildren(afterEach clear 防泄漏),不依赖 runSpawn/spawn/fs。
|
|
11
|
-
|
|
12
|
-
import type { ChildProcess } from "node:child_process";
|
|
13
|
-
|
|
14
|
-
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
15
|
-
|
|
16
|
-
const { loggerMock } = vi.hoisted(() => ({
|
|
17
|
-
loggerMock: { debug: vi.fn(), warn: vi.fn(), error: vi.fn(), info: vi.fn() },
|
|
18
|
-
}));
|
|
19
|
-
vi.mock("@zhushanwen/pi-extension-logger", () => ({ getLogger: () => loggerMock }));
|
|
20
|
-
|
|
21
|
-
import { getChildByRecord, killAllSpawnedChildren, spawnedChildren } from "../session-runner.ts";
|
|
22
|
-
|
|
23
|
-
/** 假 child:只关心 killed 标记 + kill 调用(killAllSpawnedChildren 遍历用)。 */
|
|
24
|
-
function makeFakeChild(killed = false): ChildProcess & { kill: ReturnType<typeof vi.fn> } {
|
|
25
|
-
return { killed, kill: vi.fn() } as unknown as ChildProcess & { kill: ReturnType<typeof vi.fn> };
|
|
26
|
-
}
|
|
27
|
-
|
|
28
|
-
describe("spawnedChildren Map + getChildByRecord (M2-B1)", () => {
|
|
29
|
-
afterEach(() => {
|
|
30
|
-
spawnedChildren.clear();
|
|
31
|
-
});
|
|
32
|
-
|
|
33
|
-
it("set(recordId, child) 后 getChildByRecord 能查到;未注册返回 undefined", () => {
|
|
34
|
-
const child = makeFakeChild();
|
|
35
|
-
spawnedChildren.set("rec-1", child);
|
|
36
|
-
expect(getChildByRecord("rec-1")).toBe(child);
|
|
37
|
-
expect(getChildByRecord("rec-2")).toBeUndefined();
|
|
38
|
-
});
|
|
39
|
-
|
|
40
|
-
it("delete(recordId) 后 getChildByRecord 返回 undefined(close/error 回调语义)", () => {
|
|
41
|
-
const child = makeFakeChild();
|
|
42
|
-
spawnedChildren.set("rec-1", child);
|
|
43
|
-
spawnedChildren.delete("rec-1");
|
|
44
|
-
expect(getChildByRecord("rec-1")).toBeUndefined();
|
|
45
|
-
});
|
|
46
|
-
|
|
47
|
-
it("killAllSpawnedChildren 遍历 .values() 对每个未 killed child kill + clear", () => {
|
|
48
|
-
const c1 = makeFakeChild();
|
|
49
|
-
const c2 = makeFakeChild();
|
|
50
|
-
spawnedChildren.set("rec-a", c1);
|
|
51
|
-
spawnedChildren.set("rec-b", c2);
|
|
52
|
-
|
|
53
|
-
const n = killAllSpawnedChildren();
|
|
54
|
-
|
|
55
|
-
expect(n).toBe(2);
|
|
56
|
-
expect(c1.kill).toHaveBeenCalledTimes(1);
|
|
57
|
-
expect(c2.kill).toHaveBeenCalledTimes(1);
|
|
58
|
-
expect(spawnedChildren.size).toBe(0);
|
|
59
|
-
});
|
|
60
|
-
|
|
61
|
-
it("killAllSpawnedChildren 跳过 child.killed=true(不重复 kill 已 kill 的)", () => {
|
|
62
|
-
const alive = makeFakeChild(false);
|
|
63
|
-
const alreadyKilled = makeFakeChild(true);
|
|
64
|
-
spawnedChildren.set("rec-1", alive);
|
|
65
|
-
spawnedChildren.set("rec-2", alreadyKilled);
|
|
66
|
-
|
|
67
|
-
const n = killAllSpawnedChildren();
|
|
68
|
-
|
|
69
|
-
expect(n).toBe(1);
|
|
70
|
-
expect(alive.kill).toHaveBeenCalledTimes(1);
|
|
71
|
-
expect(alreadyKilled.kill).not.toHaveBeenCalled();
|
|
72
|
-
expect(spawnedChildren.size).toBe(0);
|
|
73
|
-
});
|
|
74
|
-
|
|
75
|
-
it("killAllSpawnedChildren 对单个 child kill 抛错不阻断其他(best-effort catch)", () => {
|
|
76
|
-
const ok = makeFakeChild();
|
|
77
|
-
const throwy = makeFakeChild();
|
|
78
|
-
throwy.kill.mockImplementation(() => {
|
|
79
|
-
throw new Error("ESRCH");
|
|
80
|
-
});
|
|
81
|
-
spawnedChildren.set("rec-1", ok);
|
|
82
|
-
spawnedChildren.set("rec-2", throwy);
|
|
83
|
-
|
|
84
|
-
const n = killAllSpawnedChildren();
|
|
85
|
-
|
|
86
|
-
// throwy 抛错被 catch,不计入 n;ok 正常 kill。两者都调过 kill,Map 已 clear。
|
|
87
|
-
expect(n).toBe(1);
|
|
88
|
-
expect(ok.kill).toHaveBeenCalledTimes(1);
|
|
89
|
-
expect(throwy.kill).toHaveBeenCalledTimes(1);
|
|
90
|
-
expect(spawnedChildren.size).toBe(0);
|
|
91
|
-
});
|
|
92
|
-
});
|
|
@@ -1,150 +0,0 @@
|
|
|
1
|
-
// src/execution/__tests__/start-sync-model-guard.test.ts
|
|
2
|
-
//
|
|
3
|
-
// [U1 验收 3] start 同步期裁决:非全等 model 入参在 start 工具调用同步期 isError——
|
|
4
|
-
// 无 spawn、无子 session、record 不落盘。
|
|
5
|
-
//
|
|
6
|
-
// 用真实 SubagentService(tmpdir + mock pi)走 execute 全链路,断言:
|
|
7
|
-
// 1. execute 在 IDENTITY 解析(resolveModel 裁决)阶段 reject(问句式报错);
|
|
8
|
-
// 2. buildSpawnArgs 从未被调用(vi.mock spy 包装真实实现)——spawn 前置守卫的结构性证据;
|
|
9
|
-
// 3. node:child_process.spawn 从未被调用——无子进程;
|
|
10
|
-
// 4. pi.appendEntry 从未被调用——record 未创建(emitPendingRegister 未发生)。
|
|
11
|
-
//
|
|
12
|
-
// 放行路径(全等 → 裁决通过)由 model-ref.test.ts / model-resolver.test.ts 锁定,
|
|
13
|
-
// 全链路放行的 spawn 行为由 run-spawn-* 契约测试覆盖。
|
|
14
|
-
|
|
15
|
-
import { spawn } from "node:child_process";
|
|
16
|
-
import * as fs from "node:fs";
|
|
17
|
-
import * as os from "node:os";
|
|
18
|
-
import * as path from "node:path";
|
|
19
|
-
|
|
20
|
-
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
|
21
|
-
|
|
22
|
-
vi.mock("node:child_process", async () => {
|
|
23
|
-
const actual = await import("node:child_process");
|
|
24
|
-
return {
|
|
25
|
-
...actual,
|
|
26
|
-
spawn: vi.fn(() => {
|
|
27
|
-
throw new Error("spawn must not be called on sync-model rejection");
|
|
28
|
-
}),
|
|
29
|
-
};
|
|
30
|
-
});
|
|
31
|
-
|
|
32
|
-
// buildSpawnArgs 包装为 spy(保留真实实现)——「非全等拒单不触达 spawn 参数组装」的承重断言。
|
|
33
|
-
vi.mock("../session-runner.ts", async (importOriginal) => {
|
|
34
|
-
const actual = await importOriginal<typeof import("../session-runner.ts")>();
|
|
35
|
-
return { ...actual, buildSpawnArgs: vi.fn(actual.buildSpawnArgs) };
|
|
36
|
-
});
|
|
37
|
-
|
|
38
|
-
import { ModelConfigService } from "../model-config-service.ts";
|
|
39
|
-
import type { ModelInfo, ModelRegistryLike } from "../model-resolver.ts";
|
|
40
|
-
import { buildSpawnArgs } from "../session-runner.ts";
|
|
41
|
-
import type { PiLike } from "../subagent-service.ts";
|
|
42
|
-
import { SubagentService } from "../subagent-service.ts";
|
|
43
|
-
|
|
44
|
-
const mockSpawn = vi.mocked(spawn);
|
|
45
|
-
const mockBuildSpawnArgs = vi.mocked(buildSpawnArgs);
|
|
46
|
-
|
|
47
|
-
// ── 工具 ──
|
|
48
|
-
|
|
49
|
-
function makeModel(over: Partial<ModelInfo> = {}): ModelInfo {
|
|
50
|
-
return {
|
|
51
|
-
id: over.id ?? "GLM-5.3-Flash",
|
|
52
|
-
name: over.name ?? "GLM 5.3 Flash",
|
|
53
|
-
provider: over.provider ?? "zai-coding-cn",
|
|
54
|
-
reasoning: over.reasoning ?? false,
|
|
55
|
-
...over,
|
|
56
|
-
};
|
|
57
|
-
}
|
|
58
|
-
|
|
59
|
-
/** registry 快照:真实形态(大小写混合 id),无孪生(P-A2 放行快照形态)。 */
|
|
60
|
-
function makeRegistry(models: ModelInfo[]): ModelRegistryLike {
|
|
61
|
-
return {
|
|
62
|
-
getAvailable: () => models,
|
|
63
|
-
find: (provider, modelId) => models.find((m) => m.provider === provider && m.id === modelId),
|
|
64
|
-
hasConfiguredAuth: () => true,
|
|
65
|
-
};
|
|
66
|
-
}
|
|
67
|
-
|
|
68
|
-
function makePi(): PiLike {
|
|
69
|
-
return {
|
|
70
|
-
appendEntry: vi.fn(),
|
|
71
|
-
events: { emit: vi.fn() },
|
|
72
|
-
sendMessage: vi.fn(),
|
|
73
|
-
} as unknown as PiLike;
|
|
74
|
-
}
|
|
75
|
-
|
|
76
|
-
describe("start 同步期裁决(U1 验收 3:非全等 → isError 且无 spawn 无子 session)", () => {
|
|
77
|
-
let tmpDir: string;
|
|
78
|
-
let service: SubagentService;
|
|
79
|
-
let pi: PiLike;
|
|
80
|
-
|
|
81
|
-
beforeEach(() => {
|
|
82
|
-
vi.clearAllMocks();
|
|
83
|
-
tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "sw-start-guard-"));
|
|
84
|
-
const modelService = new ModelConfigService({ agentDir: tmpDir, cwd: tmpDir });
|
|
85
|
-
modelService.initModel({
|
|
86
|
-
modelRegistry: makeRegistry([makeModel()]),
|
|
87
|
-
sessionId: "guard-session",
|
|
88
|
-
});
|
|
89
|
-
service = new SubagentService({ cwd: tmpDir, modelService });
|
|
90
|
-
pi = makePi();
|
|
91
|
-
service.initSession({ pi, sessionId: "guard-session" });
|
|
92
|
-
});
|
|
93
|
-
|
|
94
|
-
afterEach(() => {
|
|
95
|
-
vi.restoreAllMocks();
|
|
96
|
-
fs.rmSync(tmpDir, { recursive: true, force: true });
|
|
97
|
-
});
|
|
98
|
-
|
|
99
|
-
it("小写变体入参 → execute 同步 reject(问句式报错),buildSpawnArgs/spawn/appendEntry 均未被调", async () => {
|
|
100
|
-
await expect(
|
|
101
|
-
service.execute({
|
|
102
|
-
task: "probe task",
|
|
103
|
-
slug: "guard",
|
|
104
|
-
model: "zai-coding-cn/glm-5.3-flash",
|
|
105
|
-
}),
|
|
106
|
-
).rejects.toThrow(/is not a registry entry.*Did you mean one of these\?/s);
|
|
107
|
-
|
|
108
|
-
// spawn 链路零触达:参数组装(buildSpawnArgs)与进程创建(spawn)都未发生
|
|
109
|
-
expect(mockBuildSpawnArgs).not.toHaveBeenCalled();
|
|
110
|
-
expect(mockSpawn).not.toHaveBeenCalled();
|
|
111
|
-
// record 未创建(emitPendingRegister 未发生)→ 无子 session 前置状态
|
|
112
|
-
expect(pi.appendEntry).not.toHaveBeenCalled();
|
|
113
|
-
});
|
|
114
|
-
|
|
115
|
-
it("错误含 case variant 首位建议 + 继承指引(可直接重发的合法串)", async () => {
|
|
116
|
-
let msg = "";
|
|
117
|
-
try {
|
|
118
|
-
await service.execute({
|
|
119
|
-
task: "probe task",
|
|
120
|
-
slug: "guard",
|
|
121
|
-
model: "zai-coding-cn/glm-5.3-flash",
|
|
122
|
-
});
|
|
123
|
-
} catch (e) {
|
|
124
|
-
msg = (e as Error).message;
|
|
125
|
-
}
|
|
126
|
-
expect(msg).toContain('zai-coding-cn/GLM-5.3-Flash ← case variant of "glm-5.3-flash"');
|
|
127
|
-
expect(msg).toMatch(/Or omit the `model` param to inherit the main agent model\./);
|
|
128
|
-
});
|
|
129
|
-
|
|
130
|
-
it("全等入参通过裁决(reject 不发生,放行链路触达 spawn 参数组装)——守卫零误伤对照", async () => {
|
|
131
|
-
// 放行路径会进入 kickOffBackground(detached);execute resolve 即视为裁决放行。
|
|
132
|
-
// spawn 本身由 FakeChild 缺失抛错收口——本断言只关心「越过了同步裁决」:
|
|
133
|
-
// execute 不 reject,且 buildSpawnArgs 已被调(或即将被调的 record 已创建)。
|
|
134
|
-
const promise = service.execute({
|
|
135
|
-
task: "probe task",
|
|
136
|
-
slug: "guard-exact",
|
|
137
|
-
model: "zai-coding-cn/GLM-5.3-Flash",
|
|
138
|
-
});
|
|
139
|
-
// 放行路径:裁决不抛(execute 最终可能因 mock 环境子进程收尾而 reject,
|
|
140
|
-
// 但错误不能是模型裁决错误)
|
|
141
|
-
try {
|
|
142
|
-
await promise;
|
|
143
|
-
} catch (e) {
|
|
144
|
-
expect((e as Error).message).not.toMatch(/is not a registry entry/);
|
|
145
|
-
expect((e as Error).message).not.toMatch(/ambiguous case variants/);
|
|
146
|
-
}
|
|
147
|
-
// 全等放行 → record 已创建(emitPendingRegister 发生)——与拒单路径形成对照
|
|
148
|
-
expect(pi.appendEntry).toHaveBeenCalled();
|
|
149
|
-
});
|
|
150
|
-
});
|
|
@@ -1,35 +0,0 @@
|
|
|
1
|
-
// src/execution/__tests__/startup-config-declaration.test.ts
|
|
2
|
-
//
|
|
3
|
-
// startupConfig 声明守护测试:断言 package.json `xyz-agent.startupConfig` 声明的
|
|
4
|
-
// content 与代码 DEFAULT_CONFIG 常量深相等,防止两侧任一改动未同步(漂移)。
|
|
5
|
-
// 机制:runtime 启动序列统一 ensure(extension-startup-config.ts),声明即首建内容。
|
|
6
|
-
//
|
|
7
|
-
// 运行:cd extensions/universal/subagent-workflow && npx vitest run src/execution/__tests__/startup-config-declaration.test.ts
|
|
8
|
-
|
|
9
|
-
import { readFileSync } from "node:fs";
|
|
10
|
-
import { join } from "node:path";
|
|
11
|
-
import { fileURLToPath } from "node:url";
|
|
12
|
-
|
|
13
|
-
import { describe, expect, it } from "vitest";
|
|
14
|
-
|
|
15
|
-
import { DEFAULT_CONFIG } from "../config.ts";
|
|
16
|
-
|
|
17
|
-
/** package.json `xyz-agent.startupConfig` 声明条目的最小形状(守护断言用)。 */
|
|
18
|
-
interface StartupConfigEntry {
|
|
19
|
-
path: string;
|
|
20
|
-
content: unknown;
|
|
21
|
-
}
|
|
22
|
-
|
|
23
|
-
const pkg = JSON.parse(
|
|
24
|
-
readFileSync(join(fileURLToPath(import.meta.url), "../../../../package.json"), "utf-8"),
|
|
25
|
-
) as { "xyz-agent"?: { startupConfig?: StartupConfigEntry[] } };
|
|
26
|
-
|
|
27
|
-
describe("startupConfig 声明守护", () => {
|
|
28
|
-
it("声明 content 与代码 DEFAULT_CONFIG 深相等", () => {
|
|
29
|
-
const entry = pkg["xyz-agent"]?.startupConfig?.find(
|
|
30
|
-
(e: StartupConfigEntry) => e.path === "subagents/config.json",
|
|
31
|
-
);
|
|
32
|
-
expect(entry).toBeDefined();
|
|
33
|
-
expect(entry?.content).toEqual(DEFAULT_CONFIG);
|
|
34
|
-
});
|
|
35
|
-
});
|