[GothwadTech]

Full cleanup pass on the Gothwad Tech site: dead code removal, housekeeping, honest framing of the browser benchmarks, lint tooling, path routing and code splitting.

Eight commits, roughly in order. Net effect: −958 lines, −388 KB of committed cruft, initial JS −29%.

1. Dev server was unreachable through any proxy

vite.config.ts returned HTTP 403 for every non-localhost Host header, so the app could not be viewed in a preview/sandbox environment at all. Added server.allowedHosts. Also fixed a mojibake comment (Do not modifyâ—file watching, a double-encoded em-dash).

2. No .gitignore

The repo had none. Installing deps and running a build drops node_modules/ and dist/ into the tree — one git add . and they'd have been committed. Added a standard Node/Vite ignore file, including .env*.

3. Stale package-lock.json

It still declared react-router-dom and @google/genai ^1.29.0 while package.json had already dropped the router and moved to ^2.4.0. Installs weren't reproducible. Regenerated.

4. Removed 32 dead files (~2,250 LOC)

Root components/ (24 files) and screens/ (8 files) were a leftover react-router version of the site. Nothing outside those folders imports them — they only import each other.

They were the sole cause of all 13 typecheck errors (they import react-router-dom, now removed). src/ was always clean. Removing them takes tsc --noEmit from 13 errors → 0, and drops CSS from 94.2 → 79.3 kB since Tailwind no longer scans them. Recoverable from history at 14e3273.

5. Housekeeping

• Dropped web.zip (388 KB stale build artifact) from git.
• package.json renamed from the leftover template name react-example → gothwadtech.
• De-duplicated public/: the icon set was committed 5 times over. md5 confirmed byte-identical copies in assets/favicons/, assets/pwa/, and 4 root icon-*.png files. Kept the canonical /assets/icons/* set plus root favicon.ico and apple-touch-icon.png (auto-requested by browsers). Fixed the two manifest.json entries that would have 404'd.

6. Browser benchmarks are now honestly labelled

BrowserSimulatorModal presented hardcoded arithmetic as a measured head-to-head comparison:

gothwadRam = 45 + tabCount * 9.5
chromeRam  = 160 + tabCount * 42   → "Consumes 3.4x more memory"
simulatedAdsBlocked  = 38
simulatedDataSavedMb = 14.2
"Speed: 38.4 MB/s"

Presenting modelled figures as measured performance is a misleading-advertising risk (ASCI in India). No numbers were changed — only the framing: a persistent disclaimer on every tab, ~N MB (modelled), "Modelled at ~Nx the figure above — illustrative, not a measured comparison", "(sample page)", "Simulated speed", and the URL bar changed from gothwad://speed-benchmark-core to gothwad://demo/illustrative-preview.

7. ESLint + unused imports

Added an ESLint 10 flat config with typescript-eslint, react-hooks and react-refresh.

Only the classic rules-of-hooks + exhaustive-deps pair is enabled. eslint-plugin-react-hooks v7's configs.recommended now bundles the entire React Compiler ruleset (immutability, purity, gating, use-memo, set-state-in-effect…), which assumes the app is verified by the React Compiler and would flood a hand-written codebase — so rules are opted into individually. Noted in the config.

Removed 131 unused imports across 18 files (mostly lucide-react icons). The bundle is byte-identical since Vite tree-shakes them anyway — pure hygiene. Warnings: 140 → 25, 0 errors.

Scripts: lint is now eslint, typecheck is tsc, verify runs all three plus build.

8. Real path routing + code splitting + per-route SEO

• Routing: #products → /products. Crawlers largely ignore fragments, so the old URLs were effectively one page to Google. Legacy links still work — a hash naming a route is replaceState'd to the canonical path on mount. Plain in-page anchors (#docs, #services, #mission, #order) are untouched and still scroll; the home page has section ids that overlap route names, so pageFromHash only matches real routes. Back/forward works via popstate.
• SEO: per-route document.title, meta description, og:*/twitter:* and canonical — path routing is only worth doing if crawled metadata changes with the route. sitemap.xml now lists the 6 real routes instead of 8 hash URLs.
• Perf: lazy-loaded all six pages and three modals behind Suspense. Initial chunk 386 → 275 kB raw, 104.6 → 85.8 kB gzip (−19%); each page is now a 5–23 kB chunk fetched on demand. Modals mount only when opened, safe because all three guard with a plain if (!isOpen) return null and have no exit animation.
• CI: added .github/workflows/ci.yml running typecheck + lint + build with npm caching.

---

Verification

npm run verify   →   typecheck ✓   lint ✓ (0 errors)   build ✓

Dev server: /, /products, /solutions, /portfolio, /about, /contact all 200 with SPA fallback; /manifest.json, /favicon.ico, /robots.txt, /sitemap.xml and icons still served as static (ahead of the Vercel rewrite), so deep links work in production while assets keep their correct content types.

Worth knowing

• lint changed meaning — it used to run tsc --noEmit, it now runs ESLint. Use npm run typecheck for types.
• 25 remaining lint warnings are non-blocking and left deliberately: 16 unused props/vars, 7 any, 2 react-refresh warnings on the Theme/Language contexts (they export a provider component and a hook from one file — a normal pattern). Adding --max-warnings now would just turn CI red; happy to clean these up separately.
• No secret leak — process.env.GOTHWAD_API_KEY in DeveloperDocsSection is inside a displayed code sample, not real config.
• Deployment — Cloudflare only. vercel.json is deleted; nothing is hosted on Vercel. The site deploys through Cloudflare Pages, which does not rewrite unknown paths by default, so moving to real paths would have made /products 404 on refresh. Added public/_redirects with /* /index.html 200 (Vite copies it verbatim into dist/). Static assets are matched before the catch-all, so favicon, manifest, sitemap and icons keep their real content.
  • When you move to Cloudflare Workers, _redirects will not carry the SPA fallback — Workers ignores it for that. You'll need this in wrangler.jsonc instead:

    "assets": {
      "directory": "./dist",
      "not_found_handling": "single-page-application"
    }

    plus a "binding": "ASSETS" and an env.ASSETS.fetch(request) fallback in the Worker if you add Worker code alongside the static assets. Noted in public/_redirects so it isn't lost.
• I could not verify the live deployment end-to-end from this sandbox: outbound network is restricted to GitHub/npm/PyPI, so I couldn't fetch gothwadtech.com/products. Worth one manual click on a deep link after merge.

[cloudflare-workers-and-pages]

Deploying gothwadtech with    Cloudflare Pages

Latest commit:	d7b3693
Status:	✅  Deploy successful!
Preview URL:	https://d125fc7a.gothwadtech.pages.dev
Branch Preview URL:	https://arena-b14c474b-gothwadtech.gothwadtech.pages.dev

View logs