Use input source names in the link references

Author: alcuadrado

v-next/hardhat/src/internal/builtin-plugins/solidity/build-system/artifacts.ts

@@ -1,8 +1,4 @@
-import type {
-  Artifact,
-  BuildInfo,
-  LinkReferences,
-} from "../../../../types/artifacts.js";
+import type { Artifact, BuildInfo } from "../../../../types/artifacts.js";
 import type { CompilationJob } from "../../../../types/solidity/compilation-job.js";
 import type {
   CompilerOutput,
@@ -21,7 +17,6 @@ export function getContractArtifact(
   inputSourceName: string,
   contractName: string,
   contract: CompilerOutputContract,
-  userSourceNameMap: Record<string, string>,
 ): Artifact {
   const evmBytecode = contract.evm?.bytecode;
   const bytecode: string =
@@ -47,14 +42,8 @@ export function getContractArtifact(
     abi: contract.abi,
     bytecode,
     deployedBytecode,
-    linkReferences: applyUserSourceNamesToLinkReferences(
-      linkReferences,
-      userSourceNameMap,
-    ),
-    deployedLinkReferences: applyUserSourceNamesToLinkReferences(
-      deployedLinkReferences,
-      userSourceNameMap,
-    ),
+    linkReferences,
+    deployedLinkReferences,
     immutableReferences,
     inputSourceName,
     buildInfoId,
@@ -63,18 +52,6 @@ export function getContractArtifact(
   return artifact;
 }
 
-function applyUserSourceNamesToLinkReferences(
-  linkReferences: LinkReferences,
-  userSourceNameMap: Record<string, string>,
-): LinkReferences {
-  return Object.fromEntries(
-    Object.entries(linkReferences).map(([sourceName, references]) => [
-      userSourceNameMap[sourceName] ?? sourceName,
-      references,
-    ]),
-  );
-}
-
 export function getArtifactsDeclarationFile(artifacts: Artifact[]): string {
   if (artifacts.length === 0) {
     return "";

v-next/hardhat/src/internal/builtin-plugins/solidity/build-system/build-system.ts

@@ -471,16 +471,6 @@ export class SolidityBuildSystemImplementation implements SolidityBuildSystem {
     const result = new Map<string, string[]>();
     const buildId = await compilationJob.getBuildId();
 
-    const userSourceNameMap = Object.fromEntries(
-      compilationJob.dependencyGraph
-        .getRoots()
-        .entries()
-        .map(([userSourceName, root]) => [
-          root.inputSourceName,
-          userSourceName,
-        ]),
-    );
-
     // We emit the artifacts for each root file, first emitting one artifact
     // for each contract, and then one declaration file for the entire file,
     // which defines their types and augments the ArtifactMap type.
@@ -511,7 +501,6 @@ export class SolidityBuildSystemImplementation implements SolidityBuildSystem {
             root.inputSourceName,
             contractName,
             contract,
-            userSourceNameMap,
           );
 
           await writeUtf8File(

v-next/hardhat/src/types/artifacts.ts

@@ -218,7 +218,7 @@ export interface Artifact<AbiT extends Abi = Abi> {
  * The link references of a contract, which need to be resolved before using it.
  */
 export interface LinkReferences {
-  [librarySourceName: string]: {
+  [libraryInputSourceName: string]: {
     [libraryName: string]: Array<{ length: number; start: number }>;
   };
 }