@yemi33/minions 0.1.2315 → 0.1.2317

package/engine/meeting.js

@@ -231,18 +231,14 @@ function resolveMeetingContributionContent(output, structuredCompletion) {
   return resolveStructuredMeetingContent(structuredCompletion);
 }
 
-function truncateMeetingContext(text, maxBytes, label) {
-  return shared.truncateTextBytes(text, maxBytes, `\n\n_...${label} truncated — review the meeting transcript if needed._`);
-}
-
 function formatMeetingContributions(entries, agents, emptyText, label, maxBytes) {
   const pairs = Object.entries(typeof entries === 'object' && entries ? entries : {});
   if (pairs.length === 0) return emptyText;
   const perEntryBytes = Math.max(1024, Math.floor(maxBytes / Math.max(1, pairs.length)));
   const combined = pairs.map(([agent, value]) =>
-    `### ${agents[agent]?.name || agent}\n\n${truncateMeetingContext(value?.content || emptyText, perEntryBytes, `${label} entry`)}`
+    `### ${agents[agent]?.name || agent}\n\n${shared.truncateTextBytes(value?.content || emptyText, perEntryBytes, `\n\n_...${label} entry truncated — review the meeting transcript if needed._`)}`
   ).join('\n\n---\n\n');
-  return truncateMeetingContext(combined, maxBytes, label);
+  return shared.truncateTextBytes(combined, maxBytes, `\n\n_...${label} truncated — review the meeting transcript if needed._`);
 }
 
 function stripMeetingSummaryMarkdown(text) {
@@ -542,10 +538,10 @@ function discoverMeetingWork(config) {
       const key = `${concludePrefix}${concluder}`;
       if (activeKeys.has(key)) continue;
 
-      const humanNotes = truncateMeetingContext(
+      const humanNotes = shared.truncateTextBytes(
         (Array.isArray(meeting.humanNotes) ? meeting.humanNotes : []).map(n => '- ' + n).join('\n'),
         ENGINE_DEFAULTS.maxMeetingHumanNotesBytes,
-        'human meeting notes'
+        '\n\n_...human meeting notes truncated — review the meeting transcript if needed._'
       );
       const allFindings = formatMeetingContributions(
         meeting.findings,
@@ -605,10 +601,10 @@ function discoverMeetingWork(config) {
       const key = `meeting-${meeting.id}-r${round}-${agentId}`;
       if (activeKeys.has(key)) continue;
 
-      const humanNotes = truncateMeetingContext(
+      const humanNotes = shared.truncateTextBytes(
         (meeting.humanNotes || []).map(n => '- ' + n).join('\n'),
         ENGINE_DEFAULTS.maxMeetingHumanNotesBytes,
-        'human meeting notes'
+        '\n\n_...human meeting notes truncated — review the meeting transcript if needed._'
       );
       const vars = {
         agent_name: agents[agentId]?.name || agentId,

package/engine/pipeline.js

@@ -26,10 +26,6 @@ const NOTES_INBOX_DIR = path.join(MINIONS_DIR, 'notes', 'inbox');
 const NOTES_ARCHIVE_DIR = path.join(MINIONS_DIR, 'notes', 'archive');
 const CONFIG_PATH = path.join(MINIONS_DIR, 'config.json');
 
-function truncatePipelineContext(text, maxBytes, label) {
-  return shared.truncateTextBytes(text, maxBytes, `\n\n_...${label} truncated — inspect the upstream artifacts if needed._`);
-}
-
 // ── Pipeline CRUD ────────────────────────────────────────────────────────────
 
 function getPipelines() {
@@ -619,10 +615,10 @@ async function executePlanStage(stage, stageState, run, config, pipeline = {}) {
     try {
       const mtg = safeJson(path.join(MEETINGS_DIR, mid + '.json'));
       if (mtg) {
-        const transcript = truncatePipelineContext(
+        const transcript = shared.truncateTextBytes(
           (mtg.transcript || []).map(formatTranscriptEntry).join('\n\n---\n\n'),
           ENGINE_DEFAULTS.maxPipelineMeetingContextBytes,
-          `meeting transcript for ${mtg.title || mid}`
+          `\n\n_...meeting transcript for ${mtg.title || mid} truncated — inspect the upstream artifacts if needed._`
         );
         meetingContext += '# Meeting: ' + (mtg.title || mid) + '\n\n**Agenda:** ' + (mtg.agenda || '') + '\n\n' + transcript + '\n\n';
       }
@@ -633,15 +629,15 @@ async function executePlanStage(stage, stageState, run, config, pipeline = {}) {
     for (const depId of stage.dependsOn) {
       const depStage = run.stages[depId];
       if (depStage?.output && !depStage.artifacts?.meetings?.length) {
-        meetingContext += '## From: ' + depId + '\n\n' + truncatePipelineContext(
+        meetingContext += '## From: ' + depId + '\n\n' + shared.truncateTextBytes(
           depStage.output,
           ENGINE_DEFAULTS.maxPipelineMeetingContextBytes,
-          `pipeline stage output from ${depId}`
+          `\n\n_...pipeline stage output from ${depId} truncated — inspect the upstream artifacts if needed._`
         ) + '\n\n';
       }
     }
   }
-  meetingContext = truncatePipelineContext(meetingContext, ENGINE_DEFAULTS.maxPipelineMeetingContextBytes, 'pipeline meeting context');
+  meetingContext = shared.truncateTextBytes(meetingContext, ENGINE_DEFAULTS.maxPipelineMeetingContextBytes, '\n\n_...pipeline meeting context truncated — inspect the upstream artifacts if needed._');
 
   // Use LLM to generate a structured plan (same approach as dashboard "Create Plan from Meeting" button)
   let content = '';
@@ -1259,7 +1255,7 @@ module.exports = {
   getPipelineRuns, getActiveRun, startRun, updateRunStage, upsertRunStage, completeRun,
   discoverPipelineWork,
   evaluateCondition, // exported for testing
-  truncatePipelineContext, executeTaskStage, executePlanStage, executeScheduleStage, executeApiStage, executeMeetingStage, executeMergePrsStage, isStageComplete, resolveTemplate, // exported for testing
+  executeTaskStage, executePlanStage, executeScheduleStage, executeApiStage, executeMeetingStage, executeMergePrsStage, isStageComplete, resolveTemplate, // exported for testing
   _resolvePipelineProjects, // exported for testing
   _findMeetingsInRun, _findExistingPlanForMeeting, _findExistingPrdForPlan, _canonicalPlanName, // exported for testing
 };

package/engine/pr-clone-keep.js

@@ -69,24 +69,9 @@ const GIT_OP_TIMEOUT_MS = 60 * 1000;
  * `PrActionError` (400) on missing/unrecognized input — mirrors the read-only,
  * choice-surface, and temp-clone paths so the dashboard reflects a 400, not a 500.
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object') {
-    if (input.host && input.slug && input.number) return input; // already normalized
-    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
-function _prId(ref) {
-  if (!ref) return null;
-  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
-}
+const _prId = prResolve.prIdFromRef;
 
 /** Filesystem-safe slug for the persistent project dir name (from the repo name). */
 function _safeDirSlug(name) {

package/engine/pr-devbox.js

@@ -60,24 +60,9 @@ const DEFAULT_REMOTE_TMP_ROOT = 'C:/minions-devbox-tmp';
  * `PrActionError` (400) on missing/unrecognized input — mirrors the other
  * projectless paths so the dashboard reflects a 400, not a 500.
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object') {
-    if (input.host && input.slug && input.number) return input; // already normalized
-    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
-function _prId(ref) {
-  if (!ref) return null;
-  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
-}
+const _prId = prResolve.prIdFromRef;
 
 /** Filesystem-safe slug for a PR id (used in the remote dir name). */
 function _safeDirSlug(prId) {

package/engine/pr-fix-target.js

@@ -143,21 +143,7 @@ function _routedRecord(plan, target, extra = {}) {
  * a normalized PR ref. Throws `PrActionError` (400) on missing/unrecognized input
  * so the dashboard handler reflects it as a 400 (matching the read-only path).
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object' && input.host && input.slug && input.number) {
-    return input; // already normalized
-  }
-  if (input && typeof input === 'object' && input.ref && input.ref.host) {
-    return input.ref; // a plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
 // ── Configured-project detection ──────────────────────────────────────────────
 
@@ -309,7 +295,7 @@ function buildPrFixTargetModal(plan) {
 function planPrFix(input = {}, opts = {}) {
   const ref = _resolveRef(input);
   const prUrl = opts.prUrl || (input && typeof input === 'object' ? input.url : undefined) || null;
-  const prId = ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
+  const prId = prResolve.prIdFromRef(ref);
 
   const project = findConfiguredProjectForRef(ref, opts.config);
   if (project) {

package/engine/pr-remote-patch.js

@@ -59,24 +59,9 @@ const REMOTE_PATCH_NO_VALIDATION_LABEL =
  * `PrActionError` (400) on missing/unrecognized input — mirrors the read-only,
  * choice-surface, and temp-clone paths so the dashboard reflects a 400, not 500.
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object') {
-    if (input.host && input.slug && input.number) return input; // already normalized
-    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
-function _prId(ref) {
-  if (!ref) return null;
-  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
-}
+const _prId = prResolve.prIdFromRef;
 
 // ── Patch normalization (the trivial-single-file guardrail) ───────────────────
 

package/engine/pr-resolve.js

@@ -102,6 +102,42 @@ function normalizePrRef(input) {
   return _normalizeFromScope(parsed.scope, parsed.prNumber);
 }
 
+// ── Shared ref resolution for the PR-action executors ───────────────────────
+
+/**
+ * Resolve `{ url }` (raw PR URL / canonical id), an already-normalized ref, or a
+ * plan/handle carrying `.ref` into a normalized PR ref. Throws `PrActionError`
+ * (400) on missing/unrecognized input so callers reflect a 400, not a 500.
+ *
+ * Shared by the 6 PR-action executor modules (pr-clone-keep, pr-devbox,
+ * pr-remote-patch, pr-temp-clone, pr-track, pr-fix-target), which previously
+ * each defined a byte-identical copy of this function. `PrActionError` must be
+ * passed in by the caller — pr-resolve.js does not import engine/pr-action.js
+ * (that would introduce a new cross-module dependency in that direction).
+ */
+function resolvePrActionRef(input, PrActionError) {
+  if (input && typeof input === 'object') {
+    if (input.host && input.slug && input.number) return input; // already normalized
+    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
+  }
+  const rawUrl = input && typeof input === 'object'
+    ? (typeof input.url === 'string' ? input.url.trim() : '')
+    : (typeof input === 'string' ? input.trim() : '');
+  if (!rawUrl) throw new PrActionError('url required');
+  const ref = normalizePrRef(rawUrl);
+  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
+  return ref;
+}
+
+/**
+ * Derive the canonical `host:slug#number` PR id string from a normalized ref
+ * (or its already-computed `.id`). Returns null when `ref` is falsy.
+ */
+function prIdFromRef(ref) {
+  if (!ref) return null;
+  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
+}
+
 // ── Config (for the unknown-slug account fallback) ───────────────────────────
 
 function _readConfig(opts = {}) {
@@ -432,6 +468,8 @@ async function fetchPrPayload(refOrInput, opts = {}) {
 
 module.exports = {
   normalizePrRef,
+  resolvePrActionRef,
+  prIdFromRef,
   fetchPrPayload,
   fetchPrBranches,
   // Exported for testing.

package/engine/pr-temp-clone.js

@@ -57,24 +57,9 @@ const GIT_OP_TIMEOUT_MS = 60 * 1000;
  * `PrActionError` (400) on missing/unrecognized input — mirrors the read-only
  * and choice-surface paths so the dashboard reflects a 400, not a 500.
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object') {
-    if (input.host && input.slug && input.number) return input; // already normalized
-    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
-function _prId(ref) {
-  if (!ref) return null;
-  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
-}
+const _prId = prResolve.prIdFromRef;
 
 /** Filesystem-safe slug for a PR id (used in the ephemeral dir name). */
 function _safeDirSlug(prId) {

package/engine/pr-track.js

@@ -56,24 +56,9 @@ const TRACK_RECOMMENDED_TARGET = 'clone-keep';
  * missing/unrecognized input so the dashboard reflects a 400 — mirrors the
  * read-only and choice-surface paths.
  */
-function _resolveRef(input) {
-  if (input && typeof input === 'object') {
-    if (input.host && input.slug && input.number) return input; // already normalized
-    if (input.ref && input.ref.host) return input.ref;          // plan/handle carrying a ref
-  }
-  const rawUrl = input && typeof input === 'object'
-    ? (typeof input.url === 'string' ? input.url.trim() : '')
-    : (typeof input === 'string' ? input.trim() : '');
-  if (!rawUrl) throw new PrActionError('url required');
-  const ref = prResolve.normalizePrRef(rawUrl);
-  if (!ref) throw new PrActionError(`unrecognized PR reference: ${JSON.stringify(rawUrl.slice(0, 120))}`);
-  return ref;
-}
+const _resolveRef = (input) => prResolve.resolvePrActionRef(input, PrActionError);
 
-function _prId(ref) {
-  if (!ref) return null;
-  return ref.id || (ref.host && ref.slug && ref.number ? `${ref.host}:${ref.slug}#${ref.number}` : null);
-}
+const _prId = prResolve.prIdFromRef;
 
 // ── Choice surface (Clone & keep implied) ─────────────────────────────────────
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2315",
+  "version": "0.1.2317",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"