Improve team command behavior

* lead role behavior — is_lead logic removed from _workload.py.
  The lead role now only causes a developer to appear in the table
  (they're counted in components), but doesn't inherit
  PRs/issues/bugs/features/releases from other roles. The note "Lead
  developers are counted in all role columns" is gone from the HTML.

* Configurable role names — New TeamConfig dataclass with six role keys
  (lead, developer, debugger, reviewer, support, maintainer), each
  defaulting to its own name as a single-element list. Set globally in
  [team] or per-component in [components."g:a"], accepting either a
  string or list of strings.

* [team] includes/excludes — TeamConfig has includes and excludes
  list fields, parsed from [team] in pombast.toml, using the same
  ComponentFilter glob logic as [smelt].

* Components popup — The Components count cell is now clickable (using
  the same modal as the other cells). Each component links to its
  project URL if available, or shows the G:A as plain text.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: ctrueden

src/pombast/cli/_team.py

@@ -247,7 +247,7 @@ def _gh_progress(fetched: int, total: int) -> None:
         console.print(f"  Got stats for {len(repo_stats)} repos.")
 
     # Phase 4: aggregate and display
-    workload_rows = build_workloads(entries, dev_roles, repo_stats)
+    workload_rows = build_workloads(entries, dev_roles, repo_stats, pombast_config)
 
     table = Table(title="Team Workload", show_lines=False)
     table.add_column("Developer", style="cyan", no_wrap=True)

src/pombast/config/_settings.py

@@ -12,6 +12,15 @@
     import tomli as tomllib
 
 
+def _parse_role_value(value: object, default: str) -> list[str]:
+    """Coerce a TOML role value (str or list[str]) to list[str]."""
+    if isinstance(value, str):
+        return [value]
+    if isinstance(value, list):
+        return [str(v) for v in value]
+    return [default]
+
+
 def parse_repo_spec(spec: str, fallback_id: str) -> tuple[str, str]:
     """Parse 'id=url' or bare 'url'; return (repo_id, url)."""
     name, sep, url = spec.partition("=")
@@ -35,6 +44,27 @@ class FilterConfig:
     excludes: list[str] = field(default_factory=list)
 
 
+_TEAM_ROLE_KEYS = ("lead", "developer", "debugger", "reviewer", "support", "maintainer")
+
+
+@dataclass
+class TeamConfig:
+    """Configuration for the team command, including role mappings."""
+
+    includes: list[str] = field(default_factory=list)
+    excludes: list[str] = field(default_factory=list)
+    lead: list[str] = field(default_factory=lambda: ["lead"])
+    developer: list[str] = field(default_factory=lambda: ["developer"])
+    debugger: list[str] = field(default_factory=lambda: ["debugger"])
+    reviewer: list[str] = field(default_factory=lambda: ["reviewer"])
+    support: list[str] = field(default_factory=lambda: ["support"])
+    maintainer: list[str] = field(default_factory=lambda: ["maintainer"])
+
+    def role_mapping(self) -> dict[str, list[str]]:
+        """Return {semantic_key: [pom_role_strings]} for all role keys."""
+        return {key: getattr(self, key) for key in _TEAM_ROLE_KEYS}
+
+
 @dataclass
 class StatusConfig:
     """Configuration for the status command."""
@@ -71,6 +101,7 @@ class PombastConfig:
     component_overrides: dict[str, dict[str, object]] = field(default_factory=dict)
     mega_melt: MegaMeltConfig = field(default_factory=MegaMeltConfig)
     status: StatusConfig = field(default_factory=StatusConfig)
+    team: TeamConfig = field(default_factory=TeamConfig)
 
     @classmethod
     def load(cls, path: Path) -> PombastConfig:
@@ -116,6 +147,17 @@ def resolve(section: dict, key: str) -> Path | None:
             nexus_base=status_data.get("nexus-base", ""),
         )
 
