Many functions of util should not be listed in preflight

[meirdev]

I tried this:

bring util;

class Foo {
  new() {
    util.shell("sleep 1 && echo Hello > ../file.txt");

    let output = util.shell("cat ../file.txt", throw: true);
    log("{output}");
  }
}

new Foo();

This happened:

An error has occurred:
Failed to compile.

Error: Error executing command "cat ../file.txt". Exited with error: cat: ../file.txt: No such file or directory

I expected this:

No response

Is there a workaround?

No response

Anything else?

Because async functions are called normally (without await) they continue to run in the background and the following lines of code are called.

I think in this case it is better to hide async functions from a preflight block.

Wing Version

0.62.3

Node.js Version

v20.11.0

Platform(s)

MacOS

Community Notes

• Please vote by adding a 👍 reaction to the issue to help us prioritize.
• If you are interested to work on this issue, please leave a comment.

[Chriscbr]

Ideally it would be nice to support these util functions in preflight as well - we just need to have separate preflight and inflight implementations of each relevant method. But flagging them as inflight-only for now sounds like a good solution to prevent the awkward error.

[staycoolcall911]

Seems quite trivial to just define shell and the other async functions to run inflight in https://github.com/winglang/wing/blob/main/libs/wingsdk/src/util/util.ts

[revitalbarletz]

Gowtham Kasi will take a look.

[garysassano]

This should actually be fixed by #5425 once it gets merged.

[revitalbarletz]

#5425 is fixed - @Chriscbr can we close this one as well?