[PR #4867] [MERGED] Adjust fix path to use ESM import #4800

Closed
opened 2026-02-22 18:36:33 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/4867
Author: @timothycarambat
Created: 1/15/2026
Status: Merged
Merged: 1/15/2026
Merged by: @timothycarambat

Base: masterHead: fix-path-patch


📝 Commits (6)

  • daea1f8 Adjust fix path to use ESM import
  • 02b7c6f normalize fix-path imports and usage across the app
  • ab17d08 extract path fix logic to utils for server and collector
  • c4818b1 add helpers
  • 68a77f8 repin strip-ansi in collector
  • 27cea31 fix log for localWhisper

📊 Changes

9 files changed (+84 additions, -55 deletions)

View changed files

📝 .github/workflows/dev-build.yaml (+1 -1)
📝 collector/__tests__/utils/WhisperProviders/ffmpeg/index.test.js (+8 -7)
📝 collector/package.json (+1 -1)
📝 collector/utils/WhisperProviders/ffmpeg/index.js (+7 -15)
📝 collector/utils/WhisperProviders/localWhisper.js (+2 -2)
collector/utils/shell.js (+25 -0)
📝 collector/yarn.lock (+13 -1)
📝 server/utils/MCP/hypervisor/index.js (+2 -28)
server/utils/helpers/shell.js (+25 -0)

📄 Description

fix-path v5 cannot be used in the Docker build right now as we are on Node 18.18.x.
fix-path v4 is missing the strip-ansi patch that is the only change between v4 <> v5.

This change pins all uses of fix-path to v4 with the strip-ansi patch
This also updates all tests, as well as using the dynamic import module to ensure the import works.
Currently, all ffmpeg are failing inside of the container as this code would always throw.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/4867 **Author:** [@timothycarambat](https://github.com/timothycarambat) **Created:** 1/15/2026 **Status:** ✅ Merged **Merged:** 1/15/2026 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `fix-path-patch` --- ### 📝 Commits (6) - [`daea1f8`](https://github.com/Mintplex-Labs/anything-llm/commit/daea1f8e0697bc5e70de0d404c9b8854f597cfb1) Adjust fix path to use ESM import - [`02b7c6f`](https://github.com/Mintplex-Labs/anything-llm/commit/02b7c6fb6c25d0e963e6562c07303da456fa8c3c) normalize fix-path imports and usage across the app - [`ab17d08`](https://github.com/Mintplex-Labs/anything-llm/commit/ab17d0846a15be09226d2ae44dce8829b0ce0b6c) extract path fix logic to utils for server and collector - [`c4818b1`](https://github.com/Mintplex-Labs/anything-llm/commit/c4818b12b57ff27862bcda3aac84c14f83938bb7) add helpers - [`68a77f8`](https://github.com/Mintplex-Labs/anything-llm/commit/68a77f888ee1eda711c149559457dc6e762b3368) repin strip-ansi in collector - [`27cea31`](https://github.com/Mintplex-Labs/anything-llm/commit/27cea317575c5ca06aba6160dfbd9b8ba152b391) fix log for localWhisper ### 📊 Changes **9 files changed** (+84 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/dev-build.yaml` (+1 -1) 📝 `collector/__tests__/utils/WhisperProviders/ffmpeg/index.test.js` (+8 -7) 📝 `collector/package.json` (+1 -1) 📝 `collector/utils/WhisperProviders/ffmpeg/index.js` (+7 -15) 📝 `collector/utils/WhisperProviders/localWhisper.js` (+2 -2) ➕ `collector/utils/shell.js` (+25 -0) 📝 `collector/yarn.lock` (+13 -1) 📝 `server/utils/MCP/hypervisor/index.js` (+2 -28) ➕ `server/utils/helpers/shell.js` (+25 -0) </details> ### 📄 Description `fix-path` v5 cannot be used in the Docker build right now as we are on Node 18.18.x. `fix-path` v4 is missing the `strip-ansi` patch that is the only change between v4 <> v5. This change pins all uses of `fix-path` to v4 with the strip-ansi patch This also updates all tests, as well as using the dynamic `import` module to ensure the import works. Currently, all ffmpeg are failing inside of the container as this code would always throw. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:36:33 -05:00
yindo closed this issue 2026-02-22 18:36:33 -05:00
yindo changed title from [PR #4867] Adjust fix path to use ESM import to [PR #4867] [MERGED] Adjust fix path to use ESM import 2026-06-05 15:20:25 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#4800