From fe33960c0e12bc5822516b359c8313668379b6a3 Mon Sep 17 00:00:00 2001
From: Kawa <us.fame.sq@gmail.com>
Date: Thu, 11 Jun 2026 11:27:46 +0200
Subject: [PATCH] Let standard techs use profiles without sign-in; flag
 unshared ones
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Standard technicians (TechN0/TechN1) are no longer auto-prompted for a
delegated SharePoint sign-in when selecting a profile — only admins are.
Techs operate under the profile's app (certificate) identity, so a profile
selection never forces them to authenticate.

To keep that usable, the admin profile list now shows a "No shared access"
badge on any profile that isn't certificate-configured, since standard
techs can't operate against those until an admin registers a cert.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 Components/Layout/MainLayout.razor | 18 ++++++++++++++----
 Components/Pages/Profiles.razor    |  7 +++++++
 2 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/Components/Layout/MainLayout.razor b/Components/Layout/MainLayout.razor
index b51fd15..94966aa 100644
--- a/Components/Layout/MainLayout.razor
+++ b/Components/Layout/MainLayout.razor
@@ -226,8 +226,8 @@
         }
 
         // If profile selected but no credentials → show modal (cert profiles never prompt)
-        if (Session.HasProfile && !_hasCredentials && !CurrentProfileUsesCert && _credModal is not null)
-            await _credModal.ShowAsync();
+        if (ShouldPromptForCredentials)
+            await _credModal!.ShowAsync();
     }
 
     // True when the selected profile authenticates app-only via a stored certificate —
@@ -235,6 +235,15 @@
     private bool CurrentProfileUsesCert =>
         Session.CurrentProfile is { } p && AppOnly.IsConfigured(p);
 
+    // Whether to auto-show the delegated sign-in modal. Only admins are ever asked to
+    // authenticate: standard technicians (TechN0/TechN1) operate under the profile's app
+    // (certificate) identity and must never be prompted when selecting a profile. A profile
+    // that isn't cert-configured is an admin setup concern, not a sign-in for the technician.
+    private bool ShouldPromptForCredentials =>
+        Session.HasProfile && !_hasCredentials && !CurrentProfileUsesCert
+        && UserContext.Role == UserRole.Admin
+        && _credModal is not null;
+
     private async Task HandleOAuthCallbackAsync()
     {
         var uri   = new Uri(Nav.Uri);
@@ -320,8 +329,9 @@
             // operating on the old connection.
             await RefreshCredentialState();
             // New profile selected and no valid credentials for it → prompt to connect.
-            if (Session.HasProfile && !_hasCredentials && _credModal is not null)
-                await _credModal.ShowAsync();
+            // Standard technicians are never prompted (see ShouldPromptForCredentials).
+            if (ShouldPromptForCredentials)
+                await _credModal!.ShowAsync();
         });
     }
 
diff --git a/Components/Pages/Profiles.razor b/Components/Pages/Profiles.razor
index cf665fb..45af55b 100644
--- a/Components/Pages/Profiles.razor
+++ b/Components/Pages/Profiles.razor
@@ -79,6 +79,13 @@
             {
                 <span class="chip chip-green">@T["profiles.active"]</span>
             }
+            @if (!AppOnlyFactory.IsConfigured(p))
+            {
+                <span class="chip chip-yellow"
+                      title="No certificate configured — standard technicians can't use this profile. Open it and run 'Register app' (or upload a certificate) to enable shared access.">
+                    ⚠ No shared access
+                </span>
+            }
             <button class="btn btn-secondary btn-sm" @onclick="() => SelectProfile(p)">
                 @(Session.CurrentProfile?.Id == p.Id ? T["profiles.selected"] : T["profiles.select"])
             </button>