asserts x is type doesn't work when using arrow functions

[mweststrate]

TypeScript Version: 3.7.0-dev.20191016

Search Terms: assertion signatures, asserts, asserts is type

Code

function isString1(val: any): asserts val is string {
    if (typeof val !== "string") throw "Nope"
} 

const isString2 = (val: any): asserts val is string => {
    if (typeof val !== "string") throw "Nope"
} 

const x: any = "3";

isString1(x);
isString2(x);

x.toUpperCase();

Expected behavior:

Compiles, infers that x is a string, both when using only isString1 or isString2

Actual behavior:

Compile error on isString2:

Assertions require every name in the call target to be declared with an explicit type annotation.(2775)

Playground Link: 1⃣ playground

Related Issues: #33743

Context

In principle the above problem is easy to work around, however, the result of it is when type checkers are stored on objects, the above error returns as is demonstrated in 2⃣ this playground. Note how the very same function fails when it is stored in an object, unless the type is provided explicitly.

[jcalz]

This is (perhaps unfortunately) working as intended, as per #33622.

[mweststrate]

Thanks for looking into it! Nothing I cannot work around in the end. Just felt surprising / inconsistent :)

[pstrh]

As mentioned here (#33622 (comment)) a workaround is:

const isString2: (val: unknown) => asserts val is string = (val) => {
    if (typeof val !== "string") throw new Error ("Nope");
};

const x: unknown = "3";

isString2(x);

x.toUpperCase();

[Pyrolistical]

A better error message on this design limitation would have been nice instead of the confusing TS 2775

[voltrevo]

This is (perhaps unfortunately) working as intended, as per #33622.

@jcalz Unfortunate indeed. Do you know the rationale behind this intention? What would be the complication of allowing arrow functions to have type assertions?

[sezanzeb]

A better error message on this design limitation would have been nice instead of the confusing TS 2775

I created a new issue for that: #53450

[kaspar-p]

Is there a way to type object literal members in the above way? I can't seem to get either to work:

const assertions = {
  isString(val: unknown): asserts val is string {
    if (typeof val !== "string") throw new Error("Nope");
  },

  isString2: (val: unknown): asserts val is string => {
    if (typeof val !== "string") throw new Error("Nope");
  }
}

[Moriarty47]

Just like pstrh this comment said.

const assertions: {
  isString: (val: unknown) => asserts val is string;
  isString2: (val: unknown) => asserts val is string;
} = {
  isString(val) {
    if (typeof val !== "string") throw new Error("Nope");
  },
  isString2: (val) => {
    if (typeof val !== "string") throw new Error("Nope");
  }
}

const x: unknown = '123';
assertions.isString(x)

x.toLowerCase();