Skip to content

Commit

Permalink
fix(ci): Fix CI issue where local monorepo node_modules were copied i…
Browse files Browse the repository at this point in the history
…nto export tests (langchain-ai#6774)
  • Loading branch information
jacoblee93 authored and FilipZmijewski committed Sep 27, 2024
1 parent c0e86b3 commit b906e65
Show file tree
Hide file tree
Showing 14 changed files with 54 additions and 763 deletions.
12 changes: 6 additions & 6 deletions environment_tests/scripts/docker-ci-entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,12 @@ mkdir -p ./libs/langchain-community/
mkdir -p ./libs/langchain-cohere/
mkdir -p ./libs/langchain/

cp -r ../langchain-core ./libs/
cp -r ../langchain-openai ./libs/
cp -r ../langchain-anthropic ./libs/
cp -r ../langchain-community ./libs/
cp -r ../langchain-cohere ./libs/
cp -r ../langchain ./libs/
cp -r ../langchain-core/!(node_modules) ./libs/langchain-core
cp -r ../langchain-openai/!(node_modules) ./libs/langchain-openai
cp -r ../langchain-anthropic/!(node_modules) ./libs/langchain-anthropic
cp -r ../langchain-community/!(node_modules) ./libs/langchain-community
cp -r ../langchain-cohere/!(node_modules) ./libs/langchain-cohere
cp -r ../langchain/!(node_modules) ./libs/langchain

# copy cache
mkdir -p ./.yarn
Expand Down
16 changes: 8 additions & 8 deletions langchain/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -411,15 +411,15 @@
"devDependencies": {
"@faker-js/faker": "^7.6.0",
"@jest/globals": "^29.5.0",
"@langchain/anthropic": "^0.2.8",
"@langchain/aws": "^0.0.5",
"@langchain/cohere": "^0.2.1",
"@langchain/anthropic": "*",
"@langchain/aws": "*",
"@langchain/cohere": "*",
"@langchain/core": "workspace:*",
"@langchain/google-genai": "^0.0.26",
"@langchain/google-vertexai": "~0.0",
"@langchain/groq": "^0.0.15",
"@langchain/mistralai": "^0.0.26",
"@langchain/ollama": "^0.0.2",
"@langchain/google-genai": "*",
"@langchain/google-vertexai": "*",
"@langchain/groq": "*",
"@langchain/mistralai": "*",
"@langchain/ollama": "*",
"@langchain/scripts": ">=0.1.0 <0.2.0",
"@swc/core": "^1.3.90",
"@swc/jest": "^0.2.29",
Expand Down
4 changes: 0 additions & 4 deletions libs/langchain-cloudflare/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,6 @@ index.cjs
index.js
index.d.ts
index.d.cts
langgraph/checkpointers.cjs
langgraph/checkpointers.js
langgraph/checkpointers.d.ts
langgraph/checkpointers.d.cts
node_modules
dist
.yarn
3 changes: 1 addition & 2 deletions libs/langchain-cloudflare/langchain.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ function abs(relativePath) {
export const config = {
internals: [/node\:/, /@langchain\/core\//, /@langchain\/langgraph\/web/],
entrypoints: {
index: "index",
"langgraph/checkpointers": "langgraph/checkpointers",
index: "index"
},
tsConfigPath: resolve("./tsconfig.json"),
cjsSource: "./dist-cjs",
Expand Down
24 changes: 2 additions & 22 deletions libs/langchain-cloudflare/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@
"@cloudflare/workers-types": "^4.20240909.0",
"@jest/globals": "^29.5.0",
"@langchain/core": "workspace:*",
"@langchain/langgraph": "~0.0.31",
"@langchain/scripts": ">=0.1.0 <0.2.0",
"@langchain/standard-tests": "0.0.0",
"@swc/core": "^1.3.90",
Expand All @@ -68,13 +67,7 @@
"typescript": "<5.2.0"
},
"peerDependencies": {
"@langchain/core": ">=0.2.21 <0.4.0",
"@langchain/langgraph": "*"
},
"peerDependenciesMeta": {
"@langchain/langgraph": {
"optional": true
}
"@langchain/core": ">=0.2.21 <0.4.0"
},
"publishConfig": {
"access": "public"
Expand All @@ -89,26 +82,13 @@
"import": "./index.js",
"require": "./index.cjs"
},
"./langgraph/checkpointers": {
"types": {
"import": "./langgraph/checkpointers.d.ts",
"require": "./langgraph/checkpointers.d.cts",
"default": "./langgraph/checkpointers.d.ts"
},
"import": "./langgraph/checkpointers.js",
"require": "./langgraph/checkpointers.cjs"
},
"./package.json": "./package.json"
},
"files": [
"dist/",
"index.cjs",
"index.js",
"index.d.ts",
"index.d.cts",
"langgraph/checkpointers.cjs",
"langgraph/checkpointers.js",
"langgraph/checkpointers.d.ts",
"langgraph/checkpointers.d.cts"
"index.d.cts"
]
}
212 changes: 0 additions & 212 deletions libs/langchain-cloudflare/src/langgraph/checkpointers.ts

This file was deleted.

4 changes: 0 additions & 4 deletions libs/langchain-community/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -1026,10 +1026,6 @@ chains/graph_qa/cypher.cjs
chains/graph_qa/cypher.js
chains/graph_qa/cypher.d.ts
chains/graph_qa/cypher.d.cts
langgraph/checkpointers/vercel_kv.cjs
langgraph/checkpointers/vercel_kv.js
langgraph/checkpointers/vercel_kv.d.ts
langgraph/checkpointers/vercel_kv.d.cts
node_modules
dist
.yarn
4 changes: 1 addition & 3 deletions libs/langchain-community/langchain.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -316,9 +316,7 @@ export const config = {
"experimental/llms/chrome_ai": "experimental/llms/chrome_ai",
"experimental/tools/pyinterpreter": "experimental/tools/pyinterpreter",
// chains
"chains/graph_qa/cypher": "chains/graph_qa/cypher",
// langgraph checkpointers
"langgraph/checkpointers/vercel_kv": "langgraph/checkpointers/vercel_kv"
"chains/graph_qa/cypher": "chains/graph_qa/cypher"
},
requiresOptionalDependency: [
"tools/aws_sfn",
Expand Down
Loading

0 comments on commit b906e65

Please sign in to comment.