[services] Add Vercel Services skill - #134
Merged
Merged
Conversation
ricardo-agz
marked this pull request as draft
August 6, 2026 03:08
Lead with when Services is the right call (tightly coupled frontend + backend in one project) and its concrete benefits: skew protection, synced previews, atomic deploys and rollbacks, private bindings. The separate-projects escape hatch is now about independent deploy cadence. Drop framework pinning from examples, add /api namespace-split guidance (SPA vs frameworks with their own API routes), prefix stripping via service-scoped rewrite, SPA catch-all to /index.html, and a subdomain section with the preview-environment caveat. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The proxy only reads destination.service when dispatching into a service; the path field is compiled and validated but never consumed at request time. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
No top-level rewrite means a service is unreachable from the public internet and only accessible through bindings; adding a rewrite makes it public regardless of bindings. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ricardo-agz
marked this pull request as ready for review
August 7, 2026 01:00
Melkeydev
reviewed
Aug 7, 2026
Review feedback: the frontend term made the group miss backend-led phrasings. The wider [backend, vercel] group can reach the threshold with anyOf hits, which we accept for now. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Intents: one line per job the skill teaches, anchored by domain vocabulary (services, binding, rewrites, vercel.json) rather than product-name prefixes, and covering the private-by-default, subdomain, and prefix-stripping content added in this PR. Examples: verbatim-style prompts carrying stack names (FastAPI, Vite, Express, Go) and symptom phrasings, which is what the lowest-weight retrieval field is for. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Removed 'frontend' from the 'anyOf' array in skill manifest.
The manifest is generated from SKILL.md, so removing the term only from generated/skill-manifest.json left the two out of sync and failed the manifest check. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
dnwpark
approved these changes
Aug 7, 2026
Melkeydev
approved these changes
Aug 7, 2026
Member
|
@Melkeydev the hip shooter of vercel.. my main feedback -- too many words, I'd do a pass for brevity. |
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
4 participants
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.
Adds a
vercel-servicesskill for building multi-service applications on Vercel withservices