Skip to content

Require explicit Smart CDN delivery config - #12

Merged
kvz merged 1 commit into
mainfrom
safe-smartcdn
Sep 1, 2026
Merged

kvz merged 1 commit into
mainfrom
safe-smartcdn

Conversation

@kvz

@kvz kvz commented Sep 1, 2026

Copy link
Copy Markdown
Member

Why

The public Next.js Smart CDN skill silently fell back to a fictitious demo/serve-preview/example.jpg contract. That could generate a correctly signed URL without proving that the application had selected a real, workspace-owned and origin-constrained delivery Template. It would also leave stale serve-preview guidance behind while API2 retires the arbitrary-origin Built-in.

What

  • require workspace, Template, and input alongside the server-only credentials
  • remove all serve-preview and serve-image references from the live Skills repository
  • document the fixed-input/origin-constrained security contract
  • make the reference scenario load all required local env values deterministically
  • assert the configured Smart CDN hostname and full encoded path in E2E

Verified

  • yarn check
  • focused Smart CDN scenario npm run test:e2e
  • focused scenario npm run build
  • skill-creator quick_validate.py
  • council review: no issues found
  • repository-wide scan: no remaining serve-preview or serve-image references

@kvz
kvz merged commit ac0e88e into main Sep 1, 2026
3 checks passed
@kvz
kvz deleted the safe-smartcdn branch September 1, 2026 10:36
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