Skip to content

Conversation

@mattfik
Copy link
Collaborator

@mattfik mattfik commented Nov 8, 2025

Overview

πŸ‘‹πŸ» Goodbye bun, hello pnpm!
πŸš€ 60%+ smaller bundle size for end users
⚑ Better tree-shaking with ESM support. Falls back to CommonJS.
πŸ›  Modern tooling throughout development workflow
πŸ“¦ Production-ready package structure
βœ… All tests passing with comprehensive CI/CD
🚫 No build scripts in the SDK!
🚫 Removed all files that were generated or fetched at build time. i.e. schemas, network configs, etc.
🚫 Remove fragment masking since it's generic and not used in the sdk directly.

End Result

Screenshot 2025-11-07 at 16 41 10 Screenshot 2025-11-10 at 09 25 56

Imports

Before

import { Codex } from "@codex-data/sdk";
import { TokenRankingAttribute, RankingDirection } from "@codex-data/sdk/dist/sdk/generated/graphql";

After

import { Codex, TokenRankingAttribute, RankingDirection } from "@codex-data/sdk";

@mattfik mattfik self-assigned this Nov 8, 2025
@mattfik mattfik added the p2 label Nov 8, 2025
@mattfik mattfik requested a review from bradens November 8, 2025 00:41
Copy link

@bradens bradens left a comment

Choose a reason for hiding this comment

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

Looks great. Good changes. So we'll still be building the SDK from network configs & the automatically fetched schema though, right.

@mattfik
Copy link
Collaborator Author

mattfik commented Nov 10, 2025

Looks great. Good changes. So we'll still be building the SDK from network configs & the automatically fetched schema though, right.

That's correct, @bradens. That part all stays the same. Once I get the hidden network config stuff added then we can fetch the networkconfigs without ever leaking hidden nets to the sdk.

@mattfik mattfik merged commit 71cc242 into main Nov 11, 2025
2 checks passed
@mattfik mattfik deleted the matt/remove-batch-launchpad-override branch November 11, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants