From c8c0de3b0429d7f1f5e1a66a87271e72e73ad3ba Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Sun, 5 Apr 2026 21:41:26 -0400
Subject: [PATCH] feat(settings): SSO account guard + remove redundant Office
 Days

- Account tab: SSO users see an info banner instead of email/password forms
- Briefing tab: remove Office Days section (work days now come from Profile)
- Remove unused toggleWorkDay function

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/views/SettingsView.vue | 195 +++++++++++++---------------
 1 file changed, 92 insertions(+), 103 deletions(-)

diff --git a/frontend/src/views/SettingsView.vue b/frontend/src/views/SettingsView.vue
index 61f9f98..75e4b03 100644
--- a/frontend/src/views/SettingsView.vue
+++ b/frontend/src/views/SettingsView.vue
@@ -347,13 +347,6 @@ async function geocodeLocation(key: 'home' | 'work') {
   }
 }
 
-function toggleWorkDay(day: number) {
-  const days = briefingConfig.value.work_days;
-  const idx = days.indexOf(day);
-  if (idx === -1) days.push(day);
-  else days.splice(idx, 1);
-  days.sort();
-}
 
 
 async function saveBriefingSettings() {
@@ -1578,93 +1571,104 @@ function formatUserDate(iso: string): string {
     <!-- ── Account ── -->
     <div v-show="activeTab === 'account'" class="settings-grid">
 
-      <section class="settings-section">
-        <h2>Email Address</h2>
-        <p class="section-desc">Used for password resets and notifications.</p>
-        <div class="field">
-          <label for="new-email">Email</label>
-          <input
-            id="new-email"
-            v-model="newEmail"
-            type="email"
-            placeholder="you@example.com"
-            class="input"
-          />
-        </div>
-        <div v-if="authStore.user?.has_password" class="field">
-          <label for="email-password">Current Password</label>
-          <input
-            id="email-password"
-            v-model="emailPassword"
-            type="password"
-            autocomplete="current-password"
-            class="input"
-          />
-          <p class="field-hint">Required to confirm the change.</p>
-        </div>
-        <div class="actions">
-          <button
-            class="btn-save"
-            @click="changeEmail"
-            :disabled="changingEmail || (authStore.user?.has_password && !emailPassword)"
-          >
-            {{ changingEmail ? "Saving..." : "Save Email" }}
-          </button>
-        </div>
+      <!-- SSO accounts: no local credential management -->
+      <section v-if="!authStore.user?.has_password" class="settings-section">
+        <h2>Account</h2>
+        <p class="section-desc">
+          Your account is managed by an external identity provider.
+          Email and password changes are made through your provider, not here.
+        </p>
       </section>
 
-      <section class="settings-section">
-        <h2>Change Password</h2>
-        <div class="field">
-          <label for="current-password">Current Password</label>
-          <input
-            id="current-password"
-            v-model="currentPassword"
-            type="password"
-            autocomplete="current-password"
-            class="input"
-          />
-        </div>
-        <div class="field">
-          <label for="new-password">New Password</label>
-          <input
-            id="new-password"
-            v-model="newPassword"
-            type="password"
-            autocomplete="new-password"
-            class="input"
-          />
-          <p class="field-hint">Must be at least 8 characters</p>
-        </div>
-        <div class="field">
-          <label for="confirm-new-password">Confirm New Password</label>
-          <input
-            id="confirm-new-password"
-            v-model="confirmNewPassword"
-            type="password"
-            autocomplete="new-password"
-            class="input"
-            :class="{ 'input-error': confirmNewPassword && newPassword !== confirmNewPassword }"
-          />
-          <p v-if="confirmNewPassword && newPassword !== confirmNewPassword" class="error-hint">
-            Passwords do not match
-          </p>
-        </div>
-        <div class="actions">
-          <button
-            class="btn-save"
-            @click="changePassword"
-            :disabled="changingPassword || !currentPassword || newPassword.length < 8 || newPassword !== confirmNewPassword"
-          >
-            {{ changingPassword ? "Changing..." : "Change Password" }}
-          </button>
-        </div>
-      </section>
+      <template v-if="authStore.user?.has_password">
+        <section class="settings-section">
+          <h2>Email Address</h2>
+          <p class="section-desc">Used for password resets and notifications.</p>
+          <div class="field">
+            <label for="new-email">Email</label>
+            <input
+              id="new-email"
+              v-model="newEmail"
+              type="email"
+              placeholder="you@example.com"
+              class="input"
+            />
+          </div>
+          <div class="field">
+            <label for="email-password">Current Password</label>
+            <input
+              id="email-password"
+              v-model="emailPassword"
+              type="password"
+              autocomplete="current-password"
+              class="input"
+            />
+            <p class="field-hint">Required to confirm the change.</p>
+          </div>
+          <div class="actions">
+            <button
+              class="btn-save"
+              @click="changeEmail"
+              :disabled="changingEmail || !emailPassword"
+            >
+              {{ changingEmail ? "Saving..." : "Save Email" }}
+            </button>
+          </div>
+        </section>
+
+        <section class="settings-section">
+          <h2>Change Password</h2>
+          <div class="field">
+            <label for="current-password">Current Password</label>
+            <input
+              id="current-password"
+              v-model="currentPassword"
+              type="password"
+              autocomplete="current-password"
+              class="input"
+            />
+          </div>
+          <div class="field">
+            <label for="new-password">New Password</label>
+            <input
+              id="new-password"
+              v-model="newPassword"
+              type="password"
+              autocomplete="new-password"
+              class="input"
+            />
+            <p class="field-hint">Must be at least 8 characters</p>
+          </div>
+          <div class="field">
+            <label for="confirm-new-password">Confirm New Password</label>
+            <input
+              id="confirm-new-password"
+              v-model="confirmNewPassword"
+              type="password"
+              autocomplete="new-password"
+              class="input"
+              :class="{ 'input-error': confirmNewPassword && newPassword !== confirmNewPassword }"
+            />
+            <p v-if="confirmNewPassword && newPassword !== confirmNewPassword" class="error-hint">
+              Passwords do not match
+            </p>
+          </div>
+          <div class="actions">
+            <button
+              class="btn-save"
+              @click="changePassword"
+              :disabled="changingPassword || !currentPassword || newPassword.length < 8 || newPassword !== confirmNewPassword"
+            >
+              {{ changingPassword ? "Changing..." : "Change Password" }}
+            </button>
+          </div>
+        </section>
+      </template>
 
       <section class="settings-section">
         <h2>Active Sessions</h2>
         <p class="section-desc">
-          Sign out all other devices and sessions. Use this after an SSO password change
+          Sign out all other devices and sessions. Use this after a password change
           to ensure stale sessions are revoked.
         </p>
         <div class="actions">
@@ -2112,21 +2116,6 @@ function formatUserDate(iso: string): string {
         </div>
       </section>
 
-      <!-- Work schedule -->
-      <section class="settings-section full-width">
-        <h2>Office Days</h2>
-        <p class="section-desc">Which days do you go into the office? Used to decide whether to include the 8am check-in slot.</p>
-        <div class="briefing-day-toggles">
-          <button
-            v-for="(day, idx) in ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat']"
-            :key="idx"
-            :class="['briefing-day-btn', { active: briefingConfig.work_days.includes(idx) }]"
-            @click="toggleWorkDay(idx)"
-            type="button"
-          >{{ day }}</button>
-        </div>
-      </section>
-
       <!-- Slots -->
       <section class="settings-section full-width">
         <h2>Scheduled Slots</h2>