Issue 269

.depcheckrc

@@ -4,6 +4,19 @@
     "@storybook/addon-ondevice-controls",
     "@storybook/addon-ondevice-actions",
     "storybook",
-    "@storybook/react"
+    "@storybook/react",
+    "expo-speech-recognition",
+    "expo-system-ui",
+    "prettier-plugin-tailwindcss",
+    "@lhci/cli",
+    "@size-limit/preset-app",
+    "@testing-library/jest-native",
+    "@types/babel__generator",
+    "@types/babel__template",
+    "@types/babel__traverse",
+    "react-native-nitro-modules",
+    "metro",
+    "k6",
+    "js-yaml"
   ]
 }

.github/workflows/bundle-size.yml

@@ -3,14 +3,22 @@ name: Bundle Size Tracking
 on:
   push:
     branches: [main]
-  pull_request:
+  pull_request_target:
     branches: [main]
 
+permissions:
+  contents: read
+  pull-requests: write
+  issues: write
+
 jobs:
   bundle-size:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }}
+          repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
 
       - uses: actions/setup-node@v4
         with:
@@ -136,7 +144,7 @@ jobs:
           fi
 
       - name: Comment PR with bundle size
-        if: github.event_name == 'pull_request'
+        if: github.event_name == 'pull_request' || github.event_name == 'pull_request_target'
         uses: actions/github-script@v7
         with:
           script: |
@@ -166,10 +174,13 @@ jobs:
                 body += `\n⚠️ **Warning**: Bundle size increased by more than 10%!\n`;
               }
             }
-
-            github.rest.issues.createComment({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              body: body
-            });
+            try {
+              await github.rest.issues.createComment({
+                issue_number: context.issue.number,
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                body: body
+              });
+            } catch (error) {
+              console.warn('Skipping PR commenting due to permission limits (e.g. fork PRs):', error.message);
+            }

.github/workflows/performance-regression.yml

@@ -3,14 +3,19 @@ name: Performance Regression
 on:
   push:
     branches: [main]
-  pull_request:
+  pull_request_target:
     branches: [main]
 
 # Cancel in-progress runs for the same branch
 concurrency:
   group: perf-regression-${{ github.ref }}
   cancel-in-progress: true
 
+permissions:
+  contents: read
+  pull-requests: write
+  issues: write
+
 jobs:
   # ─────────────────────────────────────────────
   # 1. Bundle Size Check (size-limit + regression)
@@ -20,6 +25,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }}
+          repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
 
       - uses: actions/setup-node@v4
         with:
@@ -132,6 +140,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }}
+          repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
 
       - uses: actions/setup-node@v4
         with:
@@ -206,6 +217,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }}
+          repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
 
       - name: Install k6
         run: |
@@ -292,6 +306,9 @@ jobs:
     if: always()
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }}
+          repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
 
       - uses: actions/setup-node@v4
         with:
@@ -321,7 +338,7 @@ jobs:
           retention-days: 30
 
       - name: Post PR comment
-        if: github.event_name == 'pull_request' && always()
+        if: (github.event_name == 'pull_request' || github.event_name == 'pull_request_target') && always()
         uses: actions/github-script@v7
         with:
           script: |

.prettierignore

