trycua/cua

Variable shadowing in computer class breaks shared directories.

Open

Aperta il 23 apr 2025

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)HTML (16.722 star) (1051 fork)batch import
good first issue

Descrizione

This section in the Computer.__init__ method:

 if shared_directories:
    for path in shared_directories:
        abs_path = os.path.abspath(os.path.expanduser(path))  # type: ignore[attr-defined]
        if not os.path.exists(abs_path):  # type: ignore[attr-defined]
            raise ValueError(f"Shared directory does not exist: {path}")
        self.shared_paths.append(abs_path)

fails with:

abs_path = os.path.abspath(os.path.expanduser(path))  # type: ignore[attr-defined]
               ^^^^^^^
AttributeError: 'str' object has no attribute 'path'

because the module name is being shadowed by the init variable os, which is a string (macos, linux).

Please fix.

Guida contributor

Tech stack
python
Dominio
backend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
basic Pythonunderstanding of variable shadowing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90
Direzione di ricerca
Il bug si trova nel metodo `Computer. init ` dove la variabile `os` oscura il modulo `os`. La soluzione è rinominare la variabile `os` in qualcosa come `os name` per evitare l'oscuramento. Il codice specifico si trova nel file che contiene la classe `Computer` (probabilmente `computer.py` o simile). Dopo la rinomina, assicurarsi che le chiamate `os.path` e `os.path.exists` funzionino correttamente.