Skip to content

[cdn-caching] Describe PPR state by category, not raw proxy values - #132

Merged
Melkeydev merged 1 commit into
mainfrom
skills/ppr-state-categories
Aug 5, 2026
Merged

[cdn-caching] Describe PPR state by category, not raw proxy values#132
Melkeydev merged 1 commit into
mainfrom
skills/ppr-state-categories

Conversation

@shinapatel

Copy link
Copy Markdown
Contributor

What

Follow-up to #128. Updates the PPR state table in the cdn-caching skill to drop the raw ppr_state proxy values (page, shell, blocking) and describe the three states purely by their user-facing category:

ppr_state Meaning
Static Fully prerendered — no postponed state, so the function is not invoked for the body
Partial A static shell serves from cache + a postponed hole the function resumes per request
Dynamic The whole body is postponed and rendered by the function per request

The surrounding prose is updated to match (a Partial serve, not a shell hit).

Why

Users see PPR state categorized as (fully) static, partial, and (fully) dynamic. The raw page/shell/blocking values are internal proxy terms and don't belong in user-facing skill guidance.

Notes

  • Body-only change — the ppr_state field name, prompt signals, and retrieval entries from [cdn-caching] Add PPR state (ppr_state) to the skill #128 are unchanged, so the generated manifests are untouched.
  • bun test → 929 pass, 0 fail. bun run validate → passed (pre-existing workflow warning only).

🤖 Generated with Claude Code

Remove the internal ppr_state proxy values (page/shell/blocking) from the
PPR state table and present the three states by their user-facing category:
Static, Partial, and Dynamic.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Melkeydev
Melkeydev merged commit 3878c45 into main Aug 5, 2026
4 checks passed
@Melkeydev
Melkeydev deleted the skills/ppr-state-categories branch August 5, 2026 00:02
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