deno bundle should support type checking worker scripts

[nekdolan]

When I try to bundle my worker.ts it raises an error

error: TS2339 [ERROR]: Property 'postMessage' does not exist on type 'Window & typeof globalThis'.
        self.postMessage(data);

Code

self.postMessage(data);

In Deno

deno 1.5.1
v8 8.7.220.3
typescript 4.0.3

[kitsonk]

We need to think about this more, but there are two workarounds.

> deno bundle --no-check

Or:

> deno bundle --config tsconfig.json

Where the tsconfig.json contains:

{
  "compilerOptions": {
    "lib": [ "deno.worker" ]
  }
}

[nekdolan]

Thanks @kitsonk deno bundle --no-check works

[kitsonk]

Actually, I realised I made a comment in the code.

deno/cli/main.rs

Line 344 in 96e03e0

// TODO(@kitsonk) support bundling for workers

Ultimately we just need to agree on how to deal with this. @ry @bartlomieju thoughts?

[nayeemrmn]

This is basically #6753 (deno cache --worker), the flag would be shared like similar ones are.

[kitsonk]

@nayeemrmn you are right as usual... deno bundle --worker is what it should be, nothing to debate... I forgot we had agreed that for deno cache.

[kitsonk]

Actually based on #8089 there is clearly things to discuss...

[nekdolan]

I'm not sure how it could be done without the proposed/reverted flag --worker . I can understand that cache could work without it but how would you keep the linkage when you bundle them, given that you don't know where or how the scripts will be deployed? Would it just keep a relative link to the bundled worker in the script? Would that work on a normal webpage? Not being able to give a name to the bundled worker file would also be an issue.

[lucacasonato]

deno bundle was removed in Deno 2.