Skip to content

update docs#64

Merged
godronus merged 2 commits intomainfrom
fix/plugin-docs
Apr 14, 2026
Merged

update docs#64
godronus merged 2 commits intomainfrom
fix/plugin-docs

Conversation

@godronus
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates FastEdge Rust SDK documentation to reflect FastEdge CDN ProxyWasm execution semantics and to improve doc formatting/clarity for CDN app developers.

Changes:

  • Clarifies CDN ProxyWasm lifecycle semantics, emphasizing per-callback HttpContext instantiation (“Hook State Isolation”) and recommending set_property/get_property for cross-callback state.
  • Expands and reformats the documented set of FastEdge CDN request/response properties (including encoding notes for response.status).
  • Fixes Markdown table formatting in the docs index and quickstart.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
fastedge-plugin-source/.generation-config.md Updates the generation guidance to encode Hook State Isolation and a more complete properties list.
docs/quickstart.md Fixes feature flag table formatting.
docs/INDEX.md Fixes the documents index table formatting/alignment.
docs/CDN_APPS.md Updates CDN app lifecycle docs (per-callback contexts), adds Hook State Isolation section, and expands the Request Properties table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/CDN_APPS.md
Comment thread fastedge-plugin-source/.generation-config.md
@godronus godronus requested review from qrdl and ruslanti April 14, 2026 09:12
@godronus godronus merged commit fc1a3b2 into main Apr 14, 2026
3 checks passed
@godronus godronus deleted the fix/plugin-docs branch April 14, 2026 11:03
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.

3 participants