-
Notifications
You must be signed in to change notification settings - Fork 179
interop: supervisor_dependencySet API #684
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.
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,7 @@ | |
| - [`SupervisorChainSyncStatus`](#supervisorchainsyncstatus) | ||
| - [`SuperRootResponse`](#superrootresponse) | ||
| - [`SafetyLevel`](#safetylevel) | ||
| - [`DependencySetConfigV1`](#dependencysetconfigv1) | ||
| - [Methods](#methods) | ||
| - [`supervisor_crossDerivedToSource`](#supervisor_crossderivedtosource) | ||
| - [`supervisor_localUnsafe`](#supervisor_localunsafe) | ||
|
|
@@ -31,6 +32,7 @@ | |
| - [`supervisor_finalizedL1`](#supervisor_finalizedl1) | ||
| - [`supervisor_superRootAtTimestamp`](#supervisor_superrootattimestamp) | ||
| - [`supervisor_syncStatus`](#supervisor_syncstatus) | ||
| - [`supervisor_dependencySetV1`](#supervisor_dependencysetv1) | ||
| - [`supervisor_allSafeDerivedAt`](#supervisor_allsafederivedat) | ||
| - [`supervisor_checkAccessList`](#supervisor_checkaccesslist) | ||
| - [Access-list contents](#access-list-contents) | ||
|
|
@@ -121,7 +123,8 @@ Regular JSON number, always integer. Assumed to always fit in 51 bits. | |
| #### `ChainID` | ||
|
|
||
| `STRING`: | ||
| Hex-encoded big-endian number, variable length up to 256 bits, prefixed with `0x`. | ||
| Decimal-encoded string, variable length up to 256 bits. | ||
| This supports decoding of hex-encoded values when prefixed with `0x`. | ||
|
|
||
| #### `Hash` | ||
|
|
||
|
|
@@ -204,6 +207,19 @@ Corresponds to a verifier [SafetyLevel](./verifier.md#safety). | |
| - `safe`: matching cross-safe, named `safe` to match the RPC label. | ||
| - `finalized` | ||
|
|
||
| #### `DependencySetConfigV1` | ||
|
|
||
| Defines an interop dependency set. | ||
| Includes information to determine the active set of chains at any particular block time. | ||
|
|
||
| This type is versioned: the format of this configuration may change over time. | ||
|
|
||
| `OBJECT`: | ||
| - `dependencies`: `MAP`: | ||
| - key: `ChainID` | ||
| - value: `ConfigDependency`: `OBJECT`: empty object. This may be extended in the future. | ||
| - `overrideMessageExpiryWindow`: `uint64`: changes the default protocol expiry window time. | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So this isn't something we only use in tests, but want to have in the specs? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I already suggested to change it to just be |
||
|
|
||
| ### Methods | ||
|
|
||
| #### `supervisor_crossDerivedToSource` | ||
|
|
@@ -257,6 +273,16 @@ Parameters: (none) | |
|
|
||
| Returns: `SupervisorSyncStatus` | ||
|
|
||
| #### `supervisor_dependencySetV1` | ||
|
|
||
| Retrieves the full dependency set configuration, | ||
| to determine which chains are available at any particular block. | ||
| The returned configuration may change with new chain upgrades. | ||
|
|
||
| Parameters: (none) | ||
protolambda marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| Returns: `DependencySetConfigV1` | ||
|
|
||
| #### `supervisor_allSafeDerivedAt` | ||
|
|
||
| Returns the last derived block for each chain, from the given L1 block. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sadiq1971