Skip to content

Fix Installation Circular Dependency Issue #317

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AmmarFahmy
Copy link

Fix Installation Circular Dependency Issue

Problem

When running yarn install, users encounter a circular dependency error because the @opencanvas/agents package tries to run yarn turbo build during installation via its postinstall script. This creates a circular dependency since:

  1. The postinstall script tries to use Turbo to build the package
  2. But Turbo itself isn't fully available during the installation process
  3. This results in a "Cannot find module 'turbo'" error that blocks installation

Solution

This PR fixes the installation process by:

  1. Removing the problematic postinstall script from apps/agents/package.json that was causing the circular dependency
  2. Updating the main README.md to include an explicit yarn build step after installation
  3. Ensuring the build process is run manually after dependency installation instead of during the installation itself

This breaks the circular dependency chain and allows for a smooth installation experience for new contributors.

Changes

  • Removed the postinstall script from apps/agents/package.json
  • Updated README.md to add the explicit build step after yarn install

These changes ensure that Open Canvas can be installed without errors and maintains a clear, straightforward setup process for new users.

Copy link

vercel bot commented Mar 23, 2025

@AmmarFahmy is attempting to deploy a commit to the LangChain Team on Vercel.

A member of the Team first needs to authorize it.

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