Skip to content

sync: port upstream isolated app testing workflow and mobile skills (#4121, #4162, #4165) - #173

Merged
cursor[bot] merged 4 commits into
mainfrom
codex/sync-20260720-docs-skills-daf2
Jul 20, 2026
Merged

sync: port upstream isolated app testing workflow and mobile skills (#4121, #4162, #4165)#173
cursor[bot] merged 4 commits into
mainfrom
codex/sync-20260720-docs-skills-daf2

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Jul 20, 2026 •
edited by cursor Bot
Loading

Copy link
Copy Markdown
Owner

Summary

Ports upstream isolated app testing workflow and mobile testing skills into the fork. Base is main (restacked after mobile reland #178).

Upstream ports

  • Isolated app testing workflow / test-t3-app skill (#4121)
  • Make test-t3-app discoverable by Claude Code (#4162)
  • Portable mobile app testing guidance (#4165)
  • Portable Claude pre-push hook for cloud agents (fork adaptation)

Fork adaptations

  • Keep fork GitHub / protected-feature / vp check rules in AGENTS.md
  • Portable .claude/settings.json hook paths for cloud agents

Test plan

  • Check / Test green
  • Base is main (do not merge if retargeted away)
  • Merge with admin after required checks
Open in Web Open in Cursor 

@cursor
cursor Bot force-pushed the codex/sync-20260720-mobile-daf2 branch from bcf0dea to cef796c Compare July 20, 2026 00:50
@cursor
cursor Bot force-pushed the codex/sync-20260720-docs-skills-daf2 branch from 240a511 to 6300fc4 Compare July 20, 2026 00:50
@coderabbitai

coderabbitai Bot commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown

Warning

Review limit reached

@cursor[bot], you've reached your PR review limit, so we couldn't start this review.

Next review available in: 20 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: defaults

Review profile: CHILL

Plan: Pro

Run ID: 77fb084e-4e9f-4f26-9154-9ee0f5d97c68

📥 Commits

Reviewing files that changed from the base of the PR and between 8570897 and 65e620a.

📒 Files selected for processing (28)
  • .agents/skills/ios-debugger-agent/LICENSE
  • .agents/skills/ios-debugger-agent/SKILL.md
  • .agents/skills/ios-debugger-agent/agents/openai.yaml
  • .agents/skills/ios-simulator-browser/LICENSE
  • .agents/skills/ios-simulator-browser/SKILL.md
  • .agents/skills/ios-simulator-browser/agents/openai.yaml
  • .agents/skills/test-t3-app/SKILL.md
  • .agents/skills/test-t3-app/agents/openai.yaml
  • .agents/skills/test-t3-app/references/sqlite-fixtures.md
  • .agents/skills/test-t3-mobile/SKILL.md
  • .agents/skills/test-t3-mobile/agents/openai.yaml
  • .claude/settings.json
  • .claude/skills
  • .codex/config.toml
  • .mcp.json
  • AGENTS.md
  • apps/desktop/src/app/DesktopEnvironment.test.ts
  • apps/desktop/src/app/DesktopEnvironment.ts
  • apps/mobile/src/features/connection/ConnectionSheetButton.tsx
  • apps/server/scripts/t3-sqlite-state.test.ts
  • apps/server/scripts/t3-sqlite-state.ts
  • apps/server/src/cli/config.test.ts
  • apps/server/src/cli/config.ts
  • apps/server/src/config.ts
  • docs/operations/observability.md
  • docs/reference/scripts.md
  • scripts/dev-runner.test.ts
  • scripts/dev-runner.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/sync-20260720-docs-skills-daf2

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.

@github-actions github-actions Bot added the size:XL 500-999 effective changed lines (test files excluded in mixed PRs). label Jul 20, 2026
@cursor
cursor Bot force-pushed the codex/sync-20260720-mobile-daf2 branch from cef796c to 59ce057 Compare July 20, 2026 01:05
@cursor
cursor Bot force-pushed the codex/sync-20260720-docs-skills-daf2 branch from 485b3bd to 6e93da7 Compare July 20, 2026 01:05
@cursor
cursor Bot force-pushed the codex/sync-20260720-mobile-daf2 branch from 59ce057 to d81b8a7 Compare July 20, 2026 01:17
@cursor
cursor Bot force-pushed the codex/sync-20260720-docs-skills-daf2 branch from 6e93da7 to c1f9d78 Compare July 20, 2026 01:17
@aaditagrawal aaditagrawal mentioned this pull request Jul 20, 2026
Merged
@aaditagrawal Graphite App

aaditagrawal commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@aaditagrawal aaditagrawal mentioned this pull request Jul 20, 2026
Merged
@cursor
cursor Bot deleted the branch main July 20, 2026 01:23
@cursor
cursor Bot changed the base branch from codex/sync-20260720-mobile-daf2 to codex/sync-20260720-mobile-reland-daf2 July 20, 2026 01:24
@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Jul 20, 2026
@cursor
cursor Bot deleted the branch main July 20, 2026 01:30
@cursor
cursor Bot changed the base branch from codex/sync-20260720-mobile-reland-daf2 to main July 20, 2026 01:30
juliusmarminge and others added 4 commits July 20, 2026 01:31
@juliusmarminge @codex @cursoragent
Add isolated app testing workflow (pingdotgg#4121)
3e8dad0 
Co-authored-by: codex <codex@users.noreply.github.com>
(cherry picked from commit 4f11740)
@juliusmarminge @claude @cursoragent
Make test-t3-app skill discoverable by Claude Code (pingdotgg#4162)
c2f4504 
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
(cherry picked from commit 69c8be2)
@juliusmarminge @codex @cursoragent
Add portable mobile app testing guidance (pingdotgg#4165)
7af2a0e 
Co-authored-by: codex <codex@users.noreply.github.com>
(cherry picked from commit e34250d)
@cursoragent @aaditagrawal
fix(claude): make pre-push hook portable for cloud agents
65e620a 
Replace the hardcoded local Mac path and bun-only gate with a
repo-root typecheck via vp so agents can push from this fork.

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
@cursor
cursor Bot force-pushed the codex/sync-20260720-docs-skills-daf2 branch from 721efd4 to 65e620a Compare July 20, 2026 01:31
@cursor
cursor Bot merged commit bb52b31 into main Jul 20, 2026
12 checks passed
@cursor
cursor Bot deleted the codex/sync-20260720-docs-skills-daf2 branch July 20, 2026 01:37
@aaditagrawal aaditagrawal mentioned this pull request Jul 20, 2026
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size:XL 500-999 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aaditagrawal @juliusmarminge @cursoragent