@saltcorn/copilot 0.8.1 → 0.8.2

package/app-constructor/tasks.js

@@ -34,6 +34,8 @@ const {
   small,
   a,
   textarea,
+  tr,
+  td,
 } = require("@saltcorn/markup/tags");
 const { getState } = require("@saltcorn/data/db/state");
 const renderLayout = require("@saltcorn/markup/layout");
@@ -47,6 +49,37 @@ const {
   available_plugins_list,
 } = require("./prompts");
 
+const doneTaskRowHtml = (task) =>
+  tr(
+    { "data-row-id": task.id },
+    td(task.body.name || ""),
+    td(task.body.description || ""),
+    td((task.body.depends_on || []).join(", ")),
+    td(task.body.priority || ""),
+    td("Done"),
+    td(
+      task.body.run_id
+        ? a(
+            {
+              target: "_blank",
+              href: `/view/Saltcorn%20Agent%20copilot?run_id=${task.body.run_id}`,
+            },
+            i({ class: "fas fa-external-link-alt" })
+          )
+        : ""
+    ),
+    td(""),
+    td(
+      button(
+        {
+          class: "btn btn-outline-danger btn-sm",
+          onclick: `view_post("${viewname}", "del_task", {id:${task.id}})`,
+        },
+        i({ class: "fas fa-trash-alt" })
+      )
+    )
+  );
+
 const makeTaskList = async (req) => {
   const rs = await MetaData.find(
     {
@@ -60,8 +93,18 @@ const makeTaskList = async (req) => {
     name: "settings",
   });
   const running = !!settings?.body?.running;
+  const stopping = !running && rs.some((t) => t.body.status === "Running");
+  const runningTask = rs.find((t) => t.body.status === "Running");
+  const statusText = runningTask
+    ? span(
+        "Running: ",
+        span({ class: "fw-bold" }, runningTask.body.name || "task")
+      )
+    : running
+    ? "Currently running"
+    : "Currently not running";
   const status = div(
-    running ? "Currently running" : "Currently not running",
+    span({ id: "copilot-status-text" }, statusText),
     running
       ? button(
           {
@@ -73,22 +116,46 @@ const makeTaskList = async (req) => {
         )
       : button(
           {
+            id: "copilot-start-btn",
             class: "btn btn-success ms-2",
-            onclick: `view_post("${viewname}", "start", {})`,
+            ...(stopping
+              ? { disabled: true }
+              : { onclick: `copilotStartRunning(this)` }),
           },
-          i({ class: "fas fa-play me-1" }),
-          "Start running now"
+          stopping
+            ? i({ class: "fas fa-spinner fa-spin me-1" })
+            : i({ class: "fas fa-play me-1" }),
+          stopping ? "Running..." : "Start running now"
         ),
+    stopping &&
+      span(
+        {
+          id: "copilot-stop-notice",
+          class:
+            "alert alert-warning alert-dismissible d-inline-block ms-2 py-1 px-2 mb-0",
+          style: "font-size:0.875rem",
+        },
+        button({
+          type: "button",
+          class: "btn-close btn-sm",
+          "data-bs-dismiss": "alert",
+        }),
+        "The current task will complete, then the queue stops. No new tasks will be started."
+      ),
     button(
       {
+        id: "copilot-run-next-btn",
         class: "btn btn-outline-success ms-2",
-        onclick: `press_store_button(this);view_post("${viewname}", "run_task", {})`,
+        style: running || stopping ? "display:none" : "",
+        onclick: `copilotRunNext(this)`,
       },
       i({ class: "fas fa-play me-1" }),
       "Run next task"
     )
   );
   if (rs.length) {
+    const runningOnLoad =
+      !stopping && rs.some((t) => t.body.status === "Running");
     return div(
       { class: "mt-2" },
       status,
@@ -126,7 +193,8 @@ const makeTaskList = async (req) => {
                 : button(
                     {
                       class: "btn btn-outline-success btn-sm",
-                      onclick: `press_store_button(this);view_post("${viewname}", "run_task", {id:${r.id}})`,
+                      "data-task-run": r.id,
+                      onclick: `copilotRunTask(this,${r.id})`,
                     },
                     i({ class: "fas fa-play" })
                   ),
@@ -159,33 +227,261 @@ const makeTaskList = async (req) => {
         ],
         {
           "To do": rs.filter(
-            (t) => !t.body.status || t.body.status === "To do"
+            (t) =>
+              !t.body.status ||
+              t.body.status === "To do" ||
+              t.body.status === "Running"
           ),
-          Running: rs.filter((t) => t.body.status === "Running"),
           Done: rs.filter((t) => t.body.status === "Done"),
         },
         { grouped: true }
       ),
-      rs.some((t) => t.body.status === "Running")
-        ? script(
-            domReady(`
-(function() {
-  function pollTasks() {
-    var spinners = document.querySelectorAll('.task-spinner[data-task-id]');
-    if (!spinners.length) return;
-    var ids = Array.from(spinners).map(function(el) { return el.getAttribute('data-task-id'); });
-    view_post(${JSON.stringify(
-      viewname
-    )}, 'task_status', { ids: ids }, function(resp) {
-      if (resp && resp.any_done) {
-        location.reload();
+      script(`
+function copilotInitStopping() {
+  const startBtn = document.getElementById('copilot-start-btn');
+  const noticeEl = document.getElementById('copilot-stop-notice');
+  const runNextBtn = document.getElementById('copilot-run-next-btn');
+  const movedToDone = copilotDoneIds();
+  const poll = () => {
+    view_post(${JSON.stringify(viewname)}, 'tasks_poll', {}, (resp) => {
+      if (!resp || !resp.tasks) return;
+      let hasRunning = false;
+      for (const task of resp.tasks) {
+        if (task.status === 'Done' && !movedToDone.has(task.id)) {
+          movedToDone.add(task.id);
+          view_post(${JSON.stringify(
+            viewname
+          )}, 'task_row_done', {id: task.id}, (rowResp) => {
+            copilotAppendDoneRow(task.id, rowResp);
+          });
+        } else if (task.status === 'Running') {
+          hasRunning = true;
+          copilotShowSpinner(task.id);
+        }
+      }
+      if (hasRunning) {
+        setTimeout(poll, 3000);
       } else {
-        setTimeout(pollTasks, 3000);
+        if (noticeEl) noticeEl.remove();
+        const statusTextEl = document.getElementById('copilot-status-text');
+        if (statusTextEl) statusTextEl.textContent = 'Currently not running';
+        if (startBtn) {
+          startBtn.disabled = false;
+          startBtn.innerHTML = '<i class="fas fa-play me-1"></i>Start running now';
+          startBtn.onclick = () => copilotStartRunning(startBtn);
+        }
+        if (runNextBtn) runNextBtn.style.display = '';
       }
     });
+  };
+  setTimeout(poll, 1000);
+}
+
+function copilotAppendDoneRow(taskId, rowResp) {
+  const row = document.querySelector('tr[data-row-id="' + taskId + '"]');
+  if (row) row.remove();
+  const doneHeader = Array.from(document.querySelectorAll('h4.list-group-header'))
+    .find(h => h.textContent.trim() === 'Done');
+  if (doneHeader && rowResp && rowResp.html) {
+    const tbody = doneHeader.closest('tr').parentNode;
+    const tmp = document.createElement('tbody');
+    tmp.innerHTML = rowResp.html;
+    tbody.appendChild(tmp.firstChild);
   }
-  setTimeout(pollTasks, 3000);
-})();
+}
+
+function copilotRunTask(btn, taskId) {
+  btn.innerHTML = '<i class="fas fa-spinner fa-spin"></i>';
+  btn.disabled = true;
+  const runNextBtn = document.getElementById('copilot-run-next-btn');
+  const startBtn = document.getElementById('copilot-start-btn');
+  if (runNextBtn) runNextBtn.disabled = true;
+  if (startBtn) startBtn.disabled = true;
+  const row = document.querySelector('tr[data-row-id="' + taskId + '"]');
+  const taskName = row ? (row.cells[0]?.textContent?.trim() || '') : '';
+  const statusTextEl = document.getElementById('copilot-status-text');
+  if (statusTextEl) statusTextEl.innerHTML =
+    'Running: <span class="fw-bold">' + (taskName || 'task') + '</span>';
+  view_post(${JSON.stringify(viewname)}, 'run_task', {id: taskId}, () => {
+    const poll = () => {
+      view_post(${JSON.stringify(
+        viewname
+      )}, 'task_status', {ids: [String(taskId)]}, (resp) => {
+        if (resp && resp.any_done) {
+          if (statusTextEl) statusTextEl.textContent = 'Currently not running';
+          if (runNextBtn) runNextBtn.disabled = false;
+          if (startBtn) startBtn.disabled = false;
+          view_post(${JSON.stringify(
+            viewname
+          )}, 'task_row_done', {id: taskId}, (rowResp) => {
+            copilotAppendDoneRow(taskId, rowResp);
+          });
+        } else {
+          setTimeout(poll, 3000);
+        }
+      });
+    };
+    setTimeout(poll, 3000);
+  });
+}
+
+function copilotDoneIds() {
+  const ids = new Set();
+  const doneHeader = Array.from(document.querySelectorAll('h4.list-group-header'))
+    .find(h => h.textContent.trim() === 'Done');
+  if (doneHeader) {
+    let sib = doneHeader.closest('tr').nextElementSibling;
+    while (sib) {
+      const id = sib.getAttribute('data-row-id');
+      if (id) ids.add(Number(id));
+      sib = sib.nextElementSibling;
+    }
+  }
+  return ids;
+}
+
+function copilotShowSpinner(taskId) {
+  const row = document.querySelector('tr[data-row-id="' + taskId + '"]');
+  if (row && !row.querySelector('.task-spinner')) {
+    const runBtn = row.querySelector('[data-task-run]');
+    if (runBtn) {
+      runBtn.closest('td').innerHTML =
+        '<span class="task-spinner" data-task-id="' + taskId + '">' +
+        '<i class="fas fa-spinner fa-spin text-warning"></i></span>';
+    }
+  }
+}
+
+function copilotRunNext(btn) {
+  btn.disabled = true;
+  const originalHtml = btn.innerHTML;
+  btn.innerHTML = '<i class="fas fa-spinner fa-spin me-1"></i>Running...';
+  const firstRunBtn = document.querySelector('[data-task-run]');
+  const optimisticId = firstRunBtn ? Number(firstRunBtn.getAttribute('data-task-run')) : null;
+  if (optimisticId) copilotShowSpinner(optimisticId);
+  const movedToDone = copilotDoneIds();
+  view_post(${JSON.stringify(viewname)}, 'run_task', {}, () => {
+    const poll = () => {
+      view_post(${JSON.stringify(viewname)}, 'tasks_poll', {}, (resp) => {
+        if (!resp || !resp.tasks) return;
+        let hasRunning = false;
+        const runningIds = new Set(resp.tasks.filter(t => t.status === 'Running').map(t => t.id));
+        if (optimisticId && !runningIds.has(optimisticId) && !movedToDone.has(optimisticId)) {
+          const staleRow = document.querySelector('tr[data-row-id="' + optimisticId + '"]');
+          const staleSpinner = staleRow?.querySelector('.task-spinner');
+          if (staleSpinner) staleSpinner.closest('td').innerHTML =
+            '<button class="btn btn-outline-success btn-sm" data-task-run="' + optimisticId + '" onclick="copilotRunTask(this,' + optimisticId + ')"><i class="fas fa-play"></i></button>';
+        }
+        for (const task of resp.tasks) {
+          if (task.status === 'Done' && !movedToDone.has(task.id)) {
+            movedToDone.add(task.id);
+            view_post(${JSON.stringify(
+              viewname
+            )}, 'task_row_done', {id: task.id}, (rowResp) => {
+              copilotAppendDoneRow(task.id, rowResp);
+            });
+          } else if (task.status === 'Running') {
+            hasRunning = true;
+            copilotShowSpinner(task.id);
+          }
+        }
+        if (hasRunning) setTimeout(poll, 3000);
+        else {
+          btn.disabled = false;
+          btn.innerHTML = originalHtml;
+        }
+      });
+    };
+    setTimeout(poll, 500);
+  });
+}
+
+function copilotStartRunning(btn) {
+  btn.disabled = true;
+  btn.innerHTML = '<i class="fas fa-spinner fa-spin me-1"></i>Running...';
+  const runNextBtn = document.getElementById('copilot-run-next-btn');
+  const statusTextEl = document.getElementById('copilot-status-text');
+  if (runNextBtn) runNextBtn.style.display = 'none';
+  if (statusTextEl) statusTextEl.innerHTML = '<i class="fas fa-spinner fa-spin me-1"></i>Running';
+  let stopped = false;
+  let stopNotice = null;
+  const stopBtn = document.createElement('button');
+  stopBtn.className = 'btn btn-danger ms-2';
+  stopBtn.innerHTML = '<i class="fas fa-stop me-1"></i>Stop';
+  stopBtn.onclick = () => {
+    stopped = true;
+    stopBtn.disabled = true;
+    view_post(${JSON.stringify(viewname)}, 'stop', {});
+    if (!stopNotice) {
+      stopNotice = document.createElement('span');
+      stopNotice.className = 'alert alert-warning alert-dismissible d-inline-block ms-2 py-1 px-2 mb-0';
+      stopNotice.style.fontSize = '0.875rem';
+      stopNotice.innerHTML =
+        '<button type="button" class="btn-close btn-sm" data-bs-dismiss="alert"></button>' +
+        'The current task will complete, then the queue stops. No new tasks will be started.';
+      stopBtn.insertAdjacentElement('afterend', stopNotice);
+    }
+  };
+  btn.insertAdjacentElement('afterend', stopBtn);
+  view_post(${JSON.stringify(viewname)}, 'start', {}, () => {
+    const movedToDone = copilotDoneIds();
+    const poll = () => {
+      view_post(${JSON.stringify(viewname)}, 'tasks_poll', {}, (resp) => {
+        if (!resp || !resp.tasks) return;
+        let hasPending = false;
+        let hasRunning = false;
+        for (const task of resp.tasks) {
+          if (task.status === 'Done' && !movedToDone.has(task.id)) {
+            movedToDone.add(task.id);
+            view_post(${JSON.stringify(
+              viewname
+            )}, 'task_row_done', {id: task.id}, (rowResp) => {
+              copilotAppendDoneRow(task.id, rowResp);
+            });
+          } else if (task.status === 'Running') {
+            hasRunning = true;
+            copilotShowSpinner(task.id);
+            if (statusTextEl) statusTextEl.innerHTML =
+              'Running: <span class="fw-bold">' + (task.name || 'task') + '</span>';
+          } else if (task.status !== 'Done') {
+            hasPending = true;
+          }
+        }
+        if (hasRunning || (!stopped && hasPending)) {
+          setTimeout(poll, 3000);
+        } else {
+          stopBtn.remove();
+          if (stopNotice) { stopNotice.remove(); stopNotice = null; }
+          btn.disabled = false;
+          btn.innerHTML = '<i class="fas fa-play me-1"></i>Start running now';
+          if (statusTextEl) statusTextEl.textContent = 'Currently not running';
+          if (runNextBtn) runNextBtn.style.display = '';
+        }
+      });
+    };
+    setTimeout(poll, 1000);
+  });
+}
+`),
+      stopping
+        ? script(domReady(`copilotInitStopping();`))
+        : runningOnLoad
+        ? script(
+            domReady(`
+const runNextBtn = document.getElementById('copilot-run-next-btn');
+const startBtn = document.getElementById('copilot-start-btn');
+if (runNextBtn) runNextBtn.disabled = true;
+if (startBtn) startBtn.disabled = true;
+const pollTasks = () => {
+  const spinners = document.querySelectorAll('.task-spinner[data-task-id]');
+  if (!spinners.length) return;
+  const ids = Array.from(spinners).map(el => el.getAttribute('data-task-id'));
+  view_post(${JSON.stringify(viewname)}, 'task_status', {ids}, (resp) => {
+    if (resp && resp.any_done) location.reload();
+    else setTimeout(pollTasks, 3000);
+  });
+};
+setTimeout(pollTasks, 3000);
 `)
           )
         : "",
@@ -211,17 +507,13 @@ const makeTaskList = async (req) => {
         ),
         script(
           domReady(`
-(function() {
-  function poll() {
-    view_post(${JSON.stringify(
-      viewname
-    )}, 'planning_status', {}, function(resp) {
-      if (resp && !resp.planning) location.reload();
-      else setTimeout(poll, 3000);
-    });
-  }
-  setTimeout(poll, 3000);
-})();
+const poll = () => {
+  view_post(${JSON.stringify(viewname)}, 'planning_status', {}, (resp) => {
+    if (resp && !resp.planning) location.reload();
+    else setTimeout(poll, 3000);
+  });
+};
+setTimeout(poll, 3000);
 `)
         )
       );
@@ -328,22 +620,38 @@ Important trigger planning rules:
 Important view planning rules:
 * Each task must create exactly one view. Never put two or more views in the same task. Edit, Show, and List for the same table are always three separate tasks with three separate names, descriptions, and dependencies.
 * Do NOT plan separate tasks for "create" and "edit" on the same table. In Saltcorn, a single Edit view handles both (no id = create, id present = edit). One task, one Edit view, description says "create and edit".
-* Edit, Show, and List views for a table form a natural group and should normally each be planned as their own task. A List without a Show leaves users with no way to inspect details; omit or adjust only when the requirements explicitly say the data is read-only or not editable. When all three are planned, the ordering of tasks must be: Edit and Show first (in either order, they are independent of each other), then List last, because the List depends on both.
-* A List view task must depend on the Edit view task and the Show view task for the same table (if both exist), since its rows link to them. Set depends_on accordingly.
+* Edit, Show, and List views for a table always go together as three separate tasks. Whenever you plan a List view AND a Show view for the same table, you MUST also plan an Edit view for that table — a List without an Edit leaves users unable to create or modify records. Only omit the Edit view when the requirements explicitly say the data is read-only.
+* The three tasks must be ordered: Edit and Show first (independent of each other, in any order), List last. The List task MUST list both the Edit task and the Show task in its depends_on — without exception. If you plan a List that depends on neither, that is a bug in the plan.
+* Before finalising the plan, for every List view task, verify that its depends_on includes the corresponding Edit task and the corresponding Show task (if they exist). If either is missing, add it.
 * When a List view links to a Show view or Edit view, the task description must say: "Add a viewlink column to [view_name] for the current row" — not just "link each row". This wording makes it unambiguous that a viewlink column must be added to the list for each target view.
 * In general, if a view embeds or links to another view, the linked view's task must be listed as a dependency.
 * When a table has foreign key fields referencing the users table, the task description must explicitly state for each one whether it is an ownership field (automatically set from the logged-in user, omit from the form) or a selector field (the user picks a value, include a selector in the form). Example: "user_id records the owner and is set automatically; shared_with_user_id must have a user selector."
 * For FK fields that represent a parent context (e.g. trip_id on packing_items), always include the field as a normal selector in the Edit view form. Do NOT say to omit it. Saltcorn automatically pre-fills the selector from the URL query parameter when the view is opened from a parent context, and the user can select it manually when the view is used standalone.
 * For every task that creates a view, include the exact view name in the task description. View names must be lowercase, snake_case, unique across all tasks in the plan, and descriptive enough to identify the table and purpose — for example 'packing_items_edit' rather than just 'edit'.
-
 Important user account rules:
-* The platform (Saltcorn) provides a built-in user account system with login, registration, and session management. Do NOT plan any tasks for user registration, login pages, password management, or authentication flows — these are already handled by the platform.
+* The platform (Saltcorn) provides a built-in user account system with login, registration, and session management. Do NOT plan any tasks for user registration, login pages, password management, authentication flows, or email verification — these are already handled by the platform. Users register at /auth/signup and log in at /auth/login.
 * User identity is always available as the logged-in user. Ownership fields (FK to users) are set automatically from the session; no custom logic is needed.
 * If a requirement mentions "user accounts", "secure login", "saving data per user", "user-specific data", or "sharing between users", treat it as already satisfied by the platform's built-in user system. Do not generate any task in response to such a requirement.
 
+Important role rules:
+* Every view and page task description MUST state the min_role explicitly, e.g. "Set min_role to admin (1)." or "Set min_role to user (80).". Never omit it.
+* Role values: admin=1, staff=40, user=80, public=100. Use the value that matches who will use the view or page — admin for management, staff for staff-only, user for logged-in users (clients, members, etc.), public only when the view or page must be accessible without login.
+
+Important home page rules:
+* Every role should land on the right page after visiting /. Plan a single task "Set home pages by role" that depends on all relevant page tasks and configures home_page_by_role for every role in one step.
+* Role IDs: public=100, user=80, staff=40, admin=1.
+* Landing/marketing page (public-facing intro): min_role must be 100 (public). It MUST include visible links to /auth/login (Log in) and /auth/signup (Create an account). Set as home for role 100 (public).
+* If there is an admin dashboard page, set it as home for role 1 (admin).
+* If there is a dashboard or main page for regular users or staff, set it as home for role 80 (user) and/or role 40 (staff) as appropriate.
+* The "Set home pages by role" task description must list every role→page mapping explicitly using the exact page names planned in this task list, e.g.: "Set home_page_by_role: public (100) → landing, user (80) → client_dashboard, staff (40) → staff_dashboard, admin (1) → app_admin_dashboard." Never use "admin_dashboard" as a page name — it is reserved by the platform.
+
 Important plugin rules:
 * If multiple plugins need to be installed, combine them ALL into a single task named "Install plugins" that lists every required plugin name. Do NOT create a separate task per plugin.
 
+Important dependency rules:
+* Every name in a task's depends_on MUST exactly match the name field of another task in the same plan_tasks call. Never reference a name that is not present in the tasks array — not a concept, not a table name, not a made-up label. If you find yourself writing a depends_on entry whose name does not appear as a task name in the list, either add the missing task or remove the dependency.
+* Before calling plan_tasks, mentally verify: for every task, every name in its depends_on array appears as the name of another task in the array.
+
 Important schema/table rules:
 * The database schema is already fully designed and implemented before task planning begins. ALL tables and fields needed by the application already exist. Do NOT plan any tasks that create tables, add fields, modify fields, or change the schema in any way. If you find yourself writing a task whose output is a table or a field, delete it — that work is already done.
 * Ownership behaviour (auto-setting a FK-to-users field from the logged-in user) is configured in the Edit view, not in the database. Do not create tasks for it at the schema level.
@@ -379,13 +687,27 @@ Before finalising the plan, you may call get_view_config for any existing view y
 
       const planCall = toolCalls.find((tc) => tc.tool_name === "plan_tasks");
       if (planCall) {
-        for (const task of planCall.input.tasks)
+        const tasks = planCall.input.tasks;
+        const plannedNames = new Set(tasks.map((t) => t.name).filter(Boolean));
+
+        for (const task of tasks) {
+          const validDeps = (task.depends_on || []).filter((nm) => {
+            if (!plannedNames.has(nm)) {
+              getState().log(
+                2,
+                `AppConstructor: dropping phantom dependency "${nm}" from task "${task.name}" — no such task in plan`
+              );
+              return false;
+            }
+            return true;
+          });
           await MetaData.create({
             type: "CopilotConstructMgr",
             name: "task",
-            body: task,
+            body: { ...task, depends_on: validDeps },
             user_id: userId,
           });
+        }
         break;
       }
 
@@ -477,12 +799,14 @@ const del_task = async (table_id, viewname, config, body, { req, res }) => {
 };
 const run_task = async (table_id, viewname, config, body, { req, res }) => {
   const reqUser = req?.user;
-  if (body.id)
+  if (body.id) {
     runTask(body.id, { user: reqUser, __: req.__ }).catch((e) =>
       console.error("run_task error", e)
     );
-  else runNextTask(true).catch((e) => console.error("run_task error", e));
-  return { json: { reload_page: true } };
+    return { json: { success: true } };
+  }
+  runNextTask(true).catch((e) => console.error("run_task error", e));
+  return { json: { success: true } };
 };
 
 const planning_status = async (
@@ -499,6 +823,35 @@ const planning_status = async (
   return { json: { planning: !!planning } };
 };
 
+const tasks_poll = async (table_id, viewname, config, body, { req, res }) => {
+  const tasks = await MetaData.find({
+    type: "CopilotConstructMgr",
+    name: "task",
+  });
+  return {
+    json: {
+      tasks: tasks.map((t) => ({
+        id: t.id,
+        name: t.body.name,
+        status: t.body.status || "To do",
+        run_id: t.body.run_id,
+      })),
+    },
+  };
+};
+
+const task_row_done = async (
+  table_id,
+  viewname,
+  config,
+  body,
+  { req, res }
+) => {
+  const task = await MetaData.findOne({ id: body.id });
+  if (!task) return { json: { html: "" } };
+  return { json: { html: doneTaskRowHtml(task) } };
+};
+
 const task_status = async (table_id, viewname, config, body, { req, res }) => {
   const ids = body.ids || [];
   const tasks = await MetaData.find({
@@ -524,7 +877,7 @@ const start = async (table_id, viewname, config, body, { req, res }) => {
       body: { running: true },
     });
   runNextTask().catch((e) => console.error("start error", e));
-  return { json: { reload_page: true } };
+  return { json: { success: true } };
 };
 const stop = async (table_id, viewname, config, body, { req, res }) => {
   const settings = await MetaData.findOne({
@@ -539,7 +892,7 @@ const stop = async (table_id, viewname, config, body, { req, res }) => {
       name: "settings",
       body: { running: false },
     });
-  return { json: { reload_page: true } };
+  return { json: { success: true } };
 };
 
 const mark_done_task = async (
@@ -578,6 +931,8 @@ const task_routes = {
   run_task,
   planning_status,
   task_status,
+  tasks_poll,
+  task_row_done,
   start,
   stop,
 };

package/app-constructor/tools.js

@@ -46,12 +46,13 @@ const task_tool = {
           type: "array",
           items: {
             type: "object",
-            required: ["requirement", "priority"],
+            required: ["name", "description", "priority", "depends_on"],
             additionalProperties: false,
             properties: {
               name: {
                 type: "string",
-                description: "A short name for the task",
+                description:
+                  "A short unique name for the task (snake_case). Every other task that depends on this task must use exactly this name in their depends_on array.",
               },
               description: {
                 type: "string",
@@ -65,7 +66,7 @@ const task_tool = {
               depends_on: {
                 type: "array",
                 description:
-                  "The names of the tasks that must be completed before this tasks can be started",
+                  "Names of tasks in THIS plan that must complete before this task starts. Every name listed here MUST exactly match the name of another task in this same plan_tasks call. Never reference a task name that is not present in the tasks array.",
                 items: {
                   type: "string",
                 },
@@ -78,4 +79,4 @@ const task_tool = {
   },
 };
 
-module.exports = { requirements_tool,task_tool };
+module.exports = { requirements_tool, task_tool };

package/app-constructor/view.js

@@ -159,6 +159,13 @@ const submit_specs = async (table_id, viewname, config, body, { req, res }) => {
       user_id: req.user?.id || undefined,
       body: spec,
     });
+  return {
+    json: {
+      success: "ok",
+      notify: "Specification saved",
+      notify_type: "success",
+    },
+  };
 };
 
 const virtual_triggers = () => {