Skip to content

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

Open
longngn wants to merge 2 commits intomainfrom
fix-exports
Open

fix lib exports and add prefixes to export type names#75
longngn wants to merge 2 commits intomainfrom
fix-exports

Conversation

@longngn
Copy link
Member

@longngn longngn commented Mar 17, 2026

No description provided.

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

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.

Signed-off-by: Nguyen Le Vu Long <vulongvn98@gmail.com>
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