fix: permanently rename index to app and landing to index to fix all routing and dark mode issues natively

Author: devanshi14malhotra

build_netlify.py

@@ -51,16 +51,7 @@ def rewrite_html(content: str) -> str:
 
 def build_html() -> None:
     for html_file in PAGES.glob("*.html"):
-        if html_file.name == "landing.html":
-            content = html_file.read_text(encoding="utf-8")
-            rewritten = rewrite_html(content)
-            (DIST / "index.html").write_text(rewritten, encoding="utf-8")
-            (DIST / "landing.html").write_text(rewritten, encoding="utf-8")
-            continue
-        elif html_file.name == "index.html":
-            target_file = DIST / "app.html"
-        else:
-            target_file = DIST / html_file.name
+        target_file = DIST / html_file.name
         content = html_file.read_text(encoding="utf-8")
         target_file.write_text(rewrite_html(content), encoding="utf-8")
 

frontend/js/app.js

@@ -81,7 +81,7 @@
 // Do NOT re-declare them here — use the globals from config.js directly.
 const IS_DEV = typeof window !== 'undefined' && ['localhost', '127.0.0.1'].includes(window.location.hostname);
 const moodAnalysisCache = new Map();
-const APP_ROUTE = window.location.pathname.endsWith('/app.html') ? 'app.html' : 'index.html';
+const APP_ROUTE = window.location.pathname.endsWith('/app.html') ? 'app.html' : 'app.html';
 
 const delay = (ms) => new Promise((res) => setTimeout(res, ms));
 

frontend/js/landing-interactions.js

@@ -201,16 +201,7 @@
         link.addEventListener('click', (event) => {
             let targetHref = link.getAttribute('href');
             if (!targetHref || targetHref.startsWith('#')) return;
-            
-            // Rewrite index.html to /app in production
-            if (targetHref.startsWith('index.html')) {
-                const isLocalPreview = window.location.protocol === 'file:' ||
-                  ['localhost', '127.0.0.1', ''].includes(window.location.hostname);
-                
-                if (!isLocalPreview) {
-                    targetHref = targetHref.replace('index.html', '/app');
-                }
-            }
+            // The link is natively app.html now. No need to intercept and rewrite URLs.
 
             event.preventDefault();
             document.body.classList.add('is-leaving');

frontend/pages/404.html

@@ -20,7 +20,7 @@
 
 <body class="error-page-body">
     <header>
-    <a href="index.html" class="logo">
+    <a href="app.html" class="logo">
       <img style="height: 40px" src="../assets/images/biblioDrift_favicon.png" alt="BiblioDrift Logo"> BiblioDrift
     </a>
     <div class="header-controls">
@@ -30,7 +30,7 @@
       </div>
       <nav class="nav-links">
         <div class="tooltip">
-          <a href="index.html" >Discovery</a>
+          <a href="app.html" >Discovery</a>
           <span class="tooltip-text"><i class="fa-solid fa-wand-magic-sparkles"></i> Explore books</span>
         </div>
         <div class="tooltip">
@@ -72,7 +72,7 @@ <h2 class="error-heading">This book is missing from the shelf.</h2>
       </p>
 
       <div class="error-actions">
-        <a href="index.html" class="btn-primary">Return to Library</a>
+        <a href="app.html" class="btn-primary">Return to Library</a>
         <a href="javascript:history.back()" class="btn-outline">Go Back</a>
       </div>
     </section>