Skip to content

Revert "Sequence"#622

Merged
Dargon789 merged 2 commits into
masterfrom
revert-616-sequence
May 26, 2026
Merged

Revert "Sequence"#622
Dargon789 merged 2 commits into
masterfrom
revert-616-sequence

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

Reverts #616

This reverts commit e08b3a8.
@codesandbox
Copy link
Copy Markdown

codesandbox Bot commented May 26, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry, we are unable to review this pull request

The GitHub API does not allow us to fetch diffs exceeding 300 files, and this pull request has 2127

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sequence.js Ready Ready Preview, Comment May 26, 2026 7:02am
wagmi-project Building Building Preview, Comment May 26, 2026 7:02am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
sequence-js-docs Skipped Skipped May 26, 2026 7:02am
sequence-js-web Skipped Skipped May 26, 2026 7:02am

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented May 26, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

protected path = '/rpc/UserData/'

constructor(hostname: string, fetch: Fetch) {
this.hostname = hostname.replace(/\/*$/, '')
Comment on lines +139 to +145

export type ExplicitSessionParams = {
chainId: number
expiresIn: SessionDuration
permissions: Permission.Permission[]
nativeTokenSpending?: NativeTokenSpending
}
Comment thread packages/wallet/primitives-cli/src/subcommands/server.ts Fixed
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 performs a major cleanup by deleting several demo dapps, solidity contracts, workflows, and configuration files. It also updates the CI dependency installation action by upgrading Node setup, PNPM to v10, and caching actions, while configuring ESLint and Next.js settings in the extras/docs and extras/web packages. The reviewer's feedback highlights potential issues with the changeset configuration using directory names instead of package names, potential lockfile incompatibilities with the PNPM v10 upgrade, and a possible runtime TypeError from spreading nextJsConfig in the ESLint configurations if it is not an array.

Comment thread .changeset/config.json
Comment thread .github/actions/install-dependencies/action.yml
Comment thread extras/docs/eslint.config.js
Comment thread extras/web/eslint.config.js
… through a stack trace'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
@vercel vercel Bot temporarily deployed to Preview – sequence-js-docs May 26, 2026 07:02 Inactive
@vercel vercel Bot temporarily deployed to Preview – sequence-js-web May 26, 2026 07:02 Inactive
@Dargon789 Dargon789 merged commit ac51ae3 into master May 26, 2026
14 of 17 checks passed
@Dargon789 Dargon789 deleted the revert-616-sequence branch May 26, 2026 07:18
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.

Sequence diagram for JSON-RPC parse error handling in handleHttpRequest

2 participants