[PR Workflow] Update REST API PR labeling bot for TypeSpec rename

[mikekistler]

Some time back we added support in our labelling bot to add the "Cadl" label to any PRs that included files with the ".cadl" suffix. With the rename of Cadl to TypeSpec, we need to update this automation to look for files with the ".tsp" suffix and when found add the "TypeSpec" label (I already created the label).

Related work:

• [PR Workflow] Add TypeSpec label if appropriate files are changed #7353

[konrad-jamrozik]

Ideally to be fixed within the next 30 days.

[maririos]

If this is enabled, this could be used by the Release Planner apps and we can stop asking for this data manually.

[konrad-jamrozik]

Relevant code:

export function isSwaggerGeneratedByCadl(swaggerFilePath:string) {
  try {
    return !!JSON.parse(readFileSync(swaggerFilePath).toString())?.info['x-cadl-generated']
  }
  catch{
    return false
  }
}

@weshaggard @mikeharder looks like we have automation that when it detects x-cadl-generated, it adds the cadl label. Do we want to follow this model, do instead regex file path matching as explained in #7353, or do both? If we want to do do such x- marker, how should it be called? x-typespec-generated?

Example of x-cadl-generated

{
  "swagger": "2.0",
  "info": {
    "title": "Microsoft.VoiceServices",
    "version": "2023-01-31",
    "x-cadl-generated": [
      {
        "emitter": "@azure-tools/cadl-autorest"
      }
    ]
  },

Example of automation adding cadl on 11/1/2023, i.e. a month ago.

[mikekistler]

We should do both -- check for TypeSpec files and check for TypeSpec-generated OpenAPI.

OpenAPI generated from TypeSpec will have an x-typespec-generated marker.

[konrad-jamrozik]

@mikekistler thank you for the attribute name pointer! When I at first used GitHub search to find attributes like that it returned no results. I think I wasn't paying enough attention and had a bug in my search query (:

[konrad-jamrozik]

Closed as implemented and deployed. For details, see here:

• [PR Workflow] Add TypeSpec label if appropriate files are changed #7353 (comment)