+        team_data = data.get("team", {})
+        team_config = TeamConfig(
+            includes=team_data.get("includes", []),
+            excludes=team_data.get("excludes", []),
+            **{
+                key: _parse_role_value(team_data[key], key)
+                for key in _TEAM_ROLE_KEYS
+                if key in team_data
+            },
+        )
+
         return cls(
             filter=filter_config,
             default_java=int(default_java) if default_java is not None else None,
@@ -126,6 +168,7 @@ def resolve(section: dict, key: str) -> Path | None:
             component_overrides={k: v for k, v in data.get("components", {}).items()},
             mega_melt=mega_melt_config,
             status=status_config,
+            team=team_config,
         )
 
     @classmethod

src/pombast/team/_html.py

@@ -52,6 +52,9 @@ def _row_data(r: DeveloperRow) -> dict:
             "maintainer_releases": [
                 {"ga": i.ga, "url": i.url} for i in r.maintainer_release_items
             ],
+            "components": [
+                {"ga": ga, "url": url} for ga, url in r.component_url_items
+            ],
         }
         return {
             "dev_link": _dev_link(r),
@@ -63,7 +66,6 @@ def _row_data(r: DeveloperRow) -> dict:
             "maintainer_releases": r.maintainer_releases,
             "total": r.total,
             "component_count": len(r.components),
-            "components": sorted(r.components),
         }
 
     row_list = [_row_data(r) for r in rows]

src/pombast/team/_workload.py

@@ -6,21 +6,11 @@
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
+    from pombast.config._settings import PombastConfig
     from pombast.status._entry import StatusEntry
     from pombast.team._github import RepoItem, RepoStats
     from pombast.team._pom_devs import Developer
 
-# Roles that represent ongoing maintenance responsibility.
-# "founder" is excluded — it's historical credit, not active duty.
-MAINTENANCE_ROLES = {
-    "lead",
-    "developer",
-    "debugger",
-    "reviewer",
-    "support",
-    "maintainer",
-}
-
 
 @dataclass
 class ReleaseItem:
@@ -38,6 +28,7 @@ class DeveloperRow:
     _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)
+    _component_urls: dict[str, str] = field(default_factory=dict)  # ga → project URL
 
     @property
     def reviewer_prs(self) -> int:
@@ -79,6 +70,10 @@ def maintainer_releases(self) -> int:
     def maintainer_release_items(self) -> list[ReleaseItem]:
         return list(self._releases.values())
 
+    @property
+    def component_url_items(self) -> list[tuple[str, str]]:
+        return [(ga, self._component_urls.get(ga, "")) for ga in sorted(self.components)]
+
     @property
     def total(self) -> int:
         return (
@@ -90,21 +85,63 @@ def total(self) -> int:
         )
 
 
+def _effective_role_mapping(
+    ga: str,
+    pombast_config: PombastConfig,
+) -> dict[str, set[str]]:
+    """Return {semantic_key: {pom_role_strings}} for this component.
+
+    Starts from global [team] role mappings and applies per-component overrides
+    from [components."g:a"] sections.
+    """
+    base: dict[str, set[str]] = {
+        key: set(vals) for key, vals in pombast_config.team.role_mapping().items()
+    }
+    ov = pombast_config.component_overrides.get(ga, {})
+    for key in base:
+        if key in ov:
+            val = ov[key]
+            base[key] = {val} if isinstance(val, str) else set(str(v) for v in val)
+    return base
+
+
+def _semantic_roles(
+    pom_roles: set[str],
+    mapping: dict[str, set[str]],
+) -> set[str]:
+    """Map POM role strings to semantic role keys using the given mapping."""
+    return {key for key, pom_names in mapping.items() if pom_roles & pom_names}
+
+
 def build_workloads(
     entries: list[StatusEntry],
     dev_roles: dict[str, list[tuple[Developer, set[str]]]],
     repo_stats: dict[str, RepoStats],
+    pombast_config: PombastConfig | None = None,
 ) -> list[DeveloperRow]:
     """Build per-developer workload rows sorted by total workload descending.
 
     Args:
         entries: StatusEntry list from query_status (provides release status per component).
         dev_roles: G:A → [(Developer, roles)] from component POM <developers> sections.
         repo_stats: GitHub repo slug → RepoStats from the GitHub search API.
+        pombast_config: Full pombast config (for role mappings, team includes/excludes).
     """
