Design Meeting 2021-07-08

[kitsonk]

• --prompt by default (Security prompt by default (instead of throw) #10183)
  • General support for it if we address the security issues with promiting first (Runtime permissons request text could be hidden using iTerm custom escape codes #9666 and Runtime permissions request should fully consume STDIN before asking for user input #9750)
  • ACTION proceed with --prompt by default after issues resolved or mitigated
• window.location undefined without --location (fix(#10825): location should be undefined without --location #10828)
  • Lots of discussion about how we can have a location without a location being explicitly set.
  • @lucacasonato and @kitsonk highlight the challenges on localStorage and fetch() with some sort of implicit location that would need to be considered.
  • Discussion around usability of types having window.location defined, but undefined at runtime. It was considered acceptable as there are other APIs that are undefined at runtime, and it allows "sniffing" without throwing.
  • ACTION proceed with window.location being undefined.
  • ACTION discussion and research to continue on how we might be able to provide a default --location without breaking everything.
• Add a --worker flag to type check workers (feat(cli): Add --worker to cache and bundle #10050)
  • General dislike of adding flags.
  • There are current documented ways of type checking with workers (https://deno.land/[email protected]/typescript/types#type-checking-web-workers)
  • ACTION we will not proceed with feat(cli): Add --worker to cache and bundle #10050
• Discussion around how many pragmas we have
  • Most of the pragmas aren't created by Deno, just supported to support wider compatability
  • ACTION Venting in Cambrian explosion of comment pragmas #11334
• Syntax highlighting for deno lint, deno doc and REPL (REPL syntax highlighting is quirky #10998)
  • This highlighted that we probably shouldn't maintain context documentation in the CLI, deno lint should link to the lint.deno.land website instead of printing out rich documentation to the console. We should improve lint.deno.land versus maintaining documentation in to places.
• Skip type checking by default (Skip type checking by default #8549)
  • This has been a long running dicussion.
  • There are usability issues and performance issues with type checking all the time.
  • Most users use an editor which does the type checking, making type checking all the time redundant.
  • Ackowledged that the community is divided on this.
  • While behaviour changes, we think it is unlikely to break workflows, it just won't not-break when it should.
  • Broad agreement of those on the call that we should do this.
  • @kitsonk concerned that if emit is swc only that it will impact the code, as we are still finding bugs with swc emit.
  • ACTION deprecate --no-check (do this by default) and --check will continue to use tsc for the emit (Skip type checking by default #11340)