add logs to azure pipelines

Author: nsdeschenes

packages/bundler-plugin-core/src/utils/providers/AzurePipelines.ts

@@ -12,54 +12,78 @@ export function detect(envs: ProviderEnvs): boolean {
   return Boolean(envs?.SYSTEM_TEAMFOUNDATIONSERVERURI);
 }
 
-function _getBuild(inputs: ProviderUtilInputs): ProviderServiceParams["build"] {
+function _getBuild(
+  inputs: ProviderUtilInputs,
+  output: Output,
+): ProviderServiceParams["build"] {
   const { args, envs } = inputs;
   if (args?.build && args.build !== "") {
+    debug(`Using build: ${args.build}`, { enabled: output.debug });
     return args.build;
   }
-  return envs?.BUILD_BUILDNUMBER ?? null;
+
+  const build = envs?.BUILD_BUILDNUMBER ?? null;
+  debug(`Using build: ${build}`, { enabled: output.debug });
+  return build;
 }
 
 function _getBuildURL(
   inputs: ProviderUtilInputs,
+  output: Output,
 ): ProviderServiceParams["buildURL"] {
   const { envs } = inputs;
+  let buildURL: string | null = null;
   if (envs?.SYSTEM_TEAMPROJECT && envs?.BUILD_BUILDID) {
-    return `${envs?.SYSTEM_TEAMFOUNDATIONSERVERURI}${envs?.SYSTEM_TEAMPROJECT}/_build/results?buildId=${envs?.BUILD_BUILDID}`;
+    buildURL = `${envs?.SYSTEM_TEAMFOUNDATIONSERVERURI}${envs?.SYSTEM_TEAMPROJECT}/_build/results?buildId=${envs?.BUILD_BUILDID}`;
   }
-  return null;
+  debug(`Using build URL: ${buildURL}`, { enabled: output.debug });
+  return buildURL;
 }
 
 function _getBranch(
   inputs: ProviderUtilInputs,
+  output: Output,
 ): ProviderServiceParams["branch"] {
   const { args, envs } = inputs;
   if (args?.branch && args.branch !== "") {
+    debug(`Using branch: ${args.branch}`, { enabled: output.debug });
     return args.branch;
   }
 
+  let branch: string | null = null;
   if (envs?.BUILD_SOURCEBRANCH) {
-    return envs?.BUILD_SOURCEBRANCH.toString().replace("refs/heads/", "");
+    branch = envs?.BUILD_SOURCEBRANCH.toString().replace("refs/heads/", "");
   }
 
-  return null;
+  debug(`Using branch: ${branch}`, { enabled: output.debug });
+  return branch;
 }
 
-function _getJob(envs: ProviderEnvs): ProviderServiceParams["job"] {
-  return envs?.BUILD_BUILDID ?? null;
+function _getJob(
+  envs: ProviderEnvs,
+  output: Output,
+): ProviderServiceParams["job"] {
+  const job = envs?.BUILD_BUILDID ?? null;
+  debug(`Using job: ${job}`, { enabled: output.debug });
+  return job;
 }
 
-function _getPR(inputs: ProviderUtilInputs): ProviderServiceParams["pr"] {
+function _getPR(
+  inputs: ProviderUtilInputs,
+  output: Output,
+): ProviderServiceParams["pr"] {
   const { args, envs } = inputs;
   if (args?.pr && args.pr !== "") {
+    debug(`Using PR: ${args.pr}`, { enabled: output.debug });
     return args.pr;
   }
 
-  return (
+  const pr =
     envs?.SYSTEM_PULLREQUEST_PULLREQUESTNUMBER ??
     envs?.SYSTEM_PULLREQUEST_PULLREQUESTID ??
-    null
-  );
+    null;
+  debug(`Using PR: ${pr}`, { enabled: output.debug });
+  return pr;
 }
 
 function _getService(): ProviderServiceParams["service"] {
@@ -84,7 +108,7 @@ function _getSHA(
 
   let commit = envs?.BUILD_SOURCEVERSION ?? null;
 
-  if (_getPR(inputs)) {
+  if (_getPR(inputs, output)) {
     const mergeCommitRegex = /^[a-z0-9]{40} [a-z0-9]{40}$/;
     const mergeCommitMessage = childProcess
       .execFileSync("git", ["show", "--no-patch", "--format=%P"])
@@ -105,20 +129,24 @@ function _getSHA(
     }
   }
 
-  debug(`Using commit: ${commit}`, {
-    enabled: output.debug,
-  });
-
+  debug(`Using commit: ${commit}`, { enabled: output.debug });
   return commit;
 }
 
-function _getSlug(inputs: ProviderUtilInputs): ProviderServiceParams["slug"] {
+function _getSlug(
+  inputs: ProviderUtilInputs,
+  output: Output,
+): ProviderServiceParams["slug"] {
   const { args, envs } = inputs;
   if (args?.slug && args.slug !== "") {
+    debug(`Using slug: ${args.slug}`, { enabled: output.debug });
     return args.slug;
   }
 
-  return envs?.BUILD_REPOSITORY_NAME ?? parseSlugFromRemoteAddr("") ?? null;
+  const slug =
+    envs?.BUILD_REPOSITORY_NAME ?? parseSlugFromRemoteAddr("") ?? null;
+  debug(`Using slug: ${slug}`, { enabled: output.debug });
+  return slug;
 }
 
 // eslint-disable-next-line @typescript-eslint/require-await
@@ -127,14 +155,14 @@ export async function getServiceParams(
   output: Output,
 ): Promise<ProviderServiceParams> {
   return {
-    branch: _getBranch(inputs),
-    build: _getBuild(inputs),
-    buildURL: _getBuildURL(inputs),
+    branch: _getBranch(inputs, output),
+    build: _getBuild(inputs, output),
+    buildURL: _getBuildURL(inputs, output),
     commit: _getSHA(inputs, output),
-    job: _getJob(inputs.envs),
-    pr: _getPR(inputs),
+    job: _getJob(inputs.envs, output),
+    pr: _getPR(inputs, output),
     service: _getService(),
-    slug: _getSlug(inputs),
+    slug: _getSlug(inputs, output),
   };
 }