@possumtech/rummy 2.0.1 → 2.1.0

package/src/server/protocol.js

@@ -1,11 +1,2 @@
-/**
- * Server↔client wire-protocol version. Bumped whenever the RPC shape
- * or notification payload shape changes in a way that breaks existing
- * clients. Clients pass their own version in `rummy/hello`; server
- * rejects MAJOR mismatch. Git commit log is the changelog.
- *
- *   MAJOR — breaking change (removed/renamed method, shape change)
- *   MINOR — additive change (new method, new optional field)
- *   PATCH — internal fix visible to the wire shape
- */
+// Wire protocol version; rummy/hello rejects MAJOR mismatch. Semver.
 export const RUMMY_PROTOCOL_VERSION = "2.0.0";

package/src/sql/functions/slugify.js

@@ -1,18 +1,16 @@
+import encodeSegment from "../../agent/pathEncode.js";
+
 export const deterministic = true;
 
-// Build URI paths the model can round-trip:
-//   "history,mongol,khan"   → "history/mongol/khan"   (commas become path separators)
-//   "contents of Document 1" → "contents_of_Document_1" (spaces become underscores)
-// Slice on decoded text, then split-encode-join per segment so / survives as
-// a separator while anything URL-unsafe inside a segment gets escaped.
+// commas→/, then encode-per-segment so / survives as separator.
+// encodeSegment handles spaces→_ + URL-encode (single rule, used everywhere).
 export default function slugify(text) {
 	if (!text) return "";
 	return text
 		.slice(0, 80)
 		.replace(/,/g, "/")
-		.replace(/ /g, "_")
 		.split("/")
 		.filter(Boolean)
-		.map(encodeURIComponent)
+		.map(encodeSegment)
 		.join("/");
 }

package/src/sql/v_model_context.sql

@@ -16,8 +16,8 @@ visible AS (
 		, e.attributes
 		, COALESCE(s.category, 'logging') AS category
 		, CASE
-			WHEN rv.visibility = 'archived' THEN NULL
 			WHEN s.model_visible = 0 THEN NULL
+			WHEN rv.visibility = 'archived' THEN NULL
 			ELSE rv.visibility
 		END AS effective_visibility
 	FROM run_views AS rv
@@ -38,10 +38,7 @@ projected AS (
 		, attributes
 		-- Category comes from schemes table — plugins declare it via registerScheme().
 		, category
-		, CASE
-			WHEN effective_visibility IN ('visible', 'summarized') THEN body
-			ELSE ''
-		END AS body
+		, body
 	FROM visible
 	WHERE effective_visibility IS NOT NULL
 )
@@ -67,13 +64,9 @@ SELECT
 				WHEN 'prompt' THEN 5
 				ELSE 5
 			END
-			, CASE scheme WHEN 'skill' THEN 0 ELSE 1 END
-			, CASE visibility
-				WHEN 'summarized' THEN 0
-				ELSE 1
-			END
+			, scheme
 			, turn
 			, updated_at
-			, id
+			, path
 	) AS ordinal
 FROM projected;