Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

CI boundary checker misses renamed codex-core dependencies in codex-tui

Abierto Apto para principiantes
  • ¿Algo no está bien?
#57,750 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
76/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, python, rust
Área
ci-cd, testing

Línea de trabajo

Empieza por .github/scripts/verify_tui_core_boundary.py, especialmente por manifest_failures() y source_failures(), y luego inspecciona cómo .github/workflows/repo-checks.yml invoca el verificador. Añade pruebas de regresión para dependencias renombradas en tablas de dependencias normales y específicas del destino, y verifica que el verificador las rechace mientras sigue aceptando los manifiestos permitidos.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug
Summary

.github/scripts/verify_tui_core_boundary.py checks only whether the literal key codex-core appears in the Cargo dependency tables. Cargo permits renaming a dependency while preserving its underlying package name, so the checker accepts a direct dependency on codex-core if it is declared under an alias.

Source evidence

In manifest_failures(), the condition is if FORBIDDEN_PACKAGE in dependencies:. This tests dependency keys, not the package field of dependency specifications. source_failures() likewise looks for the literal codex_core identifier and misses imports using the alias.

Minimal reproduction (isolated Python, matching the current parser/check logic)
[dependencies]
core_alias = { package = "codex-core", path = "../core" }

With this dependency table, manifest_failures() returns []; with codex-core = { path = "../core" } it reports a failure. A Rust import use core_alias::... also avoids the hard-coded source patterns. This is a valid Cargo renamed-dependency form.

Expected

The CI check should reject any dependency whose key is codex-core or whose dependency table has package = "codex-core", including target-specific and dev/build dependencies.

Actual

The checker reports no violation for a renamed direct dependency, despite its stated purpose of enforcing the codex-tui → codex-core boundary. The check is invoked from .github/workflows/repo-checks.yml.

Suggested fix

Inspect each dependency entry's package field when the value is a table, and add a regression test for renamed direct dependencies and target-specific renamed dependencies. Consider token-aware source analysis separately to avoid false positives in comments.

Validation scope

Confirmed with an isolated Python reproduction of the exact TOML parsing and membership condition from the referenced source. No full Bazel/Cargo build was run. Reviewed source at commit e742db9f7ba98e63e502b310857d49d047888518; searched existing issues/PRs for this checker and found no exact duplicate.

Lenguaje dominante
Rust
Estrellas
131k
Forks
20.7k
Merge medio
1 min
PR fusionados (30 d)
997

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de openai/codex

Todos los issues de openai/codex

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.