Skip to content

Fix site audit security and UX issues#52

Draft
injoon5 wants to merge 2 commits into
mainfrom
cursor/site-audit-fixes-0667
Draft

Fix site audit security and UX issues#52
injoon5 wants to merge 2 commits into
mainfrom
cursor/site-audit-fixes-0667

Conversation

@injoon5
Copy link
Copy Markdown
Owner

@injoon5 injoon5 commented May 24, 2026

Summary

  • Gate Convex write/auth helper functions behind the server-held secret and keep admin bypass explicit.
  • Make user comment deletion soft-delete instead of hard-delete.
  • Sanitize /now markdown rendering and require admin auth in the Convex mutation.
  • Fix light-mode code highlighting, footer markup, mobile padding, nav/accessibility papercuts, and language toggle semantics.
  • Fix client hydration/router console warnings from reading-time animation, random comment placeholders, and analytics autocollect.

Testing

  • npm run build
  • npx prettier --check on touched files
  • npx eslint on touched files
  • npx convex deploy --preview-create="my-preview-deployment-name"
  • Browser click-through against local SvelteKit + Convex preview: home, blog post, code block, language toggle, like, comment create/edit controls, projects, /now.

Notes

  • Convex preview deployed to https://grand-opossum-228.convex.cloud and ADMIN_SECRET was set there.
  • Full npm run lint is still blocked by pre-existing unformatted files outside this change.
  • npm run check is still blocked by pre-existing Node typings and Convex Id typing issues.
Open in Web Open in Cursor 

Co-authored-by: Injoon Oh <injoon5@icloud.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 24, 2026

Deploying web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 871996a
Status:🚫  Build failed.

View logs

@vercel
Copy link
Copy Markdown

vercel Bot commented May 24, 2026

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

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment May 24, 2026 5:01am

Co-authored-by: Injoon Oh <injoon5@icloud.com>
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.

2 participants