@particle-academy/prism-acp 0.5.1 → 0.5.3

package/README.md

@@ -97,6 +97,12 @@ non-credentials alike. If the child needs an orchestrator session or terminal
 id, a callback URL, or a feature flag, name it in `allowEnv` or it will not
 arrive. This failure is silent: the child can start and appear healthy while
 being unable to identify itself, rather than reporting a missing variable.
+An allowed variable whose parent value is `''` is forwarded as an empty string.
+That can be worse than omitting it: a referenced-but-unset value in an MCP
+config can stop every server in that file. When there is no value to give,
+omit the variable instead of passing its name with a blank value. `childEnv`
+does not filter blanks for you, because an intentionally empty value can be
+meaningful for some variable names and the package will not guess which.
 
 Nothing here mutates `process.env`. A workspace may hold an API key on purpose —
 other consumers beside this one legitimately bill per token — so the child's
@@ -161,6 +167,19 @@ drivers because their measured resume behavior differs. Codex thread ids are
 the provider's own captured ids and are the same ids accepted by its resume
 method; ACP-minted ids are refused.
 
+The human half of the conversation differs too. Against a real Claude child,
+the CLI did not echo the client's prompt as `user_message_chunk`, and loading
+its session replayed zero updates. A Claude client cannot reconstruct the whole
+conversation from ACP alone; record the human turn yourself. Codex's mapping
+does emit text from `userMessage` items as `user_message_chunk` during history
+replay, so a Codex `session/load` will carry that human text. Measured against a
+real Codex child, a live turn reports the user's text as one
+`user_message_chunk`. Releases 0.5.2 and earlier emitted it twice because our
+mapper handled both `item/started` and `item/completed`; client-side
+deduplication added for that behavior is no longer needed. Record the human
+turn yourself for both providers, and treat a live `user_message_chunk` as the
+agent's record of what it received, not as new input to append again.
+
 ### Refusing an unknown id at load, not a turn later
 
 Pass `probeSession` and an id naming **no** conversation is refused by
@@ -304,6 +323,14 @@ A client then speaks ACP on those pipes. Several sessions run at once, each with
 its own agent process, its own in-flight turn and its own updates — tagged with
 the session they belong to, because a room of agents shares one stream.
 
+### Commit streamed messages at the turn boundary
+
+`agent_message_chunk` and `agent_thought_chunk` carry no message id. The turn
+boundary is the only delimiter between replies, so commit the accumulated
+message there. Clearing the in-flight message without committing loses the
+reply (including the whole reply in a single-message turn); not clearing it
+appends the next turn to the previous one.
+
 ## Development
 
 ```sh

package/dist/codex/driver.js

@@ -474,11 +474,13 @@ export class CodexDriver {
             this.#recordUnmapped('completed Codex item had an unknown status', item);
         }
         if (type === 'userMessage') {
-            const text = textFromContent(item.content);
-            if (text.length > 0)
-                this.#emit({ sessionUpdate: 'user_message_chunk', content: { type: 'text', text } });
-            else
-                this.#recordUnmapped('Codex userMessage had no text content', item);
+            if (replay || completed) {
+                const text = textFromContent(item.content);
+                if (text.length > 0)
+                    this.#emit({ sessionUpdate: 'user_message_chunk', content: { type: 'text', text } });
+                else
+                    this.#recordUnmapped('Codex userMessage had no text content', item);
+            }
             return;
         }
         if (type === 'agentMessage') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@particle-academy/prism-acp",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Speak the Agent Client Protocol to a coding-agent CLI the user has already authenticated. No API key, no third-party adapter.",
   "license": "MIT",
   "repository": {