Incorrect selectAll typing on CTE with same tablename

[JustGreg]

Hello, there seems to be a type error when the CTE has the same name as an existing table.

// result1 should have id and name from 'pet' but got type of Person
const result1 = await db
  .with("person", (db) =>
    db
      .selectFrom("pet")
      .where("is_favorite", "=", true)
      .select(["id", "name"]),
  )
  .selectFrom("person")
  .selectAll()
  .execute();

// result2 should only have id and first_name but got type of Person
const result2 = await db
  .with("person", (db) =>
    db
      .selectFrom("person")
      .where("age", ">", 60)
      .select(["id", "first_name"]),
  )
  .selectFrom("person")
  .selectAll()
  .execute();

selectFrom always select from the existing table but postgresql is selecting the CTE.

playground link

[koskimas]

This is a similar issue to this. We probably merge the tables instead of replacing the existing table. At least in this case, we should replace the person table in the database interface with the new CTE.

But to make your query cleaner, I'd avoid using an existing name for your CTE.