Tool & Extension readpage fixes (#169)

* Add GitHub repository stats section to tool details page and remove source URL button

* Fix additional_info template to conditionally display site statistics

* Add Recent Activity section to extension and tool read pages; remove 'Datasets in Extension' section

- Added a GitHub Recent Activity sidebar section to read_extension.html and read_tool.html
  - Fetches the 5 most recent commits via GET /repos/{owner}/{repo}/commits?per_page=5
  - Displays commit message (truncated, linked to GitHub), author name, relative timestamp, and author's GitHub profile picture
  - Avatar falls back to github.com/ghost.png for unmatched git authors
  - Results cached in localStorage for 30 minutes to reduce API calls
  - Gracefully handles rate limit errors and empty responses
- Moved the 'View on GitHub' button from the Repository Stats section to the bottom of the Recent Activity section (links to /commits)
- Removed the 'Datasets in Extension' section from extension_info.html snippet as it was not relevant

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

* Extract GitHub stats/activity sidebar into shared snippet; switch to Events API

- Created snippets/github_sidebar.html as a single source of truth for all
  GitHub sidebar logic (Repository Stats + Recent Activity). Both read_extension.html
  and read_tool.html now include this snippet, passing github_url as the only
  parameter — eliminating duplicated CSS, JS, and HTML.
- Switched Recent Activity from the commits endpoint to the Events API
  (GET /repos/{owner}/events?per_page=5) to display richer activity:
  issues opened/closed, PRs, pushes, watches, forks, releases, etc.
  Each event shows: actor avatar, relative timestamp, human-readable action,
  and a linked detail line (e.g. "Issue #457: pyproject.toml").
- Shared JS helpers (timeAgo, extractRepo, fromCache, toCache) are defined
  once and reused by both the stats and activity blocks within the snippet.
- Description-source label styles (.ckanext-{extension,tool}-description-source)
  remain in their respective templates as they are type-specific.

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

---------

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

Author: a5dur

ckanext/pose_theme/custom_themes/pose_theme/templates/extension/snippets/extension_info.html

@@ -35,19 +35,5 @@ <h1 class="heading">{{ pkg.title or pkg.name }}</h1>
             </div>
         </section>
 
-        <section class="module module-narrow">
-            <h3 class="module-heading"><i class="fa fa-sitemap icon-medium icon-sitemap"></i> {{ _('Datasets in Extension') }}</h2>
-                {% if packages %}
-                    <ul class="nav nav-simple">
-                        {% for package in packages %}
-                            {% set truncate_title = truncate_title or 80 %}
-                            {% set title = package.title or package.name %}
-                            <li class="nav-item">{{ h.link_to(title|truncate(truncate_title), h.url_for('dataset.read', id=package.name)) }}</li>
-                        {% endfor %}
-                    </ul>
-                {% else %}
-                    <p class="module-content empty">{{_('There are no Datasets in this Extension')}}</p>
-                {% endif %}
-        </section>
     {% endif %}
 {% endblock %}

ckanext/pose_theme/custom_themes/pose_theme/templates/scheming/package/additional_info.html

@@ -21,16 +21,23 @@
     'sotm_month',
     ] -%}
 
-{% set org = h.get_organization(pkg_dict.owner_org) %} 
+{% set org = h.get_organization(pkg_dict.owner_org) %}
 {% block package_additional_info %}
 {% if h.check_access('package_update',{'id':pkg_dict.id}) %}
 
 {% endif %}
   {%- set always_show_fields = ['contributors', 'ckan_version', 'extension_type'] -%}
