Lift all utility functions in runner scripts

1 files changed, 11 insertions, 15 deletions

diff --git a/worker/scripts/build_docker_image.ts b/worker/scripts/build_docker_image.ts
index 5dd5b4e..1783e7c 100755
--- a/worker/scripts/build_docker_image.ts
+++ b/worker/scripts/build_docker_image.ts
@@ -7,6 +7,7 @@ import {
     LogMetricTraceable,
     Metric,
     TraceUtil,
+    Command,
 } from '@emprespresso/pengueno';
 import type { BuildDockerImageJob, BuildDockerImageJobProps } from '@emprespresso/ci_model';
 import { Bitwarden, type LoginItem } from '@emprespresso/ci_worker';
@@ -26,7 +27,7 @@ const eitherJob = getRequiredEnvVars([
             arguments: baseArgs,
         },
 );
-console.log(eitherJob);
+
 const eitherVault = Bitwarden.getConfigFromEnvironment().mapRight((config) => new Bitwarden(config));
 
 const buildImageMetric = Metric.fromName('dockerImage.build').asResult();
@@ -105,19 +106,14 @@ await LogMetricTraceable.ofLogTraceable(_logJob)
     })
     .get();
 
-const getDockerLoginCommand = (username: string, registry: string) =>
-    `docker login --username ${username} --password $REGISTRY_PASSWORD ${registry}`.split(' ');
+function getDockerLoginCommand(username: string, registry: string): Command {
+    return `docker login --username ${username} --password $REGISTRY_PASSWORD ${registry}`.split(' ');
+}
 
-const getBuildCommand = ({ buildTarget, imageTag, dockerfile, context }: BuildDockerImageJobProps) => [
-    'docker',
-    'build',
-    '--target',
-    buildTarget,
-    '-t',
-    imageTag,
-    '-f',
-    dockerfile,
-    context,
-];
+function getBuildCommand({ buildTarget, imageTag, dockerfile, context }: BuildDockerImageJobProps): Command {
+    return ['docker', 'build', '--target', buildTarget, '-t', imageTag, '-f', dockerfile, context];
+}
 
-const getPushCommand = (tag: string) => ['docker', 'push', tag];
+function getPushCommand(tag: string): Command {
+    return ['docker', 'push', tag];
+}