Skip to content

feat: land community PRs #39, #44, #45; CI on Node 22 - #47

Merged
pyramation merged 10 commits into
mainfrom
feat/community-prs-node-22
Oct 2, 2026
Merged

pyramation merged 10 commits into
mainfrom
feat/community-prs-node-22

Conversation

@pyramation

@pyramation pyramation commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Merges the three open community PRs as-is (their branches are merged with --no-ff, so authorship is kept and GitHub will mark #39/#44/#45 merged when this lands), plus a CI toolchain bump.

Lockfile

#44's lockfile commit is superseded: pnpm-lock.yaml is regenerated from main's lockfile with CI's pnpm 9.15.4. The large line count is formatting only (main's lockfile has expanded resolution: { integrity } blocks; pnpm writes them inline). Same 2119 packages before/after; the only content changes are the new wallets/eightsaf-extension importer, @types/node 20.19 → 22.19, and peer-suffix keys. No new registries, tarball or git sources.

Security review

  • Authors are long-standing GitHub accounts with unrelated public history. The EightSaf author's handle matches the Firefox add-on publisher ("8pro"), and the Chrome/AMO listings and eightsaf.io all resolve.
  • No install/lifecycle scripts beyond the standard prepack: npm run build, and no new third-party deps, network calls, CI edits, or eval/obfuscation.
  • The EightSaf logo is a base64 SVG. Decoded, it contains only <path>/<style> (no <script>, event handlers, href, or foreignObject).

Verification (local, Node 22.23 / pnpm 9.15.4)

  • pnpm install --frozen-lockfile ✓
  • pnpm run build ✓ — same TS diagnostics as main's CI (vue, mock-wallet, metamask wallets); none come from changed files
  • pnpm run test ✓ (core 128, store 177, all wallets incl. eightsaf)
  • eslint on changed files: no errors.

Link to Devin session: https://app.devin.ai/sessions/a02c84f654b94cbcb50936e98373fba5
Open in Devin Desktop: https://app.devin.ai/desktop/session/a02c84f654b94cbcb50936e98373fba5?variant=devin
Requested by: @pyramation

drimchansky and others added 10 commits March 17, 2026 18:46
Add @interchain-kit/eightsaf-extension so dApps can connect to window.eightsaf
via ExtensionWallet and CosmosWallet for Safrochain connect, sign, and broadcast flows.
Register @interchain-kit/eightsaf-extension in the pnpm workspace lockfile.
Replace the placeholder hexagon icon with the real SVG from
eightsaf/public/eightsaf-icon.svg.
Regenerates pnpm-lock.yaml with pnpm 9.15.4 (CI's version) for the
eightsaf-extension importer and @types/node 22.
@devin-ai-integration

Copy link
Copy Markdown

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@pyramation
pyramation merged commit 374e873 into main Oct 2, 2026
2 checks passed
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.

4 participants