sass/dart-sass

Legacy JS API does not handle symlinks properly

Open

#1.687 geöffnet am 4. Mai 2022

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Dart (4.192 Stars) (376 Forks)batch import
bughelp wanted

Beschreibung

this is with npm sass .

relative import fails when it starts with 2 dots @import ../some-style.

I also checked how fs.existsSync handles paths with .. in it. on linux fs.existsSync fails to find the file if the path contains .., on windows it works...

Contributor Guide

Tech Stack
javascriptnodejs
Domain
tooling
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
needs investigation
Voraussetzungen
Node.js filesystem basicssymlink concept
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
The bug is in the legacy JS API of dart sass where relative imports with '..' fail on Linux due to symlink handling. Investigate how fs.existsSync and related functions handle paths with '..' on Linux vs Windows. Look at the JS API source (likely in lib/src/legacy.js or similar) to understand the import resolution logic. Check if there are any existing tests for symlink behavior. The fix may involve using realpath or fs.realpathSync to resolve symlinks before checking file existence.