Rule proposal: `no-this-outside-of-class`

Repository metrics

Stars

5,022 stars (5,022 stars)

PR merge metrics

Avg merge 1d 16h (Avg merge 1d 16h)399 merged PRs in 30d (399 merged PRs in 30d)

Description

function foo () {
  this.bar;
}
function Foo(name) {
  this.name = name;
}
class Foo {
  foo() {
    function bar() {
      this.baz;
    }
  }
}

class Foo {
  foo() {
    this.name;
  }
}

class Foo {
  foo() {
    const bar = () => {
      this.name;
    }
  }
}

Contributor guide

Research direction

Review existing rules in the repository (e.g., rules/no array for each.js) to understand the pattern for implementing a new rule. Identify the AST node types for `this` expressions and ensure the rule only flags `this` outside of class bodies. Write corresponding tests covering both fail and pass cases from the issue. Update the plugin's documentation and rule list in readme.md.

Tech stack

javascript

Domain

tooling

Issue type

Feature

DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.

3

Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.

1-2 days

Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.

Active

ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.

Clear

Prerequisites

ESLint rule developmentJavaScript `this` bindingAST traversal

Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.

65