Skip to content

Remove stonfi provider and bump dev tooling#166

Open
0xh3rman wants to merge 2 commits intomainfrom
remove-stonfi
Open

Remove stonfi provider and bump dev tooling#166
0xh3rman wants to merge 2 commits intomainfrom
remove-stonfi

Conversation

@0xh3rman
Copy link
Copy Markdown
Collaborator

Summary

  • Remove the TypeScript stonfi provider; it lives in core (Rust) now. Drops the provider source, registration, fixtures, TON/ston-fi runtime deps, dependabot grouping, TON_URL env, and bench/justfile defaults.
  • Bump dev tooling: knip 6.6.2 → 6.12.2, oxfmt 0.46.0 → 0.48.0, oxlint 1.61.0 → 1.63.0. Reformats two files (cetus/provider.ts, mayan/sui.ts) per oxfmt 0.48's stricter import grouping.
  • Leaves the StonfiV2 enum entry in packages/types/src/primitives/SwapProvider.ts since that file is typeshare-generated upstream (same approach as the orca removal in Remove orca provider #158).

Test plan

  • pnpm install
  • pnpm build
  • pnpm test
  • pnpm lint
  • pnpm format:check

Stonfi is implemented in core (Rust) now. Drops the TypeScript
provider, its TON/ston-fi runtime deps, and related fixtures. Also
bumps knip 6.6.2 -> 6.12.2, oxfmt 0.46.0 -> 0.48.0, oxlint 1.61.0
-> 1.63.0 and reformats two files per oxfmt's stricter import
grouping.
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the Ston-fi provider and all associated TON blockchain dependencies, configurations, and tests from the codebase. It also updates several development dependencies, including knip, oxfmt, and oxlint. Feedback was provided regarding the benchmark script, which now lacks a default request for the new default provider 'mayan', potentially causing execution failures.

Comment thread scripts/provider-bench.ts Outdated
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.

1 participant