Merge pull request #2470 from CosmWasm/exports-feature

Add "exports" feature

Author: webmaster128

CHANGELOG.md

@@ -78,6 +78,8 @@ and this project adheres to
 - cosmwasm-std: Remove export of `ExternalApi`, `ExternalQuerier` and
   `ExternalStorage` as those are only needed by export implementations in
   cosmwasm-std. ([#2467])
+- cosmwasm-std: Add a new `exports` feature which needs to be enabled for the
+  primary cosmwasm_std dependency of a contract.
 
 ## Fixed
 

contracts/cyberpunk/Cargo.toml

@@ -24,6 +24,7 @@ overflow-checks = true
 cosmwasm-schema = { path = "../../packages/schema" }
 cosmwasm-std = { path = "../../packages/std", default-features = false, features = [
     "cosmwasm_1_3",
+    "exports",
     "std",
 ] }
 rust-argon2 = "2.1"

contracts/hackatom/Cargo.toml

@@ -26,6 +26,7 @@ overflow-checks = true
 cosmwasm-schema = { path = "../../packages/schema" }
 cosmwasm-std = { path = "../../packages/std", default-features = false, features = [
     "cosmwasm_2_2",
+    "exports",
     "std",
 ] }
 schemars = "0.8.12"

contracts/reflect/Cargo.toml

@@ -27,6 +27,7 @@ overflow-checks = true
 cosmwasm-schema = { path = "../../packages/schema" }
 cosmwasm-std = { path = "../../packages/std", default-features = false, features = [
     "cosmwasm_2_2",
+    "exports",
     "staking",
     "stargate",
     "std",

contracts/replier/Cargo.toml

@@ -26,6 +26,7 @@ overflow-checks = true
 cosmwasm-schema = { path = "../../packages/schema" }
 cosmwasm-std = { path = "../../packages/std", default-features = false, features = [
     "cosmwasm_1_4",
+    "exports",
     "iterator",
     "std",
 ] }

contracts/staking/Cargo.toml

@@ -25,6 +25,7 @@ overflow-checks = true
 [dependencies]
 cosmwasm-schema = { path = "../../packages/schema" }
 cosmwasm-std = { path = "../../packages/std", default-features = false, features = [
+    "exports",
     "staking",
     "std",
 ] }

docs/USING_COSMWASM_STD.md

@@ -33,16 +33,16 @@ in the dependency tree. Otherwise conflicting C exports are created.
 
 The library comes with the following features:
 
-| Feature      | Enabled by default | Description                                                                          |
-| ------------ | ------------------ | ------------------------------------------------------------------------------------ |
-| iterator     | x                  | Storage iterators                                                                    |
-| abort        | x                  | DEPRECATED A panic handler that aborts the contract execution with a helpful message |
-| stargate     |                    | Cosmos SDK 0.40+ features and IBC                                                    |
-| staking      |                    | Access to the staking module                                                         |
-| cosmwasm_1_1 |                    | Features that require CosmWasm 1.1+ on the chain                                     |
-| cosmwasm_1_2 |                    | Features that require CosmWasm 1.2+ on the chain                                     |
-| cosmwasm_1_3 |                    | Features that require CosmWasm 1.3+ on the chain                                     |
-| cosmwasm_1_4 |                    | Features that require CosmWasm 1.4+ on the chain                                     |
+| Feature      | Enabled by default | Description                                                                      |
+| ------------ | ------------------ | -------------------------------------------------------------------------------- |
+| exports      | x                  | Adds exports and imports needed for basic communication between contract and VM. |
+| iterator     | x                  | Storage iterators                                                                |
+| stargate     |                    | Cosmos SDK 0.40+ features and IBC                                                |
+| staking      |                    | Access to the staking module                                                     |
+| cosmwasm_1_1 |                    | Features that require CosmWasm 1.1+ on the chain                                 |
+| cosmwasm_1_2 |                    | Features that require CosmWasm 1.2+ on the chain                                 |
+| cosmwasm_1_3 |                    | Features that require CosmWasm 1.3+ on the chain                                 |
+| cosmwasm_1_4 |                    | Features that require CosmWasm 1.4+ on the chain                                 |
 
 ## The cosmwasm-std dependency for contract developers
 
@@ -78,13 +78,13 @@ might move certain existing functionality to that feature in the future.
 
 Also libraries should define a loose version range that allows the contract
 developer to control which cosmwasm-std version they want to use in the final
-project. E.g. if your library does not work with 1.0.0 due to a bug fixed in
-1.0.1, your min version is 1.0.1 and not the latest stable.
+project. E.g. if your library does not work with 3.0.0 due to a bug fixed in
+3.0.1, your min version is 3.0.1 and not the latest stable.
 
 A typical dependency then looks like this:
 
 ```toml
-# We really need `stargate` here as this is an IBC related library. `abort` and `iterator` are not needed.
+# We really need `stargate` here as this is an IBC related library. `exports` and `iterator` are not needed.
 # `std` should always stay enabled.
-cosmwasm-std = { version = "1.0.1", default-features = false, features = ["std", "stargate"] }
+cosmwasm-std = { version = "3.0.1", default-features = false, features = ["std", "stargate"] }
 ```

packages/std/Cargo.toml

@@ -12,7 +12,10 @@ readme = "README.md"
 features = ["cosmwasm_2_2", "staking", "stargate", "ibc2"]
 
 [features]
-default = ["iterator", "std"]
+default = ["exports", "iterator", "std"]
+# Enable if this cosmwasm-std is the primary version of cosmwas-std used in the contract. This
+# adds exports and imports needed for basic communication between contract and VM.
+exports = []
 std = []
 # iterator allows us to iterate over all DB items in a given range
 # optional as some merkle stores (like tries) don't support this

packages/std/src/exports.rs renamed to packages/std/src/exports/exports.rs

@@ -12,6 +12,9 @@ use core::{marker::PhantomData, ptr};
 
 use serde::de::DeserializeOwned;
 
+use super::imports::{ExternalApi, ExternalQuerier, ExternalStorage};
+use super::memory::{Owned, Region};
+use super::panic::install_panic_handler;
 use crate::deps::OwnedDeps;
 #[cfg(any(feature = "stargate", feature = "ibc2"))]
 use crate::ibc::IbcReceiveResponse;
@@ -24,9 +27,6 @@ use crate::ibc::{
 use crate::ibc::{IbcChannelOpenMsg, IbcChannelOpenResponse};
 #[cfg(feature = "ibc2")]
 use crate::ibc2::{Ibc2PacketReceiveMsg, Ibc2PacketTimeoutMsg};
-use crate::imports::{ExternalApi, ExternalQuerier, ExternalStorage};
-use crate::memory::{Owned, Region};
-use crate::panic::install_panic_handler;
 use crate::query::CustomQuery;
 use crate::results::{ContractResult, QueryResponse, Reply, Response};
 use crate::serde::{from_json, to_json_vec};

packages/std/src/imports.rs renamed to packages/std/src/exports/imports.rs

@@ -1,19 +1,18 @@
 use alloc::vec::Vec;
 use core::ptr;
 
+#[cfg(feature = "iterator")]
+use super::memory::get_optional_region_address;
+use super::memory::{Owned, Region};
 use crate::import_helpers::{from_high_half, from_low_half};
-use crate::memory::{Owned, Region};
+#[cfg(feature = "iterator")]
+use crate::iterator::{Order, Record};
 use crate::results::SystemResult;
 #[cfg(feature = "iterator")]
 use crate::sections::decode_sections2;
 use crate::sections::encode_sections;
 use crate::serde::from_json;
 use crate::traits::{Api, Querier, QuerierResult, Storage};
-#[cfg(feature = "iterator")]
-use crate::{
-    iterator::{Order, Record},
-    memory::get_optional_region_address,
-};
 use crate::{Addr, CanonicalAddr};
 #[cfg(feature = "cosmwasm_2_1")]
 use crate::{AggregationError, HashFunction, PairingEqualityError};