shazhou/united-workforce
8
0
Fork 0
Code Issues 8 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix: bootstrap adds Step 0 environment pre-flight check #113

Merged
xiaomo merged 2 commits from fix/112-bootstrap-preflight into main 2026-06-05 14:34:13 +00:00
Conversation 1 Commits 2 Files Changed 2
+49 -11
xiaoju commented 2026-06-05 14:11:41 +00:00
Owner
Copy Link
Copy Source

What

Add Step 0 environment pre-flight check to bootstrap prompt.

Why

New agents hit PATH cascade failures when pnpm/node/hermes are missing or not in PATH (#112).

Changes

  • Add Step 0 with 4 checks: node, pnpm/npm, global bin PATH, hermes CLI — each with FIX instructions
  • Install commands: npm → pnpm (with npm fallback)
  • hermes PATH guidance consolidated into Step 0

Ref

Fixes #112

## What Add Step 0 environment pre-flight check to bootstrap prompt. ## Why New agents hit PATH cascade failures when pnpm/node/hermes are missing or not in PATH (#112). ## Changes - Add Step 0 with 4 checks: node, pnpm/npm, global bin PATH, hermes CLI — each with FIX instructions - Install commands: npm → pnpm (with npm fallback) - hermes PATH guidance consolidated into Step 0 ## Ref Fixes #112
xiaoju added 1 commit 2026-06-05 14:11:41 +00:00
fix: bootstrap adds Step 0 environment pre-flight check
CI / check (pull_request) Successful in 3m44s
Details
3be92bfac2 
- Node.js, pnpm/npm, global bin PATH, hermes CLI checks with FIX instructions
- Agent must pass all checks before proceeding to install
- Install commands changed from npm to pnpm (with npm fallback)
- hermes PATH guidance moved from Step 1 to Step 0

Fixes #112
xiaoju added 1 commit 2026-06-05 14:11:52 +00:00
chore: add changeset for #112
CI / check (pull_request) Successful in 6m2s
Details
0ad10b9b6d
xiaomo approved these changes 2026-06-05 14:34:10 +00:00
xiaomo left a comment
Owner
Copy Link
Copy Source

LGTM ✅

Step 0 预检加得好 — 4 项检查每项都附了 FIX 指引，"All checks must pass before continuing" 卡住了不看的人。hermes PATH 的 venv/symlink 两种方案从 Step 1 提前到了预检里，逻辑更顺。pnpm 优先 + npm fallback 也合理。

LGTM ✅ Step 0 预检加得好 — 4 项检查每项都附了 FIX 指引，"All checks must pass before continuing" 卡住了不看的人。hermes PATH 的 venv/symlink 两种方案从 Step 1 提前到了预检里，逻辑更顺。pnpm 优先 + npm fallback 也合理。
xiaomo merged commit a33f12c74f into main 2026-06-05 14:34:13 +00:00
xiaomo deleted branch fix/112-bootstrap-preflight 2026-06-05 14:34:13 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
xiaomo
Labels
Clear labels
bug
Something is not working
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/united-workforce#113
Delete Branch "fix/112-bootstrap-preflight"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?