kushi-agents 6.4.0 → 6.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kushi-agents",
-  "version": "6.4.0",
+  "version": "6.5.0",
   "description": "Install Kushi — multi-source project evidence agent with Comprehensive Structured Capture (CSC) into weekly-only files across Email, Teams, OneNote, Loop, SharePoint, Meetings, CRM, ADO. Meetings retain a sibling verbatim/ audit folder. WorkIQ-only for M365 sources (Graph / m365_* FORBIDDEN as fallbacks; user-paste is first-class). Host-agnostic.",
   "type": "module",
   "bin": {

package/plugin/runners/bootstrap.mjs

@@ -234,8 +234,8 @@ async function interactiveSetup({ workspace, dryRun }) {
 function emit(obj) { process.stdout.write(JSON.stringify(obj) + '\n'); }
 
 const INTEGRATIONS_TEMPLATE = {
-  crm: { instance: 'https://iscrm.crm.dynamics.com', table: 'incidents', request_id: null, record_id: null },
-  ado: { organization: 'IndustrySolutions', project: 'IS Engagements', apiVersion: '7.1', engagement_id: null },
+  crm: { instance: 'https://iscrm.crm.dynamics.com', table: 'incidents', request_id: '<__FILL_ME_IN__>', record_id: '<__FILL_ME_IN__>' },
+  ado: { organization: 'IndustrySolutions', project: 'IS Engagements', apiVersion: '7.1', engagement_id: '<__FILL_ME_IN__>' },
   sharepoint: { allowed_tenants: [] },
 };
 

package/plugin/runners/discover.mjs

@@ -252,17 +252,26 @@ function applyRows(source, rows, currentBounds, currentInteg) {
   }
   if (source === 'meetings') {
     const existing = currentBounds.meetings?.joinUrls || [];
+    // v6.5.0: meeting boundaries MUST be real http(s) join URLs — pull-meetings
+    // can't resolve a subject string into a meeting at the WorkIQ layer (unlike
+    // teams chat topics). Reject anything that isn't a URL; track rejected
+    // subjects in `accepted` log via reason field for discover-report visibility.
+    const rejectedSubjects = [];
     const incoming = rows.map(r => {
       const url = r.join_url;
-      if (url && !isPlaceholder(url) && isValidValueFor('meetings', 'join_url', url) && url.startsWith('http')) return url;
+      if (url && !isPlaceholder(url) && isValidValueFor('meetings', 'join_url', url) && /^https?:\/\//.test(url)) return url;
       const subj = r.subject;
-      if (subj && !isPlaceholder(subj)) return subj;
+      if (subj && !isPlaceholder(subj)) rejectedSubjects.push(subj);
       return null;
     }).filter(Boolean);
     const merged = dedup([...existing, ...incoming]);
     const added = merged.filter(v => !existing.includes(v));
     if (added.length) accepted.push(...added);
-    return { boundariesPatch: added.length ? { meetings: { joinUrls: merged } } : null, accepted };
+    return {
+      boundariesPatch: added.length ? { meetings: { joinUrls: merged } } : null,
+      accepted,
+      rejected: rejectedSubjects.length ? rejectedSubjects.map(s => ({ subject: s, reason: 'no-join-url' })) : undefined,
+    };
   }
   if (source === 'onenote') {
     const existing = currentBounds.onenote?.section_file_ids || [];
@@ -303,7 +312,7 @@ function applyRows(source, rows, currentBounds, currentInteg) {
       isValidValueFor('crm', 'request_id', r.request_id) ||
       isValidValueFor('crm', 'incident_number', r.incident_number)
     );
-    if (!top) return { integrationsPatch: null, accepted: [] };
+    if (!top) return { integrationsPatch: null, accepted: [], unresolved: 'crm.request_id' };
     const id = isValidValueFor('crm', 'request_id', top.request_id) ? top.request_id : top.incident_number;
     const patch = { crm: { ...cur, request_id: id } };
     accepted.push(id);
@@ -318,7 +327,7 @@ function applyRows(source, rows, currentBounds, currentInteg) {
       isValidValueFor('ado', 'engagement_id', r.engagement_id) ||
       isValidValueFor('ado', 'work_item_id', r.work_item_id)
     );
-    if (!top) return { integrationsPatch: null, accepted: [] };
+    if (!top) return { integrationsPatch: null, accepted: [], unresolved: 'ado.engagement_id' };
     const id = isValidValueFor('ado', 'engagement_id', top.engagement_id) ? top.engagement_id : top.work_item_id;
     const patch = { ado: { ...cur, engagement_id: id } };
     accepted.push(id);
@@ -448,7 +457,7 @@ async function main() {
         : `${skipReason} after ${elapsed}ms: ${(e.message || '').split('\n')[0].slice(0, 200)}`;
       log(`  ${source}: ✗ ${detail}`);
     }
-    const { boundariesPatch, integrationsPatch, accepted } = applyRows(source, rows, bounds, integ);
+    const { boundariesPatch, integrationsPatch, accepted, rejected, unresolved } = applyRows(source, rows, bounds, integ);
     if (boundariesPatch) {
       Object.assign(bounds, mergeShallow(bounds, boundariesPatch));
       boundsDirty = true;
@@ -457,7 +466,7 @@ async function main() {
       Object.assign(integ, mergeShallow(integ, integrationsPatch));
       integDirty = true;
     }
-    sourceResults.push({ source, asked, found: rows.length, accepted, skipped_reason: skipReason });
+    sourceResults.push({ source, asked, found: rows.length, accepted, rejected, unresolved, skipped_reason: skipReason });
   }
 
   log(`done: ${sourceResults.filter(r => r.found > 0).length}/${total} sources returned data`);

package/plugin/runners/refresh.mjs

@@ -73,13 +73,14 @@ function emit(obj) { process.stdout.write(JSON.stringify(obj) + '\n'); }
  */
 export function buildTargets(merged) {
   const targets = [];
+  const isPlaceholder = (v) => v == null || /^<.*>$/.test(String(v).trim()) || /^(unknown|n\/a|none|null|tbd|todo)$/i.test(String(v).trim());
   // crm: from integrations
   const crm = merged.crm || {};
   const crmEntity = crm.request_id || crm.record_id;
-  if (crmEntity) targets.push({ source: 'crm', entity: String(crmEntity) });
+  if (crmEntity && !isPlaceholder(crmEntity)) targets.push({ source: 'crm', entity: String(crmEntity) });
   // ado
   const ado = merged.ado || {};
-  if (ado.engagement_id) targets.push({ source: 'ado', entity: String(ado.engagement_id) });
+  if (ado.engagement_id && !isPlaceholder(ado.engagement_id)) targets.push({ source: 'ado', entity: String(ado.engagement_id) });
   // email: per-user mailbox folders
   const email = merged.email || {};
   for (const f of (email.folders || [])) {