ci: update artifact hub metadata for v0.1.2

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

artifacthub-pkg.yml

@@ -1,4 +1,4 @@
-version: 0.1.1
+version: 0.1.2
 name: headlamp-polaris-plugin
 displayName: Polaris
 createdAt: "2026-02-05T19:00:00Z"
@@ -28,7 +28,7 @@ maintainers:
   - name: cpfarhood
     email: "chris@farhood.org"
 annotations:
-  headlamp/plugin/archive-url: "https://github.com/cpfarhood/headlamp-polaris-plugin/releases/download/v0.1.1/headlamp-polaris-plugin-0.1.1.tar.gz"
+  headlamp/plugin/archive-url: "https://github.com/cpfarhood/headlamp-polaris-plugin/releases/download/v0.1.2/headlamp-polaris-plugin-0.1.2.tar.gz"
   headlamp/plugin/version-compat: ">=0.26"
-  headlamp/plugin/archive-checksum: sha256:5a73bbbcfce3d6158c54409e7673b2d08d8e60c21d11fd0114e33e8eb8eb58b5
+  headlamp/plugin/archive-checksum: sha256:c1dd28df92d8c64b0ddbb27ad4ef21c1e405e2036f43add3e8eb910362b26850
   headlamp/plugin/distro-compat: in-cluster

claude.md

@@ -23,6 +23,9 @@ npx tsc --noEmit
 
 # Lint
 npx eslint src/
+
+# Run tests
+npm test
 ```
 
 ## Architecture
@@ -32,15 +35,18 @@ src/
 ├── index.tsx                           # Entry point: registers sidebar entries + routes
 ├── api/
 │   ├── polaris.ts                      # Types (AuditData schema), usePolarisData hook, countResults utilities, refresh settings
+│   ├── polaris.test.ts                 # Unit tests for utility functions (vitest)
 │   └── PolarisDataContext.tsx           # React context provider for shared data fetch
 └── components/
-    ├── DashboardView.tsx                # Overview / Full Audit page (score, check summary, cluster info)
+    ├── DashboardView.tsx                # Overview page (score, check summary with skipped count, cluster info)
+    ├── NamespacesListView.tsx           # Namespace list with scores and links to detail views
     ├── NamespaceDetailView.tsx          # Per-namespace drill-down with resource table
-    ├── DynamicSidebarRegistrar.tsx       # Registers namespace sidebar entries from live audit data
     └── PolarisSettings.tsx              # Plugin settings (refresh interval selector)
 ```
 
