Skip to content

Bump the native pin to include renderer resource forwarding - #48

Merged
SunkenInTime merged 1 commit into
masterfrom
agent/native-pin-resource-forwarding
Jul 31, 2026
Merged

Bump the native pin to include renderer resource forwarding#48
SunkenInTime merged 1 commit into
masterfrom
agent/native-pin-resource-forwarding

Conversation

@SunkenInTime

@SunkenInTime SunkenInTime commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Completes #47 — that PR merged at the commit before this pin bump landed on its branch, so master currently ships the shared-renderer cutover without SunkenInTime/native#26 (registered image + font forwarding). Until this merges, image- and font-carrying widgets (Noro) render device-less with blank art and system-font text.

One commit, cherry-picked from the #47 branch: submodule pin → native weaver-main 7368f7f5 (the #26 merge), release-audit ratchet updated, audit passing locally. The receipts for what the pin carries are in #26: Noro art + Cozette verified live under the cutover, byte-identical across a host crash + replay, 43.1 MB device-less with zero arena regions.

Summary by CodeRabbit

  • Chores
    • Updated the bundled native SDK to a newer revision.
    • Updated release validation to recognize the latest native SDK version.

The cutover is only honest with it: registered images and fonts ride
the renderer channel, so image- and font-carrying widgets (Noro) render
identically device-less — verified live, including crash-replay.
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f256081a-f6e9-40c8-b01e-6f197480167a

📥 Commits

Reviewing files that changed from the base of the PR and between a59b0c6 and c2a821f.

📒 Files selected for processing (2)
  • runtime/native-sdk
  • scripts/release-audit.mjs

📝 Walkthrough

Walkthrough

The native SDK submodule now points to commit 7368f7f59e09c372311e884190d88006a73ce97f. The release audit expects and validates this commit.

Changes

Native SDK update

Layer / File(s) Summary
Update SDK pin and audit validation
runtime/native-sdk, scripts/release-audit.mjs
The submodule reference and release audit expected commit changed to 7368f7f59e09c372311e884190d88006a73ce97f.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the native pin update and its purpose: renderer resource forwarding.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/native-pin-resource-forwarding

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

@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the native SDK submodule to revision 7368f7f59e09c372311e884190d88006a73ce97f, including renderer image and font resource forwarding, and updates the release audit to require that same revision.

The native revision alignment was verified before and after this update: the release audit passed, and its expected revision matches both the committed submodule reference and the checked-out native SDK revision. No defects were found.

Confidence Score: 5/5

T-Rex T-Rex Logs

What T-Rex did

  • Ran the release audit on both the parent revision and the updated revision; the parent audit passed with native SDK 66783435, the updated audit passed with native SDK 7368f7f5, and the updated audit constant, committed submodule gitlink, and checked-out native SDK HEAD were identical.
  • Validated that the pre-update and post-update audits align as expected, with the post-update PR audit showing the expected hash, HEAD gitlink, and checked-out submodule HEAD exactly matching, and noted no code-evident defect in the changed gitlink/audit-pin configuration.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "Bump the native pin to weaver-main with ..." | Re-trigger Greptile

@SunkenInTime
SunkenInTime merged commit 705f89d into master Jul 31, 2026
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