apps/cowswap-frontend/src/modules/wallet/containers/ConnectWalletButton/index.tsx (1)

10-10: 💤 Low value

Consider exporting ConnectWalletButtonProps

The type is internal but since ConnectWalletButton is part of the public module API (re-exported from wallet/index.ts), exporting its props type allows consumers to build typed wrappers or pass forwarded props without having to re-derive the intersection type.

♻️ Proposed change

-type ConnectWalletButtonProps = ButtonPrimaryProps & { children?: ReactNode }
+export type ConnectWalletButtonProps = ButtonPrimaryProps & { children?: ReactNode }

And add the re-export to wallet/index.ts:

 export { ConnectWalletButton } from './containers/ConnectWalletButton'
+export type { ConnectWalletButtonProps } from './containers/ConnectWalletButton'

As per coding guidelines, "Public module API must be re-exported via the module index.ts" — the exported component's companion props type is part of that surface.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@apps/cowswap-frontend/src/modules/wallet/containers/ConnectWalletButton/index.tsx`
at line 10, Export the ConnectWalletButtonProps type so consumers can use its
shape: change the declaration of ConnectWalletButtonProps (currently type
ConnectWalletButtonProps = ButtonPrimaryProps & { children?: ReactNode }) to an
exported type and add a corresponding re-export from the wallet module index (so
the public API exposes ConnectWalletButtonProps alongside ConnectWalletButton);
update any imports that should rely on the public re-export instead of internal
paths.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

libs/wallet/src/wagmi/hooks/useIsRestoringConnection.ts (1)

24-24: 💤 Low value

Remove redundant double negation.

The double negation !!currentConnectorUid is unnecessary in a boolean context (if statement). Simplify to currentConnectorUid && !account.

♻️ Simplify boolean coercion

-  if (!!currentConnectorUid && !account) return true
+  if (currentConnectorUid && !account) return true

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@libs/wallet/src/wagmi/hooks/useIsRestoringConnection.ts` at line 24, In
useIsRestoringConnection (the condition using currentConnectorUid and account),
remove the redundant double-negation by changing the if-condition to use
currentConnectorUid directly instead of coercing with !!currentConnectorUid;
leave the rest of the logic intact so the branch still returns true when a
connector UID exists and account is falsy (i.e., keep the same behavior for
currentConnectorUid and account in the useIsRestoringConnection hook).