chore: remove unused v1 secret/variable client constructors

Addresses Cursor Bugbot review. newCISecretServiceClient and
newCIVariableServiceClient were dead code after v1→v2 migration.

Made-with: Cursor

Author: 121watts

pkg/api/ci.go

@@ -103,11 +103,6 @@ func CIListRuns(ctx context.Context, token, orgID string, statuses []civ1.CIRunS
 	return allRuns, nil
 }
 
-func newCISecretServiceClient() civ1connect.SecretServiceClient {
-	baseURL := baseURLFunc()
-	return civ1connect.NewSecretServiceClient(getHTTPClient(baseURL), baseURL, WithUserAgent())
-}
-
 func newCISecretServiceV2Client() civ2connect.SecretServiceClient {
 	baseURL := baseURLFunc()
 	return civ2connect.NewSecretServiceClient(getHTTPClient(baseURL), baseURL, WithUserAgent())
@@ -195,11 +190,6 @@ func CIDeleteSecret(ctx context.Context, token, orgID, name, repo string) error
 	return err
 }
 
-func newCIVariableServiceClient() civ1connect.VariableServiceClient {
-	baseURL := baseURLFunc()
-	return civ1connect.NewVariableServiceClient(getHTTPClient(baseURL), baseURL, WithUserAgent())
-}
-
 func newCIVariableServiceV2Client() civ2connect.VariableServiceClient {
 	baseURL := baseURLFunc()
 	return civ2connect.NewVariableServiceClient(getHTTPClient(baseURL), baseURL, WithUserAgent())