style(tests): drop type assertions made unnecessary by eslint 10 toolchain (#497)

Claude-Session: https://claude.ai/code/session_012jwDZSEKBhuQmhQuGujHC1

Author: billchurch

tests/unit/config/options-terminal-shift-enter.vitest.ts

@@ -31,7 +31,7 @@ describe('options.terminal.shiftEnterNewline', () => {
       ...cfg,
       options: { ...cfg.options, terminal: { shiftEnterNewline: 'yes' } }
     }
-    const result = validateConfigSchema(bad as never)
+    const result = validateConfigSchema(bad)
     expect(result.ok).toBe(false)
   })
 

tests/unit/connection-handler/terminal-injection.vitest.ts

@@ -57,7 +57,7 @@ describe('buildTempConfig - terminal slice', () => {
   it('omits the terminal key when options.terminal is absent', () => {
     // eslint-disable-next-line @typescript-eslint/naming-convention, sonarjs/no-unused-vars -- intentional discard destructure
     const { terminal: _terminal, ...optionsWithout } = defaultConfig.options
-    const cfg = { ...defaultConfig, options: optionsWithout } as Config
+    const cfg = { ...defaultConfig, options: optionsWithout }
     const tempConfig = buildTempConfig(makeReq(), cfg)
     expect('terminal' in tempConfig).toBe(false)
   })