Merge pull request #3 from maxnorm/feat/improve-ci

dummy change for test PR

Author: maxnorm

.github/scripts/coverage-comment.js

@@ -86,23 +86,35 @@ function getBadgeColor(coverage) {
 /**
  * Generate coverage report comment body
  * @param {object} metrics - Coverage metrics
+ * @param {object} commitInfo - Optional commit information
  * @returns {string} Markdown formatted comment body
  */
-function generateCoverageReport(metrics) {
+function generateCoverageReport(metrics, commitInfo = {}) {
   const lineCoverage = calculateCoverage(metrics.coveredLines, metrics.totalLines);
   const functionCoverage = calculateCoverage(metrics.coveredFunctions, metrics.totalFunctions);
   const branchCoverage = calculateCoverage(metrics.coveredBranches, metrics.totalBranches);
 
   const badgeColor = getBadgeColor(lineCoverage);
   const badge = `![Coverage](https://img.shields.io/badge/coverage-${lineCoverage}%25-${badgeColor})`;
+  
+  // Generate timestamp
+  const timestamp = new Date().toUTCString();
+  
+  // Build commit link if info is available
+  let commitLink = '';
+  if (commitInfo.sha && commitInfo.owner && commitInfo.repo) {
+    const shortSha = commitInfo.sha.substring(0, 7);
+    commitLink = ` for commit [\`${shortSha}\`](https://github.com/${commitInfo.owner}/${commitInfo.repo}/commit/${commitInfo.sha})`;
+  }
 
   return `## Coverage Report\n` +
     `${badge}\n\n` +
     `| Metric | Coverage | Details |\n` +
     `|--------|----------|----------|\n` +
     `| **Lines** | ${lineCoverage}% | ${metrics.coveredLines}/${metrics.totalLines} lines |\n` +
     `| **Functions** | ${functionCoverage}% | ${metrics.coveredFunctions}/${metrics.totalFunctions} functions |\n` +
-    `| **Branches** | ${branchCoverage}% | ${metrics.coveredBranches}/${metrics.totalBranches} branches |\n\n`;
+    `| **Branches** | ${branchCoverage}% | ${metrics.coveredBranches}/${metrics.totalBranches} branches |\n\n` +
+    `*Last updated: ${timestamp}*${commitLink}\n`;
 }
 
 /**
@@ -157,7 +169,14 @@ function generateCoverageFile() {
   console.log('- Functions:', metrics.coveredFunctions, '/', metrics.totalFunctions);
   console.log('- Branches:', metrics.coveredBranches, '/', metrics.totalBranches);
 
-  const body = generateCoverageReport(metrics);
+  // Get commit info from environment variables
+  const commitInfo = {
+    sha: process.env.COMMIT_SHA,
+    owner: process.env.REPO_OWNER,
+    repo: process.env.REPO_NAME
+  };
+
+  const body = generateCoverageReport(metrics, commitInfo);
   fs.writeFileSync('coverage-report.md', body);
   console.log('Coverage report saved to coverage-report.md');
 }

.github/workflows/coverage-comment.yml

@@ -5,13 +5,13 @@ on:
     workflows: ["Coverage"]
     types:
       - completed
-
+ 
 permissions:
   actions: read
   pull-requests: write
   issues: write
   contents: read
-
+  
 jobs:
   comment:
     name: Post Coverage Comment

.github/workflows/coverage.yml

@@ -29,9 +29,16 @@ jobs:
 
       - name: Generate coverage report
         if: github.event_name == 'pull_request'
+        env:
+          COMMIT_SHA: ${{ github.event.pull_request.head.sha }}
+          REPO_OWNER: ${{ github.repository_owner }}
+          REPO_NAME: ${{ github.event.repository.name }}
         run: |
           node .github/scripts/coverage-comment.js
           echo "PR_NUMBER=${{ github.event.pull_request.number }}" >> coverage-data.txt
+          echo "COMMIT_SHA=${{ github.event.pull_request.head.sha }}" >> coverage-data.txt
+          echo "REPO_OWNER=${{ github.repository_owner }}" >> coverage-data.txt
+          echo "REPO_NAME=${{ github.event.repository.name }}" >> coverage-data.txt
           echo "Generated coverage report for PR #${{ github.event.pull_request.number }}"
 
       - name: Upload coverage data