@bluecopa/harness 0.1.0-snapshot.27 → 0.1.0-snapshot.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bluecopa/harness",
-  "version": "0.1.0-snapshot.27",
+  "version": "0.1.0-snapshot.29",
   "description": "Provider-agnostic TypeScript agent framework",
   "license": "UNLICENSED",
   "scripts": {

package/src/arc/arc-loop.ts

@@ -78,6 +78,8 @@ export class ArcLoop {
   private readonly traceWriter: ((event: TraceEvent) => void) | undefined;
   private readonly tracedRunning = new Set<string>();
   private readonly reportedCompletions = new Set<string>();
+  /** Maps normalized action text → process ID for dedup. */
+  private readonly actionIndex = new Map<string, string>();
   private readonly processListeners: Promise<void>[] = [];
   private readonly skillRouter: SkillRouter | undefined;
   private skillSummaries: SkillSummary[] | null = null;
@@ -266,28 +268,71 @@ export class ArcLoop {
 
         if (call.toolName === 'Thread') {
           const request = this.toProcessRequest(call.args);
-          const proc = this.dispatch(request, signal);
-          this.trace({ type: 'process_created', id: proc.id, status: 'pending' });
-          const resultText = `Process ${proc.id} dispatched: "${request.action}" (model: ${request.model ?? 'medium'})`;
-          toolResultMessages.push({
-            role: 'tool',
-            content: resultText,
-            toolResults: [{
-              toolCallId,
-              toolName: 'Thread',
-              result: resultText,
-            }],
-          });
-          const dispatchEvent: ArcEvent = {
-            type: 'process_dispatched',
-            id: proc.id,
-            action: request.action,
-            model: proc.model,
-          };
-          if (request.label != null) {
-            (dispatchEvent as { label?: string }).label = request.label;
+
+          // Require profile when processProfiles are configured
+          const hasProfiles = this.config.processProfiles && Object.keys(this.config.processProfiles).length > 0;
+          if (hasProfiles && !request.profile) {
+            const names = Object.keys(this.config.processProfiles!).map(n => `"${n}"`).join(', ');
+            const resultText = `ERROR: profile parameter is required. Available profiles: ${names}. Re-call Thread with a profile.`;
+            toolResultMessages.push({
+              role: 'tool',
+              content: resultText,
+              toolResults: [{ toolCallId, toolName: 'Thread', result: resultText }],
+            });
+          } else if (hasProfiles && request.profile && !this.config.processProfiles![request.profile]) {
+            const names = Object.keys(this.config.processProfiles!).map(n => `"${n}"`).join(', ');
+            const resultText = `ERROR: unknown profile "${request.profile}". Available profiles: ${names}. Re-call Thread with a valid profile.`;
+            toolResultMessages.push({
+              role: 'tool',
+              content: resultText,
+              toolResults: [{ toolCallId, toolName: 'Thread', result: resultText }],
+            });
+          } else {
+
+          // Dedup: reject re-dispatch of identical actions unless the previous attempt failed
+          const normAction = request.action.trim().replace(/\s+/g, ' ');
+          const existingId = this.actionIndex.get(normAction);
+          const existing = existingId ? this.processes.get(existingId) : undefined;
+
+          if (existing && (existing.status === 'running' || existing.status === 'pending')) {
+            const resultText = `DUPLICATE — thread already running for this action (process ${existing.id}). Wait for it to complete.`;
+            toolResultMessages.push({
+              role: 'tool',
+              content: resultText,
+              toolResults: [{ toolCallId, toolName: 'Thread', result: resultText }],
+            });
+          } else if (existing && existing.status === 'completed' && existing.result) {
+            const ep = existing.result.episode;
+            const resultText = `DUPLICATE — this action already completed (process ${existing.id}, episodeId: ${ep.id}). Use contextEpisodeIds: ["${ep.id}"] to reference it.`;
+            toolResultMessages.push({
+              role: 'tool',
+              content: resultText,
+              toolResults: [{ toolCallId, toolName: 'Thread', result: resultText }],
+            });
+          } else {
+            // New dispatch (or retry of a failed action)
+            const proc = this.dispatch(request, signal);
+            this.actionIndex.set(normAction, proc.id);
+            this.trace({ type: 'process_created', id: proc.id, status: 'pending' });
+            const resultText = `Process ${proc.id} dispatched: "${request.action}" (model: ${request.model ?? 'medium'})`;
+            toolResultMessages.push({
+              role: 'tool',
+              content: resultText,
+              toolResults: [{ toolCallId, toolName: 'Thread', result: resultText }],
+            });
+            const dispatchEvent: ArcEvent = {
+              type: 'process_dispatched',
+              id: proc.id,
+              action: request.action,
+              model: proc.model,
+            };
+            if (request.label != null) {
+              (dispatchEvent as { label?: string }).label = request.label;
+            }
+            yield dispatchEvent;
           }
-          yield dispatchEvent;
+
+          } // end profile validation else
         } else if (call.toolName === 'Check') {
           const proc = this.processes.get(String(call.args.id));
           let resultText: string;