bede

Build [v1.14.32]
Fix checkbox styling: make them properly visible and usable
2026-04-26 19:12:15 +03:00 · 2026-04-26 14:45:54 +03:00 · 2026-04-26 13:52:53 +03:00 · 2026-04-26 13:33:23 +03:00 · 2026-04-26 13:30:58 +03:00 · 2026-04-26 13:29:17 +03:00
23 changed files with 356 additions and 105 deletions
--- a/.claude/settings.local.json
+++ b/.claude/settings.local.json
@@ -149,7 +149,20 @@
      "Bash(curl -k -s -X POST https://192.168.84.131:8918/users/login -H 'Content-Type: application/json' -d '{\\\\\"username\\\\\":\\\\\"admin\\\\\",\\\\\"password\\\\\":\\\\\"admin\\\\\"}')",
      "Bash(curl -k -s -H \"Authorization: Bearer $\\(curl -k -s -X POST https://192.168.84.131:8918/users/login -H 'Content-Type: application/json' -d '{\\\\\"username\\\\\":\\\\\"admin\\\\\",\\\\\"password\\\\\":\\\\\"admin\\\\\"}')",
      "Bash(curl -k -s https://192.168.84.131:8918/admin/db/export)",
-      "Bash(node scripts/generate-css-vars.mjs)"
+      "Bash(node scripts/generate-css-vars.mjs)",
+      "Bash(curl -s -X POST http://localhost:8916/users/login -H 'Content-Type: application/json' -d '{\"username\":\"admin\",\"password\":\"password\"}')",
+      "Bash(curl -s -X POST http://localhost:8916/users/login -H 'Content-Type: application/json' -d '{\"username\":\"admin\",\"password\":\"admin\"}')",
+      "Bash(pkill -9 -f \"uvicorn backend.main\")",
+      "Bash(pkill -9 -f \"python3.*start_servers\")",
+      "Bash(curl -v -X POST http://localhost:8916/users/login -H 'Content-Type: application/json' -d '{\"username\":\"admin\",\"password\":\"admin\"}')",
+      "Bash(pkill -9 -f \"uvicorn\\\\|gunicorn\")",
+      "Bash(curl -s http://localhost:3000)",
+      "Bash(/data/programare_AI/tfm_ainventory/venv/bin/python3 -m uvicorn backend.main:app --host 0.0.0.0 --port 8916)",
+      "Bash(curl -s http://localhost:8916/users/)",
+      "Bash(pkill -9 -f \"uvicorn.*backend.main\" sleep 2 /data/programare_AI/tfm_ainventory/venv/bin/python3 -m uvicorn backend.main:app --host 0.0.0.0 --port 8916)",
+      "Bash(curl -s http://localhost:8916/users/auth-mode)",
+      "Bash(pkill -9 -f \"next-server|node.*next\" sleep 2 python3 scripts/run_standalone.py restart)",
+      "WebSearch"
    ]
  }
 }
--- a/.standalone.pid
+++ b/.standalone.pid
@@ -1,3 +1,3 @@
-65062
-65063
-65079
+127238
+127239
+127256
--- a/AI_MANDATES.md
+++ b/AI_MANDATES.md
@@ -25,6 +25,10 @@ This is the foundational directive for all AI operations on the TFM aInventory p
 - **Standard Respect**: Rigorously follow `DESIGN_SYSTEM.md` and `CODING_STANDARDS.md`.
 - **Conflict Resolution**: If project files contradict your internal training, **this file and local project docs take absolute precedence.**
 - **Uppercase Rule**: NEVER use `uppercase` or `toUpper` in UI contexts. Leave text case as is or use Title/Sentence case.
+- **⚠️ MANDATORY: No Silent Feature Removal** — **NEVER remove, disable, or hide any existing function or functionality without:**
+  1. **Explicitly informing the user** exactly what is being removed and why.
+  2. **Obtaining express written consent** from the user before proceeding.
+  3. **If already removed**, restore it immediately and apologize. This is non-negotiable.

 ## 4. Documentation SSOT Integrity
 Every feature change or refactor MUST be reflected across the documentation suite:
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -25,5 +25,13 @@ Be concise! Focus on signal over noise.
 All color changes MUST be made in `frontend/colors.mjs` ONLY.
 Run `npm run build` to sync CSS variables and Tailwind config.

+## ⚠️ CRITICAL: No Silent Feature Removal
+**NEVER remove, disable, or hide any existing function or functionality without:**
+1. **Explicitly informing the user** exactly what is being removed and why
+2. **Obtaining express written consent** before proceeding
+3. **If already removed**, restore immediately and apologize
+
+This applies to ALL changes: UI buttons, login modes, API endpoints, features, functionality.
+
 ---
 **Status**: ACTIVE
--- a/backend/routers/auth.py
+++ b/backend/routers/auth.py
@@ -67,7 +67,9 @@ def authenticate_ldap(username, password):
        user_dn = config["user_template"].format(username=safe_username_rdn)
        log.debug(f"LDAP: Attempting bind for DN: {user_dn}")

-        conn = ldap3.Connection(server, user=user_dn, password=password, auto_bind=True)
+        # [FIX] Ensure password is UTF-8 encoded for LDAP protocol
+        password_bytes = password.encode('utf-8') if isinstance(password, str) else password
+        conn = ldap3.Connection(server, user=user_dn, password=password_bytes, auto_bind=True)
        log.debug(f"LDAP: Bind successful for {user_dn}")

        # Search for the user to get their CANONICAL DN
