Cannot download the complete openzeppellin library in remix desktop

Cainuriel · March 18, 2023, 3:46pm

I am trying to test eth-infinitism contracts on my remix desktop.

But I can't get it to download your complete library.

I have copied them from the official repository and of course I have not touched any files. However I suspect this is not the best way to test these contracts.

ernestognw · March 22, 2023, 8:32pm

Hey @Cainuriel,

In your firest example, the openzeppelin/.../Create2.sol is missing a @ at the beginning.

I'm not sure about the second one but my best guess is that you may have a mismatched version in Remix. This usually happens because it caches versions depending on the imported files. Best practice in remix is to specify the version as:

import "@openzeppelin/contracts/v4.8.2/..."

Where v4.8.2 is the latest at the moment of writing.

Is it possible you can share a URL to remix with a minimal reproducible example? So I can confirm it's the version cache.

Cainuriel · March 23, 2023, 8:56am

ernestognw:

In your firest example, the openzeppelin/.../Create2.sol is missing a @ at the beginning.

I'm not sure about the second one but my best guess is that you may have a mismatched version in Remix. This usually happens because it caches versions depending on the imported files. Best practice in remix is to specify the version as:
import "@openzeppelin/contracts/v4.8.2/..."
Where v4.8.2 is the latest at the moment of writing.

Is it possible you can share a URL to remix with a minimal reproducible example? So I can confirm it's the version cache.

I have made the indicated corrections with the same result.

What type of URL would you like me to share with you?
I am using Remix desktop.

ernestognw · March 23, 2023, 5:56pm

Oh sorry, my bad, I made a mistake with the import line, it should be:

import "@openzeppelin/contracts@4.8.2/..."

What type of URL would you like me to share with you?
I am using Remix desktop.

I was referring to URLs with encoded code, but I thought there was an option for generating them.
If the @4.8.2 doesn't work, I think it'll be fine to just share the files if you want.

Cainuriel · March 24, 2023, 10:02am

Dont worry

Same result:

If you wish to have the code I have used the standard shared by its creators:
Here is SimplleAccount.sol:

github.com

eth-infinitism/account-abstraction/blob/develop/contracts/samples/SimpleAccount.sol

// SPDX-License-Identifier: GPL-3.0
pragma solidity ^0.8.12;

/* solhint-disable avoid-low-level-calls */
/* solhint-disable no-inline-assembly */
/* solhint-disable reason-string */

import "@openzeppelin/contracts/utils/cryptography/ECDSA.sol";
import "@openzeppelin/contracts/proxy/utils/Initializable.sol";
import "@openzeppelin/contracts/proxy/utils/UUPSUpgradeable.sol";

import "../core/BaseAccount.sol";
import "./callback/TokenCallbackHandler.sol";

/**
  * minimal account.
  *  this is sample minimal account.
  *  has execute, eth handling methods
  *  has a single signer that can send requests through the entryPoint.
  */

This file has been truncated. show original

And here is SimpleAccountFactory.sol:

github.com

eth-infinitism/account-abstraction/blob/develop/contracts/samples/SimpleAccountFactory.sol

// SPDX-License-Identifier: GPL-3.0
pragma solidity ^0.8.12;

import "@openzeppelin/contracts/utils/Create2.sol";
import "@openzeppelin/contracts/proxy/ERC1967/ERC1967Proxy.sol";

import "./SimpleAccount.sol";

/**
 * A sample factory contract for SimpleAccount
 * A UserOperations "initCode" holds the address of the factory, and a method call (to createAccount, in this sample factory).
 * The factory's createAccount returns the target account address even if it is already installed.
 * This way, the entryPoint.getSenderAddress() can be called either before or after the account is created.
 */
contract SimpleAccountFactory {
    SimpleAccount public immutable accountImplementation;

    constructor(IEntryPoint _entryPoint) {
        accountImplementation = new SimpleAccount(_entryPoint);
    }

This file has been truncated. show original

ernestognw · March 26, 2023, 12:55am

Thanks for sharing @Cainuriel.

I can't clearly see the issue without looking at the folder structure, but my guess is that it's still either a version mismatch (you have to change all the @openzeppelin/contracts imports to @4.8.2), or that you're missing some files, I see you reordered the original code.

Luckily, the dependencies are cached by workspace on Remix as referred in the docs, so it may be worth starting a new workspace from scratch so you can get the newest versions.

My recommendation would be using the Clone repository option, it'll create a new Workspace from scratch, so the dependencies should match, and you can start testing or extending from there:

Best

Cainuriel · March 26, 2023, 1:34pm

Fantastic! @ernestognw, easy solution.
This happens because I don't know the tool well.
With cloning everything compiled perfectly.