nsaspy/zara
1
0
Fork 0
Code Issues 95 Pull requests 18 Projects Releases Packages Wiki Activity Actions

RAGE #172: add Android Trealla JNI bridge boundary #256

Closed
nsaspy wants to merge 0 commits from rage/android-172-jni-bridge into master
pull from: rage/android-172-jni-bridge
merge into: nsaspy:master
Conversation 0 Commits - Files changed -
nsaspy commented 2026-09-04 22:43:51 +00:00
Owner
Copy link

Consumes the next bounded #172 slice from exact master 89e6fbc6f17df0a86083b5e66dd1496c89555daf after #283 landed the Trealla lifecycle contract.

Android-only scope:

  • concrete NativeTreallaBridge implementation of the existing TreallaBridge contract;
  • pinned zara_trealla native library loader, isolated behind an injectable JVM-testable boundary;
  • TreallaNativeApi JNI surface for init/evaluate/shutdown only;
  • fail-closed library-load/native-init/evaluation behavior;
  • preserves native semantic terms for the existing normalized comparator; no second parser/runtime authority.

TDD history:

  • RED 44427e416ce1613e8774ee42a059389f3191511d
  • GREEN b26fc07dc3b06c50c08fb14efce988092e938c48
  • evidence head bb82778cca98de8eb2c4d1220a64900b80bd7f77

This does not claim native Trealla fixture parity yet. The C/NDK implementation remains the next #172 slice.

Recovery note: draft #284 passed all exact-head CI on this identical SHA, but GitHub's ready-for-review mutation failed due to the connector's fullDatabaseId GraphQL incompatibility. This non-draft wrapper intentionally carries zero code changes; require fresh exact-head CI before merge.

Consumes the next bounded #172 slice from exact master `89e6fbc6f17df0a86083b5e66dd1496c89555daf` after #283 landed the Trealla lifecycle contract. Android-only scope: - concrete `NativeTreallaBridge` implementation of the existing `TreallaBridge` contract; - pinned `zara_trealla` native library loader, isolated behind an injectable JVM-testable boundary; - `TreallaNativeApi` JNI surface for init/evaluate/shutdown only; - fail-closed library-load/native-init/evaluation behavior; - preserves native semantic terms for the existing normalized comparator; no second parser/runtime authority. TDD history: - RED `44427e416ce1613e8774ee42a059389f3191511d` - GREEN `b26fc07dc3b06c50c08fb14efce988092e938c48` - evidence head `bb82778cca98de8eb2c4d1220a64900b80bd7f77` This does not claim native Trealla fixture parity yet. The C/NDK implementation remains the next #172 slice. Recovery note: draft #284 passed all exact-head CI on this identical SHA, but GitHub's ready-for-review mutation failed due to the connector's `fullDatabaseId` GraphQL incompatibility. This non-draft wrapper intentionally carries zero code changes; require fresh exact-head CI before merge.
nsaspy closed this pull request 2026-09-04 23:09:58 +00:00
Some checks failed
CI / test (pull_request) Failing after 5s
Details
CI / android skeleton gate (pull_request) Failing after 4s
Details
CI / shared mic / Arch Linux (pull_request) Failing after 3m11s
Details
CI / shared mic / Ubuntu 24.04 (pull_request) Has been cancelled
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something isn't working

  
documentation
Improvements or additions to documentation

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
invalid
This doesn't seem right

  
question
Further information is requested

  
wontfix
This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
nsaspy/zara!256
No description provided.