@inetafrica/open-claudia 3.0.33 → 3.0.35

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## v3.0.35 — Spaces comments are readable rich text, not a run-on wall
+
+- **The bot's Spaces replies rendered as unreadable walls.** `buildCommentContent` posted the entire reply as one text node inside one paragraph. Spaces comments are TipTap/ProseMirror docs: inside a single text node newlines don't render, so paragraphs, numbered lists and `---` separators collapsed into one run-on blob — and the model's markdown (`**bold**`, `*italic*`, backticks) printed its asterisks literally.
+- **Outgoing comments now convert markdown → a real ProseMirror doc.** Blank lines become paragraphs, single newlines hardBreaks; `**`/`*`/`` ` `` become bold/italic/code marks (word-bounded, so snake_case identifiers stay plain); `-`/`1.` runs become real bullet/ordered lists; `#`–`###` become headings; ``` fences become code blocks; `---` becomes a divider. Every emitted node type was verified against the Spaces frontend editor schema (StarterKit + HorizontalRule + CodeBlockLowlight), so nothing renders broken.
+- **Mention chips unchanged.** The proven tag-chip shape still leads the first paragraph, so notification fan-out and the "@Name text…" lead-in behave exactly as before. `flattenContent` also learned `codeBlock`/`horizontalRule` so the bot reads its own comments back cleanly.
+- **Agent-space / OpenClaw:** no new deps, no new env, no schema change. Covered by new `test-spaces-comment-content.js` in the pretest chain. Bots pick this up on their next upgrade, together with v3.0.34's /upgrade redelivery guard.
+
+## v3.0.34 — /upgrade no longer triggers a rollout storm
+
+- **One `/upgrade` used to mean up to five rollouts.** On AgentSpace pods the control plane answers `/upgrade` by rolling the deployment, which SIGTERMs the bot mid-turn. Telegram long-polling keeps no persisted offset, so the still-unconfirmed `/upgrade` update was redelivered to the freshly booted pod — which triggered *another* rollout, and so on. Observed on rahilcoder 2026-07-17: deployment revisions 47–51 (five rollouts in three minutes) from a single command; every earlier `/upgrade` had silently double-fired the same way (revs 42/43, 45/46).
+- **Fix: an upgrade-in-flight marker (`upgrade-in-flight.json` in the config dir, 10-minute freshness) guards the handler.** A redelivered or impatiently re-run `/upgrade` inside the window is acknowledged but not re-triggered: if the pod has already restarted since the request it replies "that upgrade already went through", otherwise "already in flight". The marker is cleared on a definitive trigger failure so an immediate retry stays possible, and simply ages out otherwise.
+- **The ack now goes out BEFORE the rollout is requested.** Previously the "Upgrade requested" confirmation was sent after the control-plane call, so the SIGTERM routinely killed it — Rahil saw reboot greetings *before* any acknowledgement. Ack-first means the user always sees the confirmation, then the reboot.
+- **Agent-space / OpenClaw:** no new deps, no new env, no schema change; Docker image builds FROM the existing `:base`. Existing pods pick this up on their next upgrade — that upgrade itself will still multi-fire (old code is doing the triggering), all later ones are single-shot.
+
 ## v3.0.33 — Full sudo inside the pod (re-cut of v3.0.32; the pin guard bit us again)
 
 - **`claudia` now has `NOPASSWD:ALL` sudo in the pod image.** The rebuilt `:base` widens the baked sudoers from the old command allow-list to full passwordless sudo, so the agent can install packages and operate inside its own container like a normal machine. This pairs with agent-space's IaaS-style security-policy system: the blast radius is fenced at the pod's NetworkPolicy (egress/ingress presets, DB-rendered), not by crippling the user inside the container.

package/channels/spaces/client.js

@@ -89,8 +89,9 @@ function flattenContent(node) {
   if (t === "text") return node.text || "";
   if (t === "mention") return `@${node.attrs?.label || node.attrs?.id || ""}`;
   if (t === "hardBreak") return "\n";
+  if (t === "horizontalRule") return "---\n";
   const inner = (node.content || []).map(flattenContent).join("");
-  if (["paragraph", "heading", "listItem", "blockquote"].includes(t)) return inner + "\n";
+  if (["paragraph", "heading", "listItem", "blockquote", "codeBlock"].includes(t)) return inner + "\n";
   return inner;
 }
 
@@ -98,17 +99,126 @@ function contentText(raw) {
   return flattenContent(raw).trim();
 }
 
