feat(rspack_plugin_copy): Set source_filename and copied flag when co…

…pying an asset (#8077)

crates/node_binding/binding.d.ts

@@ -397,6 +397,8 @@ export interface JsAssetInfo {
    */
   contenthash: Array<string>
   sourceFilename?: string
+  /** when asset was created from a source file (potentially transformed), it should be flagged as copied */
+  copied?: boolean
   /**
    * size in bytes, only set after asset has been emitted
    * when asset is only used for development and doesn't count towards user-facing assets
@@ -802,6 +804,7 @@ export interface JsStatsAssetInfo {
   development?: boolean
   hotModuleReplacement?: boolean
   sourceFilename?: string
+  copied?: boolean
   immutable?: boolean
   javascriptModule?: boolean
   chunkhash: Array<string>

crates/rspack_binding_values/src/asset.rs

@@ -29,6 +29,8 @@ pub struct JsAssetInfo {
   pub contenthash: Vec<String>,
   // when asset was created from a source file (potentially transformed), the original filename relative to compilation context
   pub source_filename: Option<String>,
+  /// when asset was created from a source file (potentially transformed), it should be flagged as copied
+  pub copied: Option<bool>,
   /// size in bytes, only set after asset has been emitted
   // pub size: f64,
   /// when asset is only used for development and doesn't count towards user-facing assets
@@ -63,6 +65,7 @@ impl From<JsAssetInfo> for rspack_core::AssetInfo {
       content_hash: i.contenthash.into_iter().collect(),
       version: String::from(""),
       source_filename: i.source_filename,
+      copied: i.copied,
       javascript_module: i.javascript_module,
       css_unused_idents: i.css_unused_idents.map(|i| i.into_iter().collect()),
       extras: i.extras,
@@ -97,6 +100,7 @@ impl From<rspack_core::AssetInfo> for JsAssetInfo {
       fullhash: info.full_hash.into_iter().collect(),
       contenthash: info.content_hash.into_iter().collect(),
       source_filename: info.source_filename,
+      copied: info.copied,
       javascript_module: info.javascript_module,
       css_unused_idents: info.css_unused_idents.map(|i| i.into_iter().collect()),
       extras: info.extras,

crates/rspack_binding_values/src/stats.rs

@@ -413,6 +413,7 @@ pub struct JsStatsAssetInfo {
   pub development: Option<bool>,
   pub hot_module_replacement: Option<bool>,
   pub source_filename: Option<String>,
+  pub copied: Option<bool>,
   pub immutable: Option<bool>,
   pub javascript_module: Option<bool>,
   pub chunkhash: Vec<String>,
@@ -438,6 +439,7 @@ impl From<rspack_core::StatsAssetInfo> for JsStatsAssetInfo {
       development: stats.development,
       hot_module_replacement: stats.hot_module_replacement,
       source_filename: stats.source_filename,
+      copied: stats.copied,
       immutable: stats.immutable,
       javascript_module: stats.javascript_module,
       chunkhash: stats.chunk_hash,

crates/rspack_core/src/compiler/compilation.rs

@@ -1895,6 +1895,8 @@ pub struct AssetInfo {
   pub content_hash: HashSet<String>,
   /// when asset was created from a source file (potentially transformed), the original filename relative to compilation context
   pub source_filename: Option<String>,
+  /// when asset was created from a source file (potentially transformed), it should be flagged as copied
+  pub copied: Option<bool>,
   /// size in bytes, only set after asset has been emitted
   // pub size: f64,
   /// when asset is only used for development and doesn't count towards user-facing assets

crates/rspack_core/src/stats/mod.rs

@@ -101,6 +101,7 @@ impl Stats<'_> {
                 development: asset.info.development,
                 hot_module_replacement: asset.info.hot_module_replacement,
                 source_filename: asset.info.source_filename.clone(),
+                copied: asset.info.copied,
                 is_over_size_limit: asset.info.is_over_size_limit,
               },
               emitted: self.compilation.emitted_assets.contains(name),

crates/rspack_core/src/stats/struct.rs

@@ -114,6 +114,7 @@ pub struct StatsAssetInfo {
   pub development: Option<bool>,
   pub hot_module_replacement: Option<bool>,
   pub source_filename: Option<String>,
+  pub copied: Option<bool>,
   pub immutable: Option<bool>,
   pub javascript_module: Option<bool>,
   pub chunk_hash: Vec<String>,

crates/rspack_plugin_copy/src/lib.rs

@@ -631,9 +631,15 @@ async fn process_assets(&self, compilation: &mut Compilation) -> Result<()> {
       if let Some(info) = result.info {
         set_info(&mut exist_asset.info, info);
       }
-      // TODO set info { copied: true, sourceFilename }
+      exist_asset.info.source_filename = Some(result.source_filename.to_string());
+      exist_asset.info.copied = Some(true);
     } else {
-      let mut asset_info = Default::default();
+      let mut asset_info = AssetInfo {
+        source_filename: Some(result.source_filename.to_string()),
+        copied: Some(true),
+        ..Default::default()
+      };
+
       if let Some(info) = result.info {
         set_info(&mut asset_info, info);
       }

packages/rspack/etc/api.md

@@ -3494,6 +3494,7 @@ type KnownAssetInfo = {
     chunkhash?: string | string[];
     contenthash?: string | string[];
     sourceFilename?: string;
+    copied?: boolean;
     size?: number;
     development?: boolean;
     hotModuleReplacement?: boolean;

packages/rspack/src/stats/DefaultStatsPrinterPlugin.ts

@@ -312,6 +312,8 @@ const SIMPLE_PRINTERS: Record<
 						: `from: ${sourceFilename}`
 				)
 			: undefined,
+	"asset.info.copied": (copied, { green, formatFlag }) =>
+		copied ? green(formatFlag("copied")) : undefined,
 	"asset.info.development": (development, { green, formatFlag }) =>
 		development ? green(formatFlag("dev")) : undefined,
 	"asset.info.hotModuleReplacement": (
@@ -779,6 +781,7 @@ const PREFERRED_ORDERS: Record<string, string[]> = {
 	"asset.info": [
 		"immutable",
 		"sourceFilename",
+		"copied",
 		"javascriptModule",
 		"development",
 		"hotModuleReplacement"

packages/rspack/src/stats/statsFactoryUtils.ts

@@ -63,6 +63,7 @@ export type KnownAssetInfo = {
 	// modulehash?: string | string[];
 	contenthash?: string | string[];
 	sourceFilename?: string;
+	copied?: boolean;
 	size?: number;
 	development?: boolean;
 	hotModuleReplacement?: boolean;

packages/rspack/src/util/AssetInfo.ts

@@ -16,6 +16,7 @@ class JsAssetInfo {
 			contenthash,
 			javascriptModule,
 			sourceFilename,
+			copied,
 			extras
 		} = jsAssetInfo;
 		return {
@@ -29,7 +30,8 @@ class JsAssetInfo {
 			chunkhash,
 			contenthash,
 			javascriptModule,
-			sourceFilename
+			sourceFilename,
+			copied
 		};
 	}
 
@@ -45,6 +47,7 @@ class JsAssetInfo {
 			contenthash = [],
 			javascriptModule,
 			sourceFilename,
+			copied,
 			...extras
 		} = assetInfo;
 		extras = extras ?? {};
@@ -59,7 +62,8 @@ class JsAssetInfo {
 			contenthash,
 			extras,
 			javascriptModule,
-			sourceFilename
+			sourceFilename,
+			copied
 		};
 	}
 }

tests/plugin-test/copy-plugin/CopyPlugin.test.js

@@ -146,7 +146,7 @@ describe("CopyPlugin", () => {
 		});
 
 		it("should copy a file to a new file", done => {
-			runEmit({
+			run({
 				expectedAssetKeys: ["newfile.txt"],
 				patterns: [
 					{
@@ -155,6 +155,12 @@ describe("CopyPlugin", () => {
 					}
 				]
 			})
+				.then(({ stats, compilation }) => {
+					const assetInfo = stats.compilation.getAsset("newfile.txt");
+					expect(assetInfo.info.sourceFilename).toBe("file.txt");
+					expect(assetInfo.name).toBe("newfile.txt");
+					expect(compilation.assets["newfile.txt"]).toBeDefined();
+				})
 				.then(done)
 				.catch(done);
 		});
@@ -306,7 +312,7 @@ describe("CopyPlugin", () => {
 			})
 				.then(({ stats }) => {
 					for (const name of expectedAssetKeys) {
-						const info = stats.compilation.assetsInfo.get(name);
+						const { info } = stats.compilation.getAsset(name);
 
 						expect(info.copied).toBe(true);
 
@@ -350,7 +356,7 @@ describe("CopyPlugin", () => {
 			})
 				.then(({ stats }) => {
 					for (const name of expectedAssetKeys) {
-						const info = stats.compilation.assetsInfo.get(name);
+						const { info } = stats.compilation.getAsset(name);
 
 						expect(info.immutable).toBe(true);
 
@@ -363,7 +369,7 @@ describe("CopyPlugin", () => {
 				.catch(done);
 		});
 
-		it.skip('should copy files and print "copied" in the string representation ', done => {
+		it('should copy files and print "copied" in the string representation ', done => {
 			expect.assertions(1);
 
 			const expectedAssetKeys = [
@@ -524,7 +530,7 @@ describe("CopyPlugin", () => {
 								{
 									from: path.resolve(__dirname, "./fixtures/directory"),
 									to: () => {
-										return 'directory';
+										return "directory";
 									}
 								}
 							]