addressing nits

Author: seemantaggarwal

cumulus/polkadot-parachain/Cargo.toml

@@ -22,6 +22,7 @@ hex-literal = { workspace = true, default-features = true }
 log = { workspace = true, default-features = true }
 serde = { features = ["derive"], workspace = true, default-features = true }
 serde_json = { workspace = true, default-features = true }
+clap = { features = ["derive"], workspace = true }
 
 # Local
 asset-hub-rococo-runtime = { workspace = true, default-features = true }
@@ -51,7 +52,6 @@ sp-keyring = { workspace = true, default-features = true }
 xcm = { workspace = true, default-features = true }
 
 # Cumulus
-clap = { features = ["derive"], workspace = true }
 cumulus-primitives-core = { workspace = true, default-features = true }
 
 [build-dependencies]

cumulus/polkadot-parachain/src/main.rs

@@ -1,12 +1,12 @@
 // Copyright (C) Parity Technologies (UK) Ltd.
 // This file is part of Cumulus.
 // SPDX-License-Identifier: Apache-2.0
-//
+
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
-//      http://www.apache.org/licenses/LICENSE-2.0
+// http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ -29,7 +29,6 @@ use polkadot_omni_node_lib::{
 };
 use sc_cli::ExportChainSpecCmd;
 
-/// Custom CLI configuration for metadata.
 struct CliConfig;
 
 impl CliConfigT for CliConfig {

cumulus/test/service/src/cli.rs

@@ -82,7 +82,7 @@ impl Display for AuthoringPolicy {
 #[derive(Debug, clap::Subcommand)]
 pub enum Subcommand {
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]

polkadot/cli/src/cli.rs

@@ -25,7 +25,7 @@ use std::path::PathBuf;
 #[derive(Debug, Parser)]
 pub enum Subcommand {
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]

substrate/bin/node/cli/src/cli.rs

@@ -78,7 +78,7 @@ pub enum Subcommand {
 	Sign(sc_cli::SignCmd),
 
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]

substrate/client/cli/src/commands/export_chain_spec_cmd.rs

@@ -66,14 +66,3 @@ impl ExportChainSpecCmd {
 		Ok(())
 	}
 }
-impl CliConfiguration for ExportChainSpecCmd {
-	// If ExportChainSpecCmd doesn’t have shared_params, you must provide some implementation.
-	fn shared_params(&self) -> &SharedParams {
-		unimplemented!("ExportChainSpecCmd does not implement shared_params")
-	}
-
-	// Implement the chain_id method to return the chain identifier.
-	fn chain_id(&self, _is_dev: bool) -> Result<String> {
-		Ok(self.chain.clone())
-	}
-}

templates/minimal/node/src/cli.rs

@@ -59,7 +59,7 @@ pub enum Subcommand {
 	Key(sc_cli::KeySubcommand),
 
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]

templates/parachain/node/src/cli.rs

@@ -6,7 +6,7 @@ use std::path::PathBuf;
 #[derive(Debug, clap::Subcommand)]
 pub enum Subcommand {
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]

templates/solochain/node/src/cli.rs

@@ -15,7 +15,7 @@ pub enum Subcommand {
 	Key(sc_cli::KeySubcommand),
 
 	/// Build a chain specification.
-	/// #[deprecated( note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead")]
+	/// DEPRECATED: `build-spec` command will be removed after 1/04/2026. Use `export-chain-spec` command instead.
 	#[deprecated(
 		note = "build-spec command will be removed after 1/04/2026. Use export-chain-spec command instead"
 	)]