* fixes#12015 by also checking kind of `typeNode`
If a tuple field is aliased it'll appear the same as a ref type in a
call to `getType` if only for the kind of the resulting `NimNode` is
checked (that is a `nnkBracketExpr`)
* fix test case due to #12017 and add more realistic test case
Adds an additional test case, which includes generics and is closer to
the real failure I encountered
* remove previous fix and fix differently after all
The previous fix was incomplete, because it failed for generics.
Note that the `of "tuple"` is not actually needed, the
`nnkBracketExpr` branch in the `else` branch would catch it too, but I
decided to introduce it for clarity. However, the latter is actually
needed, because it seems for aliases of `seq` we end up in it.
* update comment about global `%` proc in json test
eff0837ff4