chore(sdk): packaging hardening + bump ethereum-contracts to 1.15.2#75
Merged
Conversation
- Add "files": ["dist"] so the tarball ships only dist (+ package.json, LICENSE), dropping src, tests, configs, .turbo, and .env.example - Add MIT LICENSE to the published package - Declare engines.node >= 18 - Add attw (--profile esm-only) + publint gate as check:package, wired into CI and the release workflow to lock the export/type-resolution contract - Document the ESM/CommonJS consumption story in the SDK docs - Fix repository.url to the git+ URL form publint expects Package stays ESM-only; no API or runtime changes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Regenerating against @superfluid-finance/ethereum-contracts 1.15.2 drops the
SuperTokenFactory canonical ERC20 wrapper API, which was removed upstream:
- Removed actions (6): read/write/simulate variants of SuperTokenFactory
{GetCanonicalErc20Wrapper, ComputeCanonicalErc20WrapperAddress,
CreateCanonicalErc20Wrapper, InitializeCanonicalWrapperSuperTokens}
- Removed hooks (6): the corresponding use* variants
createERC20Wrapper and all other SuperTokenFactory functions are unaffected.
Also refreshes the OpenZeppelin SafeCast error inherited across several
contracts (SafeCastOverflowedIntToUint -> SafeCastOverflowedIntDowncast)
from the OZ v5.6.1 submodule bump.
BREAKING CHANGE: SuperTokenFactory canonical-wrapper actions and hooks removed.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Downgrade the changeset from minor to patch per maintainer decision; the combined release is now 0.2.2 -> 0.2.3. The removed SuperTokenFactory canonical-wrapper exports are still listed in the changelog entry. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR bundles two related SDK changes on one branch. Combined, the changesets bump @sfpro/sdk 0.2.2 → 0.2.3 (both
patch).1 · Packaging hardening
@sfpro/sdkstays ESM-only (correct for how it's consumed). This closes two gaps: the npm tarball shipped junk (nofilesallowlist) and nothing protected the export/type contract in CI. Grew out of reviewing an ESM/CJS consumer-compatibility hand-off.files: ["dist"]— tarball now ships onlydist/(pluspackage.json,LICENSE); no moresrc/, tests,wagmi.config.ts/vitest.config.ts/tsconfig.json,.turbo/, or.env.example.LICENSE(MIT).engines.node >= 18.check:package(attw --pack --profile esm-only && publint) wired intoci.ymlandsdk-release.yml(pre-publish) to lock the export/type contract.installation.mdx(await import()any Node /require()Node 20.19+ for the data entrypoints).repository.urlto thegit+…form publint wants.2 · Bump
@superfluid-finance/ethereum-contracts1.15.1 → 1.15.2 + regenerateRegenerating against 1.15.2 drops the SuperTokenFactory canonical ERC20 wrapper API, removed upstream (confirmed intentional).
createERC20Wrapperand all other SuperTokenFactory functions are unaffected; no whole contract ABI was dropped. The rest of the regen diff is cosmetic — the OpenZeppelin v5.6.1 SafeCast error rename (SafeCastOverflowedIntToUint→SafeCastOverflowedIntDowncast) across several contracts.@superfluid-finance/metadatais already at the latest stable (1.6.3), so no bump there.Verification
pnpm --filter=@sfpro/sdk build/typecheck— clean.pnpm --filter=@sfpro/sdk test:unit— 4/4 pass.pnpm --filter=@sfpro/sdk check:package— attw green (esm-only), publint "All good".pnpm changeset status— @sfpro/sdk bumps at patch (→ 0.2.3).npm pack --dry-run— tarball top-level isLICENSE,dist,package.jsononly.🤖 Generated with Claude Code