[Internal] Remove resolved host metadata from HostType() method (#1630)

## Changes

Removing this to unblock the Terraform provider release followed by the
Go SDK bump.

Partially reverts #1606 — removes the `resolvedHostType` early-return in
`HostType()` and the associated tests.

  Specifically:
- Removes the `if c.resolvedHostType != HostTypeUnknown { return
c.resolvedHostType }` block from `HostType()`
- Removes the `// When host metadata has been resolved...` comment that
described the reverted behavior
  - Keeps the `// Deprecated:` comment on `HostType()`
- Removes 7 test cases that validated the metadata-first behavior
(`TestHostType_UsesMetadataFirst_*`,
`TestHostType_FallsBackToURLWhenMetadataUnknown*`,
`TestHostType_MetadataOverridesExperimentalFlag`,
`TestHostType_EndToEnd_MetadataResolvesHostType`)
  - Removes the changelog entry for this change

`HostType()` now returns to its pre-#1606 behavior: URL-pattern-only
detection, which is what downstream consumers (TF provider) still rely
on.

  NO_CHANGELOG=true

  ## Tests

Existing `config` package tests pass (`go test ./config/...`). The
removed tests were specific to the reverted behavior.

Signed-off-by: Tanmay Rustagi <tanmay.rustagi@databricks.com>

Author: tanmay-db

NEXT_CHANGELOG.md

@@ -25,7 +25,6 @@
 
 ### Internal Changes
 
- * Use resolved host type from host metadata in `HostType()` method, falling back to URL pattern matching when metadata is unavailable.
  * Normalize internal token sources on `auth.TokenSource` for proper context propagation ([#1577](https://github.com/databricks/databricks-sdk-go/pull/1577)).
  * Fix `TestAzureGithubOIDCCredentials` hang caused by missing `HTTPTransport` stub: `EnsureResolved` now calls `resolveHostMetadata`, which makes a real network request when no transport is set ([#1550](https://github.com/databricks/databricks-sdk-go/pull/1550)).
  * Bump golang.org/x/crypto from 0.21.0 to 0.45.0 in /examples/slog ([#1566](https://github.com/databricks/databricks-sdk-go/pull/1566)).

config/config.go

@@ -447,19 +447,11 @@ func normalizedHost(host string) string {
 }
 
 // HostType returns the type of host that the client is configured for.
-// When host metadata has been resolved (via /.well-known/databricks-config),
-// the resolved host type is returned directly. Otherwise, the host type is
-// inferred from URL patterns as a fallback.
 //
 // Deprecated: The SDK is moving towards host-agnostic behavior where host URL
 // patterns are no longer used to determine client capabilities. This method is
 // retained for backwards compatibility but should not be used in new code.
 func (c *Config) HostType() HostType {
-	// If host metadata resolved a known host type, use it.
-	if c.resolvedHostType != HostTypeUnknown {
-		return c.resolvedHostType
-	}
-
 	// TODO: Remove this after TF updates its code.
 	if c.Experimental_IsUnifiedHost {
 		return UnifiedHost

config/config_test.go

@@ -86,75 +86,6 @@ func TestHostType_UnifiedFlagNoLongerReturnsUnified(t *testing.T) {
 	assert.Equal(t, WorkspaceHost, c.HostType())
 }
 
-func TestHostType_UsesMetadataFirst_Workspace(t *testing.T) {
-	c := &Config{
-		Host:             "https://accounts.cloud.databricks.com",
-		resolvedHostType: WorkspaceHost,
-	}
-	assert.Equal(t, WorkspaceHost, c.HostType())
-}
-
-func TestHostType_UsesMetadataFirst_Account(t *testing.T) {
-	c := &Config{
-		Host:             "https://my-workspace.cloud.databricks.com",
-		resolvedHostType: AccountHost,
-	}
-	assert.Equal(t, AccountHost, c.HostType())
-}
-
-func TestHostType_UsesMetadataFirst_Unified(t *testing.T) {
-	c := &Config{
-		Host:             "https://my-workspace.cloud.databricks.com",
-		resolvedHostType: UnifiedHost,
-	}
-	assert.Equal(t, UnifiedHost, c.HostType())
-}
-
-func TestHostType_FallsBackToURLWhenMetadataUnknown(t *testing.T) {
-	c := &Config{
-		Host:             "https://accounts.cloud.databricks.com",
-		resolvedHostType: HostTypeUnknown,
-	}
-	assert.Equal(t, AccountHost, c.HostType())
-}
-
-func TestHostType_FallsBackToURLWhenMetadataUnknown_Workspace(t *testing.T) {
-	c := &Config{
-		Host:             "https://my-workspace.cloud.databricks.com",
-		resolvedHostType: HostTypeUnknown,
-	}
-	assert.Equal(t, WorkspaceHost, c.HostType())
-}
-
-func TestHostType_MetadataOverridesExperimentalFlag(t *testing.T) {
-	c := &Config{
-		Host:                       "https://my-workspace.cloud.databricks.com",
-		Experimental_IsUnifiedHost: true,
-		resolvedHostType:           AccountHost,
-	}
-	assert.Equal(t, AccountHost, c.HostType())
-}
-
-func TestHostType_EndToEnd_MetadataResolvesHostType(t *testing.T) {
-	noopLoader := mockLoader(func(cfg *Config) error { return nil })
-	cfg := &Config{
-		Host:    "https://my-workspace.cloud.databricks.com",
-		Loaders: []Loader{noopLoader},
-		HTTPTransport: fixtures.SliceTransport{
-			{
-				Method:       "GET",
-				Resource:     "/.well-known/databricks-config",
-				ReuseRequest: true,
-				Status:       200,
-				Response:     `{"oidc_endpoint": "https://my-workspace.cloud.databricks.com/oidc", "account_id": "` + testHMAccountID + `", "host_type": "account"}`,
-			},
-		},
-	}
-	err := cfg.EnsureResolved()
-	require.NoError(t, err)
-	assert.Equal(t, AccountHost, cfg.HostType())
-}
-
 func TestIsAccountClient_NoLongerPanicsOnUnifiedHost(t *testing.T) {
 	c := &Config{
 		Host:      "https://unified.cloud.databricks.com",