chore: remove stale dead_code allows in aimdb-cli, aimdb-mcp and the zenoh profile module

Abierto Apto para principiantes
  • ¿Algo no está bien?
#462 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
74/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
tooling

Línea de trabajo

Lee las tres ubicaciones mencionadas en la issue: tools/aimdb-cli/src/error.rs, tools/aimdb-mcp/src/server.rs y aimdb-zenoh-connector/src/profile/mod.rs. Elimina las supresiones enumeradas y ejecuta las combinaciones de características de build y clippy del Makefile para encontrar cualquier advertencia restante, especialmente con la característica ros2 desactivada. El trabajo estará terminado cuando estas comprobaciones pasen sin supresiones amplias y cualquier allowance necesaria esté limitada al elemento afectado.

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

Descripción

🏷️ good first issue enhancement

Problem

Three #[allow(dead_code)] suppressions have outlived their reason. They no longer hide anything today, but they will silently hide real dead code in future.

  1. tools/aimdb-cli/src/error.rs:23 — #[allow(dead_code)] // Some variants are for future use on CliError, and :141 — #[allow(dead_code)] // Helper methods for future use on its constructor helpers. Every helper (connection_failed, record_not_found, write_protected, protocol_mismatch, invalid_json, server_error) has at least one caller in tools/aimdb-cli/src, and the variants are constructed either through those helpers or in the From<ClientError> conversion.
  2. tools/aimdb-mcp/src/server.rs:103 — #[allow(dead_code)] on set_state, which is called at line 119 and in the tests.
  3. aimdb-zenoh-connector/src/profile/mod.rs:8 — a module-wide #[allow(dead_code, unused_imports)] with the comment "Used by the ROS 2 connector, which builds on this module". ros2.rs now exists and uses most of the module (Attachment, Durability, EntityKind, History, Node, Qos, Reliability, validate_*, data_key, entity_token, node_token, gid).

Suggested work

  • Remove the three allows and build/clippy with the feature combinations the Makefile uses.
  • For anything that then warns (most likely in the zenoh profile module when the ros2 feature is off), either delete it or scope a narrow #[cfg_attr(not(feature = "…"), allow(dead_code))] to that item instead of the whole module.

Notes

Usage was checked by grep at d9c7be0, not by compiling without the allows, so a few items may still warn under some feature combinations.

Lenguaje dominante
Rust
Estrellas
101
Forks
20
Merge medio
2 h 33 min
PR fusionados (30 d)
90

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

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 aimdb-dev/aimdb

Todos los issues de aimdb-dev/aimdb

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.