+// Inline markdown → ProseMirror inline nodes. Supports `code`, **bold** and
+// *italic*/_italic_ (word-bounded, so snake_case identifiers and stray
+// asterisks in prose survive untouched). One level only — chat comments
+// don't need nesting, and a wrong guess renders worse than plain text.
+const INLINE_TOKEN_RE = /(`[^`\n]+`)|(\*\*(?=\S)[^*\n]*?\S\*\*)|(\*(?=\S)[^*\n]*?\S\*)|((?<![\w`])_(?=\S)[^_\n]*?\S_(?![\w`]))/g;
+function inlineNodes(text) {
+  const out = [];
+  const push = (t, marks) => { if (t) out.push(marks ? { type: "text", text: t, marks } : { type: "text", text: t }); };
+  let last = 0;
+  let m;
+  INLINE_TOKEN_RE.lastIndex = 0;
+  while ((m = INLINE_TOKEN_RE.exec(text))) {
+    push(text.slice(last, m.index));
+    if (m[1]) push(m[1].slice(1, -1), [{ type: "code" }]);
+    else if (m[2]) push(m[2].slice(2, -2), [{ type: "bold" }]);
+    else if (m[3]) push(m[3].slice(1, -1), [{ type: "italic" }]);
+    else push(m[4].slice(1, -1), [{ type: "italic" }]);
+    last = m.index + m[0].length;
+  }
+  push(text.slice(last));
+  return out;
+}
+
+// Block-level markdown → ProseMirror blocks. The model writes markdown; Spaces
+// renders TipTap docs — posting the raw text as one text node collapses every
+// newline and prints the asterisks literally (the unreadable-wall bug). Only
+// node types present in the Spaces editor schema (StarterKit + HorizontalRule
+// + CodeBlockLowlight) are emitted, so nothing renders broken.
+function markdownBlocks(text) {
+  const blocks = [];
+  const lines = String(text || "").replace(/\r\n/g, "\n").split("\n");
+  let para = [];
+  const flushPara = () => {
+    if (!para.length) return;
+    const content = [];
+    para.forEach((ln, idx) => {
+      if (idx > 0) content.push({ type: "hardBreak" });
+      content.push(...inlineNodes(ln));
+    });
+    para = [];
+    if (content.length) blocks.push({ type: "paragraph", content });
+  };
+  let i = 0;
+  while (i < lines.length) {
+    const trimmed = lines[i].trim();
+    if (!trimmed) { flushPara(); i += 1; continue; }
+    if (/^(-{3,}|\*{3,}|_{3,})$/.test(trimmed)) { flushPara(); blocks.push({ type: "horizontalRule" }); i += 1; continue; }
+    if (trimmed.startsWith("```")) {
+      flushPara();
+      const language = trimmed.slice(3).trim();
+      const buf = [];
+      i += 1;
+      while (i < lines.length && !lines[i].trim().startsWith("```")) { buf.push(lines[i]); i += 1; }
+      i += 1;
+      blocks.push({
+        type: "codeBlock",
+        attrs: { language: language || null },
+        ...(buf.length ? { content: [{ type: "text", text: buf.join("\n") }] } : {}),
+      });
+      continue;
+    }
+    if (/^[-*•]\s+/.test(trimmed)) {
+      flushPara();
+      const items = [];
+      while (i < lines.length && /^[-*•]\s+/.test(lines[i].trim())) {
+        const inner = inlineNodes(lines[i].trim().replace(/^[-*•]\s+/, ""));
+        items.push({ type: "listItem", content: [inner.length ? { type: "paragraph", content: inner } : { type: "paragraph" }] });
+        i += 1;
+      }
+      blocks.push({ type: "bulletList", content: items });
+      continue;
+    }
+    if (/^\d+[.)]\s+/.test(trimmed)) {
+      flushPara();
+      const start = parseInt(trimmed, 10);
+      const items = [];
+      while (i < lines.length && /^\d+[.)]\s+/.test(lines[i].trim())) {
+        const inner = inlineNodes(lines[i].trim().replace(/^\d+[.)]\s+/, ""));
+        items.push({ type: "listItem", content: [inner.length ? { type: "paragraph", content: inner } : { type: "paragraph" }] });
+        i += 1;
+      }
+      blocks.push({ type: "orderedList", attrs: { start: Number.isNaN(start) ? 1 : start }, content: items });
+      continue;
+    }
+    const heading = trimmed.match(/^(#{1,3})\s+(.*)$/);
+    if (heading) {
+      flushPara();
+      const inner = inlineNodes(heading[2]);
+      blocks.push({ type: "heading", attrs: { level: heading[1].length }, ...(inner.length ? { content: inner } : {}) });
+      i += 1;
+      continue;
+    }
+    para.push(lines[i]);
+    i += 1;
+  }
+  flushPara();
+  return blocks;
+}
+
 // Build the stringified ProseMirror doc for an outgoing comment. Optional
 // leading @mention chips (each {userId,label}) drive Spaces' notification