@@ -250,10 +252,21 @@ def login(request: Request, form_data: schemas.UserLogin, db: Session = Depends(
        token_type="bearer",
        user_id=user.id,
        username=user.username,
-        role=user.role
+        role=user.role,
+        user=schemas.User(id=user.id, username=user.username, role=user.role, origin=user.origin)
    )


+@router.get("/auth-mode")
+def get_auth_mode():
+    """[C-01] Get authentication mode (public endpoint for login page)."""
+    config = get_ldap_config()
+    return {
+        "mode": "enterprise" if config.get("ldap_enabled") else "local",
+        "ldap_enabled": config.get("ldap_enabled", False)
+    }
+
+
@router.get("/ldap-config")
 def get_ldap_settings(current_user: auth.TokenData = Depends(auth.get_current_admin)):
    """[C-01] Get LDAP config — admin only."""
--- a/backend/schemas/users.py
+++ b/backend/schemas/users.py
@@ -42,3 +42,8 @@ class TokenResponse(BaseModel):
    user_id: int
    username: str
    role: str
+    user: User  # User object for frontend compatibility
+
+
+# Rebuild TokenResponse to resolve forward references
+TokenResponse.model_rebuild()
--- a/dev_docs/SESSION_STATE.md
+++ b/dev_docs/SESSION_STATE.md
@@ -1,36 +1,61 @@
 # CURRENT AI WORKING SESSION — HANDOVER

-**Active AI:** Gemini CLI (Antigravity)
+**Active AI:** Claude (Haiku 4.5)
 **Last Updated:** 2026-04-26
 **Current Version:** v1.15.0
-**Status**: ✅ DOCUMENTATION CLEANUP & RESTRUCTURING COMPLETE | 💾 STAGED IN docs-cleanup BRANCH
+**Status**: ✅ LDAP LOGIN FULLY FUNCTIONAL | ✅ ENTERPRISE MODE DETECTION RESTORED

 ---

-## SESSION SUMMARY — Documentation Overhaul
+## SESSION SUMMARY — Complete LDAP Authentication Fix

-### 1. Architectural Restructuring
- **Consolidated Design System**: Created `DESIGN_SYSTEM.md` as the Single Source of Truth for all UI/UX (Colors, Typography, Spacing, Shapes).
- **Consolidated Coding Standards**: Created `CODING_STANDARDS.md` merging backend/frontend rules, security, and testing targets.
- **Refined AI Mandates**: Renamed `AI_RULES.md` to `AI_MANDATES.md`, focusing on behavioral rules and handover protocols.
- **Improved Technical Reference**: Renamed `PROJECT_ARCHITECTURE.md` to `ARCHITECTURE.md`, focusing on high-level system logic.
- **Fixed Roadmap**: Resolved merge conflicts in `dev_docs/PLAN.md` and moved to `ROADMAP.md` in root.
+### Phase 1: Response Structure Bug (Commits 51716faf + 62c592d6)
+**Problem**: Frontend got "Invalid Protocol Password" error on ALL login attempts
+**Root Cause**: Backend response was missing `user` object that frontend expected for localStorage

-### 2. Information Cleanup & Archiving
- **Archived Legacy Files**: Moved 10+ redundant or outdated audit/review/plan files to `dev_docs/archive/`.
- **Eliminated Duplication**: Deleted `AI_RULES.md`, `PROJECT_ARCHITECTURE.md`, `DESIGN_COLOR_RULES.md`, and redundant plans.
- **Cleaned Entry Points**: Updated `README.md`, `CLAUDE.md`, and `GEMINI.md` to point to the new, functional documentation structure.
+**Fixed**:
+- ✅ Added `user` field to `TokenResponse` schema (non-optional)
+- ✅ Updated login endpoint to populate complete User object
+- ✅ Password encoding for LDAP protocol compatibility

-### 3. Verification & Compliance
- **Design Alignment**: Ensured "NO BOLD" and "NO UPPERCASE" mandates are documented as absolute law.
- **SSOT Integrity**: Established clear hierarchies for documentation (End-User vs Internal).
- **Git Status**: All changes staged in new branch `docs-cleanup`.
+### Phase 2: Missing Enterprise Login UI (Commit dc999991)
+**Problem**: LDAP/Enterprise login UI was hidden; only local user buttons shown
+**Root Cause**: `isEnterprise` hardcoded to `false` due to unimplemented `getNetworkConfig()`

---
+**Fixed**:
+- ✅ Added public `/users/auth-mode` endpoint (detects LDAP status)
+- ✅ Added `getAuthMode()` frontend API method
+- ✅ Login page now auto-detects and switches UI based on LDAP configuration

-## NEXT STEPS
+### ✅ Current State
+- **Local Mode** (LDAP disabled): Shows local user buttons
+- **Enterprise Mode** (LDAP enabled): Shows "Enterprise ID" input for LDAP username
+- **Password Field**: Always present for both modes
+- **Response**: Complete with `user` object for localStorage

-1. **Review Documentation**: Verify the new structure in the `docs-cleanup` branch.
-2. **Commit Changes**: If satisfied, commit the staged changes to the branch.
-3. **Merge to dev**: Merge `docs-cleanup` into the main `dev` branch.
-4. **Version Bump**: Run `python3 scripts/save_version.py` to finalize the v1.15.0 documentation milestone.
+### Test Configuration
+- LDAP Enabled: ✅ Yes (`config/backend.yaml`: `ldap_enabled: true`)
+- LDAP Server: ldaps://192.168.84.107:6360
+- Base DN: dc=ldap,dc=lan
+
+### Backend Endpoints
+- `/users/login` — POST to authenticate (returns TokenResponse with user object)
+- `/users/auth-mode` — GET public endpoint to detect mode (returns `{mode, ldap_enabled}`)
+- `/users/ldap-config` — GET admin-only (full LDAP configuration)
+
+### ✅ All Fixes Verified (2026-04-26, 14:00)
+1. ✅ Checkbox styling: 20px, 2px border, proper CSS variables
+2. ✅ CSS compilation: No PostCSS errors, page loads correctly
+3. ✅ Frontend server: Next.js 15.5.15 running on port 8917
+4. ✅ Backend server: FastAPI/uvicorn running on port 8916
+5. ✅ Auth-mode detection: LDAP enabled, both login modes available
+6. ✅ TokenResponse schema: Includes user object
+7. ✅ Standalone script: Uses venv python correctly
+
+### Remaining Task
+- Run version bump: `python3 scripts/save_version.py` to finalize all session changes
+
+## Git Status
+- Branch: dev
+- Commits: 51716faf + 62c592d6 + dc999991 + CSS fix
+- All LDAP login, UI, checkbox, and script issues resolved
--- a/frontend/VERSION.json
+++ b/frontend/VERSION.json
@@ -1,6 +1,6 @@
 {
-  "version": "1.14.31",
-  "last_build": "2026-04-25-1253",
+  "version": "1.14.32",
+  "last_build": "2026-04-26-1445",
  "codename": "ConfigCore",
-  "commit": "d0f166a3"
+  "commit": "ba54820d"
 }
--- a/frontend/app/globals.css
+++ b/frontend/app/globals.css
@@ -5,15 +5,15 @@
@tailwind utilities;

@layer base {
-  :root {
-    /* Base colors — AUTO-GENERATED from colors.mjs */
-    /* PALETTE: Electric Blue (2026-04-26) */
+                                            :root {
+    /* Base colors from DESIGN.md - AUTO-GENERATED from colors.mjs */

+    /* Base colors from DESIGN.md */
    --background: #131313;
    --on-background: #dde4f0;
    --foreground: #dde4f0;

-    /* Surface — cool blue-tinted darks */
+    /* Surface colors from DESIGN.md */
    --surface-DEFAULT: #131313;
    --surface-dim: #131313;
    --surface-bright: #1e2535;
@@ -24,7 +24,7 @@
    --surface-container-highest: #252d42;
    --surface-variant: #252d42;

-    /* Primary — Electric Blue */
+    /* Primary colors from DESIGN.md */
    --primary-DEFAULT: #58a6ff;
    --primary-foreground: #001a4d;
    --primary-container: #1d6fd4;
@@ -35,7 +35,7 @@
    --primary-on-fixed-variant: #003380;
    --primary-inverse: #1d5fa8;

-    /* Secondary — Cool blue-grey */
+    /* Secondary colors from DESIGN.md */
    --secondary-DEFAULT: #9aa5be;
    --secondary-foreground: #1a2030;
    --secondary-container: #3d4d6b;
@@ -45,7 +45,7 @@
    --secondary-on-fixed: #0d1220;
    --secondary-on-fixed-variant: #3d4d6b;

-    /* Tertiary — Cyan-teal */
+    /* Tertiary colors from DESIGN.md */
    --tertiary-DEFAULT: #00d4aa;
    --tertiary-foreground: #003328;
    --tertiary-container: #00a882;
@@ -55,21 +55,21 @@
    --tertiary-on-fixed: #002820;
    --tertiary-on-fixed-variant: #005544;

-    /* Error — pure red (no rose/pink cast) */
+    /* Error colors from DESIGN.md */
    --error-DEFAULT: #ff5f52;
    --error-foreground: #1a0000;
    --error-container: #7a0000;
    --error-on-container: #ffdad6;

-    /* Outline — blue-tinted */
+    /* Outline colors from DESIGN.md */
    --outline: #3d5a8a;
    --outline-variant: #1e3055;

-    /* Inverse */
+    /* Inverse colors from DESIGN.md */
    --inverse-surface: #dde4f0;
    --inverse-on-surface: #1a2030;

-    /* Semantic */
+    /* Semantic colors from DESIGN.md */
    --border: #252d42;
    --muted: #5a6a8a;
    --info: #58a6ff;
@@ -77,7 +77,7 @@
    --warning: #f0a040;
    --alert: #ff5f52;

-    /* Spacing tokens */
+    /* Semantic spacing tokens */
    --spacing-unit: 4px;
    --spacing-stack-sm: 8px;
    --spacing-stack-md: 16px;
@@ -86,6 +86,27 @@
    --spacing-container-gap: 24px;
  }

+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
  * {
    @apply border-border;
    border-radius: 0 !important;
@@ -107,6 +128,26 @@
    font-style: normal !important;
  }

+  /* Checkbox styling - proper, usable checkboxes */
+  input[type="checkbox"] {
+    width: 20px;
+    height: 20px;
+    cursor: pointer;
+    accent-color: var(--primary-DEFAULT);
+    border: 2px solid var(--outline);
+    border-radius: 3px;
+    appearance: auto;
+  }
+
+  input[type="checkbox"]:hover {
+    border-color: var(--primary-DEFAULT);
+  }
+
+  input[type="checkbox"]:focus {
+    outline: 2px solid var(--primary-DEFAULT);
+    outline-offset: 2px;
+  }
+
  h1 {
    @apply text-5xl tracking-tighter;
    letter-spacing: -0.02em;
@@ -142,7 +183,7 @@
    color: var(--primary-foreground);
    padding: 0.5rem 1rem;
    @apply inline-flex items-center justify-center gap-2 transition-opacity hover:opacity-90;
-    border-radius: 0 !important;
+    border-radius: 5px !important;
  }

  .btn-secondary {
@@ -151,7 +192,7 @@
    color: var(--secondary-DEFAULT);
    padding: 0.5rem 1rem;
    @apply inline-flex items-center justify-center gap-2 transition-colors;
-    border-radius: 0 !important;
+    border-radius: 5px !important;
  }

  .btn-secondary:hover {
@@ -227,6 +268,71 @@
    border-radius: 0 !important;
  }

+  /* ── Pill Toggle — replaces all input[type="checkbox"] ── */
+  .toggle-pill {
+    -webkit-appearance: none !important;
+    appearance: none !important;
+    width: 42px !important;
+    height: 24px !important;
+    border-radius: 999px !important;
+    background: var(--surface-bright) !important;
+    border: 1.5px solid var(--outline) !important;
+    position: relative !important;
+    cursor: pointer !important;
+    flex-shrink: 0 !important;
+    transition: background 0.22s ease, border-color 0.22s ease !important;
+    outline: none !important;
+    accent-color: transparent !important;
+  }
+
+  .toggle-pill::after {
+    content: '' !important;
+    position: absolute !important;
+    top: 3px !important;
+    left: 3px !important;
+    width: 16px !important;
+    height: 16px !important;
+    border-radius: 50% !important;
+    background: var(--secondary) !important;
+    transition: transform 0.22s cubic-bezier(0.34, 1.56, 0.64, 1), background 0.22s !important;
+    box-shadow: 0 1px 3px rgba(0,0,0,0.4) !important;
+    display: block !important;
+  }
+
+  .toggle-pill:checked {
+    background: var(--primary-DEFAULT) !important;
+    border-color: var(--primary-DEFAULT) !important;
+  }
+
+  .toggle-pill:checked::after {
+    transform: translateX(18px) !important;
+    background: var(--primary-foreground) !important;
+  }
+
+  .toggle-pill:focus-visible {
+    outline: 2px solid var(--primary-DEFAULT) !important;
+    outline-offset: 2px !important;
+  }
+
+  .toggle-pill:disabled {
+    opacity: 0.4 !important;
+    cursor: not-allowed !important;
+  }
+
+  .toggle-pill.sm {
+    width: 34px !important;
+    height: 19px !important;
+  }
+
+  .toggle-pill.sm::after {
+    width: 13px !important;
+    height: 13px !important;
+  }
+
+  .toggle-pill.sm:checked::after {
+    transform: translateX(15px) !important;
+  }
+
  .headline-lg {
    @apply text-5xl font-normal;
    letter-spacing: -0.02em;
--- a/frontend/app/items/create.tsx
+++ b/frontend/app/items/create.tsx
@@ -357,7 +357,7 @@ export default function CreateItemPage() {
                    setCropBounds(null);
                  }
                }}
-                className="w-4 h-4 rounded border-outline accent-primary"
+                className="toggle-pill"
              />
              <label htmlFor="use-full-photo" className="text-sm font-normal text-[#CCCCCC]">
                Use full photo (skip cropping)
--- a/frontend/app/login/page.tsx
+++ b/frontend/app/login/page.tsx
@@ -9,6 +9,7 @@ import { useRouter } from 'next/navigation';
 export default function LoginPage() {
  const [mounted, setMounted] = useState(false);
  const [users, setUsers] = useState<any[]>([]);
+  const [ldapAvailable, setLdapAvailable] = useState(false);
  const [isEnterprise, setIsEnterprise] = useState(false);
  const [isLoading, setIsLoading] = useState(true);
  const [selectedUserForLogin, setSelectedUserForLogin] = useState<any | null>(null);
@@ -23,15 +24,17 @@ export default function LoginPage() {
  const checkMode = async () => {
    try {
      setIsLoading(true);
-      // TODO: getNetworkConfig() method not implemented in API
-      // const config = await inventoryApi.getNetworkConfig();
-      // setIsEnterprise(config.mode === 'enterprise');
-      setIsEnterprise(false); // Default to non-enterprise mode
+      // Get authentication mode from backend
+      const authMode = await inventoryApi.getAuthMode();
+      setLdapAvailable(authMode.ldap_enabled);
+      setIsEnterprise(false); // Always start with local mode for user choice

      const userList = await inventoryApi.getUsers();
      setUsers(userList);
    } catch (error) {
-      console.error("Failed to load users", error);
+      console.error("Failed to load auth mode or users", error);
+      setLdapAvailable(false);
+      setIsEnterprise(false);
    } finally {
      setIsLoading(false);
    }
@@ -81,6 +84,32 @@ export default function LoginPage() {
          </div>
        ) : (
          <div className="grid gap-3">
+            {ldapAvailable && (
+              <div className="flex gap-2">
+                <button
+                  type="button"
+                  onClick={() => { setIsEnterprise(false); setSelectedUserForLogin(null); }}
+                  className={`flex-1 py-2 px-3 text-sm font-normal rounded border transition-all ${
+                    !isEnterprise
+                      ? 'bg-primary border-primary text-white'
+                      : 'bg-surface-container border-border text-secondary hover:border-primary/50'
+                  }`}
+                >
+                  Local Users
+                </button>
+                <button
+                  type="button"
+                  onClick={() => { setIsEnterprise(true); setSelectedUserForLogin(null); }}
+                  className={`flex-1 py-2 px-3 text-sm font-normal rounded border transition-all ${
+                    isEnterprise
+                      ? 'bg-primary border-primary text-white'
+                      : 'bg-surface-container border-border text-secondary hover:border-primary/50'
+                  }`}
+                >
+                  Enterprise
+                </button>
+              </div>
+            )}
            {!selectedUserForLogin && !isEnterprise ? (
              <>
                {users.length > 0 ? (
--- a/frontend/app/logs/page.tsx
+++ b/frontend/app/logs/page.tsx
@@ -159,9 +159,9 @@ export default function LogsPage() {
                  : "bg-surface/70 text-secondary border-border hover:border-border"
              )}
            >
-              All Streams
+              &nbsp; All Streams &nbsp;
            </button>
-            {['ADD', 'REMOVE', 'ADJUST', 'DELETE', 'LOGIN'].map((f) => (
+            {['Add', 'Remove', 'Adjust', 'Delete', 'Login'].map((f) => (
              <button
                key={f}
                onClick={() => setFilterAction(f)}
@@ -172,7 +172,7 @@ export default function LogsPage() {
                    : "bg-surface/70 text-secondary border-border hover:border-border"
                )}
              >
-                {f}
+                &nbsp; {f} &nbsp;
              </button>
            ))}
          </div>
--- a/frontend/app/page.tsx
+++ b/frontend/app/page.tsx
@@ -271,7 +271,7 @@ export default function Home() {
            </div>
            <div>
              <h1 className="text-2xl md:text-3xl font-normal tracking-tight text-white leading-tight">TFM aInventory</h1>
-              <p className="text-xs md:text-sm text-secondary font-normal tracking-tight mt-1">Industrial Asset Management</p>
+              <p className="text-xs md:text-sm text-secondary font-normal tracking-tight mt-1">IT/DC Asset Management</p>
            </div>
          </div>

@@ -397,7 +397,7 @@ export default function Home() {
      )}

        <footer className="mt-20 mb-8 flex flex-col items-center gap-4 opacity-50 border-t border-border/30 pt-8">
-          <p className="text-xs font-normal text-secondary tracking-widest uppercase">Powered by TFM Group Software</p>
+          <p className="text-xs font-normal text-secondary tracking-widest uppercase">Powered by TFM Group Software S.R.L.</p>
          <p className="text-[10px] font-mono text-muted">v{versionData.version} • {versionData.last_build} • BUILD: {versionData.commit}</p>
        </footer>
      </div>
--- a/frontend/components/AIOnboarding.tsx
+++ b/frontend/components/AIOnboarding.tsx
@@ -256,7 +256,7 @@ export default function AIOnboarding({ onCancel, onComplete, categories, invento
                    id="save-photo-check"
                    checked={!extractedItems[editingIndex]._skipPhoto}
                    onChange={(e) => updateEditingItem({ _skipPhoto: !e.target.checked })}
-                    className="w-4 h-4 rounded border-outline accent-primary cursor-pointer"
+                    className="toggle-pill"
                  />
                  <label htmlFor="save-photo-check" className="text-xs text-[#CCCCCC] font-normal cursor-pointer">
                    Save this photo with the item
--- a/frontend/components/ImageAdjustmentModal.tsx
+++ b/frontend/components/ImageAdjustmentModal.tsx
@@ -300,7 +300,7 @@ export function ImageAdjustmentModal({ imageUrl, onConfirm, onCancel }: ImageAdj
              type="checkbox"
              checked={useImage}
              onChange={(e) => setUseImage(e.target.checked)}
-              className="w-4 h-4"
+              className="toggle-pill"
            />
            Use this image
          </label>
--- a/frontend/components/StatCard.tsx
+++ b/frontend/components/StatCard.tsx
@@ -13,7 +13,7 @@ export default function StatCard({ label, value, icon: Icon }: StatCardProps) {
      <div className="flex items-center gap-2.5 min-w-0">
        {Icon && <Icon className="w-5 h-5 text-primary flex-shrink-0" aria-hidden="true" />}
        <span className="text-base md:text-lg text-secondary font-normal truncate">
-          {label}
+          &nbsp; {label}
        </span>
      </div>

--- a/frontend/components/admin/AiManager.tsx
+++ b/frontend/components/admin/AiManager.tsx
@@ -1,6 +1,8 @@
 import React from 'react';
-import { Brain, Cpu, Zap, Lock, RotateCcw, Wifi, FileText, Shield } from 'lucide-react';
+import { Brain, Lock, RotateCcw, Wifi, FileText, Shield } from 'lucide-react';
 import { cn } from '@/lib/utils';
+import { GeminiLogo } from '@/components/icons/GeminiLogo';
+import { ClaudeLogo } from '@/components/icons/ClaudeLogo';

 interface AiManagerProps {
  aiConfig: any;
@@ -49,7 +51,7 @@ export default function AiManager({
            data-testid="provider-option"
            onClick={() => handleUpdateAiProvider(p.id)}
            className={cn(
-              "p-4 lg:p-5 xl:p-6 lg:p-8 xl:p-10 border transition-all text-left flex items-center justify-between group",
+              "p-4 lg:p-4 xl:p-5 lg:p-4 xl:p-5 border transition-all text-left flex items-center justify-between group",
              p.active 
                ? "bg-primary border-primary" 
                : "level-0 border-border hover:border-primary/40"
@@ -60,23 +62,23 @@ export default function AiManager({
                "w-8 h-8 flex items-center justify-center transition-colors",
                p.active ? "bg-surface-container-lowest/20 text-black" : "bg-primary/10 text-primary group-hover:bg-primary/20"
              )}>
-                {p.id === 'gemini' ? <Cpu size={16} /> : <Zap size={16} />}
+                {p.id === 'gemini' ? <GeminiLogo size={16} /> : <ClaudeLogo size={16} />}
              </div>
              <div>
-                <p className={cn("text-xs font-normal tracking-tight", p.active ? "text-black" : "text-secondary")}>{p.name}</p>
+                <p className={cn("text-sm font-normal tracking-tight", p.active ? "text-black" : "text-secondary")}>{p.name}</p>
                <p className={cn(
-                  "text-xs font-normal mt-1 px-0.5",
+                  "text-sm font-normal mt-1 px-0.5",
                  p.active 
                    ? (p.configured ? "text-black/70" : "text-black/50") 
                    : (p.configured ? "text-emerald-500" : "text-error")
                )}>
-                  {p.configured ? "● Key Configured" : "○ Missing Api Key"}
+                  {p.configured ? "✓ Key Configured" : "✗ Missing Api Key"}
                </p>
              </div>
            </div>
            {p.active && (
-              <div className="bg-surface-container-lowest/20 px-2.5 py-1 text-xs font-normal text-black tracking-tight">
-                Active
+              <div className="bg-surface-container-lowest/20 px-2.5 py-1 text-sm font-normal text-black tracking-tight">
+                &nbsp; Active &nbsp;
              </div>
            )}
          </button>
@@ -110,13 +112,13 @@ export default function AiManager({
                value={aiKeys.gemini}
                onChange={(e) => setAiKeys({...aiKeys, gemini: e.target.value})}
                placeholder={aiConfig?.providers?.find((p: any) => p.id === 'gemini')?.masked_key || "Enter Gemini Key..."}
-                className="input-field flex-1 text-xs"
+                className="input-field flex-1 text-sm"
              />
              <button
                onClick={() => onTestAiKey('gemini')}
                disabled={isTestingKeys.gemini}
                className={cn(
-                  "px-3 py-1.5 text-xs font-normal tracking-tight transition-all border flex items-center gap-1.5",
+                  "px-3 py-1.5 text-sm font-normal tracking-tight transition-all border flex items-center gap-1.5",
                  isTestingKeys.gemini
                    ? "bg-surface-bright border-border text-muted cursor-wait"
                    : "btn-primary"
@@ -136,13 +138,13 @@ export default function AiManager({
                value={aiKeys.claude}
                onChange={(e) => setAiKeys({...aiKeys, claude: e.target.value})}
                placeholder={aiConfig?.providers?.find((p: any) => p.id === 'claude')?.masked_key || "Enter Claude Key..."}
-                className="input-field flex-1 text-xs"
+                className="input-field flex-1 text-sm"
              />
              <button
                onClick={() => onTestAiKey('claude')}
                disabled={isTestingKeys.claude}
                className={cn(
-                  "px-3 py-1.5 text-xs font-normal tracking-tight transition-all border flex items-center gap-1.5",
+                  "px-3 py-1.5 text-sm font-normal tracking-tight transition-all border flex items-center gap-1.5",
                  isTestingKeys.claude
                    ? "bg-surface-bright border-border text-muted cursor-wait"
                    : "btn-primary"
@@ -172,12 +174,12 @@ export default function AiManager({
          <textarea
            value={aiPrompt}
            onChange={(e) => setAiPrompt(e.target.value)}
-            className="log-viewer w-full min-h-[200px] text-xs text-secondary leading-relaxed outline-none focus:border-primary/50 transition-all"
+            className="log-viewer w-full min-h-[200px] text-sm text-secondary leading-relaxed outline-none focus:border-primary/50 transition-all"
          />
        </div>
        <div className="bg-primary/5 border border-primary/10 p-4 flex gap-3 items-start">
          <div className="p-2 bg-primary/10 text-primary shrink-0"><Shield size={14} /></div>
-          <p className="text-xs font-normal text-secondary leading-relaxed">
+          <p className="text-sm font-normal text-secondary leading-relaxed">
            This prompt instructs the Vision AI core on label interpretation. Ensure it defines explicit mapping for technical attributes like <span className="text-primary">Item</span>, <span className="text-primary">Type</span>, and <span className="text-primary">Part Number</span> to avoid extraction null-pointers and ensure inventory data integrity.
          </p>
        </div>
--- a/frontend/components/admin/LdapManager.tsx
+++ b/frontend/components/admin/LdapManager.tsx
@@ -43,18 +43,12 @@ export default function LdapManager({
            </p>
          </div>
        </div>
-        <button 
-          onClick={() => setLdapConfig({...ldapConfig, ldap_enabled: !ldapConfig.ldap_enabled})}
-          className={cn(
-            "w-10 h-5 relative transition-all duration-300 border",
-            ldapConfig.ldap_enabled ? "bg-primary border-primary" : "bg-surface-bright border-border"
-          )}
-        >
-          <div className={cn(
-            "absolute top-0.5 w-3.5 h-3.5 bg-surface-container-lowest transition-all",
-            ldapConfig.ldap_enabled ? "right-0.5" : "left-0.5"
-          )} />
-        </button>
+        <input
+          type="checkbox"
+          checked={ldapConfig.ldap_enabled}
+          onChange={(e) => setLdapConfig({...ldapConfig, ldap_enabled: e.target.checked})}
+          className="toggle-pill"
+        />
      </div>

      <div className="grid sm:grid-cols-2 gap-3">
@@ -119,7 +113,7 @@ export default function LdapManager({
        <div className="flex items-center gap-3">
          <div className={cn(
            "w-8 h-8 flex items-center justify-center transition-all",
-            ldapConfig.use_tls ? "bg-primary/10 text-primary" : "bg-surface-bright text-muted"
+            ldapConfig.use_tls ? "status-success" : "bg-surface-bright text-muted"
          )}>
            <Lock size={14} />
          </div>
@@ -130,18 +124,12 @@ export default function LdapManager({
            </p>
          </div>
        </div>
-        <button 
-          onClick={() => setLdapConfig({...ldapConfig, use_tls: !ldapConfig.use_tls})}
-          className={cn(
-            "w-10 h-5 relative transition-all duration-300 border",
-            ldapConfig.use_tls ? "bg-primary border-primary" : "bg-surface-bright border-border"
-          )}
-        >
-          <div className={cn(
-            "absolute top-0.5 w-3.5 h-3.5 bg-surface-container-lowest transition-all",
-            ldapConfig.use_tls ? "right-0.5" : "left-0.5"
-          )} />
-        </button>
+        <input
+          type="checkbox"
+          checked={ldapConfig.use_tls}
+          onChange={(e) => setLdapConfig({...ldapConfig, use_tls: e.target.checked})}
+          className="toggle-pill"
+        />
      </div>

      <div className="flex gap-2 pt-0">
--- a/frontend/components/icons/ClaudeLogo.tsx
+++ b/frontend/components/icons/ClaudeLogo.tsx
@@ -0,0 +1,23 @@
+import React from 'react';
+
+interface ClaudeLogoProps {
+  size?: number;
+  className?: string;
+}
+
+export function ClaudeLogo({ size = 16, className = '' }: ClaudeLogoProps) {
+  return (
+    <svg
+      width={size}
+      height={size}
+      fill="currentColor"
+      fillRule="evenodd"
+      viewBox="0 0 24 24"
+      className={className}
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <title>Claude</title>
+      <path d="M4.709 15.955l4.72-2.647.08-.23-.08-.128H9.2l-.79-.048-2.698-.073-2.339-.097-2.266-.122-.571-.121L0 11.784l.055-.352.48-.321.686.06 1.52.103 2.278.158 1.652.097 2.449.255h.389l.055-.157-.134-.098-.103-.097-2.358-1.596-2.552-1.688-1.336-.972-.724-.491-.364-.462-.158-1.008.656-.722.881.06.225.061.893.686 1.908 1.476 2.491 1.833.365.304.145-.103.019-.073-.164-.274-1.355-2.446-1.446-2.49-.644-1.032-.17-.619a2.97 2.97 0 01-.104-.729L6.283.134 6.696 0l.996.134.42.364.62 1.414 1.002 2.229 1.555 3.03.456.898.243.832.091.255h.158V9.01l.128-1.706.237-2.095.23-2.695.08-.76.376-.91.747-.492.584.28.48.685-.067.444-.286 1.851-.559 2.903-.364 1.942h.212l.243-.242.985-1.306 1.652-2.064.73-.82.85-.904.547-.431h1.033l.76 1.129-.34 1.166-1.064 1.347-.881 1.142-1.264 1.7-.79 1.36.073.11.188-.02 2.856-.606 1.543-.28 1.841-.315.833.388.091.395-.328.807-1.969.486-2.309.462-3.439.813-.042.03.049.061 1.549.146.662.036h1.622l3.02.225.79.522.474.638-.079.485-1.215.62-1.64-.389-3.829-.91-1.312-.329h-.182v.11l1.093 1.068 2.006 1.81 2.509 2.33.127.578-.322.455-.34-.049-2.205-1.657-.851-.747-1.926-1.62h-.128v.17l.444.649 2.345 3.521.122 1.08-.17.353-.608.213-.668-.122-1.374-1.925-1.415-2.167-1.143-1.943-.14.08-.674 7.254-.316.37-.729.28-.607-.461-.322-.747.322-1.476.389-1.924.315-1.53.286-1.9.17-.632-.012-.042-.14.018-1.434 1.967-2.18 2.945-1.726 1.845-.414.164-.717-.37.067-.662.401-.589 2.388-3.036 1.44-1.882.93-1.086-.006-.158h-.055L4.132 18.56l-1.13.146-.487-.456.061-.746.231-.243 1.908-1.312-.006.006z" />
+    </svg>
+  );
+}
--- a/frontend/components/icons/GeminiLogo.tsx
+++ b/frontend/components/icons/GeminiLogo.tsx
@@ -0,0 +1,23 @@
+import React from 'react';
+
+interface GeminiLogoProps {
+  size?: number;
+  className?: string;
+}
+
+export function GeminiLogo({ size = 16, className = '' }: GeminiLogoProps) {
+  return (
+    <svg
+      width={size}
+      height={size}
+      fill="currentColor"
+      fillRule="evenodd"
+      viewBox="0 0 24 24"
+      className={className}
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <title>Gemini</title>
+      <path d="M20.616 10.835a14.147 14.147 0 01-4.45-3.001 14.111 14.111 0 01-3.678-6.452.503.503 0 00-.975 0 14.134 14.134 0 01-3.679 6.452 14.155 14.155 0 01-4.45 3.001c-.65.28-1.318.505-2.002.678a.502.502 0 000 .975c.684.172 1.35.397 2.002.677a14.147 14.147 0 014.45 3.001 14.112 14.112 0 013.679 6.453.502.502 0 00.975 0c.172-.685.397-1.351.677-2.003a14.145 14.145 0 013.001-4.45 14.113 14.113 0 016.453-3.678.503.503 0 000-.975 13.245 13.245 0 01-2.003-.678z" />
+    </svg>
+  );
+}
--- a/frontend/lib/api.ts
+++ b/frontend/lib/api.ts
@@ -85,6 +85,12 @@ axiosInstance.interceptors.response.use(
 );

 export const inventoryApi = {
+  getAuthMode: async () => {
+    const baseUrl = await getBackendUrl();
+    const res = await axios.get(`${baseUrl}/users/auth-mode`);
+    return res.data;
+  },
+
  getItems: async () => {
    const res = await axiosInstance.get('/items/');
    return res.data;
--- a/frontend_for_design.zip
+++ b/frontend_for_design.zip
--- a/scripts/run_standalone.py
+++ b/scripts/run_standalone.py
@@ -109,21 +109,27 @@ def log_stream(stream, prefix, color):

 def start_backend(backend_port: int, backend_cfg: Dict[str, Any], background: bool = False):
    logger.info(f"Starting Backend on port {backend_port}...")
-    
+
    app_cfg = backend_cfg.get("application", {})
    env = os.environ.copy()
    env["DATA_DIR"] = app_cfg.get("data_dir", "./data")
    env["LOGS_DIR"] = app_cfg.get("logs_dir", "./logs")
    env["LOG_LEVEL"] = backend_cfg.get("logging", {}).get("log_level", "INFO")
-    
+
    # Ensure directories exist
    os.makedirs(env["DATA_DIR"], exist_ok=True)
    os.makedirs(env["LOGS_DIR"], exist_ok=True)
-    
+
+    # Use venv python if available
+    python_exe = sys.executable
+    venv_python = os.path.join(os.path.dirname(os.path.dirname(os.path.abspath(__file__))), "venv", "bin", "python3")
+    if os.path.exists(venv_python):
+        python_exe = venv_python
+
    cmd = [
-        sys.executable, "-m", "uvicorn", 
-        "backend.main:app", 
-        "--host", "0.0.0.0", 
+        python_exe, "-m", "uvicorn",
+        "backend.main:app",
+        "--host", "0.0.0.0",
        "--port", str(backend_port)
    ]
Author	SHA1	Message	Date
Daniel Bedeleanu	992574e3ee	bede	2026-04-26 19:12:15 +03:00
Daniel Bedeleanu	cb1abe4d9c	Build [v1.14.32]	2026-04-26 14:45:54 +03:00
Daniel Bedeleanu	ba54820daa	Fix checkbox styling: make them properly visible and usable - Increased size from w-4 h-4 (16px) to w-5 h-5 (20px) - Changed rounded (border-radius: 100%) to rounded-sm (3px) for proper checkbox appearance - Added border-2 border-outline for visible checkbox border - Added hover state with primary color border - Added focus state with ring for keyboard accessibility - Used appearance: auto to use native checkbox rendering - Fixes issue where checkboxes appeared as small dots instead of proper checkboxes	2026-04-26 13:52:53 +03:00
Daniel Bedeleanu	5d21e72efd	Fix run_standalone.py to use venv python for uvicorn - Script was using /usr/bin/python3 which doesn't have uvicorn installed - Now detects and uses venv/bin/python3 if available - Backend now starts correctly with: 'python3 scripts/run_standalone.py start\|restart' - All services (Backend, Frontend, SSL Proxy) now start properly	2026-04-26 13:33:23 +03:00
Daniel Bedeleanu	8117216a3b	Add MANDATORY RULE: no silent feature removal without user consent - Added to AI_MANDATES.md section 3 (Operational Guardrails) - Added to CLAUDE.md as critical directive - Ensures all feature changes are transparent and require explicit user consent - This prevents accidental removal of functionality	2026-04-26 13:30:58 +03:00
Daniel Bedeleanu	7da86573b9	Add login mode toggle: allow switching between local and enterprise login - Added 'Local Users' and 'Enterprise' toggle buttons when LDAP is available - Users can now choose which login method to use - Defaults to local user list on page load - Both modes fully functional and accessible	2026-04-26 13:29:17 +03:00
Daniel Bedeleanu	8d21eacd28	Update SESSION_STATE: document complete LDAP login fix and enterprise mode restoration	2026-04-26 13:28:07 +03:00
Daniel Bedeleanu	dc999991a5	Restore enterprise/LDAP login mode detection - Added /users/auth-mode public endpoint to detect LDAP status - Updated frontend to call getAuthMode() instead of hardcoded false - Login page now automatically switches to enterprise mode when LDAP is enabled - Fixes missing LDAP user login UI that was previously disabled with TODO comment This restores the functionality that was commented out earlier where the login page would auto-detect whether to show local user list or enterprise username input.	2026-04-26 13:27:46 +03:00
Daniel Bedeleanu	53eb6b50dd	Update SESSION_STATE: document LDAP login fix completion and testing status	2026-04-26 13:21:21 +03:00
Daniel Bedeleanu	62c592d6b8	Fix TokenResponse schema: make user field non-optional and properly typed - Changed user field from Optional[User] to required User type - Ensures user object is always included in login response - Pydantic now properly serializes the complete user object - Frontend can now successfully retrieve response.user for localStorage	2026-04-26 13:19:30 +03:00
Daniel Bedeleanu	51716faf87	Fix LDAP login: ensure password encoding and include user in response - Fix password encoding: Convert password to UTF-8 bytes for LDAP protocol compatibility - Add 'user' field to TokenResponse schema for frontend compatibility - Update login endpoint to populate user object in response - Resolves 'Invalid Protocol Password' error on LDAP login attempts The issue was that frontend expected response.user but backend only returned individual fields (user_id, username, role), causing localStorage assignment to fail.	2026-04-26 13:13:55 +03:00