Skip to content

feat: add pause point capture modes and history#1729

Merged
hatayama merged 3 commits into
v3-betafrom
feat/pause-point-capture-modes
Jul 12, 2026
Merged

feat: add pause point capture modes and history#1729
hatayama merged 3 commits into
v3-betafrom
feat/pause-point-capture-modes

Conversation

@hatayama

@hatayama hatayama commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

This Phase 1 base branch adds continuous and trace capture modes plus bounded pause point hit history.

  • Add single-shot, continuous, and trace capture modes while preserving single-shot as the default.
  • Retain bounded captured-variable history with hit sequence, frame count, timestamp, truncation, and dropped-frame metadata.
  • Expose mode/history through enable-pause-point and pause-point-status, including validation and shared C#/Go response contracts.
  • Add await-pause-point contract coverage, CLI schema guidance, and the source pause-point skill documentation with regenerated copies.

Validation

  • Unity compile: 0 errors, 0 warnings.
  • Targeted Unity tests: 58/58 passed.
  • scripts/check-go-cli.sh: passed.
  • Continuous E2E: an Update line in SimulateMouseDemoScene was hit, then three Editor Steps grew history from four consecutive frames.
  • Trace E2E: playback continued without pause; 372 hits were recorded with a five-frame history and 367 dropped frames.
  • Single-shot E2E: the first hit paused Unity, recorded one history frame, and auto-disarmed.
  • The checkout has no dedicated jumping-ball scene, so the equivalent Update-path behavior was verified in SimulateMouseDemoScene using dist/darwin-arm64/uloop directly; no PATH-resolved uloop was used.

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@hatayama, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 18 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ab5f9cea-477f-4f89-b5cb-af4be04ad502

📥 Commits

Reviewing files that changed from the base of the PR and between 7318b9d and 7b29d36.

⛔ Files ignored due to path filters (4)
  • Assets/Tests/Editor/PausePointCaptureModeTests.cs.meta is excluded by none and included by none
  • Assets/Tests/Editor/PausePointToolModeTests.cs.meta is excluded by none and included by none
  • Packages/src/Runtime/PausePoints/UloopPausePointCaptureMode.cs.meta is excluded by none and included by none
  • Packages/src/Runtime/PausePoints/UloopPausePointCapturedHistoryFrame.cs.meta is excluded by none and included by none
📒 Files selected for processing (19)
  • .agents/skills/uloop-pause-point/SKILL.md
  • .claude/skills/uloop-pause-point/SKILL.md
  • Assets/Tests/Editor/PausePointCaptureModeTests.cs
  • Assets/Tests/Editor/PausePointStatusResponseContractTests.cs
  • Assets/Tests/Editor/PausePointToolModeTests.cs
  • Assets/Tests/Editor/SourcePausePointCapture/SourcePausePointCaptureTests.cs
  • Packages/src/Editor/CliOnlyTools~/PausePoint/Skill/SKILL.md
  • Packages/src/Editor/FirstPartyTools/PausePoint/PausePointTools.cs
  • Packages/src/Editor/Infrastructure/Api/PausePointStatusBridgeCommand.cs
  • Packages/src/Runtime/PausePoints/UloopPausePointCaptureMode.cs
  • Packages/src/Runtime/PausePoints/UloopPausePointCapturedHistoryFrame.cs
  • Packages/src/Runtime/PausePoints/UloopPausePointEntry.cs
  • Packages/src/Runtime/PausePoints/UloopPausePointRegistry.cs
  • Packages/src/Runtime/PausePoints/UloopPausePointSnapshot.cs
  • cli/common/tools/default-tools.json
  • cli/project-runner/internal/projectrunner/pause_point_types.go
  • cli/project-runner/internal/projectrunner/pause_point_wait.go
  • cli/project-runner/internal/projectrunner/pause_point_wait_test.go
  • tests/contracts/pause_point_status_response_contract.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/pause-point-capture-modes

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.

@hatayama hatayama merged commit 0b5e72e into v3-beta Jul 12, 2026
10 checks passed
@hatayama hatayama deleted the feat/pause-point-capture-modes branch July 12, 2026 09:39
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