fix: resolve lint errors in logs commands and tests

Add proper type assertions to eliminate unsafe-any and non-null
assertion lint errors.

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

Author: biilmann

src/commands/logs/edge-functions.ts

@@ -10,7 +10,7 @@ import { CLI_LOG_LEVEL_CHOICES_STRING, LOG_LEVELS_LIST } from './log-levels.js'
 import { getName } from './build.js'
 
 export const logsEdgeFunction = async (options: OptionValues, command: BaseCommand) => {
-  let deployId: string | undefined = options.deployId
+  let deployId = options.deployId as string | undefined
   await command.authenticate()
 
   const client = command.netlify.api
@@ -22,23 +22,24 @@ export const logsEdgeFunction = async (options: OptionValues, command: BaseComma
     return
   }
 
-  if (options.level && !options.level.every((level: string) => LOG_LEVELS_LIST.includes(level))) {
-    log(`Invalid log level. Choices are:${CLI_LOG_LEVEL_CHOICES_STRING}`)
+  const levels = options.level as string[] | undefined
+  if (levels && !levels.every((level) => LOG_LEVELS_LIST.includes(level))) {
+    log(`Invalid log level. Choices are:${CLI_LOG_LEVEL_CHOICES_STRING.toString()}`)
   }
 
-  const levelsToPrint = options.level || LOG_LEVELS_LIST
+  const levelsToPrint: string[] = levels || LOG_LEVELS_LIST
 
   if (options.from) {
-    const fromMs = parseDateToMs(options.from)
-    const toMs = options.to ? parseDateToMs(options.to) : Date.now()
+    const fromMs = parseDateToMs(options.from as string)
+    const toMs = options.to ? parseDateToMs(options.to as string) : Date.now()
 
     const url = `https://analytics.services.netlify.com/v2/sites/${siteId}/edge_function_logs?from=${fromMs.toString()}&to=${toMs.toString()}`
     const data = await fetchHistoricalLogs({ url, accessToken: client.accessToken ?? '' })
     printHistoricalLogs(data, levelsToPrint)
     return
   }
 
-  const userId = command.netlify.globalConfig.get('userId')
+  const userId = command.netlify.globalConfig.get('userId') as string
 
   if (!deployId) {
     const deploys = await client.listSiteDeploys({ siteId })
@@ -51,15 +52,15 @@ export const logsEdgeFunction = async (options: OptionValues, command: BaseComma
     if (deploys.length === 1) {
       deployId = deploys[0].id
     } else {
-      const { result } = await inquirer.prompt({
+      const { result } = (await inquirer.prompt({
         name: 'result',
         type: 'list',
         message: `Select a deploy\n\n${chalk.yellow('*')} indicates a deploy created by you`,
-        choices: deploys.map((deploy: any) => ({
+        choices: deploys.map((deploy) => ({
           name: getName({ deploy, userId }),
           value: deploy.id,
         })),
-      })
+      })) as { result: string }
 
       deployId = result
     }
@@ -79,7 +80,7 @@ export const logsEdgeFunction = async (options: OptionValues, command: BaseComma
   })
 
   ws.on('message', (data: string) => {
-    const logData = JSON.parse(data)
+    const logData = JSON.parse(data) as { level: string; message: string; timestamp?: string }
     if (!levelsToPrint.includes(logData.level.toLowerCase())) {
       return
     }
@@ -90,7 +91,7 @@ export const logsEdgeFunction = async (options: OptionValues, command: BaseComma
     log('Connection closed')
   })
 
-  ws.on('error', (err: any) => {
+  ws.on('error', (err: Error) => {
     log('Connection error')
     log(err)
   })

src/commands/logs/log-api.ts

@@ -54,7 +54,7 @@ export function formatLogEntry(entry: { timestamp?: string; level?: string; mess
 }
 
 export function printHistoricalLogs(data: unknown, levelsToPrint: string[]): void {
-  const entries = Array.isArray(data) ? data : []
+  const entries = Array.isArray(data) ? (data as { timestamp?: string; level?: string; message?: string }[]) : []
 
   if (entries.length === 0) {
     log('No logs found for the specified time range')

tests/integration/commands/logs/edge-functions.test.ts

@@ -124,13 +124,12 @@ describe('logs:edge-functions command', () => {
     const setupCall = spyOn.mock.calls.find((args) => args[0] === 'open')
     expect(setupCall).toBeDefined()
 
-    const openCallback = setupCall?.[1]
+    const openCallback = setupCall?.[1] as (() => void) | undefined
     openCallback?.()
 
     expect(spySend).toHaveBeenCalledOnce()
-    const call = spySend.mock.calls[0]
-    const [message] = call
-    const body = JSON.parse(message)
+    const call = spySend.mock.calls[0] as string[]
+    const body = JSON.parse(call[0]) as Record<string, unknown>
 
     expect(body.deploy_id).toEqual('deploy-id-1')
     expect(body.site_id).toEqual('site_id')
@@ -154,11 +153,11 @@ describe('logs:edge-functions command', () => {
     await program.parseAsync(['', '', 'logs:edge-functions', '--deploy-id', 'my-deploy-id'])
 
     const setupCall = spyOn.mock.calls.find((args) => args[0] === 'open')
-    const openCallback = setupCall?.[1]
+    const openCallback = setupCall?.[1] as (() => void) | undefined
     openCallback?.()
 
-    const call = spySend.mock.calls[0]
-    const body = JSON.parse(call[0])
+    const call = spySend.mock.calls[0] as string[]
+    const body = JSON.parse(call[0]) as Record<string, unknown>
 
     expect(body.deploy_id).toEqual('my-deploy-id')
   })
@@ -179,7 +178,7 @@ describe('logs:edge-functions command', () => {
 
     await program.parseAsync(['', '', 'logs:edge-functions', '--level', 'info'])
     const messageCallback = spyOn.mock.calls.find((args) => args[0] === 'message')
-    const messageCallbackFunc = messageCallback?.[1]
+    const messageCallbackFunc = messageCallback?.[1] as ((data: string) => void) | undefined
 
     messageCallbackFunc?.(JSON.stringify({ level: LOG_LEVELS.INFO, message: 'Hello World' }))
     messageCallbackFunc?.(JSON.stringify({ level: LOG_LEVELS.WARN, message: 'There was a warning' }))
@@ -216,8 +215,8 @@ describe('logs:edge-functions command', () => {
         args[0].includes('analytics.services.netlify.com'),
       )
       expect(analyticsCall).toBeDefined()
-      expect(analyticsCall![0]).toContain('edge_function_logs')
-      expect(analyticsCall![0]).toContain('site_id')
+      expect((analyticsCall as string[])[0]).toContain('edge_function_logs')
+      expect((analyticsCall as string[])[0]).toContain('site_id')
     } finally {
       global.fetch = originalFetch
     }

tests/integration/commands/logs/functions.test.ts

@@ -296,9 +296,9 @@ describe('logs:function command', () => {
         return parsedUrl.hostname === 'analytics.services.netlify.com'
       })
       expect(analyticsCall).toBeDefined()
-      expect(analyticsCall![0]).toContain('function_logs')
-      expect(analyticsCall![0]).toContain('cool-function')
-      expect(analyticsCall![0]).toContain('site_id')
+      expect((analyticsCall as string[])[0]).toContain('function_logs')
+      expect((analyticsCall as string[])[0]).toContain('cool-function')
+      expect((analyticsCall as string[])[0]).toContain('site_id')
     } finally {
       global.fetch = originalFetch
     }