Address PR review feedback

- Revert ci.yml change that added feature/* to PR triggers
- Add documentation to SchemaOptions model and name property
- Add clarifying comment for regex in type-utils.ts
  (addresses false-positive HTML injection warning from security bot)

Author: FionaBronwen

eng/common/pipelines/ci.yml

@@ -8,7 +8,6 @@ pr:
     include:
       - main
       - release/*
-      - feature/*
 
 extends:
   template: /eng/common/pipelines/templates/1es-redirect.yml

packages/graphql/lib/schema.tsp

@@ -5,7 +5,12 @@ using TypeSpec.Reflection;
 namespace TypeSpec.GraphQL;
 
 namespace Schema {
+  /** Options for configuring a GraphQL schema. */
   model SchemaOptions {
+    /**
+     * The name of the GraphQL schema. Used in the output filename when emitting
+     * multiple schemas (e.g., `{name}.graphql`). Defaults to `"schema"`.
+     */
     name?: string;
   }
 }

packages/graphql/src/lib/type-utils.ts

@@ -78,6 +78,8 @@ export function stripNullVariants(union: Union): {
 /** Generate a GraphQL type name for a templated model (e.g., `ListOfString`). */
 export function getTemplatedModelName(model: Model): string {
   const name = getTypeName(model, {});
+  // Strip generic type parameters from compiler type names (e.g., "List<string>" → "List").
+  // This regex matches angle-bracket syntax, not HTML — output is GraphQL SDL identifiers.
   const baseName = toTypeName(name.replace(/<[^>]*>/g, ""));
   const templateString = getTemplateString(model);
   return templateString ? `${baseName}Of${templateString}` : baseName;