docs: Record the final JOSS manuscript proof. - #6537
Closed
franciscojavierarceo wants to merge 1 commit into
Closed
franciscojavierarceo wants to merge 1 commit into
franciscojavierarceo wants to merge 1 commit into
Conversation
Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
franciscojavierarceo
requested review from
leseb and
raghotham
as code owners
September 10, 2026 20:53
franciscojavierarceo
requested review from
bbrowning,
cdoern,
mattf and
skamenan7
as code owners
September 10, 2026 20:53
Collaborator
Author
|
superseded by #6548, which includes this proof link, hash, source snapshot, and page references alongside the new manuscript corrections. the README now makes clear that the 10 September proof predates those edits and needs regeneration after merge. closing this duplicate so the source pins and proof record can be reviewed together. |
mattf
pushed a commit
to mattf/llama-stack
that referenced
this pull request
Sep 15, 2026
the JOSS manuscript still describes Safety as an interchangeable provider layer, although Responses guardrails call an external moderation endpoint. this corrects the Summary and architecture claims and the matching stale API/configuration documentation, following the [14 September findings](openjournals/joss-reviews#11234 (comment)). the paper now explicitly describes the cited v1.0.3 release: 23 inference provider types and 13 vector provider types across 10 backends. the README links the exact registries and explains why reviewed main has 25 inference types and 14 vector types across 11 backends. Interactions is identified as alpha; Containers and Skills are omitted because they are absent from v1.0.3. the reference audit records checks of all 19 DOI-less entries. no additional DOI was verified for the exact cited works. the bibliography corrects the Oracle post's authors/year and the MLflow article's metadata and source URL. this also incorporates the proof record from ogx-ai#6537 and pins the revised manuscript pair. the recorded 10 September proof predates these edits; we should regenerate it after merge. validation: provider registries, API availability, and moderation configuration checked against both source snapshots; all 22 citation keys resolve to bibliography entries; changed-file pre-commit and `git diff --check` pass. under Node 22, `npm ci --no-audit --no-fund`, `npm run gen-api-docs all`, and `npm run build` pass in `docs`, including strict broken-link and anchor checks. an independent source review found no remaining introduced inaccuracies. runtime functionality was not retested for this paper/documentation change. --------- Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
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
1 participant
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.
the paper README still points to the proof generated before #6532 merged. this records the new proof, its SHA-256, and the corresponding manuscript snapshot after #6532.
both manuscript files at the existing pin are byte-identical to merge commit
016a6079594de71b58cba2e58842126fadda2ec1. all six rendered pages were inspected: the AI disclosure is on page 4, immutable OGX/operator references are on page 5, and the corrected SGLang author list and DOI are on page 6. changed-file pre-commit andgit diff --checkpass. this updates proof provenance only; the manuscript and bibliography are unchanged.