Address code review feedback: improve path escaping, date parsing, and add inline comments for unicode escapes

Agent-Logs-Url: https://github.com/githubnext/autoloop/sessions/639174de-6b45-471f-878b-05ffaca3cdfa

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>

Author: Copilot

tests/conftest.py

@@ -56,9 +56,9 @@ def _load_workflow_functions():
 def _call_js(func_name, *args):
     """Call a JS function from the extracted workflow module and return the result."""
     args_json = json.dumps(list(args))
-    escaped_path = _JS_MODULE_PATH.replace("\\", "\\\\")
+    escaped_path = json.dumps(_JS_MODULE_PATH)
     script = (
-        "const m = require('" + escaped_path + "');\n"
+        "const m = require(" + escaped_path + ");\n"
         "const result = m." + func_name + "(..." + args_json + ");\n"
         "process.stdout.write(JSON.stringify(result === undefined ? null : result));\n"
     )

workflows/autoloop.md

@@ -117,7 +117,7 @@ steps:
               const rawVal = row[2].trim();
               if (['field', '---', ':---', ':---:', '---:'].includes(rawKey.toLowerCase())) continue;
               const key = rawKey.toLowerCase().replace(/ /g, '_');
-              const val = ['\u2014', '-', ''].includes(rawVal) ? null : rawVal;
+              const val = ['\u2014', '-', ''].includes(rawVal) ? null : rawVal; // \u2014 = em dash
               state[key] = val;
           }
           // Coerce types
@@ -391,7 +391,7 @@ steps:
               const lr = state.last_run || null;
               if (lr) {
                   try {
-                      const d = new Date(lr.endsWith('Z') ? lr : lr.replace('Z', '+00:00'));
+                      const d = new Date(lr);
                       if (!isNaN(d.getTime())) lastRun = d;
                   } catch (e) {
                       // ignore invalid date

workflows/sync-branches.md

@@ -99,11 +99,11 @@ steps:
       git('checkout', defaultBranch);
 
       if (failed.length > 0) {
-          console.log('\n\u26a0\ufe0f Failed to sync ' + failed.length + ' branch(es): ' + JSON.stringify(failed));
+          console.log('\n\u26a0\ufe0f Failed to sync ' + failed.length + " branch(es): " + JSON.stringify(failed)); // \u26a0\ufe0f = warning sign
           console.log('These branches may need manual conflict resolution.');
           // Don't fail the workflow -- log the issue but continue
       } else {
-          console.log('\n\u2705 All ' + branches.length + ' branch(es) synced successfully.');
+          console.log('\n\u2705 All ' + branches.length + " branch(es) synced successfully."); // \u2705 = checkmark
       }
       JSEOF
 ---