Conversation
…ithub to default-safe-outputs Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
💥 Automated smoke test review - all systems nominal!
💥 [THE END] — Illustrated by Smoke Claude
pkg/workflow/domains.go
Outdated
| // Covers common infrastructure certificate/OCSP hosts (via "defaults") plus popular | ||
| // developer-tool and CI/CD service domains (via "dev-tools"). | ||
| "default-safe-outputs": {"defaults", "dev-tools"}, | ||
| // developer-tool and CI/CD service domains (via "dev-tools") and GitHub domains (via "github"). |
There was a problem hiding this comment.
The comment update clearly documents the change. The default-safe-outputs expansion to include github makes sense — most workflows interact with GitHub APIs/docs.
There was a problem hiding this comment.
Pull request overview
Updates the workflow domain ecosystem configuration so the default-safe-outputs compound ecosystem includes GitHub-related domains, and refreshes tests/golden outputs accordingly.
Changes:
- Expanded
default-safe-outputsto uniondefaults + dev-tools + github. - Added/reshaped entries in
ecosystem_domains.json(notably more GitHub domains and loopback hosts indev-tools). - Updated unit tests and the Copilot WASM golden fixture to match the new expanded allow-domain set.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| pkg/workflow/testdata/wasm_golden/TestWasmGolden_CompileFixtures/smoke-copilot.golden | Updates the expected --allow-domains string to include the newly-added GitHub domains. |
| pkg/workflow/domains.go | Expands default-safe-outputs compound ecosystem to include github and updates related comments. |
| pkg/workflow/domains_test.go | Updates default-safe-outputs expansion test expectations/samples for the added GitHub + loopback entries. |
| pkg/workflow/data/ecosystem_domains.json | Adds loopback hosts to dev-tools and expands the github ecosystem domain list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| @@ -52,6 +54,7 @@ | |||
| "docs.github.com", | |||
| "drone.io", | |||
| "img.shields.io", | |||
| "localhost", | |||
| "fonts": ["fonts.googleapis.com", "fonts.gstatic.com"], | ||
| "github": [ | ||
| "*.githubusercontent.com", | ||
| "raw.githubusercontent.com", | ||
| "objects.githubusercontent.com", | ||
| "lfs.github.com", | ||
| "codeload.github.com", | ||
| "docs.github.com", | ||
| "github-cloud.githubusercontent.com", | ||
| "github-cloud.s3.amazonaws.com", | ||
| "codeload.github.com", | ||
| "github.githubassets.com" | ||
| "github.blog", | ||
| "github.com", | ||
| "github.githubassets.com", | ||
| "lfs.github.com", | ||
| "objects.githubusercontent.com", | ||
| "raw.githubusercontent.com" |
pkg/workflow/domains_test.go
Outdated
| // Should include defaults + dev-tools + github (at least 34+24+11 = 69 domains, with some overlap) | ||
| if len(result) < 60 { | ||
| t.Errorf("Expected at least 60 domains in default-safe-outputs, got %d", len(result)) | ||
| } |
…p dev-tools Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in 834316a. Added a dedicated |
Four related domain set changes: introduce a dedicated
localecosystem for loopback addresses, flesh out thegithubecosystem with first-party domains, clean updev-tools, and include bothgithubandlocalin thedefault-safe-outputsbaseline.Changes
local(new) — dedicated ecosystem forlocalhost,127.0.0.1,::1dev-tools— remove loopback addresses (moved tolocal) anddocs.github.com(now covered bygithub)github— addgithub.com,docs.github.com,github.blog(alongside existing*.githubusercontent.comentries)default-safe-outputs— expand compound from{defaults, dev-tools}→{defaults, dev-tools, github, local}dev-toolskey inecosystem_domains.json(Go's JSON decoder silently used the last one; now canonical)TestDefaultSafeOutputsEcosystemnow computes expected domain count dynamically from the union of component ecosystems instead of a brittle hard-coded thresholdsmoke-copilot.goldento reflect the updated--allow-domainslist✨ PR Review Safe Output Test - Run 23123559040
📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.