fix(ui): replace contrast severity with secondary for consistent styling (#208)

* refactor(logging): migrate to singleton logger service

- Created LoggerService singleton with WeakMap-based operation tracking
- Prevents duplicate logs and memory leaks
- CloudWatch-optimized structured JSON logging
- Updated 10 controllers, 2 middleware, 7 services, 2 utils
- Improved error handling with centralized error middleware logging
- Removed redundant error logs before next(error) calls
- Net reduction of 706 lines

LFXV2-903

Generated with [Claude Code](https://claude.ai/code)

Signed-off-by: Asitha de Silva <[email protected]>

* refactor(logging): extend logger service for infrastructure operations

Extended LoggerService to support both request-scoped and infrastructure
operations by making req parameter optional. Migrated all 41 direct
serverLogger calls across services, utilities, and routes to use the
unified logger service pattern.

Changes:
- Extended LoggerService methods to accept Request | undefined
- Migrated 11 request-scoped operations to pass req parameter
- Migrated 30 infrastructure operations to use logger with undefined
- Updated ai.service, user.service, project.service method signatures
- Replaced serverLogger calls in nats, snowflake, lock-manager services
- Updated CLAUDE.md with comprehensive logging documentation
- Removed serverLogger imports from all service/utility files

Benefits:
- Unified logging interface for all operations
- Better request correlation when context available
- Consistent error handling with err field
- Infrastructure operations now use same pattern

LFXV2-903

Signed-off-by: Asitha de Silva <[email protected]>

* docs(logging): update docs for infrastructure logging

Updated logging-monitoring.md to reflect the new dual-mode logger service
that supports both request-scoped and infrastructure operations. Added
documentation for the new server-logger.ts file that breaks the circular
dependency between server.ts and logger.service.ts.

Changes:
- Added logging architecture layers diagram showing server-logger.ts
- Updated all method signatures to show req | undefined pattern
- Added infrastructure logging examples for all methods
- Documented circular dependency resolution
- Clarified when to use request-scoped vs infrastructure logging

LFXV2-903

Signed-off-by: Asitha de Silva <[email protected]>

* refactor(meetings): move agenda generation to controller

Moved AI agenda generation logic from inline route handler to
MeetingController.generateAgenda method for better separation of concerns
and consistency with other route patterns.

Also improved logger usage in middleware and services:
- auth.middleware: better error tracking with startTime
- error-handler.middleware: use operation startTime when available
- access-check.service: move startOperation before try block
- etag.service: use debug instead of silent startOperation
- persona-helper: use success/error instead of warning

LFXV2-903

Signed-off-by: Asitha de Silva <[email protected]>

* docs(logging): update architecture diagram for server-logger

Updated CLAUDE.md logging architecture diagram to reflect the new
server-logger.ts file that breaks the circular dependency between
server.ts and logger.service.ts.

LFXV2-903

Signed-off-by: Asitha de Silva <[email protected]>

* fix: claude.md linting issues

Signed-off-by: Asitha de Silva <[email protected]>

* refactor(meetings): standardize generateAgenda error handling

LFXV2-903

Use ServiceValidationError.fromFieldErrors() and next() for validation
errors instead of manual 400 response, consistent with other controller
methods. Provides field-level error details and centralized error
handling through middleware.

Signed-off-by: Asitha de Silva <[email protected]>

* fix: linting issues for docs

Signed-off-by: Asitha de Silva <[email protected]>

* fix(ui): replace contrast severity with secondary for consistent styling

Replace 'contrast' severity with 'secondary' across meeting and committee
components for improved visual consistency and better theme compatibility.

Changes:
- Member card: secretary role badge severity
- Dashboard meeting card: button severities and meeting type badges
- Meeting card: join button severities and meeting type badges
- Meeting join: button styling and meeting type badges

LFXV2-905

Signed-off-by: Asitha de Silva <[email protected]>

---------

Signed-off-by: Asitha de Silva <[email protected]>

Author: asithade

apps/lfx-one/src/app/modules/committees/components/member-card/member-card.component.ts

@@ -48,7 +48,7 @@ export class MemberCardComponent {
     const role = this.member().role?.name?.toLowerCase();
     if (role === 'chair' || role === 'chairperson') return 'info';
     if (role === 'vice chair' || role === 'vice-chair') return 'success';
-    if (role === 'secretary') return 'contrast';
+    if (role === 'secretary') return 'secondary';
     return 'secondary';
   });
 

apps/lfx-one/src/app/modules/dashboards/components/dashboard-meeting-card/dashboard-meeting-card.component.html

@@ -99,7 +99,10 @@
     <div class="flex gap-2">
       <!-- See Meeting button - full width when alone, flex-1 when both buttons present -->
       <lfx-button
-        class="w-full border border-gray-300 text-gray-900 hover:bg-gray-50 h-8 text-sm font-medium"
+        class="w-full"
+        severity="secondary"
+        size="small"
+        styleClass="w-full"
         label="See Meeting Details"
         [href]="meetingDetailUrl()"
         target="_blank"
@@ -122,7 +125,8 @@
             type="button"
             rel="noopener noreferrer"
             icon="fa-light fa-video"
-            styleClass="w-full bg-emerald-500 hover:bg-emerald-600 text-white h-8 text-sm font-semibold"
+            severity="success"
+            styleClass="w-full h-8"
             data-testid="dashboard-meeting-card-join-button" />
         }
       }

