All three cargo steps now run at `serve,e2e-tests`. They previously used
three different sets, so the lib and the ~144 crates `serve` adds compiled
once per step. clippy gains `--all-targets` so `cargo test` reuses the test
targets instead of rebuilding them.
The test run is split by thread budget rather than feature set, so neither
half recompiles: the lib suite spawns no tmux and gets 8 threads, the
integration and e2e binaries keep the cap at 3.
git2 drops `vendored-openssl`, its only non-default feature. git2's
`default` is empty, so `https`/`ssh` were never on and openssl-sys was in
the tree solely to be vendored — building OpenSSL from C source on every
cold cache. Nothing reaches a remote through libgit2: src/git/remote.rs
reads origin's URL, and `Repository::clone` appears only in tests cloning a
tempdir over the built-in local transport. flake.nix loses `perl` with it.
Linting `serve` for the first time surfaced 16 pre-existing violations,
fixed here. The `result_large_err` family is allowed at the `server::api`
module root: those handlers return `Response` in the `Err` arm, and boxing
it would put an allocation on every error path.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Miguel Palhas
c93812b05b