Merge remote-tracking branch 'origin/main' into NoDocumentPerformanceFix

Author: dconeybe

firebase-ai/src/main/kotlin/com/google/firebase/ai/common/Exceptions.kt

@@ -90,7 +90,7 @@ internal class UnsupportedUserLocationException(cause: Throwable? = null) :
   FirebaseCommonAIException("User location is not supported for the API use.", cause)
 
 /**
- * The user's project has not been configured and enabled for the selected API.
+ * The Firebase project has not been configured and enabled for the selected API.
  *
  * For the Gemini Developer API, see
  * [steps](https://firebase.google.com/docs/ai-logic/faq-and-troubleshooting?api=dev#error-genai-config-not-found)

firebase-ai/src/main/kotlin/com/google/firebase/ai/type/Exceptions.kt

@@ -152,7 +152,7 @@ public class UnsupportedUserLocationException internal constructor(cause: Throwa
   FirebaseAIException("User location is not supported for the API use.", cause)
 
 /**
- * The user's project has not been configured and enabled for the selected API.
+ * The Firebase project has not been configured and enabled for the selected API.
  *
  * For the Gemini Developer API, see
  * [steps](https://firebase.google.com/docs/ai-logic/faq-and-troubleshooting?api=dev#error-genai-config-not-found)

firebase-ai/src/main/kotlin/com/google/firebase/ai/type/GenerateContentResponse.kt

@@ -59,7 +59,7 @@ public class GenerateContentResponse(
 
   /**
    * Convenience field representing all the text parts in the response that are marked as thoughts
-   * as a single string, if they exists.
+   * as a single string, if they exist.
    *
    * Learn more about [thinking](https://firebase.google.com/docs/ai-logic/thinking?api=dev).
    */

firebase-ai/src/main/kotlin/com/google/firebase/ai/type/Part.kt

@@ -106,8 +106,6 @@ internal constructor(
 /**
  * Represents image data sent to and received from requests. The image is converted client-side to
  * JPEG encoding at 80% quality before being sent to the server.
- *
- * @param image [Bitmap] to convert into a [Part]
  */
 public class ImagePart
 internal constructor(
@@ -116,6 +114,7 @@ internal constructor(
   internal val thoughtSignature: String?
 ) : Part {
 
+  /** @param image [Bitmap] to convert into a [Part] */
   public constructor(image: Bitmap) : this(image, false, null)
 
   internal fun toInlineDataPart() =
@@ -127,13 +126,7 @@ internal constructor(
     )
 }
 
-/**
- * Represents binary data with an associated MIME type sent to and received from requests.
- *
- * @param inlineData the binary data as a [ByteArray]
- * @param mimeType an IANA standard MIME type. For supported values, see the
- * [Firebase documentation](https://firebase.google.com/docs/vertex-ai/input-file-requirements).
- */
+/** Represents binary data with an associated MIME type sent to and received from requests. */
 public class InlineDataPart
 internal constructor(
   public val inlineData: ByteArray,
@@ -142,6 +135,11 @@ internal constructor(
   internal val thoughtSignature: String?
 ) : Part {
 
+  /**
+   * @param inlineData the binary data as a [ByteArray]
+   * @param mimeType an IANA standard MIME type. For supported values, see the
+   * [Firebase documentation](https://firebase.google.com/docs/vertex-ai/input-file-requirements).
+   */
   public constructor(
     inlineData: ByteArray,
     mimeType: String
@@ -159,14 +157,7 @@ internal constructor(
   }
 }
 
-/**
- * Represents function call name and params received from requests.
- *
- * @param name the name of the function to call
- * @param args the function parameters and values as a [Map]
- * @param id Unique id of the function call. If present, the returned [FunctionResponsePart] should
- * have a matching `id` field.
- */
+/** Represents function call name and params received from requests. */
 public class FunctionCallPart
 internal constructor(
   public val name: String,
@@ -176,6 +167,12 @@ internal constructor(
   internal val thoughtSignature: String?
 ) : Part {
 
+  /**
+   * @param name the name of the function to call
+   * @param args the function parameters and values as a [Map]
+   * @param id Unique id of the function call. If present, the returned [FunctionResponsePart]
+   * should have a matching `id` field.
+   */
   @JvmOverloads
   public constructor(
     name: String,
@@ -199,13 +196,7 @@ internal constructor(
   }
 }
 
-/**
- * Represents function call output to be returned to the model when it requests a function call.
- *
- * @param name The name of the called function.
- * @param response The response produced by the function as a [JSONObject].
- * @param id Matching `id` for a [FunctionCallPart], if one was provided.
- */
+/** Represents function call output to be returned to the model when it requests a function call. */
 public class FunctionResponsePart
 internal constructor(
   public val name: String,
@@ -215,6 +206,11 @@ internal constructor(
   internal val thoughtSignature: String?
 ) : Part {
 
+  /**
+   * @param name The name of the called function.
+   * @param response The response produced by the function as a [JSONObject].
+   * @param id Matching `id` for a [FunctionCallPart], if one was provided.
+   */
   @JvmOverloads
   public constructor(
     name: String,
@@ -242,14 +238,7 @@ internal constructor(
   }
 }
 
-/**
- * Represents file data stored in Cloud Storage for Firebase, referenced by URI.
- *
- * @param uri The `"gs://"`-prefixed URI of the file in Cloud Storage for Firebase, for example,
- * `"gs://bucket-name/path/image.jpg"`
- * @param mimeType an IANA standard MIME type. For supported MIME type values see the
- * [Firebase documentation](https://firebase.google.com/docs/vertex-ai/input-file-requirements).
- */
+/** Represents file data stored in Cloud Storage for Firebase, referenced by URI. */
 public class FileDataPart
 internal constructor(
   public val uri: String,
@@ -258,6 +247,12 @@ internal constructor(
   internal val thoughtSignature: String?
 ) : Part {
 
+  /**
+   * @param uri The `"gs://"`-prefixed URI of the file in Cloud Storage for Firebase, for example,
+   * `"gs://bucket-name/path/image.jpg"`
+   * @param mimeType an IANA standard MIME type. For supported MIME type values see the
+   * [Firebase documentation](https://firebase.google.com/docs/vertex-ai/input-file-requirements).
+   */
   public constructor(uri: String, mimeType: String) : this(uri, mimeType, false, null)
 
   @Serializable

plugins/src/main/java/com/google/firebase/gradle/plugins/ci/AffectedProjectFinder.java

@@ -16,6 +16,7 @@
 
 import com.google.common.collect.ImmutableSet;
 import com.google.common.io.CharStreams;
+import com.google.common.io.Files;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStreamReader;
@@ -39,6 +40,8 @@ public AffectedProjectFinder(Project project, List<Pattern> ignorePaths) {
     this(project, changedPaths(project.getRootDir()), ignorePaths);
   }
 
+  private static final Set<String> DOC_EXTENSIONS = ImmutableSet.of("md", "txt", "html");
+
   public AffectedProjectFinder(
       Project project, Set<String> changedPaths, List<Pattern> ignorePaths) {
     this.project = project;
@@ -53,9 +56,17 @@ public AffectedProjectFinder(
                   }
                   return true;
                 })
+            .filter(p -> !isDocFile(p))
             .collect(Collectors.toSet());
   }
 
+  private static boolean isDocFile(String path) {
+    if (path.startsWith("docs/")) {
+      return true;
+    }
+    return DOC_EXTENSIONS.contains(Files.getFileExtension(path));
+  }
+
   Set<Project> find() {
     Set<String> paths = new HashSet<>(changedPaths);
     Set<Project> projects = changedSubProjects(project, paths);