Skip to content

feat(bin): no mandatory signer on eth connections #60

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
guidiaz merged 4 commits into from
Mar 2, 2026
Merged

feat(bin): no mandatory signer on eth connections #60

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
eb7d0ff
feat(bin): no mandatory signer on eth connections
guidiaz Mar 2, 2026
666a266
fix(cli): try to set signer if only update conditions are to be settled
guidiaz Mar 2, 2026
a13d704
chore: pnpm update
guidiaz Mar 2, 2026
86f07df
chore: this is 3.2.0
guidiaz Mar 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,8 @@
"pfs:check": "node ./scripts/rulebook.cjs",
"pfs:notarizer": "node src/bin/bots/notarizer.cjs",
"pfs:reporter": "node src/bin/bots/reporter.cjs",
"pfs:settle": "npm run build && node ./scripts/settle.cjs",
"pfs:status": "npm run build && node ./scripts/status.cjs",
"pfs:settle": "npm run build && node src/bin/cli/settle.cjs",
"pfs:status": "npm run build && node src/bin/cli/status.cjs",
"prepare": "npm run build",
"test": "vitest"
},
Expand Down
68 changes: 34 additions & 34 deletions src/bin/bots/reporter.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,6 @@ async function main() {
"ETH/RPC provider host",
process.env.WITNET_PFS_ETH_RPC_PROVIDER_HOST || "http://127.0.0.1",
)
.option(
"--patron <evm_address>",
"Signer address that will pay for every update report, other than the gateway's default.",
process.env.WITNET_PFS_ETH_SIGNER || undefined,
)
.option(
"--port <url>",
"ETH/RPC provider port",
Expand All @@ -84,7 +79,12 @@ async function main() {
"Make sure the randomizer bot connects to this EVM chain.",
process.env.WITNET_PFS_ETH_NETWORK || undefined,
)
.requiredOption(
.option(
"--signer <evm_address>",
"Signer address that will pay for every update report, other than the gateway's default.",
process.env.WITNET_PFS_ETH_SIGNER || undefined,
)
.option(
"--target <evm_address>",
"Address of WitPriceFeeds contract where to report data updates.",
process.env.WITNET_PFS_ETH_TARGET || undefined,
Expand All @@ -101,49 +101,45 @@ async function main() {
maxGasPriceGwei,
network,
port,
patron,
signer,
target,
} = program.opts();

if (!debug) console.debug = () => {};

if (patron && !ethers.isAddress(patron)) {
console.error(`❌ Fatal: invalid EVM signer address: "${patron}"`);
if (signer && !ethers.isAddress(signer)) {
console.error(`❌ Fatal: invalid EVM signer address: "${signer}"`);
process.exit(0);
} else if (!target || !ethers.isAddress(target)) {
} else if (target && !ethers.isAddress(target)) {
console.error(`❌ Fatal: invalid EVM target address: "${target}"`);
process.exit(0);
} else if (gasLimit && isNaN(Number(gasLimit))) {
console.error(`❌ Fatal: invalid gas limit: "${gasLimit}"`);
process.exit(0);
}

const witOracle = patron
? await WitOracle.fromJsonRpcUrl(`${host}:${port}`, patron)
: await WitOracle.fromJsonRpcUrl(`${host}:${port}`);
const signer = witOracle.signer.address;
const kermit = await Witnet.KermitClient.fromEnv(KERMIT);

console.info(`> WIT/Kermit URL: ${kermit.url}`);
console.info(`> EVM RPC gateway: ${host}:${port}`);

if (network && witOracle.network !== network) {
console.error(
`❌ Fatal: connected to ${witOracle.network.toUpperCase()} instead of ${network.toUpperCase()}`,
);
process.exit(0);
} else {
network = witOracle.network;
const witOracle = await WitOracle.fromEthRpcUrl(`${host}:${port}`);
if (network && network !== witOracle.network) {
console.error(`Fatal: connected to wrong network: ${network.toUpperCase()}`);
process.exit(1);
}
network = witOracle.network;
console.info(`> EVM network: ${network.toUpperCase()}`);
console.info(`> Wit/Oracle bridge: ${witOracle.address}`);

const witOracleRadonRegistry = await witOracle
.getWitOracleRadonRegistry()
const witOracleRadonRegistry = await witOracle._getWitOracleRadonRegistry()
.catch((err) => {
console.error(`❌ Fatal: cannot fetch Wit/Oracle Radon Registry: ${err}`);
process.exit(0);
});
console.info(`> Wit/Oracle registry: ${witOracleRadonRegistry.address}`);

const kermit = await Witnet.KermitClient.fromEnv(KERMIT);

console.info(`> WIT/Kermit URL: ${kermit.url}`);
console.info(`> EVM RPC gateway: ${host}:${port}`);
console.info(`> EVM network: ${network.toUpperCase()}`);

const witPriceFeeds = await witOracle
.getWitPriceFeedsAt(target)
const witPriceFeeds = await witOracle._getWitPriceFeeds(target)
.catch((err) => {
console.error(`❌ Fatal: ${err}`);
process.exit(0);
Expand All @@ -160,13 +156,17 @@ async function main() {
console.error(`❌ Fatal: unsupported WitPriceFeeds version: ${serial}`);
process.exit(0);
}

console.info(`> Wit/Oracle bridge: ${witOracle.address}`);
console.info(`> Wit/Oracle registry: ${witOracleRadonRegistry.address}`);
console.info(
`> Wit/Oracle appliance: ${target} [${artifact} v${serial.split("-")[0]}]`,
);

try {
signer = (await witPriceFeeds.setSigner(signer)).address;
} catch (err) {
console.error(`❌ Fatal: failed to set signer to ${signer}: ${err.message}`);
process.exit(0);
}

const { provider } = witOracle;
const symbol = utils.getEvmNetworkSymbol(network);
balance = await provider.getBalance(signer);
Expand Down
Loading