Skip to content

Dev - #11

Merged
Man-Jain merged 6 commits into
mainfrom
dev
Sep 11, 2026
Merged

Dev#11
Man-Jain merged 6 commits into
mainfrom
dev

Conversation

@Man-Jain

Copy link
Copy Markdown
Contributor

No description provided.

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
miden-integration-example Ready Ready Preview Sep 11, 2026 7:44pm UTC

@Man-Jain
Man-Jain merged commit ba8ab66 into main Sep 11, 2026
3 checks passed
@Man-Jain
Man-Jain deleted the dev branch September 11, 2026 19:44
@github-actions

Copy link
Copy Markdown

React Doctor found 4 new issues in 4 files · 4 warnings · score 84 / 100 (Needs work) · 3 fixed · vs main

4 warnings

src/components/crosschain/IntentForm.tsx

  • ⚠️ L49 React function has high control-flow complexity no-high-complexity-react-function

src/components/crosschain/IntentStatus.tsx

  • ⚠️ L120 React function has high control-flow complexity no-high-complexity-react-function

src/components/crosschain/WithdrawForm.tsx

  • ⚠️ L36 React function has high control-flow complexity no-high-complexity-react-function

src/components/providers/MidenNetworkApp.tsx

  • ⚠️ L20 Unstable context provider value jsx-no-constructed-context-values

Reviewed by React Doctor for commit c0e5608. See inline comments for fixes.

@@ -117,6 +118,7 @@ function StatusRow({
}

export function IntentStatus({ result, error, flowStatus, isPolling }: Props) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

React Doctor · react-doctor/no-high-complexity-react-function (warning)

IntentStatus has cyclomatic complexity 18, cognitive complexity 17, and maximum nesting depth 2, so its React logic is hard to understand and change. Extract independent branches into components or hooks.

Fix → Extract independent render branches and state logic into focused components or hooks until the control flow is easy to follow.

Docs

const networkConfig = getMidenNetworkConfig(network);

return (
<MidenNetworkContext.Provider value={{ network }}>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

React Doctor · react-doctor/jsx-no-constructed-context-values (warning)

Every reader of this context redraws on each render because you build its value inline.

Fix → Wrap the context value in useMemo or move it outside the component so consumers do not redraw every render.

Docs

This branch was successfully deployed

1 active deployment
Preview — c0e56081 Deployed Sep 11, 2026 by vercel[bot]
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.

2 participants