feat: get the latest github release tag for codex#2081
feat: get the latest github release tag for codex#2081kemingy merged 4 commits intotensorchord:mainfrom
Conversation
Priority: - use the user specified version - use the latest tag - fallback to the default one Signed-off-by: Keming <[email protected]>
There was a problem hiding this comment.
Pull request overview
This PR adds functionality to automatically fetch the latest Codex release version from GitHub when no explicit version is specified by the user. The implementation follows a priority system: user-specified version → latest GitHub release → default fallback version.
Changes:
- Added
getLatestVersionfunction to fetch the latest release tag from GitHub API - Updated version resolution logic in
installAgentCodexto attempt fetching the latest version when user doesn't specify one - Updated default Codex version from "0.92.0" to "0.98.0"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- refact the getLatestReleaseVersion - cache to the ~/.cache/envd/ dir Signed-off-by: Keming <[email protected]>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- fix the envd interface args for codex/node/go - rm user/repo name sanitizer - fix codex log msg Signed-off-by: Keming <[email protected]>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (2)
envd/api/v1/install.py:98
- Changing the signature default from a pinned version to
Noneno longer reflects the implementation: the Go backend still uses a pinned default (golangDefaultVersion) when no version is specified. Either restore the default argument to the pinned version, or update the docstring to explicitly state what version will be installed whenversionis omitted.
def go(version: Optional[str] = None):
"""Install Go programming language.
Args:
version (Optional[str]): Go version, such as '1.25.3'.
envd/api/v1/install.py:106
- Same issue as
go(): the backend uses a pinned default (nodejsDefaultVersion) whenversionis not provided, but the signature now suggests there is no default. Please align the signature/docstring with the actual defaulting behavior to avoid misleading generated docs and users.
def nodejs(version: Optional[str] = None):
"""Install NodeJS programming language.
Args:
version (Optional[str]): NodeJS version, such as '25.1.0'.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Keming <[email protected]>
Priority: