Add git commit hash in output comments for Community Contracts

packages/core/solidity/openzeppelin-contracts.d.ts

@@ -1,6 +1,15 @@
 export interface OpenZeppelinContracts {
+  /**
+   * Version of `@openzeppelin/contracts` and `@openzeppelin/contracts-upgradeable`
+   */
   version: string;
+  /**
+   * Map of source file path to source code.
+   */
   sources: Record<string, string>;
+  /**
+   * Map of source file path to the list of all source file paths it depends on (including transitive dependencies).
+   */
   dependencies: Record<string, string[]>;
 }
 

packages/core/solidity/package.json

@@ -22,7 +22,7 @@
     "update-env": "rm ./src/environments/hardhat/package-lock.json && npm install --package-lock-only --prefix ./src/environments/hardhat && rm ./src/environments/hardhat/upgradeable/package-lock.json && npm install --package-lock-only --prefix ./src/environments/hardhat/upgradeable"
   },
   "devDependencies": {
-    "@openzeppelin/community-contracts": "https://github.com/OpenZeppelin/openzeppelin-community-contracts",
+    "@openzeppelin/community-contracts": "git+https://github.com/OpenZeppelin/openzeppelin-community-contracts.git#de17c8ee4b0329867f7219fbc401707be9518ff1",
     "@openzeppelin/contracts": "^5.4.0",
     "@openzeppelin/contracts-upgradeable": "^5.4.0",
     "@types/node": "^20.0.0",

packages/core/solidity/src/print.ts

@@ -17,6 +17,8 @@ import SOLIDITY_VERSION from './solidity-version.json';
 import { inferTranspiled } from './infer-transpiled';
 import { compatibleContractsSemver } from './utils/version';
 import { stringifyUnicodeSafe } from './utils/sanitize';
+import { importsCommunityContracts } from './utils/imports-libraries';
+import { getCommunityContractsGitCommit } from './utils/community-contracts-git-commit';
 
 export function printContract(contract: Contract, opts?: Options): string {
   const helpers = withHelpers(contract, opts);
@@ -29,7 +31,7 @@ export function printContract(contract: Contract, opts?: Options): string {
     ...spaceBetween(
       [
         `// SPDX-License-Identifier: ${contract.license}`,
-        `// Compatible with OpenZeppelin Contracts ${compatibleContractsSemver}`,
+        ...printCompatibleLibraryVersions(contract),
         `pragma solidity ^${SOLIDITY_VERSION};`,
       ],
 
@@ -54,6 +56,15 @@ export function printContract(contract: Contract, opts?: Options): string {
   );
 }
 
+function printCompatibleLibraryVersions(contract: Contract): string[] {
+  const lines: string[] = [];
+  lines.push(`// Compatible with OpenZeppelin Contracts ${compatibleContractsSemver}`);
+  if (importsCommunityContracts(contract)) {
+    lines.push(`// Compatible with OpenZeppelin Community Contracts commit ${getCommunityContractsGitCommit()}`);
+  }
+  return lines;
+}
+
 function printInheritance(contract: Contract, { transformName }: Helpers): [] | [string] {
   if (contract.parents.length > 0) {
     return ['is ' + contract.parents.map(p => transformName(p.contract)).join(', ')];

packages/core/solidity/src/utils/community-contracts-git-commit.ts

@@ -0,0 +1,19 @@
+import { devDependencies } from '../../package.json';
+
+export function getCommunityContractsGitCommit(): string {
+  const communityContractsVersion = devDependencies['@openzeppelin/community-contracts'];
+  if (!communityContractsVersion) {
+    throw new Error('@openzeppelin/community-contracts not found in devDependencies');
+  }
+  return extractGitCommitHash('@openzeppelin/community-contracts', communityContractsVersion);
+}
+
+function extractGitCommitHash(dependencyName: string, dependencyVersion: string): string {
+  const splitHash = dependencyVersion.split('#');
+  if (!dependencyVersion.startsWith('git+') || splitHash.length !== 2) {
+    throw new Error(
+      `Expected package dependency for ${dependencyName} in format git+<url>#<commit-hash>, but got ${dependencyVersion}`,
+    );
+  }
+  return splitHash[1]!;
+}

packages/core/solidity/src/utils/imports-libraries.ts

@@ -0,0 +1,5 @@
+import type { Contract } from '../contract';
+
+export function importsCommunityContracts(contract: Contract) {
+  return contract.imports.some(i => i.path.startsWith('@openzeppelin/community-contracts/'));
+}

packages/ui/src/solidity/inject-hyperlinks.ts

@@ -2,17 +2,32 @@ import { version as contractsVersion } from '@openzeppelin/contracts/package.jso
 
 export function injectHyperlinks(code: string) {
   // We are modifying HTML, so use HTML escaped chars. The pattern excludes paths that include /../ in the URL.
-  const contractsRegex =
+  const importContractsRegex =
     /"(@openzeppelin\/)(contracts-upgradeable\/|contracts\/)((?:(?!\.\.)[^/]+\/)*?[^/]*?)"/g;
-  const communityContractsRegex = /"(@openzeppelin\/)(community-contracts\/)((?:(?!\.\.)[^/]+\/)*?[^/]*?)"/g;
+  const importCommunityContractsRegex =
+    /"(@openzeppelin\/)(community-contracts\/)((?:(?!\.\.)[^/]+\/)*?[^/]*?)"/g;
 
-  return code
-    .replace(
-      contractsRegex,
-      `&quot;<a class="import-link" href="https://github.com/OpenZeppelin/openzeppelin-$2blob/v${contractsVersion}/contracts/$3" target="_blank" rel="noopener noreferrer">$1$2$3</a>&quot;`,
-    )
-    .replace(
-      communityContractsRegex,
-      `&quot;<a class="import-link" href="https://github.com/OpenZeppelin/openzeppelin-community-contracts/blob/master/contracts/$3" target="_blank" rel="noopener noreferrer">$1$2$3</a>&quot;`,
-    );
+  const compatibleCommunityContractsRegexSingle = /OpenZeppelin Community Contracts commit ([a-f0-9]{40})/;
+  const compatibleCommunityContractsRegexGlobal = new RegExp(compatibleCommunityContractsRegexSingle, 'g');
+
+  const compatibleCommunityContractsGitCommit = code.match(compatibleCommunityContractsRegexSingle)?.[1];
+
+  let result = code.replace(
+    importContractsRegex,
+    `&quot;<a class="import-link" href="https://github.com/OpenZeppelin/openzeppelin-$2blob/v${contractsVersion}/contracts/$3" target="_blank" rel="noopener noreferrer">$1$2$3</a>&quot;`,
+  );
+
+  if (compatibleCommunityContractsGitCommit !== undefined) {
+    result = result
+      .replace(
+        importCommunityContractsRegex,
+        `&quot;<a class="import-link" href="https://github.com/OpenZeppelin/openzeppelin-community-contracts/tree/${compatibleCommunityContractsGitCommit}/contracts/$3" target="_blank" rel="noopener noreferrer">$1$2$3</a>&quot;`,
+      )
+      .replace(
+        compatibleCommunityContractsRegexGlobal,
+        `OpenZeppelin Community Contracts commit <a class="import-link" href="https://github.com/OpenZeppelin/openzeppelin-community-contracts/tree/$1" target="_blank" rel="noopener noreferrer" title="View repository at commit $1">$1</a>`,
+      );
+  }
+
+  return result;
 }

yarn.lock

@@ -823,9 +823,9 @@
     "@nomicfoundation/solidity-analyzer-linux-x64-musl" "0.1.2"
     "@nomicfoundation/solidity-analyzer-win32-x64-msvc" "0.1.2"
 
-"@openzeppelin/community-contracts@https://github.com/OpenZeppelin/openzeppelin-community-contracts":
+"@openzeppelin/community-contracts@git+https://github.com/OpenZeppelin/openzeppelin-community-contracts.git#de17c8ee4b0329867f7219fbc401707be9518ff1":
   version "0.0.1"
-  resolved "https://github.com/OpenZeppelin/openzeppelin-community-contracts#de17c8ee4b0329867f7219fbc401707be9518ff1"
+  resolved "git+https://github.com/OpenZeppelin/openzeppelin-community-contracts.git#de17c8ee4b0329867f7219fbc401707be9518ff1"
 
 "@openzeppelin/contracts-upgradeable@^5.4.0":
   version "5.4.0"