@pellux/goodvibes-daemon 1.28.7 → 1.28.9

package/CHANGELOG.md

@@ -6,6 +6,52 @@ All notable changes to the GoodVibes daemon.
 
 ## [Unreleased]
 
+## [1.28.9] - 2026-08-02
+
+### Changes
+
+- Changed: a conversational turn is told to understand what it captures, not
+  just file it — an itinerary also means an away-span said back in plain
+  words, travelers who are people in the owner's life, and durable facts
+  about the destination, and the turn uses what it stored: it names
+  collisions with existing plans and offers the obviously useful next steps
+  once (a reminder before departure, weather at the destination). Anything
+  beyond the conversation is still proposed and waits for a yes (platform
+  runtime 2.0.7).
+
+## [1.28.8] - 2026-08-02
+
+### Changes
+
+- **Fixed: a chat message gets an answer, never workflow paperwork.** The
+  conversation gate's decision that a channel message is conversation is now
+  authoritative — the review-wording heuristic that turned "I'll review the
+  route" in a transcript into a full write-review-fix-confirm chain can no
+  longer override it. When a chain does legitimately run, the reply sent back
+  over the channel carries what the agent actually found or did; chain status
+  lines stay in the operator progress feed. Duplicate replies from the two
+  completion reporters are gone (platform runtime 2.0.6).
+- **Added: personal information shared in conversation gets captured.** A
+  conversational channel turn — which previously ran with no tools at all —
+  now carries the profile capture tool with per-run owner authority: a trip
+  itinerary pasted into Telegram lands in the owner profile's Plans section
+  with its dates, flights, travelers and confirmation number, and the reply
+  states concretely what was stored. Configurable via
+  `profile.conversationalCapture` (default on) and `profile.ownerChannels`
+  (defaults to the occasions nudge channel).
+- **Fixed: a chat session is filed as a chat.** Channel-originated sessions
+  are classified by their channel instead of as TUI project sessions rooted
+  in a filesystem path, rollover re-derives the classification, and a sender
+  on the channel's owner allowlist is attributed as the owner principal
+  instead of unknown. The daemon now hands its channel ingress policy to the
+  session intake so that attribution works end to end.
+- Fixed: this daemon's settings file is rewritten only by this daemon.
+  Clients reading the daemon settings tier migrate their in-memory view and
+  leave the file bytes untouched, so a newer client can no longer rewrite the
+  file out from under an older running daemon; the daemon's own migration
+  folds the rename, the receipt, and a reader floor into one atomic write
+  (platform runtime 2.0.6).
+
 ## [1.28.7] - 2026-08-02
 
 ### Changes

package/README.md

