Expose CLI for egglog-experimental

[Alex-Fischman]

This PR exposes the command-line interface so that the egglog-experimental crate can reuse it, as well as significantly improving the existing source code.

[codspeed-hq]

CodSpeed Performance Report

Merging #510 will not alter performance

_{Comparing Alex-Fischman:cli (6e05fe9) with main (df92e00)}

Summary

✅ 10 untouched benchmarks

[Alex-Fischman]

I would review this PR commit-by-commit, they are all pretty small and well separated, and the last one is code motion (necessary to expose the cli function to egglog-experimental).

[yihozhang]

I have a small question and otherwise it looks good

[Alex-Fischman]

After talking to people offline I believe that the bin feature is adding unnecessary complexity, so I've removed it (and cleaned up Cargo.toml in the process). I will add it back if someone has a use case for it, but no one that I've talked to on the core team has one.

[yihozhang]

@DaniPopes we are thinking about simplifying our build dependencies and removing the feature flags that used to keep the build thin. Please take a look at this PR. Since you made #467, do you have any usage for having these feature flags?

[DaniPopes]

Yes, it's for excluding clap, graphviz, chrono... dependencies that are unused when importing egglog as a library. This brings the total dependencies imported from 133 down to 56.

[Alex-Fischman]

@DaniPopes Do you have a use case where having fewer dependencies matters?

For context, the egglog team's current goal is to stabilize this crate by June, and it doesn't seem like having Cargo features is worth the smaller build to me. That said, if someone depends on this, we can support it as a library feature.

[Alex-Fischman]

I think the ping failed (I edited it in), my bad. @DaniPopes

[DaniPopes]

Well it's not really a use case, but I would say it's pretty significant the fact that it's 3 times faster to build basically for free?

Having feature gates for the library, required to build the binary is a pretty common pattern for this reason; the features are enabled by default to allow "cargo install" to work without extra flags, and library users can opt-in to lower dependencies with default-features=false

[Alex-Fischman]

I've added back the features. I've cut the serde_json dependency as it is not used in the main crate.

[yihozhang]

Thanks!!