bcnmy / bcnmy/abstractjs

[OPTIMIZATION] No need to pass the full meeVersions list in most cases

Open
#167 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
TypeScript
Stars
8
Forks
12
Avg merge
2h 15m
Merged PRs (30d)
1

Description

### Feature or Improvement Description

No need to pass the full meeVersions list in most cases

### Benefits & Outcomes

Research : No need to pass the full meeVersions list in most cases

https://github.com/bcnmy/abstractjs/pull/166#discussion_r2498925136

### Any References?
https://github.com/bcnmy/abstractjs/blob/42303248becb97a0c4bbd79e2211a13566ef9318/src/sdk/clients/decorators/mee/getFusionQuote.ts#L146

https://github.com/bcnmy/abstractjs/blob/42303248becb97a0c4bbd79e2211a13566ef9318/src/sdk/clients/decorators/mee/getOnChainQuote.ts#L91

https://github.com/bcnmy/abstractjs/blob/d9b8121a912868f010f3d9ea4465bff959c1a404/src/sdk/clients/decorators/mee/getQuote.ts#L1473

and others

### Code of Conduct

- [x] I agree to follow this project's Code of Conduct.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the referenced implementations in src/sdk/clients/decorators/mee/getFusionQuote.ts, getOnChainQuote.ts, and getQuote.ts, then inspect the linked pull request discussion. Identify where the full meeVersions list is passed unnecessarily and verify the affected quote callers no longer require it.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
api
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.