Skip to content

feat: add codex template#2054

Merged
kemingy merged 1 commit intotensorchord:mainfrom
kemingy:codex_template
Nov 7, 2025
Merged

feat: add codex template#2054
kemingy merged 1 commit intotensorchord:mainfrom
kemingy:codex_template

Conversation

@kemingy
Copy link
Copy Markdown
Member

@kemingy kemingy commented Nov 5, 2025

  • add codex.envd template, try it with envd new -t codex
  • change codex from gnu to musl to avoid the glibc version requirements
  • only install the uv python once when dev=True
  • use apt glob syntax to avoid "Unable to locate package libtinfo5" on Ubuntu 24:04

Signed-off-by: Keming <kemingyang@tensorchord.ai>
@kemingy kemingy requested review from Copilot and removed request for VoVAllen, gaocegege, lalawuu and zwpaper November 5, 2025 15:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the codex agent installation to use musl-based binaries instead of glibc-based ones, updates base APT packages to support both version 5 and 6 of libtinfo and libncursesw libraries, and adds a new codex template file. Additionally, it optimizes UV Python installation to skip for sudo when dev=True.

  • Changed codex agent downloads from linux-gnu to linux-musl builds
  • Updated base APT packages to use pattern matching for library versions (libtinfo[56] and libncursesw[56])
  • Added early return in UV compilation when dev=True to avoid redundant Python installation

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pkg/types/envd.go Updated base APT packages to use version-flexible patterns for ncurses libraries
pkg/lang/ir/v1/uv.go Added conditional early return to skip UV Python installation for sudo when dev mode is enabled
pkg/lang/ir/v1/compile.go Corrected comment from "re-install" to "install" for clarity
pkg/lang/ir/v1/agent.go Changed codex binary downloads from glibc (linux-gnu) to musl (linux-musl), removed glibc version warning and unused logrus import
pkg/app/template/codex.envd Added new template file for codex environment setup

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/types/envd.go
@kemingy kemingy requested a review from gaocegege November 6, 2025 02:09
@kemingy kemingy merged commit 5d654fe into tensorchord:main Nov 7, 2025
18 checks passed
@kemingy kemingy deleted the codex_template branch November 7, 2025 03:12
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