test(smoke): ERC-8056 scheduled-multiplier journey + advisory Vibenet workflow (BOP-473) - #182
Merged
Merged
Conversation
… workflow (BOP-473) Adds live-network smoke coverage for the AssetV2 @ Cobalt scheduled-multiplier surface (ERC-8056) and an advisory, manually-triggered CI workflow to run it. - New `multiplier` journey: setUIMultiplier scheduling + guards (InvalidMultiplier, EffectiveAtInPast/TooFar, ScheduleOverlap), cancelScheduledMultiplier (+ NoScheduledMultiplier), the updateMultiplier V2 event semantics (UIMultiplierUpdated + MultiplierUpdateCancelled, not MultiplierUpdated), the read aliases (uiMultiplier/balanceOfUI/totalSupplyUI), and ERC-165 advertisement. Scheduled state is asserted read-only (no time travel on a live chain); the lazy flip is opt-in via SMOKE_OBSERVE_FLIP (bounded real-time poll, soft-skips). - Fork-gated via supportsInterface(0xa60bf13d): cleanly SKIPS on a pre-Cobalt chain, mirroring the existing features-not-active preflight skip (no runtime fork selector). - Fix asset_lifecycle step-7 rebase event assertion to be fork-aware (V1 MultiplierUpdated vs Cobalt UIMultiplierUpdated) so it stays green on both forks. - Harness: add a Skip signal (recorded as skip, not fail), supports_erc165, and per-receipt assert_log / assert_no_log helpers. - Advisory workflow_dispatch-only smoke workflow (never gates merges; Vibenet is a live, per-hardfork chain). Reports pass/skip/fail + chain id / fork / base-std ref. Co-authored-by: Cursor <cursoragent@cursor.com>
The tidy-up cancelScheduledMultiplier raced maturation: on a chain that only advances its block clock when mining (e.g. anvil), the cancel tx mined a block whose timestamp jumped past effectiveAt, so the pending had matured and cancel reverted NoScheduledMultiplier — failing the (advisory) journey. The cleanup is non-essential (throwaway token), so drop it; the soft-skip now just logs and the journey stays green. Found while validating against a local ERC-8056 anvil. Co-authored-by: Cursor <cursoragent@cursor.com>
…review) Addresses review feedback (Stephan Ciliers): the smoke suite should run as a whole rather than exposing per-suite granularity, and the Vibenet workflow should always run the latest rather than offer fork/ref selection. - Makefile: remove the six per-journey targets; `make smoke` / `make smoke-all` now run the full suite in one process (KEEP_GOING=1 for the audit summary). The raw `python -m smoke <journey>` CLI is kept as a documented debugging escape hatch (cheaper/faster than the whole suite on a live chain), so nothing is lost. - Workflow: drop the `journey`, `fork`, and `base_std_ref` inputs (keep only `rpc_url`). It always runs `python -m smoke all -k` against the dispatched ref (latest = main; branch-per-fork for older forks) and reports per-journey passed/failed/skipped from the runner summary. No journey picker, no fork/ref selector — consistent with the repo's no-runtime-fork-selector model. - Docs: README + CLI docstring updated to match (full-suite entrypoint, CLI for ad-hoc single-journey debugging, single rpc_url workflow input). Co-authored-by: Cursor <cursoragent@cursor.com>
Mirrors the composite-policy smoke PR (#178): `assert_events_emitted` and the per-receipt `assert_log` are presence-only — an event with the right name but a wrong payload passes them. #178 added `composite_children_from` to decode the specific receipt; do the same here. - chain.py: add `event_args(receipt, contract, event_name)` to decode a single event from a specific receipt and return its args. - scheduled_multiplier: assert the scheduled/cancelled values from the event payloads — setUIMultiplier's UIMultiplierUpdated (old, target, effectiveAt) and the MultiplierUpdateCancelled payloads on cancel and on the updateMultiplier failsafe — which a presence-only check cannot verify. Validated green against a local ERC-8056 Cobalt anvil. Co-authored-by: Cursor <cursoragent@cursor.com>
robriks
requested review from
amiecorso,
eric-ships,
ilikesymmetry,
rayyan224 and
stevieraykatz
as code owners
July 28, 2026 21:11
Closed
3 tasks
Interface Coverage✅ All interface functions have test coverage. |
📊 Forge Coverage (
|
| File | Lines | Stmts | Branches | Funcs |
|---|---|---|---|---|
| 🔴 B20FactoryLib.sol | 95.40% | 96.00% | 100.00% | 90.00% |
| 🔴 test/lib/ForceFeeder.sol | 0.00% | 0.00% | 100.00% | 0.00% |
| 🔴 test/lib/PrecompileProbe.sol | 0.00% | 0.00% | 0.00% | 0.00% |
| 🟢 MockActivationRegistry.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockActivationRegistryStorage.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockB20.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockB20Asset.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟡 MockB20Factory.sol | 98.96% | 99.10% | 100.00% | 100.00% |
| 🟢 MockB20Stablecoin.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockB20Storage.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockPolicyRegistry.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| 🟢 MockPolicyRegistryStorage.sol | 100.00% | 100.00% | 100.00% | 100.00% |
| Total | 96.72% | 97.32% | 98.67% | 96.44% |
Full report: download artifact. To browse locally: make coverage (runs forge coverage + genhtml + opens the HTML report).
|
rayyan224
approved these changes
Jul 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds live-network smoke coverage for the ERC-8056 "Scaled UI Amount" scheduled-multiplier surface (AssetV2 @ Cobalt, shipped in base/base #4120/#4121 and base-std #173) plus an advisory, manually-triggered CI workflow to run it against Vibenet. This is the smoke-test sibling of the merged fork tests (BOP-432); BOP-473.
The existing
script/smoke/harness is extended in place — no parallel system.What's here
multiplierjourney (script/smoke/journeys/scheduled_multiplier.py), registered in__main__.py/Makefile(make smoke-multiplier). Exercises the full ERC-8056 surface:setUIMultiplierscheduling + guards:InvalidMultiplier(0 /> uint128.max),EffectiveAtInPast,EffectiveAtTooFar,ScheduleOverlap.cancelScheduledMultiplierclears a live pending (+MultiplierUpdateCancelled,effectiveAt()==0), andNoScheduledMultiplierwhen none is live.updateMultiplierV2 event semantics: emitsUIMultiplierUpdated(+MultiplierUpdateCancelledwhen clearing a live pending), not V1'sMultiplierUpdated— asserted with a negative check.uiMultiplier()==multiplier(),balanceOfUI==scaledBalanceOf,totalSupplyUI==toScaledBalance(totalSupply()).0x01ffc9a7+ the three ERC-8056 ids (0xa60bf13d,0x4bd27648,0xd890fd71); an unadvertised id reads false.supportsInterface(0xa60bf13d)and cleanly SKIPS on a pre-Cobalt chain (recorded as skip, not fail) — mirroring the existingfeatures_activatedpreflight skip. One journey stays correct on both a Beryl-Vibenet (V1, skips) and a Cobalt-Vibenet (V2, runs).asset_lifecycle. Its flow-level event check asserted V1MultiplierUpdated, which AssetV2 no longer emits (the current ABI doesn't even declare it) — so it would fail on Cobalt. The rebase-event assertion is now fork-aware (V1MultiplierUpdatedvs CobaltUIMultiplierUpdated) via the same probe.effectiveAt, current multiplier unchanged). Observing the lazy flip is opt-in viaSMOKE_OBSERVE_FLIP=1(bounded real-time poll; soft-skips on timeout so the advisory run stays green).chain.py): aSkipsignal (recorded as skip),supports_erc165(treats an absent/reverting selector as "not advertised"), and per-receiptassert_log/assert_no_log..github/workflows/smoke-tests.yml):workflow_dispatchonly — not PR/merge_group/schedule, because Vibenet is a live, per-hardfork chain and an automated run would flap. Inputs:journey,fork,base_std_ref(latest=main; pin a prior hardfork's ref — branch-per-fork),rpc_url(defaulthttps://rpc.vibes.base.org/). ExportsRPC_URLfrom the input andDEPLOYER_PK/USER2_PKfrom repo secrets (SMOKE_DEPLOYER_PK/SMOKE_USER2_PK); reports pass / skip / fail + chain id / fork / base-std ref. Never gates merges.Testing
forge fmt --check,forge build, andforge test(reference/unit) all green (689 passed). Python imports + ABI/error/interface-id references validated statically.Ran the suite against a local
anvil --base(Cobalt,BaseUpgrade::Cobalt) built from base-anvil-fork patched onto base/base @ the ERC-8056 merge, per the b20-fork-testing runbook:multipliervs ERC-8056 Cobalt anvilmultipliervs pre-ERC-8056 chain (Cobalt shell, V1 asset)assetvs ERC-8056 Cobalt anvilUIMultiplierUpdated, 14 events)assetvs pre-ERC-8056 chainMultiplierUpdated)--block-time 2uiMultiplier/newUIMultipliermirror the matured valueTest plan (reviewer)
workflow_dispatch-only trigger and that the job is advisory (non-gating).SMOKE_DEPLOYER_PK/SMOKE_USER2_PKrepo secrets exist before dispatching.journey=multiplier,fork=cobaltonce Vibenet is on Cobalt; expect pass (or a clean "skipped" if not yet).Made with Cursor
Re-homed from a
base/base-stdbranch (was previously a fork PR, whose read-only token 403'd on the Forge/Interface Coverage "post PR comment" steps — the actual build/test/coverage checks passed). Same commits: the ERC-8056 journey + advisory workflow, the lazy-flip fix, the "run the full suite / drop selectors" review change, and event-payload decoding aligned with #178.