Skip to content

analyzePackageVersions is fire-and-forget: missing await at line 861 #11

Description

@CodeDeficient

What

is called without await at ai-slop-detector.ts:861 inside analyzeFile, which is async. But analyzePackageVersions is also async and internally calls await this.getNpmPackageAge(...) to hit the npm registry.

Impact

The report is generated and the process exits before the HTTP fetches resolve. fresh_package_version issues are never pushed to this.issues and never appear in output.

Fix

Change line 861 from:

this.analyzePackageVersions(filePath, content);

to:

await this.analyzePackageVersions(filePath, content);

Location

ai-slop-detector.ts:861

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions