chore(interfaces): use the shared submit-label default and guard a clipboard call

The copilot tool re-stated 'Submit' as a literal, which is what
DEFAULT_FORM_SUBMIT_LABEL exists to prevent — and its test asserted the same
literal, so the two could drift from the server default while staying green.
Both now read the constant.

Also voids the interfaces-list clipboard write: denied permission rejects, and
every other copy call site in the app already handles that.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/interfaces/interfaces.tsx

@@ -311,7 +311,9 @@ export function Interfaces() {
 
   function handleCopyActiveId(): void {
     if (activeInterface) {
-      navigator.clipboard.writeText(activeInterface.id)
+      // Denied clipboard permission rejects; `void` keeps that from surfacing as
+      // an unhandled rejection, matching every other copy call site in the app.
+      void navigator.clipboard.writeText(activeInterface.id)
     }
   }
 

apps/sim/lib/copilot/tools/server/interfaces/user-interface.test.ts

@@ -4,6 +4,7 @@
 
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import type { ServerToolContext } from '@/lib/copilot/tools/server/base-tool'
+import { DEFAULT_FORM_SUBMIT_LABEL } from '@/lib/interfaces/constants'
 import type { InterfaceDefinition, InterfaceModule } from '@/lib/interfaces/types'
 
 const {
@@ -602,7 +603,7 @@ describe('userInterfaceServerTool', () => {
         placement: { row: 1, col: 0, rowSpan: 1, colSpan: 1 },
         config: {
           workflowId: 'wf_1',
-          submitLabel: 'Submit',
+          submitLabel: DEFAULT_FORM_SUBMIT_LABEL,
           fields: [
             {
               id: 'generated-id',
@@ -637,7 +638,7 @@ describe('userInterfaceServerTool', () => {
       expect(mockAddModule).toHaveBeenCalledWith(
         'int_1',
         expect.objectContaining({
-          config: expect.objectContaining({ submitLabel: 'Submit' }),
+          config: expect.objectContaining({ submitLabel: DEFAULT_FORM_SUBMIT_LABEL }),
         })
       )
     })

apps/sim/lib/copilot/tools/server/interfaces/user-interface.ts

@@ -60,7 +60,7 @@ import {
  * routing them separately keeps grid arithmetic real in tests that stub the
  * service surface.
  */
-import { DEFAULT_MODULE_SPAN } from '@/lib/interfaces/constants'
+import { DEFAULT_FORM_SUBMIT_LABEL, DEFAULT_MODULE_SPAN } from '@/lib/interfaces/constants'
 import { overlappingModules } from '@/lib/interfaces/geometry'
 import { getUserEntityPermissions } from '@/lib/workspaces/permissions/utils'
 
@@ -347,7 +347,7 @@ function normalizeFormConfig(raw: unknown): FormModuleConfig {
   }
   return {
     workflowId: nullableId(config, 'workflowId'),
-    submitLabel: configString(config, 'submitLabel', 'Submit'),
+    submitLabel: configString(config, 'submitLabel', DEFAULT_FORM_SUBMIT_LABEL),
     fields: Array.isArray(fields) ? fields.map(readFormField) : [],
   }
 }