@artblocks/contracts 1.0.3 → 1.1.0

package/contracts/DependencyRegistryV0.sol

@@ -3,9 +3,9 @@ pragma solidity 0.8.19;
 
 // Created By: Art Blocks Inc.
 
-import "./interfaces/0.8.x/IAdminACLV0.sol";
-import "./interfaces/0.8.x/IDependencyRegistryCompatibleV0.sol";
-import "./interfaces/0.8.x/IDependencyRegistryV0.sol";
+import "./interfaces/v0.8.x/IAdminACLV0.sol";
+import "./interfaces/v0.8.x/IDependencyRegistryCompatibleV0.sol";
+import "./interfaces/v0.8.x/IDependencyRegistryV0.sol";
 
 import "@openzeppelin-4.7/contracts/utils/Strings.sol";
 import "@openzeppelin-4.7/contracts/utils/structs/EnumerableSet.sol";
@@ -13,8 +13,8 @@ import "@openzeppelin-4.8/contracts-upgradeable/access/OwnableUpgradeable.sol";
 import "@openzeppelin-4.8/contracts-upgradeable/proxy/utils/Initializable.sol";
 import "@openzeppelin-4.5/contracts/utils/math/SafeCast.sol";
 
-import "./libs/0.8.x/BytecodeStorageV1.sol";
-import "./libs/0.8.x/Bytes32Strings.sol";
+import "./libs/v0.8.x/BytecodeStorageV1.sol";
+import "./libs/v0.8.x/Bytes32Strings.sol";
 
 /**
  * @title Art Blocks Dependency Registry, V0.
@@ -23,7 +23,7 @@ import "./libs/0.8.x/Bytes32Strings.sol";
  * Permissions managed by ACL contract. If/when we ever call
  * renounceOwnership() this will becom a frozen, immutable registry
  * as no upgrades will be possible.
- * This contract This contract is intended to be an auxiliary reference registry
+ * This contract is intended to be an auxiliary reference registry
  * to our non-upgradeable and immutable ERC-721 conforming core contracts,
  * and has been made upgradeable as we expect its required functionality in
  * relation to the Art Blocks ecosystem to evolve over time.
@@ -46,9 +46,12 @@ contract DependencyRegistryV0 is
     IAdminACLV0 public adminACLContract;
 
     struct Dependency {
+        bytes32 licenseType;
         string preferredCDN;
+        // mapping from additional CDN index to CDN URLr
         mapping(uint256 => string) additionalCDNs;
         string preferredRepository;
+        // mapping from additional repository index to repository URL
         mapping(uint256 => string) additionalRepositories;
         string referenceWebsite;
         // mapping from script index to address storing script in bytecode
@@ -58,9 +61,14 @@ contract DependencyRegistryV0 is
         uint24 scriptCount;
     }
 
+    // dependency types, i.e. "type@version"
     EnumerableSet.Bytes32Set private _dependencyTypes;
+    // mapping from dependencyTypes to Dependency, which stores the properties of each dependency
     mapping(bytes32 => Dependency) dependencyDetails;
+    // source code license types, MIT, GPL, etc.
+    EnumerableSet.Bytes32Set private _licenseTypes;
 
+    // set of supported core ArtBlocks contracts
     EnumerableSet.AddressSet private _supportedCoreContracts;
     mapping(address => mapping(uint256 => bytes32)) projectDependencyTypeOverrides;
 
@@ -97,6 +105,17 @@ contract DependencyRegistryV0 is
         );
     }
 
+    function _onlyInRangeIndex(uint256 _index, uint256 _length) internal pure {
+        require(_index < _length, "Index out of range");
+    }
+
+    function _onlyExistingLicenseType(bytes32 _licenseType) internal view {
+        require(
+            _licenseTypes.contains(_licenseType),
+            "License type does not exist"
+        );
+    }
+
     /**
      * @notice Initializes contract.
      * @param _adminACLContract Address of admin access control contract, to be
@@ -108,23 +127,36 @@ contract DependencyRegistryV0 is
         _transferOwnership(_adminACLContract);
     }
 
+    /**
+     * @notice Adds a new license type that can be used by dependencies.
+     */
+    function addLicenseType(bytes32 _licenseType) external {
+        _onlyAdminACL(this.addLicenseType.selector);
+        // @dev the add function returns false if set already contains value
+        require(_licenseTypes.add(_licenseType), "License type already exists");
+        require(
+            _licenseType != bytes32(""),
+            "License type cannot be empty string"
+        );
+        emit LicenseTypeAdded(_licenseType);
+    }
+
     /**
      * @notice Adds a new dependency.
      * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _licenseType License type for dependency, must be a registered license type.
      * @param _preferredCDN Preferred CDN for dependency.
      * @param _preferredRepository Preferred repository for dependency.
      */
     function addDependency(
         bytes32 _dependencyType,
+        bytes32 _licenseType,
         string memory _preferredCDN,
         string memory _preferredRepository,
         string memory _referenceWebsite
     ) external {
         _onlyAdminACL(this.addDependency.selector);
-        require(
-            !_dependencyTypes.contains(_dependencyType),
-            "Dependency type already exists"
-        );
+        _onlyExistingLicenseType(_licenseType);
         require(
             _dependencyType.containsExactCharacterQty(
                 AT_CHARACTER_CODE,
@@ -132,15 +164,21 @@ contract DependencyRegistryV0 is
             ),
             "must contain exactly one @"
         );
+        require(
+            // @dev the add function returns false if set already contains value
+            _dependencyTypes.add(_dependencyType),
+            "Dependency type already exists"
+        );
 
-        _dependencyTypes.add(_dependencyType);
-        Dependency storage dependencyType = dependencyDetails[_dependencyType];
-        dependencyType.preferredCDN = _preferredCDN;
-        dependencyType.preferredRepository = _preferredRepository;
-        dependencyType.referenceWebsite = _referenceWebsite;
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        dependency.licenseType = _licenseType;
+        dependency.preferredCDN = _preferredCDN;
+        dependency.preferredRepository = _preferredRepository;
+        dependency.referenceWebsite = _referenceWebsite;
 
         emit DependencyAdded(
             _dependencyType,
+            _licenseType,
             _preferredCDN,
             _preferredRepository,
             _referenceWebsite
@@ -149,7 +187,7 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Removes a dependency.
-     * @param _dependencyType Name of dependency type (i.e. "type@version")
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      */
     function removeDependency(bytes32 _dependencyType) external {
         _onlyAdminACL(this.removeDependency.selector);
@@ -169,10 +207,11 @@ contract DependencyRegistryV0 is
     }
 
     /**
-     * @notice Adds a script to dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @notice Adds a script to dependency `_dependencyType`, by way of
+     *         providing a string to write to bytecode storage.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _script Script to be added. Required to be a non-empty string,
-     * but no further validation is performed.
+     *                but no further validation is performed.
      */
     function addDependencyScript(
         bytes32 _dependencyType,
@@ -191,36 +230,80 @@ contract DependencyRegistryV0 is
     }
 
     /**
-     * @notice Updates script for dependencyType `_dependencyType` at script ID `_scriptId`.
-     * @param _dependencyType dependency to be updated.
-     * @param _scriptId Script ID to be updated.
+     * @notice Updates script for dependencyType `_dependencyType` at script `_index`,
+     *         by way of providing a string to write to bytecode storage.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of a given script, relative to the overall `scriptCount`.
      * @param _script The updated script value. Required to be a non-empty
-     * string, but no further validation is performed.
+     *                string, but no further validation is performed.
      */
     function updateDependencyScript(
         bytes32 _dependencyType,
-        uint256 _scriptId,
+        uint256 _index,
         string memory _script
     ) external {
         _onlyAdminACL(this.updateDependencyScript.selector);
         _onlyNonEmptyString(_script);
         _onlyExistingDependencyType(_dependencyType);
-        Dependency storage dependencyType = dependencyDetails[_dependencyType];
-        require(
-            _scriptId < dependencyType.scriptCount,
-            "scriptId out of range"
-        );
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({_index: _index, _length: dependency.scriptCount});
         // store script in contract bytecode, replacing reference address from
         // the contract that no longer exists with the newly created one
-        dependencyType.scriptBytecodeAddresses[_scriptId] = _script
-            .writeToBytecode();
+        dependency.scriptBytecodeAddresses[_index] = _script.writeToBytecode();
+
+        emit DependencyScriptUpdated(_dependencyType);
+    }
+
+    /**
+     * @notice Adds a script to dependency `_dependencyType`, by way of
+     *         providing an already written chunk of bytecode storage.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _scriptPointer Address of script to be added. Required to be a non-zero address,
+     *                       but no further validation is performed.
+     */
+    function addDependencyScriptPointer(
+        bytes32 _dependencyType,
+        address _scriptPointer
+    ) external {
+        _onlyAdminACL(this.addDependencyScriptPointer.selector);
+        _onlyNonZeroAddress(_scriptPointer);
+        _onlyExistingDependencyType(_dependencyType);
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        // store script in contract bytecode
+        dependency.scriptBytecodeAddresses[
+            dependency.scriptCount
+        ] = _scriptPointer;
+        dependency.scriptCount = dependency.scriptCount + 1;
+
+        emit DependencyScriptUpdated(_dependencyType);
+    }
+
+    /**
+     * @notice Updates script for dependencyType `_dependencyType` at script `_index`,
+     *         by way of providing an already written chunk of bytecode storage.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of a given script, relative to the overall `scriptCount`.
+     * @param _scriptPointer The updated script pointer (address of bytecode storage).
+     *                       Required to be a non-zero address, but no further validation is performed.
+     */
+    function updateDependencyScriptPointer(
+        bytes32 _dependencyType,
+        uint256 _index,
+        address _scriptPointer
+    ) external {
+        _onlyAdminACL(this.updateDependencyScriptPointer.selector);
+        _onlyNonZeroAddress(_scriptPointer);
+        _onlyExistingDependencyType(_dependencyType);
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({_index: _index, _length: dependency.scriptCount});
+        dependency.scriptBytecodeAddresses[_index] = _scriptPointer;
 
         emit DependencyScriptUpdated(_dependencyType);
     }
 
     /**
      * @notice Removes last script from dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      */
     function removeDependencyLastScript(bytes32 _dependencyType) external {
         _onlyAdminACL(this.removeDependencyLastScript.selector);
@@ -238,7 +321,7 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Updates preferred CDN for dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _preferredCDN URL for preferred CDN.
      */
     function updateDependencyPreferredCDN(
@@ -254,7 +337,7 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Updates preferred repository for dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _preferredRepository URL for preferred repository.
      */
     function updateDependencyPreferredRepository(
@@ -274,7 +357,7 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Updates project website for dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _referenceWebsite URL for project website.
      */
     function updateDependencyReferenceWebsite(
@@ -293,9 +376,9 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Adds a new CDN url to `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _additionalCDN CDN URL to be added. Required to be a non-empty string,
-     * but no further validation is performed.
+     *                       but no further validation is performed.
      */
     function addDependencyAdditionalCDN(
         bytes32 _dependencyType,
@@ -321,19 +404,19 @@ contract DependencyRegistryV0 is
      * @notice Removes additional CDN for dependency `_dependencyId` at index `_index`.
      * Removal is done by swapping the element to be removed with the last element in the array, then deleting this last element.
      * Assets with indices higher than `_index` can have their indices adjusted as a result of this operation.
-     * @param _dependencyType dependency to be updated.
-     * @param _index Additional CDN index
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional CDN, relative to the overall `additionalCDNCount`.
      */
-    function removeDependencyAdditionalCDNAtIndex(
+    function removeDependencyAdditionalCDN(
         bytes32 _dependencyType,
         uint256 _index
     ) external {
-        _onlyAdminACL(this.removeDependencyAdditionalCDNAtIndex.selector);
+        _onlyAdminACL(this.removeDependencyAdditionalCDN.selector);
         _onlyExistingDependencyType(_dependencyType);
         Dependency storage dependency = dependencyDetails[_dependencyType];
 
         uint256 additionalCDNCount = dependency.additionalCDNCount;
-        require(_index < additionalCDNCount, "Asset index out of range");
+        _onlyInRangeIndex({_index: _index, _length: additionalCDNCount});
 
         uint256 lastElementIndex = additionalCDNCount - 1;
 
@@ -349,21 +432,23 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Updates additional CDN for dependency `_dependencyType` at `_index`.
-     * @param _dependencyType dependency to be updated.
-     * @param _index Additional CDN index.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional CDN, relative to the overall `additionalCDNCount`.
      * @param _additionalCDN New CDN URL.
      */
-    function updateDependencyAdditionalCDNAtIndex(
+    function updateDependencyAdditionalCDN(
         bytes32 _dependencyType,
         uint256 _index,
         string memory _additionalCDN
     ) external {
-        _onlyAdminACL(this.updateDependencyAdditionalCDNAtIndex.selector);
+        _onlyAdminACL(this.updateDependencyAdditionalCDN.selector);
         _onlyNonEmptyString(_additionalCDN);
         _onlyExistingDependencyType(_dependencyType);
         Dependency storage dependency = dependencyDetails[_dependencyType];
-        uint24 additionalCDNCount = dependency.additionalCDNCount;
-        require(_index < additionalCDNCount, "Asset index out of range");
+        _onlyInRangeIndex({
+            _index: _index,
+            _length: dependency.additionalCDNCount
+        });
 
         dependency.additionalCDNs[_index] = _additionalCDN;
 
@@ -376,7 +461,7 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Adds a new repository URL to dependency `_dependencyType`.
-     * @param _dependencyType dependency to be updated.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @param _additionalRepository Repository URL to be added. Required to be a non-empty string,
      * but no further validation is performed.
      */
@@ -409,22 +494,19 @@ contract DependencyRegistryV0 is
      * @notice Removes additional repository for depenency `_dependencyId` at index `_index`.
      * Removal is done by swapping the element to be removed with the last element in the array, then deleting this last element.
      * Assets with indices higher than `_index` can have their indices adjusted as a result of this operation.
-     * @param _dependencyType dependency to be updated.
-     * @param _index Additional repository index.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional repository, relative to the overall `additionalRepositoryCount`.
      */
-    function removeDependencyAdditionalRepositoryAtIndex(
+    function removeDependencyAdditionalRepository(
         bytes32 _dependencyType,
         uint256 _index
     ) external {
-        _onlyAdminACL(
-            this.removeDependencyAdditionalRepositoryAtIndex.selector
-        );
+        _onlyAdminACL(this.removeDependencyAdditionalRepository.selector);
         _onlyExistingDependencyType(_dependencyType);
         Dependency storage dependency = dependencyDetails[_dependencyType];
-        uint256 additionalRepositoryCount = uint256(
-            dependency.additionalRepositoryCount
-        );
-        require(_index < additionalRepositoryCount, "Asset index out of range");
+        uint256 additionalRepositoryCount = dependency
+            .additionalRepositoryCount;
+        _onlyInRangeIndex({_index: _index, _length: additionalRepositoryCount});
 
         uint256 lastElementIndex = additionalRepositoryCount - 1;
 
@@ -440,23 +522,23 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Updates additional repository for dependency `_dependencyType` at `_index`.
-     * @param _dependencyType dependency to be updated.
-     * @param _index Additional repository index.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional repository, relative to the overall `additionalRepositoryCount`.
      * @param _additionalRepository New Repository URL.
      */
-    function updateDependencyAdditionalRepositoryAtIndex(
+    function updateDependencyAdditionalRepository(
         bytes32 _dependencyType,
         uint256 _index,
         string memory _additionalRepository
     ) external {
-        _onlyAdminACL(
-            this.updateDependencyAdditionalRepositoryAtIndex.selector
-        );
+        _onlyAdminACL(this.updateDependencyAdditionalRepository.selector);
         _onlyNonEmptyString(_additionalRepository);
         _onlyExistingDependencyType(_dependencyType);
         Dependency storage dependency = dependencyDetails[_dependencyType];
-        uint24 additionalRepositoryCount = dependency.additionalRepositoryCount;
-        require(_index < additionalRepositoryCount, "Asset index out of range");
+        _onlyInRangeIndex({
+            _index: _index,
+            _length: dependency.additionalRepositoryCount
+        });
 
         dependency.additionalRepositories[_index] = _additionalRepository;
 
@@ -475,12 +557,10 @@ contract DependencyRegistryV0 is
         _onlyAdminACL(this.addSupportedCoreContract.selector);
         _onlyNonZeroAddress(_contractAddress);
         require(
-            !_supportedCoreContracts.contains(_contractAddress),
+            // @dev the add function returns false if set already contains value
+            _supportedCoreContracts.add(_contractAddress),
             "Contract already supported"
         );
-
-        _supportedCoreContracts.add(_contractAddress);
-
         emit SupportedCoreContractAdded(_contractAddress);
     }
 
@@ -490,9 +570,11 @@ contract DependencyRegistryV0 is
      */
     function removeSupportedCoreContract(address _contractAddress) external {
         _onlyAdminACL(this.removeSupportedCoreContract.selector);
-        _onlySupportedCoreContract(_contractAddress);
-        _supportedCoreContracts.remove(_contractAddress);
-
+        require(
+            // @dev the remove function returns false if set does not contain value
+            _supportedCoreContracts.remove(_contractAddress),
+            "Core contract already removed or not in set"
+        );
         emit SupportedCoreContractRemoved(_contractAddress);
     }
 
@@ -503,7 +585,7 @@ contract DependencyRegistryV0 is
      * type (`_dependencyType`).
      * @param _contractAddress Core contract address.
      * @param _projectId Project to override script type and version for.
-     * @param _dependencyType Dependency type to return for project.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      */
     function addProjectDependencyTypeOverride(
         address _contractAddress,
@@ -553,10 +635,8 @@ contract DependencyRegistryV0 is
      * execution where there is no gas limit.
      */
     function getDependencyTypes() external view returns (string[] memory) {
-        string[] memory dependencyTypes = new string[](
-            _dependencyTypes.length()
-        );
         uint256 numDependencyTypes = _dependencyTypes.length();
+        string[] memory dependencyTypes = new string[](numDependencyTypes);
 
         for (uint256 i = 0; i < numDependencyTypes; i++) {
             dependencyTypes[i] = _dependencyTypes.at(i).toString();
@@ -574,20 +654,56 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Returns registered depenedency type at index `_index`.
-     * @return Registered dependency at `_index`.
+     * @return Registered dependency at `_index`, relative to the overall length of the dependency type set.
      */
-    function getDependencyTypeAtIndex(
+    function getDependencyType(
         uint256 _index
     ) external view returns (string memory) {
-        require(_dependencyTypes.length() > _index, "Index out of range");
+        _onlyInRangeIndex({_index: _index, _length: _dependencyTypes.length()});
         return _dependencyTypes.at(_index).toString();
     }
 
     /**
-     * @notice Returns details for depedency type `_dependencyType`.
-     * @param _dependencyType Dependency type to be queried.
+     * @notice Returns number of registered license types
+     * @return Number of registered license types.
+     * @dev This is only intended to be called outside of block
+     * execution where there is no gas limit.
+     */
+    function getLicenseTypes() external view returns (string[] memory) {
+        uint256 numLicenseTypes = _licenseTypes.length();
+        string[] memory licenseTypes = new string[](numLicenseTypes);
+
+        for (uint256 i = 0; i < numLicenseTypes; i++) {
+            licenseTypes[i] = _licenseTypes.at(i).toString();
+        }
+        return licenseTypes;
+    }
+
+    /**
+     * @notice Returns number of registered license types
+     * @return Number of registered license types.
+     */
+    function getLicenseTypeCount() external view returns (uint256) {
+        return _licenseTypes.length();
+    }
+
+    /**
+     * @notice Returns registered license type at index `_index`.
+     * @return Registered license type at `_index`, relative to the overall length of the license type set.
+     */
+    function getLicenseType(
+        uint256 _index
+    ) external view returns (string memory) {
+        _onlyInRangeIndex({_index: _index, _length: _licenseTypes.length()});
+        return _licenseTypes.at(_index).toString();
+    }
+
+    /**
+     * @notice Returns details for a given depedency type `_dependencyType`.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      * @return typeAndVersion String representation of `_dependencyType`.
-     * (e.g. "p5js(atSymbol)1.0.0")
+     *                        (e.g. "p5js(atSymbol)1.0.0")
+     * @return licenseType License type for dependency
      * @return preferredCDN Preferred CDN URL for dependency
      * @return additionalCDNCount Count of additional CDN URLs for dependency
      * @return preferredRepository Preferred repository URL for dependency
@@ -603,6 +719,7 @@ contract DependencyRegistryV0 is
         view
         returns (
             string memory typeAndVersion,
+            string memory licenseType,
             string memory preferredCDN,
             uint24 additionalCDNCount,
             string memory preferredRepository,
@@ -616,6 +733,7 @@ contract DependencyRegistryV0 is
 
         return (
             _dependencyType.toString(),
+            dependency.licenseType.toString(),
             dependency.preferredCDN,
             dependency.additionalCDNCount,
             dependency.preferredRepository,
@@ -636,19 +754,31 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Returns the address of the supported core contract at index `_index`.
-     * @param _index Index of the core contract to be returned.
+     * @param _index Index of the core contract to be returned, relative to the overall
+     *               list of supported core contracts.
      * @return address of the core contract.
      */
-    function getSupportedCoreContractAtIndex(
+    function getSupportedCoreContract(
         uint256 _index
     ) external view returns (address) {
-        require(
-            _supportedCoreContracts.length() > _index,
-            "Index out of bounds"
-        );
+        _onlyInRangeIndex({
+            _index: _index,
+            _length: _supportedCoreContracts.length()
+        });
         return _supportedCoreContracts.at(_index);
     }
 
+    /**
+     * @notice Returns whether the given contract address is a supported core contract.
+     * @param coreContractAddress Address of the core contract to be queried.
+     * @return True if the given contract address is a supported core contract.
+     */
+    function isSupportedCoreContract(
+        address coreContractAddress
+    ) external view returns (bool) {
+        return _supportedCoreContracts.contains(coreContractAddress);
+    }
+
     /**
      * @notice Returns a list of supported core contracts.
      * @return List of supported core contracts.
@@ -674,32 +804,41 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Returns the additional CDN URL at index `_index` for dependency `_dependencyType`.
-     * @param _dependencyType Dependency type to be queried.
-     * @param _index Index of the additional CDN URL to be returned.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional CDN, relative to the overall `additionalCDNCount`.
      */
-    function getDependencyAdditionalCDNAtIndex(
+    function getDependencyAdditionalCDN(
         bytes32 _dependencyType,
         uint256 _index
     ) external view returns (string memory) {
-        return dependencyDetails[_dependencyType].additionalCDNs[_index];
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({
+            _index: _index,
+            _length: dependency.additionalCDNCount
+        });
+        return dependency.additionalCDNs[_index];
     }
 
     /**
      * @notice Returns the additional repository URL at index `_index` for dependency `_dependencyType`.
-     * @param _dependencyType Dependency type to be queried.
-     * @param _index Index of the additional repository URL to be returned.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of an additional repository, relative to the overall `additionalRepositoryCount`.
      */
-    function getDependencyAdditionalRepositoryAtIndex(
+    function getDependencyAdditionalRepository(
         bytes32 _dependencyType,
         uint256 _index
     ) external view returns (string memory) {
-        return
-            dependencyDetails[_dependencyType].additionalRepositories[_index];
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({
+            _index: _index,
+            _length: dependency.additionalRepositoryCount
+        });
+        return dependency.additionalRepositories[_index];
     }
 
     /**
      * @notice Returns the count of scripts for dependency `_dependencyType`.
-     * @param _dependencyType Dependency type to be queried.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
      */
     function getDependencyScriptCount(
         bytes32 _dependencyType
@@ -709,32 +848,80 @@ contract DependencyRegistryV0 is
 
     /**
      * @notice Returns address with bytecode containing script for
-     * dependency `_dependencyTypes` at script index `_index`.
+     *         dependency `_dependencyTypes` at script index `_index`.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of a given script, relative to the overall `scriptCount`.
+     * @return The address of the bytecode storage for the script at the given index, if it can be determined.
      */
-    function getDependencyScriptBytecodeAddressAtIndex(
+    function getDependencyScriptBytecodeAddress(
         bytes32 _dependencyType,
         uint256 _index
     ) external view returns (address) {
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({_index: _index, _length: dependency.scriptCount});
+        return dependency.scriptBytecodeAddresses[_index];
+    }
+
+    /**
+     * @notice Returns the storage library version for
+     *         dependency `_dependencyTypes` at script index `_index`.
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of a given script, relative to the overall `scriptCount`.
+     * @return The storage library version for the script at the given index, if it can be determined.
+     * @dev Note that we only expect this to be determinable if the script was written using a version
+     *      of the Art Blocks `BytecodeStorage` library, and in other cases the fallback will be the
+     *      unknown version string, as defined by the `BytecodeStorage` UNKNOWN_VERSION_STRING – this
+     *      is inclusive of the in the case of `SSTORE2` written data blobs, which are an unknown version
+     *      that can be fallback-read optimistically.
+     */
+    function getDependencyScriptBytecodeStorageVersion(
+        bytes32 _dependencyType,
+        uint256 _index
+    ) external view returns (bytes32) {
+        Dependency storage dependency = dependencyDetails[_dependencyType];
+        _onlyInRangeIndex({_index: _index, _length: dependency.scriptCount});
         return
-            dependencyDetails[_dependencyType].scriptBytecodeAddresses[_index];
+            BytecodeStorageReader.getLibraryVersionForBytecode(
+                dependencyDetails[_dependencyType].scriptBytecodeAddresses[
+                    _index
+                ]
+            );
     }
 
     /**
      * @notice Returns script for dependency `_dependencyType` at script index `_index`.
-     * @param _dependencyType dependency to be queried.
-     * @param _index Index of script to be queried.
-     */
-    function getDependencyScriptAtIndex(
+     * @param _dependencyType Name of dependency type (i.e. "type@version") used to identify dependency.
+     * @param _index The index of a given script, relative to the overall `scriptCount`.
+     * @return A string containing the script content at the given script chunk index for a given dependency.
+     * @dev This method attempts to introspectively determine which library version of
+     *      `BytecodeStorage` was used to write the stored script string that is being
+     *      read back, in order to use the proper read approach. If the version is
+     *      non-determinate, a fall-back to reading using the assumption that the bytes
+     *      were written with `SSTORE2` is used.
+     *      Also note that in this `SSTORE2` fallback handling, the approach of casting bytes to string
+     *      can cause failure (e.g. unexpected continuation byte).
+     */
+    function getDependencyScript(
         bytes32 _dependencyType,
         uint256 _index
     ) external view returns (string memory) {
         Dependency storage dependency = dependencyDetails[_dependencyType];
-        // If trying to access an out-of-index script, return the empty string.
-        if (_index >= dependency.scriptCount) {
-            return "";
+        _onlyInRangeIndex({_index: _index, _length: dependency.scriptCount});
+
+        address scriptAddress = dependency.scriptBytecodeAddresses[_index];
+        bytes32 storageVersion = BytecodeStorageReader
+            .getLibraryVersionForBytecode(scriptAddress);
+
+        if (storageVersion == BytecodeStorageReader.UNKNOWN_VERSION_STRING) {
+            return
+                string(
+                    BytecodeStorageReader.readBytesFromSSTORE2Bytecode(
+                        scriptAddress
+                    )
+                );
+        } else {
+            return BytecodeStorageReader.readFromBytecode(scriptAddress);
         }
-
-        return _readFromBytecode(dependency.scriptBytecodeAddresses[_index]);
     }
 
     /**
@@ -745,7 +932,7 @@ contract DependencyRegistryV0 is
      * an override set, this will revert.
      * @param _contractAddress Core contract address.
      * @param _projectId Project to return dependency type for.
-     * @return dependencyType Dependency type used by project.
+     * @return dependencyType Identifier for the dependency (i.e. "type@version") used by project.
      */
     function getDependencyTypeForProject(
         address _contractAddress,
@@ -828,14 +1015,4 @@ contract DependencyRegistryV0 is
         OwnableUpgradeable._transferOwnership(newOwner);
         adminACLContract = IAdminACLV0(newOwner);
     }
-
-    /**
-     * Helper for calling `BytecodeStorageReader` external library reader method,
-     * added for bytecode size reduction purposes.
-     */
-    function _readFromBytecode(
-        address _address
-    ) internal view returns (string memory) {
-        return BytecodeStorageReader.readFromBytecode(_address);
-    }
 }