diff --git a/frontend/src/App.tsx b/frontend/src/App.tsx
index b277095..af5ac4b 100644
--- a/frontend/src/App.tsx
+++ b/frontend/src/App.tsx
@@ -53,13 +53,20 @@ export default function App() {
         <Route path="/techniques/:mitreId" element={<Suspense fallback={<LoadingSpinner text="Loading…" />}><TechniqueDetailPage /></Suspense>} />
 
         <Route path="/matrix" element={<Suspense fallback={<LoadingSpinner text="Loading…" />}><MatrixPage /></Suspense>} />
-        <Route path="/techniques/review-queue" element={<Suspense fallback={<LoadingSpinner text="Loading…" />}><ReviewQueuePage /></Suspense>} />
+        <Route
+          path="/techniques/review-queue"
+          element={
+            <ProtectedRoute roles={["admin", "red_lead", "blue_lead"]}>
+              <Suspense fallback={<LoadingSpinner text="Loading…" />}><ReviewQueuePage /></Suspense>
+            </ProtectedRoute>
+          }
+        />
 
-        {/* ── Executive Dashboard (leads + admin) ──────────────── */}
+        {/* ── Executive Dashboard (leads + admin + viewer) ──────── */}
         <Route
           path="/executive-dashboard"
           element={
-            <ProtectedRoute roles={["admin", "red_lead", "blue_lead"]}>
+            <ProtectedRoute roles={["admin", "red_lead", "blue_lead", "viewer"]}>
               <Suspense fallback={<LoadingSpinner text="Loading…" />}><ExecutiveDashboardPage /></Suspense>
             </ProtectedRoute>
           }
@@ -84,11 +91,11 @@ export default function App() {
         {/* ── Compliance ───────────────────────────────────────── */}
         <Route path="/compliance" element={<Suspense fallback={<LoadingSpinner text="Loading…" />}><CompliancePage /></Suspense>} />
 
-        {/* ── Comparison (leads + admin) ───────────────────────── */}
+        {/* ── Comparison (leads + admin + viewer) ──────────────── */}
         <Route
           path="/comparison"
           element={
-            <ProtectedRoute roles={["admin", "red_lead", "blue_lead"]}>
+            <ProtectedRoute roles={["admin", "red_lead", "blue_lead", "viewer"]}>
               <Suspense fallback={<LoadingSpinner text="Loading…" />}><ComparisonPage /></Suspense>
             </ProtectedRoute>
           }
diff --git a/frontend/src/pages/TechniqueDetailPage.tsx b/frontend/src/pages/TechniqueDetailPage.tsx
index e72e2ee..c575fb3 100644
--- a/frontend/src/pages/TechniqueDetailPage.tsx
+++ b/frontend/src/pages/TechniqueDetailPage.tsx
@@ -273,8 +273,8 @@ export default function TechniqueDetailPage() {
         </div>
       </div>
 
-      {/* Review required banner */}
-      {technique.review_required && (
+      {/* Review required banner — only shown to users who can act on it */}
+      {technique.review_required && canReview && (
         <div className="flex items-start gap-3 rounded-xl border border-amber-500/30 bg-amber-500/5 p-4">
           <AlertTriangle className="mt-0.5 h-5 w-5 shrink-0 text-amber-400" />
           <div className="flex-1 min-w-0">
@@ -286,15 +286,10 @@ export default function TechniqueDetailPage() {
               {technique.mitre_last_modified && (
                 <> Last modified in ATT&CK: <span className="font-mono">{technique.mitre_last_modified.slice(0, 10)}</span>.</>
               )}
-              {" "}A lead or admin should review the changes and click{" "}
-              <span className="font-semibold">Mark as Reviewed</span> to acknowledge them.
+              {" "}Click{" "}
+              <span className="font-semibold">Mark as Reviewed</span> to acknowledge the changes.
             </p>
           </div>
-          {!canReview && (
-            <span className="shrink-0 rounded-full border border-amber-500/20 bg-amber-500/10 px-2 py-0.5 text-[10px] text-amber-400">
-              Leads only
-            </span>
-          )}
         </div>
       )}