Skip to content

Remove the replay command#27

Merged
dacorvo merged 2 commits into
mainfrom
interactive_replay
Jun 11, 2026
Merged

Remove the replay command#27
dacorvo merged 2 commits into
mainfrom
interactive_replay

Conversation

@dacorvo

@dacorvo dacorvo commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Removes agentcap replay — a thin wrapper over httpx.post plus a response renderer that went unused outside the demo. Serious replay happens in purpose-built harnesses that issue their own requests against the exported captures, which a one-shot CLI can't compose into.

Removed

  • The replay command, its tests, docs/replay.md, the demo tape, the 4 MB demo gif, and the SSE/JSON response renderers.

Kept

  • The capture-loading primitive, now agentcap.captures (load_request / resolve_workspace_rid) — inspect depends on it and scripts can import it directly.
  • inspect --rid as a generic "print the selected request-id" affordance.

Commits

  1. feat: remove the replay command
  2. refactor: rename replay module to captures

Note: the README diff also carries a python -m venv setup step that was already in the working tree, unrelated to this change.

🤖 Generated with Claude Code

dacorvo and others added 2 commits June 11, 2026 13:32
`replay` re-issued a captured body to an OpenAI-compatible endpoint and rendered the result — thin glue over httpx that went unused outside the demo. Serious replay belongs in a purpose-built harness (e.g. kv-reuse-lab) that issues its own requests against the exported captures, which a one-shot CLI can't compose into. Removed the command, its tests, docs/replay.md, the demo tape + gif, and the SSE/JSON renderers. Kept the loader primitive (`agentcap.replay.load_request` / `resolve_workspace_rid`) — inspect depends on it and scripts can import it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
With the replay command gone, the module is purely the captures loader (load_request / load_requests / resolve_workspace_rid). Rename replay.py -> captures.py (and test_replay.py -> test_captures.py) so the name reflects what it does; inspect's importer and the module docstring updated to match.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 11, 2026 11:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the agentcap replay CLI command and its supporting rendering logic/docs/tests, while preserving and renaming the underlying “load captured request by id” primitive as agentcap.captures for continued use by inspect and by downstream scripts.

Changes:

  • Deleted the replay Click command implementation and the SSE/JSON response renderers from the CLI entrypoint.
  • Renamed the reusable replay-loader module to agentcap.captures and updated tests/imports accordingly.
  • Removed replay-related documentation and demo artifacts, and updated README/docs to no longer reference agentcap replay.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/test_cli.py Removes CLI tests that were specific to the deleted replay command.
tests/test_captures.py Updates imports/docs to validate the loader API under agentcap.captures.
src/agentcap/captures.py Keeps the request-loading primitive with updated module-level framing (no longer “replay”).
src/agentcap/main.py Removes the replay command and its renderers; updates internal references to use captures.
README.md Removes replay from the CLI/docs narrative and quick-start usage examples.
docs/replay.md Deletes replay command documentation (command removed).
docs/inspect.md Removes replay-specific piping example; keeps --rid described generically.
docs/demo/replay.tape Deletes the replay demo tape (command removed).
docs/capture.md Removes replay from the “see also” links.
AGENTS.md Removes the replay-related design decision entry and renumbers subsequent items.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dacorvo dacorvo merged commit 1ba1d94 into main Jun 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants