trycua/cua

Variable shadowing in computer class breaks shared directories.

Open

#124 aperta il 23 apr 2025

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

Metriche repository

Star
 (16.722 star)
Metriche merge PR
 (Merge medio 1g 9h) (119 PR mergiate in 30 g)

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

Direzione di ricerca
Rinomina la variabile `os` nel metodo `Computer. init ` in qualcosa come `os name` per evitare di oscurare il modulo `os`.
Tech stack
python
Dominio
infrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80