Dependencies view: `getParent` loops forever on untitled documents, extension host runs out of memory
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 70/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript, vscode
- Ambito
- tooling
Direzione di ricerca
Il ciclo si trova in vscode/src/dependenciesTree.ts: getParent (intorno alla riga 55) risale finché non raggiunge la radice del percorso, e activeEditorDidChange (intorno alla riga 92) rivela l'URI di qualsiasi editor attivo indipendentemente dallo scheme, quindi i documenti untitled vengono percorsi all'infinito. Inizia leggendo queste due funzioni e treeVisibilityDidChange, che rivela anche il currentVisibleItem memorizzato. È completato quando attivare un editor untitled non provoca più né un reveal né un percorso verso l'alto, e getParent restituisce undefined per un percorso padre non assoluto o che non avanza.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
Ruby LSP Information
VS Code Version
1.141.0
Ruby LSP Extension Version
0.10.6
Ruby LSP Server Version
0.26.11
Ruby LSP Add-ons
- Ruby LSP Rails (0.4.8)
- RuboCop (1.86.1)
Ruby Version
3.4.7
Ruby Version Manager
mise
Installed Extensions
Click to expand
- bruno (6.0.0)
- claude-code (2.1.296)
- gitlab-workflow (7.2.0)
- ruby-extensions-pack (0.1.14)
- ruby-lsp (0.10.6)
- sorbet-vscode-extension (0.3.47)
- sql-developer (26.3.0)
- vscode-yaml (1.25.0)
Ruby LSP Settings
Click to expand
Workspace
{}
User
{
"enabledFeatures": {
"codeActions": true,
"diagnostics": true,
"documentHighlights": true,
"documentLink": true,
"documentSymbols": true,
"foldingRanges": true,
"formatting": true,
"hover": true,
"inlayHint": true,
"onTypeFormatting": true,
"selectionRanges": true,
"semanticHighlighting": true,
"completion": false,
"codeLens": true,
"definition": true,
"workspaceSymbol": true,
"signatureHelp": true,
"typeHierarchy": true
},
"featuresConfiguration": {},
"addonSettings": {},
"rubyVersionManager": {
"identifier": "auto"
},
"customRubyCommand": "",
"formatter": "rubocop_internal",
"linters": [
"rubocop_internal"
],
"bundleGemfile": "",
"testTimeout": 30,
"pullDiagnosticsOn": "both",
"useBundlerCompose": false,
"bypassTypechecker": false,
"rubyExecutablePath": "sh",
"indexing": {},
"erbSupport": true,
"featureFlags": {},
"sigOpacityLevel": "1"
}
Also: Remote – Dev Containers on linux-arm64 (Apple Silicon host), explorer.autoReveal left at its default (true). All versions above are the latest stable releases, and the same code is still on main.
Reproduction steps
- Start the Ruby LSP in VS Code by opening a Rails project.
- In the Explorer, expand the Dependencies view so its contents are visible.
- Create an untitled document with File → New Text File (
Untitled-N). - Click a saved file, then click back to the untitled tab.
- The extension host's memory starts climbing straight away (591 MB → 1.46 GB in about 20 seconds) until V8 aborts it with
JavaScript heap out of memory. The VS Code window hangs in the meantime.
No other extensions are needed. We first hit this through Oracle SQL Developer worksheets, which are also untitled documents.
It also triggers if an untitled editor is active while the Dependencies view is hidden and you then show the view, because currentVisibleItem is remembered and revealed in treeVisibilityDidChange.
Expected: nothing happens, because an untitled document can't be part of any gem.
Code snippet or error message
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
Extension Host Process exited with code: null, signal: SIGABRT.
Heap-allocation profile of the extension host while memory was growing:
-- live allocations by extension
65.3% (vscode core)
34.3% shopify.ruby-lsp
-- top allocating functions
49.0% (vscode core) :: r extensionHostProcess.js:28
34.1% shopify.ruby-lsp :: getParent extension.js:79
15.5% (vscode core) :: (anon) extensionHostProcess.js:294
0.3% (vscode core) :: _resolveParent extensionHostProcess.js:294
0.3% (vscode core) :: joinPath extensionHostProcess.js:28
The cause seems to be in vscode/src/dependenciesTree.ts. activeEditorDidChange (line 92) wraps any active editor's URI in a GemFilePath and reveals it, whatever its scheme. TreeView.reveal then calls getParent (line 55) until it returns undefined, and getParent only stops when parentUri.path === path.parse(parentUri.path).root:
getParent(element: BundlerTreeNode): vscode.ProviderResult<BundlerTreeNode> {
const parentUri = vscode.Uri.joinPath(element.resourceUri, "..");
const rootPath = path.parse(parentUri.path).root;
if (parentUri.path === rootPath) {
return undefined;
}
// ...otherwise returns a gem root or a new GemDirectoryPath(parentUri)
}
For a file: URI the walk ends at /. An untitled document's path has no leading /, so rootPath is "", and joining .. never produces "":
/workspaces/app/db/x.sql -> /workspaces/app/db -> /workspaces/app -> /workspaces -> / (stops)
Untitled-2 -> . -> .. -> ../.. -> ../../.. -> ../../../.. -> ... (never stops)
(The trace comes from running the same check with path.posix.join in Node. The profile above matches it.)
Possible fix: skip non-file: URIs in activeEditorDidChange, ideally only revealing URIs under one of gemRootFolders. As a safety net, getParent could return undefined when the parent path isn't absolute or equals the current path:
const uri = editor?.document.uri;
if (!uri || uri.scheme !== "file") {
return;
}
Workaround: hide the Dependencies view, or set "explorer.autoReveal": false.
- Lingua principale
- Ruby
- Stelle
- 2k
- Fork
- 284
- Merge medio
- 11h 11m
- PR unite (30g)
- 10
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Shopify/ruby-lsp
-
Linux file system with square brackets in the directory/filename are failing to be indexedForse già presa @andriytyurnikov l’ha presa 207 giorni fa. Apertabug help-wanted pinned
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Shopify/ruby-lsp#3503 · 8 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Rename of a class moves its file and leaves require_relative on the old pathForse già presa @efegokdemir l’ha presa 4 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 22/100
I maintainer di solito rispondono entro 1 giorno
-
Paths need to be escapedForse già presa @Vraiment l’ha presa 8 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
I maintainer di solito rispondono entro 1 giorno
-
check-labels intermittently fails due to the organization IP allow listForse già presa @soutaro l’ha presa 11 giorni fa. Aperta
Shopify/ruby-lsp#4219 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
'Kernel#require': cannot load such file -- bundler (LoadError)Forse già presa @efegokdemir l’ha presa 3 giorni fa. Apertabug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di Shopify/ruby-lsp
Issue simili
-
ETQ admin, le select de condition d'une annotation répétable liste les champs publics deux foisAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
demarche-numerique/demarche.numerique.gouv.fr#14301 ·
I maintainer di solito rispondono entro 1 giorno
-
needs verification
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 6 giorni
-
L: java:gradle L: java:maven L: ruby:bundler T: bug 🐞
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
dependabot/dependabot-core#16547 ·
I maintainer di solito rispondono entro 1 giorno
-
Properties named open and close collide with query delimiter tokensForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 8 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno