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

package/changelog.json

@@ -1,6 +1,14 @@
 {
   "_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": [

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.507",
+  "version": "0.1.0-alpha.508",
   "main": "dist/heart/daemon/ouro-entry.js",
   "bin": {
     "cli": "dist/heart/daemon/ouro-bot-entry.js",