Skip to content

Restore production builds, contact links and phone reading layouts - #1

Merged
jadenfix merged 6 commits into
mainfrom
fix/production-build-contact-links
Oct 14, 2026
Merged

jadenfix merged 6 commits into
mainfrom
fix/production-build-contact-links

Conversation

@jadenfix

@jadenfix jadenfix commented Oct 13, 2026 •

Copy link
Copy Markdown
Owner

Fix JSX punctuation/comment-node lint blockers and unused bindings without changing visible exercise copy. Replace placeholder contact addresses with the existing homepage address. Keep technical identifiers and grid children inside narrow reading layouts.

Validation: clean install, lint and production build pass. Actual Chrome136 renders all8 routes on desktop/mobile (16 checks) without page errors or horizontal overflow. This is a presentation/exercise site, not a live Kiln API application.

Follow-up fixes the inert mobile menu button and permanently expanded fixed header. Menu starts closed, toggles with labeled expanded/control state, closes on route selection or Escape, and restores focus on Escape. Actual Navigation component passes Chrome136 mobile/desktop interaction harness with real React/framer-motion (Next routing adapters and minimal CSS; full Next build/deployment not rerun). Old source fails the interaction control. Linked Vercel deployment remains failed and its team logs are unavailable via current connection; merge remains held.

@vercel

vercel Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
kiln Ready Ready Preview Oct 14, 2026 7:33pm UTC

@jadenfix jadenfix changed the title Restore production builds and replace placeholder contact links Restore production builds, contact links and phone reading layouts Oct 13, 2026

Copy link
Copy Markdown
Owner Author

Independent exact-head audit — HOLD (truthful demo / deployment)

Rechecked open PR head a07a2b636dcfaa59f1213b02b929bf149678ecf7 (tree 579dacfe0d2520af7e9abb486ac2497337ec3137) against base 36f2c6c33a32f65e1fcf92cf8993e8e04652a9f8.

What passed

  • npm ci
  • npm run lint — no warnings/errors
  • npm run build — optimized production build, 8 static routes
  • Actual headless Chromium production-server check: all 8 routes returned 200 at 1440×900 and 320×568; no page/console errors and no horizontal overflow (16 route/viewport cases).
  • Actual mobile navigation interaction: starts closed; opens with 5 links; aria-expanded changes; Escape closes and restores focus; selecting Task 3 closes the menu; current-page state is exposed.

Blocking integrity findings

  1. The checked-in repository is a static Next presentation: the complete non-generated file inventory contains TSX/CSS/config/assets only, and there is no runtime Kiln client, fetch, axios, result dataset, receipt, or source-pinned data artifact. Nevertheless the rendered UI asserts “Real-Time API / Live Kiln Integration,” “Real Kiln API integration with live validator performance data,” “Production-Ready Code,” and “100% API Success Rate / Robust error handling.” Exact validator/APY figures are embedded as static literals. Those claims are not reproducible from this source.
  2. Task 3's View Full Code link resolves to generic https://github.com, not an implementation or source-pinned analysis.
  3. There is still no checked-in GIF/MP4, media verifier, or source/data provenance for a truthful workflow demo.
  4. GitHub reports no Actions runs for this head, and the only hosted check remains Vercel Error (bot comment updated 2026-10-13 01:44 UTC).

Required before clearing

  • Either check in and test the actual provider/API workflow plus immutable result provenance, or label the figures everywhere as static/historical/fixture data with source/date/method and remove live, real-time, production-ready, success-rate, and performance claims not established by this repository.
  • Replace the generic code link with the exact implementation/provenance target (or remove it).
  • Repair and verify the hosted deployment.
  • Capture a visibly scope-labeled GIF and MP4 from the final unchanged source; fully decode/inspect them and record SHA-256/Git blob identities.

This does not dispute the responsive/mobile-navigation repair; that bounded behavior passed. It blocks only truthful-demo/current-main credit and merge clearance. No branch changes, push, merge, close, or retarget were performed.

@jadenfix

Copy link
Copy Markdown
Owner Author

Continued repair at de82c4676d321767e3c9081e3e3c42b25755cccf, tested tree 56d25d5a20eb877453cc8556d745574cecf928cb. Next 16.4.1 / fresh compatible dependencies, real ESLint CLI and compiler configuration, and offline-safe system fonts now pass full lint and production build. Actual production Next server + Chrome 136 passes all 8 routes at 1440px and 390px (16 route checks), plus mobile toggle, Escape/focus restoration and link navigation (3 checks); no page errors or horizontal overflow. Hosted Production runtime workflow preserves this verification.

Full unfiltered audit decreased 24 → 5, critical 1 → 0; production dependency audit has 0 findings. Five HIGH development findings remain from the official Next ESLint plugin's fast-glob/micromatch/braces chain. This is explicitly not a clean full audit or a deployed Vercel verification. New exact-head CI is pending; preserve the existing deployment hold until actual provider logs/source deployment are checked.

@jadenfix

Copy link
Copy Markdown
Owner Author

Exact-head hosted production verification is now green at de82c4676d321767e3c9081e3e3c42b25755cccf: production job. Ordinary clean npm install, lint, build, Chromium installation and all real production route/navigation assertions succeeded. Duplicate PR-trigger run also succeeded. No source changes after this verified head. Existing development-only audit findings and unverified Vercel deployment boundary remain as documented above.

@jadenfix
jadenfix merged commit 3213408 into main Oct 14, 2026
5 checks passed

This branch was successfully deployed

1 active deployment
Preview — de82c467 Deployed Oct 14, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant