Skip to content

feat(supervisor-network): add request inspection seam poc#1971

Draft
afourniernv wants to merge 3 commits into
NVIDIA:mainfrom
afourniernv:feat-relay-inspection-poc
Draft

feat(supervisor-network): add request inspection seam poc#1971
afourniernv wants to merge 3 commits into
NVIDIA:mainfrom
afourniernv:feat-relay-inspection-poc

Conversation

@afourniernv

Copy link
Copy Markdown

Summary

Add an additive request-inspection seam to openshell-supervisor-network so the sandbox runtime can call an external inspector at the outbound HTTP boundary. This is a narrow POC for loose coupling with semantic middleware systems like NeMo Relay, without implementing the full middleware subsystem proposed in RFC #1738.

Related Issue

Changes

  • Add a minimal inspection contract and optional inspector client in openshell-supervisor-network
  • Thread the optional inspector through the real outbound HTTP forwarding path
  • Support allow, deny, and header mutation decisions at the runtime boundary
  • Add focused tests covering the demo inspector client roundtrip and request-inspection seam

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Alex Fournier <afournier@nvidia.com>
Signed-off-by: Alex Fournier <afournier@nvidia.com>
Signed-off-by: Alex Fournier <afournier@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 22, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

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.

1 participant