ordel-cli 0.4.2__tar.gz → 0.4.3__tar.gz

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ordel-cli
-Version: 0.4.2
+Version: 0.4.3
 Summary: Ordel free CLI — local QA-automation for individual devs. Drives the deterministic ordel-engine and exposes it to a BYO coding agent over MCP. No account, no DB, no Ordel LLM.
 Project-URL: Homepage, https://ordel.io
 Author: Ordel

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/context.py

@@ -97,6 +97,17 @@ def assess_context(store: OrdelStore) -> dict[str, Any]:
             if low.endswith(_SRC_SUFFIXES):
                 source_files += 1
 
-    return _assess(ContextScan(pages=pages, flows=flows, runs=runs, functional_specs=functional,
-                               ordel_specs=ordel, unit_specs=unit, unknown_specs=unknown,
-                               page_objects=page_objects, has_ci=has_ci, source_files=source_files))
+    res = _assess(ContextScan(pages=pages, flows=flows, runs=runs, functional_specs=functional,
+                              ordel_specs=ordel, unit_specs=unit, unknown_specs=unknown,
+                              page_objects=page_objects, has_ci=has_ci, source_files=source_files))
+    # FINDINGS-042 #3: get_app_context said initialized:false while this said "ok, MAP next" in
+    # the same folder, and mapping then failed. Both now say the same thing.
+    res["initialized"] = store.exists
+    if not store.exists:
+        before = str(res.get("required_next_phase", ""))
+        res["required_next_phase"] = ("INIT — this folder has no .ordel/ yet: run `ordel init` "
+                                      "here first (it creates .ordel/ and ORDEL.md); nothing can be "
+                                      "mapped, recorded or run until then")
+        if isinstance(res.get("ascii"), str) and before:
+            res["ascii"] = res["ascii"].replace(before, res["required_next_phase"])
+    return res

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/explore.py