apps/lfx-one/src/app/modules/dashboards/components/dashboard-meeting-card/dashboard-meeting-card.component.ts

@@ -103,7 +103,7 @@ export class DashboardMeetingCardComponent {
       if (config.textColor.includes('red')) severity = 'danger';
       else if (config.textColor.includes('blue')) severity = 'info';
       else if (config.textColor.includes('green')) severity = 'success';
-      else if (config.textColor.includes('purple')) severity = 'contrast';
+      else if (config.textColor.includes('purple')) severity = 'secondary';
       else if (config.textColor.includes('amber')) severity = 'warn';
 
       return {

apps/lfx-one/src/app/modules/meetings/components/meeting-card/meeting-card.component.html

@@ -208,7 +208,8 @@ <h3 class="text-base font-medium text-gray-900 leading-tight tracking-tight" dat
                 target="_blank"
                 rel="noopener noreferrer"
                 icon="fa-light fa-video"
-                styleClass="w-full !bg-emerald-500 hover:!bg-emerald-600 !text-white !font-semibold"
+                severity="success"
+                styleClass="w-full"
                 data-testid="join-meeting-button">
               </lfx-button>
             } @else {
@@ -221,7 +222,8 @@ <h3 class="text-base font-medium text-gray-900 leading-tight tracking-tight" dat
                 [queryParams]="joinQueryParams()"
                 rel="noopener noreferrer"
                 icon="fa-light fa-video"
-                styleClass="w-full !bg-emerald-500 hover:!bg-emerald-600 !text-white !font-semibold"
+                severity="success"
+                styleClass="w-full"
                 data-testid="join-meeting-button">
               </lfx-button>
             }

apps/lfx-one/src/app/modules/meetings/components/meeting-card/meeting-card.component.ts

@@ -571,7 +571,7 @@ export class MeetingCardComponent implements OnInit {
         case 'marketing':
           return { badgeClass: 'bg-green-100 text-green-500', severity: 'success', icon: 'fa-light fa-chart-line-up', text: meetingType };
         case 'technical':
-          return { badgeClass: 'bg-purple-100 text-purple-500', severity: 'contrast', icon: 'fa-light fa-code', text: meetingType };
+          return { badgeClass: 'bg-purple-100 text-purple-500', severity: 'secondary', icon: 'fa-light fa-code', text: meetingType };
         case 'legal':
           return { badgeClass: 'bg-amber-100 text-amber-500', severity: 'warn', icon: 'fa-light fa-scale-balanced', text: meetingType };
         default:

apps/lfx-one/src/app/modules/meetings/meeting-join/meeting-join.component.html

@@ -116,6 +116,7 @@ <h1 [attr.data-testid]="'meeting-title'">
                 <div class="w-full md:w-[296px]">
                   <lfx-button
                     class="w-full h-[40px]"
+                    styleClass="w-full"
                     label="Join Meeting"
                     icon="fa-light fa-video"
                     severity="success"

apps/lfx-one/src/app/modules/meetings/meeting-join/meeting-join.component.ts

@@ -336,7 +336,7 @@ export class MeetingJoinComponent {
         case 'marketing':
           return { badgeClass: 'bg-green-100 text-green-500', severity: 'success', icon: 'fa-light fa-chart-line-up', text: meetingType };
         case 'technical':
-          return { badgeClass: 'bg-purple-100 text-purple-500', severity: 'contrast', icon: 'fa-light fa-code', text: meetingType };
+          return { badgeClass: 'bg-purple-100 text-purple-500', severity: 'secondary', icon: 'fa-light fa-code', text: meetingType };
         case 'legal':
           return { badgeClass: 'bg-amber-100 text-amber-500', severity: 'warn', icon: 'fa-light fa-scale-balanced', text: meetingType };
         default: