chore(serviceconfig): make Path the first field in API (#4702)

This just moves the Path field in the API type to the top, expanding the
comment to explain that it's effectively the key within the list of
APIs. The existing sdk.yaml file is then regenerated.

Also adds python to the allow-list for google/maps/navconnect/v1.

When modifying sdk.yaml, it's been all to easy to find a path which is
within the middle of the relevant section, and then change the wrong
field. While it's useful for most fields to be in alphabetical order, it
makes sense to have this at the top, just as Name is at the top of the
Library type.

Fixes #4499

Author: Jon Skeet

doc/api-allowlist-schema.md

@@ -6,14 +6,14 @@ This document describes the schema for the API Allowlist.
 
 | Field | Type | Description |
 | :--- | :--- | :--- |
+| `path` | string | Is the proto directory path in github.com/googleapis/googleapis. If ServiceConfig is empty, the service config is assumed to live at this path. |
 | `description` | string | Provides the information for describing an API. |
 | `discovery` | string | Is the file path to a discovery document in github.com/googleapis/discovery-artifact-manager. Used by sidekick languages (Rust, Dart) as an alternative to proto files. |
 | `documentation_uri` | string | Overrides the product documentation URI from the service config's publishing section. |
 | `languages` | list of string | Restricts which languages can generate client libraries for this API. Empty means all languages can use this API. We should be explicit about supported languages when adding entries.<br><br>Restrictions exist for several reasons:<br>- Newer languages (Rust, Dart) skip older beta versions when stable versions exist<br>- Python has historical legacy APIs not available to other languages<br>- Some APIs (like DIREGAPIC protos) are only used by specific languages |
 | `new_issue_uri` | string | Overrides the new issue URI from the service config's publishing section. |
 | `no_rest_numeric_enums` | map[string]bool | Determines whether to use numeric enums in REST requests. The "No" prefix is used because the default behavior (when this field is `false` or omitted) is to generate numeric enums. Map key is the language name (e.g., "python", "rust"). Optional. If omitted, the generator default is used. |
 | `open_api` | string | Is the file path to an OpenAPI spec, currently in internal/testdata. This is not an official spec yet and exists only for Rust to validate OpenAPI support. |
-| `path` | string | Is the proto directory path in github.com/googleapis/googleapis. If ServiceConfig is empty, the service config is assumed to live at this path. |
 | `release_level` | map[string]string | Is the release level per language. Map key is the language name (e.g., "python", "rust"). Optional. If omitted, the generator default is used. |
 | `short_name` | string | Overrides the API short name from the service config's publishing section. |
 | `service_config` | string | Is the service config file path override. If empty, the service config is discovered in the directory specified by Path. |

internal/serviceconfig/api.go

@@ -39,6 +39,15 @@ const (
 
 // API describes an API path and its availability across languages.
 type API struct {
+	// Note: Properties should typically be added in alphabetical order, but
+	// because this order impacts YAML serialization, we keep Path at the top
+	// for ease of consumption in file-form.
+
+	// Path is the proto directory path in github.com/googleapis/googleapis.
+	// If ServiceConfig is empty, the service config is assumed to live at this
+	// path.
+	Path string `yaml:"path,omitempty"`
+
 	// Description provides the information for describing an API.
 	Description string `yaml:"description,omitempty"`
 
@@ -76,10 +85,6 @@ type API struct {
 	// This is not an official spec yet and exists only for Rust to validate OpenAPI support.
 	OpenAPI string `yaml:"open_api,omitempty"`
 
-	// Path is the proto directory path in github.com/googleapis/googleapis.
-	// If ServiceConfig is empty, the service config is assumed to live at this path.
-	Path string `yaml:"path,omitempty"`
-
 	// ReleaseLevels is the release level per language.
 	// Map key is the language name (e.g., "python", "rust").
 	// Optional. If omitted, the generator default is used.