Skip to content

fix(agent_hub): verify visibility on incremental upload to existing repo - #945

Open
mushenL wants to merge 1 commit into
modelscope:mainfrom
mushenL:bug_0820_01
Open

fix(agent_hub): verify visibility on incremental upload to existing repo#945
mushenL wants to merge 1 commit into
modelscope:mainfrom
mushenL:bug_0820_01

Conversation

@mushenL

@mushenL mushenL commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Change Summary

Related issue number

增量上传到已有仓库时校验可见性,防止 private 静默泄露为 public

fix(agent_hub): verify visibility on incremental upload to existing repo

The private-visibility guard only ran when the repo was freshly created.
Incremental pushes into an already-existing repo skipped verification
entirely and could silently leak content to a public repo while reporting
"private". Verify before building the diff and before the up-to-date early
return, so the upload aborts before any byte is pushed or deleted.

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Run pre-commit install and pre-commit run --all-files before git commit, and passed lint check.
  • Documentation reflects the changes where applicable

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