Skip to content

docs(linter/plugins): add comment about deprecated ScopeManager methods #19363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
graphite-app merged 1 commit into from
Feb 13, 2026
+8 −5
Merged

docs(linter/plugins): add comment about deprecated ScopeManager methods #19363

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 8 additions & 5 deletions apps/oxlint/src-js/plugins/scope.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,11 +260,14 @@ export function resetScopeManager() {
tsScopeManager = null;
}

/**
* @see https://eslint.org/docs/latest/developer-guide/scope-manager-interface#scopemanager-interface
*/
// This is a wrapper around `@typescript-eslint/scope-manager` package's `ScopeManager` class.
// We want to control what APIs are exposed to the user to limit breaking changes when we switch our implementation.
// Wrapper around `@typescript-eslint/scope-manager` package's `ScopeManager` class.
//
// We only expose the methods and properties which ESLint lists as part of the `ScopeManager` interface:
// https://eslint.org/docs/latest/extend/scope-manager-interface#scopemanager-interface
// This does NOT include deprecated methods and properties.
// In future, we'll re-implement the scope manager using scopes data from `oxc_semantic`, and remove the dependency
// on `@typescript-eslint/scope-manager`. That will be easier with a more minimal API surface.
// If we find real-world usage of deprecated methods/properties in ESLint plugins, we can add them to the API surface.
//
// Only one file is linted at a time, so we can reuse a single object for all files.
//
Expand Down
Loading