This comes from #35862 and the couple of examples I poked at look correct; @mjbvz you'll probably want to fix these soonish.

specific to master -- did @types/ws just update?

this is new, and correct

probably due to discord updating

probably updates to follow-redirects?

these are correct as far as TS' current understanding of JS. We don't expect people to overwrite methods in the constructor:

class C {
  constructor(fs) {
    this.readFile = fs.readFile ? fs.readFile : null
    if (this.readFile) {
      this._readJson = (path, callback) => this.readFile(...)
    }
  }
  readFile(path, callback) {
  }
}

This is from "CachedInputFileSystem.js", which tells you all need to know I think.

not clear whether to trust the types or the values in this case, but they definitely don't agree.

upon reading more examples, they look like they're just leftovers from pre-strictNullChecks that the compiler didn't notice before.