fix: move ts-node from devDependencies to dependencies for migration support#5273
Conversation
…support Co-Authored-By: akhilmhdh@infisical.com <akhilmhdh@infisical.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Greptile OverviewGreptile SummaryThis PR moves Key Changes:
Impact:
Confidence Score: 5/5
Important Files Changed
|
Co-Authored-By: akhilmhdh@infisical.com <akhilmhdh@infisical.com>
Context
Fixes #5261
The standalone Docker image (
Dockerfile.standalone-infisical) usesnpm ci --omit=devto install only production dependencies. However, the migration scripts (knexfile.tsandauditlog-knexfile.ts) importts-node/register, which requires ts-node to be available at runtime.Since ts-node was in devDependencies, it wasn't being installed in the production image, causing migration jobs to fail with a module not found error.
This PR moves both
ts-nodeandtsconfig-pathsfrom devDependencies to dependencies so they're available in the production Docker image. Thetsconfig-pathspackage is also required because the tsconfig.json configures ts-node to use it via"require": ["tsconfig-paths/register"].Steps to verify the change
docker build -f Dockerfile.standalone-infisical -t infisical-test .docker run --rm infisical-test node -e "require('ts-node/register'); console.log('success')"Type
Checklist
type(scope): short descriptionHuman Review Checklist
Link to Devin run: https://app.devin.ai/sessions/5a94da773e0d4103b807ec6a0df65455
Requested by: akhilmhdh@infisical.com