oxheadalpha / tezos-contract-metadata Goto Github PK

We probably do not need something this heavy-weight.

The culprit:

Case in point: KT1VvnQFn9v3oHA7JLTdDZD6hdAx3T4TnUA2 on Hangzhou-net fails with Contract has no valid %metadata big-map! despite containing (big_map %metadata string bytes) within the top-level tree of nested pairs (as per spec). storage is declared as

  storage
   (pair (pair <...>)
          (pair <...>)
          (big_map <...>)
          (big_map %metadata string bytes)) ;

the corresponding Micheline representation is

{
  "code": [
    {...}
    {
      "prim": "storage",
      "args": [
        {
          "prim": "pair",
          "args": [
            {...},
            {...},
            {...},
            {
              "prim": "big_map",
              "args": [
                {
                  "prim": "string"
                },
                {
                  "prim": "bytes"
                }
              ],
              "annots": [
                "%metadata"
              ]
            }
          ]
        }
      ]
    },
    {...}
  ]
}

Notice the pair constructor here has 4 arguments in the Micheline representation, while the code only looks for cases where there are exactly 2.

Instead of using >>=, we should use let* to be consistent with tezos/tezos, which is doing the same.