-// fan-out; the tool proved this exact node shape renders as tag chips.
+// fan-out; the tool proved this exact node shape renders as tag chips. Chips
+// are prepended into the first paragraph so the visual "@Name text…" lead-in
+// stays identical to the pre-markdown behaviour.
 function buildCommentContent(text, mentions = []) {
-  const inline = [];
+  const chips = [];
   for (const m of mentions) {
-    inline.push({ type: "mention", attrs: { id: m.userId, label: m.label, mentionSuggestionChar: "@" } });
-    inline.push({ type: "text", text: " " });
+    chips.push({ type: "mention", attrs: { id: m.userId, label: m.label, mentionSuggestionChar: "@" } });
+    chips.push({ type: "text", text: " " });
+  }
+  const blocks = markdownBlocks(text);
+  if (chips.length) {
+    if (blocks.length && blocks[0].type === "paragraph") {
+      blocks[0].content = [...chips, ...(blocks[0].content || [])];
+    } else {
+      blocks.unshift({ type: "paragraph", content: chips });
+    }
   }
-  if (text) inline.push({ type: "text", text });
-  return JSON.stringify({ type: "doc", content: [{ type: "paragraph", content: inline }] });
+  if (!blocks.length) blocks.push({ type: "paragraph" });
+  return JSON.stringify({ type: "doc", content: blocks });
 }
 
 class SpacesClient {
@@ -201,4 +311,4 @@ class SpacesClient {
   }
 }
 
-module.exports = { SpacesClient, flattenContent, contentText, buildCommentContent, asArray };
+module.exports = { SpacesClient, flattenContent, contentText, buildCommentContent, markdownBlocks, inlineNodes, asArray };

package/core/handlers.js

@@ -29,6 +29,7 @@ const keyring = require("./keyring");
 const { redactSensitive, registerSecrets } = require("./redact");
 const { runDoctorChecks, formatDoctorReport } = require("./doctor");
 const { isNewerVersion } = require("./version");
+const { recordUpgradeRequested, upgradeInFlightSince, clearUpgradeMarker } = require("./upgrade-marker");
 const jobs = require("./jobs");
 const scheduler = require("./scheduler");
 const skillsLib = require("./skills");
