Uh oh!
There was an error while loading. Please reload this page.
feat: remove node installation from gemini - #374
Conversation
- format shell script
DevelopmentCats
commented
Aug 25, 2025
@35C4n0r Just checking but have you tested this module with the new changes? |
35C4n0r
commented
Aug 25, 2025
Hey @DevelopmentCats, yeah it works fine for me. |
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Pull Request Overview
This PR removes the automatic Node.js and npm installation from the Gemini module, standardizing the approach across modules by requiring users to ensure these dependencies are pre-installed.
- Replaced automatic Node.js/npm installation with dependency checking that exits with an error if not found
- Updated documentation to clarify Node.js/npm prerequisites
- Fixed test file paths from install.log to agentapi-start.log
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| registry/coder-labs/modules/gemini/scripts/install.sh | Replaced install_node() with check_dependencies() and improved npm global configuration |
| registry/coder-labs/modules/gemini/main.test.ts | Updated test file paths to reference correct log files |
| registry/coder-labs/modules/gemini/README.md | Updated version to 2.0.0 and clarified Node.js/npm prerequisites |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Harsh9485
commented
Mar 21, 2026
Hey @35C4n0r, could you give me some context on why you removed the Node installation part? |
Closes #
Description
Standardising this across all modules:
Type of Change
Module Information
Path:
registry/coder-labs/modules/geminiNew version:
v2.0.0Breaking change: [x] Yes [ ] No
Testing & Validation
bun test)bun run fmt)Related Issues