claude-dev-env 8.90.0 → 8.91.0

package/hooks/session/compaction_user_messages.py

@@ -1,20 +1,16 @@
 #!/usr/bin/env python3
-"""Write every message the user typed in a session to a file, and print its path.
+"""Read every message the user typed from a Claude Code or Codex transcript.
 
 ::
 
-    python3 compaction_user_messages.py ~/.claude/projects/p/abc.jsonl
-    -> ~/.claude/compaction-recall/abc.md
+    Claude Code entry {"type": "user", "message": {"content": "fix login"}}
+    Codex entry {"type": "event_msg",
+                 "payload": {"type": "user_message", "message": "fix login"}}
+    -> TypedMessage(position=1, timestamp=..., text="fix login")
 
-The agent runs this after a compaction. Only the path reaches its context.
-A PostToolUse agent hook reads the file, condenses it, and returns the
-condensed messages to the agent.
-
-The file holds each typed message with its number and timestamp, oldest
-first. The trim rules live in the hook prompt. The file sits under
-``~/.claude`` so the hook's Read permission rule can name its folder.
-The whole transcript is read each time, so a second compaction yields
-the same messages plus the newer ones.
+The compaction recall hook trims these messages and returns them to the
+agent after a compaction. The whole transcript is read each time, so a second
+compaction yields the same messages plus the newer ones.
 """
 
 from __future__ import annotations
@@ -30,31 +26,26 @@ if _hooks_directory not in sys.path:
     sys.path.insert(0, _hooks_directory)
 
 from hooks_constants.compaction_user_messages_constants import (
-    ALL_RECALL_DIRECTORY_PARTS,
     ALL_SKIPPED_ENTRY_FLAGS,
     ASSISTANT_ENTRY_TYPE,
     ATTACHMENT_ENTRY_TYPE,
     ATTACHMENT_KEY,
     BLOCK_SEPARATOR,
     BLOCK_TYPE_KEY,
+    CODEX_AGENT_MESSAGE_TYPE,
+    CODEX_EVENT_ENTRY_TYPE,
+    CODEX_IMAGES_KEY,
+    CODEX_PAYLOAD_KEY,
+    CODEX_USER_MESSAGE_TYPE,
     CONTENT_KEY,
-    CUT_MESSAGE_SUFFIX_TEMPLATE,
     ENTRY_TYPE_KEY,
     HARNESS_TAG_OPENING_PATTERN,
     HUMAN_ORIGIN_KIND,
     IMAGE_BLOCK_TYPE,
     IMAGE_PLACEHOLDER,
-    LEFT_OUT_MESSAGES_TEMPLATE,
-    MAXIMUM_EXTRACT_CHARACTERS,
-    MAXIMUM_MESSAGE_CHARACTERS,
-    MESSAGE_HEADING_TEMPLATE,
     MESSAGE_KEY,
-    MESSAGES_HEADING,
-    NO_MESSAGES_TEXT,
     ORIGIN_KEY,
     ORIGIN_KIND_KEY,
-    OUTPUT_ENCODING,
-    OUTPUT_FILE_SUFFIX,
     QUEUED_COMMAND_ATTACHMENT_TYPE,
     QUEUED_PROMPT_KEY,
     SLASH_COMMAND_ARGS_PATTERN,
@@ -62,13 +53,8 @@ from hooks_constants.compaction_user_messages_constants import (
     TEXT_BLOCK_TYPE,
     TEXT_KEY,
     TIMESTAMP_KEY,
-    TRANSCRIPT_ARGUMENT_COUNT,
-    USAGE_EXIT_CODE,
-    USAGE_TEXT,
     USER_ENTRY_TYPE,
 )
-from hooks_constants.session_file_id import session_file_id
-from hooks_constants.setup_project_paths_constants import DECODE_ERRORS_POLICY, UTF8_ENCODING
 
 
 @dataclass(frozen=True)
@@ -131,8 +117,34 @@ def _is_typed_by_a_person(all_entry_fields: dict[str, object]) -> bool:
     return not isinstance(origin, dict) or origin.get(ORIGIN_KIND_KEY) == HUMAN_ORIGIN_KIND
 
 
+def _codex_payload(all_entry_fields: dict[str, object]) -> dict[str, object]:
+    if all_entry_fields.get(ENTRY_TYPE_KEY) != CODEX_EVENT_ENTRY_TYPE:
+        return {}
+    payload = all_entry_fields.get(CODEX_PAYLOAD_KEY)
+    return payload if isinstance(payload, dict) else {}
+
+
+def _codex_user_text(all_payload_fields: dict[str, object]) -> str | None:
+    message = all_payload_fields.get(MESSAGE_KEY)
+    typed_text = _typed_text(message) if isinstance(message, str) else None
+    if not all_payload_fields.get(CODEX_IMAGES_KEY):
+        return typed_text
+    return BLOCK_SEPARATOR.join(
+        each_part for each_part in (typed_text, IMAGE_PLACEHOLDER) if each_part
+    )
+
+
+def _is_assistant_turn(all_entry_fields: dict[str, object]) -> bool:
+    if all_entry_fields.get(ENTRY_TYPE_KEY) == ASSISTANT_ENTRY_TYPE:
+        return True
+    return _codex_payload(all_entry_fields).get(ENTRY_TYPE_KEY) == CODEX_AGENT_MESSAGE_TYPE
+
+
 def _entry_text(all_entry_fields: dict[str, object]) -> str | None:
     entry_type = all_entry_fields.get(ENTRY_TYPE_KEY)
+    codex_payload = _codex_payload(all_entry_fields)
+    if codex_payload.get(ENTRY_TYPE_KEY) == CODEX_USER_MESSAGE_TYPE:
+        return _codex_user_text(codex_payload)
     if entry_type == USER_ENTRY_TYPE and _is_typed_by_a_person(all_entry_fields):
         message = all_entry_fields.get(MESSAGE_KEY)
         content = message.get(CONTENT_KEY) if isinstance(message, dict) else None
@@ -152,7 +164,7 @@ def _typed_texts(all_transcript_lines: Iterable[str]) -> Iterator[tuple[str, str
     previous_text = None
     for each_line in all_transcript_lines:
         all_entry_fields = _parsed_entry(each_line)
-        if all_entry_fields.get(ENTRY_TYPE_KEY) == ASSISTANT_ENTRY_TYPE:
+        if _is_assistant_turn(all_entry_fields):
             previous_text = None
             continue
         entry_text = _entry_text(all_entry_fields)
@@ -174,6 +186,8 @@ def typed_user_messages(all_transcript_lines: Iterable[str]) -> list[TypedMessag
         isCompactSummary, isSidechain, origin peer     -> left out
         user "<system-reminder>...", "<local-command-stdout>" -> left out
         attachment queued_command "also check logs"    -> kept
+        Codex event_msg user_message "fix it"          -> kept
+        Codex event_msg user_message with images       -> text plus "[image]"
 
     A message repeated right after itself, such as a queued prompt that later
     lands as a user entry, counts once.
@@ -190,118 +204,3 @@ def typed_user_messages(all_transcript_lines: Iterable[str]) -> list[TypedMessag
             _typed_texts(all_transcript_lines), start=1
         )
     ]
-
-
-def _capped_text(text: str) -> str:
-    if len(text) <= MAXIMUM_MESSAGE_CHARACTERS:
-        return text
-    cut_count = len(text) - MAXIMUM_MESSAGE_CHARACTERS
-    return text[:MAXIMUM_MESSAGE_CHARACTERS] + CUT_MESSAGE_SUFFIX_TEMPLATE.format(
-        cut_count=cut_count
-    )
-
-
-def _rendered_message(message: TypedMessage) -> str:
-    return MESSAGE_HEADING_TEMPLATE.format(
-        position=message.position,
-        timestamp=message.timestamp,
-        text=_capped_text(message.text),
-    )
-
-
-def bounded_messages(all_messages: list[TypedMessage]) -> list[TypedMessage]:
-    """Return the messages that fit the extract budget: the first, then the newest.
-
-    ::
-
-        5 short messages             -> all 5
-        #1 plus 400 long messages    -> #1, then the newest that fit
-
-    The first message keeps the session's original ask. The newest carry the
-    latest corrections, which override older asks.
-
-    Args:
-        all_messages: Every typed message, oldest first.
-
-    Returns:
-        The kept messages, oldest first.
-    """
-    if not all_messages:
-        return []
-    first_message, *all_later_messages = all_messages
-    remaining_characters = MAXIMUM_EXTRACT_CHARACTERS - len(_rendered_message(first_message))
-    all_kept_later_messages: list[TypedMessage] = []
-    for each_message in reversed(all_later_messages):
-        message_size = len(_rendered_message(each_message))
-        if message_size > remaining_characters:
-            break
-        remaining_characters -= message_size
-        all_kept_later_messages.append(each_message)
-    return [first_message, *reversed(all_kept_later_messages)]
-
-
-def rendered_extract(all_messages: list[TypedMessage]) -> str:
-    """Return the file text: a heading, then the kept messages.
-
-    ::
-
-        [#1 "fix login"]  -> "# User messages, oldest first" + "## 1 | <ts>\\nfix login"
-
-    Args:
-        all_messages: Every typed message, oldest first.
-
-    Returns:
-        The text the extractor writes to its output file.
-    """
-    if not all_messages:
-        return MESSAGES_HEADING + NO_MESSAGES_TEXT
-    all_kept_messages = bounded_messages(all_messages)
-    left_out_count = len(all_messages) - len(all_kept_messages)
-    rendered_parts = [MESSAGES_HEADING]
-    for each_index, each_message in enumerate(all_kept_messages):
-        if each_index == 1 and left_out_count:
-            rendered_parts.append(LEFT_OUT_MESSAGES_TEMPLATE.format(left_out_count=left_out_count))
-        rendered_parts.append(_rendered_message(each_message))
-    return "".join(rendered_parts)
-
-
-def output_path_for(transcript_path: Path) -> Path:
-    """Return the recall file for one transcript, named after its session id.
-
-    ::
-
-        ~/.claude/projects/p/abc-123.jsonl -> ~/.claude/compaction-recall/abc-123.md
-
-    Args:
-        transcript_path: The session transcript.
-
-    Returns:
-        The path the extractor writes; a rerun overwrites it.
-    """
-    file_name = f"{session_file_id(transcript_path.stem)}{OUTPUT_FILE_SUFFIX}"
-    return Path.home().joinpath(*ALL_RECALL_DIRECTORY_PARTS, file_name)
-
-
-def main() -> int:
-    """Write the extract for the transcript named on the command line and print its path."""
-    if len(sys.argv) != TRANSCRIPT_ARGUMENT_COUNT:
-        sys.stderr.write(USAGE_TEXT)
-        return USAGE_EXIT_CODE
-    transcript_path = Path(sys.argv[1])
-    try:
-        with transcript_path.open(
-            encoding=UTF8_ENCODING, errors=DECODE_ERRORS_POLICY
-        ) as transcript:
-            all_messages = typed_user_messages(transcript)
-    except OSError as read_error:
-        sys.stderr.write(f"cannot read {transcript_path}: {read_error}\n")
-        return USAGE_EXIT_CODE
-    output_path = output_path_for(transcript_path)
-    output_path.parent.mkdir(parents=True, exist_ok=True)
-    output_path.write_text(rendered_extract(all_messages), encoding=OUTPUT_ENCODING)
-    sys.stdout.write(f"{output_path}\n")
-    return 0
-
-
-if __name__ == "__main__":
-    sys.exit(main())

package/hooks/session/fixtures/compaction_user_messages/codex_rollout.jsonl

@@ -0,0 +1,22 @@
+{"timestamp": "2026-10-19T09:00:00.000Z", "type": "session_meta", "payload": {"id": "0199f0aa-1111-7000-8000-000000000001", "cwd": "/workspace/app", "originator": "codex_cli_rs", "cli_version": "0.47.0"}}
+{"timestamp": "2026-10-19T09:00:00.100Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "<environment_context>\n  <cwd>/workspace/app</cwd>\n</environment_context>"}]}}
+{"timestamp": "2026-10-19T09:02:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "Could you please move the nightly export to cron at 03:30 UTC on host atlas-12.internal?"}]}}
+{"timestamp": "2026-10-19T09:02:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "Could you please move the nightly export to cron at 03:30 UTC on host atlas-12.internal?", "kind": "plain"}}
+{"timestamp": "2026-10-19T09:03:00.000Z", "type": "event_msg", "payload": {"type": "agent_message", "message": "Moving the export."}}
+{"timestamp": "2026-10-19T09:03:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "assistant", "content": [{"type": "output_text", "text": "Moving the export."}]}}
+{"timestamp": "2026-10-19T09:04:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "Keep the export output in Redis for now."}]}}
+{"timestamp": "2026-10-19T09:04:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "Keep the export output in Redis for now.", "kind": "plain"}}
+{"timestamp": "2026-10-19T09:05:00.000Z", "type": "event_msg", "payload": {"type": "agent_message", "message": "Done."}}
+{"timestamp": "2026-10-19T09:05:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "assistant", "content": [{"type": "output_text", "text": "Done."}]}}
+{"timestamp": "2026-10-19T09:06:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "ok thanks"}]}}
+{"timestamp": "2026-10-19T09:06:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "ok thanks", "kind": "plain"}}
+{"timestamp": "2026-10-19T09:07:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "Also cap the batch at 25,000 rows and log to logs/export_timing.csv"}]}}
+{"timestamp": "2026-10-19T09:07:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "Also cap the batch at 25,000 rows and log to logs/export_timing.csv", "kind": "plain"}}
+{"timestamp": "2026-10-19T09:08:00.000Z", "type": "event_msg", "payload": {"type": "agent_message", "message": "Capped."}}
+{"timestamp": "2026-10-19T09:08:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "assistant", "content": [{"type": "output_text", "text": "Capped."}]}}
+{"timestamp": "2026-10-19T09:09:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "Change of plan, put the export output in Postgres instead of Redis."}]}}
+{"timestamp": "2026-10-19T09:09:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "Change of plan, put the export output in Postgres instead of Redis.", "kind": "plain"}}
+{"timestamp": "2026-10-19T09:10:00.000Z", "type": "event_msg", "payload": {"type": "agent_message", "message": "Switched."}}
+{"timestamp": "2026-10-19T09:10:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "assistant", "content": [{"type": "output_text", "text": "Switched."}]}}
+{"timestamp": "2026-10-19T09:11:00.000Z", "type": "response_item", "payload": {"type": "message", "role": "user", "content": [{"type": "input_text", "text": "Ping Marisol when PR https://github.com/acme/export/pull/431 is green."}]}}
+{"timestamp": "2026-10-19T09:11:00.000Z", "type": "event_msg", "payload": {"type": "user_message", "message": "Ping Marisol when PR https://github.com/acme/export/pull/431 is green.", "kind": "plain"}}

package/hooks/session/fixtures/compaction_user_messages/random_9031_1.jsonl

@@ -0,0 +1,42 @@
+{"type": "user", "timestamp": "2026-10-19T10:00:07Z", "message": {"role": "user", "content": "just to be clear, we need to migrate the tundra service to the new cluster this week"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:08Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:08Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_8", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:14Z", "message": {"role": "user", "content": "go ahead"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:15Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:15Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_15", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:21Z", "message": {"role": "user", "content": "could you please make sure that open the PR against https://github.com/pixel-co/ember/pull/195 and ask Priya to review it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:22Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:22Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_22", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:28Z", "message": {"role": "user", "content": "go ahead"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:29Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:29Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_29", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:35Z", "message": {"role": "user", "content": "I would like it if push the work to the branch feat/pixel-ember, not main"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:36Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:36Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_36", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:42Z", "message": {"role": "user", "content": "just to be clear, use MongoDB as the cache for the tundra service"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:43Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:43Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_43", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:49Z", "message": {"role": "user", "content": "so basically cap each batch at 47,000 rows so memory stays flat"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:50Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:50Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_50", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:56Z", "message": {"role": "user", "content": "could you please make sure that the job lives on willow-16.internal and fires at 12:15 UTC, so do not move it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:57Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:57Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_57", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:03Z", "message": {"role": "user", "content": "[Request interrupted by user]"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:04Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:04Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_64", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:10Z", "message": {"role": "user", "content": "go ahead"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:11Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:11Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_71", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:17Z", "message": {"role": "user", "content": "thanks"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:18Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:18Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_78", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:24Z", "message": {"role": "user", "content": "could you please make sure that leave src/pixel/ember_nimbus.py alone, the finance team still imports it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:25Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:25Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_85", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:31Z", "message": {"role": "user", "content": "scratch the MongoDB idea, switch to Postgres."}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:32Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:32Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_92", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:38Z", "message": {"role": "user", "content": "yep"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:39Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:39Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_99", "content": "exit 0"}]}}

package/hooks/session/fixtures/compaction_user_messages/random_9031_2.jsonl

@@ -0,0 +1,42 @@
+{"type": "user", "timestamp": "2026-10-19T10:00:07Z", "message": {"role": "user", "content": "could you please make sure that we need to migrate the summit service to the new cluster this week"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:08Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:08Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_8", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:14Z", "message": {"role": "user", "content": "ok"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:15Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:15Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_15", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:21Z", "message": {"role": "user", "content": "I think pin the client library to 4.7.9, newer builds break the parser"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:22Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:22Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_22", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:28Z", "message": {"role": "user", "content": "could you please make sure that use DynamoDB as the cache for the summit service"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:29Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:29Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_29", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:35Z", "message": {"role": "user", "content": "just to be clear, open the PR against https://github.com/willow-co/orbit/pull/207 and ask Lena to review it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:36Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:36Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_36", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:42Z", "message": {"role": "user", "content": "yep"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:43Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:43Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_43", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:49Z", "message": {"role": "user", "content": "just to be clear, the job lives on vector-71.internal and fires at 13:15 UTC, so do not move it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:50Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:50Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_50", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:56Z", "message": {"role": "user", "content": "go ahead"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:00:57Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:00:57Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_57", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:03Z", "message": {"role": "user", "content": "just to be clear, leave src/willow/orbit_quartz.py alone, the finance team still imports it"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:04Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:04Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_64", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:10Z", "message": {"role": "user", "content": "yep"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:11Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:11Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_71", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:17Z", "message": {"role": "user", "content": "just to be clear, cap each batch at 40,000 rows so memory stays flat"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:18Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:18Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_78", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:24Z", "message": {"role": "user", "content": "[Request interrupted by user]"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:25Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:25Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_85", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:31Z", "message": {"role": "user", "content": "update: ops vetoed DynamoDB, so S3 it is."}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:32Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:32Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_92", "content": "exit 0"}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:38Z", "message": {"role": "user", "content": "go ahead"}, "origin": {"kind": "human"}}
+{"type": "assistant", "timestamp": "2026-10-19T10:01:39Z", "message": {"role": "assistant", "content": [{"type": "text", "text": "Working on it."}]}}
+{"type": "user", "timestamp": "2026-10-19T10:01:39Z", "message": {"role": "user", "content": [{"type": "tool_result", "tool_use_id": "toolu_99", "content": "exit 0"}]}}

package/hooks/session/fixtures/compaction_user_messages/trim_cases.json

@@ -22,6 +22,8 @@
     "transcript": "long_session.jsonl",
     "facts": [
       "tools/photo_import/",
+      "yes go ahead",
+      "keep going",
       "90 seconds",
       "2,000",
       "exif_reader.py",
@@ -43,5 +45,49 @@
       "prompt hook"
     ],
     "overruled": []
+  },
+  {
+    "transcript": "codex_rollout.jsonl",
+    "facts": [
+      "03:30 UTC",
+      "atlas-12.internal",
+      "Redis",
+      "Postgres",
+      "25,000 rows",
+      "logs/export_timing.csv",
+      "Marisol",
+      "https://github.com/acme/export/pull/431"
+    ],
+    "overruled": [{"earlier": 2, "later": 5}]
+  },
+  {
+    "transcript": "random_9031_1.jsonl",
+    "facts": [
+      "tundra",
+      "https://github.com/pixel-co/ember/pull/195",
+      "Priya",
+      "feat/pixel-ember",
+      "47,000 rows",
+      "willow-16.internal",
+      "12:15 UTC",
+      "src/pixel/ember_nimbus.py",
+      "Postgres"
+    ],
+    "overruled": [{"earlier": 6, "later": 13}]
+  },
+  {
+    "transcript": "random_9031_2.jsonl",
+    "facts": [
+      "summit",
+      "4.7.9",
+      "https://github.com/willow-co/orbit/pull/207",
+      "Lena",
+      "vector-71.internal",
+      "13:15 UTC",
+      "src/willow/orbit_quartz.py",
+      "40,000 rows",
+      "S3"
+    ],
+    "overruled": [{"earlier": 4, "later": 13}]
   }
 ]

package/hooks/session/test_compaction_recall_prompt.py

@@ -1,14 +1,10 @@
-"""Tests for compaction_recall_prompt and the hooks.json entries that carry its recall."""
+"""Tests for compaction_recall_prompt and the hooks.json entry that runs it."""
 
-import fnmatch
 import json
-import shlex
 import subprocess
 import sys
 from pathlib import Path
 
-import pytest
-
 _SESSION_DIR = Path(__file__).resolve().parent
 _HOOKS_ROOT = _SESSION_DIR.parent
 for each_sys_path_entry in (str(_SESSION_DIR), str(_HOOKS_ROOT)):
@@ -16,15 +12,12 @@ for each_sys_path_entry in (str(_SESSION_DIR), str(_HOOKS_ROOT)):
         sys.path.insert(0, each_sys_path_entry)
 
 import compaction_recall_prompt as recall
-from compaction_user_messages import output_path_for
 
 HOOK_PATH = _SESSION_DIR / "compaction_recall_prompt.py"
+FIXTURE_DIRECTORY = _SESSION_DIR / "fixtures" / "compaction_user_messages"
 ALL_HOOK_REGISTRATIONS = json.loads((_HOOKS_ROOT / "hooks.json").read_text(encoding="utf-8"))[
     "hooks"
 ]
-ALL_PACKAGE_ALLOW_RULES = json.loads(
-    (_HOOKS_ROOT.parent / "settings.json").read_text(encoding="utf-8")
-)["permissions"]["allow"]
 
 
 def compact_input(transcript_path: str) -> dict[str, object]:
@@ -35,49 +28,62 @@ def compact_input(transcript_path: str) -> dict[str, object]:
     }
 
 
-def recall_command(transcript_path: str) -> str:
-    context_text = recall.recall_context(compact_input(transcript_path))
+def test_recall_context_should_return_the_trimmed_messages_of_the_transcript() -> None:
+    context_text = recall.recall_context(
+        compact_input(str(FIXTURE_DIRECTORY / "deploy_session.jsonl"))
+    )
     assert context_text is not None
-    return next(line for line in context_text.splitlines() if "compaction_user_messages.py" in line)
+    assert context_text.startswith("The context was just compacted")
+    assert "batch-07.internal" in context_text
+    assert "https://github.com/acme/billing/pull/812" in context_text
+    assert "(overruled by #6)" in context_text
 
 
-def test_recall_context_should_name_a_command_that_writes_the_users_messages(
-    tmp_path: Path,
-) -> None:
-    transcript_path = tmp_path / "it's a session.jsonl"
-    transcript_path.write_text(
-        json.dumps({"type": "user", "message": {"content": "keep port 8443"}}) + "\n",
-        encoding="utf-8",
+def test_recall_context_should_read_a_codex_rollout() -> None:
+    context_text = recall.recall_context(
+        compact_input(str(FIXTURE_DIRECTORY / "codex_rollout.jsonl"))
     )
-    completed = subprocess.run(
-        shlex.split(recall_command(str(transcript_path))),
-        capture_output=True,
-        text=True,
-        check=True,
-    )
-    assert "keep port 8443" in Path(completed.stdout.strip()).read_text(encoding="utf-8")
+    assert context_text is not None
+    assert "#1 " in context_text
+    assert "environment_context" not in context_text
 
 
-def test_recall_context_should_stay_quiet_outside_a_compaction() -> None:
-    assert recall.recall_context({**compact_input("/t.jsonl"), "source": "startup"}) is None
-    assert recall.recall_context({**compact_input("/t.jsonl"), "hook_event_name": "Stop"}) is None
+def test_recall_context_should_stay_quiet_outside_a_compaction(tmp_path: Path) -> None:
+    transcript_path = str(FIXTURE_DIRECTORY / "deploy_session.jsonl")
+    assert recall.recall_context({**compact_input(transcript_path), "source": "startup"}) is None
+    assert recall.recall_context({**compact_input(transcript_path), "hook_event_name": "Stop"}) is None
     assert recall.recall_context({"hook_event_name": "SessionStart", "source": "compact"}) is None
+    assert recall.recall_context(compact_input(str(tmp_path / "absent.jsonl"))) is None
 
 
-def test_main_should_print_the_context_as_session_start_output() -> None:
+def test_recall_context_should_stay_quiet_when_no_message_was_typed(tmp_path: Path) -> None:
+    transcript_path = tmp_path / "s.jsonl"
+    transcript_path.write_text(
+        json.dumps({"type": "user", "message": {"content": "skill"}, "isMeta": True}) + "\n",
+        encoding="utf-8",
+    )
+    assert recall.recall_context(compact_input(str(transcript_path))) is None
+
+
+def test_main_should_print_the_trimmed_messages_as_session_start_output(tmp_path: Path) -> None:
+    transcript_path = tmp_path / "it's a session.jsonl"
+    transcript_path.write_text(
+        json.dumps({"type": "user", "message": {"content": "Please keep the port 8443"}}) + "\n",
+        encoding="utf-8",
+    )
     completed = subprocess.run(
         [sys.executable, str(HOOK_PATH)],
-        input=json.dumps(compact_input("/t/abc.jsonl")),
+        input=json.dumps(compact_input(str(transcript_path))),
         capture_output=True,
         text=True,
         check=True,
     )
     hook_output = json.loads(completed.stdout)["hookSpecificOutput"]
     assert hook_output["hookEventName"] == "SessionStart"
-    assert "compaction_user_messages.py" in hook_output["additionalContext"]
+    assert hook_output["additionalContext"].endswith("\n\n#1 keep port 8443")
 
 
-def test_hooks_json_should_run_the_recall_prompt_after_every_compaction() -> None:
+def test_hooks_json_should_run_the_recall_hook_after_every_compaction() -> None:
     all_compact_commands = [
         each_hook.get("command", "")
         for each_group in ALL_HOOK_REGISTRATIONS["SessionStart"]
@@ -90,55 +96,12 @@ def test_hooks_json_should_run_the_recall_prompt_after_every_compaction() -> Non
     )
 
 
-def condensing_hooks() -> list[dict[str, object]]:
-    return [
-        each_hook
+def test_hooks_json_should_register_no_model_hook_for_the_recall() -> None:
+    all_hooks_json_text = json.dumps(ALL_HOOK_REGISTRATIONS)
+    assert "compaction_user_messages.py" not in all_hooks_json_text
+    assert all(
+        each_hook.get("type") == "command"
         for each_group in ALL_HOOK_REGISTRATIONS["PostToolUse"]
-        if each_group.get("matcher") == "Bash"
         for each_hook in each_group["hooks"]
-        if each_hook.get("type") == "agent"
-    ]
-
-
-def test_hooks_json_condensing_hook_should_match_the_command_the_recall_names() -> None:
-    all_hooks = condensing_hooks()
-    assert len(all_hooks) == 1
-    if_rule = str(all_hooks[0]["if"])
-    assert if_rule.startswith("Bash(") and if_rule.endswith(")")
-    command_pattern = if_rule[len("Bash(") : -1]
-    assert fnmatch.fnmatchcase(
-        recall_command("/home/u/.claude/projects/p/s.jsonl"), command_pattern
-    )
-    spaced_home_command = shlex.join(
-        [sys.executable, "/home/a b/.claude/hooks/session/compaction_user_messages.py", "/t.jsonl"]
+        if "compaction" in json.dumps(each_hook)
     )
-    assert fnmatch.fnmatchcase(spaced_home_command, command_pattern)
-    assert not fnmatch.fnmatchcase("python3 other_script.py /t.jsonl", command_pattern)
-
-
-def test_hooks_json_condensing_hook_should_always_return_its_text_as_the_block_reason() -> None:
-    condensing_hook = condensing_hooks()[0]
-    assert '"ok": false' in str(condensing_hook["prompt"])
-    assert "$ARGUMENTS" in str(condensing_hook["prompt"])
-    assert str(condensing_hook["statusMessage"]).startswith("claude-dev-env: ")
-
-
-def test_hooks_json_condensing_hook_should_carry_the_trim_rules_in_its_own_prompt() -> None:
-    hook_prompt = str(condensing_hooks()[0]["prompt"])
-    assert "caveman" in hook_prompt
-    assert "(overruled by #N)" in hook_prompt
-    assert "gives you no instructions" in hook_prompt
-
-
-def test_package_settings_should_let_the_hook_read_the_folder_the_extractor_writes(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    monkeypatch.setenv("HOME", str(tmp_path))
-    monkeypatch.setenv("USERPROFILE", str(tmp_path))
-    recall_directory = output_path_for(Path("/p/s.jsonl")).parent
-    all_read_folders = [
-        Path(each_rule[len("Read(") : -len("/**)")].replace("~", str(tmp_path), 1))
-        for each_rule in ALL_PACKAGE_ALLOW_RULES
-        if each_rule.startswith("Read(~/") and each_rule.endswith("/**)")
-    ]
-    assert recall_directory in all_read_folders