Skip to content

Always show the real publishable key on Go live; shorter install snippet - #105

Merged
engmsaleh merged 1 commit into
mainfrom
feature/publishable-key-always-shown
Oct 12, 2026
Merged

engmsaleh merged 1 commit into
mainfrom
feature/publishable-key-always-shown

Conversation

@engmsaleh

Copy link
Copy Markdown
Collaborator
  • Go live always fills in a real publishable key (creates one on first visit; dashboard_publishable_key_auto_created event). No more create-and-copy-before-it-vanishes step.
  • api_keys.display_key stores publishable values only; a CHECK constraint keeps secret keys out. Migration 0015 is already applied to production.
  • Keys page shows the full publishable key with Copy; secret keys stay masked with the one-time reveal.
  • Install snippets drop data-skilly-backend-url / data-skilly-core-url (the widget defaults them since Let the official embed script work without runtime URL attributes #104).
  • Home eyebrow uses the mono "Get live" style.

Verified locally: bun test 290/290, tsc --noEmit, next build.

🤖 Generated with Claude Code

…install snippets

Builders had to create a key and copy it before it vanished, and the
snippet showed a placeholder otherwise. Publishable keys are safe in the
browser, so store their display value (secret keys never get one, enforced
by a CHECK constraint) and fill the newest one into Go live, creating it
on first visit. The widget now defaults its backend and core URLs when
loaded from cdn.tryskilly.app, so the snippet no longer needs them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@engmsaleh
engmsaleh merged commit 86f6fc2 into main Oct 12, 2026
12 checks passed
@vercel

vercel Bot commented Oct 12, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
skilly-studio Ready Ready Preview Oct 12, 2026 5:17pm UTC

Request Review

This branch was successfully deployed

1 active deployment
Preview — 027ac98e Deployed Oct 12, 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