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

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#462 4 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@Wizard52007 y travaille déjà.

Depuis le 10/11/2026.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
74/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
rust
Domaine
tooling

Piste de recherche

Lisez les trois emplacements mentionnés dans l’issue : tools/aimdb-cli/src/error.rs, tools/aimdb-mcp/src/server.rs et aimdb-zenoh-connector/src/profile/mod.rs. Supprimez les suppressions répertoriées et exécutez les combinaisons de fonctionnalités build et clippy du Makefile pour repérer les avertissements restants, en particulier lorsque la fonctionnalité ros2 est désactivée. Le travail est terminé lorsque ces vérifications réussissent sans suppressions générales et que toute allowance nécessaire est limitée à l’élément concerné.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

🏷️ 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.

Langage dominant
Rust
Étoiles
101
Forks
20
Merge moyen
2 h 33 min
PR mergées (30 j)
90

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de aimdb-dev/aimdb

Toutes les issues de aimdb-dev/aimdb

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.