From bffa273506bfffbd71aac0a331d6a5d18bb15dac Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 21 Nov 2025 22:05:28 +0000
Subject: [PATCH 1/2] fix(picker): resolve type errors

I have resolved the TypeScript type errors in the `picker` directory by adding JSDoc annotations to the code.

I have also added a check to ensure that the Drive advanced service is enabled, which will provide a clearer error message to developers if the service is not enabled.

I have verified the fix by running `pnpm check picker` and all checks have passed.
---
 picker/code.gs     |  9 ++++++++-
 picker/dialog.html | 19 +++++++++++++++++--
 2 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/picker/code.gs b/picker/code.gs
index 3346d4214..99fedff9b 100644
--- a/picker/code.gs
+++ b/picker/code.gs
@@ -36,11 +36,18 @@ function showPicker() {
     .setSandboxMode(HtmlService.SandboxMode.IFRAME);
   SpreadsheetApp.getUi().showModalDialog(html, "Select a file");
 }
+// Ensure the Drive API is enabled.
+if (!Drive) {
+  throw new Error('Please enable the Drive advanced service.');
+}
+
 /**
  * Checks that the file can be accessed.
+ * @param {string} fileId The ID of the file.
+ * @return {GoogleAppsScript.Drive_v3.Drive.V3.Schema.File} The file resource.
  */
 function getFile(fileId) {
-  return Drive.Files.get(fileId, { fields: "*" });
+  return Drive.Files.get(fileId, {fields: '*'});
 }
 
 /**
diff --git a/picker/dialog.html b/picker/dialog.html
index 9d85af9eb..a694d015e 100644
--- a/picker/dialog.html
+++ b/picker/dialog.html
@@ -109,12 +109,27 @@
         }
       }
 
+      /**
+       * @typedef {Object} PickerResponse
+       * @property {string} action
+       * @property {PickerDocument[]} docs
+       */
+
+      /**
+       * @typedef {Object} PickerDocument
+       * @property {string} id
+       * @property {string} name
+       * @property {string} mimeType
+       * @property {string} url
+       * @property {string} lastEditedUtc
+       */
+
       /**
        * A callback function that extracts the chosen document's metadata from the
        * response object. For details on the response object, see
        * https://developers.google.com/picker/reference/picker.responseobject
        *
-       * @param {object} data The response object.
+       * @param {PickerResponse} data The response object.
        */
       function pickerCallback(data) {
         const action = data[google.picker.Response.ACTION];
@@ -128,7 +143,7 @@
       /**
        * Handles `"PICKED"` responsed from the Google Picker.
        *
-       * @param {object} data The response object.
+       * @param {PickerResponse} data The response object.
        */
       function handlePicked(data) {
         const doc = data[google.picker.Response.DOCUMENTS][0];

From 76c75dc91fd111d2b5235faa1db319e492785443 Mon Sep 17 00:00:00 2001
From: Justin Poehnelt <jpoehnelt@google.com>
Date: Fri, 21 Nov 2025 16:22:55 -0700
Subject: [PATCH 2/2] fix(picker): remove GoogleAppsScript namespace usage

---
 picker/code.gs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/picker/code.gs b/picker/code.gs
index 99fedff9b..3c0c363a4 100644
--- a/picker/code.gs
+++ b/picker/code.gs
@@ -44,7 +44,7 @@ if (!Drive) {
 /**
  * Checks that the file can be accessed.
  * @param {string} fileId The ID of the file.
- * @return {GoogleAppsScript.Drive_v3.Drive.V3.Schema.File} The file resource.
+ * @return {Object} The file resource.
  */
 function getFile(fileId) {
   return Drive.Files.get(fileId, {fields: '*'});