Redesign PR comment: shields.io badges, score bar, GitHub admonitions

Replace plain table with shields.io colored badges per signal, Unicode
score bar, and native GitHub admonition blocks for tier assessment.
Each badge is contextually colored (green/blue/orange/red) based on
signal strength. Suspicious patterns use CAUTION/WARNING admonitions.

Author: iamtoruk

dist/index.js

@@ -1,14 +1,42 @@
 import { Assessment, ContributorSignals, Tier } from './types';
 
 const TIER_LABELS: Record<Tier, string> = {
-  trusted: 'Trusted',
-  familiar: 'Familiar',
-  caution: 'Review Suggested',
-  unknown: 'Unknown Contributor',
+  trusted: 'TRUSTED',
+  familiar: 'FAMILIAR',
+  caution: 'REVIEW SUGGESTED',
+  unknown: 'UNKNOWN',
+};
+
+const TIER_BADGE_COLORS: Record<Tier, string> = {
+  trusted: 'brightgreen',
+  familiar: 'yellow',
+  caution: 'orange',
+  unknown: 'red',
+};
+
+const TIER_ALERT: Record<Tier, string> = {
+  trusted: 'NOTE',
+  familiar: 'TIP',
+  caution: 'WARNING',
+  unknown: 'CAUTION',
 };
 
 export const COMMENT_MARKER = '<!-- firstlook-assessment -->';
 
