Skip to content

scripts/setup-env.ts imports an unused glob dependency with a leftover TODO-style comment #1133

Description

@vjuliaife

Description:
scripts/setup-env.ts imports globSync from "glob" with an inline comment reading "we might not have glob installed, but we can use fs.readdirSync recursively or just hardcode apps/*" — the import is never used since the script falls back to a hand-rolled findEnvExamples recursive walker. This is confusing dead code that looks unfinished and adds an unnecessary dependency.

Acceptance Criteria:

  • Remove the unused import { globSync } from "glob" line and the leftover comment
  • Verify "glob" is actually a declared dependency or remove it from package.json if no longer needed anywhere
  • Run npm run setup-env against a clean apps/ tree to confirm behavior is unchanged
  • Add a short header comment explaining the recursive .env.example discovery approach

Relevant Files:

  • scripts/setup-env.ts — unused glob import with stale reasoning comment at top of file

Metadata

Metadata

Assignees

No one assigned

    Labels

    dxDeveloper experience / docs / namingtoolingAuto-created by repotool publish-folder

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions