Add clickable popup details to team HTML report

Store actual item details (URL, title, number, repo) in RepoStats and
DeveloperRow instead of just counts, with URL-keyed deduplication across
components that share a GitHub repo. The team HTML now embeds this data
as JSON and shows a lightbox-style modal when clicking any nonzero cell,
listing the contributing PRs/issues/bugs/features/releases with links.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: ctrueden

src/pombast/team/_github.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 from collections import defaultdict
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from typing import TYPE_CHECKING
 
 from monoqueue.github import DEFAULT_CACHE_DIR, DEFAULT_MAX_AGE, fetch_items
@@ -12,12 +12,20 @@
     from pathlib import Path
 
 
+@dataclass
+class RepoItem:
+    url: str
+    title: str
+    number: int
+    repo: str  # e.g. "imagej/imagej-common"
+
+
 @dataclass
 class RepoStats:
-    prs: int = 0  # open non-draft PRs  (→ reviewer role)
-    issues: int = 0  # open issues, not labeled "question"  (→ support role)
-    bugs: int = 0  # open issues labeled "bug"  (→ debugger role)
-    enhancements: int = 0  # open issues labeled "enhancement", not milestone "unscheduled"  (→ developer role)
+    prs: list[RepoItem] = field(default_factory=list)
+    issues: list[RepoItem] = field(default_factory=list)
+    bugs: list[RepoItem] = field(default_factory=list)
+    enhancements: list[RepoItem] = field(default_factory=list)
 
 
 def fetch_repo_stats(
@@ -42,14 +50,20 @@ def fetch_repo_stats(
         slug = item["repository_url"].removeprefix("https://api.github.com/repos/")
         labels = {lbl["name"] for lbl in item.get("labels", [])}
         milestone = (item.get("milestone") or {}).get("title", "").lower()
+        ri = RepoItem(
+            url=item.get("html_url", ""),
+            title=item.get("title", ""),
+            number=item.get("number", 0),
+            repo=slug,
+        )
         if "pull_request" in item:
             if not item.get("draft", False):
-                stats[slug].prs += 1
+                stats[slug].prs.append(ri)
         else:
             if "question" not in labels:
-                stats[slug].issues += 1
+                stats[slug].issues.append(ri)
             if "bug" in labels:
-                stats[slug].bugs += 1
+                stats[slug].bugs.append(ri)
             if "enhancement" in labels and milestone != "unscheduled":
-                stats[slug].enhancements += 1
+                stats[slug].enhancements.append(ri)
     return dict(stats)

src/pombast/team/_html.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import json
 from typing import TYPE_CHECKING, Iterable
 
 from jinja2 import Environment, PackageLoader, select_autoescape
@@ -22,6 +23,15 @@ def _dev_link(row: DeveloperRow) -> str:
     return f'<a href="{url}">{label}</a>'
 
 
+def _item_dict(item) -> dict:
+    return {
+        "url": item.url,
+        "title": item.title,
+        "number": item.number,
+        "repo": item.repo,
+    }
+
+
 def generate_team_html(
     rows: Iterable[DeveloperRow],
     *,
@@ -30,10 +40,22 @@ def generate_team_html(
 ) -> str:
     """Return a complete HTML page with the team accountability table."""
 
+    popup_data: dict[str, dict] = {}
+
     def _row_data(r: DeveloperRow) -> dict:
+        dev_id = r.developer.id
+        popup_data[dev_id] = {
+            "reviewer_prs": [_item_dict(i) for i in r.reviewer_pr_items],
+            "support_issues": [_item_dict(i) for i in r.support_issue_items],
+            "debugger_bugs": [_item_dict(i) for i in r.debugger_bug_items],
+            "developer_features": [_item_dict(i) for i in r.developer_feature_items],
+            "maintainer_releases": [
+                {"ga": i.ga, "url": i.url} for i in r.maintainer_release_items
+            ],
+        }
         return {
             "dev_link": _dev_link(r),
-            "dev_id": r.developer.id,
+            "dev_id": dev_id,
             "reviewer_prs": r.reviewer_prs,
             "support_issues": r.support_issues,
             "debugger_bugs": r.debugger_bugs,
@@ -44,9 +66,11 @@ def _row_data(r: DeveloperRow) -> dict:
             "components": sorted(r.components),
         }
 
+    row_list = [_row_data(r) for r in rows]
     template = _env.get_template("team.html.j2")
     return template.render(
         title=title,
         generated=generated,
-        rows=[_row_data(r) for r in rows],
+        rows=row_list,
+        popup_data_json=json.dumps(popup_data) if popup_data else None,
     )

src/pombast/team/_workload.py

@@ -7,7 +7,7 @@
 
 if TYPE_CHECKING:
     from pombast.status._entry import StatusEntry
-    from pombast.team._github import RepoStats
+    from pombast.team._github import RepoItem, RepoStats
     from pombast.team._pom_devs import Developer
 
 # Roles that represent ongoing maintenance responsibility.
@@ -22,18 +22,63 @@
 }
 
 
+@dataclass
+class ReleaseItem:
+    ga: str  # e.g. "net.imagej:imagej-common"
+    url: str  # project URL (GitHub repo page)
+
+
 @dataclass
 class DeveloperRow:
     developer: Developer
-    reviewer_prs: int = 0  # open PRs to review (reviewer or lead)
-    support_issues: int = 0  # open issues to answer (support or lead)
-    debugger_bugs: int = 0  # open bug reports (debugger or lead)
-    developer_features: int = 0  # open enhancement requests (developer or lead)
-    maintainer_releases: int = (
-        0  # components with unreleased changes (maintainer or lead)
-    )
+    # Dicts keyed by URL for deduplication across components sharing a repo.
+    _prs: dict[str, RepoItem] = field(default_factory=dict)
+    _issues: dict[str, RepoItem] = field(default_factory=dict)
+    _bugs: dict[str, RepoItem] = field(default_factory=dict)
+    _features: dict[str, RepoItem] = field(default_factory=dict)
+    _releases: dict[str, ReleaseItem] = field(default_factory=dict)  # ga → item
     components: list[str] = field(default_factory=list)
 
+    @property
+    def reviewer_prs(self) -> int:
+        return len(self._prs)
+
+    @property
+    def reviewer_pr_items(self) -> list[RepoItem]:
+        return list(self._prs.values())
+
+    @property
+    def support_issues(self) -> int:
+        return len(self._issues)
+
+    @property
+    def support_issue_items(self) -> list[RepoItem]:
+        return list(self._issues.values())
+
+    @property
+    def debugger_bugs(self) -> int:
+        return len(self._bugs)
+
+    @property
+    def debugger_bug_items(self) -> list[RepoItem]:
+        return list(self._bugs.values())
+
+    @property
+    def developer_features(self) -> int:
+        return len(self._features)
+
+    @property
+    def developer_feature_items(self) -> list[RepoItem]:
+        return list(self._features.values())
+
+    @property
+    def maintainer_releases(self) -> int:
+        return len(self._releases)
+
+    @property
+    def maintainer_release_items(self) -> list[ReleaseItem]:
+        return list(self._releases.values())
+
     @property
     def total(self) -> int:
         return (
@@ -85,15 +130,19 @@ def build_workloads(
 
             if stats:
                 if is_lead or "reviewer" in roles:
-                    row.reviewer_prs += stats.prs
+                    for item in stats.prs:
+                        row._prs.setdefault(item.url, item)
                 if is_lead or "support" in roles:
-                    row.support_issues += stats.issues
+                    for item in stats.issues:
+                        row._issues.setdefault(item.url, item)
                 if is_lead or "debugger" in roles:
-                    row.debugger_bugs += stats.bugs
+                    for item in stats.bugs:
+                        row._bugs.setdefault(item.url, item)
                 if is_lead or "developer" in roles:
-                    row.developer_features += stats.enhancements
+                    for item in stats.enhancements:
+                        row._features.setdefault(item.url, item)
 
             if needs_release and (is_lead or "maintainer" in roles):
-                row.maintainer_releases += 1
+                row._releases.setdefault(ga, ReleaseItem(ga=ga, url=url))
 
     return sorted(rows.values(), key=lambda r: r.total, reverse=True)

src/pombast/team/templates/team.html.j2

@@ -11,6 +11,24 @@ td.nonzero { color: firebrick; font-weight: bold; }
 td.zero { color: #999; }
 td.total-cell { font-weight: bold; border-left: 1px solid gray; }
 th.role-header { text-align: center; }
+{%- if popup_data_json %}
+td.has-popup { cursor: pointer; text-decoration: underline dotted; text-underline-offset: 2px; }
+td.has-popup:hover { filter: brightness(1.3); }
+#team-modal { display: none; position: fixed; inset: 0; z-index: 9999; align-items: center; justify-content: center; }
+#team-modal.open { display: flex; }
+#team-backdrop { position: fixed; inset: 0; background: rgba(0,0,0,.5); }
+#team-box { position: relative; z-index: 1; background: #fff; border-radius: 6px; max-width: 700px; width: 90vw; max-height: 80vh; display: flex; flex-direction: column; box-shadow: 0 8px 32px rgba(0,0,0,.3); overflow: hidden; }
+#team-header { display: flex; align-items: center; gap: .5em; padding: .6em 1em; background: #f5f5f5; border-bottom: 1px solid #ddd; }
+#team-title { flex: 1; font-weight: bold; font-size: .95em; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+#team-close { background: none; border: none; color: #888; font-size: 1.4em; line-height: 1; cursor: pointer; padding: 0 .15em; flex-shrink: 0; }
+#team-close:hover { color: #333; }
+#team-list { margin: 0; padding: .75em 1em; overflow-y: auto; flex: 1; list-style: none; }
+#team-list li { padding: .3em 0; border-bottom: 1px solid #f0f0f0; font-size: .9em; }
+#team-list li:last-child { border-bottom: none; }
+#team-list li a { color: #0366d6; text-decoration: none; }
+#team-list li a:hover { text-decoration: underline; }
+.popup-repo { color: #888; font-size: .85em; margin-left: .4em; }
+{%- endif %}
 </style>
 </head>
 <body>
@@ -32,13 +50,13 @@ th.role-header { text-align: center; }
 <th class="role-header" title="Number of BOM components this developer is involved with (hover for list)">Components</th>
 </tr>
 {%- for row in rows %}
-<tr>
+<tr data-dev="{{ row.dev_id }}">
 <td>{{ row.dev_link | safe }}</td>
-<td class="{{ 'nonzero' if row.reviewer_prs > 0 else 'zero' }}">{{ row.reviewer_prs }}</td>
-<td class="{{ 'nonzero' if row.support_issues > 0 else 'zero' }}">{{ row.support_issues }}</td>
-<td class="{{ 'nonzero' if row.debugger_bugs > 0 else 'zero' }}">{{ row.debugger_bugs }}</td>
-<td class="{{ 'nonzero' if row.developer_features > 0 else 'zero' }}">{{ row.developer_features }}</td>
-<td class="{{ 'nonzero' if row.maintainer_releases > 0 else 'zero' }}">{{ row.maintainer_releases }}</td>
+<td class="{{ 'nonzero has-popup' if row.reviewer_prs > 0 else 'zero' }}" data-col="reviewer_prs">{{ row.reviewer_prs }}</td>
+<td class="{{ 'nonzero has-popup' if row.support_issues > 0 else 'zero' }}" data-col="support_issues">{{ row.support_issues }}</td>
+<td class="{{ 'nonzero has-popup' if row.debugger_bugs > 0 else 'zero' }}" data-col="debugger_bugs">{{ row.debugger_bugs }}</td>
+<td class="{{ 'nonzero has-popup' if row.developer_features > 0 else 'zero' }}" data-col="developer_features">{{ row.developer_features }}</td>
+<td class="{{ 'nonzero has-popup' if row.maintainer_releases > 0 else 'zero' }}" data-col="maintainer_releases">{{ row.maintainer_releases }}</td>
 <td class="total-cell">{{ row.total }}</td>
 <td title="{{ row.components | join(', ') }}">{{ row.component_count }}</td>
 </tr>
@@ -48,6 +66,73 @@ th.role-header { text-align: center; }
   <b>Note:</b> Lead developers are counted in all role columns for each of their components.
   GitHub data excludes draft PRs and issues labeled &ldquo;question&rdquo;.
   Enhancement counts exclude issues with milestone &ldquo;unscheduled&rdquo;.
+  Click any nonzero cell to see the contributing items.
 </p>
+{%- if popup_data_json %}
+<div id="team-modal">
+  <div id="team-backdrop"></div>
+  <div id="team-box">
+    <div id="team-header">
+      <span id="team-title"></span>
+      <button id="team-close">&times;</button>
+    </div>
+    <ul id="team-list"></ul>
+  </div>
+</div>
+<script>
+(function(){
+  var modal = document.getElementById('team-modal');
+  var titleEl = document.getElementById('team-title');
+  var list = document.getElementById('team-list');
+  var DATA = {{ popup_data_json | safe }};
+
+  var COL_LABELS = {
+    reviewer_prs: 'PRs to review',
+    support_issues: 'Issues to answer',
+    debugger_bugs: 'Bugs to fix',
+    developer_features: 'Features requested',
+    maintainer_releases: 'Releases needed'
+  };
+
+  function show(devId, col, devName) {
+    var items = DATA[devId] && DATA[devId][col];
+    if (!items || !items.length) return;
+    titleEl.textContent = devName + ' — ' + COL_LABELS[col] + ' (' + items.length + ')';
+    list.innerHTML = '';
+    items.forEach(function(item) {
+      var li = document.createElement('li');
+      if (col === 'maintainer_releases') {
+        var repoLabel = item.url ? item.url.replace('https://github.com/', '') : item.ga;
+        li.innerHTML = item.url
+          ? '<a href="' + item.url + '/commits" target="_blank">' + item.ga + '</a>'
+          : item.ga;
+      } else {
+        li.innerHTML = '<a href="' + item.url + '" target="_blank">#' + item.number + ' – ' + escHtml(item.title) + '</a>'
+          + '<span class="popup-repo">' + escHtml(item.repo) + '</span>';
+      }
+      list.appendChild(li);
+    });
+    modal.classList.add('open');
+  }
+  function hide() { modal.classList.remove('open'); }
+  function escHtml(s) {
+    return s.replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;').replace(/"/g,'&quot;');
+  }
+
+  document.querySelectorAll('td.has-popup').forEach(function(td) {
+    td.addEventListener('click', function() {
+      var tr = td.closest('tr');
+      var devId = tr.dataset.dev;
+      var col = td.dataset.col;
+      var devName = tr.querySelector('td:first-child').textContent.trim();
+      show(devId, col, devName);
+    });
+  });
+  document.getElementById('team-close').addEventListener('click', hide);
+  document.getElementById('team-backdrop').addEventListener('click', hide);
+  document.addEventListener('keydown', function(e) { if (e.key === 'Escape') hide(); });
+})();
+</script>
+{%- endif %}
 </body>
 </html>