+  {%- set site_stat_fields = ['num_datasets', 'num_organizations', 'num_groups'] -%}
+  {%- set all_site_stats_zero = (
+      pkg_dict.get('num_datasets', '0') | string in ['0', '', 'None', 'none'] and
+      pkg_dict.get('num_organizations', '0') | string in ['0', '', 'None', 'none'] and
+      pkg_dict.get('num_groups', '0') | string in ['0', '', 'None', 'none']
+  ) -%}
   {%- for field in schema.dataset_fields -%}
     {%- set field_value = pkg_dict.get(field.field_name) -%}
     {%- if field.field_name not in exclude_fields
         and field.display_snippet is not none
+        and not (field.field_name in site_stat_fields and all_site_stats_zero)
         and (field.field_name in always_show_fields
              or (field_value is not none
                  and field_value != ''

ckanext/pose_theme/custom_themes/pose_theme/templates/scheming/package/read_extension.html

@@ -14,7 +14,7 @@
 
 {% block links -%}
     {{ super() }}
-    
+
 {% endblock -%}
 
 {% block head_extras -%}
@@ -142,119 +142,13 @@ <h1>
     {% endblock %}
 
     {% if pkg.url and 'github.com' in pkg.url %}
-    {% block package_github_stats %}
-    <section class="module module-narrow module-github-stats">
-        <h3 class="module-heading">
-            <i class="fab fa-github icon-medium"></i> {{ _('Repository Stats') }}
-        </h3>
-        <div class="module-content" id="github-stats-content" data-github-url="{{ pkg.url }}">
-            <p class="github-stats-loading text-muted">
-                <i class="fa fa-spinner fa-spin"></i> {{ _('Loading stats…') }}
-            </p>
-        </div>
-    </section>
     <style>
-        .module-github-stats .github-stats-list {
-            list-style: none;
-            padding: 0;
-            margin: 0 0 12px 0;
-        }
-        .module-github-stats .github-stats-list li {
-            padding: 5px 0;
-            border-bottom: 1px solid #f0f0f0;
-            font-size: 13px;
-            display: flex;
-            align-items: center;
-            gap: 8px;
-        }
-        .module-github-stats .github-stats-list li:last-child {
-            border-bottom: none;
-        }
-        .module-github-stats .github-stats-list .stat-value {
-            font-weight: bold;
-            margin-left: auto;
-        }
-        .module-github-stats .github-stats-list i {
-            width: 16px;
-            text-align: center;
-            color: #555;
-        }
-        .ckanext-extension-description-source {
-            margin-top: 8px;
-        }
-        .ckanext-extension-description-source .label {
-            font-size: 11px;
-            font-weight: normal;
-            letter-spacing: 0.02em;
-            padding: 3px 7px;
-        }
-        .module-github-stats .github-stats-error {
-            color: #999;
-            font-size: 13px;
-        }
+        .ckanext-extension-description-source { margin-top: 8px; }
+        .ckanext-extension-description-source .label { font-size: 11px; font-weight: normal; letter-spacing: 0.02em; padding: 3px 7px; }
     </style>
-    <script>
-    (function () {
-        var CACHE_TTL_MS = 60 * 60 * 1000; // 1 hour
-        var container = document.getElementById('github-stats-content');
-        if (!container) return;
-        var ghUrl = container.getAttribute('data-github-url') || '';
-        var match = ghUrl.match(/github\.com\/([^\/]+\/[^\/\?#]+)/);
-        if (!match) return;
-        var repo = match[1].replace(/\.git$/, '');
-        var cacheKey = 'gh_stats_' + repo.replace('/', '_');
-
-        function render(data) {
-            var updated = new Date(data.pushed_at).toLocaleDateString(undefined, { year: 'numeric', month: 'short', day: 'numeric' });
-            var licenseHtml = data.license
-                ? '<li><i class="fa fa-balance-scale"></i> {{ _("License") }} <span class="stat-value">' + data.license.spdx_id + '</span></li>'
-                : '';
-            container.innerHTML =
-                '<ul class="github-stats-list">' +
-                '<li><i class="fa fa-star"></i> {{ _("Stars") }} <span class="stat-value">' + data.stargazers_count.toLocaleString() + '</span></li>' +
-                '<li><i class="fa fa-code-fork"></i> {{ _("Forks") }} <span class="stat-value">' + data.forks_count.toLocaleString() + '</span></li>' +
-                '<li><i class="fa fa-exclamation-circle"></i> {{ _("Open Issues") }} <span class="stat-value">' + data.open_issues_count.toLocaleString() + '</span></li>' +
-                '<li><i class="fa fa-eye"></i> {{ _("Watchers") }} <span class="stat-value">' + data.subscribers_count.toLocaleString() + '</span></li>' +
-                licenseHtml +
-                '<li><i class="fa fa-clock-o"></i> {{ _("Last Push") }} <span class="stat-value">' + updated + '</span></li>' +
-                '</ul>' +
-                '<a href="' + data.html_url + '" class="btn btn-default btn-block" target="_blank" rel="noopener">' +
-                '<i class="fab fa-github"></i> {{ _("View on GitHub") }}</a>';
-        }
-
-        // Serve from localStorage cache if still fresh
-        try {
-            var cached = localStorage.getItem(cacheKey);
-            if (cached) {
-                var entry = JSON.parse(cached);
-                if (Date.now() - entry.ts < CACHE_TTL_MS) {
-                    render(entry.data);
-                    return;
-                }
-            }
-        } catch (e) {}
-
-        fetch('https://api.github.com/repos/' + repo, {
-            headers: { 'Accept': 'application/vnd.github.v3+json' }
-        })
-        .then(function (r) { return r.json(); })
-        .then(function (data) {
-            if (data.message) {
-                container.innerHTML = '<p class="github-stats-error"><i class="fa fa-exclamation-triangle"></i> ' + data.message + '</p>';
-                return;
-            }
-            try {
-                localStorage.setItem(cacheKey, JSON.stringify({ ts: Date.now(), data: data }));
-            } catch (e) {}
-            render(data);
-        })
-        .catch(function () {
-            container.innerHTML = '<p class="github-stats-error"><i class="fa fa-exclamation-triangle"></i> {{ _("Could not load stats.") }}</p>';
-        });
-    })();
-    </script>
+    {% block package_github_sidebar %}
+        {% snippet 'snippets/github_sidebar.html', github_url=pkg.url %}
     {% endblock %}
     {% endif %}
 
 {% endblock %}
-

ckanext/pose_theme/custom_themes/pose_theme/templates/scheming/package/read_tool.html

@@ -135,5 +135,14 @@ <h1>
         {% snippet 'tool/snippets/tool_info.html', pkg=pkg %}
     {% endblock %}
 
+    {% if pkg.source_url and 'github.com' in pkg.source_url %}
+    <style>
+        .ckanext-tool-description-source { margin-top: 8px; }
+        .ckanext-tool-description-source .label { font-size: 11px; font-weight: normal; letter-spacing: 0.02em; padding: 3px 7px; }
+    </style>
+    {% block package_github_sidebar %}
+        {% snippet 'snippets/github_sidebar.html', github_url=pkg.source_url %}
+    {% endblock %}
+    {% endif %}
 
 {% endblock %}