+    from pombast.config._settings import PombastConfig
+    from pombast.core._filter import ComponentFilter
+
+    if pombast_config is None:
+        pombast_config = PombastConfig.empty()
+
+    team_cfg = pombast_config.team
+    team_filter = ComponentFilter(includes=team_cfg.includes, excludes=team_cfg.excludes)
+
     rows: dict[str, DeveloperRow] = {}
 
     for entry in entries:
+        if not team_filter.is_included(entry.component):
+            continue
+
         ga = entry.component.ga
         url = entry.project_url or ""
         slug = (
@@ -115,8 +152,11 @@ def build_workloads(
         stats = repo_stats.get(slug) if slug else None
         needs_release = entry.action == "Cut"
 
-        for dev, roles in dev_roles.get(ga, []):
-            if not (roles & MAINTENANCE_ROLES):
+        role_mapping = _effective_role_mapping(ga, pombast_config)
+
+        for dev, pom_roles in dev_roles.get(ga, []):
+            semantic = _semantic_roles(pom_roles, role_mapping)
+            if not semantic:
                 continue
 
             if dev.id not in rows:
@@ -125,24 +165,23 @@ def build_workloads(
 
             if ga not in row.components:
                 row.components.append(ga)
-
-            is_lead = "lead" in roles
+            row._component_urls.setdefault(ga, url)
 
             if stats:
-                if is_lead or "reviewer" in roles:
+                if "reviewer" in semantic:
                     for item in stats.prs:
                         row._prs.setdefault(item.url, item)
-                if is_lead or "support" in roles:
+                if "support" in semantic:
                     for item in stats.issues:
                         row._issues.setdefault(item.url, item)
-                if is_lead or "debugger" in roles:
+                if "debugger" in semantic:
                     for item in stats.bugs:
                         row._bugs.setdefault(item.url, item)
-                if is_lead or "developer" in roles:
+                if "developer" in semantic:
                     for item in stats.enhancements:
                         row._features.setdefault(item.url, item)
 
-            if needs_release and (is_lead or "maintainer" in roles):
+            if needs_release and "maintainer" in semantic:
                 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

@@ -47,7 +47,7 @@ td.has-popup:hover { filter: brightness(1.3); }
 <th class="role-header" title="Open issues labeled &quot;enhancement&quot;, excluding unscheduled milestone (developer role)">Features&nbsp;requested</th>
 <th class="role-header" title="Components with unreleased commits (maintainer role)">Releases&nbsp;needed</th>
 <th class="role-header" title="Sum of all role columns">Total</th>
-<th class="role-header" title="Number of BOM components this developer is involved with (hover for list)">Components</th>
+<th class="role-header" title="Number of BOM components this developer is involved with (click for list)">Components</th>
 </tr>
 {%- for row in rows %}
 <tr data-dev="{{ row.dev_id }}">
@@ -58,15 +58,14 @@ td.has-popup:hover { filter: brightness(1.3); }
 <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>
+<td class="{{ 'has-popup' if row.component_count > 0 else '' }}" data-col="components">{{ row.component_count }}</td>
 </tr>
 {%- endfor %}
 </table>
 <p style="margin: 1em; font-size: 0.8em; color: #777;">
-  <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;.
+  <b>Note:</b> 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.
+  Click any nonzero cell to see the contributing items; click the Components count to see all associated components.
 </p>
 {%- if popup_data_json %}
 <div id="team-modal">
@@ -91,7 +90,8 @@ td.has-popup:hover { filter: brightness(1.3); }
     support_issues: 'Issues to answer',
     debugger_bugs: 'Bugs to fix',
     developer_features: 'Features requested',
-    maintainer_releases: 'Releases needed'
+    maintainer_releases: 'Releases needed',
+    components: 'Components'
   };
 
   function show(devId, col, devName) {
@@ -101,11 +101,10 @@ td.has-popup:hover { filter: brightness(1.3); }
     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;
+      if (col === 'maintainer_releases' || col === 'components') {
         li.innerHTML = item.url
-          ? '<a href="' + item.url + '/commits" target="_blank">' + item.ga + '</a>'
-          : item.ga;
+          ? '<a href="' + (col === 'maintainer_releases' ? item.url + '/commits' : item.url) + '" target="_blank">' + escHtml(item.ga) + '</a>'
+          : escHtml(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>';