[Bug]: isTemplateDeclaration and isTemplateDeclarationOrInstance take TemplatedType instead of just Type

[TomerAberbach]

Describe the bug

isTemplateDeclaration and isTemplateDeclarationOrInstance take TemplatedType instead of just Type. This makes them a lot less useful and inconsistent with isTemplateInstance, which takes just a Type.

Presumably they were meant to take Type because they each have a type assertion of is TemplatedType, which is redundant with the parameter already being TemplatedType.

I ended up having to write this in my code (and checked that first) so that I could pass a type to isTemplateDeclaration:

const isTemplatedType = (type: Type): type is TemplatedType => {
  switch (type.kind) {
    case `Scalar`:
    case `Union`:
    case `Model`:
    case `Interface`:
    case `Operation`:
      return true
    default:
      return false
  }
}

Reproduction

See above

Checklist

• Follow our Code of Conduct
• Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion.
• The provided reproduction is a minimal reproducible example of the bug.