app/components/Header/ConnectorModal.vue (2)

104-109: Redundant ms-auto class.

The ms-auto on the ButtonBase (line 107) is redundant since the parent <div> already has ms-auto and uses flex with gap-2. The button is already positioned correctly without it.

♻️ Suggested fix

           <ButtonBase
             :aria-label="copied ? $t('connector.modal.copied') : $t('connector.modal.copy_command')"
             `@click`="copyCommand"
-            class="ms-auto"
             :classicon="copied ? 'i-lucide:check text-green-500' : 'i-lucide:copy'"
           />

---

30-36: Await the copy() call and consider using the DEFAULT_PORT constant.

Two observations:

1. Per the established pattern in useInstallCommand.ts (line 86), the copy() function returns a Promise that should be awaited. This ensures clipboard errors are not silently ignored.

2. The port value '31415' is hard-coded on lines 8, 22, and 32, whilst DEFAULT_PORT is defined in app/composables/useConnector.ts (line 57). However, this constant is not currently exported, so you would need to export it from that file first before using it here.

♻️ Suggested refactor

-function copyCommand() {
+async function copyCommand() {
   let command = executeNpmxConnectorCommand.value
   if (portInput.value !== '31415') {
     command += ` --port ${portInput.value}`
   }
-  copy(command)
+  await copy(command)
 }

To also use DEFAULT_PORT, you would additionally need to export it from useConnector.ts and import it here, then replace the hard-coded '31415' with String(DEFAULT_PORT).