Skip to content

Add network middleware types (NetworkMiddlewareConfig, SupervisorMiddlewareService) #36

Closed

Description

@rhuss

Proto Gap: Network Middleware Support

New proto messages for supervisor middleware pipelines and network middleware configuration. Currently marked as skipped in the coverage test with "tracked for a later PR."

Proto types to map

  • NetworkMiddlewareConfig (sandbox.proto line 71): middleware configuration with enabled, timeout_ms, address, tls_ca, endpoints
  • MiddlewareEndpointSelector (line 87): pattern-based endpoint selection for middleware routing
  • SupervisorMiddlewareService (line 374): operator-registered middleware service metadata
  • SandboxPolicy.network_middlewares field (line 33): map of middleware configs on policy
  • GetSandboxConfigResponse.supervisor_middleware_services field (line 366)

Implementation scope

  • Add NetworkMiddlewareConfig domain type
  • Add MiddlewareEndpointSelector domain type
  • Add SupervisorMiddlewareService domain type
  • Add network_middlewares to SandboxPolicy type (currently in skipped set)
  • Update policy converter to handle middleware fields
  • Move network_middlewares from skipped to handled in coverage test
  • Unit tests

Target drop

Drop D (Policy + Service + TCP + SSH)

References

Activity

  1. added
    proto-gapProto API surface not yet covered by SDK
    drop-dTarget: Drop D (Policy + Network)
    on Jul 29, 2026
  2. rhuss commented on Jul 29, 2026

    @rhuss
    OwnerAuthor

    Implementation Hints

    Implement directly with plan mode (/plan in Claude Code). Add structs to existing policy domain.

    Approach

    1. Add types to types/policy.go:

      • NetworkMiddlewareConfig (enabled, timeout_ms, address, tls_ca, endpoints)
      • MiddlewareEndpointSelector (pattern fields)
      • SupervisorMiddlewareService (operator-registered metadata)
    2. Add NetworkMiddlewares map[string]NetworkMiddlewareConfig to SandboxPolicy type

    3. Update policy converter to handle network_middlewares field

    4. Move network_middlewares from skipped to handled in coverage test

    Note

    The coverage test already has network_middlewares in the skipped set with the comment "tracked for a later PR". This issue is that later PR.

    Verification

    make lint && make build && make test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    drop-dTarget: Drop D (Policy + Network)proto-gapProto API surface not yet covered by SDK

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions