pi-crew 0.9.16 → 0.9.17
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/CHANGELOG.md +81 -0
- package/README.md +19 -0
- package/dist/build-meta.json +21790 -0
- package/dist/index.mjs +83070 -0
- package/dist/index.mjs.map +7 -0
- package/docs/A +358 -0
- package/docs/M-A +357 -0
- package/docs/REVIEW-FINDINGS-2026-06 +357 -0
- package/docs/Y +357 -0
- package/docs/a +357 -0
- package/docs/aA +358 -0
- package/docs/archive/README.md +91 -0
- package/docs/patterns/command-agent-skill.md +1 -1
- package/docs/skills/REFERENCE.md +0 -17
- package/docs//303/242mmaAAA/303/242 +357 -0
- package/index.bundle.ts +25 -0
- package/index.ts +95 -4
- package/package.json +15 -4
- package/skills/iterative-audit/SKILL.md +0 -1
- package/skills/widget-rendering/SKILL.md +17 -0
- package/src/adapters/claude-adapter.ts +1 -3
- package/src/adapters/export-util.ts +16 -16
- package/src/adapters/index.ts +5 -5
- package/src/agents/agent-config.ts +20 -14
- package/src/agents/agent-serializer.ts +1 -1
- package/src/agents/discover-agents.ts +83 -39
- package/src/benchmark/benchmark-runner.ts +51 -51
- package/src/benchmark/feedback-loop.ts +1 -1
- package/src/config/config.ts +166 -590
- package/src/config/defaults.ts +28 -2
- package/src/config/drift-detector.ts +11 -14
- package/src/config/markers.ts +201 -203
- package/src/config/resilient-parser.ts +14 -6
- package/src/config/role-tools.ts +3 -3
- package/src/config/suggestions.ts +5 -12
- package/src/config/types.ts +9 -25
- package/src/errors.ts +151 -157
- package/src/extension/action-suggestions.ts +54 -9
- package/src/extension/async-notifier.ts +43 -13
- package/src/extension/autonomous-policy.ts +77 -37
- package/src/extension/command-completions.ts +9 -8
- package/src/extension/context-status-injection.ts +14 -12
- package/src/extension/crew-autocomplete.ts +8 -16
- package/src/extension/crew-cleanup.ts +2 -1
- package/src/extension/crew-shortcuts.ts +9 -6
- package/src/extension/cross-extension-rpc.ts +136 -42
- package/src/extension/help.ts +1 -1
- package/src/extension/import-index.ts +10 -5
- package/src/extension/knowledge-injection.ts +88 -15
- package/src/extension/management.ts +89 -349
- package/src/extension/message-renderers.ts +3 -4
- package/src/extension/notification-router.ts +22 -5
- package/src/extension/notification-sink.ts +6 -3
- package/src/extension/pi-api.ts +17 -8
- package/src/extension/plan-orchestrate.ts +8 -27
- package/src/extension/project-init.ts +21 -6
- package/src/extension/register.ts +329 -975
- package/src/extension/registration/artifact-cleanup.ts +9 -4
- package/src/extension/registration/command-utils.ts +5 -1
- package/src/extension/registration/commands.ts +806 -314
- package/src/extension/registration/compaction-guard.ts +15 -15
- package/src/extension/registration/lifecycle.ts +259 -0
- package/src/extension/registration/observability.ts +293 -0
- package/src/extension/registration/subagent-helpers.ts +29 -7
- package/src/extension/registration/subagent-tools.ts +253 -44
- package/src/extension/registration/team-tool.ts +19 -84
- package/src/extension/registration/ui.ts +168 -0
- package/src/extension/registration/viewers.ts +64 -24
- package/src/extension/result-watcher.ts +18 -7
- package/src/extension/run-bundle-schema.ts +21 -6
- package/src/extension/run-export.ts +15 -6
- package/src/extension/run-import.ts +64 -29
- package/src/extension/run-index.ts +28 -9
- package/src/extension/run-maintenance.ts +32 -12
- package/src/extension/session-summary.ts +1 -1
- package/src/extension/team-manager-command.ts +64 -7
- package/src/extension/team-onboard.ts +151 -150
- package/src/extension/team-recommendation.ts +121 -21
- package/src/extension/team-tool/anchor.ts +31 -64
- package/src/extension/team-tool/api.ts +929 -141
- package/src/extension/team-tool/auto-summarize.ts +14 -26
- package/src/extension/team-tool/cache-control.ts +1 -5
- package/src/extension/team-tool/cancel.ts +138 -38
- package/src/extension/team-tool/chain-dispatch.ts +5 -13
- package/src/extension/team-tool/chain-executor.ts +29 -51
- package/src/extension/team-tool/config-patch.ts +1 -1
- package/src/extension/team-tool/context.ts +40 -20
- package/src/extension/team-tool/destructive-gate.ts +10 -5
- package/src/extension/team-tool/doctor.ts +170 -51
- package/src/extension/team-tool/explain.ts +228 -214
- package/src/extension/team-tool/failure-patterns.ts +3 -9
- package/src/extension/team-tool/goal-wrap.ts +71 -29
- package/src/extension/team-tool/goal.ts +185 -35
- package/src/extension/team-tool/handle-schedule.ts +34 -15
- package/src/extension/team-tool/handle-settings.ts +94 -56
- package/src/extension/team-tool/health-monitor.ts +27 -108
- package/src/extension/team-tool/inspect.ts +42 -9
- package/src/extension/team-tool/intent-policy.ts +4 -1
- package/src/extension/team-tool/lifecycle-actions.ts +244 -48
- package/src/extension/team-tool/orchestrate.ts +7 -18
- package/src/extension/team-tool/parallel-dispatch.ts +51 -29
- package/src/extension/team-tool/plan.ts +29 -5
- package/src/extension/team-tool/respond.ts +49 -16
- package/src/extension/team-tool/run-not-found.ts +3 -8
- package/src/extension/team-tool/run.ts +80 -317
- package/src/extension/team-tool/status.ts +135 -43
- package/src/extension/team-tool/workflow-manage.ts +92 -25
- package/src/extension/team-tool-types.ts +8 -1
- package/src/extension/team-tool.ts +142 -551
- package/src/extension/validate-resources.ts +39 -8
- package/src/hooks/registry.ts +66 -17
- package/src/hooks/types.ts +1 -1
- package/src/i18n.ts +25 -11
- package/src/observability/correlation.ts +17 -3
- package/src/observability/event-bus.ts +51 -56
- package/src/observability/event-to-metric.ts +117 -15
- package/src/observability/exporters/otlp-exporter.ts +56 -17
- package/src/observability/exporters/prometheus-exporter.ts +1 -1
- package/src/observability/metric-registry.ts +16 -4
- package/src/observability/metric-retention.ts +8 -2
- package/src/observability/metric-sink.ts +11 -3
- package/src/observability/metrics-primitives.ts +42 -9
- package/src/plugins/plugin-define.ts +2 -2
- package/src/plugins/plugin-registry.ts +25 -25
- package/src/plugins/plugins/index.ts +1 -1
- package/src/plugins/plugins/nextjs.ts +16 -16
- package/src/plugins/plugins/vite.ts +7 -11
- package/src/plugins/plugins/vitest.ts +6 -11
- package/src/runtime/adaptive-plan.ts +225 -40
- package/src/runtime/agent-control.ts +58 -15
- package/src/runtime/agent-memory.ts +15 -9
- package/src/runtime/agent-observability.ts +10 -2
- package/src/runtime/anchor-manager.ts +27 -25
- package/src/runtime/async-marker.ts +7 -1
- package/src/runtime/async-runner.ts +27 -23
- package/src/runtime/auto-summarize.ts +7 -13
- package/src/runtime/background-runner.ts +189 -251
- package/src/runtime/batch-barrier.ts +18 -16
- package/src/runtime/cancellation-token.ts +16 -6
- package/src/runtime/cancellation.ts +46 -8
- package/src/runtime/capability-inventory.ts +6 -4
- package/src/runtime/chain-parser.ts +44 -11
- package/src/runtime/chain-runner.ts +63 -69
- package/src/runtime/checkpoint.ts +12 -56
- package/src/runtime/child-pi.ts +343 -78
- package/src/runtime/coalesce-tasks.ts +268 -0
- package/src/runtime/code-summary.ts +71 -26
- package/src/runtime/compact-stages/index.ts +15 -5
- package/src/runtime/compact-stages/tail-capture-stage.ts +7 -2
- package/src/runtime/compact-stages/truncation-stage.ts +4 -1
- package/src/runtime/compaction-summary.ts +17 -10
- package/src/runtime/completion-guard.ts +33 -16
- package/src/runtime/crash-classification.ts +28 -7
- package/src/runtime/crash-recovery.ts +177 -37
- package/src/runtime/crew-agent-records.ts +145 -47
- package/src/runtime/crew-hooks.ts +9 -24
- package/src/runtime/cross-extension-rpc.ts +25 -23
- package/src/runtime/custom-tools/irc-tool.ts +43 -15
- package/src/runtime/custom-tools/submit-result-tool.ts +16 -5
- package/src/runtime/delivery-coordinator.ts +34 -7
- package/src/runtime/delta-conflict.ts +9 -21
- package/src/runtime/deterministic-ast.ts +1 -1
- package/src/runtime/diagnostic-export.ts +53 -20
- package/src/runtime/direct-run.ts +12 -2
- package/src/runtime/dwf-state-store.ts +1 -1
- package/src/runtime/dynamic-workflow-context.ts +186 -58
- package/src/runtime/dynamic-workflow-runner.ts +42 -20
- package/src/runtime/effectiveness.ts +16 -8
- package/src/runtime/errors/crew-errors.ts +7 -11
- package/src/runtime/event-stream-bridge.ts +9 -3
- package/src/runtime/foreground-control.ts +54 -14
- package/src/runtime/foreground-watchdog.ts +9 -6
- package/src/runtime/goal-achievement.ts +27 -10
- package/src/runtime/goal-evaluator.ts +54 -19
- package/src/runtime/goal-loop-runner.ts +280 -99
- package/src/runtime/goal-state-store.ts +27 -17
- package/src/runtime/green-contract.ts +11 -2
- package/src/runtime/group-join.ts +64 -31
- package/src/runtime/handoff-manager.ts +37 -42
- package/src/runtime/heartbeat-gradient.ts +10 -2
- package/src/runtime/heartbeat-watcher.ts +32 -6
- package/src/runtime/hidden-handoff.ts +12 -30
- package/src/runtime/important-line-classifier.ts +12 -2
- package/src/runtime/iteration-hooks.ts +25 -15
- package/src/runtime/live-agent-control.ts +55 -7
- package/src/runtime/live-agent-manager.ts +130 -27
- package/src/runtime/live-control-realtime.ts +23 -3
- package/src/runtime/live-irc.ts +4 -1
- package/src/runtime/live-session-health.ts +12 -4
- package/src/runtime/live-session-runtime.ts +379 -117
- package/src/runtime/manifest-cache.ts +28 -12
- package/src/runtime/mcp-proxy.ts +8 -16
- package/src/runtime/metric-parser.ts +1 -5
- package/src/runtime/model-fallback.ts +55 -18
- package/src/runtime/model-resolver.ts +2 -6
- package/src/runtime/model-scope.ts +19 -3
- package/src/runtime/notebook-helpers.ts +60 -62
- package/src/runtime/orphan-worker-registry.ts +37 -26
- package/src/runtime/output-validator.ts +18 -3
- package/src/runtime/overflow-recovery.ts +5 -4
- package/src/runtime/parallel-research.ts +29 -5
- package/src/runtime/parallel-utils.ts +9 -11
- package/src/runtime/path-overlap.ts +150 -0
- package/src/runtime/peer-dep.ts +12 -17
- package/src/runtime/per-write-validator.ts +1 -3
- package/src/runtime/phase-tracker.ts +342 -330
- package/src/runtime/pi-args.ts +26 -8
- package/src/runtime/pi-json-output.ts +1 -1
- package/src/runtime/pi-spawn.ts +30 -19
- package/src/runtime/pipeline-runner.ts +56 -55
- package/src/runtime/plan-templates.ts +5 -6
- package/src/runtime/policy-engine.ts +43 -7
- package/src/runtime/post-checks.ts +12 -4
- package/src/runtime/post-exit-stdio-guard.ts +2 -2
- package/src/runtime/process-lifecycle.ts +20 -10
- package/src/runtime/process-status.ts +16 -5
- package/src/runtime/progress-event-coalescer.ts +2 -1
- package/src/runtime/progress-tracker.ts +103 -103
- package/src/runtime/prose-compressor.ts +9 -11
- package/src/runtime/recovery-recipes.ts +118 -24
- package/src/runtime/replace.ts +25 -10
- package/src/runtime/resilient-edit.ts +10 -25
- package/src/runtime/result-extractor.ts +1 -2
- package/src/runtime/retry-executor.ts +20 -4
- package/src/runtime/retry-runner.ts +26 -50
- package/src/runtime/role-permission.ts +6 -1
- package/src/runtime/run-coalesced-task-group.ts +256 -0
- package/src/runtime/run-drift.ts +14 -15
- package/src/runtime/run-tracker.ts +6 -28
- package/src/runtime/runtime-policy.ts +1 -6
- package/src/runtime/runtime-resolver.ts +80 -15
- package/src/runtime/scheduler.ts +47 -18
- package/src/runtime/semaphore.ts +5 -7
- package/src/runtime/sensitive-paths.ts +2 -1
- package/src/runtime/session-usage.ts +1 -1
- package/src/runtime/settings-store.ts +16 -12
- package/src/runtime/sidechain-output.ts +6 -2
- package/src/runtime/single-agent-compose.ts +1 -4
- package/src/runtime/skill-effectiveness.ts +29 -97
- package/src/runtime/skill-instructions.ts +40 -83
- package/src/runtime/stale-reconciler.ts +73 -197
- package/src/runtime/stream-preview.ts +1 -1
- package/src/runtime/streaming-output.ts +1 -1
- package/src/runtime/subagent-manager.ts +44 -169
- package/src/runtime/subprocess-tool-registry.ts +4 -1
- package/src/runtime/supervisor-contact.ts +10 -5
- package/src/runtime/task-display.ts +19 -5
- package/src/runtime/task-graph-scheduler.ts +95 -21
- package/src/runtime/task-graph.ts +5 -11
- package/src/runtime/task-health.ts +54 -47
- package/src/runtime/task-id.ts +11 -18
- package/src/runtime/task-output-context.ts +265 -81
- package/src/runtime/task-packet.ts +12 -20
- package/src/runtime/task-quality.ts +6 -14
- package/src/runtime/task-runner/context-retrieval.ts +4 -13
- package/src/runtime/task-runner/live-executor.ts +114 -36
- package/src/runtime/task-runner/output-splitter.ts +152 -0
- package/src/runtime/task-runner/progress.ts +50 -12
- package/src/runtime/task-runner/prompt-builder.ts +31 -7
- package/src/runtime/task-runner/prompt-pipeline.ts +31 -7
- package/src/runtime/task-runner/result-utils.ts +3 -1
- package/src/runtime/task-runner/retrieval-orchestrator.ts +310 -0
- package/src/runtime/task-runner/run-projection.ts +27 -8
- package/src/runtime/task-runner/state-helpers.ts +29 -8
- package/src/runtime/task-runner/tail-read.ts +2 -6
- package/src/runtime/task-runner.ts +180 -326
- package/src/runtime/team-runner-artifacts.ts +13 -0
- package/src/runtime/team-runner.ts +456 -974
- package/src/runtime/tool-output-pruner.ts +6 -9
- package/src/runtime/tool-progress.ts +11 -21
- package/src/runtime/verification-gates.ts +33 -24
- package/src/runtime/verification-integrity.ts +1 -4
- package/src/runtime/verification-worktree.ts +62 -12
- package/src/runtime/worker-heartbeat.ts +5 -1
- package/src/runtime/worker-startup.ts +29 -5
- package/src/runtime/workflow-state.ts +17 -9
- package/src/runtime/workspace-lock.ts +30 -35
- package/src/runtime/workspace-tree.ts +20 -32
- package/src/runtime/yield-handler.ts +43 -9
- package/src/runtime/zombie-scanner.ts +5 -4
- package/src/schema/config-schema.ts +266 -172
- package/src/schema/team-tool-schema.ts +29 -74
- package/src/schema/validation-types.ts +25 -17
- package/src/skills/discover-skills.ts +35 -10
- package/src/skills/skill-templates.ts +109 -27
- package/src/skills/validate.ts +26 -8
- package/src/state/active-run-registry.ts +87 -28
- package/src/state/artifact-store.ts +9 -5
- package/src/state/atomic-write-v2.ts +85 -63
- package/src/state/atomic-write.ts +105 -26
- package/src/state/blob-store.ts +47 -20
- package/src/state/contracts.ts +20 -5
- package/src/state/crew-init.ts +5 -22
- package/src/state/decision-ledger.ts +19 -73
- package/src/state/event-log-rotation.ts +41 -15
- package/src/state/event-log.ts +168 -53
- package/src/state/event-reconstructor.ts +11 -1
- package/src/state/gitignore-manager.ts +2 -8
- package/src/state/health-store.ts +57 -57
- package/src/state/hook-instinct-bridge.ts +1 -1
- package/src/state/hook-integrations.ts +1 -1
- package/src/state/instinct-store.ts +17 -5
- package/src/state/jsonl-writer.ts +1 -1
- package/src/state/locks.ts +23 -9
- package/src/state/mailbox.ts +151 -28
- package/src/state/observation-store.ts +20 -14
- package/src/state/run-cache.ts +142 -135
- package/src/state/run-graph.ts +6 -15
- package/src/state/run-metrics.ts +5 -18
- package/src/state/schedule.ts +15 -9
- package/src/state/state-store.ts +129 -45
- package/src/state/task-claims.ts +12 -2
- package/src/state/tiered-eval.ts +52 -43
- package/src/state/types-eval.ts +2 -2
- package/src/state/types.ts +24 -20
- package/src/state/usage.ts +20 -4
- package/src/state/worker-atomic-writer.ts +6 -3
- package/src/subagents/index.ts +2 -2
- package/src/teams/discover-teams.ts +21 -6
- package/src/tools/safe-bash-extension.ts +5 -6
- package/src/tools/safe-bash.ts +10 -11
- package/src/types/new-api-types.ts +6 -10
- package/src/ui/agent-management-overlay.ts +52 -40
- package/src/ui/card-colors.ts +13 -4
- package/src/ui/crew-footer.ts +8 -7
- package/src/ui/crew-select-list.ts +1 -1
- package/src/ui/dashboard-panes/agents-pane.ts +41 -25
- package/src/ui/dashboard-panes/cancellation-pane.ts +1 -1
- package/src/ui/dashboard-panes/capability-pane.ts +32 -15
- package/src/ui/dashboard-panes/health-pane.ts +2 -1
- package/src/ui/dashboard-panes/metrics-pane.ts +4 -1
- package/src/ui/dashboard-panes/progress-pane.ts +12 -9
- package/src/ui/heartbeat-aggregator.ts +15 -4
- package/src/ui/keybinding-map.ts +40 -7
- package/src/ui/live-conversation-overlay.ts +36 -20
- package/src/ui/live-duration.ts +1 -4
- package/src/ui/live-run-sidebar.ts +88 -25
- package/src/ui/loaders.ts +2 -8
- package/src/ui/mascot.ts +20 -36
- package/src/ui/overlays/agent-picker-overlay.ts +11 -3
- package/src/ui/overlays/confirm-overlay.ts +4 -2
- package/src/ui/overlays/help-overlay.ts +25 -14
- package/src/ui/overlays/mailbox-compose-overlay.ts +48 -11
- package/src/ui/overlays/mailbox-compose-preview.ts +20 -5
- package/src/ui/overlays/mailbox-detail-overlay.ts +27 -6
- package/src/ui/pi-ui-compat.ts +7 -7
- package/src/ui/powerbar-publisher.ts +120 -42
- package/src/ui/render-diff.ts +10 -3
- package/src/ui/render-scheduler.ts +12 -4
- package/src/ui/run-action-dispatcher.ts +81 -18
- package/src/ui/run-dashboard.ts +172 -76
- package/src/ui/run-event-bus.ts +59 -37
- package/src/ui/run-snapshot-cache.ts +276 -86
- package/src/ui/settings-overlay.ts +361 -74
- package/src/ui/status-colors.ts +12 -1
- package/src/ui/syntax-highlight.ts +1 -1
- package/src/ui/theme-adapter.ts +16 -14
- package/src/ui/theme-discovery.ts +13 -2
- package/src/ui/tool-progress-formatter.ts +7 -7
- package/src/ui/tool-render.ts +128 -56
- package/src/ui/tool-renderers/brief-mode.ts +45 -27
- package/src/ui/tool-renderers/index.ts +109 -43
- package/src/ui/transcript-cache.ts +32 -6
- package/src/ui/transcript-entries.ts +25 -23
- package/src/ui/transcript-viewer.ts +78 -29
- package/src/ui/widget/index.ts +99 -40
- package/src/ui/widget/widget-formatters.ts +13 -6
- package/src/ui/widget/widget-model.ts +19 -8
- package/src/ui/widget/widget-renderer.ts +23 -13
- package/src/ui/widget/widget-types.ts +1 -1
- package/src/utils/bm25-search.ts +199 -199
- package/src/utils/conflict-detect.ts +22 -21
- package/src/utils/env-filter.ts +14 -7
- package/src/utils/file-coalescer.ts +5 -1
- package/src/utils/fingerprint.ts +3 -6
- package/src/utils/frontmatter.ts +3 -1
- package/src/utils/fs-watch.ts +2 -6
- package/src/utils/gh-protocol.ts +119 -44
- package/src/utils/git.ts +13 -15
- package/src/utils/guards.ts +2 -5
- package/src/utils/ids.ts +9 -2
- package/src/utils/incremental-reader.ts +14 -3
- package/src/utils/internal-error.ts +2 -1
- package/src/utils/names.ts +12 -3
- package/src/utils/paths.ts +49 -5
- package/src/utils/project-detector.ts +2 -2
- package/src/utils/redaction.ts +29 -19
- package/src/utils/resolve-shell.ts +9 -7
- package/src/utils/run-watcher-registry.ts +19 -31
- package/src/utils/safe-paths.ts +45 -24
- package/src/utils/scan-cache.ts +9 -2
- package/src/utils/session-utils.ts +2 -4
- package/src/utils/sleep.ts +12 -5
- package/src/utils/sse-parser.ts +5 -17
- package/src/utils/visual.ts +52 -32
- package/src/workflows/discover-workflows.ts +41 -109
- package/src/workflows/intermediate-store.ts +5 -21
- package/src/workflows/preflight-validator.ts +6 -33
- package/src/workflows/topology-analyzer.ts +5 -21
- package/src/workflows/workflow-config.ts +7 -6
- package/src/worktree/branch-freshness.ts +66 -8
- package/src/worktree/cleanup.ts +130 -20
- package/src/worktree/worktree-manager.ts +212 -69
- package/skills/artifact-analysis-loop/SKILL.md +0 -303
- package/skills/detection-pipeline-design/SKILL.md +0 -286
- package/skills/hunting-investigation-loop/SKILL.md +0 -402
- package/skills/incident-playbook-construction/SKILL.md +0 -384
- package/skills/security-review/SKILL.md +0 -561
- package/skills/threat-hypothesis-framework/SKILL.md +0 -176
- package/skills/ui-render-performance/SKILL.md +0 -58
- /package/docs/{followup-review-round3-2026-05-12.md → archive/followup-review-round3-2026-05-12.md} +0 -0
- /package/docs/{followup-review-round4-2026-05-13.md → archive/followup-review-round4-2026-05-13.md} +0 -0
- /package/docs/{pi-crew-bugs.md → archive/pi-crew-bugs.md} +0 -0
- /package/docs/{pi-crew-test-final.md → archive/pi-crew-test-final.md} +0 -0
- /package/docs/{pi-crew-test-results.md → archive/pi-crew-test-results.md} +0 -0
- /package/docs/{pi-crew-test-round2.md → archive/pi-crew-test-round2.md} +0 -0
- /package/docs/{pi-crew-test-round4.md → archive/pi-crew-test-round4.md} +0 -0
- /package/docs/{pi-crew-test-round5.md → archive/pi-crew-test-round5.md} +0 -0
- /package/docs/{pi-crew-test-round6.md → archive/pi-crew-test-round6.md} +0 -0
- /package/docs/{pi-crew-v0.5.10-audit-fix-plan.md → archive/pi-crew-v0.5.10-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.11-audit-fix-plan.md → archive/pi-crew-v0.5.11-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.12-audit-fix-plan.md → archive/pi-crew-v0.5.12-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.13-audit-fix-plan.md → archive/pi-crew-v0.5.13-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.14-audit-fix-plan.md → archive/pi-crew-v0.5.14-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.16-audit-fix-plan.md → archive/pi-crew-v0.5.16-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.17-audit-fix-plan.md → archive/pi-crew-v0.5.17-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.5-audit-fix-plan.md → archive/pi-crew-v0.5.5-audit-fix-plan.md} +0 -0
- /package/docs/{pi-crew-v0.5.9-audit-fix-plan.md → archive/pi-crew-v0.5.9-audit-fix-plan.md} +0 -0
- /package/docs/{pi-mono-opportunities.md → archive/pi-mono-opportunities.md} +0 -0
- /package/docs/{pi-mono-review.md → archive/pi-mono-review.md} +0 -0
- /package/docs/{pi-subagent4-comparison.md → archive/pi-subagent4-comparison.md} +0 -0
- /package/docs/{pi-subagents3-deep-analysis.md → archive/pi-subagents3-deep-analysis.md} +0 -0
|
@@ -1,5 +1,6 @@
|
|
|
1
|
-
import type { CrewAgentRecord } from "./crew-agent-runtime.ts";
|
|
2
1
|
import type { TeamRunManifest } from "../state/types.ts";
|
|
2
|
+
import type { CrewAgentRecord } from "./crew-agent-runtime.ts";
|
|
3
|
+
|
|
3
4
|
export { hasAsyncStartMarker } from "./async-marker.ts";
|
|
4
5
|
|
|
5
6
|
export interface ProcessLiveness {
|
|
@@ -34,7 +35,12 @@ export function checkProcessLiveness(pid: number | undefined): ProcessLiveness {
|
|
|
34
35
|
return { pid, alive: true, detail: "process is alive" };
|
|
35
36
|
} catch (error) {
|
|
36
37
|
const nodeError = error as NodeJS.ErrnoException;
|
|
37
|
-
if (nodeError.code === "EPERM")
|
|
38
|
+
if (nodeError.code === "EPERM")
|
|
39
|
+
return {
|
|
40
|
+
pid,
|
|
41
|
+
alive: true,
|
|
42
|
+
detail: "process exists but permission is denied",
|
|
43
|
+
};
|
|
38
44
|
if (nodeError.code === "ESRCH") return { pid, alive: false, detail: "process does not exist" };
|
|
39
45
|
const message = error instanceof Error ? error.message : String(error);
|
|
40
46
|
return { pid, alive: false, detail: message };
|
|
@@ -56,7 +62,12 @@ export function isFinishedRunStatus(status: string): boolean {
|
|
|
56
62
|
*/
|
|
57
63
|
const ORPHANED_NO_PROGRESS_MS = 5 * 60 * 1000;
|
|
58
64
|
|
|
59
|
-
export function isLikelyOrphanedActiveRun(
|
|
65
|
+
export function isLikelyOrphanedActiveRun(
|
|
66
|
+
run: TeamRunManifest,
|
|
67
|
+
agents: CrewAgentRecord[] = [],
|
|
68
|
+
now = Date.now(),
|
|
69
|
+
staleMs = ORPHANED_ACTIVE_RUN_MS,
|
|
70
|
+
): boolean {
|
|
60
71
|
if (!isActiveRunStatus(run.status)) return false;
|
|
61
72
|
if (run.async?.pid !== undefined) return false;
|
|
62
73
|
const updatedAt = new Date(run.updatedAt).getTime();
|
|
@@ -91,8 +102,8 @@ function hasDurableActiveAgentEvidence(agent: CrewAgentRecord): boolean {
|
|
|
91
102
|
// spawned-but-never-executed agents should not appear as active.
|
|
92
103
|
return Boolean(
|
|
93
104
|
(agent.progress && (agent.progress.toolCount > 0 || agent.progress.recentOutput.length > 0)) ||
|
|
94
|
-
|
|
95
|
-
|
|
105
|
+
(agent.jsonEvents && agent.jsonEvents > 0) ||
|
|
106
|
+
(agent.toolUses && agent.toolUses > 0),
|
|
96
107
|
);
|
|
97
108
|
}
|
|
98
109
|
return false;
|
|
@@ -38,6 +38,7 @@ export function shouldAppendProgressEventUpdate(input: ProgressEventCoalesceInpu
|
|
|
38
38
|
if (numericIncrease(input.previous.turns, input.next.turns) > 0) return { shouldAppend: true, reason: "turns_increased" };
|
|
39
39
|
const tokenIncrease = numericIncrease(input.previous.tokens, input.next.tokens);
|
|
40
40
|
if (tokenIncrease >= (input.tokenThreshold ?? DEFAULT_TOKEN_THRESHOLD)) return { shouldAppend: true, reason: "tokens_increased" };
|
|
41
|
-
if (input.lastAppendMs === undefined || input.nowMs - input.lastAppendMs >= input.minIntervalMs)
|
|
41
|
+
if (input.lastAppendMs === undefined || input.nowMs - input.lastAppendMs >= input.minIntervalMs)
|
|
42
|
+
return { shouldAppend: true, reason: "interval" };
|
|
42
43
|
return { shouldAppend: false, reason: "coalesced" };
|
|
43
44
|
}
|
|
@@ -2,123 +2,123 @@ import type { AgentSessionEvent } from "@earendil-works/pi-coding-agent";
|
|
|
2
2
|
import { crewEventBus } from "../observability/event-bus.ts";
|
|
3
3
|
|
|
4
4
|
export interface AgentProgress {
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
5
|
+
toolCalls: number;
|
|
6
|
+
currentTool: string | null;
|
|
7
|
+
toolStartTime: number | null;
|
|
8
|
+
errors: string[];
|
|
9
|
+
turns: number;
|
|
10
|
+
tokens: { input: number; output: number };
|
|
11
|
+
status: "idle" | "running" | "completed" | "error";
|
|
12
12
|
}
|
|
13
13
|
|
|
14
14
|
export class ProgressTracker {
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
15
|
+
private sessions = new Map<
|
|
16
|
+
string,
|
|
17
|
+
{
|
|
18
|
+
unsubscribe: () => void;
|
|
19
|
+
progress: AgentProgress;
|
|
20
|
+
}
|
|
21
|
+
>();
|
|
19
22
|
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
23
|
+
track(
|
|
24
|
+
session: {
|
|
25
|
+
subscribe: (listener: (event: AgentSessionEvent) => void) => () => void;
|
|
26
|
+
},
|
|
27
|
+
agentId: string,
|
|
28
|
+
runId: string,
|
|
29
|
+
): AgentProgress {
|
|
30
|
+
if (this.sessions.has(agentId)) {
|
|
31
|
+
return this.sessions.get(agentId)!.progress;
|
|
32
|
+
}
|
|
28
33
|
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
34
|
+
const progress: AgentProgress = {
|
|
35
|
+
toolCalls: 0,
|
|
36
|
+
currentTool: null,
|
|
37
|
+
toolStartTime: null,
|
|
38
|
+
errors: [],
|
|
39
|
+
turns: 0,
|
|
40
|
+
tokens: { input: 0, output: 0 },
|
|
41
|
+
status: "running",
|
|
42
|
+
};
|
|
38
43
|
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
44
|
+
const unsubscribe = session.subscribe((event: AgentSessionEvent) => {
|
|
45
|
+
this.handleEvent(event, progress, agentId, runId);
|
|
46
|
+
});
|
|
42
47
|
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
48
|
+
this.sessions.set(agentId, { unsubscribe, progress });
|
|
49
|
+
return progress;
|
|
50
|
+
}
|
|
46
51
|
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
agentId,
|
|
62
|
-
payload: { ...progress },
|
|
63
|
-
timestamp: Date.now(),
|
|
64
|
-
});
|
|
65
|
-
break;
|
|
52
|
+
private handleEvent(event: AgentSessionEvent, progress: AgentProgress, agentId: string, runId: string): void {
|
|
53
|
+
switch (event.type) {
|
|
54
|
+
case "tool_execution_start":
|
|
55
|
+
progress.toolCalls++;
|
|
56
|
+
progress.currentTool = event.toolName;
|
|
57
|
+
progress.toolStartTime = Date.now();
|
|
58
|
+
crewEventBus.emit({
|
|
59
|
+
type: "agent:progress",
|
|
60
|
+
runId,
|
|
61
|
+
agentId,
|
|
62
|
+
payload: { ...progress },
|
|
63
|
+
timestamp: Date.now(),
|
|
64
|
+
});
|
|
65
|
+
break;
|
|
66
66
|
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
67
|
+
case "tool_execution_end":
|
|
68
|
+
progress.currentTool = null;
|
|
69
|
+
progress.toolStartTime = null;
|
|
70
|
+
if (event.isError) {
|
|
71
|
+
progress.errors.push(String(event.result ?? "Unknown error"));
|
|
72
|
+
crewEventBus.emit({
|
|
73
|
+
type: "agent:error",
|
|
74
|
+
runId,
|
|
75
|
+
agentId,
|
|
76
|
+
payload: String(event.result ?? "Unknown error"),
|
|
77
|
+
timestamp: Date.now(),
|
|
78
|
+
});
|
|
79
|
+
}
|
|
80
|
+
crewEventBus.emit({
|
|
81
|
+
type: "agent:progress",
|
|
82
|
+
runId,
|
|
83
|
+
agentId,
|
|
84
|
+
payload: { ...progress },
|
|
85
|
+
timestamp: Date.now(),
|
|
86
|
+
});
|
|
87
|
+
break;
|
|
88
88
|
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
89
|
+
case "turn_start":
|
|
90
|
+
progress.turns++;
|
|
91
|
+
break;
|
|
92
92
|
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
93
|
+
case "agent_end":
|
|
94
|
+
progress.status = "completed";
|
|
95
|
+
crewEventBus.emit({
|
|
96
|
+
type: "agent:complete",
|
|
97
|
+
runId,
|
|
98
|
+
agentId,
|
|
99
|
+
payload: { ...progress },
|
|
100
|
+
timestamp: Date.now(),
|
|
101
|
+
});
|
|
102
|
+
break;
|
|
103
103
|
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
|
|
104
|
+
case "agent_start":
|
|
105
|
+
progress.status = "running";
|
|
106
|
+
break;
|
|
107
|
+
}
|
|
108
|
+
}
|
|
109
109
|
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
110
|
+
untrack(agentId: string): void {
|
|
111
|
+
const tracked = this.sessions.get(agentId);
|
|
112
|
+
if (tracked) {
|
|
113
|
+
tracked.unsubscribe();
|
|
114
|
+
this.sessions.delete(agentId);
|
|
115
|
+
}
|
|
116
|
+
}
|
|
117
117
|
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
118
|
+
getProgress(agentId: string): AgentProgress | undefined {
|
|
119
|
+
return this.sessions.get(agentId)?.progress;
|
|
120
|
+
}
|
|
121
121
|
}
|
|
122
122
|
|
|
123
123
|
// Export singleton instance
|
|
124
|
-
export const globalProgressTracker = new ProgressTracker();
|
|
124
|
+
export const globalProgressTracker = new ProgressTracker();
|
|
@@ -14,14 +14,14 @@
|
|
|
14
14
|
// ---------------------------------------------------------------------------
|
|
15
15
|
|
|
16
16
|
const PROTECTED_PATTERNS: readonly RegExp[] = [
|
|
17
|
-
/```[\s\S]*?```/g,
|
|
18
|
-
/`[^`\n]+`/g,
|
|
19
|
-
/\bhttps?:\/\/\S+/gi,
|
|
20
|
-
/\b[\w.-]*[/\\][\w./\\-]+/g,
|
|
21
|
-
/\b[A-Z][A-Z0-9]*(?:_[A-Z][A-Z0-9]*)+\b/g,
|
|
22
|
-
/\b\w+(?:\.\w+)+\(\)/g,
|
|
23
|
-
/[A-Za-z_][A-Za-z0-9_]*\s*\([^)]*\)/g,
|
|
24
|
-
/\b\d+\.\d+\.\d+\b/g,
|
|
17
|
+
/```[\s\S]*?```/g, // fenced code blocks
|
|
18
|
+
/`[^`\n]+`/g, // inline code
|
|
19
|
+
/\bhttps?:\/\/\S+/gi, // URLs
|
|
20
|
+
/\b[\w.-]*[/\\][\w./\\-]+/g, // paths with / or \
|
|
21
|
+
/\b[A-Z][A-Z0-9]*(?:_[A-Z][A-Z0-9]*)+\b/g, // CONSTANT_CASE
|
|
22
|
+
/\b\w+(?:\.\w+)+\(\)/g, // dotted.method() calls
|
|
23
|
+
/[A-Za-z_][A-Za-z0-9_]*\s*\([^)]*\)/g, // function calls: name(args)
|
|
24
|
+
/\b\d+\.\d+\.\d+\b/g, // version numbers x.y.z
|
|
25
25
|
];
|
|
26
26
|
|
|
27
27
|
// ---------------------------------------------------------------------------
|
|
@@ -143,9 +143,7 @@ export function compressProse(text: string): CompressResult {
|
|
|
143
143
|
|
|
144
144
|
const originalLength = text.length;
|
|
145
145
|
const compressedLength = restored.length;
|
|
146
|
-
const savingsPercent = originalLength > 0
|
|
147
|
-
? Math.round(((originalLength - compressedLength) / originalLength) * 100 * 100) / 100
|
|
148
|
-
: 0;
|
|
146
|
+
const savingsPercent = originalLength > 0 ? Math.round(((originalLength - compressedLength) / originalLength) * 100 * 100) / 100 : 0;
|
|
149
147
|
|
|
150
148
|
return {
|
|
151
149
|
compressed: restored,
|
|
@@ -1,7 +1,27 @@
|
|
|
1
1
|
import type { PolicyDecision, PolicyDecisionReason } from "../state/types.ts";
|
|
2
2
|
|
|
3
|
-
export type FailureScenario =
|
|
4
|
-
|
|
3
|
+
export type FailureScenario =
|
|
4
|
+
| "trust_prompt_unresolved"
|
|
5
|
+
| "prompt_misdelivery"
|
|
6
|
+
| "stale_branch"
|
|
7
|
+
| "compile_red_cross_crate"
|
|
8
|
+
| "mcp_handshake_failure"
|
|
9
|
+
| "partial_plugin_startup"
|
|
10
|
+
| "provider_failure"
|
|
11
|
+
| "task_failed"
|
|
12
|
+
| "worker_stale"
|
|
13
|
+
| "green_unsatisfied";
|
|
14
|
+
export type RecoveryStep =
|
|
15
|
+
| "accept_trust_prompt"
|
|
16
|
+
| "redirect_prompt_to_agent"
|
|
17
|
+
| "rebase_branch"
|
|
18
|
+
| "clean_build"
|
|
19
|
+
| "retry_mcp_handshake"
|
|
20
|
+
| "restart_plugin"
|
|
21
|
+
| "restart_worker"
|
|
22
|
+
| "rerun_task"
|
|
23
|
+
| "collect_verification_evidence"
|
|
24
|
+
| "escalate_to_human";
|
|
5
25
|
export type RecoveryResultState = "planned" | "skipped" | "escalation_required";
|
|
6
26
|
|
|
7
27
|
export interface RecoveryRecipe {
|
|
@@ -28,26 +48,91 @@ export interface RecoveryLedger {
|
|
|
28
48
|
|
|
29
49
|
export function scenarioForPolicyReason(reason: PolicyDecisionReason): FailureScenario {
|
|
30
50
|
switch (reason) {
|
|
31
|
-
case "branch_stale":
|
|
32
|
-
|
|
33
|
-
case "
|
|
34
|
-
|
|
35
|
-
|
|
51
|
+
case "branch_stale":
|
|
52
|
+
return "stale_branch";
|
|
53
|
+
case "worker_stale":
|
|
54
|
+
return "worker_stale";
|
|
55
|
+
case "green_unsatisfied":
|
|
56
|
+
return "green_unsatisfied";
|
|
57
|
+
case "task_failed":
|
|
58
|
+
return "task_failed";
|
|
59
|
+
default:
|
|
60
|
+
return "provider_failure";
|
|
36
61
|
}
|
|
37
62
|
}
|
|
38
63
|
|
|
39
64
|
export function recipeFor(scenario: FailureScenario): RecoveryRecipe {
|
|
40
65
|
switch (scenario) {
|
|
41
|
-
case "trust_prompt_unresolved":
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
case "
|
|
49
|
-
|
|
50
|
-
|
|
66
|
+
case "trust_prompt_unresolved":
|
|
67
|
+
return {
|
|
68
|
+
scenario,
|
|
69
|
+
steps: ["accept_trust_prompt"],
|
|
70
|
+
maxAttempts: 1,
|
|
71
|
+
escalationPolicy: "alert_human",
|
|
72
|
+
};
|
|
73
|
+
case "prompt_misdelivery":
|
|
74
|
+
return {
|
|
75
|
+
scenario,
|
|
76
|
+
steps: ["redirect_prompt_to_agent"],
|
|
77
|
+
maxAttempts: 1,
|
|
78
|
+
escalationPolicy: "alert_human",
|
|
79
|
+
};
|
|
80
|
+
case "stale_branch":
|
|
81
|
+
return {
|
|
82
|
+
scenario,
|
|
83
|
+
steps: ["rebase_branch", "clean_build"],
|
|
84
|
+
maxAttempts: 1,
|
|
85
|
+
escalationPolicy: "alert_human",
|
|
86
|
+
};
|
|
87
|
+
case "compile_red_cross_crate":
|
|
88
|
+
return {
|
|
89
|
+
scenario,
|
|
90
|
+
steps: ["clean_build"],
|
|
91
|
+
maxAttempts: 1,
|
|
92
|
+
escalationPolicy: "alert_human",
|
|
93
|
+
};
|
|
94
|
+
case "mcp_handshake_failure":
|
|
95
|
+
return {
|
|
96
|
+
scenario,
|
|
97
|
+
steps: ["retry_mcp_handshake"],
|
|
98
|
+
maxAttempts: 1,
|
|
99
|
+
escalationPolicy: "abort",
|
|
100
|
+
};
|
|
101
|
+
case "partial_plugin_startup":
|
|
102
|
+
return {
|
|
103
|
+
scenario,
|
|
104
|
+
steps: ["restart_plugin", "retry_mcp_handshake"],
|
|
105
|
+
maxAttempts: 1,
|
|
106
|
+
escalationPolicy: "log_and_continue",
|
|
107
|
+
};
|
|
108
|
+
case "worker_stale":
|
|
109
|
+
return {
|
|
110
|
+
scenario,
|
|
111
|
+
steps: ["restart_worker"],
|
|
112
|
+
maxAttempts: 1,
|
|
113
|
+
escalationPolicy: "alert_human",
|
|
114
|
+
};
|
|
115
|
+
case "green_unsatisfied":
|
|
116
|
+
return {
|
|
117
|
+
scenario,
|
|
118
|
+
steps: ["collect_verification_evidence"],
|
|
119
|
+
maxAttempts: 1,
|
|
120
|
+
escalationPolicy: "alert_human",
|
|
121
|
+
};
|
|
122
|
+
case "task_failed":
|
|
123
|
+
return {
|
|
124
|
+
scenario,
|
|
125
|
+
steps: ["rerun_task"],
|
|
126
|
+
maxAttempts: 1,
|
|
127
|
+
escalationPolicy: "alert_human",
|
|
128
|
+
};
|
|
129
|
+
case "provider_failure":
|
|
130
|
+
return {
|
|
131
|
+
scenario,
|
|
132
|
+
steps: ["restart_worker"],
|
|
133
|
+
maxAttempts: 1,
|
|
134
|
+
escalationPolicy: "alert_human",
|
|
135
|
+
};
|
|
51
136
|
}
|
|
52
137
|
}
|
|
53
138
|
|
|
@@ -93,17 +178,26 @@ export interface RerunDecision {
|
|
|
93
178
|
reason: string;
|
|
94
179
|
}
|
|
95
180
|
|
|
96
|
-
export function shouldRerunFailedTask(
|
|
97
|
-
task: { policy?: { retryCount?: number } },
|
|
98
|
-
limits?: { maxRetriesPerTask?: number },
|
|
99
|
-
): RerunDecision {
|
|
181
|
+
export function shouldRerunFailedTask(task: { policy?: { retryCount?: number } }, limits?: { maxRetriesPerTask?: number }): RerunDecision {
|
|
100
182
|
const maxRetries = limits?.maxRetriesPerTask ?? 0;
|
|
101
183
|
const retryCount = task.policy?.retryCount ?? 0;
|
|
102
184
|
if (maxRetries <= 0) {
|
|
103
|
-
return {
|
|
185
|
+
return {
|
|
186
|
+
rerun: false,
|
|
187
|
+
newRetryCount: retryCount,
|
|
188
|
+
reason: "maxRetriesPerTask not set (opt-in) — no whole-task rerun",
|
|
189
|
+
};
|
|
104
190
|
}
|
|
105
191
|
if (retryCount >= maxRetries) {
|
|
106
|
-
return {
|
|
192
|
+
return {
|
|
193
|
+
rerun: false,
|
|
194
|
+
newRetryCount: retryCount,
|
|
195
|
+
reason: `retryCount ${retryCount} >= maxRetriesPerTask ${maxRetries} — rerun budget exhausted`,
|
|
196
|
+
};
|
|
107
197
|
}
|
|
108
|
-
return {
|
|
198
|
+
return {
|
|
199
|
+
rerun: true,
|
|
200
|
+
newRetryCount: retryCount + 1,
|
|
201
|
+
reason: `whole-task rerun ${retryCount + 1}/${maxRetries}`,
|
|
202
|
+
};
|
|
109
203
|
}
|
package/src/runtime/replace.ts
CHANGED
|
@@ -462,12 +462,7 @@ const REPLACERS: Replacer[] = [
|
|
|
462
462
|
* occurrences. Only safe for exact matches (simple replacer).
|
|
463
463
|
* @returns ReplaceResult with the new content and match strategy info.
|
|
464
464
|
*/
|
|
465
|
-
export function replace(
|
|
466
|
-
content: string,
|
|
467
|
-
oldString: string,
|
|
468
|
-
newString: string,
|
|
469
|
-
options?: { replaceAll?: boolean },
|
|
470
|
-
): ReplaceResult {
|
|
465
|
+
export function replace(content: string, oldString: string, newString: string, options?: { replaceAll?: boolean }): ReplaceResult {
|
|
471
466
|
if (oldString.length === 0) {
|
|
472
467
|
return { content, changed: false, strategy: "none", count: 0 };
|
|
473
468
|
}
|
|
@@ -482,7 +477,12 @@ export function replace(
|
|
|
482
477
|
const result = content.replaceAll(oldString, newString);
|
|
483
478
|
if (result !== content) {
|
|
484
479
|
const count = countOccurrences(content, oldString);
|
|
485
|
-
return {
|
|
480
|
+
return {
|
|
481
|
+
content: result,
|
|
482
|
+
changed: true,
|
|
483
|
+
strategy: "simple-replaceAll",
|
|
484
|
+
count,
|
|
485
|
+
};
|
|
486
486
|
}
|
|
487
487
|
}
|
|
488
488
|
|
|
@@ -494,7 +494,12 @@ export function replace(
|
|
|
494
494
|
if (idx === lastIdx) {
|
|
495
495
|
// Exactly one occurrence
|
|
496
496
|
const result = content.slice(0, idx) + newString + content.slice(idx + oldString.length);
|
|
497
|
-
return {
|
|
497
|
+
return {
|
|
498
|
+
content: result,
|
|
499
|
+
changed: true,
|
|
500
|
+
strategy: "simple",
|
|
501
|
+
count: 1,
|
|
502
|
+
};
|
|
498
503
|
}
|
|
499
504
|
// Multiple occurrences — fall through to fuzzy strategies
|
|
500
505
|
// (we never auto-pick among duplicates)
|
|
@@ -529,7 +534,12 @@ export function replace(
|
|
|
529
534
|
.replace("Replacer", "")
|
|
530
535
|
.replace(/([a-z])([A-Z])/g, "$1-$2")
|
|
531
536
|
.toLowerCase();
|
|
532
|
-
return {
|
|
537
|
+
return {
|
|
538
|
+
content: result,
|
|
539
|
+
changed: true,
|
|
540
|
+
strategy: `${strategyName}-replaceAll`,
|
|
541
|
+
count: totalCount,
|
|
542
|
+
};
|
|
533
543
|
}
|
|
534
544
|
continue;
|
|
535
545
|
}
|
|
@@ -544,7 +554,12 @@ export function replace(
|
|
|
544
554
|
.replace("Replacer", "")
|
|
545
555
|
.replace(/([a-z])([A-Z])/g, "$1-$2")
|
|
546
556
|
.toLowerCase();
|
|
547
|
-
return {
|
|
557
|
+
return {
|
|
558
|
+
content: result,
|
|
559
|
+
changed: true,
|
|
560
|
+
strategy: strategyName,
|
|
561
|
+
count: 1,
|
|
562
|
+
};
|
|
548
563
|
}
|
|
549
564
|
}
|
|
550
565
|
// Multiple candidates — skip this strategy (safety first)
|
|
@@ -23,7 +23,7 @@
|
|
|
23
23
|
* }
|
|
24
24
|
*/
|
|
25
25
|
import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
|
|
26
|
-
import {
|
|
26
|
+
import { type ReplaceResult, replace } from "../runtime/replace.ts";
|
|
27
27
|
|
|
28
28
|
interface ToolLike {
|
|
29
29
|
name: string;
|
|
@@ -50,13 +50,7 @@ interface EditResult {
|
|
|
50
50
|
[key: string]: unknown;
|
|
51
51
|
}
|
|
52
52
|
|
|
53
|
-
const NOT_FOUND_PATTERNS = [
|
|
54
|
-
/old_string not found/i,
|
|
55
|
-
/oldstring not found/i,
|
|
56
|
-
/no match/i,
|
|
57
|
-
/could not find/i,
|
|
58
|
-
/string not found/i,
|
|
59
|
-
];
|
|
53
|
+
const NOT_FOUND_PATTERNS = [/old_string not found/i, /oldstring not found/i, /no match/i, /could not find/i, /string not found/i];
|
|
60
54
|
|
|
61
55
|
function isNotFoundResult(result: unknown): boolean {
|
|
62
56
|
if (!result || typeof result !== "object") return false;
|
|
@@ -72,7 +66,7 @@ function isPiDiffLoaded(pi: ExtensionAPI): boolean {
|
|
|
72
66
|
const piAny = pi as any;
|
|
73
67
|
const extensions = piAny?.extensions ?? piAny?._extensions ?? [];
|
|
74
68
|
const names = Array.isArray(extensions)
|
|
75
|
-
? extensions.map((e: unknown) => (typeof e === "string" ? e : (e as { name?: string })?.name ?? ""))
|
|
69
|
+
? extensions.map((e: unknown) => (typeof e === "string" ? e : ((e as { name?: string })?.name ?? "")))
|
|
76
70
|
: Object.keys(extensions);
|
|
77
71
|
return names.some((n: string) => typeof n === "string" && n.includes("pi-diff"));
|
|
78
72
|
} catch {
|
|
@@ -94,7 +88,7 @@ export function wrapEditWithResilientReplace(pi: ExtensionAPI, tools?: { edit: T
|
|
|
94
88
|
return false;
|
|
95
89
|
}
|
|
96
90
|
|
|
97
|
-
const t = tools ?? (
|
|
91
|
+
const t = tools ?? (pi as unknown as { tools?: { edit?: ToolLike } }).tools;
|
|
98
92
|
if (!t?.edit?.execute) return false;
|
|
99
93
|
|
|
100
94
|
const nativeExecute = t.edit.execute.bind(t.edit);
|
|
@@ -117,12 +111,7 @@ export function wrapEditWithResilientReplace(pi: ExtensionAPI, tools?: { edit: T
|
|
|
117
111
|
|
|
118
112
|
return true;
|
|
119
113
|
|
|
120
|
-
async function retryWithReplace(
|
|
121
|
-
params: EditParams,
|
|
122
|
-
toolCallId: string,
|
|
123
|
-
signal: any,
|
|
124
|
-
onUpdate: any,
|
|
125
|
-
): Promise<unknown> {
|
|
114
|
+
async function retryWithReplace(params: EditParams, toolCallId: string, signal: any, onUpdate: any): Promise<unknown> {
|
|
126
115
|
const filePath = params.path ?? params.filePath;
|
|
127
116
|
const oldStr = params.oldString ?? params.old_string;
|
|
128
117
|
const newStr = params.newString ?? params.new_string;
|
|
@@ -139,18 +128,14 @@ export function wrapEditWithResilientReplace(pi: ExtensionAPI, tools?: { edit: T
|
|
|
139
128
|
try {
|
|
140
129
|
content = await fs.readFile(filePath, "utf8");
|
|
141
130
|
} catch (readErr) {
|
|
142
|
-
throw new Error(
|
|
143
|
-
`resilient edit: could not read ${filePath}: ${
|
|
144
|
-
readErr instanceof Error ? readErr.message : String(readErr)
|
|
145
|
-
}`,
|
|
146
|
-
);
|
|
131
|
+
throw new Error(`resilient edit: could not read ${filePath}: ${readErr instanceof Error ? readErr.message : String(readErr)}`);
|
|
147
132
|
}
|
|
148
133
|
|
|
149
|
-
const result: ReplaceResult = replace(content, oldStr, newStr, {
|
|
134
|
+
const result: ReplaceResult = replace(content, oldStr, newStr, {
|
|
135
|
+
replaceAll,
|
|
136
|
+
});
|
|
150
137
|
if (!result.changed) {
|
|
151
|
-
throw new Error(
|
|
152
|
-
`old_string not found (resilient cascade exhausted, strategy=${result.strategy})`,
|
|
153
|
-
);
|
|
138
|
+
throw new Error(`old_string not found (resilient cascade exhausted, strategy=${result.strategy})`);
|
|
154
139
|
}
|
|
155
140
|
|
|
156
141
|
await fs.writeFile(filePath, result.content, "utf8");
|