Repository navigation
[safe-output-integrator] Daily Safe Output Integrator: add missing steer test coverage - #62220
Conversation
All safe-output types already have pkg/cli/workflows/test-*.md fixtures. This adds the one missing Go compiler-test case: the global 'steer' boolean flag in SafeOutputsConfig, following the existing table-driven pattern in compiler_safe_outputs_config_test.go. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
@copilot This PR still needs forward progress before a maintainer can merge it. Please refresh the branch if needed, verify the added safe-output steer test coverage, update the branch if GitHub allows it, and run the Run: https://github.com/github/gh-aw/actions/runs/35531716641 Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "github.com"See Network Configuration for more information.
|
Verified commit |
|
@copilot This PR still needs forward progress before a maintainer can merge it. Please refresh the branch if needed, address any remaining review feedback or failed checks, update the branch if GitHub allows it, and run the Run: https://github.com/github/gh-aw/actions/runs/35537069559
|
Rechecked commit |
There was a problem hiding this comment.
Warning
Firewall blocked 2 domains
The following domains were blocked by the firewall during workflow execution:
clients2.google.comupdate.googleapis.com
To allow these domains, add them to the network.allowed list in your workflow frontmatter:
network:
allowed:
- defaults
- "clients2.google.com"
- "update.googleapis.com"See Network Configuration for more information.
💥 [THE END] — Illustrated by Smoke Claude · claude · sonnet46 · 65.4 AIC · ⌖ 17.8 AIC · ⊞ 7.9K
Comment /smoke-claude to run again
|
@copilot This PR still needs forward progress before a maintainer can merge it. Please refresh the branch if needed, review any remaining checks or feedback, update the branch if GitHub allows it, and run the Run: https://github.com/github/gh-aw/actions/runs/35562335069 Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "github.com"See Network Configuration for more information.
|
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Pushed |
|
@copilot This PR looks ready for another pass. Please refresh the branch if needed, review current checks, and run the Warning Firewall blocked 2 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "github.com"
- "registry.npmjs.org"See Network Configuration for more information.
|
Re-ran the finisher pass on |
|
@copilot This PR still needs forward progress before a maintainer can merge it. Please refresh the branch if needed, review any remaining checks or feedback, update the branch if GitHub allows it, and run the Run: https://github.com/github/gh-aw/actions/runs/35578515677
|
Re-ran the finisher pass on |
|
🎉 This pull request is included in a new release. Release: |
Daily Safe Output Integrator
Overview
Audited every YAML key in
SafeOutputsConfig(pkg/workflow/safe_outputs_config_types.go) against:pkg/cli/workflows/test-*.mdfixturespkg/workflow/compiler_safe_outputs_config_test.gotable-driven casesKey Metrics
jobs,github-app,env,github-token,allowed-domains,allowed-github-references,staged,threat-detection).pkg/cli/workflows/test-*.mdfile covering it. No new fixtures needed.steerboolean flag had no case inTestAddHandlerManagerConfigEnvVar.Change
Added a
steer configtable-test case tocompiler_safe_outputs_config_test.go, following the existing pattern (paired withCreateIssuesso the handler-config JSON is non-empty and assertable), verified with:All 66 subtests pass, including the new
steer_configcase.Coverage detail
No missing fixture types were found — this run required no new
test-copilot-*.mdfiles. Only the Go-side gap forsteerneeded a new case.Next Actions
None — coverage is now complete for both fixtures and Go compiler tests across all supported safe-output types.
Warning
Firewall blocked 3 domains
The following domains were blocked by the firewall during workflow execution:
go.uber.orggo.yaml.instorage.googleapis.comTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
hub.lumenfield.workTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
✨ PR Review Safe Output Test - Run 35547324855
Warning
Firewall blocked 2 domains
The following domains were blocked by the firewall during workflow execution:
clients2.google.comupdate.googleapis.comTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
Run: https://github.com/github/gh-aw/actions/runs/35562335069
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
hub.lumenfield.workTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
hub.lumenfield.workTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
Run: https://github.com/github/gh-aw/actions/runs/35607531481
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
hub.lumenfield.workTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.