shared: Make option parsing and optional feature

This is a breaking change for anyone using default-features = false and option parsing.

Author: evestera

ci/script.sh

@@ -7,6 +7,7 @@ DIR="$(cd "$(dirname $0)/.." && pwd)"
 cd "$DIR/json_typegen_shared"
 cargo build --target "$TARGET" --verbose
 cargo test --target "$TARGET" --verbose
+cargo build --target "$TARGET" --no-default-features --verbose
 
 cd "$DIR/json_typegen_cli"
 cargo build --target "$TARGET" --verbose

json_typegen_shared/Cargo.toml

@@ -12,9 +12,10 @@ edition = "2018"
 
 [features]
 unstable = []
-default = ["remote-samples", "local-samples"]
+default = ["remote-samples", "local-samples", "option-parsing"]
 remote-samples = ["reqwest"]
 local-samples = []
+option-parsing = ["syn", "synom"]
 
 [dependencies]
 serde = "1.0"
@@ -26,8 +27,8 @@ lazy_static = "1.2"
 linked-hash-map = "0.5.3"
 Inflector = "0.11"
 regex = "1.1"
-syn = { version = "0.11", features = ["full", "parsing"] }
-synom = "0.11.3"
+syn = { version = "0.11", features = ["full", "parsing"], optional = true }
+synom = { version = "0.11.3", optional = true }
 unindent = "0.1.2"
 
 [dev-dependencies]

json_typegen_shared/README.md

@@ -11,3 +11,27 @@ If you want to use this crate directly, be prepared for breaking changes to happ
 to let me know what you are using. (Breaking changes may still happen,
 but then I'll at least try to keep your use-case in mind if possible.
 This has happened enough by now that there are parts I already consider public API.)
+
+## Crate feature flags
+
+All of these flags are on by default,
+but you can avoid dependencies you don't care about disabling some or all of them,
+with e.g. something like this to only enable option parsing:
+
+```
+json_typegen_shared = { version = "*", default-features = false, features = ["option-parsing"] }
+```
+
+### `remote-samples`
+
+Required to load samples from URLs.
+
+### `local-samples`
+
+Required to load samples from local paths.
+
+### `option-parsing`
+
+Required to parse an options object from a string.
+Since this is required for code generation from macro-like strings,
+this is also required for the functions `codegen_from_macro` and `codegen_from_macro_input`.

json_typegen_shared/src/generation/serde_case.rs

@@ -2,7 +2,7 @@
 //! case of the source (e.g. `my-field`, `MY_FIELD`).
 //!
 //! Manually vendored from serde_derive/internals
-//! https://github.com/serde-rs/serde/blob/4eb580790dd6c96089b92942a5f481b21df4feaf/serde_derive/src/internals/case.rs
+//! <https://github.com/serde-rs/serde/blob/4eb580790dd6c96089b92942a5f481b21df4feaf/serde_derive/src/internals/case.rs>
 #![allow(dead_code)]
 
 // See https://users.rust-lang.org/t/psa-dealing-with-warning-unused-import-std-ascii-asciiext-in-today-s-nightly/13726

json_typegen_shared/src/lib.rs

@@ -24,6 +24,7 @@ mod generation;
 mod hints;
 mod inference;
 mod options;
+#[cfg(feature = "option-parsing")]
 pub mod parse;
 mod shape;
 mod unwrap;
@@ -68,6 +69,7 @@ enum SampleSource<'a> {
     Text(&'a str),
 }
 
+#[cfg(feature = "option-parsing")]
 /// Generate code from a `json_typegen` macro invocation
 pub fn codegen_from_macro(input: &str) -> Result<String, JTError> {
     let macro_input = parse::full_macro(input)?;
@@ -79,6 +81,7 @@ pub fn codegen_from_macro(input: &str) -> Result<String, JTError> {
     )
 }
 
+#[cfg(feature = "option-parsing")]
 /// Generate code from the arguments to a `json_typegen` macro invocation
 pub fn codegen_from_macro_input(input: &str) -> Result<String, JTError> {
     let macro_input = parse::macro_input(input)?;

json_typegen_shared/src/options.rs

@@ -41,6 +41,7 @@ impl Default for Options {
     }
 }
 
+#[cfg(feature = "option-parsing")]
 impl Options {
     pub(crate) fn macro_default() -> Options {
         let mut options = Options::default();

json_typegen_shared/src/parse.rs

@@ -1,4 +1,6 @@
 //! Functions for parsing `json_typegen` macro invocations and their arguments
+//!
+//! Requires the "option-parsing" feature
 
 use syn;
 use syn::parse::{boolean, ident, string};

json_typegen_wasm/Cargo.toml

@@ -14,7 +14,7 @@ crate-type = ["cdylib"]
 [dependencies]
 cfg-if = "0.1.5"
 wasm-bindgen = "0.2.71"
-json_typegen_shared = { path = "../json_typegen_shared", default-features = false, features = [] }
+json_typegen_shared = { path = "../json_typegen_shared", default-features = false, features = ["option-parsing"] }
 
 # The `console_error_panic_hook` crate provides better debugging of panics by
 # logging them with `console.error`. This is great for development, but requires