RDoc-4091 Agentic Documentation & RDoc-4100 - #2660
Open
poissoncorp wants to merge 2 commits into
Open
Conversation
Serve the RavenDB agent skill from static/skills/ravendb/ at /skills/ravendb/, verbatim, outside the version prefix. No plugin, no build step. handle_redirects.js gains `md` in staticAssetRegex; without it a versionless .md URL 301s to /7.2/<path>.md and every skill URL 404s. The tree is kept as published because SKILL.md navigates it with relative links.
The viewer-request function and the CSP live in this repo but never travelled with the S3 upload, so both needed a manual publish that fails silently when skipped: the site deploys cleanly while the edge keeps behaving like the previous release. deploy.ps1 phase 3 now pushes them alongside the KVS table. The CSP becomes scripts/lib/csp-policy.js, shaped like version-policy.js: a directive map with a comment per host naming the tag that needs it, plus a render function, runnable directly to print the header. It renders byte-identical to what production serves, so prod is a no-op; staging is on an April CSP and gets brought up to date. Both AWS APIs replace the whole config, so both scripts re-send it verbatim. For the CSP the patch is also proved to touch one field before writing and read back afterwards, since that policy carries CORS and a one-year HSTS besides.
redknightlois
approved these changes
Sep 10, 2026
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
2 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.
Issue link
https://issues.hibernatingrhinos.com/issue/RDoc-4091
https://issues.hibernatingrhinos.com/issue/RDoc-4100
Additional description
Two commits, one per ticket.
RDoc-4091 serves the agent skill from
static/skills/ravendb/at/skills/ravendb/, verbatim, no plugin. NeedsmdinstaticAssetRegexor every skill URL 404s.RDoc-4100 makes deploy phase 3 publish the edge function and the CSP. Both needed a manual step that fails silently when skipped: the site deploys green while the edge stays old. The CSP moves from an uncommitted JS generator to
scripts/lib/csp-policy.js.Type of change
/templatesor readme)Changes in docs URLs
/scripts/redirects.jsonfile, setDocuments MovedPR label)Changes in UX/UI