@@ -140,6 +140,16 @@ def persist_capture(store: OrdelStore, doc: dict[str, Any], page_id: str,
     The fingerprint transform is the pure ``fingerprints_from_capture``; this adapter only
     writes the result to the store."""
     fps, url, title = fingerprints_from_capture(doc, fallback_url=fallback_url)
+    # FINDINGS-042 #1: after a failed navigation the browser sits on chrome-error://chromewebdata/;
+    # mapping it under the agent's page_id took that id from the real page, which was then saved
+    # under a site-qualified name, and coverage counted an error page.
+    scheme = url.split(":", 1)[0].lower() if ":" in url else ""
+    if scheme not in ("http", "https", "file"):
+        return {"status": "not_mapped", "page_id": page_id, "url": url, "title": title,
+                "elements": 0,
+                "note": (f"nothing was mapped: the browser is on {url or 'no page'}, not a page of "
+                         "the app (a navigation failed, or nothing is loaded yet); goto a working "
+                         "address and capture again")}
     # A capture replaces its page's elements. When the id already belongs to ANOTHER site (a
     # second app's root is HomePage too), replacing it would silently discard that site's map:
     # save under a site-qualified id and say so. The same site re-captured still refreshes.

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/mcp_server.py

@@ -89,7 +89,7 @@ _FAILURE_STATUSES: frozenset[str] = frozenset({
     "unknown_element", "record_failed", "capture_failed", "http_error", "no_playwright",
     "no_node", "timeout", "no_session", "session_error", "nav_failed", "failed",
     "circuit_open", "error", "exists", "action_failed", "ambiguous", "not_actionable",
-    "invalid_target", "flow_rejected",
+    "invalid_target", "flow_rejected", "not_mapped",
 })
 
 
@@ -911,6 +911,9 @@ def build_server(root: str | Path = ".") -> FastMCP:
         Returns {page_id, url, title, elements}. No LLM."""
         from ordel_cli.explore import page_id_from_url, persist_capture
         def _do() -> dict[str, Any]:
+            if not store.exists:
+                return {"status": "not_initialized",
+                        "note": "this folder has no .ordel/ yet: run `ordel init` here first"}
             doc = _sess().capture()
             if not doc.get("ok", True):
                 return {"status": "session_error", "note": doc.get("error", "capture failed")}
@@ -963,14 +966,21 @@ def build_server(root: str | Path = ".") -> FastMCP:
         if isinstance(results, tuple) and len(results) == 2 and isinstance(results[1], dict):
             content, structured = results
             blocks = list(content)
+            failed = _is_failure(name, structured)
+            if failed and not structured.get("ascii"):
+                # FINDINGS-042 #4: a failure is a milestone the person must see too; the skill's
+                # checkpoints paste `ascii`, which only successes carried.
+                why = str(structured.get("note") or structured.get("error") or "").strip()
+                first = why.splitlines()[0][:300] if why else ""
+                structured = {**structured, "ascii": f"✗ {name}: {structured.get('status') or 'failed'}"
+                              + (f" - {first}" if first else "")}
             summary = structured.get("ascii")
             if isinstance(summary, str) and summary:
                 # The step summary as its own first, plain-text block: a host that shows tool
                 # output shows the map or the verdicts, not an escaped JSON line (the 0.4.0
                 # evaluation's F5: the summary reached the agent but not the person).
                 blocks = [TextContent(type="text", text=summary), *blocks]
-            return CallToolResult(content=blocks, structuredContent=structured,
-                                  isError=_is_failure(name, structured))
+            return CallToolResult(content=blocks, structuredContent=structured, isError=failed)
         return results
 
     mcp._mcp_server.call_tool(validate_input=False)(_wire_call_tool)

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/record.py

@@ -484,6 +484,8 @@ def record_human(store: OrdelStore, name: str, url: str, headed: bool = True,
         # the id the page was actually saved under (another site may already hold the plain one)
         saved = persist_capture(store, {**page, "url": page_url}, page_id_from_url(page_url),
                                 fallback_url=page_url)
+        if saved.get("status") == "not_mapped":
+            continue
         page_ids[page_url] = str(saved["page_id"])
     for s in steps:
         u = strip_query(str(s.get("url", "")))

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/scenarios.py

@@ -81,6 +81,17 @@ def _load_flow_doc(store: OrdelStore, flow_name: str) -> tuple[dict[str, Any] |
                 return None, {"status": "invalid_flow",
                               "note": f"flow '{flow_name}': step {i} field '{key}' must be a "
                                       f"string, number or boolean, got {type(value).__name__}"}
+    # FINDINGS-042 #5: a hand-written or 0.3-era flow with "transitions": 4 crashed with
+    # "'int' object is not iterable" when the scenarios read each step's page.
+    transitions = doc.get("transitions")
+    if transitions is not None and (not isinstance(transitions, list)
+                                    or not all(isinstance(t, dict) for t in transitions)):
+        return None, {"status": "invalid_flow",
+                      "note": f"flow '{flow_name}': 'transitions' must be a list of step records "
+                              f"(got {type(transitions).__name__}); record the flow again"}
+    if not isinstance(doc.get("final_url", ""), str):
+        return None, {"status": "invalid_flow",
+                      "note": f"flow '{flow_name}': 'final_url' must be a string"}
     return doc, None
 
 

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/ordel_cli/session_driver.cjs

@@ -563,6 +563,26 @@ function requirePage(msg) {
   if (needsPage && !page) throw new Error("no open session — call 'open' first");
 }
 
+// FINDINGS-042 #2: a click whose request never answered held the chain for as long as the
+// server took (forever, on a hung login POST), so every later command - current_url included -
+// timed out behind it and the session was wedged. Each page command now answers within this
+// deadline (under the MCP side's 60 s) and frees the chain; open and close keep their own.
+const STEP_DEADLINE_MS = 45000;
+function withDeadline(work, cmd) {
+  work.catch(() => {}); // an abandoned step may still fail later: never an unhandled rejection
+  let timer;
+  const deadline = new Promise((_, reject) => {
+    timer = setTimeout(() => {
+      const e = new Error(`'${cmd}' did not finish within ${STEP_DEADLINE_MS / 1000}s: the page is still `
+        + 'waiting (a request that has not answered, or a page that never settles). The session is '
+        + 'still usable: current_url, goto another address, or browser_close.');
+      e.deadline = true;
+      reject(e);
+    }, STEP_DEADLINE_MS);
+  });
+  return Promise.race([work, deadline]).finally(() => clearTimeout(timer));
+}
+
 // Serialize commands so overlapping stdin lines can't race the single page.
 let chain = Promise.resolve();
 const rl = readline.createInterface({ input: process.stdin });
@@ -574,10 +594,17 @@ rl.on('line', (line) => {
     try { msg = JSON.parse(text); } catch (e) { write({ ok: false, error: 'bad json' }); return; }
     try {
       requirePage(msg);
-      const res = await handle(msg);
+      const bounded = msg.cmd !== 'open' && msg.cmd !== 'close';
+      const res = bounded ? await withDeadline(handle(msg), msg.cmd) : await handle(msg);
       write({ id: msg.id, ...res });
       if (msg.cmd === 'close') await shutdown(0);
     } catch (e) {
+      if (e && e.deadline) { // no element diagnosis: probing the busy page could hang again
+        let url = '';
+        try { url = page ? page.url() : ''; } catch (_) { /* page gone */ }
+        write({ id: msg.id, ok: false, error_kind: 'no_response', url, error: e.message });
+        return;
+      }
       const res = ELEMENT_CMDS.has(msg.cmd) && page ? await diagnose(msg, e)
         : { ok: false, error: String((e && e.message) || e).replace(ANSI, '') };
       write({ id: msg.id, ...res });

{ordel_cli-0.4.2 → ordel_cli-0.4.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "ordel-cli"
-version = "0.4.2"
+version = "0.4.3"
 description = "Ordel free CLI — local QA-automation for individual devs. Drives the deterministic ordel-engine and exposes it to a BYO coding agent over MCP. No account, no DB, no Ordel LLM."
 readme = "README.md"
 requires-python = ">=3.11"