tests: cover structural_impact.py pure helpers; thread repo_root and align path resolution
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 56/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- github-actions, python
- Área
- ci-cd, testing-qa, tooling
Línea de trabajo
Empieza por .agents/tools/structural_impact.py, especialmente _get_package, _dedup, _collect_source_files, _unknown_package_dirs, _rel, _changed_files_mode y _full_mode; después inspecciona el PR #567 y los workflows de CI agentic existentes. Añade .agents/tools/tests/test_structural_impact.py para los casos indicados, intégralo en CI y verifica el manejo explícito de repo_root y la resolución simétrica de rutas con el conjunto de pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Priority Level
Medium
Task Summary
Add unit-test coverage for the pure helpers in .agents/tools/structural_impact.py and clean up two latent issues that get cheap to fix in the same pass: the module-level _REPO_ROOT global and the asymmetric .resolve() between changed-files paths and source-file paths.
Follow-up to PR #567 review feedback from @nabinchha.
Technical Details & Implementation Plan
-
Unit tests — add
.agents/tools/tests/test_structural_impact.pycovering:_get_package— engine / config / interface / external paths, including thedata-designervsdata-designer-enginesubstring-precedence ordering_dedup— collisions on shared label prefixes (regression guard for the bug fixed inbfc5db69)_collect_source_files— smoke test against a tmp dir with mockpackages/<name>/src/...layout_unknown_package_dirs— synthetic packages outside_KNOWN_PACKAGE_DIRS
-
CI wiring —
.agents/tools/lives outside the three package test suites, somake testwon't pick it up. Decide between (a) a new Makefile target, (b) a step inagentic-ci-pr-review.yml/agentic-ci-daily.yml, or (c) folding into an existing CI workflow. Lean toward (b) since the file's own change cadence is tied to the agentic-CI surface. -
Thread
repo_rootthrough helpers — replace the module-level_REPO_ROOTglobal +global _REPO_ROOTmutation inmain()with explicit parameter passing through_collect_source_files,_unknown_package_dirs,_rel,_changed_files_mode,_full_mode. Removes test-time fragility (no need to mutate a global to vary input) and decouples the otherwise-pure helpers from module state. -
Symmetric path resolution — once helpers take
repo_rootexplicitly, either drop.resolve()onchanged_paths(current_changed_files_mode:198) or also.resolve()paths inside_collect_source_files. Latent today because no symlinks live underpackages/, but the symmetric form removes a future-divergence trap.
Investigation / Context
PR #567 review thread: https://github.com/NVIDIA-NeMo/DataDesigner/pull/567 (review by @nabinchha and follow-up at https://github.com/NVIDIA-NeMo/DataDesigner/pull/567#issuecomment-4373073701)
Relevant prior fixes that justify the test coverage:
bfc5db69—_dedup30-char prefix-collision bug (pre-merge)55f0bdda—changed_node_idsID-stability bug (pre-merge)
Both bugs were caught during review; tests would have surfaced them earlier and would protect against the most likely future failure mode (graphify API drift at the loose graphifyy==0.4.23 pin).
Dependencies
None
- Lenguaje dominante
- Python
- Estrellas
- 2.3k
- Forks
- 227
- Merge medio
- 22 h 51 min
- PR fusionados (30 d)
- 61
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de NVIDIA-NeMo/DataDesigner
-
docs: required_columns description is incomplete for LLM and multimodal columnsPosiblemente ocupada @nightcityblade la tomó hace 10 días. Abiertobug triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
NVIDIA-NeMo/DataDesigner#1002 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
-
enforce \from future import annotations` via ruff FA102 rule`Posiblemente ocupada @chethanuk la tomó hace 38 días. Abiertotask
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
NVIDIA-NeMo/DataDesigner#760 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
NVIDIA-NeMo/DataDesigner#1048 ·
Los mantenedores suelen responder en 1 día
-
bug needs-discussion
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
NVIDIA-NeMo/DataDesigner#1045 ·
Los mantenedores suelen responder en 1 día
-
Structured-output parsing drops valid dictionary entriesPosiblemente ocupada @nabinchha la tomó hace 2 días. Abiertobug
NVIDIA-NeMo/DataDesigner#1043 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de NVIDIA-NeMo/DataDesigner
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
SublimeText/AdvancedNewFile#211 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Imbad0202/academic-research-skills#987 ·
Los mantenedores suelen responder en 1 día
-
Add date of request to the UIAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Pull-through from gcr.io returns 500 when pulling using podman/buildahPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoIssue Triage-Needed
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pulp/pulp_container#2581 ·
Los mantenedores suelen responder en 1 día