Skip to content

docs(linter/plugins): alter comments on FILE_CONTEXT used in ESLint-compat Context shim #15605

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
Nov 11, 2025
+2 −10
Merged

docs(linter/plugins): alter comments on FILE_CONTEXT used in ESLint-compat Context shim #15605

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
12 changes: 2 additions & 10 deletions apps/oxlint/src-js/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,8 @@ let cwd: string | null = null;
//
// Only `cwd` property and `extends` method are available in `createOnce`, so only those are implemented here.
// All other getters/methods throw, same as they do in main implementation.
//
// See `FILE_CONTEXT` in `plugins/context.ts` for details of all the getters/methods.
const FILE_CONTEXT: FileContext = freeze({
get filename(): string {
throw new Error('Cannot access `context.filename` in `createOnce`');
Expand All @@ -152,9 +154,6 @@ const FILE_CONTEXT: FileContext = freeze({
throw new Error('Cannot access `context.physicalFilename` in `createOnce`');
},

/**
* Current working directory.
*/
get cwd(): string {
// Note: We can allow accessing `cwd` in `createOnce`, as it's global
if (cwd === null) cwd = process.cwd();
Expand All @@ -173,13 +172,6 @@ const FILE_CONTEXT: FileContext = freeze({
throw new Error('Cannot access `context.settings` in `createOnce`');
},

/**
* Create a new object with the current object as the prototype and
* the specified properties as its own properties.
* @param extension - The properties to add to the new object.
* @returns A new object with the current object as the prototype
* and the specified properties as its own properties.
*/
extend(this: FileContext, extension: Record<string | number | symbol, unknown>): FileContext {
// Note: We can allow calling `extend` in `createOnce`, as it involves no file-specific state
return freeze(ObjectAssign(ObjectCreate(this), extension));
Expand Down
Loading