fix: real logo on PDFs, login redesign, password-reset diagnostics, dashboard strip-down#242
Merged
thecelestialmismatch merged 1 commit intoJul 23, 2026
Conversation
…strip-down Four founder-reported issues + a login-page reference redesign. All prior "shipped" fixes were already on main (branch == origin/main), so these are real bugs + observability, not stale deploys. - PDF: embed the real doberman logo (base64, lib/reports/logo-data.ts) via addImage in a white chip on the dark cover band; vector shield kept as the feature-detected fallback. One change covers every report type. - Login/signup: reference redesign — shared Sign in/Sign up segmented toggle (components/auth/AuthTabs.tsx, preserves ?redirect=), Work-email labels, Terms line, real Logo. All working auth preserved (password/OTP/2FA/OAuth/ forgot-password). - Password reset: the code chain was already correct; the enumeration-safe route's silent 200 hid every config failure. Added value-free reset_* diagnostics to /api/health (lib/auth/reset-diagnostics.ts) + an honest "didn't get it?" line on the success screen. - Dashboard: signed-in operators default-hide the simulated telemetry (SIGNED_IN_STRIPPED_HIDDEN), landing on the real next actions; the public demo is unchanged. +23 tests (1520/1520 pass), tsc/lint clean (0 errors), build clean. Doc: docs/FIX-BATCH-2026-07-23.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JYL5MSPhuqivN4m7ectp33
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
thecelestialmismatch
marked this pull request as ready for review
July 23, 2026 03:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes four founder-reported issues ("still no password reset · didn't like the after-login dashboard · buttons not working · pdf sucks, no my logo") plus a login-page redesign to match a supplied reference. Root-cause finding up front: this branch was identical to
origin/main(0 ahead/behind), so every prior "shipped" fix was already deployed — these are real bugs + observability, not stale deploys.drawBadge()drew a vector "HS" shield and never used the real logo. Now embeds the doberman-shield mark as a compiled-in base64 PNG (lib/reports/logo-data.ts, generated byscripts/gen-logo-data.pyfrompublic/logo.png, flattened to a white 256×256, ~35 KB) viadoc.addImage, seated in a white chip on the dark cover band.addImageis feature-detected with the vector shield kept as fallback (test mock still works). One change covers every report type (sample, compliance/audit, SSP/POA&M/Evidence, /demo snapshot).components/auth/AuthTabs.tsx, preserves?redirect=), "Work email" labels, a Terms of Service + Privacy Policy line, and the real Logo. No working auth removed — password, passwordless OTP/magic-link, email 2FA, GitHub + Google OAuth, and forgot-password all remain.reset_*diagnostics to/api/health(lib/auth/reset-diagnostics.ts:service_role,resend,app_urlflagging kaelus/localhost/preview,sender_domain) — shape/host only, never a secret — plus an honest "didn't get it?" line on the success screen. Remaining fixes are founder-side config (see doc), surfaced by the new diagnostics.SIGNED_IN_STRIPPED_HIDDEN), landing on the real next actions (Ask Brain AI + the activation checklist that ends on the PDF). The anonymous public demo is unchanged.Doc:
docs/FIX-BATCH-2026-07-23.md.Fixes # (no tracked issue — founder-reported)
Type of change
Jordan's Test Plan (CMMC Buyer — required before merge)
npm run buildpasses incompliance-firewall-agent/logo-data.tsis a base64 image;reset-diagnostics.tsis value-free (shape/host only, no-leak asserted in tests)proxy/patterns/Verification
/login+/signupbrowser-verified against a production build (mobile + desktop), 0 console errors; sample PDF renders the real logo (257 KB)npm run buildsucceeds locallynpm run lint) — 0 errors (40 pre-existing Next-16react-hookswarnings repo-wide, none introduced here)Checklist:
anytypes added in compliance-critical code🤖 Generated with Claude Code
https://claude.ai/code/session_01JYL5MSPhuqivN4m7ectp33
Generated by Claude Code