Release 0.77.5 - #3775
Merged
Merged
Release 0.77.5#3775
Conversation
* adding toggle setting for retrieving resources results from payload * add constants for excluding items from payload * fetch directly from payload * ensure vector search does not prefetch opensearch results * make sure nested content file is retrieved * remove irrelevant field
…3772) * perf(metadata): skip the jsdom sanitize for markup-free descriptions standardizeMetadata calls htmlToPlainText on every server-rendered page, and on the server isomorphic-dompurify is jsdom, so every render allocated a real DOM fragment -- including for descriptions containing no markup at all. The homepage pays it for the constant "Learn with MIT". Return early when the input has neither a tag nor an entity to decode. The guard tests for `<` or `&` rather than `<` alone: "Tom & Jerry" has no tags but still needs decoding, and skipping that would silently leave the entity in the meta description. collapseWhitespace still runs on the fast path, so trimming behaviour is unchanged. This narrows the per-render allocation, not the module-level jsdom import, which is a one-time cost. Refs #3771 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XBP7CuQXmxLnPNjQvk4aCz * test(metadata): assert the fast path skips sanitize, not just its output The previous assertions could not fail if the fast path were removed: DOMPurify round-trips markup-free input unchanged, so "2 > 1" and friends produce the same string down either branch. Verified by deleting the guard -- all 11 output-level tests still passed. Spy on DOMPurify.sanitize instead and assert it does not run for markup-free input, which is the allocation this change exists to avoid. Also assert the inverse for tags and entities, so widening the guard until entities skip decoding fails loudly rather than silently regressing meta descriptions. With the guard removed, exactly the 4 new negative assertions fail. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XBP7CuQXmxLnPNjQvk4aCz --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
* add fix by constraining to percolate index * paginate results
OpenAPI ChangesNo changes detected Unexpected changes? Ensure your branch is up-to-date with |
odlbot
added
waiting for checkboxes
all checkboxes checked
and removed
deploying to rc
waiting for checkboxes
all checkboxes checked
labels
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
3 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.
Shankar Ambady
Tobias Macey
renovate[bot]