fix: check exists too early

src/__tests__/action.test.ts

@@ -18,6 +18,8 @@ jest.mock('@actions/core', () => {
 			.fn()
 			.mockImplementation((key: string) => (inputs as any)[key]),
 		debug: jest.fn(),
+		startGroup: jest.fn(),
+		endGroup: jest.fn(),
 	};
 });
 

src/check.ts

@@ -1,10 +1,14 @@
+// Packages
+import * as core from '@actions/core';
+
 // Ours
 import { ActionContext } from './types';
 
 import {
 	IssueManager,
 	DependencyResolver,
 	DependencyExtractor,
+	formatDependency,
 } from './helpers';
 
 export async function checkIssues(context: ActionContext) {
@@ -15,16 +19,27 @@ export async function checkIssues(context: ActionContext) {
 	const resolver = new DependencyResolver(client, context.issues, repo);
 
 	for (const issue of context.issues) {
+		core.startGroup(`Checking #${issue.number}`);
 		let dependencies = extractor.fromIssue(issue);
 
 		if (dependencies.length === 0) {
+			core.debug('No dependencies found. Running clean-up');
 			await manager.removeLabel(issue);
-			await manager.removeAnyComments(issue);
-			return await manager.updateCommitStatus(issue, []);
+			await manager.removeActionComments(issue);
+			await manager.updateCommitStatus(issue, []);
+
+			core.endGroup();
+			continue;
 		}
 
 		let isBlocked = false;
 
+		core.debug(
+			`Depends on: ${dependencies
+				.map((dep) => formatDependency(dep, repo))
+				.join(', ')}`
+		);
+
 		dependencies = await Promise.all(
 			dependencies.map(async (dep) => {
 				const issue = await resolver.get(dep);
@@ -36,17 +51,30 @@ export async function checkIssues(context: ActionContext) {
 			})
 		);
 
+		core.debug(
+			`Blocked by: ${dependencies
+				.filter((dep) => dep.blocker)
+				.map((dep) => formatDependency(dep, repo))
+				.join(', ')}`
+		);
+
+		core.debug('Updating labels');
 		// Toggle label
 		isBlocked
 			? await manager.addLabel(issue)
 			: await manager.removeLabel(issue);
 
+		core.debug('Updating Action comments');
 		await manager.writeComment(
 			issue,
 			manager.generateComment(dependencies, dependencies, config),
 			!isBlocked
 		);
 
+		core.debug(
+			`Updating PR status${issue.pull_request ? '' : '. Skipped'}`
+		);
 		await manager.updateCommitStatus(issue, dependencies);
+		core.endGroup();
 	}
 }

src/context.ts

@@ -6,6 +6,8 @@ import * as github from '@actions/github';
 import { ActionContext, Issue } from './types';
 
 export async function getActionContext(): Promise<ActionContext> {
+	core.startGroup('Context');
+
 	const config = {
 		actionName: 'Dependent Issues',
 		actionRepoURL: 'https://github.com/z0al/dependent-issues',
@@ -36,6 +38,7 @@ export async function getActionContext(): Promise<ActionContext> {
 
 	// Only run checks for the context.issue (if any)
 	if (issue?.number) {
+		core.debug(`Payload issue: #${issue?.number}`);
 		const remoteIssue = (
 			await client.issues.get({ ...repo, issue_number: issue.number })
 		).data;
@@ -48,6 +51,7 @@ export async function getActionContext(): Promise<ActionContext> {
 
 	// Otherwise, check all open issues
 	else {
+		core.debug(`Payload issue: None`);
 		const options = {
 			...repo,
 			state: 'open' as 'open',
@@ -60,8 +64,11 @@ export async function getActionContext(): Promise<ActionContext> {
 				: client.pulls.list;
 
 		issues = (await client.paginate(method, options)) as Issue[];
+		core.debug(`No. of open issues: ${issues.length}`);
 	}
 
+	core.endGroup();
+
 	return {
 		client,
 		config,

src/helpers.ts

@@ -282,7 +282,7 @@ export class IssueManager {
 			  });
 	}
 
-	async removeAnyComments(issue: Issue) {
+	async removeActionComments(issue: Issue) {
 		const issueComments = await this.gh.paginate(
 			this.gh.issues.listComments,
 			{ ...this.repo, issue_number: issue.number, per_page: 100 }