Skip to content

Conversation

@durran
Copy link
Member

@durran durran commented Oct 28, 2025

langchainjs updated their directory structure prior to the 1.0 release and also moved to pnpm from yarn.

Requires langchain-ai/langchainjs#9276

Once the langchain PR is merged I will update the branch we are pointing to.

@durran durran force-pushed the NODE-7278 branch 9 times, most recently from a819a13 to cdbea2f Compare October 28, 2025 17:52
@durran durran force-pushed the NODE-7278 branch 2 times, most recently from 6c4d331 to aab4af5 Compare October 28, 2025 19:05
@durran durran changed the title test(NODE-7278): update langchainjs location test(NODE-7278): fix langchainjs tests Oct 28, 2025
@durran durran force-pushed the NODE-7278 branch 6 times, most recently from 397d164 to eb142a5 Compare October 28, 2025 21:07
@durran durran marked this pull request as ready for review October 29, 2025 12:39
- command: attach.xunit_results
params:
file: src/langchain-js/langchainjs/libs/langchain-mongodb/results.xml
file: src/langchain-js/langchainjs/libs/providers/langchain-mongodb/results.xml
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Langchain moved locations


setup_langchain_integration() {
cd libs/langchain-mongodb
pnpm install
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Langchain is using pnpm with corepack now instead of yarn.

setup_langchain_integration

yarn test:int --reporters=default --reporters=jest-junit
pnpm test:int --reporter=default --reporter=junit --outputFile=./results.xml
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using vitest instead of jest, reporter options are different.

@Jibola Jibola requested a review from baileympearson October 29, 2025 15:50
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