fixes #12125 (#12131)

* fixes #12125

compiler/scriptconfig.nim

@@ -62,8 +62,14 @@ proc setupVM*(module: PSym; cache: IdentCache; scriptName: string;
   cbos listDirs:
     listDirs(a, {pcDir})
   cbos removeDir:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.removeDir getString(a, 0)
   cbos removeFile:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.removeFile getString(a, 0)
   cbos createDir:
     os.createDir getString(a, 0)
@@ -76,12 +82,24 @@ proc setupVM*(module: PSym; cache: IdentCache; scriptName: string;
   cbos getCurrentDir:
     setResult(a, os.getCurrentDir())
   cbos moveFile:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.moveFile(getString(a, 0), getString(a, 1))
   cbos moveDir:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.moveDir(getString(a, 0), getString(a, 1))
   cbos copyFile:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.copyFile(getString(a, 0), getString(a, 1))
   cbos copyDir:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       os.copyDir(getString(a, 0), getString(a, 1))
   cbos getLastModificationTime:
     setResult(a, getLastModificationTime(getString(a, 0)).toUnix)
@@ -89,6 +107,9 @@ proc setupVM*(module: PSym; cache: IdentCache; scriptName: string;
     setResult(a, os.findExe(getString(a, 0)))
 
   cbos rawExec:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       setResult(a, osproc.execCmd getString(a, 0))
 
   cbconf getEnv:
@@ -164,9 +185,15 @@ proc setupVM*(module: PSym; cache: IdentCache; scriptName: string;
   cbconf cppDefine:
     options.cppDefine(conf, a.getString(0))
   cbexc stdinReadLine, EOFError:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       setResult(a, "")
       setResult(a, stdin.readLine())
   cbexc stdinReadAll, EOFError:
+    if defined(nimsuggest) or graph.config.cmd == cmdCheck:
+      discard
+    else:
       setResult(a, "")
       setResult(a, stdin.readAll())
 

compiler/vm.nim

@@ -1549,6 +1549,9 @@ proc rawExecute(c: PCtx, start: int, tos: PStackFrame): TFullReg =
       regs[ra].node.strVal = opSlurp(regs[rb].node.strVal, c.debug[pc],
                                      c.module, c.config)
     of opcGorge:
+      if defined(nimsuggest) or c.config.cmd == cmdCheck:
+        discard "don't run staticExec for 'nim suggest'"
+      else:
         when defined(nimcore):
           decodeBC(rkNode)
           inc pc

compiler/vmops.nim

@@ -77,6 +77,14 @@ template wrap2svoid(op, modop) {.dirty.} =
     op(getString(a, 0), getString(a, 1))
   modop op
 
+template wrapDangerous(op, modop) {.dirty.} =
+  proc `op Wrapper`(a: VmArgs) {.nimcall.} =
+    if defined(nimsuggest) or c.config.cmd == cmdCheck:
+      discard
+    else:
+      op(getString(a, 0), getString(a, 1))
+  modop op
+
 proc getCurrentExceptionMsgWrapper(a: VmArgs) {.nimcall.} =
   setResult(a, if a.currentException.isNil: ""
                else: a.currentException.sons[3].skipColon.strVal)
@@ -133,7 +141,7 @@ proc registerAdditionalOps*(c: PCtx) =
     wrap2svoid(putEnv, osop)
     wrap1s(dirExists, osop)
     wrap1s(fileExists, osop)
-    wrap2svoid(writeFile, ioop)
+    wrapDangerous(writeFile, ioop)
     wrap1s(readFile, ioop)
     wrap2si(readLines, ioop)
     systemop getCurrentExceptionMsg