Skip to content

fix lib exports and add prefixes to export type names#75

Merged
longngn merged 5 commits intomainfrom
fix-exports
Mar 19, 2026
Merged

fix lib exports and add prefixes to export type names#75
longngn merged 5 commits intomainfrom
fix-exports

Conversation

@longngn
Copy link
Copy Markdown
Member

@longngn longngn commented Mar 17, 2026

No description provided.

Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds module-specific prefixes to exported type names (e.g., SwapOptionsStableswapSwapOptions, DepositOptionsDexV2DepositOptions) to avoid naming conflicts, and broadens index.ts exports from selective to wildcard re-exports for several modules. It also removes unused networkEnv and dexVersion fields from multiple classes.

Changes:

  • Prefixed all exported type names in dex.ts, dex-v2.ts, stableswap.ts, and lbe-v2/type.ts with their module identifier (DexV1, DexV2, Stableswap, LbeV2)
  • Changed index.ts from selective exports to export * for stableswap, dao, lbe-v2/type, and types/factory
  • Removed unused networkEnv, networkEnv, and dexVersion private fields from Dex, DexV2, Stableswap, and LbeV2 classes

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/index.ts Broadened exports to wildcard for several modules
src/dex.ts Prefixed types with DexV1, removed unused fields
src/dex-v2.ts Prefixed types with DexV2, removed unused fields
src/stableswap.ts Prefixed types with Stableswap, removed unused fields, added local StableswapCustomReceiver type
src/lbe-v2/type.ts Prefixed types with LbeV2
src/lbe-v2/lbe-v2.ts Updated imports/usages, removed unused fields
src/lbe-v2/validation.ts Updated imports/usages to match renamed types

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

longngn added 4 commits March 17, 2026 16:05
Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
@longngn longngn merged commit 5afa985 into main Mar 19, 2026
1 check passed
@longngn longngn deleted the fix-exports branch March 19, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants