Skip to content

chore(live-test): point missing-CLI hint at pnpm exec vp - #94

Merged
altaywtf merged 1 commit into
mainfrom
fix/vp-build-hint
Oct 4, 2026
Merged

altaywtf merged 1 commit into
mainfrom
fix/vp-build-hint

Conversation

@altaywtf

@altaywtf altaywtf commented Oct 4, 2026

Copy link
Copy Markdown
Member

Summary

Running node live-test/run-live-test.ts without a packed CLI now tells you to run pnpm exec vp run build, which works from a fresh checkout. The old hint said vp run build, but vp is the pinned vite-plus devDependency and isn't on PATH, so following it failed with command not found.

  • Only the hint text changes; the live:test:* package scripts already build before running the harness.
  • live-test/ is not in the npm package, so this is a non-release chore.

Verification

  • Fresh worktree without dist/ or global vp: before the fix, running the hinted command exits 127 and builds nothing; after it, the hinted command produces dist/taizn.mjs.
  • pnpm exec vp run verify passes (128 tests).

Notes

Live Tizen scripts were not run.


Written by an agent (Claude Code, Opus 5.5)

Copilot AI balanced review requested due to automatic review settings October 4, 2026 15:42
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@altaywtf
altaywtf merged commit e49fe28 into main Oct 4, 2026
6 of 7 checks passed
@altaywtf
altaywtf deleted the fix/vp-build-hint branch October 4, 2026 15:45
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.

2 participants