feat(cli): Noir contract compiler CLI #1561
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Please could you also write a section of the docs for how to compile a noir contract? (See list of placeholder docs pages: #1456). |
benesjan
approved these changes
Aug 15, 2023
| @@ -116,13 +116,13 @@ await viewUnencryptedLogs(); | |||
| ////////////// QUERYING THE TOKEN BALANCE FOR EACH ACCOUNT ////////////// | |||
|
|
|||
| // Create the contract abstraction and link to Alice's wallet for future signing | |||
| const tokenContractAlice = await PrivateTokenContract.create( | |||
| const tokenContractAlice = await PrivateTokenContract.at( | |||
| "rebuildPatterns": ["^circuits/cpp/barretenberg/cpp/"], | ||
| "rebuildPatterns": [ | ||
| "^circuits/cpp/barretenberg/cpp/" | ||
| ], |
There was a problem hiding this comment.
Incorrect formatting has been annoying me for a while so I've created an issue for this.
spalladino
force-pushed
the
palla/compiler
branch
from
4e9f49f to
ce9dd7b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Exposes a Noir contract compiler to end users via the CLI. Adds a new
compilecommand in the aztec-cli, also exposed via thenoir-compilerpackage asaztec-compile contract. This command compiles a noir contract using nargo or noir-wasm (defaulting to nargo until noir-lang/noir#2210 is fixed), then reformats it for Aztec consumption, and optionally generates typescript wrappers and noir interfaces. All these actions are also exported as functions from thenoir-compilerpackage, and are now used in thecopy-outputscript.This PR also makes a few changes to the generated typescript type:
creategets renamed toat(to make it clear we're not creating a new contract but creating an instance of the ts wrapper for a contract at an address) and the import path to the ABI is directly to the JSON file.We could add more (sub)commands just for generating the typescript or noir interfaces from the CLI, so it can be used in flows where nargo is used directly. We should also update the project README and add a page in the docs about how to compile one's own contracts (not sure if there's already one in the skeleton).
Fixes #1457