From 39ef32a9ac453852cf330461cdaf525e761d3258 Mon Sep 17 00:00:00 2001
From: Zack Rosen <69942576+zjrosen@users.noreply.github.com>
Date: Sun, 23 Nov 2025 17:58:58 -0500
Subject: [PATCH] fix: use nullish coalescing to fix priority 0 bug display

Fixes display bug where P0 issues were shown as P2 due to JavaScript treating 0 as falsy. Also adds P0 to the priority filter dropdown.

Changes:
- Replace || with ?? operator for priority defaults in monitor UI
- Add P0 option to priority filter selector

Co-authored-by: Zack Rosen <zjrosen@users.noreply.github.com>
---
 examples/monitor-webui/web/index.html       | 1 +
 examples/monitor-webui/web/static/js/app.js | 8 ++++----
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/examples/monitor-webui/web/index.html b/examples/monitor-webui/web/index.html
index bbc46fb5..07d626a7 100644
--- a/examples/monitor-webui/web/index.html
+++ b/examples/monitor-webui/web/index.html
@@ -64,6 +64,7 @@
                 <div class="filter-group">
                     <label for="filter-priority">Priority</label>
                     <select id="filter-priority" multiple>
+                        <option value="0" selected>P0</option>
                         <option value="1" selected>P1</option>
                         <option value="2" selected>P2</option>
                         <option value="3" selected>P3</option>
diff --git a/examples/monitor-webui/web/static/js/app.js b/examples/monitor-webui/web/static/js/app.js
index 202d418c..6209efbb 100644
--- a/examples/monitor-webui/web/static/js/app.js
+++ b/examples/monitor-webui/web/static/js/app.js
@@ -127,18 +127,18 @@ function renderIssues(issues) {
     // Render table view
     tbody.innerHTML = issues.map(issue => {
         const statusClass = 'status-' + (issue.status || 'open').toLowerCase().replace('_', '-');
-        const priorityClass = 'priority-' + (issue.priority || 2);
-        return '<tr onclick="showIssueDetail(\'' + issue.id + '\')"><td>' + issue.id + '</td><td>' + issue.title + '</td><td class="' + statusClass + '">' + (issue.status || 'open') + '</td><td class="' + priorityClass + '">P' + (issue.priority || 2) + '</td><td>' + (issue.issue_type || 'task') + '</td><td>' + (issue.assignee || '-') + '</td></tr>';
+        const priorityClass = 'priority-' + (issue.priority ?? 2);
+        return '<tr onclick="showIssueDetail(\'' + issue.id + '\')"><td>' + issue.id + '</td><td>' + issue.title + '</td><td class="' + statusClass + '">' + (issue.status || 'open') + '</td><td class="' + priorityClass + '">P' + (issue.priority ?? 2) + '</td><td>' + (issue.issue_type || 'task') + '</td><td>' + (issue.assignee || '-') + '</td></tr>';
     }).join('');
 
     // Render card view for mobile
     cardView.innerHTML = issues.map(issue => {
         const statusClass = 'status-' + (issue.status || 'open').toLowerCase().replace('_', '-');
-        const priorityClass = 'priority-' + (issue.priority || 2);
+        const priorityClass = 'priority-' + (issue.priority ?? 2);
         let html = '<div class="issue-card" onclick="showIssueDetail(\'' + issue.id + '\')">';
         html += '<div class="issue-card-header">';
         html += '<span class="issue-card-id">' + issue.id + '</span>';
-        html += '<span class="' + priorityClass + '">P' + (issue.priority || 2) + '</span>';
+        html += '<span class="' + priorityClass + '">P' + (issue.priority ?? 2) + '</span>';
         html += '</div>';
         html += '<h3 class="issue-card-title">' + issue.title + '</h3>';
         html += '<div class="issue-card-meta">';