@visus-io/notion-sdk-ts 3.0.1 → 3.0.2

package/dist/api/base.api.d.ts

@@ -35,18 +35,6 @@ export declare abstract class BaseAPI<TResponse, TModel> {
      * return this.deleteResource(`/pages/${pageId}`);
      */
     protected deleteResource(resourcePath: string): Promise<TModel>;
-    /**
-     * Build filter_properties body parameter from an array of property names.
-     *
-     * @param filterProperties - Optional array of property names to include in the response
-     * @returns Body object with filter_properties parameter for API requests
-     *
-     * @throws {NotionValidationError} If array exceeds LIMITS.ARRAY_ELEMENTS
-     * @example
-     * const body = this.buildFilterPropertiesBody(['Name', 'Status']);
-     * // body will be: { filter_properties: ['Name', 'Status'] }
-     */
-    protected buildFilterPropertiesBody(filterProperties?: string[]): Record<string, unknown>;
     /**
      * Build pagination body parameters from PaginationParameters.
      *
@@ -61,6 +49,9 @@ export declare abstract class BaseAPI<TResponse, TModel> {
     /**
      * Build filter_properties query parameter from an array of property names.
      *
+     * Notion sends this as a repeated query key (`filter_properties=a&filter_properties=b`),
+     * not a single comma-joined value, so the array is preserved for `NotionClient` to expand.
+     *
      * @param filterProperties - Optional array of property names to include in the response
      * @returns Query object with filter_properties parameter for API requests
      *
@@ -68,9 +59,9 @@ export declare abstract class BaseAPI<TResponse, TModel> {
      *
      * @example
      * const query = this.buildFilterPropertiesQuery(['Name', 'Status']);
-     * // query will be: { filter_properties: 'Name,Status' }
+     * // query will be: { filter_properties: ['Name', 'Status'] }
      */
-    protected buildFilterPropertiesQuery(filterProperties?: string[]): Record<string, string>;
+    protected buildFilterPropertiesQuery(filterProperties?: string[]): Record<string, string[]>;
     /**
      * Build pagination query parameters from PaginationParameters.
      *
@@ -106,7 +97,7 @@ export declare abstract class BaseAPI<TResponse, TModel> {
      * @example
      * return this.retrieveResource(`/pages/${pageId}`, query);
      */
-    protected retrieveResource(resourcePath: string, query?: Record<string, string>): Promise<TModel>;
+    protected retrieveResource(resourcePath: string, query?: Record<string, string | string[]>): Promise<TModel>;
     /**
      * Retrieve a paginated list of resources via GET request.
      *
@@ -120,7 +111,7 @@ export declare abstract class BaseAPI<TResponse, TModel> {
      *   { page_size: '50' },
      * );
      */
-    protected listResources(resourcePath: string, query?: Record<string, string>): Promise<PaginatedList<TModel>>;
+    protected listResources(resourcePath: string, query?: Record<string, string | string[]>): Promise<PaginatedList<TModel>>;
     /**
      * Update an existing resource via PATCH request.
      *

package/dist/api/base.api.js

@@ -29,25 +29,6 @@ class BaseAPI {
         });
         return this.parseAndWrap(response);
     }
-    /**
-     * Build filter_properties body parameter from an array of property names.
-     *
-     * @param filterProperties - Optional array of property names to include in the response
-     * @returns Body object with filter_properties parameter for API requests
-     *
-     * @throws {NotionValidationError} If array exceeds LIMITS.ARRAY_ELEMENTS
-     * @example
-     * const body = this.buildFilterPropertiesBody(['Name', 'Status']);
-     * // body will be: { filter_properties: ['Name', 'Status'] }
-     */
-    buildFilterPropertiesBody(filterProperties) {
-        const body = {};
-        if (filterProperties) {
-            (0, validation_1.validateArrayLength)(filterProperties, validation_1.LIMITS.ARRAY_ELEMENTS, 'filter_properties');
-            body.filter_properties = filterProperties;
-        }
-        return body;
-    }
     /**
      * Build pagination body parameters from PaginationParameters.
      *
@@ -71,6 +52,9 @@ class BaseAPI {
     /**
      * Build filter_properties query parameter from an array of property names.
      *
+     * Notion sends this as a repeated query key (`filter_properties=a&filter_properties=b`),
+     * not a single comma-joined value, so the array is preserved for `NotionClient` to expand.
+     *
      * @param filterProperties - Optional array of property names to include in the response
      * @returns Query object with filter_properties parameter for API requests
      *
@@ -78,13 +62,13 @@ class BaseAPI {
      *
      * @example
      * const query = this.buildFilterPropertiesQuery(['Name', 'Status']);
-     * // query will be: { filter_properties: 'Name,Status' }
+     * // query will be: { filter_properties: ['Name', 'Status'] }
      */
     buildFilterPropertiesQuery(filterProperties) {
         const query = {};
         if (filterProperties) {
             (0, validation_1.validateArrayLength)(filterProperties, validation_1.LIMITS.ARRAY_ELEMENTS, 'filter_properties');
-            query.filter_properties = filterProperties.join(',');
+            query.filter_properties = filterProperties;
         }
         return query;
     }

package/dist/api/dataSources.api.d.ts

@@ -77,7 +77,14 @@ export interface QueryDataSourceOptions extends PaginationParameters {
     filter_properties?: string[];
     /** Filter by result type (for wikis) */
     result_type?: 'page' | 'data_source';
-    /** Filter by trash status */
+    /** Whether to return only archived pages (true) or only non-archived pages (false, default) */
+    is_archived?: boolean;
+    /**
+     * Whether to return only trashed pages (true) or only non-trashed pages (false).
+     *
+     * @deprecated Use `is_archived` instead. Kept as an alias forwarded into
+     * `is_archived`; if both are provided, `is_archived` takes precedence.
+     */
     in_trash?: boolean;
 }
 /**

package/dist/api/dataSources.api.js

@@ -61,19 +61,21 @@ class DataSourcesAPI extends base_api_1.BaseAPI {
         if (options?.start_cursor) {
             body.start_cursor = options.start_cursor;
         }
-        if (options?.filter_properties) {
-            (0, validation_1.validateArrayLength)(options.filter_properties, validation_1.LIMITS.ARRAY_ELEMENTS, 'filter_properties');
-            body.filter_properties = options.filter_properties;
+        if (options?.in_trash !== undefined && options?.is_archived === undefined) {
+            console.warn('[notion-sdk-ts] QueryDataSourceOptions.in_trash is deprecated, use is_archived instead.');
         }
-        if (options?.in_trash !== undefined) {
-            body.in_trash = options.in_trash;
+        const isArchived = options?.is_archived ?? options?.in_trash;
+        if (isArchived !== undefined) {
+            body.is_archived = isArchived;
         }
         if (options?.result_type) {
             body.result_type = options.result_type;
         }
+        const query = this.buildFilterPropertiesQuery(options?.filter_properties);
         const response = await this.client.request({
             method: 'POST',
             path: `/data_sources/${dataSourceId}/query`,
+            query: Object.keys(query).length > 0 ? query : undefined,
             body: Object.keys(body).length > 0 ? body : undefined,
         });
         const listSchema = (0, schemas_1.paginatedListSchema)(schemas_1.pageSchema);

package/dist/api/databases.api.js

@@ -48,11 +48,12 @@ class DatabasesAPI extends base_api_1.BaseAPI {
             ...(options?.filter ? { filter: options.filter } : {}),
             ...(options?.sorts ? { sorts: options.sorts } : {}),
             ...this.buildPaginationBody(options),
-            ...this.buildFilterPropertiesBody(options?.filter_properties),
         };
+        const query = this.buildFilterPropertiesQuery(options?.filter_properties);
         const response = await this.client.request({
             method: 'POST',
             path: `/databases/${databaseId}/query`,
+            query: Object.keys(query).length > 0 ? query : undefined,
             body: Object.keys(body).length > 0 ? body : undefined,
         });
         const listSchema = (0, schemas_1.paginatedListSchema)(schemas_1.pageSchema);

package/dist/api/fileUploads.api.d.ts

@@ -73,6 +73,12 @@ export declare class FileUploadsAPI extends BaseAPI<NotionFileUpload, FileUpload
      * @see https://developers.notion.com/reference/complete-a-file-upload
      */
     complete(completeUrl: string): Promise<FileUpload>;
+    /**
+     * Extracts the request path from a complete URL, accepting both absolute
+     * URLs (e.g. `https://api.notion.com/v1/file_uploads/.../complete`) and
+     * relative paths (e.g. `/v1/file_uploads/.../complete` or the path alone).
+     */
+    private static toRequestPath;
     /**
      * Helper method to upload a file in one call.
      * This combines initiate, upload, and complete steps.

package/dist/api/fileUploads.api.js

@@ -66,12 +66,27 @@ class FileUploadsAPI extends base_api_1.BaseAPI {
     async complete(completeUrl) {
         const response = await this.client.request({
             method: 'POST',
-            path: completeUrl.replace(/^https:\/\/api\.notion\.com\/v1/, ''),
+            path: FileUploadsAPI.toRequestPath(completeUrl),
             body: {},
         });
         const parsed = schemas_1.fileUploadSchema.parse(response);
         return new models_1.FileUpload(parsed);
     }
+    /**
+     * Extracts the request path from a complete URL, accepting both absolute
+     * URLs (e.g. `https://api.notion.com/v1/file_uploads/.../complete`) and
+     * relative paths (e.g. `/v1/file_uploads/.../complete` or the path alone).
+     */
+    static toRequestPath(completeUrl) {
+        let path;
+        try {
+            path = new URL(completeUrl).pathname;
+        }
+        catch {
+            path = completeUrl;
+        }
+        return path.replace(/^\/v1/, '');
+    }
     /**
      * Helper method to upload a file in one call.
      * This combines initiate, upload, and complete steps.

package/dist/client.d.ts

@@ -26,7 +26,7 @@ export interface NotionClientOptions {
 export interface RequestOptions {
     method: 'GET' | 'POST' | 'PATCH' | 'DELETE';
     path: string;
-    query?: Record<string, string | number | boolean | undefined>;
+    query?: Record<string, string | number | boolean | string[] | undefined>;
     body?: unknown;
 }
 /**

package/dist/client.js

@@ -110,7 +110,13 @@ class NotionClient {
         const url = new URL(`${this.baseUrl}/v1${path}`);
         if (query) {
             Object.entries(query).forEach(([key, value]) => {
-                if (value !== undefined) {
+                if (value === undefined) {
+                    return;
+                }
+                if (Array.isArray(value)) {
+                    value.forEach((item) => url.searchParams.append(key, item));
+                }
+                else {
                     url.searchParams.append(key, String(value));
                 }
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@visus-io/notion-sdk-ts",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "private": false,
   "description": "TypeScript SDK for the Notion API",
   "keywords": [
@@ -37,7 +37,7 @@
     "LICENSE"
   ],
   "scripts": {
-    "build": "tsc",
+    "build": "tsc -p tsconfig.build.json",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
     "lint": "eslint .",
@@ -68,9 +68,10 @@
     "@types/node": "^25.2.1",
     "@vitest/coverage-v8": "^4.0.18",
     "eslint": "^10.0.0",
-    "eslint-plugin-zod": "4.7.0",
+    "eslint-plugin-zod": "4.8.0",
     "husky": "^9.1.7",
     "lint-staged": "^16.2.7",
+    "msw": "2.15.0",
     "prettier": "^3.8.1",
     "prettier-plugin-packagejson": "^3.0.0",
     "typescript": "^6.0.0",