wogiflow 2.6.3 → 2.7.0

package/lib/workspace-intelligence.js

@@ -246,6 +246,81 @@ function addSharedDecision(workspaceRoot, title, content) {
   fs.writeFileSync(decisionsPath, existing + entry);
 }
 
+// ============================================================
+// S5: Decision Propagation (Active Broadcast)
+// ============================================================
+
+/**
+ * Add a shared decision AND broadcast it to all workspace members.
+ * Extends addSharedDecision with active notification.
+ *
+ * @param {string} workspaceRoot
+ * @param {string} fromRepo — repo that created the decision
+ * @param {string} title
+ * @param {string} content
+ * @param {Object} manifest — workspace manifest for member discovery
+ * @returns {{ saved: boolean, broadcastCount: number }}
+ */
+function propagateDecision(workspaceRoot, fromRepo, title, content, manifest) {
+  // 1. Save the decision
+  addSharedDecision(workspaceRoot, title, content);
+
+  // 2. Broadcast to all members
+  let broadcastCount = 0;
+  if (manifest && manifest.members) {
+    const { broadcastDecision, saveMessage } = require('./workspace-messages');
+    const targetRepos = Object.keys(manifest.members).filter(n => n !== fromRepo);
+    const messages = broadcastDecision(fromRepo, title, content, targetRepos);
+
+    for (const msg of messages) {
+      try {
+        saveMessage(workspaceRoot, msg);
+        broadcastCount++;
+      } catch (_err) {
+        // Best effort
+      }
+    }
+  }
+
+  return { saved: true, broadcastCount };
+}
+
+/**
+ * Check if there are new shared decisions since a given timestamp.
+ * Used by session-start hooks to inject new decisions into worker context.
+ *
+ * @param {string} workspaceRoot
+ * @param {string} sinceDate — ISO date string
+ * @returns {Array<{ title: string, content: string, date: string }>}
+ */
+function getNewDecisionsSince(workspaceRoot, sinceDate) {
+  const decisions = getSharedDecisions(workspaceRoot);
+  if (!decisions) return [];
+
+  const sinceTime = new Date(sinceDate).getTime();
+  const results = [];
+
+  // Parse decisions.md for entries with *Added: YYYY-MM-DD* markers
+  const sections = decisions.split(/^### /m).filter(Boolean);
+  for (const section of sections) {
+    const lines = section.trim().split('\n');
+    const title = lines[0]?.trim() || '';
+    const dateMatch = section.match(/\*Added:\s*(\d{4}-\d{2}-\d{2})\*/);
+    if (dateMatch) {
+      const entryDate = new Date(dateMatch[1]).getTime();
+      if (entryDate > sinceTime) {
+        results.push({
+          title,
+          content: lines.slice(1).join('\n').replace(/\*Added:.*\*/, '').trim(),
+          date: dateMatch[1]
+        });
+      }
+    }
+  }
+
+  return results;
+}
+
 // ============================================================
 // S5: API Changelog (Criterion 4) — delegates to workspace-contracts
 // S5: Cross-Repo Ready Queue (Criterion 5) — workspace state/ready.json
@@ -580,6 +655,402 @@ function getMultiUserSchema() {
 // Exports
 // ============================================================
 
+// ============================================================
+// Workspace Review Mode (Cross-Repo Contract Impact)
+// ============================================================
+
+/**
+ * Analyze code review findings for cross-repo impact.
+ * Called during /wogi-review when workspace is active.
+ *
+ * @param {string} workspaceRoot
+ * @param {Object} manifest
+ * @param {string[]} changedFiles — files changed in the review
+ * @param {string} repoName — repo being reviewed
+ * @returns {Object} review impact analysis
+ */
+function analyzeReviewForCrossRepoImpact(workspaceRoot, manifest, changedFiles, repoName) {
+  const result = {
+    hasContractImpact: false,
+    endpointChanges: [],
+    missingContractUpdates: [],
+    affectedConsumers: [],
+    recommendations: []
+  };
+
+  const member = manifest.members?.[repoName];
+  if (!member) return result;
+
+  // Check if any changed files are in API/route directories
+  const apiPatterns = ['route', 'controller', 'endpoint', 'api', 'handler'];
+  const changedApiFiles = changedFiles.filter(f => {
+    const lower = f.toLowerCase();
+    return apiPatterns.some(p => lower.includes(p));
+  });
+
+  if (changedApiFiles.length > 0) {
+    result.hasContractImpact = true;
+    result.endpointChanges = changedApiFiles;
+
+    // Check if contract was also updated
+    const contractsDir = path.join(workspaceRoot, '.workspace', 'contracts');
+    const contractFiles = changedFiles.filter(f => f.includes('.workspace/contracts'));
+
+    if (contractFiles.length === 0 && changedApiFiles.length > 0) {
+      result.missingContractUpdates.push(
+        `API files changed (${changedApiFiles.join(', ')}) but no contract was updated. ` +
+        `Run \`flow workspace sync\` and regenerate contracts.`
+      );
+    }
+
+    // Find affected consumers
+    const { buildIntegrationMap } = require('./workspace-contracts');
+    const integrationMap = buildIntegrationMap(manifest);
+    const consumerSet = new Set();
+
+    for (const matched of integrationMap.matched || []) {
+      if ((matched.providers || []).includes(repoName)) {
+        for (const consumer of matched.consumers || []) {
+          consumerSet.add(consumer);
+        }
+      }
+    }
+
+    result.affectedConsumers = [...consumerSet];
+
+    if (result.affectedConsumers.length > 0) {
+      result.recommendations.push(
+        `Notify consumers: ${result.affectedConsumers.join(', ')}`,
+        'Update shared contracts if endpoint signatures changed',
+        'Consider creating verification tasks in consumer repos'
+      );
+    }
+  }
+
+  return result;
+}
+
+// ============================================================
+// Workspace Morning Briefing
+// ============================================================
+
+/**
+ * Generate a workspace-aware morning briefing.
+ *
+ * @param {string} workspaceRoot
+ * @param {Object} manifest
+ * @param {string} [repoName] — current repo (for filtering)
+ * @returns {Object} briefing data
+ */
+function generateWorkspaceBriefing(workspaceRoot, manifest, repoName) {
+  const briefing = {
+    unreadMessages: [],
+    contractChanges: [],
+    blockedTasks: [],
+    healthIssues: [],
+    activeLocks: [],
+    recentEvents: [],
+    summary: ''
+  };
+
+  // 1. Unread messages
+  try {
+    const { getUnreadMessages } = require('./workspace-messages');
+    briefing.unreadMessages = getUnreadMessages(workspaceRoot, repoName || 'all');
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 2. Contract changes (drift detection)
+  try {
+    briefing.contractChanges = detectContractDrift(workspaceRoot, manifest);
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 3. Blocked tasks
+  try {
+    const { updateCrossRepoBlocking } = require('./workspace-routing');
+    const blocking = updateCrossRepoBlocking(workspaceRoot, manifest);
+    briefing.blockedTasks = blocking.blockedTasks;
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 4. Health issues
+  try {
+    const health = checkWorkspaceHealth(workspaceRoot, manifest);
+    briefing.healthIssues = health.issues || [];
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 5. Active locks
+  try {
+    const { listActiveLocks } = require('./workspace-locks');
+    briefing.activeLocks = listActiveLocks(workspaceRoot);
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 6. Recent events (last 24h)
+  try {
+    const { readEvents } = require('./workspace-events');
+    const yesterday = new Date(Date.now() - 24 * 60 * 60 * 1000).toISOString();
+    briefing.recentEvents = readEvents(workspaceRoot, { since: yesterday, limit: 20 });
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 7. Summary
+  const parts = [];
+  if (briefing.unreadMessages.length > 0) parts.push(`${briefing.unreadMessages.length} unread message(s)`);
+  if (briefing.contractChanges.length > 0) parts.push(`${briefing.contractChanges.length} contract drift(s)`);
+  if (briefing.blockedTasks.length > 0) parts.push(`${briefing.blockedTasks.length} blocked task(s)`);
+  if (briefing.healthIssues.length > 0) parts.push(`${briefing.healthIssues.length} health issue(s)`);
+  if (briefing.activeLocks.length > 0) parts.push(`${briefing.activeLocks.length} active lock(s)`);
+  briefing.summary = parts.length > 0 ? parts.join(', ') : 'All clear';
+
+  return briefing;
+}
+
+/**
+ * Format workspace briefing as readable text.
+ *
+ * @param {Object} briefing — from generateWorkspaceBriefing()
+ * @returns {string}
+ */
+function formatWorkspaceBriefing(briefing) {
+  const lines = ['Workspace Briefing', '━'.repeat(40), ''];
+
+  if (briefing.unreadMessages.length > 0) {
+    lines.push(`Messages (${briefing.unreadMessages.length} unread):`);
+    for (const msg of briefing.unreadMessages.slice(0, 5)) {
+      lines.push(`  ${msg.from}: ${msg.subject}`);
+    }
+    lines.push('');
+  }
+
+  if (briefing.contractChanges.length > 0) {
+    lines.push(`Contract Drifts (${briefing.contractChanges.length}):`);
+    for (const drift of briefing.contractChanges.slice(0, 5)) {
+      lines.push(`  ${drift.severity}: ${drift.endpoint || drift.type}`);
+    }
+    lines.push('');
+  }
+
+  if (briefing.blockedTasks.length > 0) {
+    lines.push(`Blocked Tasks (${briefing.blockedTasks.length}):`);
+    for (const bt of briefing.blockedTasks.slice(0, 5)) {
+      lines.push(`  ${bt.repo}: ${bt.task.title} [blocked by: ${bt.blockedBy?.join(', ')}]`);
+    }
+    lines.push('');
+  }
+
+  if (briefing.activeLocks.length > 0) {
+    lines.push(`Active Locks (${briefing.activeLocks.length}):`);
+    try {
+      const { formatLocksForDisplay } = require('./workspace-locks');
+      lines.push(formatLocksForDisplay(briefing.activeLocks));
+    } catch (_err) {
+      for (const lock of briefing.activeLocks) {
+        lines.push(`  ${lock.interface} — held by ${lock.owner}`);
+      }
+    }
+    lines.push('');
+  }
+
+  if (briefing.healthIssues.length > 0) {
+    lines.push(`Health Issues (${briefing.healthIssues.length}):`);
+    for (const issue of briefing.healthIssues.slice(0, 5)) {
+      lines.push(`  ${issue.severity || 'warning'}: ${issue.message || issue}`);
+    }
+    lines.push('');
+  }
+
+  lines.push(`Summary: ${briefing.summary}`);
+
+  return lines.join('\n');
+}
+
+// ============================================================
+// Workspace Audit Dimension
+// ============================================================
+
+/**
+ * Run workspace-specific audit checks.
+ * Returns findings for the workspace dimension of /wogi-audit.
+ *
+ * @param {string} workspaceRoot
+ * @param {Object} manifest
+ * @returns {Object} audit results
+ */
+function auditWorkspaceDimension(workspaceRoot, manifest) {
+  const audit = {
+    dimension: 'workspace',
+    score: 100,
+    findings: [],
+    metrics: {}
+  };
+
+  if (!manifest || !manifest.members) {
+    audit.score = 0;
+    audit.findings.push({ severity: 'error', message: 'No workspace manifest found' });
+    return audit;
+  }
+
+  const memberCount = Object.keys(manifest.members).length;
+  audit.metrics.memberCount = memberCount;
+
+  // 1. Type consistency — check for type drift
+  try {
+    const memberMetadata = {};
+    const configPath = path.join(workspaceRoot, 'wogi-workspace.json');
+    const config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+
+    for (const [name, memberConfig] of Object.entries(config.members || {})) {
+      const memberPath = path.resolve(workspaceRoot, memberConfig.path);
+      const workflowPath = path.join(memberPath, '.workflow');
+      if (fs.existsSync(workflowPath)) {
+        const { readMemberMetadata } = require('./workspace');
+        memberMetadata[name] = readMemberMetadata(workflowPath);
+      }
+    }
+
+    const { detectTypeDrift } = require('./workspace-contracts');
+    const drifts = detectTypeDrift(manifest, memberMetadata);
+    audit.metrics.typeDrifts = drifts.length;
+
+    if (drifts.length > 0) {
+      audit.score -= drifts.length * 5;
+      audit.findings.push({
+        severity: 'high',
+        message: `${drifts.length} type drift(s) detected across repos`,
+        details: drifts.map(d => `${d.type}: ${d.entries?.length || 0} conflicting definitions`)
+      });
+    }
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 2. Contract coverage — what % of integrations have contracts
+  try {
+    const { buildIntegrationMap } = require('./workspace-contracts');
+    const map = buildIntegrationMap(manifest);
+    const totalIntegrations = (map.matched || []).length;
+    const contractsDir = path.join(workspaceRoot, '.workspace', 'contracts');
+    const contractCount = fs.existsSync(contractsDir)
+      ? fs.readdirSync(contractsDir).filter(f => !f.startsWith('.')).length
+      : 0;
+
+    audit.metrics.totalIntegrations = totalIntegrations;
+    audit.metrics.contractCount = contractCount;
+    audit.metrics.contractCoverage = totalIntegrations > 0
+      ? Math.round((contractCount / totalIntegrations) * 100)
+      : 100;
+
+    if (audit.metrics.contractCoverage < 50) {
+      audit.score -= 15;
+      audit.findings.push({
+        severity: 'high',
+        message: `Contract coverage is ${audit.metrics.contractCoverage}% (${contractCount}/${totalIntegrations})`
+      });
+    } else if (audit.metrics.contractCoverage < 80) {
+      audit.score -= 5;
+      audit.findings.push({
+        severity: 'medium',
+        message: `Contract coverage is ${audit.metrics.contractCoverage}% — consider adding contracts for uncovered integrations`
+      });
+    }
+
+    // Orphaned endpoints
+    audit.metrics.orphanedConsumers = (map.orphanedConsumers || []).length;
+    audit.metrics.orphanedProviders = (map.orphanedProviders || []).length;
+
+    if (audit.metrics.orphanedConsumers > 0) {
+      audit.score -= audit.metrics.orphanedConsumers * 3;
+      audit.findings.push({
+        severity: 'high',
+        message: `${audit.metrics.orphanedConsumers} consumer(s) calling endpoints with no provider`
+      });
+    }
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 3. Communication health — message acknowledgment rate
+  try {
+    const { readMessages } = require('./workspace-messages');
+    const allMessages = readMessages(workspaceRoot);
+    const actionRequired = allMessages.filter(m => m.actionRequired);
+    const acknowledged = actionRequired.filter(m => m.status !== 'pending');
+
+    audit.metrics.totalMessages = allMessages.length;
+    audit.metrics.pendingActionRequired = actionRequired.filter(m => m.status === 'pending').length;
+    audit.metrics.acknowledgmentRate = actionRequired.length > 0
+      ? Math.round((acknowledged.length / actionRequired.length) * 100)
+      : 100;
+
+    if (audit.metrics.pendingActionRequired > 5) {
+      audit.score -= 10;
+      audit.findings.push({
+        severity: 'medium',
+        message: `${audit.metrics.pendingActionRequired} action-required message(s) still pending`
+      });
+    }
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 4. Dependency freshness — is the manifest up to date?
+  try {
+    const manifestPath = path.join(workspaceRoot, '.workspace', 'state', 'workspace-manifest.json');
+    if (fs.existsSync(manifestPath)) {
+      const stat = fs.statSync(manifestPath);
+      const ageHours = (Date.now() - stat.mtime.getTime()) / (60 * 60 * 1000);
+      audit.metrics.manifestAgeHours = Math.round(ageHours);
+
+      if (ageHours > 48) {
+        audit.score -= 10;
+        audit.findings.push({
+          severity: 'medium',
+          message: `Workspace manifest is ${Math.round(ageHours)}h old — run \`flow workspace sync\``
+        });
+      }
+    }
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // 5. Contract drift
+  try {
+    const contractDrifts = detectContractDrift(workspaceRoot, manifest);
+    audit.metrics.contractDrifts = contractDrifts.length;
+
+    if (contractDrifts.length > 0) {
+      const highSeverity = contractDrifts.filter(d => d.severity === 'high');
+      if (highSeverity.length > 0) {
+        audit.score -= highSeverity.length * 10;
+        audit.findings.push({
+          severity: 'critical',
+          message: `${highSeverity.length} high-severity contract drift(s) — implementation doesn't match spec`
+        });
+      }
+    }
+  } catch (_err) {
+    // Non-critical
+  }
+
+  // Clamp score
+  audit.score = Math.max(0, Math.min(100, audit.score));
+
+  return audit;
+}
+
+// ============================================================
+// Exports
+// ============================================================
+
 module.exports = {
   // S5: Cross-Repo Intelligence
   detectContractDrift,
@@ -587,6 +1058,10 @@ module.exports = {
   getSharedDecisions,
   addSharedDecision,
 
+  // Decision propagation (active broadcast)
+  propagateDecision,
+  getNewDecisionsSince,
+
   // S7: N-Repo Scaling
   buildDependencyGraph,
   getLibraryConsumers,
@@ -596,5 +1071,15 @@ module.exports = {
   // S8: Cloud Preparation
   exportForDashboard,
   getMessageTransport,
-  getMultiUserSchema
+  getMultiUserSchema,
+
+  // Review mode
+  analyzeReviewForCrossRepoImpact,
+
+  // Morning briefing
+  generateWorkspaceBriefing,
+  formatWorkspaceBriefing,
+
+  // Audit dimension
+  auditWorkspaceDimension
 };