config: remove fields not documented in config/README.md

Summary:
Intent:
- This branch builds on #190 (config: update fields, defaults, and required
  checks to match README), which aligned names/defaults/required-ness with
  config/README.md but left several fields that the README doesn't document
  at all. This removes them so the Go structs match the documented surface
  exactly.

Changes:
- Removed RepositoryConfig.FullHashRepos, ExcludedFiles, ExcludeExternalTargets,
  and StreamBazelLogs, and ServiceConfig.WorkerRootPath.
- These fields drove real behavior, so this is a narrow behavior change, not
  just a doc cleanup: graphrunner/native.go now always includes external
  targets in the Bazel query and no longer applies a repo-level full-hash or
  excluded-file list (per-request excludes via RequestOptions still work),
  orchestrator/native_orchestrator.go no longer streams Bazel logs, and
  example/main.go derives the worker checkout path as
  workspaces_root/.workers instead of reading a separate config field.
- Updated config/config_test.go and the YAML fixtures
  (example/tango-config.yaml, orchestrator/testdata/config.yaml,
  integration/testdata/tango-config.yaml.tmpl) to drop the removed keys.

---

<sub>Generated by the 🪄 [pr-create](https://sg.uberinternal.com/code.uber.internal/uber-code/devexp-agent-marketplace/-/blob/claude-code/plugins/dev/uber-dev/skills/pr-create/SKILL.md) skill in devexp-agent-marketplace</sub>

Author: yushan8

config/config.go

@@ -17,7 +17,6 @@ package config
 import (
 	"fmt"
 	"os"
-	"path/filepath"
 
 	yaml "github.com/goccy/go-yaml"
 )
@@ -72,9 +71,6 @@ func Parse(configFilePath string) (*Config, error) {
 	if config.Service.WorkspacesRoot == "" {
 		return nil, fmt.Errorf("service.workspaces_root must be set")
 	}
-	if config.Service.WorkerRootPath == "" {
-		config.Service.WorkerRootPath = filepath.Join(config.Service.WorkspacesRoot, ".workers")
-	}
 	if config.Service.MaxWorkerPoolSize <= 0 {
 		return nil, fmt.Errorf("service.max_worker_pool_size must be > 0, got %d", config.Service.MaxWorkerPoolSize)
 	}

config/config_test.go

@@ -74,33 +74,16 @@ func TestParse_ServiceDefaults(t *testing.T) {
 	tests := []struct {
 		name                      string
 		give                      string
-		wantWorkerRootPath        string
 		wantMaxNumTargets         int
 		wantMaxNumChangedTargets  int
 		wantMaxNumMetadataEntries int
 	}{
 		{
-			name: "worker_root_path and streaming default when unset",
+			name: "streaming defaults when unset",
 			give: _baseServiceYAML + `
 repository:
   - remote: "r1"
 `,
-			wantWorkerRootPath:        filepath.Join("/tmp/x", ".workers"),
-			wantMaxNumTargets:         _defaultMaxNumTargets,
-			wantMaxNumChangedTargets:  _defaultMaxNumChangedTargets,
-			wantMaxNumMetadataEntries: _defaultMaxNumMetadataEntries,
-		},
-		{
-			name: "worker_root_path explicit value preserved",
-			give: `
-service:
-  workspaces_root: "/tmp/x"
-  worker_root_path: "/tmp/custom-workers"
-  max_worker_pool_size: 1
-repository:
-  - remote: "r1"
-`,
-			wantWorkerRootPath:        "/tmp/custom-workers",
 			wantMaxNumTargets:         _defaultMaxNumTargets,
 			wantMaxNumChangedTargets:  _defaultMaxNumChangedTargets,
 			wantMaxNumMetadataEntries: _defaultMaxNumMetadataEntries,
@@ -115,7 +98,6 @@ repository:
 repository:
   - remote: "r1"
 `,
-			wantWorkerRootPath:        filepath.Join("/tmp/x", ".workers"),
 			wantMaxNumTargets:         10,
 			wantMaxNumChangedTargets:  20,
 			wantMaxNumMetadataEntries: 30,
@@ -126,7 +108,6 @@ repository:
 		t.Run(tt.name, func(t *testing.T) {
 			cfg, err := Parse(writeConfig(t, tt.give))
 			require.NoError(t, err)
-			assert.Equal(t, tt.wantWorkerRootPath, cfg.Service.WorkerRootPath)
 			assert.Equal(t, tt.wantMaxNumTargets, cfg.Service.Streaming.MaxNumTargets)
 			assert.Equal(t, tt.wantMaxNumChangedTargets, cfg.Service.Streaming.MaxNumChangedTargets)
 			assert.Equal(t, tt.wantMaxNumMetadataEntries, cfg.Service.Streaming.MaxNumMetadataEntries)

config/repository_config.go

@@ -20,23 +20,19 @@ type RepositoryConfig struct {
 	// this URL and uses it as the lookup key for per-repo settings. Must be
 	// unique across all entries and match exactly what clients send in
 	// BuildDescription.remote.
-	Remote                 string   `yaml:"remote"`
-	FullHashRepos          []string `yaml:"full_hash_repos"`
-	ExcludedFiles          []string `yaml:"excluded_files"`
-	ExcludeExternalTargets bool     `yaml:"exclude_external_targets"`
+	Remote string `yaml:"remote"`
 	// BzlmodEnabled indicates whether this repository uses Bzlmod for external
 	// dependency management. Defaults to true if unset. Set to false only for
 	// repositories still using WORKSPACE.
 	BzlmodEnabled *bool `yaml:"bzlmod_enabled"`
 	// BazelCommand overrides the Bazel binary path. When empty, Tango
 	// automatically downloads and caches Bazelisk from GitHub.
 	BazelCommand string `yaml:"bazel_command"`
-	// QueryTimeoutSeconds is the Bazel query timeout in seconds. Defaults to 600.
-	QueryTimeoutSeconds int64 `yaml:"query_timeout_seconds"`
 	// BazelExtraArgs are extra arguments passed to `bazel query` invocations,
 	// inserted between the `query` subcommand and the query expression.
-	BazelExtraArgs  []string `yaml:"bazel_extra_args"`
-	StreamBazelLogs bool     `yaml:"stream_bazel_logs"`
+	BazelExtraArgs []string `yaml:"bazel_extra_args"`
+	// QueryTimeoutSeconds is the Bazel query timeout in seconds. Defaults to 600.
+	QueryTimeoutSeconds int64 `yaml:"query_timeout_seconds"`
 }
 
 // RepositoryConfigProvider looks up per-repository configuration by remote.

config/service_config.go

@@ -25,8 +25,7 @@ type ServiceConfig struct {
 	// origin clones and <workspaces_root>/.workers/<repo>/worker-{1..N}/ for
 	// worker checkouts.
 	WorkspacesRoot string      `yaml:"workspaces_root"`
-	WorkerRootPath string      `yaml:"worker_root_path"` // root directory for worker workspace checkouts; defaults to workspaces_root/.workers
-	Streaming      ChunkConfig `yaml:"streaming"`        // streaming chunk sizes; zero values fall back to package defaults
+	Streaming      ChunkConfig `yaml:"streaming"` // streaming chunk sizes; zero values fall back to package defaults
 }
 
 // ChunkConfig controls the number of entries per gRPC stream message.

example/main.go

@@ -72,7 +72,7 @@ func run() error {
 
 	// Repo manager and orchestrator
 	repoManagerClonePath := cfg.Service.WorkspacesRoot
-	workerRootPath := cfg.Service.WorkerRootPath
+	workerRootPath := filepath.Join(repoManagerClonePath, ".workers")
 	if err := os.MkdirAll(repoManagerClonePath, 0o755); err != nil {
 		return fmt.Errorf("failed to create repo manager clone path: %w", err)
 	}

example/tango-config.yaml

@@ -9,11 +9,6 @@ storage:
 # Repository configuration
 repository:
   - remote: "https://github.com/uber/tango.git"
-    full_hash_repos:
-      - ""
-    excluded_files:
-      - "^@@?bazel_tools/"
-    exclude_external_targets: true
     bzlmod_enabled: true
     query_timeout_seconds: 300
 
@@ -22,5 +17,3 @@ service:
   max_worker_pool_size: 5
   # root for origin clones; required
   workspaces_root: "/tmp/tango-repo-manager"
-  # root for worker checkouts; defaults to workspaces_root/.workers
-  worker_root_path: "/tmp/tango/workers"

graphrunner/native.go

@@ -59,9 +59,6 @@ func NewNativeGraphRunner(p NativeGraphRunnerParams) GraphRunner {
 
 func (g *nativeGraphRunner) Compute(ctx context.Context, ws workspace.Workspace) (targethasher.Result, error) {
 	query := "//external:all-targets + deps(//...:all-targets)"
-	if g.config.ExcludeExternalTargets {
-		query = "deps(//...:all-targets)"
-	}
 	additionalArgs := append(
 		[]string{"--order_output=no", "--proto:locations", "--noproto:default_values"},
 		g.config.BazelExtraArgs...,
@@ -92,8 +89,7 @@ func (g *nativeGraphRunner) Compute(ctx context.Context, ws workspace.Workspace)
 
 	hashConfig := targethasher.HashConfig{
 		KnownSourceHashes: knownSourceHashes,
-		FullHashRepos:     g.config.FullHashRepos,
-		ExcludedRegex:     append(g.config.ExcludedFiles, g.extraExcludedFiles...),
+		ExcludedRegex:     g.extraExcludedFiles,
 		UseBzlmod:         g.config.BzlmodEnabled == nil || *g.config.BzlmodEnabled,
 	}
 

integration/testdata/tango-config.yaml.tmpl

@@ -7,13 +7,9 @@ repository:
 {{- if .BazelCommand}}
     bazel_command: {{.BazelCommand}}
 {{- end}}
-    full_hash_repos: [""]
-    excluded_files: ["^@@?bazel_tools/"]
-    exclude_external_targets: true
     bzlmod_enabled: true
     query_timeout_seconds: 600
 
 service:
   max_worker_pool_size: 2
   workspaces_root: {{.ClonePath}}
-  worker_root_path: {{.WorkerPath}}

orchestrator/native_orchestrator.go

@@ -188,7 +188,6 @@ func (b *nativeOrchestrator) GetTargetGraph(ctx context.Context, param GetTarget
 			Logger:        b.logger,
 			BazelCommand:  repoCfg.BazelCommand,
 			QueryTimeout:  time.Duration(repoCfg.QueryTimeoutSeconds) * time.Second,
-			StreamLogs:    repoCfg.StreamBazelLogs,
 		})
 		if err != nil {
 			return nil, fmt.Errorf("create bazel client: %w", err)

orchestrator/testdata/config.yaml

@@ -1,11 +1,5 @@
 repository:
   - remote: "git@github:uber/tango"
-    full_hash_repos:
-      - "//"
-      - "//external"
-    excluded_files:
-      - "*.gen.go"
-    exclude_external_targets: true
     bzlmod_enabled: true
     query_timeout_seconds: 15