Skip to content

Implicit type guard by other variables are not always workingΒ #46915

Closed
@Egor-Koldasov

Description

@Egor-Koldasov

Bug Report

πŸ”Ž Search Terms

Implicit type guard by other variables.

Related

#44730

πŸ•— Version & Regression Information

  • This is the behavior in every version I tried, and I reviewed the FAQ for entries about type guards

⏯ Playground Link

Playground link

πŸ’» Code

const state = {
  prop1: 'value1',
  prop2: 'value2',
  prop3: 'value3',
}

type State = typeof state;

type Keys = (keyof State)

const makeSetProp = (path: Keys | undefined) => {
  // "setProp" is not null if "path" is not undefined
  const setProp = 
    path ?
      (value: string) => state[path] = value :
      null;
  // "data" is not null is "path" is not undefined
  const data =
    path ?
      'nextValue' :
      null;

  // "data" is always not null if "setProp" is not null, but the compiler does not recognize it
  if (setProp) setProp(data); // Argument of type 'string | null' is not assignable to parameter of type 'string'
  if (data) setProp(data); // Cannot invoke an object which is possibly 'null'.
  if (path) setProp(data); // Cannot invoke an object which is possibly 'null'. Argument of type 'string | null' is not assignable to parameter of type 'string'.
  if (setProp && data) setProp(data); // works
}

πŸ™ Actual behavior

  • data does not narrow by checking the setProp
  • setProp does not narrow by checking the data
  • data and setProp both do not narrow by checking the path even though they were defined by it

πŸ™‚ Expected behavior

Logically they all can be narrowed by each other:

  • data can be narrowed by checking the setProp or path
  • setProp can be narrowed by checking the data or path

Metadata

Metadata

Assignees

No one assigned

    Labels

    Design LimitationConstraints of the existing architecture prevent this from being fixed

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions