chore(gateway): remove PR-number breadcrumbs from basePath comments

Author: mehara-rothila

gateway/gateway-controller/pkg/transform/restapi.go

@@ -465,7 +465,7 @@ func (t *RestAPITransformer) addUpstreamCluster(
 // unconditionally (see the upstreamDefinition cluster loop in Transform), so a per-op
 // route reuses it rather than minting its own cluster. One cluster per definition
 // serves both the main and sandbox vhosts — the key carries no env component. Reuse
-// also means the route inherits the definition's authoritative basePath (#2065),
+// also means the route inherits the definition's authoritative basePath,
 // avoiding the URL-derived-basePath divergence a separate per-op cluster would have.
 func perOpDefinitionClusterKey(kind, uuid string, target *api.RestAPIOperationUpstreamTarget, upstreamDefinitions *[]api.UpstreamDefinition) (string, error) {
 	def, err := upstreamref.FindByName(target.Ref, upstreamDefinitions)
@@ -489,7 +489,7 @@ func resolveUpstreamURL(name string, up *api.Upstream, defs *[]api.UpstreamDefin
 		refName := strings.TrimSpace(*up.Ref)
 		// Resolve through the shared upstreamref helper (one source of truth for ref
 		// lookup, shared with the per-op transformer and the xDS translator), and return
-		// the definition's base path (from basePath, #2065) so the caller rewrites the
+		// the definition's base path (from basePath) so the caller rewrites the
 		// upstream path correctly. The "no URLs" check stays here since FindByName
 		// resolves the definition, not its endpoints.
 		def, err := upstreamref.FindByName(refName, defs)

gateway/gateway-controller/pkg/xds/translator.go

@@ -836,7 +836,7 @@ func (t *Translator) translateAPIConfig(cfg *models.StoredConfig, allConfigs []*
 			}
 			// Reuse the referenced upstreamDefinition's cluster (built unconditionally
 			// below) instead of minting a per-op cluster. routeURLPath carries the
-			// definition's base path (#2065) so the route's static rewrite prepends it
+			// definition's base path so the route's static rewrite prepends it
 			// exactly once. Keep cluster_header ON with that cluster as the default so a
 			// dynamic-endpoint policy can still steer this operation. Precedence:
 			// op-policy > api-policy > per-op ref > api-level upstream.
@@ -917,7 +917,7 @@ func (t *Translator) translateAPIConfig(cfg *models.StoredConfig, allConfigs []*
 					return nil, nil, fmt.Errorf("per-op sandbox upstream for %s %s: %w", string(op.Method), op.Path, err)
 				}
 				// Reuse the referenced upstreamDefinition's cluster; routeURLPath carries
-				// its base path (#2065). Keep cluster_header ON so a sandbox dynamic-endpoint
+				// its base path. Keep cluster_header ON so a sandbox dynamic-endpoint
 				// policy can override the per-op default — consistent with the API-level
 				// sandbox routing fixed in #2059 (no static-pin bypass).
 				sbRouteCluster = defClusterName
@@ -1063,7 +1063,7 @@ func (t *Translator) resolveUpstreamCluster(apiID, upstreamName string, up *api.
 // (upstream_<kind>_<apiID>_<defName>) and base path. The definition's cluster is
 // created unconditionally for every definition, so a per-op route reuses it rather
 // than minting its own — one cluster per definition serves both vhosts (no env in
-// the key). The base path comes from the definition's basePath field (#2065); the
+// the key). The base path comes from the definition's basePath field; the
 // caller passes it as the route's upstream path so the static rewrite prepends it.
 func (t *Translator) resolvePerOpDefinitionCluster(kind, apiID string, target *api.RestAPIOperationUpstreamTarget, upstreamDefinitions *[]api.UpstreamDefinition) (string, string, *resolvedTimeout, error) {
 	refName := strings.TrimSpace(target.Ref)