lekss361/gendesign

Fork 0

chore: add landing.html static page #279

Merged

lekss361 merged 1 commit from chore/landing-html into main

2026-05-17 11:03:16 +00:00

lekss361 commented

2026-05-17 10:53:28 +00:00

Owner

Summary

Marketing landing «gendsgn — аналитика спроса для девелоперов» добавлен как static asset в frontend/public/landing.html.

Self-contained HTML (50KB, ~790 lines, inline assets)
Будет доступен через Next.js public/ serving после deploy
URL после merge: https://gendsgn.ru/landing.html

Test plan

Файл скопирован, no lint applies (static HTML)
Post-deploy: curl -sI https://gendsgn.ru/landing.html → 200 OK, content-type text/html
Open в браузере, визуальный QA

## Summary Marketing landing «gendsgn — аналитика спроса для девелоперов» добавлен как static asset в `frontend/public/landing.html`. - Self-contained HTML (50KB, ~790 lines, inline assets) - Будет доступен через Next.js `public/` serving после deploy - **URL после merge:** https://gendsgn.ru/landing.html ## Test plan - [x] Файл скопирован, no lint applies (static HTML) - [ ] Post-deploy: `curl -sI https://gendsgn.ru/landing.html` → 200 OK, content-type text/html - [ ] Open в браузере, визуальный QA

lekss361 commented

2026-05-17 10:56:28 +00:00

Author

Owner

⚠️ Empty diff — push commit missing

Findings:

changed_files: 0, +0/-0 — head SHA (8d95bf0c) равен merge_base. Ветка chore/landing-html не содержит новых коммитов поверх main.
Head 8d95bf0c — это merge commit PR #277 (fix(sf-04b): bulk seed mappings), уже на main.
PR description упоминает frontend/public/landing.html (50KB, ~790 lines), но diff пуст → файл не был commit'нут / push'нут в ветку.

Действие:

Локально: git checkout chore/landing-html → git add frontend/public/landing.html → git commit -m "chore: add landing.html" → git push forgejo chore/landing-html
После push diff появится, тогда re-trigger review

Не мержу: empty diff merge создаст пустой merge commit без value.

### ⚠️ Empty diff — push commit missing **Findings:** - `changed_files: 0`, `+0/-0` — head SHA (`8d95bf0c`) равен merge_base. Ветка `chore/landing-html` не содержит новых коммитов поверх main. - Head `8d95bf0c` — это merge commit PR #277 (`fix(sf-04b): bulk seed mappings`), уже на main. - PR description упоминает `frontend/public/landing.html` (50KB, ~790 lines), но diff пуст → файл не был commit'нут / push'нут в ветку. **Действие:** 1. Локально: `git checkout chore/landing-html` → `git add frontend/public/landing.html` → `git commit -m "chore: add landing.html"` → `git push forgejo chore/landing-html` 2. После push diff появится, тогда re-trigger review Не мержу: empty diff merge создаст пустой merge commit без value.

lekss361 added 1 commit 2026-05-17 10:59:26 +00:00

chore: add landing.html static page a3066733c2

Marketing landing «gendsgn — аналитика спроса для девелоперов».
Self-contained HTML (50KB, inline assets) — будет доступен через
Next.js public/ serving по URL https://gendsgn.ru/landing.html

lekss361 merged commit 28ca7bf6a8 into main

2026-05-17 11:03:16 +00:00

lekss361 referenced this pull request from a commit

2026-05-17 11:03:17 +00:00

