@saltcorn/server 1.6.0-beta.1 → 1.6.0-beta.11

package/routes/actions.js

@@ -75,6 +75,8 @@ const {
   h4,
   p,
   style,
+  h5,
+  textarea,
 } = require("@saltcorn/markup/tags");
 const Table = require("@saltcorn/data/models/table");
 const { getActionConfigFields } = require("@saltcorn/data/plugin-helper");
@@ -265,10 +267,14 @@ const triggerForm = async (req, trigger) => {
   const hasChannel = Object.entries(getState().eventTypes)
     .filter(([k, v]) => v.hasChannel)
     .map(([k, v]) => k);
-
+  const actionExplainers = Trigger.action_explainers();
+  Trigger.find({}).forEach((tr) => {
+    if (tr.description && tr.name) actionExplainers[tr.name] = tr.description;
+  });
   const allActions = Trigger.action_options({
     notRequireRow: false,
     workflow: true,
+    allTriggers: true,
   });
   const table_triggers = ["Insert", "Update", "Delete", "Validate"];
   const additional_triggers_with_onlyif = ["Login", "PageLoad"];
@@ -276,6 +282,7 @@ const triggerForm = async (req, trigger) => {
   const actionsNotRequiringRow = Trigger.action_options({
     notRequireRow: true,
     workflow: true,
+    allTriggers: true,
   });
 
   Trigger.when_options.forEach((t) => {
@@ -376,6 +383,7 @@ const triggerForm = async (req, trigger) => {
         attributes: {
           options: actionsNotRequiringRow,
           onChange: "$('select[name=action]').val(event.target.value)",
+          explainers: actionExplainers,
         },
         showIf: {
           when_trigger: "Never",
@@ -392,6 +400,7 @@ const triggerForm = async (req, trigger) => {
         attributes: {
           options: allActions,
           onChange: "$('select[name=action]').val(event.target.value)",
+          explainers: actionExplainers,
         },
         showIf: {
           when_trigger: "Never",
@@ -439,9 +448,14 @@ const triggerForm = async (req, trigger) => {
         label: req.__("Only if"),
         type: "String",
         class: "validate-expression",
-        sublabel: req.__(
-          "Optional JavaScript expression to determine if the trigger should run."
-        ),
+        sublabel:
+          req.__(
+            "Optional JavaScript expression to determine if the trigger should run."
+          ) +
+          " " +
+          req.__(
+            "On updates, use <code>row.old_row</code> to access values before the update."
+          ),
         parent_field: "configuration",
         showIf: {
           when_trigger: [...table_triggers, ...additional_triggers_with_onlyif],
@@ -1194,6 +1208,12 @@ router.get(
       ? Table.findOne({ id: trigger.table_id })
       : null;
 
+    const edit_redirect =
+      req.query.on_done_redirect &&
+      is_relative_url("/" + req.query.on_done_redirect)
+        ? `?on_done_redirect=${req.query.on_done_redirect}`
+        : "";
+
     const subtitle =
       span(
         { class: "ms-2" },
@@ -1203,7 +1223,7 @@ router.get(
         table ? ` on ` + a({ href: `/table/${table.name}` }, table.name) : ""
       ) +
       a(
-        { href: `/actions/edit/${id}`, class: "ms-2" },
+        { href: `/actions/edit/${id}${edit_redirect}`, class: "ms-2" },
         req.__("Edit"),
         '&nbsp;<i class="fas fa-edit"></i>'
       ) +
@@ -1216,6 +1236,52 @@ router.get(
 
         req.__("Test run") + "&nbsp;&raquo;"
       );
+    // Check if trigger.action is a reference to another trigger by name
+    const refTrigger =
+      !action &&
+      trigger.action !== "Workflow" &&
+      trigger.action !== "Multi-step action"
+        ? Trigger.findOne({ name: trigger.action })
+        : null;
+
+    if (refTrigger) {
+      send_events_page({
+        res,
+        req,
+        active_sub: "Triggers",
+        sub2_page: "Configure",
+        page_title: req.__(`%s configuration`, trigger.name),
+        contents: {
+          type: "card",
+          title: req.__(
+            "Configure trigger %s",
+            span({ class: "copy-to-clipboard" }, trigger.name)
+          ),
+          subtitle,
+          contents:
+            p(
+              req.__(
+                "This trigger delegates to %s.",
+                a(
+                  { href: `/actions/configure/${refTrigger.id}` },
+                  refTrigger.name
+                )
+              )
+            ) +
+            p(
+              a(
+                {
+                  href: `/actions/configure/${refTrigger.id}`,
+                  class: "btn btn-primary",
+                },
+                req.__("Configure %s &raquo;", refTrigger.name)
+              )
+            ),
+        },
+      });
+      return;
+    }
+
     if (trigger.action === "Workflow") {
       const wfCfg = await getWorkflowConfig(req, id, table, trigger);
       send_events_page({
@@ -1352,6 +1418,146 @@ router.get(
       });
       // populate form values
       form.values = trigger.configuration;
+
+      const hasCopilot =
+        trigger.action === "run_js_code" &&
+        !!getState().functions.copilot_generate_javascript;
+
+      if (hasCopilot) {
+        form.additionalButtons = [
+          {
+            label: req.__("Edit with AI"),
+            onclick: "showJsCopilotModal(this)",
+            class: "btn btn-secondary",
+          },
+        ];
+      }
+
+      const copilotModal = hasCopilot
+        ? div(
+            {
+              class: "modal fade",
+              id: "jsCopilotModal",
+              tabindex: "-1",
+            },
+            div(
+              { class: "modal-dialog" },
+              div(
+                { class: "modal-content" },
+                div(
+                  { class: "modal-header" },
+                  h5({ class: "modal-title" }, req.__("Edit with AI")),
+                  button({
+                    type: "button",
+                    class: "btn-close",
+                    "data-bs-dismiss": "modal",
+                  })
+                ),
+                div(
+                  { class: "modal-body" },
+                  p(
+                    { class: "text-muted small" },
+                    req.__(
+                      "Describe what the code should do. The existing code will be used as context."
+                    )
+                  ),
+                  textarea({
+                    id: "jsCopilotPrompt",
+                    class: "form-control",
+                    rows: 3,
+                    placeholder: req.__("Enter your prompt..."),
+                  }),
+                  div({
+                    id: "jsCopilotError",
+                    class: "alert alert-danger mt-2 mb-0 d-none",
+                  })
+                ),
+                div(
+                  { class: "modal-footer" },
+                  button(
+                    { class: "btn btn-secondary", "data-bs-dismiss": "modal" },
+                    req.__("Cancel")
+                  ),
+                  button(
+                    {
+                      id: "jsCopilotGenBtn",
+                      class: "btn btn-primary",
+                      onclick: "runJsCopilot()",
+                      disabled: true,
+                    },
+                    req.__("Generate")
+                  )
+                )
+              )
+            )
+          ) +
+          script(`
+function showJsCopilotModal() {
+  var modal = new bootstrap.Modal(document.getElementById("jsCopilotModal"));
+  modal.show();
+}
+document.addEventListener("DOMContentLoaded", function () {
+  var ta = document.getElementById("jsCopilotPrompt");
+  var btn = document.getElementById("jsCopilotGenBtn");
+  if (ta && btn) {
+    ta.addEventListener("input", function () {
+      btn.disabled = !ta.value.trim();
+    });
+  }
+});
+function runJsCopilot() {
+  var ta = document.getElementById("jsCopilotPrompt");
+  var prompt = ta.value.trim();
+  if (!prompt) return;
+  var btn = document.getElementById("jsCopilotGenBtn");
+  var errDiv = document.getElementById("jsCopilotError");
+  btn.disabled = true;
+  btn.textContent = ${JSON.stringify(req.__("Generating..."))};
+  errDiv.classList.add("d-none");
+  fetch("/actions/gen-js-copilot/${id}", {
+    method: "POST",
+    headers: {
+      "Content-Type": "application/json",
+      "X-Requested-With": "XMLHttpRequest",
+      "CSRF-Token": _sc_globalCsrf,
+    },
+    body: JSON.stringify({ description: prompt }),
+  })
+    .then(function (r) {
+      return r.json();
+    })
+    .then(function (data) {
+      btn.textContent = ${JSON.stringify(req.__("Generate"))};
+      if (data.error) {
+        btn.disabled = !ta.value.trim();
+        errDiv.textContent = data.error;
+        errDiv.classList.remove("d-none");
+        return;
+      }
+      ta.value = "";
+      btn.disabled = true;
+      var editorDiv = document.querySelector('textarea[name="code"]');
+      if (editorDiv) {
+        var monacoDiv = editorDiv.nextElementSibling;
+        if (monacoDiv) {
+          var ed = $(monacoDiv).data("monaco-editor");
+          if (ed) ed.setValue(data.code);
+        }
+      }
+      bootstrap.Modal.getInstance(
+        document.getElementById("jsCopilotModal")
+      ).hide();
+    })
+    .catch(function (err) {
+      btn.disabled = !ta.value.trim();
+      btn.textContent = ${JSON.stringify(req.__("Generate"))};
+      errDiv.textContent = err.message || "Request failed";
+      errDiv.classList.remove("d-none");
+    });
+}
+          `)
+        : "";
+
       // send events page
       send_events_page({
         res,
@@ -1367,7 +1573,7 @@ router.get(
             span({ class: "copy-to-clipboard" }, trigger.name)
           ),
           subtitle,
-          contents: renderForm(form, req.csrfToken()),
+          contents: copilotModal + renderForm(form, req.csrfToken()),
         },
       });
     }
@@ -1935,6 +2141,45 @@ router.post(
   })
 );
 
+router.post(
+  "/gen-js-copilot/:id",
+  isAdminOrHasConfigMinRole("min_role_edit_triggers"),
+  error_catcher(async (req, res) => {
+    const { id } = req.params;
+    const trigger = await Trigger.findOne({ id });
+    if (!trigger) {
+      req.flash("warning", req.__("Action not found"));
+      res.redirect(`/actions/`);
+      return;
+    }
+    const description = (req.body || {}).description;
+    // await Trigger.update(trigger.id, { description });
+    const table = trigger.table_id
+      ? Table.findOne({ id: trigger.table_id })
+      : null;
+    const existing_code = trigger.configuration?.code || "";
+    const generated =
+      await getState().functions.copilot_generate_javascript.run(
+        description,
+        existing_code,
+        table?.name || null
+      );
+    await Trigger.update(trigger.id, {
+      configuration: { ...trigger.configuration, code: generated },
+    });
+    await getState().refresh_triggers();
+    Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+      entity_type: "Trigger",
+      entity_name: trigger.name,
+    });
+    if (req.xhr) {
+      res.json({ code: generated });
+    } else {
+      res.redirect(`/actions/configure/${trigger.id}`);
+    }
+  })
+);
+
 router.post(
   "/gen-copilot/:trigger_id",
   isAdminOrHasConfigMinRole("min_role_edit_triggers"),
@@ -1980,8 +2225,7 @@ router.get(
   error_catcher(async (req, res) => {
     const trNames = {};
     const { _page, trigger } = req.query;
-    for (const trig of await Trigger.find({ action: "Workflow" }))
-      trNames[trig.id] = trig.name;
+    for (const trig of await Trigger.find({})) trNames[trig.id] = trig.name;
     const q = {};
     const selOpts = { orderBy: "started_at", orderDesc: true, limit: 20 };
     if (_page) selOpts.offset = 20 * (parseInt(_page) - 1);
@@ -2207,9 +2451,9 @@ router.get(
   error_catcher(async (req, res) => {
     const { id } = req.params;
 
-    const run = await WorkflowRun.findOne({ id });
+    const run = await WorkflowRun.findOne({ id: +id });
 
-    if (!run.user_allowed_to_fill_form(req.user)) {
+    if (!run || !run.user_allowed_to_fill_form(req.user)) {
       if (req.xhr) res.json({ error: "Not authorized" });
       else {
         req.flash("danger", req.__("Not authorized"));
@@ -2249,22 +2493,31 @@ const workflowRunPromiseHandler = (promise, run, req) => {
         ...retDirs,
         page_load_tag: req.headers["page-load-tag"],
       };
-      getState().emitDynamicUpdate(db.getTenantSchema(), emitData);
+      const userIds = req.user ? [req.user.id] : null;
+      getState().emitDynamicUpdate(db.getTenantSchema(), emitData, userIds);
       if (
         !emitData.resume_workflow &&
         !emitData.popup?.startsWith?.("/actions/fill-workflow-form/")
       )
-        getState().emitDynamicUpdate(db.getTenantSchema(), {
-          eval_js: "reset_spinners()",
-          page_load_tag: req.headers["page-load-tag"],
-        });
+        getState().emitDynamicUpdate(
+          db.getTenantSchema(),
+          {
+            eval_js: "reset_spinners()",
+            page_load_tag: req.headers["page-load-tag"],
+          },
+          userIds
+        );
     })
     .catch((e) => {
       console.error(e);
-      getState().emitDynamicUpdate(db.getTenantSchema(), {
-        error: e.message,
-        page_load_tag: req.headers["page-load-tag"],
-      });
+      getState().emitDynamicUpdate(
+        db.getTenantSchema(),
+        {
+          error: e.message,
+          page_load_tag: req.headers["page-load-tag"],
+        },
+        req.user ? [req.user.id] : null
+      );
     });
 };
 
@@ -2274,7 +2527,7 @@ router.post(
     const { id } = req.params;
 
     const run = await WorkflowRun.findOne({ id });
-    if (!run.user_allowed_to_fill_form(req.user)) {
+    if (!run || !run.user_allowed_to_fill_form(req.user)) {
       if (req.xhr) res.json({ error: "Not authorized" });
       else {
         req.flash("danger", req.__("Not authorized"));
@@ -2332,9 +2585,9 @@ router.post(
   error_catcher(async (req, res) => {
     const { id } = req.params;
 
-    const run = await WorkflowRun.findOne({ id });
+    const run = await WorkflowRun.findOne({ id: +id });
     //TODO session if not logged in
-    if (run.started_by !== req.user?.id) {
+    if (!run || run.started_by !== req.user?.id) {
       if (req.xhr) res.json({ error: "Not authorized" });
       else {
         req.flash("danger", req.__("Not authorized"));