@@ -672,23 +673,45 @@ register({
     // delegate to the control plane so the deployment can be rolled with a
     // fresh image pull. The local npm path can't do that.
     if (process.env.AGENTSPACE_POD_TOKEN && process.env.AGENTSPACE_API_URL) {
+      // Redelivery guard: the rollout SIGTERMs this pod mid-turn and Telegram
+      // redelivers the still-unconfirmed /upgrade update to the rebooted pod.
+      // Unguarded, each replay triggered another rollout (one /upgrade was
+      // observed to cause five). A fresh marker means the request was already
+      // honoured — by this boot or the previous one.
+      const since = upgradeInFlightSince();
+      if (since) {
+        const bootedAt = Date.now() - Math.round(process.uptime() * 1000);
+        if (since < bootedAt) {
+          await send(`That upgrade already went through — I restarted onto the freshly pulled image and I'm running v${CURRENT_VERSION}. Skipping the duplicate trigger.`);
+        } else {
+          const mins = Math.max(1, Math.round((Date.now() - since) / 60000));
+          await send(`An upgrade rollout is already in flight (requested ~${mins} min ago) — not triggering another. If I seem stuck, the guard clears 10 minutes after the request.`);
+        }
+        return;
+      }
+      // Ack BEFORE triggering: once the control plane rolls the deployment we
+      // can be SIGTERM'd at any moment, and an unsent ack dies with us.
+      await send("Upgrade requested — AgentSpace will pull latest image and restart, see you in a minute.");
+      recordUpgradeRequested();
       try {
         const result = await requestAgentSpaceUpgrade();
         if (result && (result.status === 202 || result.status === 200)) {
-          await send("Upgrade requested — AgentSpace will pull latest image and restart, see you in a minute.");
           return;
         }
         // A client-side timeout is not a failure: the control plane triggers the
         // rollout fire-and-forget, and a pod being rolled is exactly when its own
         // in-flight request gets cut off. Framing it as "failed" invites a retry
-        // storm of redundant rollouts, so surface it as pending instead.
+        // storm of redundant rollouts, so surface it as pending instead — and
+        // keep the marker, since the rollout is most likely underway.
         if (result && result.status === 0 && /timeout/i.test(result.body || "")) {
           await send("Upgrade request sent but the reply timed out — likely because I'm already restarting. Give it a minute before re-running; I should come back on the new image.");
           return;
         }
+        clearUpgradeMarker();
         await send(`Upgrade request failed (status ${result?.status || "?"}). ${(result?.body || "").slice(0, 300)}`);
         return;
       } catch (e) {
+        clearUpgradeMarker();
         await send(`Upgrade request error: ${e.message || e}`);
         return;
       }

package/core/upgrade-marker.js

@@ -0,0 +1,42 @@
+// Guards /upgrade on AgentSpace-managed pods against the redelivery loop:
+// the control-plane rollout SIGTERMs this pod mid-turn, Telegram long-polling
+// keeps no persisted offset, so the still-unconfirmed /upgrade update is
+// redelivered to the freshly booted pod — which would trigger ANOTHER rollout
+// (observed: one /upgrade → five rollouts in three minutes). A fresh marker
+// on disk tells the handler the request was already honoured.
+
+const fs = require("fs");
+const path = require("path");
+const { CONFIG_DIR } = require("./config");
+
+const FILE = path.join(CONFIG_DIR, "upgrade-in-flight.json");
+// An image pull + pod recreate completes well inside this window; after it a
+// re-run of /upgrade is treated as a genuine new request.
+const FRESH_MS = 10 * 60 * 1000;
+
+// Call right before triggering the rollout. Best-effort: never throws.
+function recordUpgradeRequested() {
+  try {
+    fs.writeFileSync(FILE, JSON.stringify({ at: Date.now() }));
+  } catch (e) { /* the recorder must never itself throw */ }
+}
+
+// Epoch ms of the in-flight upgrade request, or null when none is fresh.
+// Deliberately does NOT clear the file: Telegram may redeliver more than one
+// unconfirmed copy of the command, so the guard must hold until it ages out.
+function upgradeInFlightSince() {
+  try {
+    const rec = JSON.parse(fs.readFileSync(FILE, "utf8"));
+    if (rec && typeof rec.at === "number" && Date.now() - rec.at <= FRESH_MS) {
+      return rec.at;
+    }
+  } catch (e) { /* absent/unreadable ⇒ no upgrade in flight */ }
+  return null;
+}
+
+// Call when the trigger definitively failed, so an immediate retry is allowed.
+function clearUpgradeMarker() {
+  try { fs.unlinkSync(FILE); } catch (e) {}
+}
+
+module.exports = { recordUpgradeRequested, upgradeInFlightSince, clearUpgradeMarker };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "3.0.33",
+  "version": "3.0.35",
   "description": "An always-on, provider-agnostic coding-agent harness for Claude Code and OpenAI Codex via chat",
   "main": "bot.js",
   "bin": {
@@ -9,7 +9,7 @@
   "scripts": {
     "setup": "node setup.js",
     "start": "node bot.js",
-    "pretest": "node test-provider-fixture.js && node test-provider-core-prompt.js && node test-provider-prompt-parity.js && node test-provider-env-isolation.js && node test-provider-resume-parity.js && node test-provider-bootstrap.js && node test-provider-registry.js && node test-provider-stream-decoder.js && node test-provider-events.js && node test-provider-claude.js && node test-provider-codex.js && node test-provider-runner.js && node test-provider-run-lifecycle.js && node test-provider-gateway.js && node test-provider-side-chat.js && node test-single-runtime.js && node test-provider-migration-backup.js && node test-provider-state-migration.js && node test-provider-session-history.js && node test-provider-session-commands.js && node test-provider-compaction.js && node test-provider-scheduler.js && node test-cursor-removal.js && node test-utility-provider-policy.js && node test-provider-subagent.js && node test-provider-recall.js && node test-provider-pack-review.js && node test-memory-mutation-queue.js && node test-provider-dream.js && node test-provider-enforcer.js && node test-provider-tool-hooks.js && node test-provider-boot-matrix.js && node test-provider-capabilities.js && node test-provider-language.js && node test-provider-coupling-audit.js && node test-provider-parity-e2e.js && node test-kazee-channel-health.js && node test-prompt-budget.js && node test-agency-ledger.js && node test-agency-deliberate.js && node test-guardrail.js",
+    "pretest": "node test-provider-fixture.js && node test-provider-core-prompt.js && node test-provider-prompt-parity.js && node test-provider-env-isolation.js && node test-provider-resume-parity.js && node test-provider-bootstrap.js && node test-provider-registry.js && node test-provider-stream-decoder.js && node test-provider-events.js && node test-provider-claude.js && node test-provider-codex.js && node test-provider-runner.js && node test-provider-run-lifecycle.js && node test-provider-gateway.js && node test-provider-side-chat.js && node test-single-runtime.js && node test-provider-migration-backup.js && node test-provider-state-migration.js && node test-provider-session-history.js && node test-provider-session-commands.js && node test-provider-compaction.js && node test-provider-scheduler.js && node test-cursor-removal.js && node test-utility-provider-policy.js && node test-provider-subagent.js && node test-provider-recall.js && node test-provider-pack-review.js && node test-memory-mutation-queue.js && node test-provider-dream.js && node test-provider-enforcer.js && node test-provider-tool-hooks.js && node test-provider-boot-matrix.js && node test-provider-capabilities.js && node test-provider-language.js && node test-provider-coupling-audit.js && node test-provider-parity-e2e.js && node test-kazee-channel-health.js && node test-prompt-budget.js && node test-agency-ledger.js && node test-agency-deliberate.js && node test-guardrail.js && node test-spaces-comment-content.js",
     "test": "OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node -e \"require('./vault'); console.log('OK')\" && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-runner-watchdog-static.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-usage-accounting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-recall-engine.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-recall-graph.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-recall-discoverer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-project-transcripts-smoke.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-abilities.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-ability-extraction.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-ability-couse.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-ability-transfer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-ability-tiers.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-ability-merge-guard.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-learning-e2e.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-pack-nesting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-journal-dedupe.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-read-signal.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-tools.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-tool-graph.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-tool-guard.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-tooling-mode.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-tool-manifest.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-recall-evolution.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-approval-async.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-unified-identity.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-queue-routing.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-persona-packs.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-speaker-persona.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-persona-pipeline.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-recall-relationship-gate.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-relationship.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-enforcer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-identity-prune.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-kazee-message-id.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-delivery-contract.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-run-lock.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-web-sessions.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-telegram-poll-recovery.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-telegram-409-grace.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-telegram-conflict-heal-guard.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-single-instance.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-streaming-split.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=./test-fixtures/fake-agent-cli.js FAKE_AGENT_KIND=claude AGENT_ENV_PASSTHROUGH=FAKE_AGENT_KIND node test-spaces-autoenable.js"
   },
   "files": [

package/test-provider-language.js

@@ -32,16 +32,15 @@ assert.doesNotMatch(setup, /Your Claude Code bot is connected|Description=Claude
 
 const pkg = JSON.parse(read("package.json"));
 assert.match(pkg.description, /provider-agnostic coding-agent harness/i);
-// 3.0.33 approved by Sumeet 2026-07-16 ("build and push it all out"): full-sudo
-// pod image — the rebuilt :base widens claudia's sudoers from a command
-// allow-list to NOPASSWD:ALL so the agent can operate its own container like a
-// normal machine (pairs with agent-space's IaaS-style security policy, which
-// fences the blast radius at the NetworkPolicy instead of crippling the user).
-// Re-cut of v3.0.32, whose build tripped THIS guard again — package.json was
-// bumped but the pin below wasn't, so pretest asserted 3.0.31, the build job
-// failed, docker was skipped, and nothing was published (same failure mode as
-// v3.0.30 -> v3.0.31).
-assert.strictEqual(pkg.version, "3.0.33", "release version must remain unchanged without explicit approval");
+// 3.0.35 approved by Sumeet 2026-07-17 ("yes will this also ship the issue of
+// new lines not showing properly?"): Spaces comments become real ProseMirror
+// docs. buildCommentContent used to post the whole reply as ONE text node in
+// ONE paragraph, so TipTap collapsed every newline and printed markdown
+// asterisks literally — the unreadable-wall screenshot from the euvy bot
+// thread. Now markdown converts to paragraphs/hardBreaks, bold/italic/code
+// marks, bullet/ordered lists, headings, code blocks and dividers — all node
+// types verified present in the Spaces editor schema.
+assert.strictEqual(pkg.version, "3.0.35", "release version must remain unchanged without explicit approval");
 for (const keyword of ["claude", "codex", "coding-agent", "provider-agnostic"]) {
   assert.ok(pkg.keywords.includes(keyword), `package keyword missing: ${keyword}`);
 }