Inkeep OSS Sync#621
Merged
Merged
Conversation
… (#2625) * fix(open-knowledge): preserve agent byte-forms through same-block WYSIWYG edits Blockquote lazy continuations round-trip via a 'lazy' marker-spacing capture with a block-start replay guard; tight ATX heading-paragraph adjacency reuses the setext sourceContiguousNext capture; Observer A's map-driven splice recurses into a changed blockquote, list, or list item so untouched sibling children keep their source bytes. Beyond-tolerance test fixtures that pinned the old lazy canonicalization move to still canonicalizing constructs (NBSP, nested-blockquote lazy, intra-item blank runs). * test(open-knowledge): record fuzz and stress residual evidence for the splice change Deep fuzz at 1000 seeds measured rate 0.012 (12 failing seeds, all in the known convergence-budget residual class; historical band is 0.010-0.026). Stress passed on retry under normal machine load; the first stress append records a 120s test timeout from a heavily loaded host (30s scenario ran 161s wall), kept for the trend record. * review: harden lazy-replay guard and pin the narrowing drop-path Bare ATX openers (1-6 hashes with no text) now classify as lazy-unsafe per CommonMark 4.2, a clarifying comment makes the blank-line invariant in the lazy capture local, and a new unit test drives the splice narrowing bail on a container-data-only bullet-marker flip. * review: pin the lazy-replay guard branches and close two robustness gaps Unit tests drive each unsafe branch of the lazy guard through stale-attrs simulation; the replay now also refuses 'lazy' on the first content line (unreachable from capture, reachable from foreign attrs). Adjacency pins cover blockquote and fenced-code siblings, the child-splice walk is wrapped in the onFallback contract, and the narrowable-type set documents its admission criteria. * review: align sourceContiguousNext docs, widen guard pins, fix covers tags The canonical mdast-augmentation JSDoc now matches the ATX-inclusive behavior, the lazy-guard suite pins the nested-blockquote and fence branches, and the integration tests declare the constructs they actually cover. GitOrigin-RevId: 56ca4fa6174133eccbbbf2b8038337b66e0894f0
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.
Copybara-translated commits for Inkeep OSS Sync. Rebase-merge this PR so the prepared commits land directly on public main.