Skip to content

feat: add preflight command for development environment check#852

Closed
caoergou wants to merge 1 commit intodifferent-ai:devfrom
caoergou:feat/preflight-check
Closed

feat: add preflight command for development environment check#852
caoergou wants to merge 1 commit intodifferent-ai:devfrom
caoergou:feat/preflight-check

Conversation

@caoergou
Copy link
Copy Markdown
Contributor

Summary

  • Add pnpm preflight command to validate development environment
  • Check required tools: Node.js (18+), pnpm (10.27.0+), Bun (1.3.9+)
  • Check optional tools: cargo, opencode, tauri (for desktop development)
  • Provide actionable hints when tools are missing or outdated

Changes

File Type Description
scripts/preflight.mjs New Preflight check script with colored output
package.json Modified Add preflight script
README.md Modified Document preflight command

Features

  • Required tools check: Node.js, pnpm, Bun with version validation
  • Optional tools check: cargo, opencode, tauri for desktop development
  • Version comparison: Warn if tool version is below minimum required
  • Color support: Respects NO_COLOR and CI environments
  • Actionable hints: Provides installation instructions for missing tools

Test Plan

# Run preflight check
pnpm preflight

# Expected output:
# ✓ node: v22.x.x
# ✓ pnpm: 10.27.x
# ✓ bun: 1.3.x
# ...
# ✓ Preflight passed

This is split from the original PR #788 as suggested by @OmarMcAdam.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 Apr 7, 2026 6:12am
openwork-software Ready Ready Preview, Comment Apr 7, 2026 6:12am

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Mar 12, 2026

@caoergou is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

@caoergou
Copy link
Copy Markdown
Contributor Author

Hi @OmarMcAdam — since you mentioned you check PRs daily on #854, wanted to flag that this PR (#852 preflight + #853 verify) are part of the same dev-workflow improvement theme as the CI typecheck in #854.

Happy to consolidate or adjust scope if that'd be easier to review together!

@jcllobet
Copy link
Copy Markdown
Collaborator

jcllobet commented Apr 4, 2026

@caoergou can you solve the merge conflicts for preflight?

Regarding the type validation in #853 , it may be trickier as we would have to change or skip a lot of things with the TS linter atm but i'll defer to @OmarMcAdam for his opinion.

- Add scripts/preflight.mjs to validate required tools (Node.js, pnpm, Bun)
- Check optional tools (cargo, opencode, tauri) for desktop development
- Provide actionable hints when tools are missing or outdated
- Support NO_COLOR and CI environments
- Update README.md with preflight command documentation

Usage: pnpm preflight
@caoergou
Copy link
Copy Markdown
Contributor Author

caoergou commented Apr 7, 2026

Hi @jcllobet, the merge conflicts have been resolved! The branch has been rebased onto the current dev branch.

@src-opn src-opn closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants