Spell 2023-10-11

[0xdecr1pto]

Description

Contribution Checklist

• [ ] PR title starts with (PE-<TICKET_NUMBER>)
• [ ] Code approved
• [ ] Tests approved
• [ ] CI Tests pass

Checklist

• [ ] Every contract variable/method declared as public/external private/internal
• [ ] Consider if this PR needs the officeHours modifier override
• [ ] Verify expiration (30 days unless otherwise specified)
• [ ] Verify hash in the description matches here
• [ ] Validate all addresses used are in changelog or known
• [ ] Notify any external teams affected by the spell so they have the opportunity to review
• [ ] Deploy spell ETH_GAS_LIMIT="XXX" ETH_GAS_PRICE="YYY" make deploy
• [ ] Verify mainnet contract on etherscan
• [ ] Change test to use mainnet spell address and deploy timestamp
• [ ] Run make archive-spell or make date="YYYY-MM-DD" archive-spell to make an archive directory and copy DssSpell.sol, DssSpell.t.sol, DssSpell.t.base.sol, and DssSpellCollateralOnboarding.sol
• [ ] squash and merge this PR

[amusingaxl]

Good to deploy :shipit:

Mainnet Executive Spell Review Checklist

Mainnet 2023-10-11

Spell Actions (Per Exec Doc):

• [x] Non-Scope Defined Parameter Changes - WBTC DC-IAM Changes
  • [x] Reduce the WBTC-A DC-IAM Target Available Debt from 10 million DAI to 2 million DAI.
  • [x] Reduce the WBTC-B DC-IAM Target Available Debt from 5 million DAI to 2 million DAI.
  • [x] Reduce the WBTC-C DC-IAM Target Available Debt from 10 million DAI to 2 million DAI.
• [x] Stability Fee Changes
  • [x] Increase the ETH-A Stability Fee (SF) by 1.55%, from 3.70% to 5.25%.
  • [x] Increase the ETH-B Stability Fee (SF) by 1.55%, from 4.20% to 5.75%.
  • [x] Increase the ETH-C Stability Fee (SF) by 1.55%, from 3.45% to 5.00%.
  • [x] Increase WBTC-A Stability Fee (SF) by 0.06%, from 5.8% to 5.86%
  • [x] Increase WBTC-B Stability Fee (SF) by 0.06%, from 6.30% to 6.36%.
  • [x] Increase WBTC-C Stability Fee (SF) by 0.06%, from 5.55% to 5.61%.
• [x] Initial RETH-A Offboarding
  • [x] Set rETH-A Debt Ceiling (line) to 0 (zero).
  • [x] Remove RETH-A from Autoline.
• [x] Reconfiguring Andromeda RWA015-A
  • [x] Set the Maximum Debt Ceiling (line) to 3 billion DAI.
• [x] Reconfiguring Clydesdale RWA007-A
  • [x] Reactivate the Debt Ceiling Instant Access Module for this vault type.
  • [x] Set the Target Available Debt (gap) to 50 million DAI.
  • [x] Set the Ceiling Increase Cooldown (ttl) to 86400 (24 hours).
  • [x] Set the Maximum Debt Ceiling (line) to 3 billion DAI.
• [x] Set up Governance Facilitator Streams
  • [x] JanSky | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 504,000.00 DAI | 0xf3F868534FAD48EF5a228Fe78669cf242745a755
  • [x] VoteWizard | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 504,000.00 DAI | 0x9E72629dF4fcaA2c2F5813FbbDc55064345431b1
  • [x] JanSky | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 216.00 MKR | 0xf3F868534FAD48EF5a228Fe78669cf242745a755
  • [x] VoteWizard | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 216.00 MKR | 0x9E72629dF4fcaA2c2F5813FbbDc55064345431b1
• [x] USDP-PSM Facilitation Incentives
  • [x] DAO Resolution 1: Initial Setup (copy only)
  • [x] Add dao_resolution parameter: QmWg43PNNGfEyXnTv1qN8dRXFJz5ZchrmZU8qH57Ki6D62
• [x] BA Labs MKR Distribution
  • [x] BA Labs - 175 MKR - 0x5d67d5B1fC7EF4bfF31967bE2D2d7b9323c1521c
• [x] AVC Member Compensation
  • [x] opensky - 20.85 MKR - 0x8e67ee3bbeb1743dc63093af493f67c3c23c6f04
  • [x] DAI-Vinci - 12.51 MKR - 0x9ee47F0f82F1A6F45C4E1D25Ce95C321D8C8356a
  • [x] IamMeeoh - 20.85 MKR - 0x47f7A5d8D27f259582097E1eE59a07a816982AE9
  • [x] ACRE DAOs - 20.85 MKR - 0xBF9226345F601150F64Ea4fEaAE7E40530763cbd
  • [x] Harmony - 20.85 MKR - 0xE20A2e231215e9b7Aa308463F1A7490b2ECE55D3
  • [x] Res - 20.85 MKR - 0x8c5c8d76372954922400e4654AF7694e158AB784
  • [x] seedlatam.eth - 20.85 MKR - 0x0087a081a9b430fd8f688c6ac5dd24421bfb060d
• [x] Delegate Compensation
  • [x] 0xDefensor - 41.67 MKR - 0x9542b441d65B6BF4dDdd3d4D2a66D8dCB9EE07a9
  • [x] TRUE NAME - 41.67 MKR - 0x612F7924c367575a0Edf21333D96b15F1B345A5d
  • [x] BONAPUBLICA - 41.67 MKR - 0x167c1a762B08D7e78dbF8f24e5C3f1Ab415021D3
  • [x] Navigator - 41.67 MKR - 0x11406a9CC2e37425F15f920F494A51133ac93072
  • [x] vigilant - 34.5 MKR - 0x2474937cB55500601BCCE9f4cb0A0A72Dc226F61
  • [x] Cloaky - 21.06 MKR - 0x869b6d5d8FA7f4FFdaCA4D23FFE0735c5eD1F818
  • [x] UPMaker - 13.89 MKR - 0xbB819DF169670DC71A16F58F55956FE642cc6BcD
  • [x] PALC - 13.89 MKR - 0x78Deac4F87BD8007b9cb56B8d53889ed5374e83A
  • [x] BLUE - 12.78 MKR - 0xb6C09680D822F162449cdFB8248a7D3FC26Ec9Bf
  • [x] PBG - 6.48 MKR - 0x8D4df847dB7FfE0B46AF084fE031F7691C6478c2
• [x] Spark Proxy Spell
  • [x] Execute Spark Proxy Spell at 0xDE7C2758db29B53cbD2898a5584d6A719C17815E which includes:

Development Stage

• [x] Office Hours
  • [ ] ON
  • [x] OFF
  • [x] Matches Exec Doc
• [x] 30 days spell expiry in constructor (block.timestamp + 30 days)
• [x] Exec Doc Hash
  • [x] Search the 'Community' GitHub repo for the corresponding Exec Doc
  • [x] Ensure Exec Doc file name follows the format Executive vote - Month DD, YYYY.md
  • [x] Extract permanent raw Url link via Github :white_check_mark:k: https://raw.githubusercontent.com/makerdao/community/344c183374c0cd9a91ec3537d0bbb0cb0c59945a/governance/votes/Executive%20Vote%20-%20October%2011%2C%202023.md
  • [x] Generate Exec Doc Hash using this URL
  • [ ] Automatically: using make exec-hash $URL :warning: Script is not working for some reason. Not blocking progress.
  • [x] Manually: using cast keccak -- "$(curl '$URL' -o - 2>/dev/null)" 0xa6a361c0f32f118b71990a98422e2d5a353499fcbc6c21d5ef44c54a362cd2e1
• [x] Spell Description
  • [x] Description follows the format TARGET_DATE MakerDAO Executive Spell | Hash: EXEC_DOC_HASH)
  • [x] Target date in description matches the Exec Doc target date
  • [x] Exec Doc Hash in description matches your locally generated Exec Doc Hash
  • [x] Accompanying comment above spell description
  • [x] Comment follows the format // Hash: cast keccak -- "$(wget 'EXEC_DOC_URL' -q -O - 2>/dev/null)"
  • [x] Exec Doc URL in comment matches your Raw Exec Doc URL
  • [x] Exec Doc URL in comment refers to the 'Community' GitHub repo
• [x] Local Environment Actions
  • [x] Update Foundry by running foundryup
  • [x] Reinstall libraries
  • [x] Remove libraries by deleting the lib folder
  • [x] Install libraries using git submodule update --init --recursive

    Submodule path 'lib/dss-exec-lib': checked out '69b658f35d8618272cd139dfc18c5713caf6b96b'
    Submodule path 'lib/dss-exec-lib/lib/dss-interfaces': checked out '9bfd7afadd1f8c217ef05850b2555691786286cb'
    Submodule path 'lib/dss-exec-lib/lib/forge-std': checked out '0aa99eb8456693c015350c5e6c4f442ebe912f77'
    Submodule path 'lib/dss-exec-lib/lib/forge-std/lib/ds-test': checked out 'cd98eff28324bfac652e63a239a60632a761790b'
    Submodule path 'lib/dss-test': checked out '4ad127cf53eeaddfb7b8ad56dd4b13e57d6a0067'
    Submodule path 'lib/dss-test/lib/dss-interfaces': checked out '9bfd7afadd1f8c217ef05850b2555691786286cb'
    Submodule path 'lib/dss-test/lib/forge-std': checked out 'aea0b2685bebc883c09f5554d7fb481e85d0564d'
    Submodule path 'lib/dss-test/lib/forge-std/lib/ds-test': checked out 'cd98eff28324bfac652e63a239a60632a761790b'

  • [ ] Dependency checks :information_source: Skipped
  • [ ] dss-exec-lib
    • [ ] if submodule upgrades are present make sure dss-exec-lib is synced as well
    • [ ] git submodule hash (run git submodule status) matches the latest release version or newer (NOTE: dss-exec-lib as installed locally will use GitHub code more recent than the 0.0.9 release)
  • [ ] dss-test
    • [ ] dss-interfaces
    • [ ] git submodule hash matches version used by dss-test (Non-critical)
    • [ ] forge-std
    • [ ] git submodule hash matches version used by dss-test (Non-critical)
• [x] Interface Checks
  • [x] dss-interfaces
  • [x] used in the current spell
  • [x] cleanup previous ones
  • [x] ensure only single import layout is used (e.g. import "dss-interfaces/dss/VatAbstract.sol";)
  • [x] Static Interfaces
  • [x] ensure they match dss-interfaces (Where there is a mismatch, use cast interface as the source of truth)
  • [x] check on-chain interface of deployed contract via cast interface <contract_address> to ensure correctness
  • [x] interface naming style should match with Like suffix (e.g. VatLike), with some exceptions
  • [x] ensure they only list used functions in spell code
• [x] Rate constants used are correct
  • [x] Manual check 1: using make rates pct=<pct> (e.g. pct=0.75, for 0.75%)
  • [x] Manual check 2: Compare against IPFS
  • [x] Variable name conforms to X_PT_Y_Z_PCT_RATE (e.g. ZERO_PT_SEVEN_FIVE_PCT_RATE for 0.75%)
  • [x] Variable visibility declared as internal
  • [x] State mutability declared as constant
• [x] Constants Match
  • [x] Precision unit constants used match their defined values
  • [x] WAD = 10 ** 18
  • [ ] RAY = 10 ** 27
  • [ ] RAD = 10 ** 45
  • [x] Variable visibility declared as internal
  • [x] State mutability declared as constant
  • [ ] Ensure they match with ds-math and the Numerical Ranges
  • [ ] Math unit constants used match their defined values
  • [ ] HUNDRED = 10 ** 2
  • [ ] THOUSAND = 10 ** 3
  • [x] MILLION = 10 ** 6
  • [x] BILLION = 10 ** 9
  • [x] Variable visibility declared as internal
  • [x] State mutability declared as constant
  • [x] Ensure they match with config
  • [x] Timestamp constants used are correct
  • [x] Ensure the timestamp converts to the correct date
  • [x] Variable naming matches MONTH_DD_YYYY (e.g. MAY_01_2023 for 2023-05-01)
  • [x] Time of day makes logical sense (i.e. 11:59:59 for the final day of something, 00:00:00 for the first day of something) in the context of timestamp usage
  • [x] Variable visibility declared as internal
  • [x] State mutability declared as constant
• [ ] Deployed Contracts (not yet on chainlog or new to chainlog)
  • [ ] Verified on etherscan
  • [ ] Optimizations match Repo
  • [ ] GNU AGPLv3 license
  • [ ] Constructor args ok (e.g. vat, dai, dog, ...)
  • [ ] Match ChainLog
  • [ ] Wards ok (pause proxy relied, deployer denied)
  • [ ] MCD_ESM is already relied / being relied in this spell (as approved by GovAlpha) in order to allow de-authing the pause proxy during Emergency Shutdown, via denyProxy.
  • [ ] Matches corresponding github source code (i.e. diffcheck via vscode code --diff etherscan.sol github.sol)
  • [ ] Ensure deployer address is included into addresses_deployers.sol (to keep up to date)
• [x] Core System Parameter Changes
  • [x] Stability Fee jug.ilk.duty (setIlkStabilityFee)
  • [x] Comment matches pattern // Increase ILK-A Stability Fee by X.XX% from X.XX% to X.XX%.
  • [ ] Dai Savings Rate pot.dsr (setDSR)
  • [ ] Double check that PCT_RATE is correct
    • [ ] Check manually via make rates pct=<pct> (e.g. pct=0.75, for 0.75%)
    • [ ] Compare against IPFS
  • [ ] spotter.ilk.mat (liquidationRatio)
  • [ ] dog.ilk.hole (setIlkMaxLiquidationAmount)
  • [ ] vat.ilk.dust(setIlkMinVaultAmount)
  • [ ] dog.ilk.chop (liquidationPenalty)
  • [ ] clip.buf (startingPriceFactor)
  • [ ] clipperMom.clip.tolerance (setLiquidationBreakerPriceTolerance)
  • [ ] clip.tail (auctionDuration)
  • [ ] clip.cusp (permittedDrop)
  • [ ] clip.chip (kprPctReward)
  • [ ] clip.tip(kprFlatReward)
  • [ ] calc.tau (setLinearDecrease)
  • [ ] setStairstepExponentialDecrease
  • [ ] calc.cut
  • [ ] calc.step
• [ ] Debt Ceiling Changes
  • [ ] vat.ilk.line changes (per ilk)
  • [ ] Collateral type (ilk) is not RWAXXX (For RWA refer to the designated section of the checklist)
  • [ ] Either is used depending on the EXEC DOC contents:
    • [ ] setIlkDebtCeiling
      • [ ] ilk
      • [ ] line
    • [ ] increaseIlkDebtCeiling
      • [ ] ilk
      • [ ] amount
      • [ ] global
    • [ ] decreaseIlkDebtCeiling
      • [ ] ilk
      • [ ] amount
      • [ ] global
  • [ ] vat.Line changes (Global Line)
    • [ ] Either is used:
      • [ ] setGlobalDebtCeiling
        • [ ] amount
      • [ ] increaseGlobalDebtCeiling
        • [ ] amount
      • [ ] decreaseGlobalDebtCeiling
        • [ ] amount Autoline Changes
  • setIlkAutoLineDebtCeiling
    • [ ] ilk
    • [ ] line
  • setIlkAutoLineParameters
    • [x] ilk
    • [x] line
    • [x] gap
    • [x] ttl
• [ ] Onboarding (insert relevant checklists inline here)
  • [ ] Collateral Onboarding
  • [ ] RWA Onboarding
  • [ ] Teleport Onboarding
• [x] Offboarding (Lerp mat)
  • [x] 1st Stage Spell Actions
  • [x] Remove Ilk from Autoline
  • [x] Set Ilks Debt Ceilings to 0
  • [x] Cache Ilks line to Reduce in the Global Debt Ceiling
  • [x] Decrease Global Debt Ceiling by Total Amount of Offboarded Ilks line Cached
  • [ ] 2nd Stage Spell Actions
  • [ ] Set Ilk Liquidation Penalty chop to 0
  • [ ] Set Keeper Incentive Flat Rate tip to 0
  • [ ] Check IF chip is required to be adjusted as well
  • [ ] Use DssExecLib.linearInterpolation
    • [ ] name Format matches "XXX-A Offboarding"
    • [ ] target matches spotter
    • [ ] ilk Format matches "XXX-A"
    • [ ] what matches mat
    • [ ] startTime matches block.timestamp
    • [ ] start matches Var CURRENT_XXX_A_MAT
    • [ ] end matches Var TARGET_XXX_A_MAT (Match Exec Doc & Risk Computations)
    • [ ] Check IF Target mat Covers All Remaining Vaults CR times Risk Multiplier Factor
    • [ ] duration matches Exec Doc
• [x] RWA Updates
  • [ ] doc (Using the _updateDoc helper or otherwise)
  • [ ] init the RwaLiquidationOracle to reset the doc
  • [ ] Sanity Check pip must be set (not the zero address)
  • [ ] ilk follows format "RWAXXX-A"
  • [ ] val price ignored (0) if init has already been called
  • [ ] doc new legal document (IPFS HASH) matches Exec Doc
  • [ ] tau parameter used is the old tau value
  • [x] Autoline (line) + Liquidation Oracle Price Bump (val)
  • [x] Enable Autoline
    • [x] ilk follows format "RWAXXX-A"
    • [x] line (max debt ceiling)
    • [x] gap
    • [x] ttl
  • [x] bump RwaLiquidationOracle with new computed increased price (val)
    • [x] ensure val is set accordingly with autoline max debt ceiling (line)
    • [x] val should enable DAI to be drawn over the loan period while taking into account the configured ink amount, interest rate and liquidation ratio (see below)
    • [x] New val is calculated with line * [(1 + duty) ** years] * mat - rounded up - and makes sense in context of the rate mechanism. Minimum duration is usually in the Exec Doc of the spell with the RWAXXX ilk onboarding.
    • [x] Comment explaining val formula (Debt ceiling * [ (1 + RWA stability fee ) ^ (minimum deal duration in years) ] * liquidation ratio) is present
    • [x] Accompanying comment above bump line in format // XXM * 1.XX^X * X.XX as a WAD corresponding to the val calculation formula (e.g. // 15M * 1.03^2 * 1.00 as a WAD) is present along with the calculation formula on the line above
    • [x] IF combining val of multiple RWA ilks being combined, val calculation is done once per ilk and added to make the total, with workings provided in code comments. The existing val value can be retrieved by calling read() on PIP_RWAXX and converting the result into decimal.
  • [x] Poke spotter to pull in the new price
  • [ ] Debt Ceiling (line) + Liquidation Oracle Price Bump (val)
  • [ ] Increase Ilk Debt Ceiling (set DC + increase Global DC)
  • [ ] bump RwaLiquidationOracle with new computed increased price (val)
    • [ ] val should enable DAI to be drawn over the loan period while taking into account the configured ink amount, interest rate and liquidation ratio (see below)
    • [ ] New val is calculated with line * [(1 + duty) ** years] * mat - rounded up - and makes sense in context of the rate mechanism. Minimum duration is usually in the Exec Doc of the spell with the RWAXXX ilk onboarding.
    • [ ] Comment explaining val formula (Debt ceiling * [ (1 + RWA stability fee ) ^ (minimum deal duration in years) ] * liquidation ratio) is present
    • [ ] Accompanying comment above bump line in format // XXM * 1.XX^X * X.XX as a WAD corresponding to the val calculation formula (e.g. // 15M * 1.03^2 * 1.00 as a WAD) is present along with the calculation formula on the line above
    • [ ] IF combining val of multiple RWA ilks being combined, val calculation is done once per ilk and added to make the total, with workings provided in code comments. The existing val value can be retrieved by calling read() on PIP_RWAXX and converting the result into decimal.
  • [ ] Poke spotter to pull in the new price
• [x] Payments
  • [x] MKR transfers
  • [x] Recipient addresses match Exec Doc
  • [x] Transfer values match Exec Doc
  • [x] Follows archive patterns
  • [ ] DAI Surplus Buffer transfers
  • [ ] Recipient Addresses match Exec Doc
  • [ ] Payment Amounts match Exec Doc
  • [ ] Follows previous patterns
  • [x] Create New MKR/DAI Streams (DssVest)
  • [x] DssVestLike interface is correct
  • [x] Check for all streams (MKR, DAI)
    • [x] restrict is used for each stream unless otherwise stated in the Exec Doc
    • [x] usr (Vest recipient address) matches Exec Doc
    • [x] tot (Total stream amount) matches Exec Doc
    • [x] Use ether to express decimal (non-integer) values of tot
    • [x] Where vest amount is expressed in 'per year' or similar in the Exec Doc, account for leap days
    • [x] bgn (Vest start timestamp) matches Exec Doc
    • [x] tau (Vest total duration) matches Exec Doc
    • [x] tau is expressed as 'bgn - fin' (i.e. MONTH_DD_YYYY - MONTH_DD_YYYY)
    • [x] fin (Vest end timestamp) matches Exec Doc
    • [x] eta (Vest cliff duration) matches Exec Doc
    • [x] If the Exec Doc does not specify a cliff date (clf), eta is 0
    • [x] clf (Cliff end timestamp) matches Exec Doc
    • [x] If clf <= bgn, eta is 0
    • [ ] If clf > bgn
      • [ ] eta is expressed as 'clf - bgn' (i.e. MONTH_DD_YYYY - MONTH_DD_YYYY)
    • [x] mgr (Vest manager address) matches Exec Doc
    • [x] If the Exec Doc does not specify a manager address, mgr is address(0)
  • [x] Max vesting rate (cap) check (MKR, DAI)
    • [x] The maximum vesting rate (tot divided by tau) <= the maximum vest streaming rate (cap)
    • [ ] The maximum vesting rate (tot divided by tau) > the maximum vest streaming rate (cap)
    • [ ] Calculate new cap value equal to 10% greater than the new maximum vesting rate
    • [ ] Round new cap up with 2 significant figure precision (i.e. 2446 becomes 2500)
    • [ ] Notify Governance Facilitators for addition in Exec Doc AND Exec Sheet
    • [ ] Ensure that cap change is noted in the Exec Doc
    • [ ] New cap value matches Exec Doc
  • [x] MKR Streams (DssVestTransferrable)
    • [x] Increase vest contract's MKR allowance by the cumulative tot (total of new MKR vest amounts)
    • [x] Ensure allowance increase follows archive patterns
  • [ ] Terminate Existing MKR/DAI Streams (Yank)
  • [ ] Yanked stream ID matches Exec Doc
  • [ ] Use MCD_VEST_MKR_TREASURY for MKR yank
  • [ ] Use MCD_VEST_DAI for DAI yank
  • [x] Ensure Recipient Addresses match addresses_wallets.sol
• [x] SubDAO Content
  • [x] SubDAO SubProxy spell execution
  • [x] SubDAO spell address matches Exec Doc
  • [ ] IF SubDAO spell deployer is a smart contract (e.g. multisig or factory)
    • [ ] Ensure the deployer address is in addresses_deployers.sol as an entry
  • [x] Executed using ProxyLike(SUBDAO_PROXY).exec(SUBDAO_SPELL, abi.encodeWithSignature("execute()"));
  • [x] Execution is NOT delegate call
  • [x] Reviewer Note: Gas cost may be very high as SubDAO spells execute within the main cast execution. (Also note that low level call gas estimation is not done by our scripts)
  • [ ] Maker Core (main spell) SubDAO actions (i.e. operate in Pause Proxy DelegateCall context)
  • [ ] No SubDAO contract being interacted with is authed on a core contract like vat, etc. (Check comprehensively where the risk is high)
  • [ ] SubDAO contract licensing and optimizations generally do not matter (except where they pose a security risk)
  • [ ] SubDAO contracts and all libraries / dependencies are verified (Blocking if not true)
  • [ ] Upgradable SubDAO contracts
    • [ ] Any upgradable contracts have the PAUSE_PROXY as their admin (i.e. the party that can upgrade)
    • [ ] Any upgradable SubDAO contracts with an admin that is not PAUSE_PROXY are not authed on any core contracts (Blocking)
  • [ ] All SubDAO content addresses (i.e. provided contract addresses or EOAs) present in the Maker Core spell are present in the Exec Doc and are correct. SubDAO addresses being authed or given any permissions MUST be in the Exec Doc. SubDAO addresses being called must be confirmed by the SubDAO spell team.
    • [ ] IF addresses not PR'ed in by the SubDAO team (use git blame for example), SubDAO content addresses all have inline comment for provenance or source being OKed by SubDAO
  • [ ] SubDAO actions match Exec Doc (only where inline with main spell code) and do not affect core contracts
  • [ ] Core contract knock-on actions (such as offboarding or setting DC to 0) are present in the exec and match the code
  • [ ] External calls for SubDAO content are NOT delegate call
  • [ ] Code does not have untoward behavior within the scope of Maker Core Contracts (e.g. up to the SubDAO proxy)
• [ ] External Contracts Calls (Not SubDAOs, e.g. Starknet)
  • [ ] Target Contract don't block spell execution
  • [ ] External call is NOT delegate call
  • [ ] Target Contract doesn't have permissions on the Vat
  • [ ] Target Contract doesn't do anything untoward (e.g. interacting with unsafe contracts)
  • [ ] Contracts deployed via CREATE2 (e.g. if it looks like a vanity address) do not have selfdestruct in their code
  • [ ] MCD Pause Proxy doesn't give any approvals
  • [ ] All possible actions of the Target Contract are documented
  • [ ] Target contract is not upgradable
  • [ ] Target Contract is included in the ChainLog
  • [ ] Test Coverage is comprehensive
• [ ] ChainLog
  • [ ] Increment ChainLog version based on update type
  • [ ] Major -> New Vat (++.0.0)
  • [ ] Minor -> Core Module (DSS) Update (e.g. Flapper) (0.++.0)
  • [ ] Patch -> Collateral addition or addition/modification (0.0.++)
• [x] addresses_mainnet.sol matches spell code
• [x] Ensure every spell variable is declared as public/internal
• [x] Ensure immutable visibility is only used when fetching addresses from the ChainLog via DssExecLib.getChangelogAddress and constant is used instead for static addresses
  • [x] Fetch addresses as type address and wrap with Like suffix interfaces inline (when making calls) unless archive patterns permit otherwise (Such as MKR)
  • [x] Use the DssExecLib Core Address Helpers where possible (e.g. DssExecLib.vat())
  • [x] Where addresses are fetched from the ChainLog, the variable name must match the value of the ChainLog key for that address (e.g. MCD_VAT rather than vat), except where the archive pattern differs from this pattern (e.g. MKR)
• [x] Spell actions match the corresponding Exec Doc
• [x] Tests
  • [x] Ensure each spell action has sufficient test coverage
  • [x] Non-Scope Defined Parameter Changes - WBTC DC-IAM Changes :white_check_mark: testGeneral
  • [x] Stability Fee Changes :white_check_mark: testGeneral
  • [x] Initial RETH-A Offboarding :white_check_mark: testGeneral
  • [x] Reconfiguring Andromeda RWA015-A :white_check_mark: testGeneral / :white_check_mark: test_RWA015_Update
  • [x] Reconfiguring Clydesdale RWA007-A :white_check_mark: testGeneral / :white_check_mark: test_RWA007_Update
  • [x] Set up Governance Facilitator Streams :white_check_mark: testVestDAI / :white_check_mark: testVestMKR
  • [x] USDP-PSM Facilitation Incentives :white_check_mark: Not applicable
  • [x] BA Labs MKR Distribution :white_check_mark: testMKRPayments
  • [x] AVC Member Compensation :white_check_mark: testMKRPayments
  • [x] Delegate Compensation :white_check_mark: testMKRPayments
  • [x] Spark Proxy Spell :white_check_mark: testSparkSpellIsExecuted
  • [x] Ensure every test function is declared as public if enabled or private if disabled
  • [x] Ensure that the DssExecLib.address file is not being modified by the spell PR
  • [x] Check all CI tests are passing as at the latest commit TODO.
  • [x] Check all tests are passing locally using make test
  • [x] Ensure that only ETH_RPC_URL is being used from env (i.e. no match, block or similar are active in your env)

Using DssExecLib at: 0x8De6DDbCd5053d32292AAA0D2105A32d108484a6
[⠊] Compiling...
[⠊] Compiling 4 files with 0.8.16
[⠆] Solc 0.8.16 finished in 1.91s
Compiler run successful!

Running 2 tests for src/test/starknet.t.sol:StarknetTests
[PASS] testStarknet() (gas: 5456079)
[PASS] testStarknetSpell() (gas: 2346)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 128.28s

Running 21 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testBytecodeMatches() (gas: 3791346)
[PASS] testCastCost() (gas: 5301472)
[PASS] testChainlogValues() (gas: 14167554)
[PASS] testChainlogVersionBump() (gas: 8930988)
[PASS] testContractSize() (gas: 8984)
[PASS] testDeployCost() (gas: 3773058)
[PASS] testFailNotScheduled() (gas: 14420)
[PASS] testFailTooEarly() (gas: 13607)
[PASS] testFailTooLate() (gas: 13584)
[PASS] testFailWrongDay() (gas: 13563)
[PASS] testGeneral() (gas: 42066368)
[PASS] testMKRPayments() (gas: 5450402)
[PASS] testNextCastTime() (gas: 353703)
[PASS] testOnTime() (gas: 5297156)
[PASS] testPSMs() (gas: 6648208)
[PASS] testSparkSpellIsExecuted() (gas: 5300692)
[PASS] testUseEta() (gas: 352324)
[PASS] testVestDAI() (gas: 5485745)
[PASS] testVestMKR() (gas: 5422467)
[PASS] test_RWA007_Update() (gas: 5541585)
[PASS] test_RWA015_Update() (gas: 5541199)
Test result: ok. 21 passed; 0 failed; 0 skipped; finished in 1102.50s

Ran 2 test suites: 23 tests passed, 0 failed, 0 skipped (23 total tests)

[SidestreamSweatyPumpkin]

Good to deploy

Spell Actions (Per Exec Doc):

• Read spell actions and instructions from the Exec Doc

• List the actions being performed in this spell

• [x] Non-Scope Defined Parameter Changes - WBTC DC-IAM Changes

  • [x] Reduce the WBTC-A DC-IAM Target Available Debt from 10 million DAI to 2 million DAI.
  • [x] Reduce the WBTC-B DC-IAM Target Available Debt from 5 million DAI to 2 million DAI.
  • [x] Reduce the WBTC-C DC-IAM Target Available Debt from 10 million DAI to 2 million DAI.

• [x] Stability Fee Changes

  • [x] Increase the ETH-A Stability Fee (SF) by 1.55%, from 3.70% to 5.25%.
  • [x] Increase the ETH-B Stability Fee (SF) by 1.55%, from 4.20% to 5.75%.
  • [x] Increase the ETH-C Stability Fee (SF) by 1.55%, from 3.45% to 5.00%.
  • [x] Increase WBTC-A Stability Fee (SF) by 0.06%, from 5.8% to 5.86%
  • [x] Increase WBTC-B Stability Fee (SF) by 0.06%, from 6.30% to 6.36%.
  • [x] Increase WBTC-C Stability Fee (SF) by 0.06%, from 5.55% to 5.61%.

• [x] Initial RETH-A Offboarding

  • [x] Set rETH-A Debt Ceiling (line) to 0 (zero).
  • [x] Remove RETH-A from Autoline.

• [x] Reconfiguring Andromeda RWA015-A

  • [x] Set the Maximum Debt Ceiling (line) to 3 billion DAI.

• [x] Reconfiguring Clydesdale RWA007-A

  • [x] Reactivate the Debt Ceiling Instant Access Module for this vault type.
  • [x] Set the Target Available Debt (gap) to 50 million DAI.
  • [x] Set the Ceiling Increase Cooldown (ttl) to 86400 (24 hours).
  • [x] Set the Maximum Debt Ceiling (line) to 3 billion DAI.

• [x] Set up Governance Facilitator Streams

  • [x] JanSky | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 504,000.00 DAI | 0xf3F868534FAD48EF5a228Fe78669cf242745a755
  • [x] VoteWizard | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 504,000.00 DAI | 0x9E72629dF4fcaA2c2F5813FbbDc55064345431b1
  • [x] JanSky | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 216.00 MKR | 0xf3F868534FAD48EF5a228Fe78669cf242745a755
  • [x] VoteWizard | 2023-10-01 00:00:00 to 2024-09-30 23:59:59 | 216.00 MKR | 0x9E72629dF4fcaA2c2F5813FbbDc55064345431b1

• [x] USDP-PSM Facilitation Incentives

  • [x] DAO Resolution 1: Initial Setup (copy only)
  • [x] Add dao_resolution parameter: QmWg43PNNGfEyXnTv1qN8dRXFJz5ZchrmZU8qH57Ki6D62

• [x] BA Labs MKR Distribution

  • [x] BA Labs - 175 MKR - 0x5d67d5B1fC7EF4bfF31967bE2D2d7b9323c1521c

• [x] AVC Member Compensation

  • [x] opensky - 20.85 MKR - 0x8e67ee3bbeb1743dc63093af493f67c3c23c6f04
  • [x] DAI-Vinci - 12.51 MKR - 0x9ee47F0f82F1A6F45C4E1D25Ce95C321D8C8356a
  • [x] IamMeeoh - 20.85 MKR - 0x47f7A5d8D27f259582097E1eE59a07a816982AE9
  • [x] ACRE DAOs - 20.85 MKR - 0xBF9226345F601150F64Ea4fEaAE7E40530763cbd
  • [x] Harmony - 20.85 MKR - 0xE20A2e231215e9b7Aa308463F1A7490b2ECE55D3
  • [x] Res - 20.85 MKR - 0x8c5c8d76372954922400e4654AF7694e158AB784
  • [x] seedlatam.eth - 20.85 MKR - 0x0087a081a9b430fd8f688c6ac5dd24421bfb060d

• [x] Delegate Compensation

  • [x] 0xDefensor - 41.67 MKR - 0x9542b441d65B6BF4dDdd3d4D2a66D8dCB9EE07a9
  • [x] TRUE NAME - 41.67 MKR - 0x612F7924c367575a0Edf21333D96b15F1B345A5d
  • [x] BONAPUBLICA - 41.67 MKR - 0x167c1a762B08D7e78dbF8f24e5C3f1Ab415021D3
  • [x] Navigator - 41.67 MKR - 0x11406a9CC2e37425F15f920F494A51133ac93072
  • [x] vigilant - 34.5 MKR - 0x2474937cB55500601BCCE9f4cb0A0A72Dc226F61
  • [x] Cloaky - 21.06 MKR - 0x869b6d5d8FA7f4FFdaCA4D23FFE0735c5eD1F818
  • [x] UPMaker - 13.89 MKR - 0xbB819DF169670DC71A16F58F55956FE642cc6BcD
  • [x] PALC - 13.89 MKR - 0x78Deac4F87BD8007b9cb56B8d53889ed5374e83A
  • [x] BLUE - 12.78 MKR - 0xb6C09680D822F162449cdFB8248a7D3FC26Ec9Bf
  • [x] PBG - 6.48 MKR - 0x8D4df847dB7FfE0B46AF084fE031F7691C6478c2

• [x] Spark Proxy Spell

  • [x] Execute Spark Proxy Spell at 0xDE7C2758db29B53cbD2898a5584d6A719C17815E which includes:

Development Stage

• [X] Office Hours
  • [X] ON
  • [X] OFF
  • [X] Matches Exec Doc
• [X] 30 days spell expiry in constructor (block.timestamp + 30 days)
• [X] Exec Doc Hash
  • [X] Search the 'Community' GitHub repo for the corresponding Exec Doc
  • [X] Ensure Exec Doc file name follows the format Executive vote - Month DD, YYYY.md
  • [X] Extract permanent raw Url link via Github https://raw.githubusercontent.com/makerdao/community/344c183374c0cd9a91ec3537d0bbb0cb0c59945a/governance/votes/Executive%20Vote%20-%20October%2011%2C%202023.md
  • [X] Generate Exec Doc Hash using this URL
  • [X] Automatically: using make exec-hash $URL
  • [X] Manually: using cast keccak -- "$(curl '$URL' -o - 2>/dev/null)" 0xa6a361c0f32f118b71990a98422e2d5a353499fcbc6c21d5ef44c54a362cd2e1
• [X] Spell Description
  • [X] Description follows the format TARGET_DATE MakerDAO Executive Spell | Hash: EXEC_DOC_HASH)
  • [X] Target date in description matches the Exec Doc target date
  • [X] Exec Doc Hash in description matches your locally generated Exec Doc Hash
  • [X] Accompanying comment above spell description
  • [X] Comment follows the format // Hash: cast keccak -- "$(wget 'EXEC_DOC_URL' -q -O - 2>/dev/null)"
  • [X] Exec Doc URL in comment matches your Raw Exec Doc URL
  • [X] Exec Doc URL in comment refers to the 'Community' GitHub repo
• [X] Local Environment Actions
  • [X] Update Foundry by running foundryup
  • [X] Reinstall libraries
  • [X] Remove libraries by deleting the lib folder
  • [X] Install libraries using git submodule update --init --recursive

    Submodule path 'lib/dss-exec-lib': checked out '69b658f35d8618272cd139dfc18c5713caf6b96b'
    Submodule path 'lib/dss-exec-lib/lib/dss-interfaces': checked out '9bfd7afadd1f8c217ef05850b25556917
    86286cb'
    Submodule path 'lib/dss-exec-lib/lib/forge-std': checked out '0aa99eb8456693c015350c5e6c4f442ebe912f
    77'
    Submodule path 'lib/dss-exec-lib/lib/forge-std/lib/ds-test': checked out 'cd98eff28324bfac652e63a239
    a60632a761790b'
    Submodule path 'lib/dss-test': checked out '4ad127cf53eeaddfb7b8ad56dd4b13e57d6a0067'
    Submodule path 'lib/dss-test/lib/dss-interfaces': checked out '9bfd7afadd1f8c217ef05850b255569178628
    6cb'
    Submodule path 'lib/dss-test/lib/forge-std': checked out 'aea0b2685bebc883c09f5554d7fb481e85d0564d'
    Submodule path 'lib/dss-test/lib/forge-std/lib/ds-test': checked out 'cd98eff28324bfac652e63a239a606
    32a761790b'

  • [X] Dependency checks
  • [X] dss-exec-lib
    • [X] if submodule upgrades are present make sure dss-exec-lib is synced as well not present
    • [X] git submodule hash (run git submodule status) matches the latest release version or newer (NOTE: dss-exec-lib as installed locally will use GitHub code more recent than the 0.0.9 release)
• [X] Interface Checks
  • [X] dss-interfaces
  • [X] used in the current spell
  • [X] cleanup previous ones
  • [X] ensure only single import layout is used (e.g. import "dss-interfaces/dss/VatAbstract.sol";)
  • [X] Static Interfaces
  • [X] ensure they match dss-interfaces (Where there is a mismatch, use cast interface as the source of truth)
  • [X] check on-chain interface of deployed contract via cast interface <contract_address> to ensure correctness
  • [X] interface naming style should match with Like suffix (e.g. VatLike), with some exceptions
  • [X] ensure they only list used functions in spell code
• [X] Rate constants used are correct
  • [X] Manual check 1: using make rates pct=<pct> (e.g. pct=0.75, for 0.75%)
  • [X] Manual check 2: Compare against IPFS
  • [X] Variable name conforms to X_PT_Y_Z_PCT_RATE (e.g. ZERO_PT_SEVEN_FIVE_PCT_RATE for 0.75%)
  • [X] Variable visibility declared as internal
  • [X] State mutability declared as constant
• [X] Constants Match
  • [X] Precision unit constants used match their defined values
  • [X] WAD = 10 ** 18
  • [X] Variable visibility declared as internal
  • [X] State mutability declared as constant
  • [X] Ensure they match with ds-math and the Numerical Ranges
  • [X] Math unit constants used match their defined values
  • [X] MILLION = 10 ** 6
  • [X] BILLION = 10 ** 9
  • [X] Variable visibility declared as internal
  • [X] State mutability declared as constant
  • [X] Ensure they match with config
  • [X] Timestamp constants used are correct
  • [X] Ensure the timestamp converts to the correct date
  • [X] Variable naming matches MONTH_DD_YYYY (e.g. MAY_01_2023 for 2023-05-01)
  • [X] Time of day makes logical sense (i.e. 11:59:59 for the final day of something, 00:00:00 for the first day of something) in the context of timestamp usage
  • [X] Variable visibility declared as internal
  • [X] State mutability declared as constant
• [X] Core System Parameter Changes
  • [X] Stability Fee jug.ilk.duty (setIlkStabilityFee)
  • [X] Comment matches pattern // Increase ILK-A Stability Fee by X.XX% from X.XX% to X.XX%.
• [X] Autoline Changes
  • setIlkAutoLineParameters
    • [X] ilk
    • [X] line
    • [X] gap
    • [X] ttl
• [X] RWA Updates
  • [X] Autoline (line) + Liquidation Oracle Price Bump (val)
  • [X] Enable Autoline
    • [X] ilk follows format "RWAXXX-A"
    • [X] line (max debt ceiling)
    • [X] gap
    • [X] ttl
  • [X] bump RwaLiquidationOracle with new computed increased price (val)
    • [X] ensure val is set accordingly with autoline max debt ceiling (line)
    • [X] val should enable DAI to be drawn over the loan period while taking into account the configured ink amount, interest rate and liquidation ratio (see below)
    • [X] New val is calculated with line * [(1 + duty) ** years] * mat - rounded up - and makes sense in context of the rate mechanism. Minimum duration is usually in the Exec Doc of the spell with the RWAXXX ilk onboarding.
    • [X] Comment explaining val formula (Debt ceiling * [ (1 + RWA stability fee ) ^ (minimum deal duration in years) ] * liquidation ratio) is present
    • [X] Accompanying comment above bump line in format // XXM * 1.XX^X * X.XX as a WAD corresponding to the val calculation formula (e.g. // 15M * 1.03^2 * 1.00 as a WAD) is present along with the calculation formula on the line above
    • [X] IF combining val of multiple RWA ilks being combined, val calculation is done once per ilk and added to make the total, with workings provided in code comments. The existing val value can be retrieved by calling read() on PIP_RWAXX and converting the result into decimal.
  • [X] Poke spotter to pull in the new price
  • [X] Debt Ceiling (line) + Liquidation Oracle Price Bump (val)
  • [X] Increase Ilk Debt Ceiling (set DC + increase Global DC)
  • [X] bump RwaLiquidationOracle with new computed increased price (val)
    • [X] val should enable DAI to be drawn over the loan period while taking into account the configured ink amount, interest rate and liquidation ratio (see below)
    • [X] New val is calculated with line * [(1 + duty) ** years] * mat - rounded up - and makes sense in context of the rate mechanism. Minimum duration is usually in the Exec Doc of the spell with the RWAXXX ilk onboarding.
    • [X] Comment explaining val formula (Debt ceiling * [ (1 + RWA stability fee ) ^ (minimum deal duration in years) ] * liquidation ratio) is present
    • [X] Accompanying comment above bump line in format // XXM * 1.XX^X * X.XX as a WAD corresponding to the val calculation formula (e.g. // 15M * 1.03^2 * 1.00 as a WAD) is present along with the calculation formula on the line above
    • [X] IF combining val of multiple RWA ilks being combined, val calculation is done once per ilk and added to make the total, with workings provided in code comments. The existing val value can be retrieved by calling read() on PIP_RWAXX and converting the result into decimal.
  • [X] Poke spotter to pull in the new price
• [X] Payments
  • [X] MKR transfers
  • [X] Recipient addresses match Exec Doc
  • [X] Transfer values match Exec Doc
  • [X] Follows archive patterns
  • [X] DAI Surplus Buffer transfers
  • [X] Recipient Addresses match Exec Doc
  • [X] Payment Amounts match Exec Doc
  • [X] Follows previous patterns
  • [X] Create New MKR/DAI Streams (DssVest)
  • [X] DssVestLike interface is correct
  • [X] Check for all streams (MKR, DAI)
    • [X] restrict is used for each stream unless otherwise stated in the Exec Doc
    • [X] usr (Vest recipient address) matches Exec Doc
    • [X] tot (Total stream amount) matches Exec Doc
    • [X] Use ether to express decimal (non-integer) values of tot
    • [X] Where vest amount is expressed in 'per year' or similar in the Exec Doc, account for leap days
    • [X] bgn (Vest start timestamp) matches Exec Doc
    • [X] tau (Vest total duration) matches Exec Doc
    • [X] tau is expressed as 'bgn - fin' (i.e. MONTH_DD_YYYY - MONTH_DD_YYYY)
    • [X] fin (Vest end timestamp) matches Exec Doc
    • [X] eta (Vest cliff duration) matches Exec Doc
    • [X] If the Exec Doc does not specify a cliff date (clf), eta is 0
    • [X] clf (Cliff end timestamp) matches Exec Doc
    • [X] If clf <= bgn, eta is 0
    • [X] If clf > bgn
      • [X] eta is expressed as 'clf - bgn' (i.e. MONTH_DD_YYYY - MONTH_DD_YYYY)
    • [X] mgr (Vest manager address) matches Exec Doc
    • [X] If the Exec Doc does not specify a manager address, mgr is address(0)
  • [X] Max vesting rate (cap) check (MKR, DAI)
    • [X] The maximum vesting rate (tot divided by tau) <= the maximum vest streaming rate (cap)
    • [X] The maximum vesting rate (tot divided by tau) > the maximum vest streaming rate (cap)
    • [X] Calculate new cap value equal to 10% greater than the new maximum vesting rate
    • [X] Round new cap up with 2 significant figure precision (i.e. 2446 becomes 2500)
    • [X] Notify Governance Facilitators for addition in Exec Doc AND Exec Sheet
    • [X] Ensure that cap change is noted in the Exec Doc
    • [X] New cap value matches Exec Doc
  • [X] MKR Streams (DssVestTransferrable)
    • [X] Increase vest contract's MKR allowance by the cumulative tot (total of new MKR vest amounts)
    • [X] Ensure allowance increase follows archive patterns
  • [X] Ensure Recipient Addresses match addresses_wallets.sol
• [X] SubDAO Content
  • [X] SubDAO SubProxy spell execution
  • [X] SubDAO spell address matches Exec Doc
  • [X] Executed using ProxyLike(SUBDAO_PROXY).exec(SUBDAO_SPELL, abi.encodeWithSignature("execute()"));
  • [X] Execution is NOT delegate call
  • [X] Reviewer Note: Gas cost may be very high as SubDAO spells execute within the main cast execution. (Also note that low level call gas estimation is not done by our scripts)
  • [X] Maker Core (main spell) SubDAO actions (i.e. operate in Pause Proxy DelegateCall context)
  • [X] No SubDAO contract being interacted with is authed on a core contract like vat, etc. (Check comprehensively where the risk is high)
  • [X] SubDAO contract licensing and optimizations generally do not matter (except where they pose a security risk)
  • [X] SubDAO contracts and all libraries / dependencies are verified (Blocking if not true)
  • [X] Upgradable SubDAO contracts
    • [X] Any upgradable contracts have the PAUSE_PROXY as their admin (i.e. the party that can upgrade)
    • [X] Any upgradable SubDAO contracts with an admin that is not PAUSE_PROXY are not authed on any core contracts (Blocking)
  • [X] All SubDAO content addresses (i.e. provided contract addresses or EOAs) present in the Maker Core spell are present in the Exec Doc and are correct. SubDAO addresses being authed or given any permissions MUST be in the Exec Doc. SubDAO addresses being called must be confirmed by the SubDAO spell team.
    • [X] IF addresses not PR'ed in by the SubDAO team (use git blame for example), SubDAO content addresses all have inline comment for provenance or source being OKed by SubDAO
  • [X] SubDAO actions match Exec Doc (only where inline with main spell code) and do not affect core contracts
  • [X] Core contract knock-on actions (such as offboarding or setting DC to 0) are present in the exec and match the code
  • [X] External calls for SubDAO content are NOT delegate call
  • [X] Code does not have untoward behavior within the scope of Maker Core Contracts (e.g. up to the SubDAO proxy)
• [X] addresses_mainnet.sol matches spell code
• [X] Ensure every spell variable is declared as public/internal
• [X] Ensure immutable visibility is only used when fetching addresses from the ChainLog via DssExecLib.getChangelogAddress and constant is used instead for static addresses
  • [X] Fetch addresses as type address and wrap with Like suffix interfaces inline (when making calls) unless archive patterns permit otherwise (Such as MKR)
  • [X] Use the DssExecLib Core Address Helpers where possible (e.g. DssExecLib.vat())
  • [X] Where addresses are fetched from the ChainLog, the variable name must match the value of the ChainLog key for that address (e.g. MCD_VAT rather than vat), except where the archive pattern differs from this pattern (e.g. MKR)
• [X] Spell actions match the corresponding Exec Doc
• [X] Tests
  • [X] Ensure each spell action has sufficient test coverage
    • [X] Non-Scope Defined Parameter Changes
    • [X] Stability Fee Changes
    • [X] Initial RETH-A Offboarding
    • [X] RWA015-A
    • [X] RWA007-A
    • [X] Governance Facilitator Streams
    • [X] USDP-PSM
    • [X] BA Labs MKR
    • [X] AVC Member Compensation
    • [X] Delegate Compensation
    • [X] Spark Proxy Spell
  • [X] Ensure every test function is declared as public if enabled or private if disabled
  • [X] Ensure that the DssExecLib.address file is not being modified by the spell PR
  • [X] Check all CI tests are passing as at the latest commit https://github.com/makerdao/spells-mainnet/pull/368/commits/471dd69e6486fd761d75b99c590f0b20978ce092
  • [X] Check all tests are passing locally using make test
  • [X] Ensure that only ETH_RPC_URL is being used from env (i.e. no match, block or similar are active in your env)

./scripts/test-dssspell-forge.sh no-match="" match="" block=""
Using DssExecLib at: 0x8De6DDbCd5053d32292AAA0D2105A32d108484a6
[⠒] Compiling...
[⠑] Compiling 7 files with 0.8.16
[⠔] Solc 0.8.16 finished in 1.99s
Compiler run successful!

Running 2 tests for src/test/starknet.t.sol:StarknetTests
[PASS] testStarknet() (gas: 5456308)
[PASS] testStarknetSpell() (gas: 2346)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 171.04s

Running 23 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testAuth() (gas: 9223371487105683244)
[PASS] testAuthInSources() (gas: 9223371487099286666)
[PASS] testBytecodeMatches() (gas: 3803830)
[PASS] testCastCost() (gas: 5301701)
[PASS] testChainlogValues() (gas: 14167783)
[PASS] testChainlogVersionBump() (gas: 8931217)
[PASS] testContractSize() (gas: 8984)
[PASS] testDeployCost() (gas: 3785506)
[PASS] testFailNotScheduled() (gas: 14420)
[PASS] testFailTooEarly() (gas: 13607)
[PASS] testFailTooLate() (gas: 13584)
[PASS] testFailWrongDay() (gas: 13563)
[PASS] testGeneral() (gas: 42080917)
[PASS] testMKRPayments() (gas: 5450631)
[PASS] testNextCastTime() (gas: 353703)
[PASS] testOnTime() (gas: 5297385)
[PASS] testPSMs() (gas: 6648437)
[PASS] testSparkSpellIsExecuted() (gas: 5300921)
[PASS] testUseEta() (gas: 352324)
[PASS] testVestDAI() (gas: 5485974)
[PASS] testVestMKR() (gas: 5422696)
[PASS] test_RWA007_Update() (gas: 5542044)
[PASS] test_RWA015_Update() (gas: 5541313)
Test result: ok. 23 passed; 0 failed; 0 skipped; finished in 1625.95s

Ran 2 test suites: 25 tests passed, 0 failed, 0 skipped (25 total tests)

[amusingaxl]

Good to handover.

Deployed Stage

• [x] Deployed Spell is Verified
  • [x] Optimization Enabled: No
  • [x] Other Settings: default evmVersion, GNU AGPLv3 license ℹ️ EVM version is shown as "London". Not critical.
• [x] Deployed Spell Code matches GitHub
  • [x] diffcheck etherscan source against spell PR (via make diff-deployed-spell)