@@ -5,6 +5,10 @@
 **/.next/**
 **/coverage/**
 
+# Lock files — must not be reformatted (breaks npm ci integrity checks)
+package-lock.json
+npm-shrinkwrap.json
+
 # Generated
 *.log
 .DS_Store

app/(tabs)/_layout.tsx

@@ -7,6 +7,18 @@ import { IconSymbol } from '@/components/ui/icon-symbol';
 import { Colors } from '@/constants/theme';
 import { useTheme } from '@/store';
 
+const HomeIcon = ({ color }: { color: string }) => (
+  <IconSymbol size={28} name="house.fill" color={color} />
+);
+
+const SearchIcon = ({ color }: { color: string }) => (
+  <IconSymbol size={28} name="magnifyingglass" color={color} />
+);
+
+const ProfileIcon = ({ color }: { color: string }) => (
+  <IconSymbol size={28} name="person.fill" color={color} />
+);
+
 const TabLayout = () => {
   const theme = useTheme();
 
@@ -25,23 +37,21 @@ const TabLayout = () => {
           name="index"
           options={{
             title: 'Home',
-            tabBarIcon: ({ color }) => <IconSymbol size={28} name="house.fill" color={color} />,
+            tabBarIcon: HomeIcon,
           }}
         />
         <Tabs.Screen
           name="search"
           options={{
             title: 'Search',
-            tabBarIcon: ({ color }) => (
-              <IconSymbol size={28} name="magnifyingglass" color={color} />
-            ),
+            tabBarIcon: SearchIcon,
           }}
         />
         <Tabs.Screen
           name="profile"
           options={{
             title: 'Profile',
-            tabBarIcon: ({ color }) => <IconSymbol size={28} name="person.fill" color={color} />,
+            tabBarIcon: ProfileIcon,
           }}
         />
       </Tabs>

app/_layout.tsx

@@ -7,13 +7,13 @@
 
 import '../global.css'; // NativeWind CSS
 import { MemoryProfilerOverlay } from '../components/DevTools';
 import { RetryErrorBoundary } from '../components/ErrorBoundary/RetryErrorBoundary';
 
 import { AnalyticsProvider, ErrorBoundary, OfflineIndicatorProvider } from '../src/components';
 import { useAnalytics } from '../src/hooks';
 import { useDeepLink } from '../src/hooks/useDeepLink';
 import { sessionRestorationService } from '../src/services/sessionRestoration';
 import { preloadService } from '../src/services/preloadService';
 import { useAppStore } from '../src/store';
 import { getPathFromDeepLink } from '../src/utils/linkParser';
 import { prefetchExternalResources } from '../src/utils/resourceHints';
@@ -36,7 +36,7 @@
   useEffect(() => {
     if (pathname) {
       trackScreen(pathname, { segments: segments.join('/') });
-      
+
       // Track and record transitions + trigger predictive preloading
 
       if (prevPathname.current !== pathname) {
@@ -48,7 +48,7 @@
         }
 
         sessionRestorationService.saveRoute(pathname);
-        
+
         // Trigger background preloading for predicted destinations
         preloadService.preload(pathname, router);
       }
@@ -73,7 +73,7 @@
   const router = useRouter();
 
   const handleDeepLink = useCallback(
-    deepLink => {
+    (deepLink: any) => {
       const path = getPathFromDeepLink(deepLink);
       if (path) {
         router.replace(path);
@@ -133,9 +133,29 @@
 
   return (
     <ErrorBoundary boundaryName="RootLayout">
-      {/* ✅ Wrap with RetryErrorBoundary */}
       <RetryErrorBoundary>
         <AnalyticsProvider>
           <ScreenTracker />
           <ThemeSync />
+          <OfflineIndicatorProvider>
+            <GestureHandlerRootView style={{ flex: 1 }}>
+              <Stack>
+                <Stack.Screen name="(tabs)" options={{ headerShown: false }} />
+                <Stack.Screen name="course-viewer" options={{ headerShown: false }} />
+                <Stack.Screen name="profile/[userId]" options={{ headerShown: false }} />
+                <Stack.Screen name="search" options={{ headerShown: false }} />
+                <Stack.Screen name="settings" options={{ headerShown: false }} />
+                <Stack.Screen name="qr-scanner" options={{ headerShown: false }} />
+                <Stack.Screen name="quiz" options={{ headerShown: false }} />
+                <Stack.Screen name="modal" options={{ presentation: 'modal' }} />
+              </Stack>
+              <MemoryProfilerOverlay />
+            </GestureHandlerRootView>
+          </OfflineIndicatorProvider>
+        </AnalyticsProvider>
+      </RetryErrorBoundary>
+    </ErrorBoundary>
+  );
+};
 
+export default RootLayout;