Skip to content

fix: remove agent from pnpm workspace to fix Render build#54

Merged
GeneralJerel merged 3 commits intomainfrom
fix/native-python-agent
Mar 25, 2026
Merged

fix: remove agent from pnpm workspace to fix Render build#54
GeneralJerel merged 3 commits intomainfrom
fix/native-python-agent

Conversation

@GeneralJerel
Copy link
Collaborator

@GeneralJerel GeneralJerel commented Mar 25, 2026 •
edited
Loading

Summary

  • Remove apps/agent/package.json so the Python agent is no longer a pnpm workspace member
  • Update root package.json to run agent dev server via direct shell command instead of Turbo
  • Remove agent entry from pnpm-workspace.yaml

Problem

PR #53 switched the agent from Docker to native FastAPI but left apps/agent/package.json with a postinstall: "uv sync" hook. When Render's frontend Node service runs pnpm install, it installs all workspace members — triggering uv sync in an environment where uv doesn't exist, crashing the build.

Solution

Match the Shadify pattern: keep the Python agent in apps/agent/ but outside the pnpm workspace entirely. Without a package.json, pnpm ignores it. The agent's Render service already has its own buildCommand in render.yaml.

Test plan

  • pnpm install succeeds without triggering any uv commands
  • pnpm ls shows no @repo/agent workspace member
  • pnpm dev:agent starts uvicorn on port 8123
  • Render frontend build passes

🤖 Generated with Claude Code
@GeneralJerel
fix: remove agent postinstall that breaks frontend build on Render
20cd544 
The postinstall "uv sync" hook runs during pnpm install on the frontend Node service where uv is not available. The agent service has its own buildCommand in render.yaml. Also updates dev script to match the new native FastAPI setup and drops the unused langgraph-cli dependency.
@GeneralJerel
fix: remove agent from pnpm workspace to fix Render build
f64850d 
The agent's package.json made it a pnpm workspace member, so pnpm install on the Node frontend service triggered "uv sync" in an environment where uv doesn't exist — breaking Render deploys. Remove apps/agent/package.json entirely (matching the Shadify pattern) and run the agent dev server via a direct shell command in the root package.json instead of through Turbo.
@GeneralJerel
fix: remove Python/uv from CI and fix smoke test SIGTERM failure
70f9184 
The agent is no longer a pnpm workspace member, so setup-python and setup-uv are unnecessary in CI. Their post-job cache step failed because uv never ran. Also drop Python from the smoke matrix (8→4 jobs) and add `wait` after killing the frontend process to suppress the SIGTERM exit code that was failing the smoke step.
@GeneralJerel GeneralJerel force-pushed the fix/native-python-agent branch from 62fcea9 to 70f9184 Compare March 25, 2026 22:26
@GeneralJerel GeneralJerel merged commit 259e92c into main Mar 25, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@GeneralJerel