cozy-viewer 26.1.2 → 26.2.1

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [26.2.1](https://github.com/cozy/cozy-libs/compare/cozy-viewer@26.2.0...cozy-viewer@26.2.1) (2025-12-15)
+
+**Note:** Version bump only for package cozy-viewer
+
+# [26.2.0](https://github.com/cozy/cozy-libs/compare/cozy-viewer@26.1.2...cozy-viewer@26.2.0) (2025-12-15)
+
+### Bug Fixes
+
+- Remove json parser when getting drive.summary flag :bug: ([a565f84](https://github.com/cozy/cozy-libs/commit/a565f848918c9dbf2819e370b3b84a77188ba28f))
+
+### Features
+
+- Save result from summarization by AI :sparkles: ([c89a034](https://github.com/cozy/cozy-libs/commit/c89a034ea912e0ff444920a7611488fcc8734c23))
+
 ## [26.1.2](https://github.com/cozy/cozy-libs/compare/cozy-viewer@26.1.1...cozy-viewer@26.1.2) (2025-12-12)
 
 **Note:** Version bump only for package cozy-viewer

package/dist/Panel/AI/AIAssistantPanel.js

@@ -9,6 +9,8 @@ exports.default = void 0;
 
 var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"));
 
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
 
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
@@ -29,6 +31,8 @@ var _ai = require("cozy-client/dist/models/ai");
 
 var _file = require("cozy-client/dist/models/file");
 
+var _cozyFlags = _interopRequireDefault(require("cozy-flags"));
+
 var _cozyLogger = _interopRequireDefault(require("cozy-logger"));
 
 var _Buttons = _interopRequireDefault(require("cozy-ui/transpiled/react/Buttons"));
@@ -63,6 +67,10 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+
 var styles = {
   "loaderContainer": "styles__loaderContainer___RuLAO",
   "loaderBar": "styles__loaderBar___2_kw1",
@@ -101,6 +109,9 @@ var AIAssistantPanel = function AIAssistantPanel(_ref) {
 
   var location = (0, _reactRouterDom.useLocation)();
   var navigate = (0, _reactRouterDom.useNavigate)();
+  var fetchedFileIdRef = (0, _react.useRef)(null);
+  var inFlightFileIdRef = (0, _react.useRef)(null);
+  var activeFileIdRef = (0, _react.useRef)((file === null || file === void 0 ? void 0 : file._id) || null);
 
   var handleClose = function handleClose() {
     var _location$state;
@@ -135,7 +146,7 @@ var AIAssistantPanel = function AIAssistantPanel(_ref) {
 
             case 7:
               textContent = _context.sent;
-              summaryConfig = (0, _helpers.getSummaryConfig)();
+              summaryConfig = (0, _cozyFlags.default)('drive.summary');
 
               if (!(summaryConfig !== null && summaryConfig !== void 0 && summaryConfig.maxTokens && (0, _helpers.roughTokensEstimation)(textContent) > summaryConfig.maxTokens)) {
                 _context.next = 13;
@@ -185,63 +196,145 @@ var AIAssistantPanel = function AIAssistantPanel(_ref) {
     };
   }();
 
-  var fetchSummary = (0, _react.useCallback)( /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2() {
-    var response, errorMessage;
-    return _regenerator.default.wrap(function _callee2$(_context2) {
+  var persistedSummary = /*#__PURE__*/function () {
+    var _ref4 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2(fileMetadata, targetFileId, summaryContent) {
+      return _regenerator.default.wrap(function _callee2$(_context2) {
+        while (1) {
+          switch (_context2.prev = _context2.next) {
+            case 0:
+              _context2.prev = 0;
+              _context2.next = 3;
+              return client.collection('io.cozy.files').updateMetadataAttribute(targetFileId, _objectSpread(_objectSpread({}, fileMetadata), {}, {
+                description: summaryContent
+              }));
+
+            case 3:
+              fetchedFileIdRef.current = targetFileId;
+              _context2.next = 9;
+              break;
+
+            case 6:
+              _context2.prev = 6;
+              _context2.t0 = _context2["catch"](0);
+
+              _cozyLogger.default.error('Error when persisting summary to file metadata:', _context2.t0);
+
+            case 9:
+            case "end":
+              return _context2.stop();
+          }
+        }
+      }, _callee2, null, [[0, 6]]);
+    }));
+
+    return function persistedSummary(_x2, _x3, _x4) {
+      return _ref4.apply(this, arguments);
+    };
+  }();
+
+  (0, _react.useEffect)(function () {
+    activeFileIdRef.current = (file === null || file === void 0 ? void 0 : file._id) || null;
+  }, [file]);
+  var fetchSummary = (0, _react.useCallback)( /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee3() {
+    var force,
+        targetFileId,
+        _response$choices,
+        _response$choices$,
+        _response$choices$$me,
+        response,
+        summaryContent,
+        fileMetadata,
+        errorMessage,
+        _args3 = arguments;
+
+    return _regenerator.default.wrap(function _callee3$(_context3) {
       while (1) {
-        switch (_context2.prev = _context2.next) {
+        switch (_context3.prev = _context3.next) {
           case 0:
-            if (file) {
-              _context2.next = 2;
+            force = _args3.length > 0 && _args3[0] !== undefined ? _args3[0] : false;
+            targetFileId = file === null || file === void 0 ? void 0 : file._id;
+
+            if (targetFileId) {
+              _context3.next = 4;
               break;
             }
 
-            return _context2.abrupt("return");
+            return _context3.abrupt("return");
+
+          case 4:
+            if (!(!force && (fetchedFileIdRef.current === targetFileId || inFlightFileIdRef.current === targetFileId))) {
+              _context3.next = 6;
+              break;
+            }
 
-          case 2:
+            return _context3.abrupt("return");
+
+          case 6:
+            inFlightFileIdRef.current = targetFileId;
             setIsLoading(true);
             setError(null);
-            _context2.prev = 4;
-            _context2.next = 7;
+            _context3.prev = 9;
+            _context3.next = 12;
             return summarizeFile({
               client: client,
               file: file,
               stream: false
             });
 
-          case 7:
-            response = _context2.sent;
+          case 12:
+            response = _context3.sent;
+            summaryContent = (response === null || response === void 0 ? void 0 : response.content) || (response === null || response === void 0 ? void 0 : (_response$choices = response.choices) === null || _response$choices === void 0 ? void 0 : (_response$choices$ = _response$choices[0]) === null || _response$choices$ === void 0 ? void 0 : (_response$choices$$me = _response$choices$.message) === null || _response$choices$$me === void 0 ? void 0 : _response$choices$$me.content); // Ignore results if the user switched to another file meanwhile
 
-            if (response && response.content) {
-              setSummary(response.content);
-            } else if (response && response.choices && response.choices[0]) {
-              setSummary(response.choices[0].message.content);
+            if (!(activeFileIdRef.current !== targetFileId)) {
+              _context3.next = 16;
+              break;
             }
 
-            _context2.next = 15;
+            return _context3.abrupt("return");
+
+          case 16:
+            setSummary(summaryContent);
+            fileMetadata = file.metadata || {};
+            _context3.next = 20;
+            return persistedSummary(fileMetadata, targetFileId, summaryContent);
+
+          case 20:
+            _context3.next = 25;
             break;
 
-          case 11:
-            _context2.prev = 11;
-            _context2.t0 = _context2["catch"](4);
-            errorMessage = _context2.t0.code === 'DOCUMENT_TOO_LARGE' ? t('Viewer.ai.error.documentTooLarge') : t('Viewer.ai.error.summary');
-            setError(errorMessage);
+          case 22:
+            _context3.prev = 22;
+            _context3.t0 = _context3["catch"](9);
 
-          case 15:
-            _context2.prev = 15;
-            setIsLoading(false);
-            return _context2.finish(15);
+            if (activeFileIdRef.current === targetFileId) {
+              errorMessage = _context3.t0.code === 'DOCUMENT_TOO_LARGE' ? t('Viewer.ai.error.documentTooLarge') : t('Viewer.ai.error.summary');
+              setError(errorMessage);
+            }
+
+          case 25:
+            _context3.prev = 25;
+
+            if (inFlightFileIdRef.current === targetFileId) {
+              inFlightFileIdRef.current = null;
+            }
 
-          case 18:
+            if (activeFileIdRef.current === targetFileId) {
+              setIsLoading(false);
+            }
+
+            return _context3.finish(25);
+
+          case 29:
           case "end":
-            return _context2.stop();
+            return _context3.stop();
         }
       }
-    }, _callee2, null, [[4, 11, 15, 18]]);
+    }, _callee3, null, [[9, 22, 25, 29]]);
   })), [client, file, t]);
 
   var handleRefresh = function handleRefresh() {
-    fetchSummary();
+    fetchedFileIdRef.current = null;
+    fetchSummary(true);
   };
 
   var handleCopy = function handleCopy() {

package/dist/helpers.d.ts

@@ -13,7 +13,6 @@ export function makeWebLink({ client, slug, path }: {
 }): string;
 export function removeFilenameFromPath(path: string): string;
 export function roughTokensEstimation(text: string): number;
-export function getSummaryConfig(): object | null;
 export function isFileSummaryCompatible(file: object, options?: {
     pageCount: number;
 }): boolean;

package/dist/helpers.js

@@ -5,7 +5,7 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.roughTokensEstimation = exports.removeFilenameFromPath = exports.normalizeAndSpreadAttributes = exports.makeWebLink = exports.isFileSummaryCompatible = exports.isFileEncrypted = exports.isEditableAttribute = exports.getSummaryConfig = exports.formatDate = void 0;
+exports.roughTokensEstimation = exports.removeFilenameFromPath = exports.normalizeAndSpreadAttributes = exports.makeWebLink = exports.isFileSummaryCompatible = exports.isFileEncrypted = exports.isEditableAttribute = exports.formatDate = void 0;
 
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 
@@ -15,8 +15,6 @@ var _file = require("cozy-client/dist/models/file");
 
 var _cozyFlags = _interopRequireDefault(require("cozy-flags"));
 
-var _cozyLogger = _interopRequireDefault(require("cozy-logger"));
-
 function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
 
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
@@ -117,29 +115,6 @@ exports.removeFilenameFromPath = removeFilenameFromPath;
 var roughTokensEstimation = function roughTokensEstimation(text) {
   return Math.ceil(text.length / 4);
 };
-/**
- * Get and parse the drive.summary flag configuration
- * @returns {object|null} Parsed summary config or null if not available/invalid
- */
-
-
-exports.roughTokensEstimation = roughTokensEstimation;
-
-var getSummaryConfig = function getSummaryConfig() {
-  var summaryConfigRawValue = (0, _cozyFlags.default)('drive.summary');
-
-  if (!summaryConfigRawValue) {
-    return null;
-  }
-
-  try {
-    return JSON.parse(summaryConfigRawValue);
-  } catch (e) {
-    _cozyLogger.default.error('Failed to parse drive.summary flag:', e);
-
-    return null;
-  }
-};
 /**
  * Check if a file is compatible with AI summary feature
  * Compatible file types are defined in the drive.summary flag
@@ -151,7 +126,7 @@ var getSummaryConfig = function getSummaryConfig() {
  */
 
 
-exports.getSummaryConfig = getSummaryConfig;
+exports.roughTokensEstimation = roughTokensEstimation;
 
 var isFileSummaryCompatible = function isFileSummaryCompatible(file) {
   var options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {
@@ -162,13 +137,13 @@ var isFileSummaryCompatible = function isFileSummaryCompatible(file) {
     return false;
   }
 
-  var summaryConfig = getSummaryConfig();
+  var summaryConfig = (0, _cozyFlags.default)('drive.summary');
 
   if (!summaryConfig) {
     return false;
   }
 
-  if (!summaryConfig || !Array.isArray(summaryConfig.types) || summaryConfig.types.length === 0) {
+  if (!Array.isArray(summaryConfig.types) || summaryConfig.types.length === 0) {
     return false;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-viewer",
-  "version": "26.1.2",
+  "version": "26.2.1",
   "description": "Cozy-Viewer provides a component to show files in a viewer.",
   "main": "dist/index.js",
   "license": "MIT",
@@ -31,10 +31,10 @@
     "babel-preset-cozy-app": "^2.8.2",
     "cozy-client": "^60.20.0",
     "cozy-device-helper": "2.0.0",
-    "cozy-harvest-lib": "^36.0.8",
+    "cozy-harvest-lib": "^36.0.10",
     "cozy-intent": "^2.30.1",
     "cozy-logger": "^1.17.0",
-    "cozy-sharing": "^28.1.1",
+    "cozy-sharing": "^28.1.2",
     "cozy-ui": "^135.0.0",
     "cozy-ui-plus": "^4.1.0",
     "identity-obj-proxy": "3.0.0",
@@ -70,5 +70,5 @@
     "react-router-dom": ">=6.14.2",
     "twake-i18n": ">=0.3.0"
   },
-  "gitHead": "47ff1f9d71e58f22aafb44c29dc68dd9658f5206"
+  "gitHead": "ea1e051ec4fb1033cdddc76cab56bc7ec174d6be"
 }

package/src/Panel/AI/AIAssistantPanel.jsx

@@ -1,12 +1,13 @@
 import cx from 'classnames'
 import PropTypes from 'prop-types'
-import React, { useCallback, useState, useEffect } from 'react'
+import React, { useCallback, useState, useEffect, useRef } from 'react'
 import { useLocation, useNavigate } from 'react-router-dom'
 import { useI18n } from 'twake-i18n'
 
 import { useClient } from 'cozy-client'
 import { extractText, chatCompletion } from 'cozy-client/dist/models/ai'
 import { fetchBlobFileById } from 'cozy-client/dist/models/file'
+import flag from 'cozy-flags'
 import logger from 'cozy-logger'
 import Button from 'cozy-ui/transpiled/react/Buttons'
 import Icon from 'cozy-ui/transpiled/react/Icon'
@@ -22,7 +23,7 @@ import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert'
 
 import { SUMMARY_SYSTEM_PROMPT, getSummaryUserPrompt } from './prompts'
 import styles from './styles.styl'
-import { getSummaryConfig, roughTokensEstimation } from '../../helpers'
+import { roughTokensEstimation } from '../../helpers'
 import { useViewer } from '../../providers/ViewerProvider'
 
 const AIAssistantPanel = ({ className }) => {
@@ -37,6 +38,9 @@ const AIAssistantPanel = ({ className }) => {
 
   const location = useLocation()
   const navigate = useNavigate()
+  const fetchedFileIdRef = useRef(null)
+  const inFlightFileIdRef = useRef(null)
+  const activeFileIdRef = useRef(file?._id || null)
 
   const handleClose = () => {
     setIsOpenAiAssistant(false)
@@ -54,7 +58,7 @@ const AIAssistantPanel = ({ className }) => {
         mime: file.mime
       })
 
-      const summaryConfig = getSummaryConfig()
+      const summaryConfig = flag('drive.summary')
       if (
         summaryConfig?.maxTokens &&
         roughTokensEstimation(textContent) > summaryConfig.maxTokens
@@ -84,31 +88,80 @@ const AIAssistantPanel = ({ className }) => {
     }
   }
 
-  const fetchSummary = useCallback(async () => {
-    if (!file) return
-
-    setIsLoading(true)
-    setError(null)
+  const persistedSummary = async (
+    fileMetadata,
+    targetFileId,
+    summaryContent
+  ) => {
     try {
-      const response = await summarizeFile({ client, file, stream: false })
-      if (response && response.content) {
-        setSummary(response.content)
-      } else if (response && response.choices && response.choices[0]) {
-        setSummary(response.choices[0].message.content)
-      }
-    } catch (err) {
-      const errorMessage =
-        err.code === 'DOCUMENT_TOO_LARGE'
-          ? t('Viewer.ai.error.documentTooLarge')
-          : t('Viewer.ai.error.summary')
-      setError(errorMessage)
-    } finally {
-      setIsLoading(false)
+      await client
+        .collection('io.cozy.files')
+        .updateMetadataAttribute(targetFileId, {
+          ...fileMetadata,
+          description: summaryContent
+        })
+      fetchedFileIdRef.current = targetFileId
+    } catch (error) {
+      logger.error('Error when persisting summary to file metadata:', error)
     }
-  }, [client, file, t])
+  }
+
+  useEffect(() => {
+    activeFileIdRef.current = file?._id || null
+  }, [file])
+
+  const fetchSummary = useCallback(
+    async (force = false) => {
+      const targetFileId = file?._id
+      if (!targetFileId) return
+
+      // Prevent duplicate fetches for the same file
+      if (
+        !force &&
+        (fetchedFileIdRef.current === targetFileId ||
+          inFlightFileIdRef.current === targetFileId)
+      ) {
+        return
+      }
+
+      inFlightFileIdRef.current = targetFileId
+      setIsLoading(true)
+      setError(null)
+
+      try {
+        const response = await summarizeFile({ client, file, stream: false })
+        const summaryContent =
+          response?.content || response?.choices?.[0]?.message?.content
+        // Ignore results if the user switched to another file meanwhile
+        if (activeFileIdRef.current !== targetFileId) {
+          return
+        }
+        setSummary(summaryContent)
+        const fileMetadata = file.metadata || {}
+        await persistedSummary(fileMetadata, targetFileId, summaryContent)
+      } catch (err) {
+        if (activeFileIdRef.current === targetFileId) {
+          const errorMessage =
+            err.code === 'DOCUMENT_TOO_LARGE'
+              ? t('Viewer.ai.error.documentTooLarge')
+              : t('Viewer.ai.error.summary')
+          setError(errorMessage)
+        }
+      } finally {
+        if (inFlightFileIdRef.current === targetFileId) {
+          inFlightFileIdRef.current = null
+        }
+        if (activeFileIdRef.current === targetFileId) {
+          setIsLoading(false)
+        }
+      }
+    },
+    [client, file, t]
+  )
 
   const handleRefresh = () => {
-    fetchSummary()
+    fetchedFileIdRef.current = null
+    fetchSummary(true)
   }
 
   const handleCopy = () => {

package/src/helpers.js

@@ -5,7 +5,6 @@ import {
   normalize
 } from 'cozy-client/dist/models/file'
 import flag from 'cozy-flags'
-import logger from 'cozy-logger'
 
 /**
  * @typedef {object} Reference
@@ -85,24 +84,6 @@ export const roughTokensEstimation = text => {
   return Math.ceil(text.length / 4)
 }
 
-/**
- * Get and parse the drive.summary flag configuration
- * @returns {object|null} Parsed summary config or null if not available/invalid
- */
-export const getSummaryConfig = () => {
-  const summaryConfigRawValue = flag('drive.summary')
-  if (!summaryConfigRawValue) {
-    return null
-  }
-
-  try {
-    return JSON.parse(summaryConfigRawValue)
-  } catch (e) {
-    logger.error('Failed to parse drive.summary flag:', e)
-    return null
-  }
-}
-
 /**
  * Check if a file is compatible with AI summary feature
  * Compatible file types are defined in the drive.summary flag
@@ -120,16 +101,12 @@ export const isFileSummaryCompatible = (
     return false
   }
 
-  const summaryConfig = getSummaryConfig()
+  const summaryConfig = flag('drive.summary')
   if (!summaryConfig) {
     return false
   }
 
-  if (
-    !summaryConfig ||
-    !Array.isArray(summaryConfig.types) ||
-    summaryConfig.types.length === 0
-  ) {
+  if (!Array.isArray(summaryConfig.types) || summaryConfig.types.length === 0) {
     return false
   }
 

package/src/helpers.spec.js

@@ -11,11 +11,11 @@ jest.mock('cozy-flags')
 // Default mock for drive.summary flag
 flag.mockImplementation(flagName => {
   if (flagName === 'drive.summary') {
-    return JSON.stringify({
+    return {
       types: ['application/pdf', 'text/*'],
       pageLimit: 50,
       maxTokens: 100000
-    })
+    }
   }
   return null
 })
@@ -110,64 +110,58 @@ describe('helpers', () => {
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
-    it('should return false if flag JSON is invalid', () => {
-      flag.mockReturnValue('invalid json')
+    it('should return false if flag config is invalid', () => {
+      flag.mockReturnValue('invalid config')
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
     it('should return false if flag config is missing types array', () => {
-      flag.mockReturnValue(JSON.stringify({}))
+      flag.mockReturnValue({})
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
     it('should return false if types array is empty', () => {
-      flag.mockReturnValue(JSON.stringify({ types: [] }))
+      flag.mockReturnValue({ types: [] })
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
     it('should return false if types is not an array', () => {
-      flag.mockReturnValue(JSON.stringify({ types: 'application/pdf' }))
+      flag.mockReturnValue({ types: 'application/pdf' })
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
     it('should return true if mime matches exactly', () => {
-      flag.mockReturnValue(
-        JSON.stringify({ types: ['application/pdf', 'text/plain'] })
-      )
+      flag.mockReturnValue({ types: ['application/pdf', 'text/plain'] })
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(true)
       expect(isFileSummaryCompatible({ mime: 'text/plain' })).toBe(true)
     })
 
     it('should handle case-insensitive mime matching', () => {
-      flag.mockReturnValue(
-        JSON.stringify({ types: ['application/PDF', 'TEXT/plain'] })
-      )
+      flag.mockReturnValue({ types: ['application/PDF', 'TEXT/plain'] })
       expect(isFileSummaryCompatible({ mime: 'APPLICATION/pdf' })).toBe(true)
       expect(isFileSummaryCompatible({ mime: 'text/PLAIN' })).toBe(true)
     })
 
     it('should return false if mime does not match', () => {
-      flag.mockReturnValue(JSON.stringify({ types: ['application/pdf'] }))
+      flag.mockReturnValue({ types: ['application/pdf'] })
       expect(isFileSummaryCompatible({ mime: 'text/plain' })).toBe(false)
     })
 
     it('should handle wildcard types', () => {
-      flag.mockReturnValue(JSON.stringify({ types: ['text/*'] }))
+      flag.mockReturnValue({ types: ['text/*'] })
       expect(isFileSummaryCompatible({ mime: 'text/plain' })).toBe(true)
       expect(isFileSummaryCompatible({ mime: 'text/markdown' })).toBe(true)
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(false)
     })
 
     it('should handle case-insensitive wildcard matching', () => {
-      flag.mockReturnValue(JSON.stringify({ types: ['TEXT/*'] }))
+      flag.mockReturnValue({ types: ['TEXT/*'] })
       expect(isFileSummaryCompatible({ mime: 'text/plain' })).toBe(true)
       expect(isFileSummaryCompatible({ mime: 'TEXT/markdown' })).toBe(true)
     })
 
     it('should respect page limit when pageCount is provided', () => {
-      flag.mockReturnValue(
-        JSON.stringify({ types: ['application/pdf'], pageLimit: 50 })
-      )
+      flag.mockReturnValue({ types: ['application/pdf'], pageLimit: 50 })
       expect(
         isFileSummaryCompatible({ mime: 'application/pdf' }, { pageCount: 10 })
       ).toBe(true)
@@ -180,9 +174,7 @@ describe('helpers', () => {
     })
 
     it('should return false if pageCount is 0 or negative', () => {
-      flag.mockReturnValue(
-        JSON.stringify({ types: ['application/pdf'], pageLimit: 50 })
-      )
+      flag.mockReturnValue({ types: ['application/pdf'], pageLimit: 50 })
       expect(
         isFileSummaryCompatible({ mime: 'application/pdf' }, { pageCount: 0 })
       ).toBe(false)
@@ -192,14 +184,12 @@ describe('helpers', () => {
     })
 
     it('should return true if pageCount is not provided and no pageLimit is set', () => {
-      flag.mockReturnValue(JSON.stringify({ types: ['application/pdf'] }))
+      flag.mockReturnValue({ types: ['application/pdf'] })
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(true)
     })
 
     it('should return true if pageCount is not provided but pageLimit is set', () => {
-      flag.mockReturnValue(
-        JSON.stringify({ types: ['application/pdf'], pageLimit: 50 })
-      )
+      flag.mockReturnValue({ types: ['application/pdf'], pageLimit: 50 })
       expect(isFileSummaryCompatible({ mime: 'application/pdf' })).toBe(true)
     })
   })