Skip to content

Feat/platform knowledge - #46

Merged
godronus merged 4 commits into
mainfrom
feat/platform-knowledge
Sep 24, 2026
Merged

godronus merged 4 commits into
mainfrom
feat/platform-knowledge

Conversation

@godronus

Copy link
Copy Markdown
Collaborator

Updating plugin with basic knowledge - things I have found whilst developing apps that agents have had to write probes to discover usage. This helps reduce other developers from having to complete the same probes.

fastedge-test, fastedge-sdk-js, fastedge-sdk-rust and proxy-wasm-sdk-as now
sync from their latest GitHub release instead of main/master, so reference
docs track what users can install and only regenerate on a new release.
fastedge-templates stays on main (no releases; portal deploys from main).

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Unresolved documentation correctness and Codex index/mirror issues remain.

Review effort: Lite
Findings: None

Resolved since last review (10)
Previously missed (1)

In code that hasn't changed since last review

Low severity Per-request expiry can extend rate-limit windows indefinitely

plugins/​gcore-fastedge/​skills/​fastedge-docs/​reference/​platform/​storage.md:70

This says to call expire after every incr, but that resets the relative TTL on every request; under continuous traffic the counter can then remain alive indefinitely. The existing rate-limit guidance correctly expires only when the window is first created (count === 1). Recommend winner-side expiry with a bounded retry rather than resetting the deadline for losing callers.

@godronus
godronus requested a review from qrdl September 24, 2026 08:53
@godronus
godronus merged commit 503a8fc into main Sep 24, 2026
6 checks passed
@godronus
godronus deleted the feat/platform-knowledge branch September 24, 2026 09:23
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