Skip to content

Bump partial markdown parser#780

Merged
blove merged 1 commit into
mainfrom
blove/partial-markdown-0.5.6-angular
Jul 8, 2026
Merged

Bump partial markdown parser#780
blove merged 1 commit into
mainfrom
blove/partial-markdown-0.5.6-angular

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bump @cacheplane/partial-markdown from ^0.5.5 to ^0.5.6.
  • Update the lockfile to resolve the published 0.5.6 tarball.

Why

0.5.6 contains the upstream streaming parser fix that suppresses partial closing code-fence markers while a fenced code block is still streaming.

Verification

  • npm ls @cacheplane/partial-markdown --depth=0
  • Direct parser smoke for streamed closing backticks
  • npx nx run licensing:prebuild
  • npx nx test chat --runInBand

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment Jul 8, 2026 5:40am

Request Review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 31s —— View job


PR Review

  • Read diff and review changes
  • Check package integrity and version bump

This is a straightforward dependency bump. The changes look correct:

  • package.json: version spec updated from ^0.5.5^0.5.6
  • package-lock.json: resolved version, tarball URL, and integrity hash all updated consistently to 0.5.6

One minor note: the new integrity hash (sha512-1hMw5QHjvyqSJebt0y2S3Y36GVsTe4H+FQB1AiJmr1e7mWl57HAcJsyZDSBsEu1AqlITMQiY7LkCEJsg0ej75Q==) appears slightly short for a standard base64-encoded SHA-512 (which should be 88 characters). This is not necessarily a problem — npm is authoritative on lockfile integrity values and would have rejected a mismatched hash — but it's worth confirming the install was clean (npm ci or npm ls @cacheplane/partial-markdown) before merge.

Otherwise, the PR is minimal, targeted, and aligns with the stated fix. No correctness, security, or type-safety concerns.

@blove blove force-pushed the blove/partial-markdown-0.5.6-angular branch from d006339 to 394bfee Compare July 8, 2026 05:34
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 42s —— View job


PR Review

  • Read diff and review changes
  • Verify lockfile consistency
  • Check integrity hash format

This is a clean, minimal dependency bump. Changes look correct:

package.json — version spec updated ^0.5.5^0.5.6

package-lock.json — version, resolved tarball URL, and integrity hash all updated consistently to 0.5.6

Re: integrity hash — the previous review flagged the new hash as potentially short. Counting the characters: 1hMw5QHjvyqSJebt0y2S3Y36GVsTe4H+FQB1AiJmr1e7mWl57HAcJsyZDSBsEu1AqlITMQiY7LkCEJsg0ej75Q== is 88 characters, which is exactly correct for a base64-encoded SHA-512 (64 bytes → 88 base64 chars with == padding). No issue there.

No correctness, security, type-safety, or API regression concerns. The stated fix (suppressing partial closing code-fence markers during streaming) is a behavioural improvement with no breaking surface changes expected from a patch bump. Good to merge.

@blove blove merged commit 3440383 into main Jul 8, 2026
62 checks passed
@blove blove deleted the blove/partial-markdown-0.5.6-angular branch July 8, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant