pr feedback

Author: frostebite

dist/index.js

@@ -3,12 +3,16 @@ import * as core from '@actions/core';
 import {
   CloudFormation,
   CreateStackCommand,
+  // eslint-disable-next-line import/named
   CreateStackCommandInput,
   DescribeStacksCommand,
+  // eslint-disable-next-line import/named
   DescribeStacksCommandInput,
   ListStacksCommand,
+  // eslint-disable-next-line import/named
   Parameter,
   UpdateStackCommand,
+  // eslint-disable-next-line import/named
   UpdateStackCommandInput,
   waitUntilStackCreateComplete,
   waitUntilStackUpdateComplete,

dist/index.js.map

@@ -1,6 +1,7 @@
 import {
   CloudFormation,
   CreateStackCommand,
+  // eslint-disable-next-line import/named
   CreateStackCommandInput,
   DescribeStackResourcesCommand,
   DescribeStacksCommand,

src/model/cloud-runner/providers/aws/aws-base-stack.ts

@@ -1,3 +1,4 @@
+// eslint-disable-next-line import/named
 import { StackResource } from '@aws-sdk/client-cloudformation';
 
 class CloudRunnerAWSTaskDef {

src/model/cloud-runner/providers/aws/aws-job-stack.ts

@@ -4,6 +4,7 @@ import {
   ListStacksCommand,
 } from '@aws-sdk/client-cloudformation';
 import type { StackSummary } from '@aws-sdk/client-cloudformation';
+// eslint-disable-next-line import/named
 import { DescribeLogGroupsCommand, DescribeLogGroupsCommandInput } from '@aws-sdk/client-cloudwatch-logs';
 import type { LogGroup } from '@aws-sdk/client-cloudwatch-logs';
 import { DescribeTasksCommand, ListClustersCommand, ListTasksCommand } from '@aws-sdk/client-ecs';
@@ -201,9 +202,9 @@ export class TaskService {
   public static async getLocks(): Promise<Array<{ Key: string }>> {
     process.env.AWS_REGION = Input.region;
     if (CloudRunner.buildParameters.storageProvider === 'rclone') {
-      const objects = await (
-        SharedWorkspaceLocking as unknown as { listObjects(prefix: string): Promise<string[]> }
-      ).listObjects('');
+      // eslint-disable-next-line no-unused-vars
+      type ListObjectsFunction = (prefix: string) => Promise<string[]>;
+      const objects = await (SharedWorkspaceLocking as unknown as { listObjects: ListObjectsFunction }).listObjects('');
 
       return objects.map((x: string) => ({ Key: x }));
     }

src/model/cloud-runner/providers/aws/cloud-runner-aws-task-def.ts

@@ -61,9 +61,9 @@ class KubernetesPods {
       }
 
       // Check if only PreStopHook failed but container succeeded
-      const hasPreStopHookFailure = events.some((e) => e.reason === 'FailedPreStopHook');
-      const wasKilled = events.some((e) => e.reason === 'Killing');
-      const hasExceededGracePeriod = events.some((e) => e.reason === 'ExceededGracePeriod');
+      const hasPreStopHookFailure = events.some((event) => event.reason === 'FailedPreStopHook');
+      const wasKilled = events.some((event) => event.reason === 'Killing');
+      const hasExceededGracePeriod = events.some((event) => event.reason === 'ExceededGracePeriod');
 
       // If container succeeded (exit code 0), PreStopHook failure is non-critical
       // Also check if pod was killed but container might have succeeded

src/model/cloud-runner/providers/aws/services/task-service.ts

@@ -143,7 +143,10 @@ export class RemoteClient {
         const uniqueJobFolderLinux = CloudRunnerFolders.ToLinuxFolder(
           CloudRunnerFolders.uniqueCloudRunnerJobFolderAbsolute,
         );
-        if (fs.existsSync(CloudRunnerFolders.uniqueCloudRunnerJobFolderAbsolute) || fs.existsSync(uniqueJobFolderLinux)) {
+        if (
+          fs.existsSync(CloudRunnerFolders.uniqueCloudRunnerJobFolderAbsolute) ||
+          fs.existsSync(uniqueJobFolderLinux)
+        ) {
           await CloudRunnerSystem.Run(`rm -r ${uniqueJobFolderLinux} || true`);
         } else {
           RemoteClientLogger.log(`Skipping cleanup; unique job folder missing`);

src/model/cloud-runner/providers/k8s/kubernetes-pods.ts

@@ -81,7 +81,7 @@ export class SharedWorkspaceLocking {
         const output = await SharedWorkspaceLocking.rclone(`lsjson ${path}`);
         const json = JSON.parse(output) as { Name: string; IsDir: boolean }[];
 
-        return json.map((e) => (e.IsDir ? `${e.Name}/` : e.Name));
+        return json.map((entry) => (entry.IsDir ? `${entry.Name}/` : entry.Name));
       } catch {
         return [];
       }

src/model/cloud-runner/remote-client/index.ts

@@ -76,7 +76,9 @@ describe('Cloud Runner pre-built rclone steps', () => {
         try {
           const lines = await CloudRunnerSystem.RunAndReadLines(`rclone lsf ${remote}`);
           CloudRunnerLogger.log(lines.join(','));
-        } catch {}
+        } catch {
+          // Ignore errors when listing remote root (best-effort validation)
+        }
       }, 1_000_000_000);
     } else {
       it.skip('Run build and prebuilt rclone steps - rclone not configured', () => {