rtk-ai/rtk

fix: ls tracking baseline should compare against `ls` not `ls -la`

Open

#561 ouverte le 13 mars 2026

Voir sur GitHub
 (1 commentaire) (4 réactions) (0 assignés)Rust (2 914 forks)batch import
area:clibugeffort-smallfilter-qualitygood first issuepriority:medium

Métriques du dépôt

Stars
 (48 085 stars)
Métriques de merge PR
 (Merge moyen 11j 1h) (45 PRs mergées en 30 j)

Description

Problem

rtk ls internally runs ls -la to get file metadata for grouping/filtering, then compresses the output. The savings % is tracked against the ls -la output, not plain ls (which is what the agent would have run without RTK).

This inflates the reported savings. Example from user report:

  • ls = 332 tokens
  • ls -la = 4173 tokens
  • rtk ls = 993 tokens
  • Reported: 76.2% savings (vs ls -la)
  • Actual: -199% vs plain ls

Fix

Track against the baseline command the agent would have run (ls), not the internal enriched command (ls -la).

In src/ls.rs line 113-118, change the tracking input from the ls -la raw output to a plain ls capture (or estimate).

Reference

https://www.reddit.com/r/ClaudeAI/comments/1rjkf9r/comment/oa3trnn/

Guide contributeur

Direction de recherche
Localisez les lignes 113-118 dans src/ls.rs, comprenez la logique actuelle de suivi de la référence et modifiez la pour comparer avec la sortie de `ls` simple au lieu de `ls la`. Testez le changement avec l'exemple fourni.
Stack technique
rust
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70