Skip to content

Drop JSON FFI boundary APIs#456

Merged
DorianZheng merged 1 commit into
mainfrom
drop-ffi-json-api
May 1, 2026
Merged

Drop JSON FFI boundary APIs#456
DorianZheng merged 1 commit into
mainfrom
drop-ffi-json-api

Conversation

@DorianZheng

Copy link
Copy Markdown
Member

Removes the legacy JSON FFI boundary in favor of typed C/Go APIs.

Also fixes OCI layer extraction for container-root absolute symlinks so Alpine command execution can resolve /bin/sh while preserving host escape protections.

Validation:

  • cargo fmt --check
  • BOXLITE_DEPS_STUB=1 cargo clippy -p boxlite --tests -- -D warnings
  • BOXLITE_DEPS_STUB=1 cargo clippy -p boxlite-ffi -p boxlite-c --tests -- -D warnings
  • cargo build -p boxlite-c
  • ctest --test-dir sdks/c/tests/build --verbose --output-on-failure -R test_execute_cmd
  • ctest --test-dir sdks/c/tests/build --verbose --output-on-failure -R test_basic
  • make lint:c
  • go test -tags boxlite_dev -run 'Test(Error|Is|BoxOptions|RuntimeOptions|BuildCOptions|StateConstants|With)' .\n- go test ./cmd/setup

@DorianZheng DorianZheng force-pushed the drop-ffi-json-api branch 4 times, most recently from 1dd8b76 to d25f37a Compare May 1, 2026 11:36
@DorianZheng DorianZheng force-pushed the drop-ffi-json-api branch from d25f37a to f2e09cb Compare May 1, 2026 14:13
@DorianZheng DorianZheng merged commit ba4ac12 into main May 1, 2026
32 checks passed
@DorianZheng DorianZheng deleted the drop-ffi-json-api branch May 1, 2026 14:20
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