Skip to content

test: Verify HybridObject memory pressure - #1524

Draft
mrousavy wants to merge 1 commit into
mainfrom
test/hybrid-object-memory-pressure
Draft

test: Verify HybridObject memory pressure#1524
mrousavy wants to merge 1 commit into
mainfrom
test/hybrid-object-memory-pressure

Conversation

@mrousavy

Copy link
Copy Markdown
Owner

Summary

  • make C++, Kotlin, and Swift test HybridObjects report their native string allocation through memorySize
  • verify NitroModules.updateMemorySize() changes Hermes js_externalBytes by the exact reported amount
  • reuse one Hermes instrumentation helper for HybridObject and ArrayBuffer coverage

Validation

  • bun typecheck
  • bun --cwd example lint-ci
  • C++, Kotlin, and Swift format scripts
  • bun --cwd example build:android
  • focused Android/Hermes harness test: 2 passed across C++ and Kotlin HybridObjects

Local iOS compilation could not start because this worktree has no Pods installation. The PR iOS jobs will compile and run the shared test against the Swift implementation.

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
nitro-docs Skipped Skipped Aug 21, 2026 4:47pm

Request Review

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