Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Improve version string matching in Add plugin recipe

Aperta Adatta ai principianti
  • Qualcosa non va?
#270 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
86/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
java
Ambito
cli

Direzione di ricerca

Inizia in src/main/java/com/canonical/devpackspring/rewrite/visitors/AddPluginVisitorSupport.java intorno alle righe 228-229 ed esamina versionMatches insieme a pluginNameFilter(). Verifica come vengono gestiti gli argomenti di versione non letterali, quindi assicurati che la corrispondenza sia conservativa o utilizzi testo sorgente affidabile, in modo che un’istruzione del plugin che corrisponde effettivamente non venga sostituita inutilmente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

─── src/main/java/com/canonical/devpackspring/rewrite/visitors/AddPluginVisitorSupport.java:228-229 ───
[bug · medium] For non-literal version arguments (variables, Groovy GString / Kotlin string
interpolation), `versionMatches` falls back to `expr.toString()`, which on an OpenRewrite tree node
does not reliably return the expression's source text, so the equality check against pluginVersion
is unreliable. This is also inconsistent with `pluginNameFilter()` (below), which conservatively
returns false for non-literals. As a result, a version that actually matches may be treated as a
mismatch and have its plugin statement needlessly replaced. Consider conservatively returning false
for non-literals (as pluginNameFilter does) or using a source-printing helper to extract the text.

-               String versionStr = (expr instanceof J.Literal literal && literal.getValue() != null)
-                               ? literal.getValue().toString() : expr.toString();
+               if (!(expr instanceof J.Literal literal) || literal.getValue() == null) {
+                       // Cannot reliably determine version from non-literal expression
+                       return false;
+               }
+               String versionStr = literal.getValue().toString();
Lingua principale
Java
Stelle
1
Fork
5
Merge medio
1g 14h
PR unite (30g)
27

Preparare l'ambiente

Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di canonical/devpack-for-spring-cli

Tutte le issue di canonical/devpack-for-spring-cli

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.