Skip to content

docs(sandboxes): document the copilot/github credential split - #25821

Draft
mdelapenya wants to merge 2 commits into
docker:mainfrom
mdelapenya:docs/copilot-credential-secret-split
Draft

docs(sandboxes): document the copilot/github credential split#25821
mdelapenya wants to merge 2 commits into
docker:mainfrom
mdelapenya:docs/copilot-credential-secret-split

Conversation

@mdelapenya

Copy link
Copy Markdown
Member

Summary

  • Adds a copilot row to the built-in services table (COPILOT_GITHUB_TOKEN, Copilot API domains)
  • Adds a note explaining why github and copilot are separate secrets (different token type requirements), and why some of their API domains overlap depending on which kit is running

Related

Follows the credential split shipped in docker/sandboxes#5041.

Adds the copilot service row and a note explaining why github and
copilot are separate secrets, and why their API domains overlap
depending on which kit is running.

Signed-off-by: Manuel de la Peña <manuel.delapena@docker.com>
@mdelapenya
mdelapenya requested a review from dvdksn as a code owner August 13, 2026 16:30
@netlify

netlify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit e748f83
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a7df29a6c84000008dd56ee
😎 Deploy Preview https://deploy-preview-25821--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mdelapenya
mdelapenya marked this pull request as draft August 13, 2026 16:32

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The new copilot row in the built-in services table and the accompanying NOTE callout are clear, well-scoped, and accurate. No high or medium severity issues found.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The PR adds a copilot row to the built-in services table and a NOTE callout explaining why github and copilot are separate secrets. The table row is correctly placed alphabetically, the domains are consistent with the existing github row, and the NOTE accurately describes the credential split and domain overlap. No high or medium issues found.

Signed-off-by: Manuel de la Peña <manuel.delapena@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants