Skip to content

Add debug logging to GHA #1981

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
riaankleinhans merged 1 commit into from
Dec 3, 2025
+24 −2
Merged

Add debug logging to GHA #1981

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 24 additions & 2 deletions .github/workflows/create-tech-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,20 +104,33 @@ jobs:
const issueNumber = issue.number;
const issueBody = issue.body;

console.log(`🔍 Processing issue #${issueNumber}`);
console.log(`📄 Issue body length: ${issueBody ? issueBody.length : 0}`);

// Extract fields
const projectName = extractFormField(issueBody, 'name');
const projectLink = extractFormField(issueBody, 'projectLink');
const ddLink = extractFormField(issueBody, 'ddLink');
const projectContact = extractFormField(issueBody, 'projectContact');
const additionalInfo = extractFormField(issueBody, 'additionalInfo');

console.log(`📋 Extracted fields:
projectName: ${projectName || 'NOT FOUND'}
projectLink: ${projectLink || 'NOT FOUND'}
ddLink: ${ddLink || 'NOT FOUND'}
projectContact: ${projectContact || 'NOT FOUND'}
additionalInfo: ${additionalInfo ? 'FOUND' : 'NOT FOUND'}
`);

// Validate required fields
if (!projectName) {
console.log('❌ Missing project name - commenting and exiting');
await commentOnce(issueNumber, COMMENT_MARKERS.missingProjectName,
`❌ Could not extract project name from issue body. Please ensure the issue was created from a template with a "Project name" field.`);
return;
}
if (!projectLink) {
console.log('❌ Missing project link - commenting and exiting');
await commentOnce(issueNumber, COMMENT_MARKERS.missingProjectLink,
`❌ Could not extract project link from issue body. Please ensure the issue contains a "Project link" or "GitHub URL" field.`);
return;
Expand All @@ -127,8 +140,10 @@ jobs:
const projectNameLower = normalize(projectName);

// Check if tech review already created for this issue
if (await hasExistingComment(issueNumber, COMMENT_MARKERS.techReviewCreated)) {
console.log('ℹ️ Tech review issue already created for this issue.');
const hasExistingCommentCheck = await hasExistingComment(issueNumber, COMMENT_MARKERS.techReviewCreated);
console.log(`🔍 Checked for existing comment: ${hasExistingCommentCheck}`);
if (hasExistingCommentCheck) {
console.log('ℹ️ Tech review issue already created for this issue - exiting');
return;
}

Expand All @@ -155,9 +170,12 @@ jobs:
});

if (existingIssue) {
console.log(`ℹ️ Found existing tech review issue #${existingIssue.number} - commenting and exiting`);
await commentOnce(issueNumber, COMMENT_MARKERS.techReviewExists,
`A tech review issue already exists for this project: [#${existingIssue.number} - ${existingIssue.title}](${existingIssue.html_url})`);
return;
} else {
console.log('✅ No existing tech review issue found - proceeding to create');
}
} catch (error) {
console.log('⚠️ Error checking for existing issues:', error.message);
Expand All @@ -173,6 +191,7 @@ jobs:
issueBodyContent += `---\n\n_This issue was automatically created from [issue #${issueNumber}](${originalIssueUrl})_`;

// Create tech review issue
console.log(`🚀 Attempting to create tech review issue for: ${normalizedProjectName}`);
try {
const newIssue = await github.rest.issues.create({
owner: context.repo.owner,
Expand All @@ -182,9 +201,12 @@ jobs:
labels: LABELS_TECH_REVIEW
});

console.log(`✅ Successfully created issue #${newIssue.data.number}`);
await commentOnce(issueNumber, COMMENT_MARKERS.techReviewCreated,
`✅ Created tech review issue: [#${newIssue.data.number} - ${newIssue.data.title}](${newIssue.data.html_url})`);
} catch (error) {
console.error(`❌ Error creating issue: ${error.message}`);
console.error(`❌ Error stack: ${error.stack}`);
await commentOnce(issueNumber, COMMENT_MARKERS.createTechReviewFailed,
`❌ Failed to create tech review issue: ${error.message}`);
throw error;
Expand Down