feat: Improve pause point observability with cleared reasons, collection previews, and raw capture

[hatayama]

Refs: 2026-07-12_uloop検証フィードバック改善 実装計画.md#Group 2

Summary

• Cleared pause points now report why they were cleared instead of only showing Cleared.
• Captured collection variables preview as shallow JSON in pause-point status responses.
• While Unity is paused on a hit, execute-dynamic-code can read live captured references via UloopPausePoint.TryGetCapturedValue.
• Pause-point and execute-dynamic-code skills document uloop-specific contracts (pre-line snapshots, hit preconditions, marker types, raw capture APIs).

Child PRs

• fix: Report why pause points were cleared instead of only Cleared #1715 PR 2-1: ClearedReason, StatusBeforeClear, LateHitDiscardedAfterClear; run-tests auto-clear is observable
• feat: Preview captured collections as JSON in pause point status #1716 PR 2-2: ICollection/IDictionary JSON preview in CapturedVariables.Value (depth 2, capped)
• feat: Expose raw captured pause point variables while Unity is paused #1717 PR 2-3: Latest-hit raw capture holder + TryGetCapturedValue / GetCapturedNames / GetCapturedPausePointId
• docs: Document pause point contracts and raw capture APIs in skills #1718 PR 2-4: Skill docs for C-1 through C-4 and raw capture cross-links

User Impact

• Agents can tell whether a marker was cleared by run-tests, expiry, or explicit clear instead of guessing from Cleared alone.
• List<T> and dictionary locals are readable from CapturedVariables without execute-dynamic-code for common collection types.
• During a paused hit, agents can inspect live collection/object references, not only string previews.
• Skill text now states uloop-specific pause-point contracts so agents do not confuse input delivery with logic reach or pre-line snapshots with post-interrupt state.

Real-device evidence (summary)

• Collection JSON preview (feat: Preview captured collections as JSON in pause point status #1716): enable-pause-point on CollectionPreviewPausePointFixture.cs:11 → CapturedVariables for scores shows Value: "[10,20,30]" with CapturedVariablesTruncated: false.
• Raw capture while paused (feat: Expose raw captured pause point variables while Unity is paused #1717): same fixture hit → TryGetCapturedValue("scores") returns Found=true with list contents 10,20,30; GetCapturedPausePointId() matches the enabled id.
• Resume clears raw refs (feat: Expose raw captured pause point variables while Unity is paused #1717): after control-play-mode --action Play, TryGetCapturedValue("scores").Found is false.
• Step retains holder (feat: Expose raw captured pause point variables while Unity is paused #1717): with PlayMode paused, control-play-mode --action Step keeps TryGetCapturedValue readable (held-after-step).

Verification

• uloop compile 0 errors / 0 warnings across child PRs
• EditMode: PausePointTests, SourcePausePointVariableFormatterTests, shared contract tests green
• Fable 5 final review LGTM on all child PRs (fix: Report why pause points were cleared instead of only Cleared #1715, feat: Preview captured collections as JSON in pause point status #1716, feat: Expose raw captured pause point variables while Unity is paused #1717, docs: Document pause point contracts and raw capture APIs in skills #1718)

Protocol

• No protocol version bump (additive response fields and skill/docs only)

Made with Cursor

[coderabbitai]

Warning

Review limit reached

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

Next review available in: 47 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: cba9f344-db37-41f4-ae91-f0320f5abf59

📥 Commits

Reviewing files that changed from the base of the PR and between 8845709 and 1729287.

⛔ Files ignored due to path filters (9)

• Assets/Tests/Editor/CollectionPreviewPausePointFixture.cs.meta is excluded by none and included by none
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointCollectionPreviewSerializer.cs.meta is excluded by none and included by none
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointVariableCollector.cs.meta is excluded by none and included by none
• Packages/src/Editor/FirstPartyTools/PausePoint/UnityCLILoop.FirstPartyTools.PausePoint.Editor.asmdef is excluded by none and included by none
• Packages/src/Runtime/PausePoints/UloopPausePointCapturedVariableEntry.cs.meta is excluded by none and included by none
• Packages/src/Runtime/PausePoints/UloopPausePointCapturedVariableFrame.cs.meta is excluded by none and included by none
• Packages/src/Runtime/PausePoints/UloopPausePointClearedReason.cs.meta is excluded by none and included by none
• Packages/src/Runtime/PausePoints/UloopPausePointRawCaptureHolder.cs.meta is excluded by none and included by none
• Packages/src/Runtime/PausePoints/UloopPausePointRawCaptureLifecycle.cs.meta is excluded by none and included by none

📒 Files selected for processing (29)

• .agents/skills/uloop-execute-dynamic-code/SKILL.md
• .agents/skills/uloop-pause-point/SKILL.md
• .claude/skills/uloop-execute-dynamic-code/SKILL.md
• .claude/skills/uloop-pause-point/SKILL.md
• Assets/Tests/Editor/CollectionPreviewPausePointFixture.cs
• Assets/Tests/Editor/PausePointStatusResponseContractTests.cs
• Assets/Tests/Editor/PausePointTests.cs
• Assets/Tests/Editor/SourcePausePointCapture/SourcePausePointVariableFormatterTests.cs
• Packages/src/Editor/CliOnlyTools~/PausePoint/Skill/SKILL.md
• Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/Skill/SKILL.md
• Packages/src/Editor/FirstPartyTools/PausePoint/PausePointTools.cs
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointCapture.cs
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointCollectionPreviewSerializer.cs
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointConstants.cs
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointVariableCollector.cs
• Packages/src/Editor/FirstPartyTools/PausePoint/SourcePausePointVariableFormatter.cs
• Packages/src/Editor/FirstPartyTools/RunTests/RunTestsUseCase.cs
• Packages/src/Editor/Infrastructure/Api/PausePointStatusBridgeCommand.cs
• Packages/src/Runtime/PausePoints/UloopPausePoint.cs
• Packages/src/Runtime/PausePoints/UloopPausePointCapturedVariableEntry.cs
• Packages/src/Runtime/PausePoints/UloopPausePointCapturedVariableFrame.cs
• Packages/src/Runtime/PausePoints/UloopPausePointClearedReason.cs
• Packages/src/Runtime/PausePoints/UloopPausePointEntry.cs
• Packages/src/Runtime/PausePoints/UloopPausePointRawCaptureHolder.cs
• Packages/src/Runtime/PausePoints/UloopPausePointRawCaptureLifecycle.cs
• Packages/src/Runtime/PausePoints/UloopPausePointRegistry.cs
• Packages/src/Runtime/PausePoints/UloopPausePointSnapshot.cs
• cli/project-runner/internal/projectrunner/pause_point_wait.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-observability

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}