[PR #84] [CLOSED] fix: make vinext bin resilient in workspace installs #295

New issue

Closed

opened 2026-05-06 12:39:01 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 12:39:01 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/84
Author: @ruskzane
Created: 2/26/2026
Status: ❌ Closed

Base: main ← Head: fix/vinext-prepare-build

📝 Commits (1)

171ae15 fix: make vinext bin resilient in workspace installs

📊 Changes

2 files changed (+24 additions, -2 deletions)

View changed files

➕ packages/vinext/cli.js (+20 -0)
📝 packages/vinext/package.json (+4 -2)

📄 Description

What

When installing the monorepo with pnpm, fixture/example workspaces that depend on vinext can emit warnings because the package bin points at dist/cli.js before the build runs.

This change adds a small top-level cli.js wrapper (checked into the repo and included in npm files) and updates bin to point to it. The wrapper forwards to dist/cli.js when present, and otherwise prints a clear instruction to run the build.

Why

Avoids ENOENT warnings during workspace installs
Keeps published package behavior the same (prepack still builds dist)

Notes

prepare still runs tsc so dist exists after install.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/84 **Author:** [@ruskzane](https://github.com/ruskzane) **Created:** 2/26/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/vinext-prepare-build` --- ### 📝 Commits (1) - [`171ae15`](https://github.com/cloudflare/vinext/commit/171ae1542ea0236387ed45f27d776abfc5a8f9d6) fix: make vinext bin resilient in workspace installs ### 📊 Changes **2 files changed** (+24 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `packages/vinext/cli.js` (+20 -0) 📝 `packages/vinext/package.json` (+4 -2) </details> ### 📄 Description ### What When installing the monorepo with pnpm, fixture/example workspaces that depend on `vinext` can emit warnings because the package `bin` points at `dist/cli.js` before the build runs. This change adds a small top-level `cli.js` wrapper (checked into the repo and included in npm files) and updates `bin` to point to it. The wrapper forwards to `dist/cli.js` when present, and otherwise prints a clear instruction to run the build. ### Why - Avoids ENOENT warnings during workspace installs - Keeps published package behavior the same (prepack still builds dist) ### Notes - `prepare` still runs `tsc` so dist exists after install. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>