fix(emcn): stop Enter double-submit and close registration gap

- stopPropagation on field Enter-submit so a parent onKeyDown can't re-fire the primary (double OAuth connect, etc.)
- register primary via useLayoutEffect so it's set before paint (no null-Enter window)
- drop now-redundant parent Enter handlers in connect-oauth/create-workspace/rename-document modals

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/components/connect-oauth-modal/connect-oauth-modal.tsx

@@ -1,6 +1,6 @@
 'use client'
 
-import { type ComponentType, type KeyboardEvent, useEffect, useMemo, useRef, useState } from 'react'
+import { type ComponentType, useEffect, useMemo, useRef, useState } from 'react'
 import {
   Badge,
   ChipModal,
@@ -309,18 +309,6 @@ export function ConnectOAuthModal(props: ConnectOAuthModalProps) {
     ? !displayName.trim() || isPending || Boolean(existingCredential)
     : isPending
 
-  /**
-   * Submits the connect form on Enter, mirroring the Connect button's enabled
-   * state and excluding the multi-line description. Restores the keyboard
-   * affordance the pre-consolidation workflow modal provided.
-   */
-  const handleBodyKeyDown = (event: KeyboardEvent<HTMLDivElement>) => {
-    if (event.key !== 'Enter' || !isConnect || isDisabled) return
-    if (event.target instanceof HTMLTextAreaElement) return
-    event.preventDefault()
-    void handleConnect()
-  }
-
   const displayNameError =
     validationError ??
     (existingCredential
@@ -334,7 +322,7 @@ export function ConnectOAuthModal(props: ConnectOAuthModalProps) {
       <ChipModalHeader icon={ProviderIcon} onClose={handleClose}>
         {title}
       </ChipModalHeader>
-      <ChipModalBody onKeyDown={handleBodyKeyDown}>
+      <ChipModalBody>
         {!isConnect && (
           <p className='text-[var(--text-tertiary)] text-caption'>
             The "{props.toolName}" tool requires access to your account.

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/rename-document-modal/rename-document-modal.tsx

@@ -74,17 +74,10 @@ export function RenameDocumentModal({
     }
   }
 
-  const handleKeyDown = (e: React.KeyboardEvent) => {
-    if (e.key === 'Enter') {
-      e.preventDefault()
-      void handleSubmit()
-    }
-  }
-
   return (
     <ChipModal open={open} onOpenChange={onOpenChange} srTitle='Rename Document'>
       <ChipModalHeader onClose={() => onOpenChange(false)}>Rename Document</ChipModalHeader>
-      <ChipModalBody onKeyDown={handleKeyDown}>
+      <ChipModalBody>
         <ChipModalField
           type='input'
           title='Name'

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workspace-header/components/create-workspace-modal/create-workspace-modal.tsx

@@ -69,13 +69,6 @@ export function CreateWorkspaceModal({
     }
   }
 
-  const handleKeyDown = (e: React.KeyboardEvent) => {
-    if (e.key === 'Enter') {
-      e.preventDefault()
-      void handleSubmit()
-    }
-  }
-
   const handleNameChange = (value: string) => {
     setName(value)
     setError(null)
@@ -86,7 +79,7 @@ export function CreateWorkspaceModal({
   return (
     <ChipModal open={open} onOpenChange={onOpenChange} srTitle={copy.title}>
       <ChipModalHeader onClose={() => onOpenChange(false)}>{copy.title}</ChipModalHeader>
-      <ChipModalBody onKeyDown={handleKeyDown}>
+      <ChipModalBody>
         <p className='px-2 text-[var(--text-muted)] text-sm'>{copy.description}</p>
         <ChipModalField
           type='input'

packages/emcn/src/components/chip-modal/chip-modal.tsx

@@ -646,13 +646,17 @@ function renderChipModalControl(
             if (event.key !== 'Enter' || event.nativeEvent.isComposing) return
             if (onSubmit) {
               event.preventDefault()
+              event.stopPropagation()
               onSubmit()
               return
             }
             if (props.submitOnEnter === false) return
             const submit = submitRef?.current
             if (submit && !submit.disabled) {
               event.preventDefault()
+              // Stop bubbling so a parent Enter handler (e.g. a modal body that
+              // also submits) can't fire the same primary action a second time.
+              event.stopPropagation()
               submit.trigger()
             }
           }}
@@ -1095,13 +1099,18 @@ function ChipModalFooter({
    * the modal on Enter without per-field wiring. Kept in a ref (updated each
    * render) rather than state so fields read the latest handler at Enter-time.
    *
+   * A layout effect (not a passive effect) so the ref is populated before the
+   * browser paints the modal — otherwise there is a window between first paint
+   * and effect commit where an enabled primary is visible but Enter does
+   * nothing because `submitRef.current` is still `null`.
+   *
    * A `destructive` primary is deliberately NOT published: Enter must never
    * trigger a destructive action from a text field. Destructive flows that DO
    * want Enter (e.g. a guarded "change address") wire an explicit field-level
    * `onSubmit`, which takes precedence over this fallback.
    */
   const submitRef = React.useContext(ChipModalSubmitContext)
-  React.useEffect(() => {
+  React.useLayoutEffect(() => {
     if (!submitRef) return
     if (primaryAction.variant === 'destructive') {
       submitRef.current = null