Skip to content

Reorganize runtime targets for scalable plugin support#1577

Merged
ajemory merged 4 commits into
apple:mainfrom
ajemory:organize-runtime
May 20, 2026
Merged

Reorganize runtime targets for scalable plugin support#1577
ajemory merged 4 commits into
apple:mainfrom
ajemory:organize-runtime

Conversation

@ajemory
Copy link
Copy Markdown
Contributor

@ajemory ajemory commented May 20, 2026

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Motivation and Context

Closes #1569

Separates generic runtime infrastructure from Linux-specific implementation into distinct SPM targets:

  • ContainerRuntimeClient — generic runtime plugin client API
  • ContainerRuntimeLinuxClient — Linux-specific types
  • ContainerRuntimeLinuxServer — Linux runtime service + interface strategy implementations

Renames types to reflect the runtime/sandbox distinction:

  • SandboxClient → RuntimeClient
  • SandboxService → RuntimeService
  • SandboxRoutes → RuntimeRoutes
  • SandboxKeys → RuntimeKeys

Breaking for external consumers:

  • ContainerSandboxService removed, replaced by ContainerRuntimeClient + ContainerRuntimeLinuxServer
  • Public types renamed

Testing

  • Tested locally
  • Added/updated tests
  • Added/updated docs

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

Code Coverage

Tier Line Coverage
Unit 33.63%
Integration 19.95%
Combined 53.01%

Copy link
Copy Markdown
Contributor

@katiewasnothere katiewasnothere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the merge conflicts, this looks good to me

@ajemory ajemory force-pushed the organize-runtime branch from 0cd8d3a to 93a432a Compare May 20, 2026 18:58
@ajemory ajemory merged commit afeceb0 into apple:main May 20, 2026
3 checks passed
@ajemory ajemory deleted the organize-runtime branch May 20, 2026 19:57
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.

[Request]: LinuxRuntimeData is not accessible by callers of the API client

2 participants