refactor(cli): add resolver utils

[fengmk2]

TL;DR

Refactored CLI utilities and added command help message tests.

What changed?

• Created a new utils.ts file with common utility functions for resolving binaries and default environment variables
• Refactored all command modules (lib, fmt, lint, test, doc, vite) to use the new utility functions
• Added snapshot tests for command help messages
• Improved semver version replacement in test utilities to handle more patterns
• Added new test files for command helper functionality

How to test?

1. Run the CLI commands with -h flag to verify help messages are displayed correctly:

   vite lib -h
   vite fmt -h
   vite lint -h
   vite build -h
   vite test -h
   

2. Run the snapshot tests to ensure command help messages are captured correctly

Why make this change?

This change improves code maintainability by reducing duplication across command modules. The common utility functions centralize binary resolution and environment variable handling, making the codebase more DRY. The added tests for command help messages ensure that CLI documentation remains accurate and consistent as the project evolves.

[fengmk2]

• chore(cli): display ASCII-art logo #183 : 2 dependent PRs (#184 , #185 )
• feat(global): improve package manager selection and add dependency installation #179
• chore(cli): add fmt to ready task #178
• chore(cli): temporarily fix pnpm install warnning messages #177
• fix(cli): add "lib" to builtin commands #174
• refactor(cli): add resolver utils #173 👈 (View in Graphite)
• feat(cli): add doc command for VitePress documentation #172
• fix(cli): add missing .npmrc #175
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull Request Overview

This PR refactors CLI utilities by extracting common functionality into a shared utils module and adds comprehensive test coverage for command help messages.

• Created centralized utility functions for binary resolution and environment variable defaults
• Refactored all command modules to use the shared utilities, reducing code duplication
• Added snapshot tests for CLI command help messages to ensure documentation consistency

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/tools/src/utils.ts	Enhanced semver replacement regex to handle more version patterns
packages/tools/src/tests/utils.spec.ts	Added test cases for new semver pattern matching
packages/tools/src/tests/snapshots/utils.spec.ts.snap	Updated snapshots to reflect improved semver replacement
packages/cli/src/utils.ts	New utility module with shared resolver and environment defaults
packages/cli/src/vite.ts	Refactored to use shared utilities, removing duplicate resolver code
packages/cli/src/test.ts	Refactored to use shared utilities and environment defaults
packages/cli/src/lint.ts	Refactored to use shared utilities and environment defaults
packages/cli/src/lib.ts	Refactored to use shared utilities and environment defaults
packages/cli/src/fmt.ts	Refactored to use shared utilities and environment defaults
packages/cli/src/doc.ts	Refactored to use shared utilities and environment defaults
packages/cli/snap-tests/command-lib/snap.txt	Updated snapshot to use normalized semver format
packages/cli/snap-tests/command-helper/*	New test suite for comprehensive CLI help message validation

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Brooooooklyn]

cursor review

[Brooooooklyn]

@codex review

[chatgpt-codex-connector]

Codex Review: Here are some suggestions.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

[fengmk2]

Merge activity

• Sep 22, 1:58 AM UTC: @fengmk2 merged this pull request with Graphite.