Handle when workflow parsing fails

Author: JarLob

dist/index.js

@@ -32,7 +32,7 @@ async function run_repo(owner, repo, octokit) {
             continue;
         }
 
-        debug(`Checking branch ${branchName} for workflows.`);
+        debug(`Checking branch https://github.com/${owner}/${repo}/blob/${branchName} for workflows.`);
         let contents;
         try {
             contents = await octokit.rest.repos.getContent({
@@ -43,7 +43,7 @@ async function run_repo(owner, repo, octokit) {
             });
         } catch (error) {
             if (error.status === 404) {
-                debug(`Branch ${branchName} does not contain any workflows.`);
+                debug(`https://github.com/${owner}/${repo}/blob/${branchName} does not contain any workflows.`);
                 continue;
             }
             throw (`Error: ${error.status}. Failed to retrieve contents for branch ${branchName}`, error);
@@ -52,7 +52,7 @@ async function run_repo(owner, repo, octokit) {
         for (const file of contents.data) {
             const lowerCaseFile = file.name.toLowerCase();
             if (file.type === 'file' && (lowerCaseFile.endsWith('.yml') || lowerCaseFile.endsWith('.yaml'))) {
-                debug(`Retrieving workflow file ${file.path}.`);
+                debug(`Retrieving workflow file https://github.com/${owner}/${repo}/blob/${branchName}/${file.path}.`);
                 const fileContent = await octokit.rest.repos.getContent({
                     owner,
                     repo,
@@ -63,26 +63,31 @@ async function run_repo(owner, repo, octokit) {
                 const content = Buffer.from(fileContent.data.content, 'base64').toString('utf-8');
 
                 if (!content.includes('pull_request_target')) {
-                    debug(`No occurrence of 'pull_request_target' in ${file.path}.`);
+                    debug(`No occurrence of 'pull_request_target' in https://github.com/${owner}/${repo}/blob/${branchName}/${file.path}.`);
                     continue;
                 }
 
-                debug(`Parsing workflow file ${file.path}.`);
+                debug(`Parsing workflow file https://github.com/${owner}/${repo}/blob/${branchName}/${file.path}.`);
                 const result = parseWorkflow(
                     {
                         name: file.name,
                         content: content
                     },
                     new NoOperationTraceWriter()
                 );
+                if (result.value === undefined) {
+                    found = true;
+                    console.log(`Failed to parse https://github.com/${owner}/${repo}/blob/${branchName}/${file.path}.`);
+                    continue;
+                }
                 const workflow = await convertWorkflowTemplate(result.context, result.value);
                 if (workflow.events['pull_request_target'] === undefined) {
-                    debug(`No usage of 'pull_request_target' in ${file.path}.`);
+                    debug(`No usage of 'pull_request_target' in https://github.com/${owner}/${repo}/blob/${branchName}/${file.path}.`);
                     continue;
                 }
 
                 if (workflow.events['pull_request_target'].branches !== undefined) {
-                    debug(`${file.path} is using branch filter ${JSON.stringify(workflow.events['pull_request_target'].branches)}.`);
+                    debug(`https://github.com/${owner}/${repo}/blob/${branchName}/${file.path} is using branch filter ${JSON.stringify(workflow.events['pull_request_target'].branches)}.`);
                     continue;
                 }