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

RAGE #172: add Android native Trealla adapter contract #254

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

Recovery wrapper for #287 on the identical tested head after GitHub's draft→ready mutation failed on the fullDatabaseId GraphQL field.

Exact implementation head: f956574916dff85e8ca92ebdcf4f5d7725245daf.

This Android-only #172 slice adds the native Trealla JNI adapter plus reproducible pinned Android packaging:

  • fail-closed JNI init/evaluate/shutdown over one mutex-serialized Trealla engine;
  • bounded semantic answer collection with canonical Trealla term serialization and correct pl_free cleanup;
  • pinned NDK 29.0.14206865, CMake 3.22.1, Trealla commit b25ccfb8e485a697bb47f1947d6fb8e0ad4e6aaf, arm64-v8a + x86_64;
  • network-free android/build-trealla.sh cross-compile before Gradle APK assembly;
  • Android gate covers unit tests, native Trealla build, debug APK, and secret inspection.

The implementation head already passed CI run 33587363916 after recovery onto current master: Android gate, full repository tests + nix flake check + packaged Zara, and both shared mic jobs all succeeded. This wrapper still requires fresh exact-head PR CI before merge.

No Python/core/desktop implementation ownership is taken. RAGE evidence remains append-only under rage/172-android-native-adapter.org.

Recovery wrapper for #287 on the identical tested head after GitHub's draft→ready mutation failed on the `fullDatabaseId` GraphQL field. Exact implementation head: `f956574916dff85e8ca92ebdcf4f5d7725245daf`. This Android-only #172 slice adds the native Trealla JNI adapter plus reproducible pinned Android packaging: - fail-closed JNI init/evaluate/shutdown over one mutex-serialized Trealla engine; - bounded semantic answer collection with canonical Trealla term serialization and correct `pl_free` cleanup; - pinned NDK 29.0.14206865, CMake 3.22.1, Trealla commit `b25ccfb8e485a697bb47f1947d6fb8e0ad4e6aaf`, arm64-v8a + x86_64; - network-free `android/build-trealla.sh` cross-compile before Gradle APK assembly; - Android gate covers unit tests, native Trealla build, debug APK, and secret inspection. The implementation head already passed CI run 33587363916 after recovery onto current master: Android gate, full repository tests + nix flake check + packaged Zara, and both shared mic jobs all succeeded. This wrapper still requires fresh exact-head PR CI before merge. No Python/core/desktop implementation ownership is taken. RAGE evidence remains append-only under `rage/172-android-native-adapter.org`.
nsaspy closed this pull request 2026-09-04 23:10:02 +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 3m54s
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!254
No description provided.