Merge pull request 'chore: add landing.html static page' (#279) from chore/landing-html into main

lekss361 commented

2026-05-17 11:03:44 +00:00

Author

Owner

Merged via deep-code-reviewer — verdict ✅ APPROVE (MINOR notes ниже, не блокирующие).

Security scan: чисто.

Inline CSS + JS, no <script src="..."> к ненадёжным CDN (только Google Fonts preconnect).
Нет analytics-трекеров (gtag/ym/hotjar/sentry/etc).
Нет API keys / JWT / bcrypt / hardcoded secrets / internal hosts.
Нет <form>/<input> → нулевой XSS surface, CSRF неактуален.
Все target="_blank" обрабатываются inline JS-хендлером с noopener,noreferrer + protocol allowlist (http/https/mailto).

Cross-file impact: конфликтов нет. frontend/public/ содержал только docs/, landing.html — новый файл. Next.js root / (page.tsx) не задет — /landing.html обслуживается как static asset.

MINOR (follow-up отдельным PR, не блокирует):

DEV-навбар видимый продакшну. Жёлтый sticky bar «DEV · ТОЛЬКО ДЛЯ РАЗРАБОТКИ» (lines ~344-356) со ссылками на analytics.html, site-finder.html, monitoring.html, audit.html, gendsgn-audit.html — все вернут 404 после deploy, файлов нет. Убрать для public URL.
Битые CTA-ссылки. 14 ссылок на несуществующие *.html файлы. Основная CTA «Запросить пилот» — <a href="#">, без формы. Маркетинг неэффективен пока не починим.
Tooling-скрипты от HTML-builder'а (data-od-sandbox-shim, data-od-snapshot-bridge) — overrride localStorage/sessionStorage, intercept всех кликов <a>, listen postMessage от window.parent для snapshot canvas. Не уязвимость (отдаёт только скриншот самой страницы, нет app-секретов), но мусор от dev-tooling — убрать для прода.
Нет <meta name="robots">. Будет индексироваться по умолчанию. Рекомендую noindex пока CTA битые, или явное решение в decisions/.
Нет favicon / OG / Twitter card — cosmetic.

Post-deploy smoke:

curl -sI https://gendsgn.ru/landing.html
# expect: HTTP/2 200, content-type: text/html

Recommendation: robots.txt / meta-robots decision записать в decisions/ перед публичным анонсом. Follow-up PR для очистки dev-навбара + tooling-скриптов.

Merged via deep-code-reviewer — verdict ✅ APPROVE (MINOR notes ниже, не блокирующие). **Security scan:** чисто. - Inline CSS + JS, no `<script src="...">` к ненадёжным CDN (только Google Fonts preconnect). - Нет analytics-трекеров (gtag/ym/hotjar/sentry/etc). - Нет API keys / JWT / bcrypt / hardcoded secrets / internal hosts. - Нет `<form>`/`<input>` → нулевой XSS surface, CSRF неактуален. - Все `target="_blank"` обрабатываются inline JS-хендлером с `noopener,noreferrer` + protocol allowlist (http/https/mailto). **Cross-file impact:** конфликтов нет. `frontend/public/` содержал только `docs/`, landing.html — новый файл. Next.js root `/` (`page.tsx`) не задет — `/landing.html` обслуживается как static asset. **MINOR (follow-up отдельным PR, не блокирует):** 1. **DEV-навбар видимый продакшну.** Жёлтый sticky bar «DEV · ТОЛЬКО ДЛЯ РАЗРАБОТКИ» (lines ~344-356) со ссылками на `analytics.html`, `site-finder.html`, `monitoring.html`, `audit.html`, `gendsgn-audit.html` — все вернут 404 после deploy, файлов нет. Убрать для public URL. 2. **Битые CTA-ссылки.** 14 ссылок на несуществующие `*.html` файлы. Основная CTA «Запросить пилот» — `<a href="#">`, без формы. Маркетинг неэффективен пока не починим. 3. **Tooling-скрипты от HTML-builder'а** (`data-od-sandbox-shim`, `data-od-snapshot-bridge`) — overrride `localStorage`/`sessionStorage`, intercept всех кликов `<a>`, listen `postMessage` от `window.parent` для snapshot canvas. Не уязвимость (отдаёт только скриншот самой страницы, нет app-секретов), но мусор от dev-tooling — убрать для прода. 4. **Нет `<meta name="robots">`.** Будет индексироваться по умолчанию. Рекомендую `noindex` пока CTA битые, или явное решение в `decisions/`. 5. **Нет favicon / OG / Twitter card** — cosmetic. **Post-deploy smoke:** ```bash curl -sI https://gendsgn.ru/landing.html # expect: HTTP/2 200, content-type: text/html ``` **Recommendation:** robots.txt / meta-robots decision записать в `decisions/` перед публичным анонсом. Follow-up PR для очистки dev-навбара + tooling-скриптов.

lekss361 referenced this pull request from a commit

2026-05-17 12:37:34 +00:00

chore: refresh preview pages + add tradein landing

lekss361 referenced this pull request

2026-05-17 12:37:47 +00:00

chore: refresh preview pages + add tradein landing #283

lekss361 referenced this pull request

2026-05-17 12:40:17 +00:00

chore: refresh preview pages + add tradein landing #283