fixes #24641; quote do captures no variables without backticks

[ringabout]

fixes #24641

ref #17426

TODO:

• revert Quote do now works with result in block #7343

This PR solves problems in nim-lang/RFCs#122

import macros

macro genPrintX(): untyped =
  let x = "mystring"
  result = quote do:
    echo x # Error: internal error: environment misses: x

proc main(): void =
  let x = 123
  genPrintX()

main()

The quoted ast won't capture local variables from macros anymore. It solves #7323 which is a special case of capturing result. Then we can revert #7343 that did some hack on result.

[ringabout]

related to #8574

[ringabout]

It allows top level syms to be captured for backwards compatibility, but still benefits from not capturing local varibles in macros/procs

  result = quote do:
    initFromJson(`baseTyp`(`dst`), `jsonNode`, `jsonPath`)

This is wrong because initFromJson should be backticked

[ringabout]

The latest changes skip the current owner scope if the kind of owner belongs to routine kinds

[Araq]

Has the potential to break somebody's subtle macro trickery. Not sure what to do.

[tersec]

Has the potential to break somebody's subtle macro trickery. Not sure what to do.

No possible subtle macro trickery can depend on an internal compiler error happening.

[Araq]

No possible subtle macro trickery can depend on an internal compiler error happening.

The diff doesn't contain any "removes internalError", instead it changes the AST which is exactly what you can inspect in a macro.