Skip to content

Codesandbox#600

Merged
Dargon789 merged 31 commits into
0xsequence-masterfrom
codesandbox
May 22, 2026
Merged

Codesandbox#600
Dargon789 merged 31 commits into
0xsequence-masterfrom
codesandbox

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

@Dargon789 Dargon789 commented May 22, 2026

Summary by Sourcery

Add a wagmi-based demo app and integrate new security and CI workflows.

New Features:

  • Introduce a new Vite-based React "wagmi-project" demonstrating wallet connection using wagmi and viem.

Enhancements:

  • Record a new major version entry in the indexer service changelog.

Build:

  • Add Vite, TypeScript, and React tooling configuration for the new wagmi demo project.

CI:

  • Add a Fortify Application Security Testing GitHub Actions workflow for automated security scanning.
  • Add an Azure Pipelines configuration to build the Node.js project on master branch pushes.

Deployment:

  • Add CNAME and Codesandbox-related configuration files to support hosting and online sandboxing setups.

Documentation:

  • Add SECURITY policy documentation and standardized GitHub issue templates for bugs, features, and custom issues.

Chores:

  • Check in generated cache/map artifacts under v8-compile-cache-0 for preconstruct CLI.

corbanbrook and others added 28 commits June 13, 2025 01:57
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
* Upgrade uuid from v13 to v14 (0xsequence#996)

* Upgrading workspace root deps (0xsequence#997)

* New userdata.gen.ts

* 3.0.6

* 3.0.7

* Fix fee options for undeployed wallets (0xsequence#998)

* Bump next from 15.5.14 to 15.5.15 (0xsequence#989)

Bumps [next](https://github.com/vercel/next.js) from 15.5.14 to 15.5.15.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v15.5.14...v15.5.15)

---
updated-dependencies:
- dependency-name: next
  dependency-version: 15.5.15
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update to latest pnpm

* 3.0.8

* Fix fee options stub signature (0xsequence#999)

* 3.0.9

* Enforce minimum age limitation for packages

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Corban Riley <corbanbrook@gmail.com>
Co-authored-by: Taylan Pince <taylanpince@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: tolgahan-arikan <tolgahan.arikan@gmail.com>
* Bump the npm_and_yarn group across 1 directory with 1 update

Bumps the npm_and_yarn group with 1 update in the / directory: [express](https://github.com/expressjs/express).


Updates `express` from 4.18.2 to 4.19.2
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.18.2...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:development
  dependency-group: npm_and_yarn-security-group
...

Signed-off-by: dependabot[bot] <support@github.com>

* Create SECURITY.md

* Set up CI with Azure Pipelines

[skip ci]

* Create CNAME

* Create fortify.yml

* Update issue templates

* Update CNAME

* Update CNAME

* Update CNAME

* Update CNAME

* update

* Update wagmi-project/src/main.tsx

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>

* Update wagmi-project/package.json

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>

* Update .github/ISSUE_TEMPLATE/custom.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>

* Enforce minimum age limitation for packages (#556)

* Bump next from 15.5.14 to 15.5.15 (0xsequence#989)

Bumps [next](https://github.com/vercel/next.js) from 15.5.14 to 15.5.15.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v15.5.14...v15.5.15)

---
updated-dependencies:
- dependency-name: next
  dependency-version: 15.5.15
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update to latest pnpm

* 3.0.8

* Fix fee options stub signature (0xsequence#999)

* 3.0.9

* Enforce minimum age limitation for packages

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Taylan Pince <taylanpince@gmail.com>
Co-authored-by: tolgahan-arikan <tolgahan.arikan@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: googleworkspace-bot <googleworkspace-bot@google.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: Taylan Pince <taylanpince@gmail.com>
Co-authored-by: tolgahan-arikan <tolgahan.arikan@gmail.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
* Bump next.js deps to resolve security issue (0xsequence#1000)

* Add Claude review workflow (0xsequence#1001)

---------

Co-authored-by: Taylan Pince <taylanpince@gmail.com>
* Bump next.js deps to resolve security issue (0xsequence#1000)

* Add Claude review workflow (0xsequence#1001)

---------

Co-authored-by: Taylan Pince <taylanpince@gmail.com>
* Bump next.js deps to resolve security issue (0xsequence#1000)

* Add Claude review workflow (0xsequence#1001)

---------

Co-authored-by: Taylan Pince <taylanpince@gmail.com>
@bolt-new-by-stackblitz
Copy link
Copy Markdown

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

@codesandbox
Copy link
Copy Markdown

codesandbox Bot commented May 22, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel

This comment was marked as resolved.

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented May 22, 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.

@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented May 22, 2026

Reviewer's Guide

Adds a new wagmi-based Vite demo app and various project-scaffolding/configuration artifacts (security and issue templates, CI pipelines, Fortify SAST workflow), plus bumps the indexer service changelog to 2.0.0 with a placeholder entry.

Sequence diagram for wagmi wallet connect/disconnect flow

sequenceDiagram
  actor User
  participant App
  participant WagmiProvider
  participant useConnect
  participant useAccount
  participant useDisconnect

  User->>App: click connector button
  App->>useConnect: connect({ connector })
  useConnect->>WagmiProvider: connect
  WagmiProvider-->>useAccount: update account.status='connected'
  useAccount-->>App: account (status, addresses, chainId)
  App-->>User: render account info and Disconnect button

  User->>App: click Disconnect
  App->>useDisconnect: disconnect()
  useDisconnect->>WagmiProvider: disconnect
  WagmiProvider-->>useAccount: update account.status='disconnected'
  useAccount-->>App: account (status)
  App-->>User: hide Disconnect button, show Connect buttons
Loading

File-Level Changes

Change Details Files
Introduce a minimal wagmi + React Vite demo application under wagmi-project for wallet connection testing.
  • Bootstrap a Vite React TypeScript app with standard index.html, main entry, React root, and CSS baseline.
  • Configure WagmiProvider with mainnet and sepolia chains, injected/WalletConnect/Coinbase connectors, and a shared QueryClient for react-query.
  • Implement App component to display connection status, addresses, chainId, and allow connecting/disconnecting via available connectors.
  • Add TypeScript, Vite, wagmi, viem, react-query, and build tooling configuration (tsconfig, tsconfig.node, vite.config, biome, .gitignore, .npmrc, vite env types, README).
wagmi-project/package.json
wagmi-project/index.html
wagmi-project/src/main.tsx
wagmi-project/src/App.tsx
wagmi-project/src/wagmi.ts
wagmi-project/src/index.css
wagmi-project/src/vite-env.d.ts
wagmi-project/tsconfig.json
wagmi-project/tsconfig.node.json
wagmi-project/vite.config.ts
wagmi-project/biome.json
wagmi-project/.gitignore
wagmi-project/.npmrc
wagmi-project/README.md
Add security and issue management documentation/templates to standardize project processes.
  • Create SECURITY.md with placeholders for supported versions and vulnerability reporting process.
  • Add GitHub issue templates for bug reports, feature requests, and a custom template.
SECURITY.md
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/custom.md
Add CI workflows for security scanning and basic Node.js builds.
  • Introduce a GitHub Actions workflow to run Fortify AST SAST scans on master pushes, PRs, schedules, and manual runs, configured to use Fortify on Demand secrets.
  • Add an Azure Pipelines configuration that installs Node.js 10.x, runs npm install, and builds the project on master branch.
.github/workflows/fortify.yml
azure-pipelines.yml
Update indexer service changelog to start 2.0.0 with a placeholder major-change entry.
  • Add 2.0.0 section with a 'Major Changes' header and placeholder 'changeset' entry above 1.10.14 in the indexer changelog.
packages/services/indexer/CHANGELOG.md
Check in various repository-level and tooling artifacts for Codesandbox and build caching.
  • Add Codesandbox tasks configuration and related repo-level metadata files.
  • Introduce v8 compile cache map artifacts under v8-compile-cache-0 directory.
.codesandbox/tasks.json
CNAME
v8-compile-cache-0/x64/11.3.244.8-node.19/zSprojectzSsequence.jszSnode_moduleszS.pnpmzS@preconstruct+cli@2.8.7zSnode_moduleszS@preconstructzSclizSbin.js.MAP
v8-compile-cache-0/x64/11.3.244.8-node.19/zSprojectzSworkspacezSnode_moduleszS.pnpmzS@preconstruct+cli@2.8.7zSnode_moduleszS@preconstructzSclizSbin.js.MAP

Possibly linked issues

  • Feature/integration #17: They both describe adding the wagmi React project plus SECURITY.md, Azure Pipelines, and related placeholder/config files.

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

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 initializes a new Vite project using create-wagmi and adds various configuration files for CI/CD, CodeSandbox, and GitHub. The review feedback identifies several issues: the Azure pipeline uses an obsolete Node.js version and the wrong package manager, and environment-specific V8 cache artifacts were mistakenly committed. Additionally, the reviewer recommends pinning dependencies to specific versions instead of using the latest tag and suggests validating the WalletConnect project ID environment variable to ensure runtime stability.

Comment thread azure-pipelines.yml
Comment thread azure-pipelines.yml
Comment thread wagmi-project/package.json Outdated
Comment thread wagmi-project/src/wagmi.ts Outdated
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.

Hey - I've found 2 issues, and left some high level feedback:

  • It looks like various generated/cache artifacts (e.g. v8-compile-cache-0/...bin.js.MAP) and sandbox/infra files (e.g. .codesandbox/tasks.json, CNAME) are being committed; consider excluding these via .gitignore and keeping the PR focused on the actual app and workflow changes.
  • The Azure pipeline is pinned to Node 10 (versionSpec: '10.x'), which is long out of support; aligning this to the runtime you actually use (and to what your app expects, e.g. Node 18+) would avoid surprises in the CI environment.
  • In wagmi-project/src/wagmi.ts, walletConnect depends on import.meta.env.VITE_WC_PROJECT_ID; adding a simple runtime check or failing fast when this env var is missing would make misconfiguration easier to catch.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- It looks like various generated/cache artifacts (e.g. `v8-compile-cache-0/...bin.js.MAP`) and sandbox/infra files (e.g. `.codesandbox/tasks.json`, `CNAME`) are being committed; consider excluding these via `.gitignore` and keeping the PR focused on the actual app and workflow changes.
- The Azure pipeline is pinned to Node 10 (`versionSpec: '10.x'`), which is long out of support; aligning this to the runtime you actually use (and to what your app expects, e.g. Node 18+) would avoid surprises in the CI environment.
- In `wagmi-project/src/wagmi.ts`, `walletConnect` depends on `import.meta.env.VITE_WC_PROJECT_ID`; adding a simple runtime check or failing fast when this env var is missing would make misconfiguration easier to catch.

## Individual Comments

### Comment 1
<location path="wagmi-project/src/wagmi.ts" line_range="5-12" />
<code_context>
+  connectors: [
+    injected(),
+    coinbaseWallet(),
+    walletConnect({ projectId: import.meta.env.VITE_WC_PROJECT_ID }),
+  ],
+  transports: {
</code_context>
<issue_to_address>
**suggestion:** Handle missing or misconfigured `VITE_WC_PROJECT_ID` to avoid runtime failures.

If `VITE_WC_PROJECT_ID` is missing or empty, `walletConnect` will likely fail at runtime in a hard-to-debug way. Consider validating this value when creating the config (e.g., throw a clear error or skip the connector) so misconfiguration is surfaced early.

```suggestion
const wcProjectId = import.meta.env.VITE_WC_PROJECT_ID

if (!wcProjectId) {
  // Fail fast with a clear error instead of an opaque runtime failure inside walletConnect
  throw new Error(
    'VITE_WC_PROJECT_ID is not set. Please define it in your environment to enable WalletConnect.',
  )
}

export const config = createConfig({
  chains: [mainnet, sepolia],
  connectors: [
    injected(),
    coinbaseWallet(),
    walletConnect({ projectId: wcProjectId }),
  ],
  transports: {
```
</issue_to_address>

### Comment 2
<location path="wagmi-project/src/main.tsx" line_range="12" />
<code_context>
+
+import './index.css'
+
+globalThis.Buffer = Buffer
+
+const queryClient = new QueryClient()
</code_context>
<issue_to_address>
**suggestion (bug_risk):** Consider guarding or narrowing the global `Buffer` assignment to avoid unintended overrides.

Unconditionally setting `globalThis.Buffer` can clobber an existing implementation in some environments. Consider guarding it, e.g. `if (!globalThis.Buffer) globalThis.Buffer = Buffer`, so you only provide the polyfill when no `Buffer` is already defined.

```suggestion
if (!globalThis.Buffer) {
  globalThis.Buffer = Buffer
}
```
</issue_to_address>

Fix all in Cursor


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Comment thread wagmi-project/src/wagmi.ts
Comment thread wagmi-project/src/main.tsx Outdated
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

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

Project Deployment Actions Updated (UTC)
sequence-js-docs Canceled Canceled May 22, 2026 10:31am
sequence-js-web Canceled Canceled May 22, 2026 10:31am
wagmi-project Ready Ready Preview, Comment May 22, 2026 10:31am

Dargon789 and others added 2 commits May 22, 2026 17:25
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
@Dargon789 Dargon789 linked an issue May 22, 2026 that may be closed by this pull request
@vercel vercel Bot temporarily deployed to Preview – sequence-js-web May 22, 2026 10:31 Inactive
@vercel vercel Bot temporarily deployed to Preview – sequence-js-docs May 22, 2026 10:31 Inactive
@Dargon789 Dargon789 merged commit 563e0e1 into 0xsequence-master May 22, 2026
13 of 19 checks passed
@Dargon789 Dargon789 deleted the codesandbox branch May 22, 2026 10:39
@Dargon789 Dargon789 restored the codesandbox branch May 22, 2026 10:41
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 wagmi wallet connect/disconnect flow

3 participants