fix(local-inference): point at Metal nil-pipeline guard

[lalalune]

Summary

• update the local-inference llama.cpp submodule to fix(metal): guard nil compute pipelines llama.cpp#39
• the submodule guard replaces the observed NULL Metal pipeline dereference with an explicit nil Metal compute pipeline backend failure at both pipeline dereference points
• add issue evidence documenting the local checks and the A18 Pro device verification still required

Refs #11612.

Evidence

• PASS: git diff --check inside plugins/plugin-local-inference/native/llama.cpp
• PASS: parent git diff --check
• PASS: parent pointer reviewed with git diff --submodule=log -- plugins/plugin-local-inference/native/llama.cpp

Not captured here

• N/A: iPhone 16 Pro Max / A18 Pro runtime verification. This host is Linux and cannot build/run Apple Metal or reproduce the device-only crash.
• Required before closing On-device local LLM crash-loops on iPhone 16 Pro Max (A18 Pro): NULL Metal pipeline in ggml mul_mat during llama_decode #11612: build the app with fix(metal): guard nil compute pipelines llama.cpp#39, run the affected on-device local-generation path, and attach crash-free logs or the new explicit backend failure plus syslog/.ips artifacts.

Residual

This is the defensive stop-the-NULL-deref patch. It does not resolve why the A18 Pro mul_mat pipeline is nil; root-cause kernel selection/compilation still needs real-device Metal diagnostics.

[greptile-apps]

Your trial has ended. Reactivate Greptile to resume code reviews.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 298f6f48-164f-4bff-a5da-934abb99a4ce

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/11612-ios-metal-null-pipeline

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Your trial has ended. Reactivate Greptile to resume code reviews.

[lalalune]

Post-merge review note: I verified the llama.cpp submodule pointer against elizaOS/llama.cpp PR #39 and inspected the actual diff (2bdcef89..299d5b78): it is a 10-line defensive nil-pipeline guard at the two Metal pipeline dereference points, and the submodule diff itself is clean.

One parent-repo issue remained after merge: git diff --check origin/develop flagged whitespace in three evidence log files under .github/issue-evidence/11612-ios-metal-null-pipeline/mac-metal-regression/. I opened follow-up PR #11645 with only that evidence whitespace cleanup: #11645

[claude]

Claude encountered an error —— View job

---

I'll analyze this and get back to you.