-
br-m
<jberman> Won't be able to make tomorrow's meeting. Update: we're squaring away final items for beta v3. The rebase onto master was massive and also exposed some issues we needed to solve. There is one remaining Mac specific issue causing failing tests in CI, and we're also looking to get all consensus changes included (including RandomX v2 into the v3). Beta v3 is very close
-
br-m
<rbrunner7> Meeting in a bit more than 1 hour
-
br-m
<jeffro256> Heads up, my internet is very spotty, but I should be present
-
br-m
<rbrunner7> Ok
-
br-m
<rbrunner7> Meeting time. Hello!
monero-project/meta #1459
-
br-m
<jpk68:matrix.org> Hello
-
br-m
<jeffro256> Howdy
-
br-m
<vtnerd> Hi
-
br-m
<sneedlewoods_xmr:matrix.org> Hey
-
br-m
<rbrunner7> Alright, on to the reports about last week. @jberman:monero.social can't attend, he left this before the meeting:
-
br-m
<rbrunner7> Update: we're squaring away final items for beta v3. The rebase onto master was massive and also exposed some issues we needed to solve. There is one remaining Mac specific issue causing failing tests in CI, and we're also looking to get all consensus changes included (including RandomX v2 into the v3). Beta v3 is very close
-
br-m
<jeffro256> Will tobtoht and tevador be present today, or are they still out?
-
br-m
<sgp_> Hello
-
br-m
<sneedlewoods_xmr:matrix.org> AFAICT fixed all the AI review comments I received (for 9464, 10232, 10233, 10819), now working on rebasing and fixing conflicts
-
br-m
<rbrunner7> @jeffro256:monero.social: Not sure. Are they expected to attend for something particular?
-
selsta
hi
-
br-m
<jeffro256> Yeah this week we did the fcmp++-stage rebase, and I'm trying to debug a weird Mac-specific bug, probably has something to do with mx25519, but I'm not sure
-
br-m
<rbrunner7> Me: The Polyseed PR was merged :)
-
br-m
<jeffro256> I'd like to discuss
monero-project/monero #11288 as it relates to merging a LGLP dependency
-
br-m
<jeffro256> *LGPL
-
br-m
<jpk68:matrix.org> Me: did a bunch of reviews, LLM testing for various patches, worked on improving fuzz testing coverage for multisig code, attempted to learn some math-related stuff (yikes)
-
br-m
<jeffro256> I see that Polyseed was merged. Good work rbrunner
-
selsta
tobtoht is not back to fully available yet
-
br-m
<rbrunner7> Thanks!
-
br-m
<jeffro256> So I guess tobtoht was okay with a copyleft dependency then?
-
br-m
<jpk68:matrix.org> Nope, it was re-licensed to Apache 2.0
-
br-m
<rbrunner7> What is this dependency?
-
br-m
<jeffro256> Ah I see
-
br-m
<jeffro256> Polyseed
-
br-m
<jeffro256> Used to be LGPL
-
br-m
<rbrunner7> Yup, that was a pleasant surprise
-
selsta
I ran a review of fcmp++-stage with latest OpenAI model, it found a couple things that appear serious but no inflation bug
-
br-m
<jeffro256> I'd take that report if that's alright
-
selsta
yes I can send it to you on matrix
-
br-m
<rbrunner7> The whole code? Didn't this burn through tokens like crazy?
-
br-m
<thomasbuilds:matrix.org> selsta: astra?
-
selsta
it was not a complete scan, it hits cyber warning quickly so I try to be careful with how I ask
-
selsta
yes
-
br-m
<jeffro256> cyber warning?
-
br-m
<rbrunner7> This AI stuff asks for brand new talents and capabilities :)
-
br-m
<rbrunner7> On the side of devs
-
br-m
<jpk68:matrix.org> It's like the Bat-Signal /s
-
selsta
cyber warning means it stops as soon as it hits anything related to cybersecurity or something that sounds scary
-
br-m
<jeffro256> isn't that the whole point of doing this?
-
selsta
so you basically have to be some "selected" person with whitelisted account to do proper audits
-
selsta
it still works but I would not call it complete
-
selsta
it triggers randomly
-
br-m
<rbrunner7> Splendid
-
br-m
<jeffro256> ah okay. so it's like KYC for a bug report exchange ;)
-
br-m
<thomasbuilds:matrix.org> selsta: aka only gov/cia or whatever can do proper audits to monero (I bet they already did)
-
br-m
<rbrunner7> Well, we still have paid competent humans to review the code, no?
-
br-m
<rbrunner7> Or better said audit
-
br-m
<thomasbuilds:matrix.org> @jeffro256: still limited, anthropic does so but just with opus 5, not fable, who is much more capable
-
br-m
<rbrunner7> Humans are still in the race
-
br-m
<thomasbuilds:matrix.org> thing is AI are capable to take a thing you think is fine/small and amplify it to a (big) vuln
-
br-m
<rbrunner7> I saw, with AI Polyseed PR reviews, that it's a pretty mixed bag. Sometimes you really don't know what do with some edge cases they come up with
-
selsta
jeffro256: sent everything to you, you can filter what is and what isn't interesting
-
br-m
<jeffro256> okay will do
-
br-m
<jeffro256> thanks
-
br-m
<rbrunner7> If you want to hunt down each and every last case we won't be able to finish larger PRs anymore ...
-
br-m
<rbrunner7> Problem is of course to see what is what
-
br-m
<rbrunner7> People seem to write longer comments ...
-
br-m
<thomasbuilds:matrix.org> yeah though I'm talking about like actual audits of codebase here, where goal are exploits
-
br-m
<thomasbuilds:matrix.org> it takes something you think is fine and creates the entire exploit chain
-
br-m
<thomasbuilds:matrix.org> ofc this is limited with guardrails...
-
br-m
<rbrunner7> Didn't want to claim in any way that these are easy questions. But as I said, if you want to react to everything, you won't finish anything anymore. AI reviews will turn into something like a DOS attack. Just if you near completition, throw another one at the dev.
-
br-m
<rbrunner7> And well, Monero is running for over a decade now without AI code reviews, and damned few exploits. That's not nothing.
-
br-m
<ofrnxmr> If you throw ai at it enough times, it starts coming up with unreachable or nonsense issues
-
br-m
<thomasbuilds:matrix.org> I mean rnn with AI there are so many more CVEs everyday, just look at linux
-
br-m
<ofrnxmr> its good as finding mistakes, but not gets real noisy after a couple rounds
-
br-m
<ofrnxmr> But it* gets
-
br-m
<rbrunner7> Yeah, for Polyseed I do not do some things, fully on purpose, as part of a design decision. AIs may now forever moan about that because they don't get the change.
-
br-m
<rbrunner7> I wonder whether there is a good way to write such things into comments which then the AIs read and "understand" ...
-
br-m
<ofrnxmr> I had an ai review of some of my work that claimed finding 1, then assumed finding 1 (wrong) would be fixed, and then pushed a finding 2 and 3 on the assumption that 1 would be changed to its suggestion
-
br-m
<thomasbuilds:matrix.org> though there are repos out there that are genuinely vuln free as in AI says "no issues"
-
br-m
-
br-m
<jpk68:matrix.org> Keep in mind, we still have a quarter or so left of the year...
-
br-m
<ofrnxmr> @jpk68:matrix.org: They added rust to the kernal in 2026 didnt they 🧠 /s
-
br-m
<rbrunner7> With that rate exploits will fill the known universe in 10 years :)
-
br-m
<rbrunner7> Anyway, interesting discussion, but is there still something important that belongs into this meeting proper?
-
br-m
<thomasbuilds:matrix.org> @thomasbuilds:matrix.org: (randomx, mx25519)
-
br-m
<thomasbuilds:matrix.org> in my testing
-
br-m
<ofrnxmr> (jpk, thats a joke saying that rust is why there is more cves)
-
br-m
<jpk68:matrix.org> @rbrunner7: XKCD 605 ;)
-
br-m
<jpk68:matrix.org> @ofrnxmr: Yes, I know, lol
-
br-m
<rbrunner7> I think I can close the meeting proper while discussions continue. Thanks everybody for attending, read you again next week!
-
br-m
<sneedlewoods_xmr:matrix.org> Thanks everyone
-
br-m
<thomasbuilds:matrix.org> btw I'd like to bring back the question on drahtbot for monero repo?
-
br-m
<jeffro256> nah I'm good, there's already way too many notifications on Github
-
br-m
<jeffro256> If no human deems it worth their time to ping someone to rebase a PR, it's not worth making a bot to do the pinging and spamming everyone involved
-
br-m
<thomasbuilds:matrix.org> all right also worth saying that the drahtbot is supposed to work based off ACK, Concept ACKs, NACK, etc
-
br-m
<thomasbuilds:matrix.org> To be fair these are used somewhat but never consitently
-
br-m
<ofrnxmr> The one feature i liekd about the bot, is that it knows when other open prs conflict
-
br-m
<thomasbuilds:matrix.org> I agree but a bot just for this seems off
-
br-m
<ofrnxmr> Thanks rbrunner
-
br-m
<jeffro256> Thanks everyone
-
br-m
<jpk68:matrix.org> I think it would be good to use "ACK commit-hash" rather than GH approvals, though DrahtBot does seem a bit unneeded
-
br-m
<thomasbuilds:matrix.org> draht bot would put that in stale ACK when commit changes, which is useful imo
-
br-m
<jpk68:matrix.org> It's just more stuff people have to worry about running, with very little benefit
-
br-m
<thomasbuilds:matrix.org> wdym?
-
br-m
<jpk68:matrix.org> Someone has to set up DrahtBot, no? And I think most people here seem to not agree that it'd be helpful
-
br-m
<thomasbuilds:matrix.org> yeah I understand what you meant now
-
br-m
<jpk68:matrix.org> @jeffro256:monero.social Looks like this links to the wrong PR:
-
br-m
-
br-m
<jpk68:matrix.org> Should be #10964 instead, I think
-
br-m
<thomasbuilds:matrix.org> wdym 10964 is merged
-
br-m
<jpk68:matrix.org> Are you sure?
-
br-m
<thomasbuilds:matrix.org> oops my bad
-
br-m
<thomasbuilds:matrix.org> I had that memory for some reason
-
br-m
<ack-j:matrix.org> I didnt make the meeting but wanted to chime in on the ai reviewing stuff since we (MAGIC) are doing a lot of it.
-
br-m
<ack-j:matrix.org> We have the “cyber” version of claude opus so it will create working exploits to prove vulns are reachable. (Very useful at ruling out false positives). This is what is running on the monero-review repo.
-
br-m
<ack-j:matrix.org> We can try to also get the “cyber” version for openai if needed.[... more lines follow, see
mrelay.p2pool.observer/e/5dHK-qwLcF9FM0RD ]
-
br-m
<jpk68:matrix.org> IMO, it is very useful :)