fix: change commentsImplCreationCode visibility to internal for dynamic test linking compatibility (#1363)

Changes function visibility from pure to internal to fix compilation errors
when dynamic test linking transforms type(Contract).creationCode into
VmContractHelper87().getCode() calls, which require access to vm context.

Author: oveddan

packages/coins/foundry.toml

@@ -6,6 +6,7 @@ via_ir = true
 optimizer = true
 solc_version = '0.8.28'
 optimizer_runs = 200
+dynamic_test_linking = true
 fs_permissions = [
   { access = "readwrite", path = "./addresses" },
   { access = "readwrite", path = "./addresses/dev" },

packages/comments/foundry.toml

@@ -6,6 +6,7 @@ via_ir = true
 solc_version = '0.8.23'
 optimizer = true
 optimizer_runs = 1_000_000
+dynamic_test_linking = true
 fs_permissions = [
   { access = "readwrite", path = "./addresses" },
   { access = "read", path = "./package.json" },

packages/comments/script/CommentsDeployerBase.sol

@@ -64,7 +64,7 @@ contract CommentsDeployerBase is ProxyDeployerScript {
         deployment.callerAndCommenterVersion = readStringOrDefaultToEmpty(json, "CALLER_AND_COMMENTER_VERSION");
     }
 
-    function commentsImplCreationCode() internal pure returns (bytes memory) {
+    function commentsImplCreationCode() internal returns (bytes memory) {
         return abi.encodePacked(type(CommentsImpl).creationCode, abi.encode(SPARK_VALUE, PROTOCOL_REWARDS));
     }