Skip to content

removes hard "privy" dependencies from molecule / aura skill#1

Open
elmariachi111 wants to merge 5 commits into
mainfrom
arch/drop-hard-privy-requirement
Open

removes hard "privy" dependencies from molecule / aura skill#1
elmariachi111 wants to merge 5 commits into
mainfrom
arch/drop-hard-privy-requirement

Conversation

@elmariachi111

@elmariachi111 elmariachi111 commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator
  • adds reference to agentic wallet / copies Privy's original skill

- adds agentic wallet reference

Signed-off-by: Stefan Adolf <stefan@molecule.to>
Signed-off-by: Stefan Adolf <stefan@molecule.to>
- documents mcp setup

Signed-off-by: Stefan Adolf <stefan@molecule.to>
Signed-off-by: Stefan Adolf <stefan@molecule.to>
Signed-off-by: Stefan Adolf <stefan@molecule.to>
@elmariachi111 elmariachi111 marked this pull request as ready for review June 29, 2026 11:00
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@elmariachi111

Copy link
Copy Markdown
Collaborator Author

I ran the flow with an hermes and an openclaw agent. These changes mark "Privy" fully optional (but recommended). I tested this with a self maintained wallet and a full privy "agentic wallet" onramp (it mustn't be a concern of this skill, consumers must never be forced to use some service provider or wallet when interacting with a sufficiently permissionless protocol).

@elmariachi111

Copy link
Copy Markdown
Collaborator Author

IPNFT minted with hermes / using local credentials: https://testnet.molecule.xyz/ipnfts/73429765202772756150668042846820572544867199961301927641201490145477240372754

IPNFT minted with openclaw / privy agent wallets:
https://testnet.molecule.xyz/ipnfts/103201780770515326590323307614599594043668087674535376195550587694540778425040

both URLs are created and don't work - that's 100% out of this PR's scope. The IPNFTs are on chain, the "labs" are created, all EIP-712 signatures worked, x402 payments worked.

Ideally incorporate these changes with your upcoming PR on the new lab creation flow ( #2 )

@DemidovVladimir

DemidovVladimir commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

If it would then the links should also work. How did you validated that the labs were created? Also can you check file upload please? If labs would be created the the file upload would also work.
I created a new PR for OCL changes. I thought we discussed that we will wait until OCL is in place, before adding some changes. Because OCL PR should have some adjustments to optional privy wallet usage.

@elmariachi111

Copy link
Copy Markdown
Collaborator Author

I started this already on Thursday. If you remove privy as hard dependency I'm fine. On your PR I see that the env vars are still mandatory. Your staging backend does reply with hard error messages, I filed them. The most recent one being that my token isn't "whitelisted". that's imo completely out of scope of this pr. If you don't like it never mind, merge yours first and apply my original prompt again if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants