thatcher 1.0.58 → 1.0.59

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thatcher",
-  "version": "1.0.58",
+  "version": "1.0.59",
   "description": "A config-driven application framework for building data-intensive web apps without code.",
   "main": "src/index.js",
   "type": "module",

package/src/lib/config-generator-engine.js

@@ -54,6 +54,16 @@ export class ConfigGeneratorEngine {
     return this.masterConfig?.system?.multi_tenancy?.enabled === true;
   }
 
+  updateWorkflow(workflowName, updatedDef) {
+    if (!workflowName || typeof workflowName !== 'string') {
+      throw new Error('[ConfigGeneratorEngine] updateWorkflow: workflowName required');
+    }
+    const nextWorkflows = { ...(this.masterConfig.workflows || {}), [workflowName]: updatedDef };
+    this.masterConfig = deepFreeze({ ...this.masterConfig, workflows: nextWorkflows });
+    this.specCache.clear();
+    return this;
+  }
+
   registerPlugin(entityName, plugin = {}) {
     if (!entityName || typeof entityName !== 'string') {
       throw new Error('[ConfigGeneratorEngine] registerPlugin: entityName required');

package/src/lib/workflow-engine.js

@@ -30,6 +30,11 @@ const log = createLogger('[WorkflowEngine]');
 const LOCKOUT_SECONDS = 300; // 5 minutes default
 const workflowCache = new Map();
 
+export function clearWorkflowCache(workflowName) {
+  if (workflowName) workflowCache.delete(workflowName);
+  else workflowCache.clear();
+}
+
 function getWorkflowDef(workflowName) {
   if (workflowCache.has(workflowName)) {
     return workflowCache.get(workflowName);

package/src/server/server.js

@@ -114,6 +114,10 @@ export function createServer(options) {
           return await handleListMemberships(req, res, thatcher, configEngine);
         }
 
+        if (req.method === 'POST' && entity === 'workflow' && id && action === 'update') {
+          return await handleUpdateWorkflow(req, res, id, thatcher, configEngine);
+        }
+
         // Check if user has custom route for this
         const userRoutePath = path.join(process.cwd(), 'app/api', ...parts, 'route.js');
         const routeExists = await fileExists(userRoutePath);
@@ -341,6 +345,85 @@ async function handleGenericCrud(req, res, entity, id, action, thatcher, configE
   }
 }
 
+async function handleUpdateWorkflow(req, res, workflowName, thatcher, configEngineArg) {
+  const user = await resolveRequestUser(req);
+  if (!user) {
+    res.writeHead(401, { 'Content-Type': 'application/json' });
+    res.end(JSON.stringify({ error: 'Authentication required' }));
+    return;
+  }
+  const { isPartner } = await import('../ui/permissions-ui.js');
+  if (!isPartner(user)) {
+    res.writeHead(403, { 'Content-Type': 'application/json' });
+    res.end(JSON.stringify({ error: 'Forbidden' }));
+    return;
+  }
+
+  let configEngine = configEngineArg || thatcher?.configEngine || globalThis.__thatcherConfigEngine;
+  if (!configEngine) {
+    const { getConfigEngineSync } = await import('../lib/config-generator-engine.js');
+    configEngine = getConfigEngineSync();
+  }
+  const existingDef = configEngine.getConfig().workflows?.[workflowName];
+  if (!existingDef) {
+    res.writeHead(404);
+    res.end(JSON.stringify({ error: `Workflow "${workflowName}" not found` }));
+    return;
+  }
+
+  let body;
+  try {
+    body = await readBody(req);
+  } catch (e) {
+    res.writeHead(400);
+    res.end(JSON.stringify({ error: e.message }));
+    return;
+  }
+  const stages = Array.isArray(body?.stages) ? body.stages : null;
+  if (!stages || !stages.length) {
+    res.writeHead(400);
+    res.end(JSON.stringify({ error: 'stages array required and must be non-empty' }));
+    return;
+  }
+  const seenNames = new Set();
+  for (const s of stages) {
+    if (!s || typeof s.name !== 'string' || !s.name.trim()) {
+      res.writeHead(400);
+      res.end(JSON.stringify({ error: 'every stage requires a non-empty name' }));
+      return;
+    }
+    if (seenNames.has(s.name)) {
+      res.writeHead(400);
+      res.end(JSON.stringify({ error: `duplicate stage name "${s.name}"` }));
+      return;
+    }
+    seenNames.add(s.name);
+  }
+  const validNames = new Set(stages.map(s => s.name));
+  for (const s of stages) {
+    for (const target of (s.forward || [])) {
+      if (!validNames.has(target)) {
+        res.writeHead(400);
+        res.end(JSON.stringify({ error: `stage "${s.name}" has a forward transition to unknown stage "${target}"` }));
+        return;
+      }
+    }
+  }
+
+  try {
+    const updatedDef = { ...existingDef, stages };
+    configEngine.updateWorkflow(workflowName, updatedDef);
+    const { clearWorkflowCache } = await import('../lib/workflow-engine.js');
+    clearWorkflowCache(workflowName);
+    res.writeHead(200, { 'Content-Type': 'application/json' });
+    res.end(JSON.stringify({ ok: true, workflow: workflowName, stageCount: stages.length }));
+  } catch (err) {
+    apiLog.error(err.message);
+    res.writeHead(500);
+    res.end(JSON.stringify({ error: err.message }));
+  }
+}
+
 async function handleListMemberships(req, res, thatcher, configEngineArg) {
   const user = await resolveRequestUser(req);
   if (!user) {

package/src/ui/page-handler-admin.js

@@ -6,6 +6,7 @@ import { renderSettingsTemplates, renderSettingsChecklists, renderSettingsEntity
 import { renderSettingsNotifications, renderSettingsIntegrations, renderSettingsRecreation, renderSettingsReviewSettings, renderSettingsFileReview, renderSettingsMwrPermissions } from '@/ui/settings/review.js';
 import { renderChecklistsManagement } from '@/ui/checklist-renderer.js';
 import { renderJobManagement } from '@/ui/job-management-renderer.js';
+import { renderWorkflowList, renderWorkflowEditor } from '@/ui/workflow-builder-renderer.js';
 import { isPartner, isManager } from '@/ui/permissions-ui.js';
 import { getSystemConfig, getSettingsCounts, getAuditData, getSystemHealth, renderBuildLogsContent } from '@/ui/page-handler-helpers.js';
 import { fileURLToPath } from 'url';
@@ -127,5 +128,19 @@ export async function handleAdminPage(normalized, segments, user) {
     let logs = []; try { logs = (await list('job_log', {})).slice(0, 20); } catch {}
     return renderJobManagement(user, jobs, logs);
   }
+  if (normalized === '/admin/workflows') {
+    const { getConfigEngineSync } = await import('@/lib/config-generator-engine.js');
+    const engine = getConfigEngineSync();
+    const config = engine.getConfig();
+    return renderWorkflowList(user, Object.keys(config.workflows || {}));
+  }
+  if (segments.length === 3 && segments[1] === 'workflows') {
+    const { getConfigEngineSync } = await import('@/lib/config-generator-engine.js');
+    const engine = getConfigEngineSync();
+    const config = engine.getConfig();
+    const workflowDef = config.workflows?.[segments[2]];
+    if (!workflowDef) return null;
+    return renderWorkflowEditor(user, segments[2], workflowDef);
+  }
   return null;
 }

package/src/ui/workflow-builder-renderer.js

@@ -0,0 +1,109 @@
+import { page } from '@/ui/layout.js';
+import { esc } from '@/ui/render-helpers.js';
+
+export function renderWorkflowList(user, workflowNames) {
+  const rows = workflowNames.map(name =>
+    `<tr data-row data-navigate="/admin/workflows/${esc(name)}" style="cursor:pointer"><td>${esc(name)}</td></tr>`
+  ).join('') || '<tr><td>No workflows configured</td></tr>';
+  const content = `<div class="page-header"><h1 class="page-title">Workflows</h1></div>
+    <div class="table-wrap"><table class="data-table"><thead><tr><th>Name</th></tr></thead><tbody>${rows}</tbody></table></div>`;
+  return page(user, 'Workflows | Thatcher', [{ href: '/admin/settings', label: 'Settings' }, { label: 'Workflows' }], content);
+}
+
+function stageRow(stage, allStageNames) {
+  const forwardOpts = allStageNames.filter(n => n !== stage.name).map(n =>
+    `<label style="display:inline-flex;align-items:center;gap:4px;margin-right:10px"><input type="checkbox" name="forward_${esc(stage.name)}" value="${esc(n)}"${(stage.forward || []).includes(n) ? ' checked' : ''}>${esc(n)}</label>`
+  ).join('');
+  return `<div class="card-clean" style="margin-bottom:12px" data-stage-card="${esc(stage.name)}">
+    <div class="card-clean-body">
+      <div style="display:flex;gap:12px;align-items:center;margin-bottom:8px">
+        <input type="text" name="name_${esc(stage.name)}" value="${esc(stage.name)}" placeholder="stage name" style="width:140px" readonly>
+        <input type="text" name="label_${esc(stage.name)}" value="${esc(stage.label || '')}" placeholder="Label" style="flex:1">
+        <input type="text" name="role_${esc(stage.name)}" value="${esc((stage.requires_role || []).join(','))}" placeholder="requires_role (comma-separated)" style="width:200px">
+        <button type="button" class="btn-ghost-clean" data-action="removeStage" data-args='["${esc(stage.name)}"]'>Remove</button>
+      </div>
+      <div style="font-size:12px;color:var(--color-text-muted,#666)">Forward transitions:</div>
+      <div>${forwardOpts || '<span style="font-size:12px;color:var(--color-text-muted,#666)">(no other stages)</span>'}</div>
+    </div>
+  </div>`;
+}
+
+export function renderWorkflowEditor(user, workflowName, workflowDef) {
+  const stages = workflowDef?.stages || [];
+  const stageNames = stages.map(s => s.name);
+  const stageCards = stages.map(s => stageRow(s, stageNames)).join('');
+
+  const content = `<div class="page-header"><h1 class="page-title">Workflow: ${esc(workflowName)}</h1></div>
+    <form id="workflow-form">
+      <div id="stage-list">${stageCards}</div>
+      <div style="margin:16px 0;display:flex;gap:8px">
+        <input type="text" id="new-stage-name" placeholder="new stage name" style="width:200px">
+        <button type="button" class="btn-ghost-clean" data-action="addStage">Add Stage</button>
+      </div>
+      <button type="button" class="btn-primary-clean" data-action="saveWorkflow" data-args='["${esc(workflowName)}"]'>Save Workflow</button>
+      <span id="save-status" style="margin-left:12px;font-size:13px"></span>
+    </form>`;
+
+  const script = `(function(){
+    function collectStages(){
+      var cards=document.querySelectorAll('[data-stage-card]');
+      var stages=[];
+      cards.forEach(function(card,idx){
+        var origName=card.getAttribute('data-stage-card');
+        var nameInput=card.querySelector('input[name^="name_"]');
+        var labelInput=card.querySelector('input[name^="label_"]');
+        var roleInput=card.querySelector('input[name^="role_"]');
+        var checked=Array.prototype.slice.call(card.querySelectorAll('input[type="checkbox"]:checked')).map(function(cb){return cb.value});
+        var roles=(roleInput.value||'').split(',').map(function(s){return s.trim()}).filter(Boolean);
+        stages.push({name:origName,label:labelInput.value||origName,order:idx,forward:checked,requires_role:roles});
+      });
+      return stages;
+    }
+    window.addStage=function(){
+      var input=document.getElementById('new-stage-name');
+      var name=(input.value||'').trim();
+      if(!name)return;
+      if(document.querySelector('[data-stage-card="'+name+'"]')){alert('Stage already exists');return}
+      var div=document.createElement('div');
+      div.className='card-clean';
+      div.style.marginBottom='12px';
+      div.setAttribute('data-stage-card',name);
+      var body=document.createElement('div');
+      body.className='card-clean-body';
+      var row=document.createElement('div');
+      row.style.cssText='display:flex;gap:12px;align-items:center;margin-bottom:8px';
+      var nameInput=document.createElement('input');
+      nameInput.type='text';nameInput.name='name_'+name;nameInput.value=name;nameInput.readOnly=true;nameInput.style.width='140px';
+      var labelInput=document.createElement('input');
+      labelInput.type='text';labelInput.name='label_'+name;labelInput.placeholder='Label';labelInput.style.flex='1';
+      var roleInput=document.createElement('input');
+      roleInput.type='text';roleInput.name='role_'+name;roleInput.placeholder='requires_role (comma-separated)';roleInput.style.width='200px';
+      var removeBtn=document.createElement('button');
+      removeBtn.type='button';removeBtn.className='btn-ghost-clean';removeBtn.textContent='Remove';
+      removeBtn.addEventListener('click',function(){window.removeStage(name)});
+      row.appendChild(nameInput);row.appendChild(labelInput);row.appendChild(roleInput);row.appendChild(removeBtn);
+      var hint=document.createElement('div');
+      hint.style.cssText='font-size:12px;color:var(--color-text-muted,#666)';
+      hint.textContent='Forward transitions: (save and reload to link)';
+      body.appendChild(row);body.appendChild(hint);
+      div.appendChild(body);
+      document.getElementById('stage-list').appendChild(div);
+      input.value='';
+    };
+    window.removeStage=function(name){
+      var card=document.querySelector('[data-stage-card="'+name+'"]');
+      if(card)card.remove();
+    };
+    window.saveWorkflow=function(workflowName){
+      var stages=collectStages();
+      var status=document.getElementById('save-status');
+      status.textContent='Saving...';
+      fetch('/api/workflow/'+workflowName+'/update',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({stages:stages})})
+        .then(function(r){return r.json().then(function(d){return {ok:r.ok,d:d}})})
+        .then(function(res){if(res.ok){status.textContent='Saved';status.style.color='var(--color-success,#22c55e)';setTimeout(function(){location.reload()},500)}else{status.textContent='Error: '+(res.d.error||'save failed');status.style.color='var(--color-danger,#ef4444)'}})
+        .catch(function(err){status.textContent='Error: '+err.message;status.style.color='var(--color-danger,#ef4444)'});
+    };
+  })();`;
+
+  return page(user, `Workflow: ${workflowName} | Thatcher`, [{ href: '/admin/settings', label: 'Settings' }, { href: '/admin/workflows', label: 'Workflows' }, { label: workflowName }], content, [script]);
+}