Repository navigation
docs: make the BYOT guide provider-neutral and SAS-facing - #48
Merged
Hou (SciencePotato) merged 5 commits intoOct 14, 2026
Conversation
Replace the adapter checklist with request, authentication, JWE, and response guidance. Leave outbound service integration customer-owned and align navigation and runtime binding notes with current source. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7075cdd0-122d-4ffa-9889-e23bc11453da
Follow the request in four plain-language steps, retain core payload and response fields, and link detailed setup and error guidance. Preserve existing anchors and clarify runtime binding notes. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7075cdd0-122d-4ffa-9889-e23bc11453da
Map provider hooks, registration, shared credentials and configuration, tests, and custom packaging for each runtime without vendor-specific examples. Keep shared SAS authentication, decryption, evaluation, and nonce responses unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7075cdd0-122d-4ffa-9889-e23bc11453da
siyixian
approved these changes
Oct 14, 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
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.
What changed
Replace the merged provider checklist with a provider-neutral SAS/BYOT guide that explains the request and response, then shows customers where to implement their own integration.
The guide follows the request path: verify the caller, read and decrypt the payload, evaluate or send, and respond to SAS. Core fields remain in short tables; detailed runtime differences and settings are linked from the contract and setup references.
The new implementation section gives customers concrete repository locations:
Customers own outbound mapping, authentication, and acceptance/failure interpretation. The shared handler keeps SAS authentication boundaries, envelope validation, decryption, evaluation, and nonce responses. The guide names no phone provider and includes no vendor API recipes or inline implementation samples.
The existing guide path is retained, with navigation updated in the main, technical, and runtime guides. Closely related contract corrections reflect actual runtime binding and response differences without changing implementation behavior.
Review and verification
Three earlier passes checked flow, source accuracy, and natural wording. The implementation-map follow-up was reread as “which file do I open, what do I add, and what comes next?” Exact provider hooks, registration locations, config readers, credential services, and test directories were checked against source.
The latest follow-up passes 44 local link/anchor checks, including all five inbound references; preserves all 13 preceding guide anchors; and renders four tables correctly. Required fields and implementation hooks are covered, with no named providers, code fences, or images in the guide. git diff --check passes.
Seven Markdown files in the overall PR; the latest follow-up changes only the BYOT guide. No runtime, infrastructure, catalog, Azure, or provider actions. Runtime tests were not run for documentation-only changes.
Reviewed head: 01973a4. Open for review; no merge requested.