Skip to content

Fix network review findings - #892

Merged
SaladDay merged 1 commit into
mainfrom
fix-network-review-findings
Oct 27, 2026
Merged

SaladDay merged 1 commit into
mainfrom
fix-network-review-findings

Conversation

@SaladDay

@SaladDay SaladDay commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes a batch of review findings on the sandbox network cap.

  • State the E2B exceptions to the blocked-address rule in the contract (EN + zh) and in the Provider comments.
  • Delete the unused setup cautions end to end: the declaration field, the generated projection, the Web confirmation dialog and its CSS and strings.
  • Drop the policy artifact role from the Provider docs and remove an unused import in NetworkSettings.
  • Reject a submitted network without public or domains with a 400 naming the member.
  • Log only the helper's unresolved/unsupported name lines.
  • Share one exact-host rule between the Environment allowed_domains check and the deployment cap, so a name such as x.1 fails at Session creation.
  • Point the troubleshooting guide to Settings -> Sandbox and scheduling -> Network.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@SaladDay
SaladDay merged commit efe01b8 into main Oct 27, 2026
22 checks passed
@SaladDay
SaladDay deleted the fix-network-review-findings branch October 27, 2026 19:03
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