Skip to content

remove node_modules from git#8

Open
christophsturm wants to merge 1 commit into
patruff:mainfrom
christophsturm:remove-node_modules
Open

remove node_modules from git#8
christophsturm wants to merge 1 commit into
patruff:mainfrom
christophsturm:remove-node_modules

Conversation

@christophsturm
Copy link
Copy Markdown

this should not be in git and it causes fascinating problems when running on a mac (and probably any os other than windows)

@JakeStevenson
Copy link
Copy Markdown

Agreed, I was looking at making this change too.

Shouldn't we add the packages to package.json and package-lock.json too?

In my local test, I came up with this list in package.json:

{
"name": "mcp-llm-bridge",
"version": "1.0.0",
"main": "dist/main.js",
"scripts": {
"build": "tsc",
"start": "ts-node src/main.ts",
"test": "jest --runInBand",
"test:ollama": "jest tests/ollama.test.ts --runInBand",
"test:filesystem": "jest tests/mcp/filesystem.test.ts --runInBand --detectOpenHandles --forceExit",
"test:flux": "jest tests/mcp/flux.test.ts --runInBand --detectOpenHandles --forceExit",
"test:brave": "jest tests/mcp/brave.test.ts --runInBand --detectOpenHandles --forceExit",
"test:github": "jest tests/mcp/github.test.ts --runInBand --detectOpenHandles --forceExit",
"test:memory": "jest tests/mcp/memory.test.ts --runInBand --detectOpenHandles --forceExit",
"test:gmail-drive": "jest tests/mcp/gmail-drive.test.ts --runInBand --detectOpenHandles --forceExit",
"reset-ollama": "ts-node src/ollama-manager.ts"
},
"dependencies": {
"@modelcontextprotocol/server-brave-search": "^0.6.2",
"@modelcontextprotocol/server-filesystem": "^0.6.2",
"@modelcontextprotocol/server-github": "^0.6.2",
"@modelcontextprotocol/server-memory": "^0.6.2",
"@patruff/server-flux": "^0.1.0",
"@patruff/server-gmail-drive": "^1.0.3",
"chalk": "^4.1.2",
"dotenv": "^16.0.3",
"openai": "^4.0.0",
"winston": "^3.8.2"
},
"devDependencies": {
"@jest/globals": "^29.7.0",
"@types/jest": "^29.5.11",
"@types/node": "^18.0.0",
"@types/node-fetch": "^2.6.9",
"jest": "^29.7.0",
"node-fetch": "^2.6.7",
"ts-jest": "^29.1.1",
"ts-node": "^10.9.1",
"typescript": "^4.9.5"
}
}

@christophsturm
Copy link
Copy Markdown
Author

for me it worked great with the included package.json.

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