imapflow 1.0.184 → 1.0.185

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.0.185](https://github.com/postalsys/imapflow/compare/v1.0.184...v1.0.185) (2025-04-14)
+
+
+### Bug Fixes
+
+* **search:** add a `not` option for search options ([#281](https://github.com/postalsys/imapflow/issues/281)) ([000686b](https://github.com/postalsys/imapflow/commit/000686b2ace1ef0e21cca64d8e056d1f7f7f8914))
+
 ## [1.0.184](https://github.com/postalsys/imapflow/compare/v1.0.183...v1.0.184) (2025-03-11)
 
 

package/lib/commands/delete.js

@@ -11,7 +11,7 @@ module.exports = async (connection, path) => {
 
     path = normalizePath(connection, path);
 
-    if (connection.states.SELECTED && connection.mailbox.path === path) {
+    if (connection.state === connection.states.SELECTED && connection.mailbox.path === path) {
         await connection.run('CLOSE');
     }
 

package/lib/commands/rename.js

@@ -12,7 +12,7 @@ module.exports = async (connection, path, newPath) => {
     path = normalizePath(connection, path);
     newPath = normalizePath(connection, newPath);
 
-    if (connection.states.SELECTED && connection.mailbox.path === path) {
+    if (connection.state === connection.states.SELECTED && connection.mailbox.path === path) {
         await connection.run('CLOSE');
     }
 

package/lib/imap-flow.js

@@ -281,6 +281,8 @@ class ImapFlow extends EventEmitter {
         this.socket = false;
         this.writeSocket = false;
 
+        this.isClosed = false;
+
         this.states = states;
         this.state = this.states.NOT_AUTHENTICATED;
 
@@ -391,7 +393,7 @@ class ImapFlow extends EventEmitter {
     }
 
     write(chunk) {
-        if (this.socket.destroyed) {
+        if (!this.socket || this.socket.destroyed) {
             // do not write after connection end or logout
             const error = new Error('Socket is already closed');
             error.code = 'NoConnection';
@@ -507,7 +509,7 @@ class ImapFlow extends EventEmitter {
     }
 
     async exec(command, attributes, options) {
-        if (this.socket.destroyed) {
+        if (!this.socket || this.socket.destroyed) {
             let error = new Error('Connection closed');
             error.code = 'EConnectionClosed';
             throw error;
@@ -873,7 +875,7 @@ class ImapFlow extends EventEmitter {
         };
 
         Object.defineProperty(this.writeSocket, 'destroyed', {
-            get: () => this.socket.destroyed
+            get: () => !this.socket || this.socket.destroyed
         });
 
         // we need to force flush deflated data to socket so we can't
@@ -1990,6 +1992,7 @@ class ImapFlow extends EventEmitter {
      * @property {String} [keyword] Matches messages that have the custom flag set
      * @property {String} [unKeyword] Matches messages that do not have the custom flag set
      * @property {Object.<string, Boolean|String>} [header] Matches messages with header key set if value is `true` (**NB!** not supported by all servers) or messages where header partially matches a string value
+     * @property {SearchObject} [not] A {@link SearchObject} object. It must not match.
      * @property {SearchObject[]} [or] An array of 2 or more {@link SearchObject} objects. At least on of these must match
      */
 
@@ -2480,7 +2483,7 @@ class ImapFlow extends EventEmitter {
 
         let res;
         while ((res = await getNext())) {
-            if (this.isClosed || this.socket.destroyed) {
+            if (this.isClosed || !this.socket || this.socket.destroyed) {
                 let error = new Error('Connection closed');
                 error.code = 'EConnectionClosed';
                 throw error;
@@ -3000,7 +3003,7 @@ class ImapFlow extends EventEmitter {
             return false;
         }
 
-        if (this.socket.destroyed) {
+        if (!this.socket || this.socket.destroyed) {
             const error = new Error('Connection not available');
             error.code = 'NoConnection';
             throw error;
@@ -3069,7 +3072,7 @@ class ImapFlow extends EventEmitter {
         const lock = this.locks.shift();
         const { resolve, reject, path, options, lockId } = lock;
 
-        if (!this.usable || this.socket.destroyed) {
+        if (!this.usable || !this.socket || this.socket.destroyed) {
             this.log.trace({ msg: 'Failed to acquire mailbox lock', path, lockId, idling: this.idling });
             // reject all
             let error = new Error('Connection not available');

package/lib/search-compiler.js

@@ -190,6 +190,19 @@ module.exports.searchCompiler = (connection, query) => {
                     }
                     break;
 
+                case 'NOT':
+                    {   
+                        if (!params[term]) {
+                            break;
+                        }
+
+                        if (typeof params[term] === 'object') {
+                            attributes.push({ type: 'ATOM', value: 'NOT' });
+                            walk(params[term]);
+                        }
+                    }
+                    break;
+                    
                 case 'OR':
                     {
                         if (!params[term] || !Array.isArray(params[term]) || !params[term].length) {

package/lib/tools.js

@@ -849,10 +849,9 @@ const tools = {
     },
 
     packMessageRange(list) {
-        if (typeof uidList === 'string')
-            if (!Array.isArray(list)) {
-                list = [].concat(list || []);
-            }
+        if (!Array.isArray(list)) {
+            list = [].concat(list || []);
+        }
 
         if (!list.length) {
             return '';

package/lib/types.d.ts

@@ -904,6 +904,7 @@ declare type SequenceString = string;
  * @property [keyword] - Matches messages that have the custom flag set
  * @property [unKeyword] - Matches messages that do not have the custom flag set
  * @property [header] - Matches messages with header key set if value is `true` (**NB!** not supported by all servers) or messages where header partially matches a string value
+ * @property [not] - A {@link SearchObject} object. It must not match.
  * @property [or] - An array of 2 or more {@link SearchObject} objects. At least on of these must match
  */
 declare type SearchObject = {
@@ -940,6 +941,7 @@ declare type SearchObject = {
     header?: {
         [key: string]: Boolean | String;
     };
+    not?: SearchObject;
     or?: SearchObject[];
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.0.184",
+    "version": "1.0.185",
     "description": "IMAP Client for Node",
     "main": "./lib/imap-flow.js",
     "scripts": {