Address PR #253 review comments and adjust preview workflow

Agent-Logs-Url: https://github.com/SillyLittleTech/Portf/sessions/e0e67947-5d7c-4bdd-9aba-1aa674165a7b

Co-authored-by: kiyarose <75678535+kiyarose@users.noreply.github.com>

Author: Copilot

.github/ISSUE_TEMPLATE/content_update.yml

@@ -22,7 +22,7 @@ body:
     attributes:
       label: "Affected pages or sections"
       description: "List routes, components, or assets that need updates."
-      placeholder: "Home hero copy, About bio, public/index.html gradient."
+      placeholder: "Home hero copy, About bio, public/fallback/index.html gradient."
     validations:
       required: true
   - type: textarea

.github/workflows/github-pages-preview.yml

@@ -10,6 +10,7 @@ permissions:
   contents: read
   pages: write
   id-token: write
+  pull-requests: write
 
 concurrency:
   group: pages-preview-${{ github.event.pull_request.number || github.ref }}

AGENTS.md

@@ -42,7 +42,7 @@
 - Privacy and legal copy should pull their “Last updated” value from `getBuildUpdatedAt()` so the date reflects the latest build automatically—never hardcode the date.
 - Developing skills use a dotted accent pill—follow the existing `isDeveloping` pattern in `SkillsSection` when adding new skills.
 - The contact form generates a `mailto:` draft; keep it client-only (no POST handlers) and preserve the copy-to-clipboard CTA.
-- The fallback page in `public/index.html` should mirror the warm accent palette; update gradients/buttons there when the brand colors change.
+- The fallback page in `public/fallback/index.html` should mirror the warm accent palette; update gradients/buttons there when the brand colors change.
 
 ## Testing Guidelines
 

CONTRIBUTING.md

@@ -19,7 +19,7 @@ Below are suggestions and quick checks to help you convert this repo into your o
     - Important: this template includes a BSD-3-Clause `LICENSE` that must be preserved as-is in the source
       when redistributing or publishing derived works (see `LICENSE.example` for how to add your own copyright
       attribution without removing the original line).
-  - `public/index.html` and `src/data/*` — replace demo content with your personal content.
+  - `public/fallback/index.html` and `src/data/*` — replace demo content with your personal content.
 
 - If you plan to accept outside contributions on your fork, add a `CODE_OF_CONDUCT.md`, a `PULL_REQUEST_TEMPLATE.md`, and/or an `ISSUE_TEMPLATE/` directory so contributors have clear expectations.
 

LICENSE.example

@@ -39,7 +39,7 @@ Choose the license that aligns with how you want others to use and contribute to
 
 - Fork the repo.
 - Update `LICENSE` (change copyright line and year).
-- Update `README.md` and `public/index.html` with your site name and contact info.
+- Update `README.md` and `public/fallback/index.html` with your site name and contact info.
 - Replace `src/data/*` content with your portfolio data.
 - Update `package.json` `name` and `license` fields.
 

package-lock.json

@@ -33,8 +33,7 @@
     "react-dom": "^19.2.0",
     "tailwind-merge": "^3.3.1",
     "tailwindcss": "^4.1.8",
-    "wrangler": "^4.16.0",
-    "zod": "^4.1.12"
+    "wrangler": "^4.16.0"
   },
   "devDependencies": {
     "@astrojs/check": "^0.9.9",

package.json

@@ -10,7 +10,7 @@
     <title>Something Went Wrong · Your Name</title>
     <meta
       name="description"
-      content="Fallback page for the template portfolio when the main application cannot load. Replace this content in public/index.html if desired."
+      content="Fallback page for the template portfolio when the main application cannot load. Replace this content in public/fallback/index.html if desired."
     />
     <style>
       @font-face {

public/index.html public/fallback/index.htmlpublic/index.html renamed to public/fallback/index.html

@@ -11,7 +11,10 @@ const DATA_BASE_URL = resolveRemoteDataBaseUrl(
   import.meta.env.VITE_REMOTE_DATA_URL as string | undefined,
 );
 const REMOTE_DATA_ENABLED = DATA_BASE_URL !== null;
-const REMOTE_DATA_PATH = "/__remote-data";
+const REMOTE_DATA_PATH = `${import.meta.env.BASE_URL}__remote-data`.replace(
+  /\/{2,}/g,
+  "/",
+);
 
 // LocalStorage namespace for cached remote data. Rename if you fork the template
 // to avoid collisions with other apps in the browser.

src/hooks/useRemoteData.ts

@@ -10,6 +10,10 @@ const {
   title = "Portfolio Starter",
   description = "A modular portfolio starter built with Astro and React.",
 } = Astro.props as Props;
+const faviconHref = `${import.meta.env.BASE_URL}favicon.svg`.replace(
+  /\/{2,}/g,
+  "/",
+);
 ---
 
 <!doctype html>
@@ -18,6 +22,7 @@ const {
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <meta name="description" content={description} />
+    <link rel="icon" type="image/svg+xml" href={faviconHref} />
     <script is:inline>
       (() => {
         try {