Fix lint errors: variable naming and unnecessary type assertion

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: nickwesselman

packages/app/src/cli/services/dev/ui/components/DevSessionUI.tsx

@@ -162,7 +162,7 @@ const DevSessionUI: FunctionComponent<DevSesionUIProps> = ({
     },
   ]
 
-  const activeShortcuts = devStatusShortcuts.filter((s) => s.condition?.() ?? true)
+  const activeShortcuts = devStatusShortcuts.filter((shortcut) => shortcut.condition?.() ?? true)
 
   const tabs: {[key: string]: Tab} = {
     // eslint-disable-next-line id-length
@@ -182,7 +182,7 @@ const DevSessionUI: FunctionComponent<DevSesionUIProps> = ({
                 <Text key={shortcut.key}>
                   {figures.pointerSmall} <Text bold>({shortcut.key})</Text>{' '}
                   {terminalSupportsHyperlinks() && shortcut.url ? (
-                    <Link url={shortcut.url} label={shortcut.shortcutLabel!} />
+                    <Link url={shortcut.url} label={shortcut.shortcutLabel} />
                   ) : (
                     shortcut.shortcutLabel
                   )}
@@ -198,7 +198,7 @@ const DevSessionUI: FunctionComponent<DevSesionUIProps> = ({
                 {status.isReady && !(canUseShortcuts && terminalSupportsHyperlinks()) && (
                   <>
                     {activeShortcuts
-                      .filter((s) => s.url)
+                      .filter((shortcut) => shortcut.url)
                       .map((shortcut) => (
                         <Text key={shortcut.key}>
                           {shortcut.linkLabel} URL: <Link url={shortcut.url!} />