[cretz]

🚀 What

• Remove show_secrets from SandboxClient.get_info / AsyncSandboxClient.get_info; the backend no longer honors it
  • 💥 BACKWARDS INCOMPATIBLE CHANGE

💻 How

• Drop the parameter and stop sending show_secrets
• Remove the show_secrets reference from the SandboxEnvValue.secret doc
• Leave the generated management client alone until the spec drops the param

🔬 Testing

• Remove the secrets-revealed e2e test; the get_info e2e test already checks the masked values
• Remove the show_secrets query-param unit test

[Copilot]

🟢 Approval recommended

No active review issues are recorded for the removal of the unsupported show_secrets option.

No open issues 🎉

What changed in this PR

Removes the unsupported show_secrets option from the sandbox SDK’s synchronous and asynchronous get_info methods. This is a breaking API change; the generated management client remains unchanged pending a spec update.

Changes:

• Remove the parameter and stop forwarding it to the management API.
• Update environment-value documentation to state that secrets are masked.
• Remove tests for revealing secrets and forwarding the removed parameter.

File	Description
tests/​sandbox/​test_e2e.py	Remove the secrets-revealed end-to-end test.
tests/​sandbox/​test_client.py	Remove the show_secrets query-parameter test.
baseten/​sandbox/​_info.py	Remove the documented exception to secret masking.
baseten/​sandbox/​_client.py	Remove show_secrets from both get_info implementations.

_{Review effort: Balanced}