Newspaper rule does not catch functions implemented above and used within a class

Repository metrics

Stars

175 stars (175 stars)

PR merge metrics

No merged PRs in 30d (No merged PRs in 30d)

Description

function doStuff() {};

class StuffDoer {
  myMethod() {
    doStuff();
  }
}

class StuffDoer {
  myMethod() {
    doStuff();
  }
}

function doStuff() {};

Contributor guide

Research direction

Inspect the newspaper rule's AST walker and check why it only detects functions declared below classes, not above. Look at the rule's source to see if it uses a single pass traversal or a two pass approach.

Tech stack

typescript

Domain

tooling

Issue type

Bug

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

2

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

1-3 hours

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

Fresh

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

Clear

Prerequisites

TypeScriptTSLint rule development

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

70