-Top-level sidebar section at `/polaris` with sub-routes for full audit (`/polaris/full-audit`) and per-namespace views (`/polaris/ns/:namespace`). Data is fetched via `ApiProxy.request` to the Polaris dashboard service proxy and refreshed on a user-configurable interval (stored in localStorage under `polaris-plugin-refresh-interval`, default 5 minutes). Score is computed from result counts (pass/total).
+Top-level sidebar section at `/polaris` with sub-routes for namespaces list (`/polaris/namespaces`) and per-namespace views (`/polaris/ns/:namespace`). Data is fetched via `ApiProxy.request` to the Polaris dashboard service proxy and refreshed on a user-configurable interval (stored in localStorage under `polaris-plugin-refresh-interval`, default 5 minutes). Score is computed from result counts (pass/total). Skipped checks are always displayed in summaries.
+
+**Sidebar limitation**: Headlamp's sidebar only supports 2-level nesting (parent → children). The `Collapse` component is driven by route-based selection, not click-to-toggle, so 3-level hierarchies don't expand properly. Namespace navigation is handled via the in-content table on the Namespaces page instead.
 
 ## Security / RBAC Requirements
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "headlamp-polaris-plugin",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Headlamp plugin for Fairwinds Polaris audit results",
   "scripts": {
     "start": "headlamp-plugin start",
@@ -10,7 +10,9 @@
     "lint": "eslint --ext .ts,.tsx src/",
     "lint:fix": "eslint --ext .ts,.tsx --fix src/",
     "format": "prettier --write src/",
-    "format:check": "prettier --check src/"
+    "format:check": "prettier --check src/",
+    "test": "vitest run",
+    "test:watch": "vitest"
   },
   "devDependencies": {
     "@kinvolk/headlamp-plugin": "^0.13.0"

src/api/polaris.test.ts

@@ -0,0 +1,264 @@
+import { describe, expect, it, vi } from 'vitest';
+
+vi.mock('@kinvolk/headlamp-plugin/lib', () => ({
+  ApiProxy: { request: vi.fn() },
+}));
+
+import {
+  AuditData,
+  computeScore,
+  countResults,
+  countResultsForItems,
+  filterResultsByNamespace,
+  getNamespaces,
+  Result,
+  ResultCounts,
+} from './polaris';
+
+// --- Fixtures ---
+
+function makeResult(overrides: Partial<Result> = {}): Result {
+  return {
+    Name: 'my-deploy',
+    Namespace: 'default',
+    Kind: 'Deployment',
+    Results: {},
+    CreatedTime: '2025-01-01T00:00:00Z',
+    ...overrides,
+  };
+}
+
+function makeAuditData(results: Result[]): AuditData {
+  return {
+    PolarisOutputVersion: '1.0',
+    AuditTime: '2025-01-01T00:00:00Z',
+    SourceType: 'Cluster',
+    SourceName: 'test',
+    DisplayName: 'test',
+    ClusterInfo: { Version: '1.28', Nodes: 3, Pods: 10, Namespaces: 2, Controllers: 5 },
+    Results: results,
+  };
+}
+
+// --- computeScore ---
+
+describe('computeScore', () => {
+  it('returns 0 when total is 0', () => {
+    const counts: ResultCounts = { total: 0, pass: 0, warning: 0, danger: 0, skipped: 0 };
+    expect(computeScore(counts)).toBe(0);
+  });
+
+  it('returns 100 when all checks pass', () => {
+    const counts: ResultCounts = { total: 10, pass: 10, warning: 0, danger: 0, skipped: 0 };
+    expect(computeScore(counts)).toBe(100);
+  });
+
+  it('rounds to nearest integer', () => {
+    const counts: ResultCounts = { total: 3, pass: 1, warning: 1, danger: 1, skipped: 0 };
+    expect(computeScore(counts)).toBe(33);
+  });
+
+  it('includes skipped in total denominator', () => {
+    const counts: ResultCounts = { total: 10, pass: 5, warning: 2, danger: 1, skipped: 2 };
+    expect(computeScore(counts)).toBe(50);
+  });
+});
+
+// --- countResults / countResultsForItems ---
+
+describe('countResults', () => {
+  it('returns zero counts for empty results', () => {
+    const data = makeAuditData([]);
+    const counts = countResults(data);
+    expect(counts).toEqual({ total: 0, pass: 0, warning: 0, danger: 0, skipped: 0 });
+  });
+
+  it('counts top-level result set entries', () => {
+    const result = makeResult({
+      Results: {
+        check1: {
+          ID: 'check1',
+          Message: 'ok',
+          Details: [],
+          Success: true,
+          Severity: 'warning',
+          Category: 'Security',
+        },
+        check2: {
+          ID: 'check2',
+          Message: 'bad',
+          Details: [],
+          Success: false,
+          Severity: 'danger',
+          Category: 'Security',
+        },
+      },
+    });
+    const counts = countResults(makeAuditData([result]));
+    expect(counts.total).toBe(2);
+    expect(counts.pass).toBe(1);
+    expect(counts.danger).toBe(1);
+    expect(counts.warning).toBe(0);
+    expect(counts.skipped).toBe(0);
+  });
+
+  it('counts skipped (severity=ignore, success=false) entries', () => {
+    const result = makeResult({
+      Results: {
+        skipped1: {
+          ID: 'skipped1',
+          Message: 'skipped',
+          Details: [],
+          Success: false,
+          Severity: 'ignore',
+          Category: 'Security',
+        },
+      },
+    });
+    const counts = countResults(makeAuditData([result]));
+    expect(counts.total).toBe(1);
+    expect(counts.skipped).toBe(1);
+    expect(counts.pass).toBe(0);
+  });
+
+  it('counts PodResult and ContainerResults', () => {
+    const result = makeResult({
+      Results: {
+        top: {
+          ID: 'top',
+          Message: 'ok',
+          Details: [],
+          Success: true,
+          Severity: 'warning',
+          Category: 'Reliability',
+        },
+      },
+      PodResult: {
+        Name: 'pod-1',
+        Results: {
+          podCheck: {
+            ID: 'podCheck',
+            Message: 'warn',
+            Details: [],
+            Success: false,
+            Severity: 'warning',
+            Category: 'Reliability',
+          },
+        },
+        ContainerResults: [
+          {
+            Name: 'container-1',
+            Results: {
+              containerCheck: {
+                ID: 'containerCheck',
+                Message: 'danger',
+                Details: [],
+                Success: false,
+                Severity: 'danger',
+                Category: 'Security',
+              },
+            },
+          },
+        ],
+      },
+    });
+    const counts = countResults(makeAuditData([result]));
+    expect(counts.total).toBe(3);
+    expect(counts.pass).toBe(1);
+    expect(counts.warning).toBe(1);
+    expect(counts.danger).toBe(1);
+  });
+
+  it('aggregates across multiple results', () => {
+    const r1 = makeResult({
+      Name: 'deploy-a',
+      Results: {
+        c1: {
+          ID: 'c1',
+          Message: '',
+          Details: [],
+          Success: true,
+          Severity: 'warning',
+          Category: 'X',
+        },
+      },
+    });
+    const r2 = makeResult({
+      Name: 'deploy-b',
+      Results: {
+        c2: {
+          ID: 'c2',
+          Message: '',
+          Details: [],
+          Success: false,
+          Severity: 'warning',
+          Category: 'X',
+        },
+      },
+    });
+    const counts = countResults(makeAuditData([r1, r2]));
+    expect(counts.total).toBe(2);
+    expect(counts.pass).toBe(1);
+    expect(counts.warning).toBe(1);
+  });
+});
+
+describe('countResultsForItems', () => {
+  it('works on a subset of results', () => {
+    const results: Result[] = [
+      makeResult({
+        Results: {
+          a: {
+            ID: 'a',
+            Message: '',
+            Details: [],
+            Success: false,
+            Severity: 'danger',
+            Category: 'X',
+          },
+        },
+      }),
+    ];
+    const counts = countResultsForItems(results);
+    expect(counts.danger).toBe(1);
+    expect(counts.total).toBe(1);
+  });
+});
+
+// --- getNamespaces ---
+
+describe('getNamespaces', () => {
+  it('returns empty array for no results', () => {
+    expect(getNamespaces(makeAuditData([]))).toEqual([]);
+  });
+
+  it('returns sorted unique namespaces', () => {
+    const data = makeAuditData([
+      makeResult({ Namespace: 'beta' }),
+      makeResult({ Namespace: 'alpha' }),
+      makeResult({ Namespace: 'beta' }),
+      makeResult({ Namespace: 'gamma' }),
+    ]);
+    expect(getNamespaces(data)).toEqual(['alpha', 'beta', 'gamma']);
+  });
+});
+
+// --- filterResultsByNamespace ---
+
+describe('filterResultsByNamespace', () => {
+  it('returns only results matching the namespace', () => {
+    const data = makeAuditData([
+      makeResult({ Name: 'a', Namespace: 'ns1' }),
+      makeResult({ Name: 'b', Namespace: 'ns2' }),
+      makeResult({ Name: 'c', Namespace: 'ns1' }),
+    ]);
+    const filtered = filterResultsByNamespace(data, 'ns1');
+    expect(filtered).toHaveLength(2);
+    expect(filtered.map(r => r.Name)).toEqual(['a', 'c']);
+  });
+
+  it('returns empty array for non-existent namespace', () => {
+    const data = makeAuditData([makeResult({ Namespace: 'ns1' })]);
+    expect(filterResultsByNamespace(data, 'ns-missing')).toEqual([]);
+  });
+});

src/components/DashboardView.tsx

@@ -6,7 +6,7 @@ import {
   StatusLabel,
 } from '@kinvolk/headlamp-plugin/lib/CommonComponents';
 import React from 'react';
-import { AuditData, countResults, ResultCounts } from '../api/polaris';
+import { AuditData, computeScore, countResults, ResultCounts } from '../api/polaris';
 import { usePolarisDataContext } from '../api/PolarisDataContext';
 
 function scoreStatus(score: number): 'success' | 'warning' | 'error' {
@@ -15,41 +15,11 @@ function scoreStatus(score: number): 'success' | 'warning' | 'error' {
   return 'error';
 }
 
-function OverviewSection(props: {
-  data: AuditData;
-  counts: ResultCounts;
-  includeSkipped: boolean;
-}) {
-  const { counts, includeSkipped } = props;
-
-  const displayTotal = includeSkipped ? counts.total : counts.total - counts.skipped;
-  const displayPass = counts.pass;
-  const score = displayTotal === 0 ? 0 : Math.round((displayPass / displayTotal) * 100);
+function OverviewSection(props: { data: AuditData; counts: ResultCounts }) {
+  const { counts } = props;
+  const score = computeScore(counts);
   const status = scoreStatus(score);
 
-  const summaryRows: { name: string; value: React.ReactNode }[] = [
-    { name: 'Total Checks', value: String(displayTotal) },
-    {
-      name: 'Pass',
-      value: <StatusLabel status="success">{counts.pass}</StatusLabel>,
-    },
-    {
-      name: 'Warning',
-      value: <StatusLabel status="warning">{counts.warning}</StatusLabel>,
-    },
-    {
-      name: 'Danger',
-      value: <StatusLabel status="error">{counts.danger}</StatusLabel>,
-    },
-  ];
-
-  if (includeSkipped) {
-    summaryRows.push({
-      name: 'Skipped',
-      value: <StatusLabel status="">{counts.skipped}</StatusLabel>,
-    });
-  }
-
   return (
     <>
       <SectionBox title="Score">
@@ -63,7 +33,27 @@ function OverviewSection(props: {
         />
       </SectionBox>
       <SectionBox title="Check Summary">
-        <NameValueTable rows={summaryRows} />
+        <NameValueTable
+          rows={[
+            { name: 'Total Checks', value: String(counts.total) },
+            {
+              name: 'Pass',
+              value: <StatusLabel status="success">{counts.pass}</StatusLabel>,
+            },
+            {
+              name: 'Warning',
+              value: <StatusLabel status="warning">{counts.warning}</StatusLabel>,
+            },
+            {
+              name: 'Danger',
+              value: <StatusLabel status="error">{counts.danger}</StatusLabel>,
+            },
+            {
+              name: 'Skipped',
+              value: <StatusLabel status="">{counts.skipped}</StatusLabel>,
+            },
+          ]}
+        />
       </SectionBox>
       <SectionBox title="Cluster Info">
         <NameValueTable
@@ -79,9 +69,8 @@ function OverviewSection(props: {
   );
 }
 
-export default function DashboardView(props: { includeSkipped: boolean }) {
+export default function DashboardView() {
   const { data, loading, error } = usePolarisDataContext();
-  const title = props.includeSkipped ? 'Polaris — Full Audit' : 'Polaris — Overview';
 
   if (loading) {
     return <Loader title="Loading Polaris audit data..." />;
@@ -91,7 +80,7 @@ export default function DashboardView(props: { includeSkipped: boolean }) {
 
   return (
     <>
-      <SectionHeader title={title} />
+      <SectionHeader title="Polaris — Overview" />
 
       {error && (
         <SectionBox title="Error">
@@ -106,9 +95,7 @@ export default function DashboardView(props: { includeSkipped: boolean }) {
         </SectionBox>
       )}
 
-      {data && counts && (
-        <OverviewSection data={data} counts={counts} includeSkipped={props.includeSkipped} />
-      )}
+      {data && counts && <OverviewSection data={data} counts={counts} />}
 
       {!data && !error && (
         <SectionBox title="No Data">

src/components/NamespaceDetailView.tsx

@@ -118,6 +118,10 @@ export default function NamespaceDetailView() {
               name: 'Danger',
               value: <StatusLabel status="error">{counts.danger}</StatusLabel>,
             },
+            {
+              name: 'Skipped',
+              value: <StatusLabel status="">{counts.skipped}</StatusLabel>,
+            },
           ]}
         />
       </SectionBox>

src/components/NamespacesListView.tsx

@@ -1,4 +1,5 @@
 import {
+  Link,
   Loader,
   NameValueTable,
   SectionBox,
@@ -7,7 +8,6 @@ import {
   StatusLabel,
 } from '@kinvolk/headlamp-plugin/lib/CommonComponents';
 import React from 'react';
-import { Link } from 'react-router-dom';
 import {
   computeScore,
   countResultsForItems,
@@ -28,6 +28,7 @@ interface NamespaceRow {
   pass: number;
   warning: number;
   danger: number;
+  skipped: number;
 }
 
 export default function NamespacesListView() {
@@ -77,6 +78,7 @@ export default function NamespacesListView() {
       pass: counts.pass,
       warning: counts.warning,
       danger: counts.danger,
+      skipped: counts.skipped,
     };
   });
 
@@ -89,7 +91,9 @@ export default function NamespacesListView() {
             {
               label: 'Namespace',
               getter: (row: NamespaceRow) => (
-                <Link to={`/polaris/ns/${row.namespace}`}>{row.namespace}</Link>
+                <Link routeName="polaris-namespace" params={{ namespace: row.namespace }}>
+                  {row.namespace}
+                </Link>
               ),
             },
             {
@@ -100,9 +104,7 @@ export default function NamespacesListView() {
             },
             {
               label: 'Pass',
-              getter: (row: NamespaceRow) => (
-                <StatusLabel status="success">{row.pass}</StatusLabel>
-              ),
+              getter: (row: NamespaceRow) => <StatusLabel status="success">{row.pass}</StatusLabel>,
             },
             {
               label: 'Warning',
@@ -112,9 +114,11 @@ export default function NamespacesListView() {
             },
             {
               label: 'Danger',
-              getter: (row: NamespaceRow) => (
-                <StatusLabel status="error">{row.danger}</StatusLabel>
-              ),
+              getter: (row: NamespaceRow) => <StatusLabel status="error">{row.danger}</StatusLabel>,
+            },
+            {
+              label: 'Skipped',
+              getter: (row: NamespaceRow) => <StatusLabel status="">{row.skipped}</StatusLabel>,
             },
           ]}
           data={rows}

src/index.tsx

@@ -28,14 +28,6 @@ registerSidebarEntry({
   icon: 'mdi:view-dashboard',
 });
 
-registerSidebarEntry({
-  parent: 'polaris',
-  name: 'polaris-full',
-  label: 'Full Audit',
-  url: '/polaris/full-audit',
-  icon: 'mdi:clipboard-text-search',
-});
-
 registerSidebarEntry({
   parent: 'polaris',
   name: 'polaris-namespaces',
@@ -53,19 +45,7 @@ registerRoute({
   exact: true,
   component: () => (
     <PolarisDataProvider>
-      <DashboardView includeSkipped={false} />
-    </PolarisDataProvider>
-  ),
-});
-
-registerRoute({
-  path: '/polaris/full-audit',
-  sidebar: 'polaris-full',
-  name: 'polaris-full-audit',
-  exact: true,
-  component: () => (
-    <PolarisDataProvider>
-      <DashboardView includeSkipped />
+      <DashboardView />
     </PolarisDataProvider>
   ),
 });

vitest.config.mts

@@ -0,0 +1,8 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({
+  test: {
+    globals: true,
+    environment: 'jsdom',
+  },
+});