Repository navigation
chore: remove files nothing references - #2055
Conversation
Demo media, stale benchmarks and parity scripts, game scripts, and test and config files for systems that no longer exist. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Repository rules and review records .npmignore, FLOW_NOTES.md, REVIEW_VERDICT.json, tests/integration/broker/evals-reports/eval-master-summary.html |
The npm exclusion file, node-claim notes, review verdict, and broker evaluation report were deleted. |
Demo and game assets scripts/demos/*, scripts/games/*, scripts/tictactoe-setup.sh |
Negotiation demos, a debate recording script, and game protocol and setup materials were deleted. |
Validation and terminal scripts scripts/run-relay-validation-suite.sh, scripts/test-interactive-terminal.sh |
The relay validation-suite and interactive-terminal environment scripts were deleted. |
Benchmark suite tests/benchmarks/*, .prettierignore |
The benchmark harness and benchmark programs were deleted. The latency benchmark was removed from the Prettier ignore list. |
Integration and parity tests tests/mcp_merge_e2e.rs, tests/parity/*, vitest.connect.config.mjs |
The MCP merge end-to-end test, parity tests, and connect-package Vitest configuration were deleted. |
Estimated code review effort: 3 (Moderate) | ~20 minutes
Change: Other
Possibly related PRs
- AgentWorkforce/relay#2011: Changes MCP launch resolution and MCP configuration updates; this PR removes an end-to-end test that covered related configuration behavior.
Suggested reviewers: willwashburn, miyaontherelay, agentrelaybot
Merge Risk: 
1b68e
-
No specific impact from the removed files is established; merge after normal checks.
Pre-merge checks: 
-
Check name Status Explanation Resolution Description check The description explains the cleanup scope and rationale, but it does not use the required template sections. It omits the Test Plan, RelayFlow Proof, and Screenshots sections, including the required … Add the required Summary, Test Plan, RelayFlow Proof, and Screenshots sections. Complete the Test Plan checkboxes and set RelayFlow Proof to Change type: non-functional and RelayFlow case: n/a if runtime behavior is unchanged. -
Check name Status Explanation Title check The title clearly summarizes the main change: removing unreferenced files. It is concise and relevant to the changeset. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.
-
Full details: Description check
-
Explanation
The description explains the cleanup scope and rationale, but it does not use the required template sections. It omits the Test Plan, RelayFlow Proof, and Screenshots sections, including the required Change type and RelayFlow case values.
Resolution
Add the required Summary, Test Plan, RelayFlow Proof, and Screenshots sections. Complete the Test Plan checkboxes and set RelayFlow Proof to Change type: non-functional and RelayFlow case: n/a if runtime behavior is unchanged.
-
Finishing Touches
-
Generate docstrings
-
- Commit to this branch
- Create a new PR
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
A rabbit checks the emptied tray,
Old test scripts hop out of sight.
Demo cards and notes depart,
Benchmarks fold their measuring charts.
The bunny stamps the cleanup page,
Then bounds away from the repo stage.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
 [Low impact] This cleanup appears safe to merge.
|
First cleanup pass: 30 tracked files, about 4.3 MB. Each path was checked by searching
origin/main, and nothing else references it. The one match outside these files was a stale.prettierignoreline, removed here.Removed
scripts/demos/: demo video, GIF and tape, 4.1 MB.tests/benchmarks/,tests/parity/: no callers.tests/mcp_merge_e2e.rs: outside every crate, so it never compiles.scripts/games/,scripts/tictactoe-setup.sh.scripts/test-interactive-terminal.sh,scripts/run-relay-validation-suite.sh: both need files that are gone.tests/integration/broker/evals-reports/eval-master-summary.html: generated report.FLOW_NOTES.md,REVIEW_VERDICT.json: leftover run notes..npmignore(the root package is private) andvitest.connect.config.mjs.Removing files doesn't shrink git history, so clone size won't change.
No changelog entry: nothing here ships to users.
🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.Agent Relay sessions
claudesessiona994d8be· opened viagh pr create· last active 2026-11-02