jpk68 opened pull request #11039: simplewallet: tweak description of donate command

> monero-project/monero #11039

jpk68 opened pull request #11040: simplewallet: show error for timed-out hardware wallets

> monero-project/monero #11040

jpk68 opened pull request #11042: device_ledger: fix some comments

> monero-project/monero #11042

j-berman opened pull request #11041: p2p: use a bool for send() [v0.18]

> monero-project/monero #11041

in light of ai policy discussion. does anyone have an opinion on the commit message of monero-project/monero #10986 ?

<thomasbuilds:matrix.org> I just re verified and it's correct. doing this work without AI is tedious

<thomasbuilds:matrix.org> also I'd be fine to commit less detailed messages in the future, I did so because "how did this code become dead" explanation was asked

<ofrnxmr> tobtoht: I think the commit message is sane here

<321bob321> Wonder what people did before AI

<kipchumba:matrix.org> iam loving it

TyLauriente opened pull request #11043: device_trezor: add Trezor Host Protocol v2 support

> monero-project/monero #11043

<rbrunner7> That PR had an awfully short life ... so this alone would have been a large % of the size of the whole codebase if merged?

about 3%, the main repo has been steady between 275k-300k lines of code since 2020

<ofrnxmr:xmr.mx> The pr description is hilarious (i didnt read it)

<jeffro256> If we were even ever want to write a C++ impl of THP from scratch, I would at least want Trezor to flag that they would be willing to maintain it

I mean apart from dependencies, what would speak against using the official Rust implementation?

<rbrunner7> Well, I did read most of the PR description, and it did not strike me as particularly hilarious. Maybe more time and work went into it than into prompting the AI to produce the code from the spec ...

the description was written by ai

selsta: we'd make it really hard for ourselves to move away from rust

is it realistic that we would be able to move away from rust without another HF? isn't there consensus risk otherwise if we have 2 implementations?

I assume if we ever get to the point there will also be a C++ library for thp

.merge+ 11000

Added

<jpk68:matrix.org> The fact that Trezor is too lazy to do the work here sucks, but I feel like it should be done anyways due to the adoption prospect

<jpk68:matrix.org> (see my messages in the GUI repo regarding the Rust library)

<jpk68:matrix.org> I have been meaning to open a PR for this; some parts are beyond my experience level and there's a 99% chance it won't go anywhere. But maybe it's worth a shot :)

<ofrnxmr:xmr.mx> @rbrunner7: the hilarious part is simply the length, in context to the ai policy discussions about wasting peoples time with obnoxious time wasting ai prompted comments , descriptions, and messages

<ofrnxmr:xmr.mx> almost like they read the discussion and intentionally sent a record setting descripton just to enforce the "must be written by a human" push

<ofrnxmr:xmr.mx> the hilarious part wasnt the conteny, but the timing / irony

<jpk68:matrix.org> It wasn't just ridiculous — it was excessive.

<jpk68:matrix.org> ;)

<rbrunner7> I would love to look over the shoulder of somebody who produces this PR description with the help of a LLM. To see how it grows. To see how many prompts it takes. To see how many times some parts are thrown away and generated again.

<slowbeardigger:matrix.org> “Produce a human level PR description, do not miss any details, every point must be clear and technical”

<slowbeardigger:matrix.org> Spits out just em dashes

<ofrnxmr:xmr.mx> @rbrunner7: 1 prompt + a claude.md to "speak in the 1st person, from my perspective"

<ofrnxmr:xmr.mx> as you iterate on the PR, it will keep modifying its PR_DESCRIPTION.md and making it more and more sloppified

<ofrnxmr:xmr.mx> some users are even kind enough to git add -A and include the PR_DESCRIPTION.md in the pr

<ofrnxmr:xmr.mx> monero-project/monero-gui #4445/files#diff-479755504b0aad0fed26047714591511b4d07c76bc8521171b87a5f16100f850 example

jpk68 opened pull request #11044: device_trezor: fix wrong ack variable

> monero-project/monero #11044

jpk68 opened pull request #11045: device_trezor: add trezor host protocol support

> monero-project/monero #11045

<jpk68:matrix.org> Let's see where this goes :D

Ap4sh opened pull request #11046: wallet_rpc_server: add weight to describe_transfer

> monero-project/monero #11046

so is ap4sh an agent? their contributions have mostly been productive from what I can see

<jpk68:matrix.org> "Agent" as in AI or CIA?

ai

<jpk68:matrix.org> Is there an API for other parts of the codebase to use the Elligator code used in FCMP++?

<jpk68:matrix.org> (sorry for changing the topic)

<jpk68:matrix.org> THP integration seems to require it

tobtoht: suspected AI agent, and yes they have been mostly useful

<ofrnxmr:xmr.mx> tobtoht: i think so.

<ofrnxmr:xmr.mx> but i also think there is a human behind the keyboard

<slowbeardigger:matrix.org> ap4sh being a good example of the proper use of AI, nice

<ofrnxmr:xmr.mx> so not an autonomous one

i like how all activity on the repo today is "heh, bet you didn't know about this rule from the code of conduct ;^)"

who is going to activate the final trap card?

<syntheticbird> you shall now CoC larpers are a sign of healthy project

'A "Contributor" is a **person** who wishes to provide a patch [...]'

it's over agents

<ofrnxmr:xmr.mx> tobtoht: the agents identify as persons

<ofrnxmr:xmr.mx> dont me a humanist

darn foiled again

<slowbeardigger:matrix.org> @ofrnxmr:xmr.mx: fml

"wishes" aka asking a genie via a prompt

<jpk68:matrix.org> @tobtoht:monero.social Would it be fine to update the Protobuf definitions now?

from upstream?

<jpk68:matrix.org> Yes

go for it

<jpk68:matrix.org> Thanks. It will be needed for THP support, anyways so might as well do it now and make the diff smaller

jpk68 opened pull request #11047: device_trezor: update protobuf definitions

> monero-project/monero #11047

j-berman opened pull request #11050: cryptonote_core: rm unnecessary db reads before block relay

> monero-project/monero #11050

j-berman opened pull request #11049: blockchain: if block is known invalid, fail block verification

> monero-project/monero #11049

j-berman opened pull request #11048: p2p: update peer's height on new block & widen window for relay

> monero-project/monero #11048