Skip to content

chore: remove internal scratchpad / handoff docs from repo#1603

Merged
f-trycua merged 1 commit into
mainfrom
chore/remove-leaked-internal-docs
May 20, 2026
Merged

chore: remove internal scratchpad / handoff docs from repo#1603
f-trycua merged 1 commit into
mainfrom
chore/remove-leaked-internal-docs

Conversation

@f-trycua
Copy link
Copy Markdown
Collaborator

@f-trycua f-trycua commented May 20, 2026

Summary

Removes 6 .md / .sh / .py files that were committed alongside other PRs but are internal coordination artifacts — status reports, dev-loop scratchpads, design plans, quick-test scripts — rather than user-facing documentation.

Removed

Safety

Verified via repo-wide grep that no tracked code, docs, or CI references these paths. Their content belongs in PR descriptions or linked internal docs, not as committed scratchpads.

Test plan

  • git grep for each filename returns no hits
  • CI green (no consumer to break)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Removed Windows Session-0 status report.
    • Removed Modal-S3 sync workflow documentation.
    • Removed training loop architecture documentation.
    • Removed Windows VM development guide.
  • Chores

    • Removed MCP server testing utilities and scripts.

Review Change Stack

These files were committed alongside other changes but are internal
coordination artifacts (status reports, dev-loop scratchpads, design
plans, quick-test scripts) rather than user-facing documentation. They
add noise to the repo tree, suggest internal process leakage to people
browsing the codebase, and have no callers in any tracked build /
test / docs flow.

Removed:
- WINDOWS_SESSION0_STATUS.md                         (status report)
- libs/cua-driver-rs/DEV_LOOP_WINDOWS_VM.md          (dev-loop scratchpad)
- docs/scripts/HANDOFF-modal-s3-sync.md              (deploy handoff)
- libs/cua-bench/plan.md                             (Tinker RL design plan)
- libs/python/mcp-server/QUICK_TEST_COMMANDS.sh      (ad-hoc test commands)
- libs/python/mcp-server/quick_test_local_option.py  (ad-hoc test script)

No code references any of these files (verified via repo-wide grep).
The signal/intent that lived in them belongs in PR descriptions or
linked internal docs, not as committed scratchpads in the repo root.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 20, 2026 12:55pm

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

📦 Publishable packages changed

  • pypi/bench
  • pypi/mcp-server

Add release:<service> labels to auto-release on merge (+ optional bump:minor or bump:major, default is patch).
Or add no-release to skip.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: dd2a9a88-1237-4a38-8889-1e39c7fdda2b

📥 Commits

Reviewing files that changed from the base of the PR and between d4f219f and a67831b.

📒 Files selected for processing (6)
  • WINDOWS_SESSION0_STATUS.md
  • docs/scripts/HANDOFF-modal-s3-sync.md
  • libs/cua-bench/plan.md
  • libs/cua-driver-rs/DEV_LOOP_WINDOWS_VM.md
  • libs/python/mcp-server/QUICK_TEST_COMMANDS.sh
  • libs/python/mcp-server/quick_test_local_option.py
💤 Files with no reviewable changes (6)
  • WINDOWS_SESSION0_STATUS.md
  • docs/scripts/HANDOFF-modal-s3-sync.md
  • libs/cua-driver-rs/DEV_LOOP_WINDOWS_VM.md
  • libs/cua-bench/plan.md
  • libs/python/mcp-server/QUICK_TEST_COMMANDS.sh
  • libs/python/mcp-server/quick_test_local_option.py

📝 Walkthrough

Walkthrough

This PR removes six documentation and script files across the repository: Windows Session-0 stabilization status, Modal-to-S3 handoff guidance, RL training architecture planning, Windows VM development reference, and MCP server quick-test infrastructure. The removal totals 1,158 lines of development-time documentation and testing utilities.

Changes

Documentation and Script Cleanup

Layer / File(s) Summary
MCP server quick-test script removal
libs/python/mcp-server/QUICK_TEST_COMMANDS.sh, libs/python/mcp-server/quick_test_local_option.py
The QUICK_TEST_COMMANDS.sh bash script and quick_test_local_option.py helper module, which provided local testing entry points for CUA_USE_HOST_COMPUTER_SERVER mode validation, are removed.
Windows and deployment handoff documentation
WINDOWS_SESSION0_STATUS.md, docs/scripts/HANDOFF-modal-s3-sync.md, libs/cua-driver-rs/DEV_LOOP_WINDOWS_VM.md
Windows Session-0 stabilization status, Modal-to-S3 sync workflow documentation, and Windows VM development iteration guidance are deleted.
RL training architecture planning
libs/cua-bench/plan.md
The documented RL training loop architecture for Tinker PPO integration with cua-bench trace collection and checkpoint management is removed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • trycua/cua#1548: This PR removes Session-0 stabilization and Windows VM development documentation (WINDOWS_SESSION0_STATUS.md and DEV_LOOP_WINDOWS_VM.md) that accompanied the Windows hardening work introduced in that PR.

Poem

🐰 Away with the drafts and the helper scripts go,
Old handoff notes fade as the seasons blow,
The quick-tests are gone, the plans filed away,
Cleaner directories greet the new day! 📝✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing internal documentation and scratchpad files that were accidentally committed. It is concise, specific, and directly related to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-leaked-internal-docs

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sentry
Copy link
Copy Markdown

sentry Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@f-trycua f-trycua merged commit f83c43c into main May 20, 2026
16 of 17 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.

1 participant