+function shieldsParam(text: string): string {
+  return text
+    .replace(/-/g, '--')
+    .replace(/_/g, '__')
+    .replace(/ /g, '_')
+    .replace(/\//g, '%2F');
+}
+
+function badge(label: string, value: string, color: string, style = 'flat-square'): string {
+  const l = shieldsParam(label);
+  const v = shieldsParam(value);
+  return `![${label}](https://img.shields.io/badge/${l}-${v}-${color}?style=${style})`;
+}
+
 function ageText(days: number): string {
   if (days >= 365) {
     const years = Math.floor(days / 365);
@@ -19,67 +47,87 @@ function ageText(days: number): string {
   return `${days} days`;
 }
 
-function indicator(good: boolean): string {
-  return good ? ':white_check_mark:' : ':warning:';
+function scoreBar(score: number): string {
+  const filled = Math.round(score / 10);
+  return '\u2588'.repeat(filled) + '\u2591'.repeat(10 - filled);
 }
 
-function profileText(profile: ContributorSignals['profile']): string {
-  const fields: string[] = [];
-  if (profile.bio) fields.push('Bio');
-  if (profile.company) fields.push('Company');
-  if (profile.blog) fields.push('Blog');
-  if (profile.twitter) fields.push('Twitter');
-  if (profile.email) fields.push('Email');
-  return fields.length > 0 ? fields.join(', ') : 'None provided';
-}
+function signalBadges(s: ContributorSignals): string[] {
+  const badges: string[] = [];
+
+  badges.push(badge('account', ageText(s.accountAgeDays),
+    s.accountAgeDays >= 365 ? 'blue' : s.accountAgeDays >= 180 ? 'blue' : s.accountAgeDays >= 30 ? 'orange' : 'red'));
+
+  badges.push(badge('repos', `${s.publicRepos}`,
+    s.publicRepos >= 3 ? 'blue' : s.publicRepos >= 1 ? 'orange' : 'red'));
+
+  badges.push(badge('merged', `${s.mergedPRs}`,
+    s.mergedPRs >= 10 ? 'brightgreen' : s.mergedPRs >= 3 ? 'blue' : s.mergedPRs >= 1 ? 'orange' : 'red'));
+
+  badges.push(badge('rejected', `${s.closedPRs}`,
+    s.closedPRs === 0 ? 'blue' : s.closedPRs <= s.mergedPRs ? 'orange' : 'red'));
+
+  badges.push(badge('unique mergers', `${s.uniqueMergers}`,
+    s.uniqueMergers >= 3 ? 'brightgreen' : s.uniqueMergers >= 1 ? 'blue' : 'lightgrey'));
 
-function mergeQualityText(s: ContributorSignals): string {
-  const parts: string[] = [];
-  if (s.uniqueMergers > 0) parts.push(`${s.uniqueMergers} unique mergers`);
-  if (s.highStarRepos > 0) parts.push(`${s.highStarRepos} repos with 100+ stars`);
-  if (parts.length > 0) return parts.join(', ');
-  if (s.selfMergeCount > 0) return `${s.selfMergeCount} self-merged only`;
-  return 'No merge data';
+  badges.push(badge('100%2B%E2%98%85 repos', `${s.highStarRepos}`,
+    s.highStarRepos >= 3 ? 'brightgreen' : s.highStarRepos >= 1 ? 'blue' : 'lightgrey'));
+
+  badges.push(badge('activity', `${s.activeMonths}/${s.totalMonths} mo`,
+    s.activeMonths >= 6 ? 'blue' : s.activeMonths >= 3 ? 'blue' : s.activeMonths >= 1 ? 'orange' : 'red'));
+
+  badges.push(badge('followers', `${s.followers}`,
+    s.followers >= 10 ? 'blue' : s.followers >= 3 ? 'blue' : 'lightgrey'));
+
+  badges.push(badge('signed', s.commitsSigned ? 'yes' : 'no',
+    s.commitsSigned ? 'brightgreen' : 'orange'));
+
+  if (s.profile.filledCount > 0) {
+    const fields: string[] = [];
+    if (s.profile.bio) fields.push('bio');
+    if (s.profile.company) fields.push('co');
+    if (s.profile.blog) fields.push('blog');
+    if (s.profile.twitter) fields.push('tw');
+    if (s.profile.email) fields.push('email');
+    badges.push(badge('profile', fields.join(' '), 'blue'));
+  }
+
+  for (const f of s.securityFiles.slice(0, 3)) {
+    badges.push(badge('security', f, 'red'));
+  }
+  if (s.securityFiles.length > 3) {
+    badges.push(badge('security', `+${s.securityFiles.length - 3} more`, 'red'));
+  }
+
+  return badges;
 }
 
 export function buildComment(assessment: Assessment): string {
   const { signals: s, tier, score, summary, patterns } = assessment;
 
-  const rows = [
-    `| **Account** | Created ${ageText(s.accountAgeDays)} ago | ${indicator(s.accountAgeDays >= 180)} |`,
-    `| **Repos** | ${s.publicRepos} public | ${indicator(s.publicRepos >= 3)} |`,
-    `| **Profile** | ${profileText(s.profile)} | ${indicator(s.profile.filledCount >= 1)} |`,
-    `| **History** | ${s.mergedPRs} merged, ${s.closedPRs} rejected elsewhere | ${indicator(s.mergedPRs >= 3 && s.closedPRs <= s.mergedPRs)} |`,
-    `| **Merge quality** | ${mergeQualityText(s)} | ${indicator(s.uniqueMergers >= 1 || s.highStarRepos >= 1)} |`,
-    `| **Activity** | ${s.activeMonths}/${s.totalMonths} months active | ${indicator(s.activeMonths >= 3)} |`,
-    `| **Followers** | ${s.followers} | ${indicator(s.followers >= 3)} |`,
-    `| **Signed** | ${s.commitsSigned ? 'Yes' : 'No'} | ${indicator(s.commitsSigned)} |`,
-  ];
+  const tierBadge = badge(TIER_LABELS[tier], `${score}%2F100`, TIER_BADGE_COLORS[tier], 'for--the--badge');
+  const bar = scoreBar(score);
+  const badges = signalBadges(s);
 
-  if (s.securityFiles.length > 0) {
-    const fileList = s.securityFiles.slice(0, 5).map(f => `\`${f}\``).join(', ');
-    const extra = s.securityFiles.length > 5 ? ` (+${s.securityFiles.length - 5} more)` : '';
-    rows.push(`| **Security paths** | ${fileList}${extra} | :rotating_light: |`);
-  }
-
-  const lines = [
+  const lines: string[] = [
     COMMENT_MARKER,
-    '### firstlook',
     '',
-    '| Signal | Detail | |',
-    '|--------|--------|---|',
-    ...rows,
+    `### firstlook &nbsp; ${tierBadge}`,
+    '',
+    `\`${bar}\``,
+    '',
+    badges.join(' '),
   ];
 
   if (patterns.length > 0) {
     lines.push('');
     for (const p of patterns) {
-      const icon = p.severity === 'critical' ? ':rotating_light:' : ':warning:';
-      lines.push(`${icon} **${p.name}** -- ${p.detail}`);
+      const alertType = p.severity === 'critical' ? 'CAUTION' : 'WARNING';
+      lines.push(`> [!${alertType}]`, `> **${p.name}** -- ${p.detail}`, '');
     }
   }
 
-  lines.push('', `> **${TIER_LABELS[tier]}** (score: ${score}/100) -- ${summary}`);
+  lines.push('', `> [!${TIER_ALERT[tier]}]`, `> ${summary}`);
 
   const details: string[] = [];
   if (s.codeReviews > 0) details.push(`- Code reviews given: ${s.codeReviews}`);