feat: android support

[Ludea]

Add android support

[branchseer]

fspy needs fspy_preload_unix on macOS and glibc Linux

[Ludea]

Done

[branchseer]

This ipc channel is only used by LD_PRELOAD. Since we won't use that in android (same as musl), we don't need to fix it for android. You can completely skip compiling this module.

[Ludea]

https://github.com/voidzero-dev/vite-task/blob/main/crates/fspy_shared_unix/src/payload.rs#L12 ChannelConf is declared here https://github.com/voidzero-dev/vite-task/blob/main/crates/fspy_shared/src/ipc/channel/mod.rs#L18 without any cfg

[branchseer]

Good catch. I've gated them in #327 and #328. You can now try adding not(android) to the cfg and removing the code changes to the ipc module.

[Ludea]

Done, I excluded ipc module, like musl host

[branchseer]

Could you try adding not(target_os = "android") to

vite-task/crates/fspy_shared/src/ipc/mod.rs

Line 1 in 728413c

#[cfg(not(target_env = "musl"))]

? That way we won't need any changes under fspy_shared/src/ipc.

[Ludea]

Done

[branchseer]

Could you undo all the changes in ‎crates/fspy_shared/src/ipc/channel? Thanks.

They aren't necessary since the channel module is properly gated.

[Ludea]

Done, crates/fspy_shared/src/ipc/channel content (mod.rs and shm_io.rs) is the same as main branch

[branchseer]

crates/fspy_shared/src/ipc/channel/mod.rs still contains lots of code changes, and they cause the CI failures. Could you undo them all?

[Ludea]

Better?

[branchseer]

Looks like the macOS test failures come from the gate change in crates/fspy_preload_unix/src/lib.rs, where you narrowed unix down to linux:

-#[cfg(all(unix, not(target_env = "musl")))]
+#[cfg(all(target_os = "linux", not(target_env = "musl")))]
 mod client;  // same for interceptions, libc, macros

On macOS that makes the preload crate compile empty, so DYLD_INSERT_LIBRARIES still injects the dylib into Node but it hooks nothing, and the fs calls just pass through.

Probably want all(unix, not(target_os = "android"), not(target_env = "musl")) there.

[branchseer]

@copilot resolve the merge conflicts in this pull request

[branchseer]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 747e02555e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".