-
Notifications
You must be signed in to change notification settings - Fork 178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: move llm bots to separate package #4629
refactor: move llm bots to separate package #4629
Conversation
Signed-off-by: Pablo Maldonado <pablo@umaproject.org>
UMA-1702 Create a separate package for llm-bot in protocol repo
Move the llm-bot from packages/monitor-v2 to a separate package packages/llm-bot |
Signed-off-by: Pablo Maldonado <pablo@umaproject.org>
Signed-off-by: Pablo Maldonado <pablo@umaproject.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Only nits and minor comments
@@ -0,0 +1,24 @@ | |||
import { formatBytes32String, parseUnits } from "./utils"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are some/all of these test utilities files copied? If so, could you comment that in the PR so we know which to review as fresh code?
packages/llm-bot/src/utils/logger.ts
Outdated
const optimisticOracleV2UIBaseUrl = "https://oracle.uma.xyz"; | ||
const testnetOptimisticOracleV2UIBaseUrl = "https://testnet.oracle.uma.xyz"; | ||
|
||
// monitor-v2 package is only using Optimistic Oracle V3, so currently there is no need to generalize this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: outdated comment
packages/llm-bot/src/utils/logger.ts
Outdated
export const createSnapshotProposalLink = (baseUrl: string, space: string, proposalId: string): string => { | ||
return `<${baseUrl}/#/${space}/proposal/${proposalId}|Snapshot UI>`; | ||
}; | ||
|
||
export const createTenderlySimulationLink = (simulationResult?: TenderlySimulationResult): string => { | ||
if (simulationResult === undefined) { | ||
return "No Tenderly simulation available"; | ||
} else if (simulationResult.status) { | ||
return `<${simulationResult.resultUrl.url}|Tenderly simulation successful${ | ||
!simulationResult.resultUrl.public ? " (private)" : "" | ||
}>`; | ||
} else { | ||
return `<${simulationResult.resultUrl.url}|Tenderly simulation reverted${ | ||
!simulationResult.resultUrl.public ? " (private)" : "" | ||
}>`; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was a little surprised to see this code here? Is this used (or placed to be used) in the LLC bit codebase? If so, no problem, just wanted to call it out!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
im assuming this is just moving stuff, but i dont see why the changes from old location arent showing
Signed-off-by: Pablo Maldonado <pablo@umaproject.org>
Motivation
Move LLM bots to a separate package