Skip to content

feat(ios): route plugin.json scripts through segkit#60

Merged
abueide merged 1 commit intomainfrom
feat/segkit-ios-plugin
May 1, 2026
Merged

feat(ios): route plugin.json scripts through segkit#60
abueide merged 1 commit intomainfrom
feat/segkit-ios-plugin

Conversation

@abueide
Copy link
Copy Markdown
Contributor

@abueide abueide commented May 1, 2026

Summary

Updates iOS plugin convenience scripts to call segkit ios instead of ios.sh directly. No fallbacks — if segkit isn't on PATH, the command fails immediately.

Changes

  • Replace ios.sh with segkit ios in 4 plugin.json scripts: ios:devices:eval, start:ios, start:sim, stop:sim
  • start:sim / stop:sim simplified from inline multi-source bash one-liners to clean segkit ios simulator start/stop calls

Why

Phase 1.3 of the shell-to-Rust migration. Same pattern as PR #58 (Android). Routes user-facing plugin commands through segkit's delegation layer for timing/error observability.

CI E2E tests are unaffected — they use process-compose which calls ios.sh directly.


🤖 Generated with Claude Code

Replace direct ios.sh calls and inline script sourcing in the iOS
plugin's convenience scripts with segkit ios delegation. No fallbacks.

Scripts updated: ios:devices:eval, start:ios, start:sim, stop:sim.
start:sim and stop:sim are simplified from multi-source bash one-liners
to clean segkit ios simulator start/stop calls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@abueide abueide added the feature New feature or functionality label May 1, 2026
@abueide abueide marked this pull request as ready for review May 1, 2026 23:48
@abueide abueide merged commit 2d58c24 into main May 1, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant