Skip to content

Add missing portless dependency#143

Merged
ctate merged 6 commits intovercel-labs:mainfrom
michalczaplinski:fix-missing-portless
Mar 2, 2026
Merged

Add missing portless dependency#143
ctate merged 6 commits intovercel-labs:mainfrom
michalczaplinski:fix-missing-portless

Conversation

@michalczaplinski
Copy link
Copy Markdown
Contributor

portless was added in #133 but that PR failed to add it to dependencies which causes pnpm dev to fail:

➤ pnpm dev

> json-render@ dev /Users/czapla/Projects/json-render
> turbo run dev --concurrency 15

• turbo 2.7.4
• Packages in scope: @json-render/codegen, @json-render/core, @json-render/react, @json-render/react-native, @json-render/react-pdf, @json-render/remotion, @json-render/shadcn, @repo/eslint-config, @repo/typescript-config, @repo/ui, com.example.json-render-demo, com.example.json-render-fullpage-demo, example-chat, example-dashboard, example-no-ai, example-react-native, example-react-pdf, example-remotion, stripe-app-api, web
• Running dev in 20 packages
• Remote caching disabled
┌─ stripe-app-api#dev > cache bypass, force executing c48a48aaab5b6e98 

> stripe-app-api@0.1.0 dev /Users/czapla/Projects/json-rende
r/examples/stripe-app/api
> portless stripe-api-demo.json-render next dev

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ stripe-app-api#dev ──
┌─ example-chat#dev > cache bypass, force executing d2da43018e6643f9 

> example-chat@0.1.0 dev /Users/czapla/Projects/json-render/
examples/chat
> portless chat-demo.json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ example-chat#dev ──
┌─ web#dev > cache bypass, force executing c48bb87577100e14 

> web@0.1.0 dev /Users/czapla/Projects/json-render/apps/web
> portless json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ web#dev ──
┌─ example-react-pdf#dev > cache bypass, force executing c158735ab4126bff 

> example-react-pdf@0.1.0 dev /Users/czapla/Projects/json-re
nder/examples/react-pdf
> portless react-pdf-demo.json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ example-react-pdf#dev ──
┌─ example-remotion#dev > cache bypass, force executing ffef47a78d738465 

> example-remotion@0.1.0 dev /Users/czapla/Projects/json-ren
der/examples/remotion
> portless remotion-demo.json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ example-remotion#dev ──
┌─ example-no-ai#dev > cache bypass, force executing d5a661422ea54e7b 

> example-no-ai@0.1.0 dev /Users/czapla/Projects/json-render
/examples/no-ai
> portless no-ai-demo.json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
└─ example-no-ai#dev ──
┌─ @json-render/react#dev > cache bypass, force executing 0a6392aa0a32ef33 

> @json-render/react@0.8.0 dev /Users/czapla/Projects/json-r
ender/packages/react
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/react#dev ──
┌─ @json-render/react-pdf#dev > cache bypass, force executing e9f9413fbb243a85 

> @json-render/react-pdf@0.8.0 dev /Users/czapla/Projects/js
on-render/packages/react-pdf
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/react-pdf#dev ──
┌─ @json-render/codegen#dev > cache bypass, force executing e755a8379b732d84 

> @json-render/codegen@0.8.0 dev /Users/czapla/Projects/json
-render/packages/codegen
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/codegen#dev ──
┌─ @json-render/remotion#dev > cache bypass, force executing 5631762af253a3e3 

> @json-render/remotion@0.8.0 dev /Users/czapla/Projects/jso
n-render/packages/remotion
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/remotion#dev ──
┌─ @json-render/core#dev > cache bypass, force executing c9b0cab78f18e9b5 

> @json-render/core@0.8.0 dev /Users/czapla/Projects/json-re
nder/packages/core
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/core#dev ──
┌─ @json-render/react-native#dev > cache bypass, force executing fe8ea87ba199663c 

> @json-render/react-native@0.8.0 dev /Users/czapla/Projects
/json-render/packages/react-native
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/react-native#dev ──
┌─ @json-render/shadcn#dev > cache bypass, force executing 9fa7c1e3f528531b 

> @json-render/shadcn@0.8.0 dev /Users/czapla/Projects/json-
render/packages/shadcn
> tsup --watch

 ELIFECYCLE  Command failed.
└─ @json-render/shadcn#dev ──
┌─ example-dashboard#dev > cache bypass, force executing a1f46c49997c6895 

> example-dashboard@0.1.0 dev /Users/czapla/Projects/json-re
nder/examples/dashboard
> portless dashboard-demo.json-render next dev --turbopack

sh: portless: command not found
 ELIFECYCLE  Command failed.
command finished with error: command (/Users/czapla/Projects
/json-render/examples/dashboard) /Users/czapla/Library/pnpm/
.tools/pnpm/10.29.3/bin/pnpm run dev exited (1)
└─ example-dashboard#dev ──
example-dashboard#dev: command (/Users/czapla/Projects/json-render/examples/dashboard) /Users/czapla/Library/pnpm/.tools/pnpm/10.29.3/bin/pnpm run dev exited (1)

 Tasks:    0 successful, 14 total
Cached:    0 cached, 14 total
  Time:    408ms 
Failed:    example-dashboard#dev

 ERROR  run failed: command  exited (1)
 ELIFECYCLE  Command failed with exit code 1.

This PR adds it to package.json.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Feb 23, 2026

@michalczaplinski is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@ctate
Copy link
Copy Markdown
Collaborator

ctate commented Feb 24, 2026

@michalczaplinski Thanks for the report

Adding portless as a workspace dependency conflicts with the project convention in AGENTS.md and with already-running global portless daemons

I think the best approach might be to add a predev script instead. That way portless stays global, but new contributors get a clear error. Something like:

"predev": "command -v portless >/dev/null 2>&1 || (echo '\\nportless is required but not installed. Run: npm i -g portless\\nSee: https://github.com/vercel-labs/portless\\n' && exit 1)"

@michalczaplinski
Copy link
Copy Markdown
Contributor Author

Hi @ctate

Yeah, that makes sense. I've updated the main package.json and the individual packages to include the predev script.

@ctate ctate merged commit 8968bd6 into vercel-labs:main Mar 2, 2026
3 of 6 checks passed
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