Skip to content

sync upstream llama.cpp (b7179)#194

Merged
ngxson merged 2 commits intomasterfrom
xsn/sync27
Nov 27, 2025
Merged

sync upstream llama.cpp (b7179)#194
ngxson merged 2 commits intomasterfrom
xsn/sync27

Conversation

@ngxson
Copy link
Owner

@ngxson ngxson commented Nov 27, 2025

Summary by CodeRabbit

  • Chores
    • Bumped package to v2.3.7.
    • Updated embedded AI library reference and runtime version label.
    • Switched CDN Wasm URLs to point at the new release.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Nov 27, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Updates dependency pointers and version strings: advances the llama.cpp submodule commit, bumps package version, updates LIBLLAMA_VERSION, and changes CDN URLs for the wasm artifacts. No code-level API or control-flow changes.

Changes

Cohort / File(s) Summary
Submodule & generated constant
llama.cpp, src/workers-code/generated.ts
Bumped llama.cpp submodule pointer from commit 1d49ca37594f... to 4abef75f2cf2.... Updated exported LIBLLAMA_VERSION from 'b6694-1d49ca3' to 'b7179-4abef75'.
Package manifest
package.json
Bumped package version from 2.3.6 to 2.3.7.
WASM CDN URLs
src/wasm-from-cdn.ts
Updated CDN URLs for single-thread and multi-thread wllama.wasm from @wllama/wllama@2.3.6 to @wllama/wllama@2.3.7.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Review focus: ensure version strings and CDN URLs are consistent and no accidental edits to other code.
  • Files to spot-check: src/workers-code/generated.ts, src/wasm-from-cdn.ts, package.json.

Possibly related PRs

Poem

🐰
The commit hop is light and spry,
New hashes gleam beneath the sky,
Packages nudged and CDNs sing,
A tiny bump — a gentle spring,
I nibble versions, then I fly. 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'sync upstream llama.cpp (b7179)' accurately reflects the primary change: updating the llama.cpp submodule pointer and the corresponding LIBLLAMA_VERSION constant to the new commit b7179.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5e17fb0 and 4e5108f.

📒 Files selected for processing (2)
  • package.json (1 hunks)
  • src/wasm-from-cdn.ts (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

@ngxson ngxson merged commit 5e63e3b into master Nov 27, 2025
5 of 6 checks passed
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