• [x] Deployed Spell Etherscan Checks
  • [x] automated checks via make check-deployed-spell
  • [x] verified
  • [x] license type matches
  • [x] solc version matches
  • [x] optimizations are disabled
  • [x] dss-exec-lib library address used (under 'Libraries Used') matches the hardcoded local DssExecLib.address file
    • [x] Check again that the PR did not modify the DssExecLib.address file (e.g. look under the 'Files Changed' PR tab, etc.)
  • [x] deployed_spell_created matches deployment timestamp
  • [x] deployed_spell_block matches deployment block number
  • [x] manual checks
    • [x] Ensure make deploy-info tx=<tx> matches config
    • [x] deployed_spell_created timestamp
    • [x] deployed_spell_block block number
    • [x] Ensure Etherscan Libraries Used matches DssExecLib Latest Release
    • [ ] (For your tests to be accurate) git submodule hash matches dss-exec-lib latest release's tag commit and inspect diffs if doesn't match to ensure expected behaviour (Currently Non-Critical pending the next DssExecLib release, double check that the ExecLib used by the contract matches the latest release) :information_source: Ignored
• [x] Archive matches src
  • [x] make diff-archive-spell for current date or or date="YYYY-MM-DD" make diff-archive-spell (date as per target Exec Doc date)
  • [x] Ensure date corresponds to target Exec Doc date
• [x] Tests
  • [x] Ensure that the DssExecLib.address file is not being modified by the spell PR
  • [x] Check all CI tests are passing as at the latest commit :white_check_mark: 1235709
  • [x] Check all tests are passing locally using make test
  • [x] Ensure that only ETH_RPC_URL is being used from env (i.e. no match, block or similar are active in your env)

Using DssExecLib at: 0x8De6DDbCd5053d32292AAA0D2105A32d108484a6
[⠆] Compiling...
[⠃] Compiling 6 files with 0.8.16
[⠢] Solc 0.8.16 finished in 2.48s
Compiler run successful!

Running 2 tests for src/test/starknet.t.sol:StarknetTests
[PASS] testStarknet() (gas: 5456079)
[PASS] testStarknetSpell() (gas: 2346)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 128.08s

Running 21 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testBytecodeMatches() (gas: 3803830)
[PASS] testCastCost() (gas: 5301472)
[PASS] testChainlogValues() (gas: 14167554)
[PASS] testChainlogVersionBump() (gas: 8930988)
[PASS] testContractSize() (gas: 8984)
[PASS] testDeployCost() (gas: 3785506)
[PASS] testFailNotScheduled() (gas: 14420)
[PASS] testFailTooEarly() (gas: 13607)
[PASS] testFailTooLate() (gas: 13584)
[PASS] testFailWrongDay() (gas: 13563)
[PASS] testGeneral() (gas: 42082785)
[PASS] testMKRPayments() (gas: 5450402)
[PASS] testNextCastTime() (gas: 353703)
[PASS] testOnTime() (gas: 5297156)
[PASS] testPSMs() (gas: 6648208)
[PASS] testSparkSpellIsExecuted() (gas: 5300692)
[PASS] testUseEta() (gas: 352324)
[PASS] testVestDAI() (gas: 5485745)
[PASS] testVestMKR() (gas: 5422467)
[PASS] test_RWA007_Update() (gas: 5541930)
[PASS] test_RWA015_Update() (gas: 5540566)
Test result: ok. 21 passed; 0 failed; 0 skipped; finished in 759.28s

Ran 2 test suites: 23 tests passed, 0 failed, 0 skipped (23 total tests)

[SidestreamSweatyPumpkin]

Good to handover

Deployed Stage

• [X] Deployed Spell is Verified
  • [X] Optimization Enabled: No
  • [X] Other Settings: default evmVersion, GNU AGPLv3 license
• [X] Deployed Spell Code matches GitHub
  • [X] diffcheck etherscan source against spell PR (via make diff-deployed-spell)
• [X] Deployed Spell Etherscan Checks
  • [X] automated checks via make check-deployed-spell
  • [X] verified
  • [X] license type matches
  • [X] solc version matches
  • [X] optimizations are disabled
  • [X] dss-exec-lib library address used (under 'Libraries Used') matches the hardcoded local DssExecLib.address file
    • [X] Check again that the PR did not modify the DssExecLib.address file (e.g. look under the 'Files Changed' PR tab, etc.)
  • [X] deployed_spell_created matches deployment timestamp
  • [X] deployed_spell_block matches deployment block number
  • [X] manual checks
    • [X] Ensure make deploy-info tx=<tx> matches config
    • [X] deployed_spell_created timestamp
    • [X] deployed_spell_block block number
    • [X] Ensure Etherscan Libraries Used matches DssExecLib Latest Release
    • [X] (For your tests to be accurate) git submodule hash matches dss-exec-lib latest release's tag commit and inspect diffs if doesn't match to ensure expected behaviour (Currently Non-Critical pending the next DssExecLib release, double check that the ExecLib used by the contract matches the latest release)
• [X] Archive matches src
  • [X] make diff-archive-spell for current date or or date="YYYY-MM-DD" make diff-archive-spell (date as per target Exec Doc date)
  • [X] Ensure date corresponds to target Exec Doc date
• [X] Tests
  • [X] Ensure that the DssExecLib.address file is not being modified by the spell PR
  • [X] Check all CI tests are passing as at the latest commit https://github.com/makerdao/spells-mainnet/pull/368/commits/123570982226635ece45deb5192f821783ced279
  • [X] Check all tests are passing locally using make test
  • [X] Ensure that only ETH_RPC_URL is being used from env (i.e. no match, block or similar are active in your env)

./scripts/test-dssspell-forge.sh no-match="" match="" block=""
Using DssExecLib at: 0x8De6DDbCd5053d32292AAA0D2105A32d108484a6
[⠒] Compiling...
[⠔] Compiling 4 files with 0.8.16
[⠢] Solc 0.8.16 finished in 1.81s
Compiler run successful!

Running 2 tests for src/test/starknet.t.sol:StarknetTests
[PASS] testStarknet() (gas: 5457459)
[PASS] testStarknetSpell() (gas: 2346)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 999.23s

Running 23 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testAuth() (gas: 9223371487105683244)
[PASS] testAuthInSources() (gas: 9223371487099286666)
[PASS] testBytecodeMatches() (gas: 3803830)
[PASS] testCastCost() (gas: 5302852)
[PASS] testChainlogValues() (gas: 14168934)
[PASS] testChainlogVersionBump() (gas: 8932368)
[PASS] testContractSize() (gas: 8984)
[PASS] testDeployCost() (gas: 3785506)
[PASS] testFailNotScheduled() (gas: 14420)
[PASS] testFailTooEarly() (gas: 13607)
[PASS] testFailTooLate() (gas: 13584)
[PASS] testFailWrongDay() (gas: 13563)
[PASS] testGeneral() (gas: 42084165)
[PASS] testMKRPayments() (gas: 5451782)
[PASS] testNextCastTime() (gas: 353703)
[PASS] testOnTime() (gas: 5298536)
[PASS] testPSMs() (gas: 6649588)
[PASS] testSparkSpellIsExecuted() (gas: 5302072)
[PASS] testUseEta() (gas: 352324)
[PASS] testVestDAI() (gas: 5487125)
[PASS] testVestMKR() (gas: 5423847)
[PASS] test_RWA007_Update() (gas: 5543080)
[PASS] test_RWA015_Update() (gas: 5542176)
Test result: ok. 23 passed; 0 failed; 0 skipped; finished in 999.23s

Ran 2 test suites: 25 tests passed, 0 failed, 0 skipped (25 total tests)