@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/mgd34msu/goodvibes-daemon/actions/workflows/ci.yml/badge.svg)](https://github.com/mgd34msu/goodvibes-daemon/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Version](https://img.shields.io/badge/version-1.28.7-blue.svg)](https://github.com/mgd34msu/goodvibes-daemon)
+[![Version](https://img.shields.io/badge/version-1.28.9-blue.svg)](https://github.com/mgd34msu/goodvibes-daemon)
 
 The GoodVibes daemon: one long-running process per machine that holds the control plane every
 GoodVibes client talks to. It answers the operator verb families over HTTP, reads and replies on

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-daemon",
-  "version": "1.28.7",
+  "version": "1.28.9",
   "description": "The GoodVibes daemon \u2014 the one long-running host for the control plane, channels, cluster membership, scheduled work, knowledge and memory stores, and the verb families every GoodVibes client calls.",
   "type": "module",
   "main": "src/daemon/cli.ts",
@@ -65,8 +65,8 @@
     "@anthropic-ai/bedrock-sdk": "^0.28.1",
     "@anthropic-ai/sdk": "^0.82.0",
     "@ast-grep/napi": "^0.42.0",
-    "@pellux/goodvibes-sdk": "2.0.5",
-    "@pellux/goodvibes-terminal-shell": "2.0.5",
+    "@pellux/goodvibes-sdk": "2.0.7",
+    "@pellux/goodvibes-terminal-shell": "2.0.7",
     "bash-language-server": "^5.6.0",
     "fuse.js": "^7.1.0",
     "graphql": "^16.13.2",

package/src/runtime/services.ts

@@ -5,6 +5,7 @@ import { ChannelPolicyManager } from '@pellux/goodvibes-sdk/platform/channels';
 import { ApprovalBroker, GatewayMethodCatalog, SharedSessionBroker, buildSharedSessionAgentSpawnRoutingInput } from '@pellux/goodvibes-sdk/platform/control-plane';
 import { AcpHostService } from '@pellux/goodvibes-sdk/platform/acp';
 import { continuationChainOptions } from '@pellux/goodvibes-sdk/platform/agents';
+import { PersonalCaptureHolder, conversationalTurnSpawnOptions } from '@pellux/goodvibes-sdk/platform/personal-capture';
 import { resolvePairingWebOrigin } from '@pellux/goodvibes-sdk/platform/pairing';
 import { attachWsOnlyGatewayVerbHandlers } from '@pellux/goodvibes-terminal-shell';
 import { composeMailDeps } from './mail-composition.ts';
@@ -205,6 +206,13 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   } = createAgentGraph({
     runtimeBus: options.runtimeBus, workingDirectory, configManager, providerRegistry,
   });
+  // The one late-binding holder for the personal-capture port. The gateway verb
+  // groups fill it (they own the owner-profile store and the occasions service),
+  // and the agent orchestrator reads it when it builds a run's tool registry.
+  // Registration happens before setDependencies in this file, but the registry
+  // is built per run, so by the time a conversational turn asks for `profile`
+  // the port is already in place.
+  const personalCapture = new PersonalCaptureHolder();
   const hookDispatcher = new HookDispatcher({ agentManager, toolLLM, projectRoot: workingDirectory }, hookActivityTracker);
   configManager.attachHookDispatcher(hookDispatcher);
   const hookWorkbench = createHookWorkbench({
@@ -238,11 +246,23 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
           getCategory: (name: string) => configManager.getCategory(name as never),
         },
       }),
+      // What the answering agent is given and what it is told. `restrictTools:
+      // true` with no tool list used to mean an EMPTY registry: the agent could
+      // emit text and nothing else, which is why a trip pasted into a channel
+      // was answered and stored nowhere. This names the tools a conversational
+      // turn actually needs (read, find, fetch, and the `profile` capture tool),
+      // supplies the instruction that makes recording part of answering, and
+      // carries the write authority for the surface the message arrived on.
+      // It also supplies the run's context, which is why the bare
+      // `context: shared-session:<id>` line that used to sit below is gone.
+      // Spread BEFORE the routing builder so an explicit tool list coming from
+      // a routing intent still wins — that builder emits `tools` only when it
+      // actually has one.
+      ...conversationalTurnSpawnOptions(input, { configReader: configManager }),
       // Spawn routing through the SDK's shared model-reference resolver
       // (unique-across-registry auto-qualifies; ambiguous and unknown ids throw
       // errors naming real candidates), against the live registry's models.
       ...buildSharedSessionAgentSpawnRoutingInput(input.routing, { restrictTools: true, modelCandidates: providerRegistry.listModels() }),
-      context: `shared-session:${input.sessionId}`,
     });
     return { agentId: record.id };
   });
@@ -511,6 +531,13 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     // an unregistered workspace refuses with something actionable.
     workspaceCheckpointManager: checkpointing.gatewayManager,
     conversationRewindPort: createSessionConversationRewindPort(), sessionBroker, secretsManager, stepUpService,
+    // Fills the capture port with the owner-profile store and the occasions
+    // service, so the `profile` tool a conversational turn is given has
+    // somewhere real to write.
+    personalCapture,
+    // The session intake attributes a sender on this policy's owner allowlist
+    // to the owner principal instead of unknown.
+    channelPolicy,
     approvalBroker, requestApproval: (input) => approvalBroker.requestApproval(input),
     // approvals.raise — a surface CREATING an ask in this broker. Without it the
     // verb is cataloged and unhandled, and a client whose prompt runs outside
@@ -596,6 +623,9 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     sandboxSessionRegistry,
     workflowServices: workflow,
     contextAccountingHolder,
+    // Without this the `profile` tool is never registered, and a conversational
+    // turn that was told to record what the owner said has nothing to call.
+    personalCapture,
   });
 
   // Continuity reads (recovery-file presence, last-session pointer) scoped to

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '1.28.7';
+let _version = '1.28.9';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   // Only trust a version read from OUR OWN package.json. A Bun single-file