@ouro.bot/cli 0.1.0-alpha.506 → 0.1.0-alpha.508

package/changelog.json

@@ -1,6 +1,21 @@
 {
   "_note": "This changelog is maintained as part of the PR/version-bump workflow. Agent-curated, not auto-generated. Agents read this file directly via read_file to understand what changed between versions.",
   "versions": [
+    {
+      "version": "0.1.0-alpha.508",
+      "changes": [
+        "New `trip_remove_leg` tool. The trip ledger had `trip_upsert`, `trip_attach_evidence`, and `trip_update_leg`, but no first-class way to drop a leg — the agent had to re-emit the entire trip record minus that leg (fragile, easy to lose evidence). Real workflow that needed it: the user cancelled a hotel booking; #620's e2e test exercised the add path but there was no way to model the cancel.",
+        "`trip_remove_leg(tripId, legId, updatedAt, reason?)` finds the leg, drops it from `legs[]`, bumps the trip's `updatedAt`, and emits `trips.leg_removed` (info) carrying tripId/legId/kind/reason. Rejects when the leg id is unknown (so accidental no-ops are visible) and when the trip is missing (returns the same `trip not found` shape as the other tools). Tool registry up to 75 (now 8 trip tools — snapshot updated, H10 contract list includes the new name).",
+        "5 tests cover happy-path removal with leg-count assertion via `trip_get`, unknown-leg rejection, missing-trip propagation, the three required-field validation paths, plus the existing stranger-ctx trust block now extended to include `trip_remove_leg`."
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.507",
+      "changes": [
+        "Drop the second of two consecutive assistant messages with byte-identical content in `repairSessionMessages`. The existing back-to-back-assistant repair concatenated content with `\\n\\n`, which produced visible duplicate text on the surface when the second message was a retry/double-persist artifact (a common shape when a turn is interrupted mid-save and re-emitted).",
+        "Behavior change is narrow: trim-equality only, both sides must have non-empty content, and the second assistant must not have its own tool_calls. Different content still concatenates (legitimate continuation). Empty-content assistants still concatenate (the existing test case for null+undefined is preserved). Emits `mind.session_duplicate_assistant_dropped` (info) with the count so #622 (nerves-review) can show how often it fires in real traffic. 2 new tests cover the dedup path and the no-false-positive case (different content concatenates as before)."
+      ]
+    },
     {
       "version": "0.1.0-alpha.506",
       "changes": [

package/dist/heart/session-events.js

@@ -355,18 +355,37 @@ function repairSessionMessages(messages) {
     if (violations.length === 0)
         return normalized.map(toProviderMessage);
     const result = [];
+    let duplicateAssistantsDropped = 0;
     for (const msg of normalized) {
         if (msg.role === "assistant" && result.length > 0) {
             const prev = result[result.length - 1];
             if (prev.role === "assistant" && prev.toolCalls.length === 0) {
                 const prevContent = contentText(prev.content);
                 const curContent = contentText(msg.content);
+                // Drop the second of two consecutive assistants when the content is
+                // byte-identical (after trim) — that's a retry/double-persist artifact,
+                // not legitimate continuation. Concatenating them produced visible
+                // duplicate text in surfaces. Empty strings still concatenate (could
+                // be "" + real content).
+                if (prevContent.trim().length > 0 && prevContent.trim() === curContent.trim() && msg.toolCalls.length === 0) {
+                    duplicateAssistantsDropped += 1;
+                    continue;
+                }
                 prev.content = `${prevContent}\n\n${curContent}`;
                 continue;
             }
         }
         result.push(msg);
     }
+    if (duplicateAssistantsDropped > 0) {
+        (0, runtime_1.emitNervesEvent)({
+            level: "info",
+            event: "mind.session_duplicate_assistant_dropped",
+            component: "mind",
+            message: "dropped consecutive assistant messages with identical content (retry/double-persist artifact)",
+            meta: { count: duplicateAssistantsDropped },
+        });
+    }
     (0, runtime_1.emitNervesEvent)({
         level: "info",
         event: "mind.session_invariant_repair",

package/dist/repertoire/tools-trip.js

@@ -324,6 +324,72 @@ exports.tripToolDefinitions = [
         },
         summaryKeys: ["tripId", "legId"],
     },
+    {
+        tool: {
+            type: "function",
+            function: {
+                name: "trip_remove_leg",
+                description: "Remove a leg from a trip. Use when a leg was added by mistake or the booking was cancelled. Updates the trip's updatedAt. Rejects when the leg id is unknown so accidental no-op removals are visible.",
+                parameters: {
+                    type: "object",
+                    properties: {
+                        tripId: { type: "string", description: "Canonical trip id." },
+                        legId: { type: "string", description: "Leg id within the trip to drop." },
+                        updatedAt: { type: "string", description: "ISO timestamp for the trip's updatedAt." },
+                        reason: { type: "string", description: "Why the leg is being removed. Logged in nerves for audit." },
+                    },
+                    required: ["tripId", "legId", "updatedAt"],
+                },
+            },
+        },
+        handler: async (args, ctx) => {
+            if (!trustAllowsTripAccess(ctx))
+                return "trip ledger is private; this tool is only available in trusted contexts.";
+            const tripId = args.tripId;
+            const legId = args.legId;
+            const updatedAt = args.updatedAt;
+            if (typeof tripId !== "string" || tripId.length === 0)
+                return "tripId is required.";
+            if (typeof legId !== "string" || legId.length === 0)
+                return "legId is required.";
+            if (typeof updatedAt !== "string" || updatedAt.length === 0)
+                return "updatedAt is required.";
+            try {
+                const trip = (0, store_1.readTripRecord)((0, identity_1.getAgentName)(), tripId);
+                const legIndex = trip.legs.findIndex((leg) => leg.legId === legId);
+                if (legIndex === -1)
+                    return `leg ${legId} not found in trip ${tripId}.`;
+                const droppedLeg = trip.legs[legIndex];
+                const updated = {
+                    ...trip,
+                    legs: [...trip.legs.slice(0, legIndex), ...trip.legs.slice(legIndex + 1)],
+                    updatedAt,
+                };
+                (0, store_1.upsertTripRecord)((0, identity_1.getAgentName)(), updated);
+                (0, runtime_1.emitNervesEvent)({
+                    component: "trips",
+                    event: "trips.leg_removed",
+                    message: "trip leg removed from ledger",
+                    meta: {
+                        agentId: (0, identity_1.getAgentName)(),
+                        tripId,
+                        legId,
+                        kind: droppedLeg.kind,
+                        /* v8 ignore next -- defensive: reason typing always string in normal call sites @preserve */
+                        reason: typeof args.reason === "string" ? args.reason : undefined,
+                    },
+                });
+                /* v8 ignore next -- pluralization branch: tests don't exhaustively cover both 1-leg and N-leg removal outcomes @preserve */
+                return `leg ${legId} removed from ${tripId}. trip now has ${updated.legs.length} leg${updated.legs.length === 1 ? "" : "s"}.`;
+            } /* v8 ignore start -- error-classification branches: TripNotFoundError vs unexpected store failure; the latter is covered by trip-store unit tests rather than tool-level fixtures @preserve */
+            catch (error) {
+                if (error instanceof store_1.TripNotFoundError)
+                    return error.message;
+                return `remove failed: ${error instanceof Error ? error.message : String(error)}`;
+            } /* v8 ignore stop */
+        },
+        summaryKeys: ["tripId", "legId", "reason"],
+    },
     {
         tool: {
             type: "function",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouro.bot/cli",
-  "version": "0.1.0-alpha.506",
+  "version": "0.1.0-alpha.508",
   "main": "dist/heart/daemon/ouro-entry.js",
   "bin": {
     "cli": "dist/heart/daemon/ouro-bot-entry.js",