fix: remote git repo doesn't need token if it's public repo (#775)

Author: christinewangcw

.pre-commit-config.yaml

@@ -80,27 +80,13 @@ repos:
     rev: 39.169.3
     hooks:
       - id: renovate-config-validator
+
   - repo: https://github.com/astral-sh/uv-pre-commit
     rev: "0.5.31"
     hooks:
       - id: uv-sync
         args: ["--frozen", "--all-packages", "--all-extras"]
 
-  - repo: "local"
-    hooks:
-      # Disabled as part of LFS removal.
-      # - id: disallowed-words-check
-      #   name: Check for disallowed words
-      #   entry: scripts/disallowed-words-check.sh
-      #   language: script
-      #   files: ''  # Check all files
-      - id: generate-runner-imports
-        name: Generate Runner Imports
-        entry: bash -c "uv run --frozen python -m codegen.gscli.cli generate runner-imports src/codegen/shared/compilation/function_imports.py"
-        language: system
-        pass_filenames: false
-        always_run: true
-
   - repo: https://github.com/hukkin/mdformat
     rev: 0.7.22  # Use the ref you want to point at
     hooks:

src/codegen/git/repo_operator/repo_operator.py

@@ -20,7 +20,7 @@
 from codegen.git.clients.git_repo_client import GitRepoClient
 from codegen.git.configs.constants import CODEGEN_BOT_EMAIL, CODEGEN_BOT_NAME
 from codegen.git.repo_operator.local_git_repo import LocalGitRepo
-from codegen.git.schemas.enums import CheckoutResult, FetchResult, SetupOption
+from codegen.git.schemas.enums import CheckoutResult, FetchResult, RepoVisibility, SetupOption
 from codegen.git.schemas.repo_config import RepoConfig
 from codegen.git.utils.clone import clone_or_pull_repo, clone_repo, pull_repo
 from codegen.git.utils.clone_url import add_access_token_to_url, get_authenticated_clone_url_for_repo_config, get_clone_url_for_repo_config, url_to_github
@@ -89,7 +89,7 @@ def repo_path(self) -> str:
 
     @property
     def remote_git_repo(self) -> GitRepoClient:
-        if not self.access_token:
+        if not self.access_token and self.repo_config.visibility != RepoVisibility.PUBLIC:
             msg = "Must initialize with access_token to get remote"
             raise ValueError(msg)