Closed CodeDragonVN closed 6 months ago
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@ethereumjs/util@9.0.0, npm/@nomicfoundation/hardhat-chai-matchers@1.0.6, npm/@nomicfoundation/hardhat-network-helpers@1.0.8, npm/@nomicfoundation/hardhat-toolbox@2.0.2, npm/@nomiclabs/hardhat-ethers@2.2.3, npm/@nomiclabs/hardhat-etherscan@3.1.7, npm/@openzeppelin/contracts@4.9.3, npm/@typechain/hardhat@6.1.6, npm/@types/node@18.17.4, npm/@types/react-dom@18.2.7, npm/@types/react@18.2.19, npm/axios@1.4.0, npm/blake2@5.0.0, npm/dotenv@16.3.1, npm/hardhat-gas-reporter@1.0.9, npm/hardhat@2.17.1, npm/keccak256@1.0.6, npm/next@13.4.19, npm/react-dom@18.2.0, npm/react-loader-spinner@5.3.4, npm/react-toastify@9.1.3, npm/react@18.2.0
🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎
To accept the risk, merge this PR and you will not be notified again.
Alert | Package | Note |
---|---|---|
Install scripts | npm/bigint-buffer@1.1.5 |
|
Install scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts.
Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.
Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.
If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.
To ignore an alert, reply with a comment starting with @SocketSecurity ignore
followed by a space separated list of ecosystem/package-name@version
specifiers. e.g. @SocketSecurity ignore npm/foo@1.0.0
or ignore all packages with @SocketSecurity ignore-all
@SocketSecurity ignore npm/bigint-buffer@1.1.5
Hey @CodeDragonVN im tried to make this work and was blocked since backend_barretenberg 0.24.0 points to bb.js 0.23.0 which causes serialization issues.
Did you find any such problems?
Ok so I stopped being lazy and checked this out, indeed I get the same serialization error:
Error: Circuit execution failed: Error: Failed to deserialize circuit. This is likely due to differing serialization formats between ACVM_JS and your compiler
I'm glad to merge this branch though instead of my own branch because this frontend looks sweet! Let's just wait for 0.24.1
Hey @signorecello,
I'm running into the serialization error you pointed out, and it's perplexing because it was previously resolved on my end. Testing now with the backend_barretenberg
at version 0.24.0 but using the recursion tests from the noir-lang/noir repository, everything still passes without issue however.
I'm finding it perplexing how It's working on my repo here: https://github.com/CodeDragonVN/noir-recursion-v0.24.0, but now it's giving me that serialization issue on this repo. The version of bb.js is also 0.23.0 on my repo.
I've reopened the issue about the serialization error since it has resurfaced. I'm encountering it again, similar to before. My repo https://github.com/CodeDragonVN/noir-recursion-v0.24.0 shows no issues, but the serialization error persists in this noir-examples repo. The issue is Issue #4459
Thanks @CodeDragonVN this is a known problem, caused by feedback loop from aztec-packages repo that wasn't included in 0.24.0 by mistake.
This is resolved in a next version 0.24.1 so it's just a matter of waiting for it. One way to check it will work correctly is just to use nightlies.
Let's hold on this PR until then, I'll let you know once 0.24.1 is out and you can update this PR to reflect that, then we merge 👍
I get a bit distracted with so many PRs though so please tag me here, or on discord, if I fail to see changes!
@signorecello Appreciated. I'll keep a close watch for the 0.24.1 release and update also. For now, I'll close the issue.
hey @signorecello,
I've got good news! After updating the packages to v0.25.0 the deserialization issue is gone.
@noir-lang
packages in package.json
to the latest version 0.25.0.Yep this one looks good, let's merge :)
@SocketSecurity ignore npm/bigint-buffer@1.1.5
When running nargo prove
in recursion/packages/noir/recursion on Nargo v0.25.0, I'm getting this error:
The parameter proof is expected to be a Array { length: 93, typ: Field } but found incompatible value Vec([Field(2), Field(10824393579292640276679628408780923644399), Field(245259312760651750434636148266637791), Field(16220608892709085881729549461059840933531), Field(29086728926417763668684368412851575), Field(86960175229511384923347249685703605573976), Field(119632588394922407903828536975742837), Field(8133670842954062125526877195552803097095), Field(77866749597867336167111883273880859), Field(71471231940114255485442979054702681468666), Field(23764813297134295735649906350412067), Field(7995811797098406394649926543277505625825), Field(129596692976341405364457231933926819), Field(42584003987106105622750099282556088552981), Field(83704113108611586325261273358909475), Field(23381474794378998045519623024244166683364), Field(58281244622276626442984716163814997), Field(29804766582689924054821691856432438879238), Field(57444262120063519768548212507957905), Field(16491241657511778684900216372384253634593), Field(41953517797133300127696735937977142), Field(68151564160769547547256134235862571492297), Field(216407266845215061217912493787155913), Field(69207649048610159988269278414443034093515), Field(15807053400682008906696955367456414), Field(10880481356747158641109320953091993808257), Field(2349200668467182599985201040167423), Field(31364921378378437615613880362128784294056), Field(39443454228419067228006125108034268), Field(20581768292632203344331303479038549047339), Field(102061500812406260120526807869958311), Field(24236701373264253026159203273786348563223), Field(7160361939492749527211926769788388), Field(63868105380635015923004029232685365912965), Field(238542531654646981940655082881081771), Field(5090616018687404140975576095173384749885), Field(43017538151095093102124813241236341), Field(4259038585770966974321899586833487122888), Field(114227111701630081242643662775400230), Field(68251179676788864790405053043314937453270), Field(9692402620334585232468065140487021), Field(2847735185949821907881170022267580238807), Field(120156717726338775729839371898339058), Field(14121884846476890623069165971469385506920), Field(83592227982249769367956318927602374), Field(10989296828477066965048323571107898543133799215759153574795593757312752813519), Field(1692002355413023306367267213642771784531727362864458766547849688766899694066), Field(3798529920491737195625363558999108934776211473679815282888537467136909974482), Field(-256275798624413393622062906902542915583317396439971975433346841900939843487), Field(8322388201023808012870399100180878830759974505798980413797440919580578045873), Field(-331192619096629662370620122212819689390196235627555437196146139630222834199), Field(2326251939241883614051705621367670708421552049794746171429699716122070839702), Field(-7868490775523227390360146959476593742454345219540388306194662761959805805275), Field(-4837239561612933720655417249630881609435728158302830702586561038269749529928), Field(-7059937854679246682309316952465684718563777857501408377784875648693204430077), Field(6421501004493454161172102601980226091172351583430306948019244908204961232450), Field(4852473425435083078587353700659053667116259568488230291627352583170990936124), Field(5433448795313040042239951003824916154714225490913731149207937064525935516374), Field(-3142739218127369117205097167650835598608566413466050106336386581732664370231), Field(8562001339324605548229470135973634788229802111240409264025659877606614976600), Field(9632251506740181241758153902970339136758527375145460422028867362307441848675), Field(10702501674155756935286837669967043485287252639050511580032074847008268720750), Field(-6875404873706023841285549468457545876118998885813823945980951136710009668145), Field(-2194778855981724790525730095918588475642910296457745123471813052655564156776), Field(3246089640769931248471626203364017474211177134741223905869740628722309947987), Field(8017291352040390036011586600418057224606339042029062360308162719474465628451), Field(-9045240862852366899902200541296822906203661233555420447659714370165886030717), Field(-7974990695436791206373516774300118557674935969650369289656506885465059158642), Field(-6904740528021215512844833007303414209146210705745318131653299400764232286567), Field(-5834490360605639819316149240306709860617485441840266973650091916063405414492), Field(11772751841571332628815521436963747833815977902955562738035282331709095592825), Field(2⁴×515015349249856027793884090616948420426494535406831519958183777312496410329), Field(-655027780306546841755476148175915721013996680692586176295616779147906181421), Field(-8176815891054077554202674580291235244747826429980231733892916755568034816733), Field(148798921886863903132781229482758217694), Field(62863334824747313451323085736833590057545117208672818611185365657622432422), Field(-9062027930855658129182172085881471197837145603556764231416228119122676787706), Field(4388110873538727912125896417094102125704040646884279841003347343229956208498), Field(-5850461569401064351417303611077900569291818716207946797419492139347994041222), Field(3842724590614690011918435231717575290652682634947443304006488952910782196646), Field(-8698646621100834496777834498192652591855389177989914219131719387988111437188), Field(4473383684656860969041250902476789967434903924167511947768870312280571111410), Field(-8784184561544884307705408906719145657180879950466192276174921944567370067929), Field(-2⁴×24756171567120416088220245109700615107341757051980874332202939304670668776), Field(7991987071397030992882361063208725973745943724802804297544427886817908667097), Field(-5508169983971286579070159697084613299339008837978731759294101384065260461007), Field(36992940453337112128528440701242819743352), Field(117689838452265778650879227913523939), Field(64927607873248691436990295648907245579578), Field(241691637338611060083000632953347194), Field(17880647516323685297027980762545736722513), Field(199138603335721403350161534864642739), Field(83146432756370824696010155233778509844385), Field(2⁴×7388079081685829759436069250324641)])
Is that expected?
Description
Problem*
This Pull Request updates the recursion example to be compatible with Noir v0.24.0, addressing breaking changes introduced in the update. It resolves the issue described in: Update recursion to Noir v0.24.0 #16.
Summary*
The PR makes several changes to provide compatibility with Noir v0.24.0:
generateIntermediateProof
togenerateProof
,verifyIntermediateProof
toverifyProof
, etc.).yarn dev localhost
to enhance user experience.#[recursive]
attribute and updatedNargo.toml
files in line with Noir v0.24.0 requirements.Additional Context
PR Checklist*
cargo fmt
on default settings.