fix(internal/config): add IncludeList to dart config (#3837)

Prepare for Dart migration, add `IncludeList` to DartPackage config.
Pass the value to sidekick config in library generation.

For #3580

Author: JoeWang1127

internal/config/language.go

@@ -290,6 +290,9 @@ type DartPackage struct {
 	// ExtraImports is additional imports to include in the generated library.
 	ExtraImports string `yaml:"extra_imports,omitempty"`
 
+	// IncludeList is a list of items to include.
+	IncludeList []string `yaml:"include_list,omitempty"`
+
 	// IssueTrackerURL is the URL for the issue tracker.
 	IssueTrackerURL string `yaml:"issue_tracker_url,omitempty"`
 

internal/librarian/dart/generate.go

@@ -18,6 +18,7 @@ package dart
 
 import (
 	"context"
+	"strings"
 
 	"github.com/googleapis/librarian/internal/command"
 	"github.com/googleapis/librarian/internal/config"
@@ -65,6 +66,9 @@ func toSidekickConfig(library *config.Library, ch *config.API, googleapisDir str
 	if library.Dart != nil && library.Dart.TitleOverride != "" {
 		source["title-override"] = library.Dart.TitleOverride
 	}
+	if library.Dart != nil && len(library.Dart.IncludeList) > 0 {
+		source["include-list"] = strings.Join(library.Dart.IncludeList, ",")
+	}
 
 	api, err := serviceconfig.Find(googleapisDir, ch.Path)
 	if err != nil {

internal/librarian/dart/generate_test.go

@@ -440,6 +440,36 @@ func TestToSidekickConfig(t *testing.T) {
 				Codec: map[string]string{},
 			},
 		},
+		{
+			name: "with include list",
+			library: &config.Library{
+				Dart: &config.DartPackage{
+					IncludeList: []string{
+						"api.proto",
+						"duration.proto",
+						"empty.proto",
+						"field_mask.proto",
+					},
+				},
+			},
+			channel: &config.API{
+				Path: "google/cloud/secretmanager/v1",
+			},
+			googleapisDir: googleapisDir,
+			want: &sidekickconfig.Config{
+				General: sidekickconfig.GeneralConfig{
+					Language:            "dart",
+					SpecificationFormat: "protobuf",
+					ServiceConfig:       "",
+					SpecificationSource: "google/cloud/secretmanager/v1",
+				},
+				Source: map[string]string{
+					"googleapis-root": googleapisDir,
+					"include-list":    "api.proto,duration.proto,empty.proto,field_mask.proto",
+				},
+				Codec: map[string]string{},
+			},
+		},
 		{
 			name: "with dart package",
 			library: &config.Library{

tool/cmd/migrate/main.go

@@ -336,6 +336,9 @@ func buildGAPIC(files []string, repoPath string) ([]*config.Library, error) {
 		if extraImports, ok := sidekick.Codec["extra-imports"]; ok && extraImports != "" {
 			dartPackage.ExtraImports = extraImports
 		}
+		if includeList, ok := sidekick.Source["include-list"]; ok && includeList != "" {
+			dartPackage.IncludeList = strings.Split(includeList, ",")
+		}
 		if partFile, ok := sidekick.Codec["part-file"]; ok && partFile != "" {
 			// part-file in .sidekick.toml starts with src/, however, the file path is
 			// actually {output}/lib/src/*.

tool/cmd/migrate/main_test.go

@@ -297,10 +297,10 @@ API key as an argument when initializing the client.
 			},
 			want: []*config.Library{
 				{
-					Name: "google_cloud_protobuf",
+					Name: "google_cloud_appengine_v1",
 					APIs: []*config.API{
 						{
-							Path: "google/protobuf",
+							Path: "google/appengine/v1",
 						},
 					},
 					Output: "testdata/read-sidekick-files/roots-update",
@@ -343,6 +343,37 @@ API key as an argument when initializing the client.
 				},
 			},
 		},
+		{
+			name: "with include list",
+			files: []string{
+				"testdata/read-sidekick-files/include-list/.sidekick.toml",
+			},
+			want: []*config.Library{
+				{
+					Name: "google_cloud_protobuf",
+					APIs: []*config.API{
+						{
+							Path: "google/protobuf",
+						},
+					},
+					Output:              "testdata/read-sidekick-files/include-list",
+					SpecificationFormat: "protobuf",
+					Dart: &config.DartPackage{
+						IncludeList: []string{
+							"api.proto",
+							"duration.proto",
+							"empty.proto",
+							"field_mask.proto",
+							"source_context.proto",
+							"struct.proto",
+							"timestamp.proto",
+							"type.proto",
+							"wrappers.proto",
+						},
+					},
+				},
+			},
+		},
 	} {
 		t.Run(test.name, func(t *testing.T) {
 			got, err := buildGAPIC(test.files, test.repoPath)

tool/cmd/migrate/testdata/read-sidekick-files/include-list/.sidekick.toml

@@ -0,0 +1,30 @@
+# Copyright 2025 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+[general]
+specification-source = 'google/protobuf'
+
+[source]
+# Don't generate 'any.proto'; use the hand-written version in protobuf.p.dart.
+include-list = """\
+  api.proto,\
+  duration.proto,\
+  empty.proto,\
+  field_mask.proto,\
+  source_context.proto,\
+  struct.proto,\
+  timestamp.proto,\
+  type.proto,\
+  wrappers.proto\
+"""

tool/cmd/migrate/testdata/read-sidekick-files/roots-update/.sidekick.toml

@@ -13,19 +13,7 @@
 # limitations under the License.
 
 [general]
-specification-source = 'google/protobuf'
+specification-source = 'google/appengine/v1'
 
 [source]
-roots                = 'protobuf'
-# Don't generate 'any.proto'; use the hand-written version in protobuf.p.dart.
-include-list = """\
-  api.proto,\
-  duration.proto,\
-  empty.proto,\
-  field_mask.proto,\
-  source_context.proto,\
-  struct.proto,\
-  timestamp.proto,\
-  type.proto,\
-  wrappers.proto\
-"""
+roots                = 'protobuf'