Mintplex-Labs/anything-llm
1
0
Fork 0
mirror of https://github.com/Mintplex-Labs/anything-llm.git synced 2026-07-19 14:13:35 -04:00
Code Issues 328 Packages Projects Releases 10 Wiki Activity

[GH-ISSUE #4782] [CHORE]: Refactor measureStream to use option argument #3012

New Issue
Closed
opened 2026-02-22 18:32:15 -05:00 by yindo · 0 comments
yindo commented 2026-02-22 18:32:15 -05:00
Owner
Copy Link
Copy Source

Originally created by @timothycarambat on GitHub (Dec 14, 2025).
Original GitHub issue: https://github.com/Mintplex-Labs/anything-llm/issues/4782

Originally assigned to: @angelplusultra on GitHub.

What would you like to see?

The LLMPerformanceMonitor measureStream method currently takes positional arguments.

This method have become messy and should be refactored to use an option argument with the exact same arguments it currently takes but now we can unambiguously set the argument for each invocation without needing to specify every positional arg every call.

This should result in no functional change to the operation of the method. Only its invocation signature.

Originally created by @timothycarambat on GitHub (Dec 14, 2025). Original GitHub issue: https://github.com/Mintplex-Labs/anything-llm/issues/4782 Originally assigned to: @angelplusultra on GitHub. ### What would you like to see? The [`LLMPerformanceMonitor`](https://github.com/Mintplex-Labs/anything-llm/blob/master/server/utils/helpers/chat/LLMPerformanceMonitor.js#L65) `measureStream` method currently takes positional arguments. This method have become messy and should be refactored to use an `option` argument with the exact same arguments it currently takes but now we can unambiguously set the argument for each invocation without needing to specify every positional arg every call. This should result in no functional change to the operation of the method. Only its invocation signature.
yindo added the enhancementcore-team-onlyfeature request labels 2026-02-22 18:32:15 -05:00
yindo closed this issue 2026-02-22 18:32:15 -05:00
yindo changed title from [CHORE]: Refactor `measureStream` to use option argument to [GH-ISSUE #4782] [CHORE]: Refactor `measureStream` to use option argument 2026-06-05 14:49:48 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Desktop
Docker
Integration Request
Integration Request
OS: Linux
OS: Mobile
OS: Windows
UI/UX
blocked
bug
bug
core-team-only
documentation
duplicate
embed-widget
enhancement
feature request
github_actions
good first issue
investigating
needs info / can't replicate
possible bug
pull-request
Mirrored from GitHub Pull Request
 question
stage: specifications
wontfix
No Label core-team-only enhancement feature request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3012
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?