Skip to content

fix(studio): normalize TOS bucket policy comparison - #1058

Merged
yaozheng-fang merged 3 commits into
volcengine:mainfrom
zyn080302:fix/studio-policy-normalization-20260904
Sep 4, 2026
Merged

fix(studio): normalize TOS bucket policy comparison#1058
yaozheng-fang merged 3 commits into
volcengine:mainfrom
zyn080302:fix/studio-policy-normalization-20260904

Conversation

@zyn080302

@zyn080302 zyn080302 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • compare TOS bucket Policy statements semantically across scalar/list response normalization
  • accept only the observed server-added Version: 1.0 when the submitted Policy omitted Version
  • preserve and verify all unrelated Policy statements while continuing to reject owned-statement conflicts
  • add the repository-standard Apache-2.0 header to four Python tests newly added by base PR feat(studio): support VeStack deployment and personal sandbox agents #1057, repairing the inherited License-Check failure without changing test logic

Validation

  • 148 focused Release Server/Studio release/artifact/self-update tests passed
  • 25 affected VeStack/AgentKit tests passed
  • full pre-commit passed, including Ruff formatting and hardcoded-secret detection
  • changed Release Server source passed Pyright with 0 errors
  • uv lock --check and git diff --check passed
  • read-only validation against the real normalized TOS Policy matched without requesting a rewrite

@yaozheng-fang
yaozheng-fang merged commit c567d78 into volcengine:main Sep 4, 2026
11 checks passed
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.

2 participants