components/microsoft_outlook/sources/new-email/new-email.mjs (1)

74-76: Log error details in catch to aid debugging (status/code/message).

-          } catch {
-            console.log(`Could not fetch message with ID: ${resourceId}`);
+          } catch (err) {
+            console.log(`Could not fetch message with ID: ${resourceId}`, {
+              status: err?.response?.status,
+              code: err?.response?.data?.error?.code,
+              message: err?.message,
+            });
           }

components/microsoft_outlook/actions/list-folders/list-folders.mjs (1)

32-34: Use strict equality in pluralization (nit).

-    $.export("$summary", `Successfully retrieved ${folders.length} folder${folders.length != 1
+    $.export("$summary", `Successfully retrieved ${folders.length} folder${folders.length !== 1
       ? "s"
       : ""}.`);

components/microsoft_outlook/sources/new-email-in-shared-folder/new-email-in-shared-folder.mjs (2)

70-73: Use message ID for dedupe and a numeric timestamp.
conversationId can collide across multiple messages; ts should be a number.

-            id: item.conversationId,
-            summary: `A new email with id: "${item.conversationId}" was received!`,
-            ts: item.createdDateTime,
+            id: item.id,
+            summary: `A new email with id: "${item.id}" was received!`,
+            ts: Date.parse(item.createdDateTime),

---

21-35: Clarify/verify required Graph permissions for userId options.
Populate of userId typically needs delegated User.Read.All or Directory.Read.All with admin consent; otherwise 403s occur. Ensure the app scopes and consent flow reflect this and update the prop description in the app to inform users.

components/microsoft_outlook/actions/list-labels/list-labels.mjs (1)

13-19: Default value to an empty array to avoid .length on undefined.

-    const { value } = await this.microsoftOutlook.listLabels({
+    const { value = [] } = await this.microsoftOutlook.listLabels({
       $,
     });
-    $.export("$summary", `Successfully retrieved ${value.length} label${value.length != 1
+    $.export("$summary", `Successfully retrieved ${value.length} label${value.length !== 1
       ? "s"
       : ""}.`);

components/microsoft_outlook/sources/new-attachment-received/new-attachment-received.mjs (3)

73-80: Avoid unnecessary base64 round‑trip; use Buffer directly.

-      const messageAttachment =  await this.microsoftOutlook.getAttachment({
+      const messageAttachment = await this.microsoftOutlook.getAttachment({
         messageId: item.messageId,
         attachmentId: item.id,
         responseType: "arraybuffer",
       });
-      const rawcontent = messageAttachment.toString("base64");
-      const buffer = Buffer.from(rawcontent, "base64");
+      const buffer = Buffer.isBuffer(messageAttachment)
+        ? messageAttachment
+        : Buffer.from(messageAttachment);

---

100-105: Stable dedupe key for attachments; fall back when contentId is missing.
Some attachments lack contentId. Prefer id, then contentId, then a composite.

-      return {
-        id: item.contentId,
-        summary: `New attachment ${item.name}`,
-        ts: Date.parse(item.messageReceivedDateTime),
-      };
+      return {
+        id: item.id ?? item.contentId ?? `${item.messageId}:${item.name}`,
+        summary: `New attachment ${item.name}`,
+        ts: Date.parse(item.messageReceivedDateTime),
+      };

---

130-132: Log error details in catch to aid debugging (status/code/message).

-          } catch {
-            console.log(`Could not fetch message with ID: ${resourceId}`);
+          } catch (err) {
+            console.log(`Could not fetch message with ID: ${resourceId}`, {
+              status: err?.response?.status,
+              code: err?.response?.data?.error?.code,
+              message: err?.message,
+            });
           }

components/microsoft_outlook/actions/update-contact/update-contact.mjs (1)

8-8: Update deprecated docs.microsoft.com URL.

Prefer learn.microsoft.com to avoid redirects.

- description: "Update an existing contact, [See the docs](https://docs.microsoft.com/en-us/graph/api/user-post-contacts)",
+ description: "Update an existing contact, [See the docs](https://learn.microsoft.com/graph/api/user-post-contacts)",

components/microsoft_outlook/actions/download-attachment/download-attachment.mjs (1)

41-53: Use the provided syncDir and avoid base64 round‑trip.

Write directly to the selected dir and avoid unnecessary encode/decode.

-    const response = await this.microsoftOutlook.getAttachment({
+    const data = await this.microsoftOutlook.getAttachment({
       $,
       messageId: this.messageId,
       attachmentId: this.attachmentId,
       responseType: "arraybuffer",
     });
-
-    const rawcontent = response.toString("base64");
-    const buffer = Buffer.from(rawcontent, "base64");
-    const downloadedFilepath = `/tmp/${this.filename}`;
-    fs.writeFileSync(downloadedFilepath, buffer);
+    const downloadedFilepath = `${this.syncDir}/${this.filename}`;
+    fs.writeFileSync(downloadedFilepath, Buffer.from(data));

Please confirm getAttachment returns the raw body (Buffer/ArrayBuffer) and not an Axios response object. If it returns { data }, adjust accordingly.

components/microsoft_outlook/actions/approve-workflow/approve-workflow.mjs (1)

31-35: Avoid spreading this into message options.

Limit fields to intended props to prevent leaking internal fields.

-    const opts = {
-      content: `Click here to approve the workflow: ${resume_url}, \nand cancel here: ${cancel_url}`,
-      ccRecipients: [],
-      bccRecipients: [],
-      ...this,
-    };
+    const opts = {
+      content: `Click here to approve the workflow: ${resume_url}, \nand cancel here: ${cancel_url}`,
+      ccRecipients: [],
+      bccRecipients: [],
+      recipients: this.recipients,
+      subject: this.subject,
+    };

components/microsoft_outlook/actions/create-contact/create-contact.mjs (1)

49-55: Guard against undefined emailAddresses.

Current code throws if emailAddresses is not provided.

-        emailAddresses: this.emailAddresses.map((a, i) => ({
+        emailAddresses: (this.emailAddresses || []).map((a, i) => ({
           address: a,
           name: `Email #${i + 1}`,
         })),

components/microsoft_outlook/actions/find-contacts/find-contacts.mjs (1)

31-46: Make matching case‑insensitive and use strict equality.

Improves UX for searches.

-      if (
-        contact?.displayName?.includes(this.searchString) ||
-        contact?.givenName?.includes(this.searchString) ||
-        contact?.surname?.includes(this.searchString) ||
-        contact?.emailAddresses?.find(
-          (e) => e?.address == this.searchString || e?.name?.includes(this.searchString),
-        )
-      ) {
+      const q = (this.searchString || "").toLowerCase();
+      const match =
+        contact?.displayName?.toLowerCase?.().includes(q) ||
+        contact?.givenName?.toLowerCase?.().includes(q) ||
+        contact?.surname?.toLowerCase?.().includes(q) ||
+        contact?.emailAddresses?.some((e) =>
+          e?.address?.toLowerCase?.() === q || e?.name?.toLowerCase?.().includes(q),
+        );
+      if (match) {

components/microsoft_outlook/actions/remove-label-from-email/remove-label-from-email.mjs (1)

34-39: Guard against missing categories to avoid runtime error.

labels can be undefined; labels.indexOf will throw.

Apply this diff:

-    let labels = message?.categories;
-
-    const index = labels.indexOf(this.label);
+    let labels = message?.categories ?? [];
+    if (!Array.isArray(labels)) labels = [];
+    const index = labels.indexOf(this.label);

components/microsoft_outlook/microsoft_outlook.app.mjs (2)

274-279: Auto-add ConsistencyLevel for advanced queries ($search/$count).

Graph requires ConsistencyLevel: eventual for these; set it centrally.

Apply this diff:

       const config = {
         url: this._getUrl(path),
         headers: this._getHeaders(headers),
         ...otherConfig,
       };
+      if (otherConfig?.params?.$search != null || otherConfig?.params?.$count != null) {
+        config.headers["ConsistencyLevel"] = "eventual";
+      }

---

91-95: Fix typos and labels in prop definitions.

Minor text polish.

Apply this diff:

-    emailAddresses: {
-      label: "Email adresses",
+    emailAddresses: {
+      label: "Email addresses",
       description: "Email addresses",
       type: "string[]",
       optional: true,
     },
-    businessPhones: {
-      label: "Recipients",
+    businessPhones: {
+      label: "Business Phones",
       description: "Array of phone numbers",
       type: "string[]",
       optional: true,
     },
-    search: {
+    search: {
       type: "string",
       label: "Search",
-      description: "Search for an email in Microsoft Outlook. Can search for specific message properties such as `to:[email protected]` or `subject:example`. If the property is excluded, the search targets the default propertes `from`, `subject`, and `body`. For example, `pizza` will search for messages with the word `pizza` in the subject, body, or from address, but `to:[email protected]` will only search for messages to `[email protected]`.",
+      description: "Search for an email in Microsoft Outlook. Can search for specific message properties such as `to:[email protected]` or `subject:example`. If the property is excluded, the search targets the default properties `from`, `subject`, and `body`. For example, `pizza` will search for messages with the word `pizza` in the subject, body, or from address, but `to:[email protected]` will only search for messages to `[email protected]`.",
       optional: true,
     },

Also applies to: 97-101, 239-243

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro