Skip to content

CLI keeps a dead void readFileSync with a comment claiming a disk self-check it does not perform #120

@nficano

Description

@nficano

Category: clean-code Severity: minor
Location: packages/sdk/src/cli.ts:163-165

What

void readFileSync; merely references the imported function and performs no filesystem access; the comment claims it 'ensures the package can find its own version on disk,' which is false. readFileSync is otherwise unused.

Evidence

// Self-check ensures the package can find its own version on disk; no-op
// otherwise, but kept here for readers of the CLI to reason about packaging.
void readFileSync;

Proposed fix

Remove the unused import and misleading comment, or implement an actual version self-check that reads package.json.

Acceptance criteria

  • The CLI has no dead void readFileSync; statement and any retained comment is accurate.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions