feat: Update officer photo workflow

.github/workflows/sync-firebase-images.yml

@@ -0,0 +1,102 @@
+name: Sync Firebase Officer Images
+
+on:
+  pull_request:
+  workflow_dispatch:
+    inputs:
+      dry_run:
+        description: 'Perform a dry run without creating PR'
+        required: false
+        default: 'false'
+        type: choice
+        options:
+          - 'true'
+          - 'false'
+      storage_bucket:
+        description: 'Firebase Storage bucket name (leave empty for default)'
+        required: false
+        type: string
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  sync-images:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.ref || 'main' }}
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '24'
+
+      - name: Install project dependencies
+        run: npm ci
+
+      - name: Install sync script dependencies
+        run: npm install --no-save firebase-admin @opentelemetry/api
+
+      - name: Create Firebase config
+        run: |
+          cat <<'EOF' > firebase-config.json
+          ${{ secrets.FIREBASE_SERVICE_ACCOUNT_JSON }}
+          EOF
+
+      - name: Run sync script
+        env:
+          FIREBASE_CONFIG_PATH: firebase-config.json
+          DRY_RUN: ${{ github.event.inputs.dry_run || 'false' }}
+          FIREBASE_STORAGE_BUCKET: ${{ github.event.inputs.storage_bucket || secrets.FIREBASE_STORAGE_BUCKET || '' }}
+        run: node scripts/sync-firebase-images.js
+
+      - name: Validate generated TypeScript
+        run: npx tsc --noEmit
+
+      - name: Create Pull Request
+        if: (github.event.inputs.dry_run || 'false') == 'false'
+        id: create-pr
+        uses: peter-evans/create-pull-request@v5
+        with:
+          commit-message: 'chore: sync officer images from firebase'
+          title: 'chore: sync officer images from firebase'
+          body: |
+            This PR automatically syncs officer images from Firebase Storage.
+
+            **Changes:**
+            - Cleared old images from `public/assets/officer/`
+            - Downloaded latest officer images from Firebase Storage
+            - Exported Firestore officers grouped by division to `config/officers.config.ts`
+
+            The sync script:
+            1. Removes all existing images
+            2. Fetches current officer data from Firestore
+            3. Downloads images from Firebase Storage
+            4. Renames files based on officer names (firstName-lastName)
+            5. Exports officers from `officers` collection as typed division-based arrays
+
+            Generated by: [Sync Firebase Officer Images](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})
+          branch: sync/firebase-officer-images
+          base: ${{ github.event.pull_request.head.ref || 'main' }}
+          delete-branch: true
+          labels: automated
+          add-paths: |
+            public/assets/officer/**
+            config/officers.config.ts
+
+      - name: Summary
+        if: always()
+        run: |
+          echo "## Sync Summary" >> $GITHUB_STEP_SUMMARY
+          echo "**Status:** Sync completed" >> $GITHUB_STEP_SUMMARY
+          if [ "${{ github.event.inputs.dry_run || 'false' }}" == "true" ]; then
+            echo "Mode: dry run" >> $GITHUB_STEP_SUMMARY
+          elif [ -n "${{ steps.create-pr.outputs.pull-request-number }}" ]; then
+            echo "PR: #${{ steps.create-pr.outputs.pull-request-number }}" >> $GITHUB_STEP_SUMMARY
+          else
+            echo "No PR created" >> $GITHUB_STEP_SUMMARY
+          fi

config/officers.config.ts

@@ -7,20 +7,29 @@ import { industryOfficers } from './industry.config';
 import { mediaOfficers } from './media.config';
 import { projectsOfficers } from './projects.config';
 import { researchOfficers } from './research.config';
+
 export type Officer = {
   name: string;
   position: string;
   image: string;
+  level?: number;
+  socialLinks?: Record<string, string>;
 };
 
-const defaultOfficer: Officer = {
-  name: 'Saksham Sangrula',
-  position: 'President',
-  image: '/assets/officer/OfficerImage.png',
+export type ExportedOfficer = {
+  firstName: string;
+  lastName: string;
+  photo: {
+    url: string;
+  };
+  socialLinks: Record<string, string>;
+  level: number;
+  title: string;
 };
 
 type Divisions =
   | 'advisor'
+  | 'executive'
   | 'board'
   | 'media'
   | 'research'
@@ -32,11 +41,14 @@ type Divisions =
   | 'industry';
 
 export const divisionOfficerMap: Record<Divisions, Officer[]> = {
-  advisor: [{
-    image: '/assets/officer/JohnCole.png',
-    name: 'John Cole',
-    position: 'ACM Faculty Advisor',
-  }],
+  advisor: [
+    {
+      image: '/assets/officer/JohnCole.png',
+      name: 'John Cole',
+      position: 'ACM Faculty Sponsor',
+    },
+  ],
+  executive: boardOfficers,
   board: boardOfficers,
   community: communityOfficers,
   development: developmentOfficers,
@@ -47,3 +59,49 @@ export const divisionOfficerMap: Record<Divisions, Officer[]> = {
   research: researchOfficers,
   projects: projectsOfficers,
 };
+
+function splitName(name: string) {
+  const trimmedName = name.trim();
+  const nameParts = trimmedName.split(/\s+/);
+
+  if (nameParts.length === 0) {
+    return { firstName: '', lastName: '' };
+  }
+
+  if (nameParts.length === 1) {
+    return { firstName: nameParts[0], lastName: '' };
+  }
+
+  return {
+    firstName: nameParts[0],
+    lastName: nameParts.slice(1).join(' '),
+  };
+}
+
+function toExportedOfficer(officer: Officer): ExportedOfficer {
+  const { firstName, lastName } = splitName(officer.name);
+
+  return {
+    firstName,
+    lastName,
+    photo: {
+      url: officer.image,
+    },
+    socialLinks: {},
+    level: 0,
+    title: officer.position,
+  };
+}
+
+export const officersByDivision: Record<string, ExportedOfficer[]> = {
+  Advisor: divisionOfficerMap.advisor.map(toExportedOfficer),
+  Executive: divisionOfficerMap.executive.map(toExportedOfficer),
+  Media: divisionOfficerMap.media.map(toExportedOfficer),
+  Research: divisionOfficerMap.research.map(toExportedOfficer),
+  Development: divisionOfficerMap.development.map(toExportedOfficer),
+  Projects: divisionOfficerMap.projects.map(toExportedOfficer),
+  Education: divisionOfficerMap.education.map(toExportedOfficer),
+  Community: divisionOfficerMap.community.map(toExportedOfficer),
+  HackUTD: divisionOfficerMap.hackutd.map(toExportedOfficer),
+  Industry: divisionOfficerMap.industry.map(toExportedOfficer),
+};