Skip to content

feat/947 repository credential isolation - #967

Closed
sabbour wants to merge 1 commit into
devfrom
feat/947-repository-credential-isolation
Closed

feat/947 repository credential isolation#967
sabbour wants to merge 1 commit into
devfrom
feat/947-repository-credential-isolation

Conversation

@sabbour

@sabbour sabbour commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • removes AgentHost Key Vault, CSI, shared-home, and repository token-store paths; the pod has no workload identity or service-account token
  • restricts /configure to bounded Copilot inference material held only in AgentHost memory
  • introduces a trusted API launch issuer that derives Copilot purpose from persisted run state and uses feat: add purpose-bound GitHub credential broker #944 snapshots with before/after broker fencing
  • removes obsolete production configuration and documents NetworkPolicy as defense in depth rather than credential authority

Validation

  • dotnet test tests\\Agentweaver.Tests\\Agentweaver.Tests.csproj -p:CopilotSkipCliDownload=true --filter "FullyQualifiedName~TwoAppCredentialArchitectureTests|FullyQualifiedName~KubernetesSandboxExecutorClaimTests|FullyQualifiedName~AgentHostConfigureIdentityTests|FullyQualifiedName~PreviewRunnerAuthAndScrubTests|FullyQualifiedName~AgentHostReadinessProbeTests" (54 passed)
  • dotnet build tests\\Agentweaver.Tests\\Agentweaver.Tests.csproj --no-restore -p:CopilotSkipCliDownload=true
  • kubectl kustomize k8s\\overlays\\production

Closes #947

Remove AgentHost repository credential stores and Key Vault delivery paths. Route bounded Copilot inference material through a run-derived, snapshot-fenced API broker adapter.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3f94bc67-d29d-4d47-a8f7-5d99540c4b42
@sabbour

sabbour commented Aug 28, 2026

Copy link
Copy Markdown
Owner Author

Closing this PR because it uses the replaced #947 design. The approved sandbox-governed design is tracked in #968.

@sabbour sabbour closed this Aug 28, 2026
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.

feat: broker sandbox GitHub operations

1 participant