Skip to content

Conversation

@asvishnyakov
Copy link
Member

@asvishnyakov asvishnyakov commented Sep 2, 2025

Fixes #2467
Caused by #2415

@asvishnyakov asvishnyakov force-pushed the fix/installation-from-sources branch 4 times, most recently from fb2a591 to 7a65841 Compare September 3, 2025 00:18
@asvishnyakov asvishnyakov force-pushed the fix/installation-from-sources branch 3 times, most recently from 65a2d3a to 86b0e9b Compare September 3, 2025 01:04
@asvishnyakov
Copy link
Member Author

asvishnyakov commented Sep 3, 2025

@francisjervis @andrePankraz You can use this branch until it gets merged.

@hayescode @sandangel I’d like to create another PR before merging this one, so I can test the publish workflow by running it manually with a test PyPi key. Until then, I’ll leave this as a draft. Also:

uv run --no-project feels like black magic to me, but it was needed because otherwise lint-backend fails as pnpm isn’t installed here. Instead of adding it (and increasing the overall build time), I used --no-install-project --no-editable, which prevents default project installation in editable mode by uv and, therefore, the automatic build of the frontend and copilot. Keep this in mind if you add or change workflows.

@francisjervis
Copy link

francisjervis commented Sep 3, 2025

@asvishnyakov thanks for looking at this.

EDIT 2: this does work. @asvishnyakov's helpful comment below should probably be added to the docs for a clear end-to-end explanation of how to use a forked version in a python project.

@asvishnyakov
Copy link
Member Author

asvishnyakov commented Sep 3, 2025

@francisjervis The issue seems to occur when you add Chainlit from source code, whether it’s a fork, the Chainlit repo, or a local folder. No CI is involved here - the only relevant changes are the ones related to the build hook in build.py and pyproject.toml. I believe pip won’t update and rebuild dependency from your repo unless you specify --force-reinstall or remove and then re-add the dependency. You might want to try it in a new empty project first to confirm it works.

@asvishnyakov
Copy link
Member Author

@francisjervis Does it fixed issue for you?

@francisjervis
Copy link

@francisjervis Does it fixed issue for you?

@asvishnyakov thanks, it turned out to be a sneaky (IDE settings vs terminal) wrong venv issue lol

@asvishnyakov asvishnyakov changed the title fix: build frontend and copilot on installation from source code fix: build frontend and copilot on installation from source code (fork, git, local folder) Sep 11, 2025
@asvishnyakov asvishnyakov marked this pull request as ready for review September 13, 2025 00:29
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 13, 2025
@dosubot dosubot bot added the dev-tooling Relating to developer/contributor toolings. label Sep 13, 2025
@asvishnyakov asvishnyakov added this pull request to the merge queue Sep 13, 2025
Merged via the queue into main with commit 66ec364 Sep 13, 2025
9 checks passed
@asvishnyakov asvishnyakov deleted the fix/installation-from-sources branch September 13, 2025 01:14
asvishnyakov added a commit to asvishnyakov/chainlit that referenced this pull request Sep 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-tooling Relating to developer/contributor toolings. size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to install Chainlit from source code because github/frontend dist dirs not found

4 participants