fix: add custom github auth using metadata

[shreysingla11]

Important

Add custom GitHub authentication using metadata in git_clone_cmd and GitClone.execute.

• Behavior:
  • git_clone_cmd now requires github_access_token as a parameter instead of fetching from environment.
  • execute method in GitClone retrieves github_access_token from metadata if not in environment.
• Error Handling:
  • Raises RuntimeError in git_clone_cmd if no token is provided and ALLOW_CLONE_WITHOUT_REPO is not "true".

^{This description was created by for 7e4994b. It will automatically update as commits are pushed.}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
composio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 30, 2024 1:46pm

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 7e4994b in 11 seconds

More details

• Looked at 35 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. python/composio/tools/local/filetool/actions/git_clone.py:31

• Draft comment:
  Consider handling the logic for checking the presence of github_access_token in the execute method to avoid duplication.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The github_access_token is being retrieved from the environment variables and metadata, but the logic for checking its presence is duplicated in git_clone_cmd. It would be more efficient to handle this logic once in the execute method.

Workflow ID: wflow_ltz4nGhTvugr8qcO

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.