@rockhopper-co/mcp-server 2.1.82-staging.65cfbf4 → 2.1.84-staging.bac34fe

package/dist/tools/list-unenrolled-files.js

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import { GRAPH_LINK_FAILURE_TEXT, graphLinkFailureFrom, } from '../graph-link-failure.js';
+import { renderEmpty, renderFound } from './list-unenrolled-render.js';
 /**
  * ENG-2785 — "which of my workbooks are NOT in Rockhopper yet?"
  *
@@ -37,27 +37,23 @@ import { GRAPH_LINK_FAILURE_TEXT, graphLinkFailureFrom, } from '../graph-link-fa
  */
 export function registerListUnenrolledFilesTool(server, api) {
     server.registerTool('list_unenrolled_files', {
-        title: 'List Workbooks Not Yet in Rockhopper',
-        description: 'List the spreadsheets Rockhopper has seen for this user that are ' +
-            'NOT enrolled — the answer to "what could I add?". Use this for ' +
-            'browsing, when the user cannot name a specific file; use ' +
-            '`search_drive_files` when they can. Read-only: it enrolls nothing ' +
-            'and asks nothing. The answer comes from stored records rather than a ' +
-            'live Microsoft read, so it is dated — pass a listed file\'s `msId` ' +
-            'and `driveMsId` to `enroll_file` to add it. ' +
-            // ENG-4283. Stated up front, in `enroll_file`'s words, because the
-            // model has to know the scope BEFORE it picks the tool: offered as the
-            // answer to a Google Workspace customer's "what could I add?", this
-            // returns an empty list whose emptiness means nothing about their
-            // drive, and the tool cannot un-ask the question afterwards.
-            'MICROSOFT ONLY — it lists OneDrive and SharePoint workbooks and ' +
-            'covers no other storage. An account with no Microsoft link gets ' +
-            'nothing from it, which is not a statement about what that user has. ' +
-            // ENG-4958: the remedy, not just the limit. `search_drive_files` now
-            // reaches Google Drive, so a model that lands here for a Google file
-            // has somewhere to go instead of reporting an empty drive.
-            'For a Google file, call `search_drive_files` with ' +
-            'provider="google" instead. ' +
+        title: 'List Files Not Yet in Rockhopper',
+        description: 'List the files Rockhopper has seen for this user that are NOT ' +
+            'enrolled — the answer to "what could I add?". Use this for browsing, ' +
+            'when the user cannot name a specific file; use `search_drive_files` ' +
+            'when they can. Read-only: it enrolls nothing and asks nothing. The ' +
+            'answer comes from stored records rather than a live read of the ' +
+            'user\'s drive, so it is dated. ' +
+            // ENG-6410 — scope stated up front, because the model has to know it
+            // BEFORE it picks the tool. ENG-4283 made this 'MICROSOFT ONLY' and
+            // ENG-4958 sent Google callers to `search_drive_files`; ENG-6409 gave
+            // the inventory a Google lane, so both are now false.
+            'It covers OneDrive and SharePoint (Microsoft) and Google Drive, for ' +
+            'whichever of those the user has connected, and every row names its ' +
+            'provider. To add a Microsoft row, pass its `msId` and `driveMsId` to ' +
+            '`enroll_file`; to add a Google row, pass its link to `enroll_file` as ' +
+            '`url`. An account with neither connected gets nothing from it, which ' +
+            'is not a statement about what that user has. ' +
             // ENG-4271: an organisation's automatic-enrollment scope governs only
             // which files its own background sweep enrolls on its own; it never
             // stops `enroll_file` from adding any file listed here by hand.
@@ -130,160 +126,4 @@ export function registerListUnenrolledFilesTool(server, api) {
         }
     });
 }
-/**
- * ENG-2814 — the "there is more" line, and it is never optional when a cursor
- * came back.
- *
- * Written to the MODEL, not the user: it names the exact next call, because a
- * hint that only says "more available" gets summarised away and the walk stops
- * one page in.
- */
-function moreToCome(nextCursor) {
-    if (!nextCursor)
-        return '';
-    return (`\n\nMORE FILES REMAIN — this is not the whole list. Call ` +
-        `\`list_unenrolled_files\` again with \`cursor="${nextCursor}"\`, and ` +
-        'keep going until a response comes back with no cursor. Do not tell the ' +
-        'user what they do or do not have until then. The cursor stops working 30 ' +
-        'minutes after the first page; if it is refused as expired, start again ' +
-        'from the first page rather than retrying it.');
-}
-function renderFound(items, freshness, nextCursor = null) {
-    const lines = items.map((item) => {
-        const where = item.parentPath ? ` in ${item.parentPath}` : '';
-        const modified = item.lastModifiedAt
-            ? `, modified ${item.lastModifiedAt}`
-            : '';
-        // `hidden` means the user REMOVED this file from Rockhopper before. Its
-        // history is still held, so enrolling restores rather than duplicates —
-        // a different next step from a file that was never added, and the only
-        // thing distinguishing the two.
-        const previously = item.enrollmentState === 'hidden' ? ' [previously removed]' : '';
-        return (`- **${item.name}**${where}${modified}${previously}\n` +
-            `  msId: ${item.msId}, driveMsId: ${item.driveMsId}`);
-    });
-    // "on this page", never a bare count: with a cursor outstanding the number is
-    // a page size, and calling it a total is the same lie as the old 200-row
-    // ceiling that reported nothing about itself.
-    const counted = nextCursor
-        ? `${items.length} workbook(s) not yet in Rockhopper on this page:`
-        : `${items.length} workbook(s) not yet in Rockhopper:`;
-    return (`${counted}\n\n` +
-        `${lines.join('\n')}\n\n${dateline(freshness)}${moreToCome(nextCursor)}`);
-}
-/**
- * ENG-4283 — the backend's code for "no Microsoft tenant on this account".
- *
- * Duplicated from `DriveInventoryFreshnessDto` rather than imported, for the
- * same reason `graph-link-failure.ts` duplicates its four: this package ships
- * over npm and takes no build dependency on the API tree. EXACT match, and no
- * case folding — a value this package has not learned about is not "probably
- * this one", and guessing at a spelling is precisely what left ENG-4311's
- * comparison dead for a release.
- */
-const NO_MICROSOFT_TENANT = 'NO_MICROSOFT_TENANT';
-/**
- * What to tell the assistant when the lane does not serve this account.
- *
- * Three rules, each answering a specific way the old message did harm:
- *  - **No retry language.** Nothing about this changes on a later call, and a
- *    model complies with "try again shortly" indefinitely.
- *  - **No `connect_microsoft`.** There is no tenant behind it for this user, so
- *    the link would send them to Microsoft to be refused (ENG-2614).
- *  - **Names the scope and the alternative**, so the answer is usable rather
- *    than only correct — a Google customer's files are enrolled a different way
- *    and the assistant should say so instead of stopping at a refusal.
- */
-const NO_MICROSOFT_TENANT_TEXT = 'Rockhopper\'s drive inventory covers OneDrive and SharePoint, and this ' +
-    'account has no Microsoft link — so there is nothing for it to list and ' +
-    'nothing to retry. This is NOT evidence that every workbook is already in ' +
-    'Rockhopper, and it does not mean the user has no files: it means this ' +
-    'particular list cannot see them. Do not call `list_unenrolled_files` again ' +
-    'for this user, and do not report their drive as covered. If they use ' +
-    'Google Workspace, their spreadsheets are added through Rockhopper directly ' +
-    'rather than from this list.';
-function renderEmpty(freshness, nextCursor = null) {
-    // ORDER IS LOAD-BEARING. A broken link and a first refresh that has not
-    // finished both mean there is nothing to page THROUGH, so sending the model
-    // after another page instead of `connect_microsoft` starts a loop that cannot
-    // end. Those two branches come first and keep their own instruction.
-    //
-    // ENG-4311 — this used to compare `lastFailureReason === 'no_delegated_token'`
-    // against a producer that writes the SCREAMING_SNAKE `GraphLinkFailure` enum,
-    // so it never matched and this branch was dead: a user with no working link
-    // fell through to the never-refreshed message below and was told a scan had
-    // been started and to try again shortly. Neither was true. The four codes are
-    // kept APART rather than collapsed, because three of them name a different
-    // person who has to act — see `graph-link-failure.ts`.
-    // ENG-4283 — FIRST, ahead of every other branch including the link failures.
-    //
-    // The four states below all assume this lane COVERS the caller and something
-    // went wrong on the way to filling it. An account with no Microsoft tenant is
-    // a fifth thing: the refresh returns before it writes a sync-state row, so
-    // `asOf` stays null and `refreshing` stays false PERMANENTLY, and the
-    // never-refreshed branch below fires on every call — asserting a scan "has
-    // been started" when none was, and instructing a retry that cannot terminate.
-    //
-    // It outranks the link failures rather than following them because a link
-    // failure names something a user or an administrator can fix, and this names
-    // something neither can: sending a Google-only customer to
-    // `connect_microsoft` is ENG-2614's loop with an extra hop.
-    if (freshness.inapplicableReason === NO_MICROSOFT_TENANT) {
-        return NO_MICROSOFT_TENANT_TEXT;
-    }
-    const linkFailure = graphLinkFailureFrom(freshness.lastFailureReason);
-    if (linkFailure)
-        return GRAPH_LINK_FAILURE_TEXT[linkFailure];
-    if (!freshness.asOf) {
-        return ('No answer yet — the first scan of this user\'s drive has not finished. ' +
-            `${freshness.refreshing ? 'One is running now.' : 'One has been started.'} ` +
-            'Try again shortly. This is NOT evidence that every workbook is already ' +
-            `in Rockhopper.${failureNote(freshness)}`);
-    }
-    /**
-     * ENG-2814 — AN EMPTY PAGE WITH A CURSOR IS THE MIDDLE OF A WALK.
-     *
-     * The backend filters enrolled rows out AFTER cutting a chunk and stops at a
-     * per-request scan budget, so "this page had none" is routine and says
-     * nothing about the drive. Reporting it as "everything is already covered" is
-     * the exact failure this whole tool was written against — an empty list that
-     * reads as reassurance — and pagination gives it a brand new way to happen.
-     */
-    if (nextCursor) {
-        return ('No un-enrolled workbooks on this page — but the search is NOT ' +
-            'finished, and this says nothing yet about what the user has.' +
-            `${moreToCome(nextCursor)}`);
-    }
-    if (freshness.consecutiveFailures > 0) {
-        return ('No un-enrolled workbooks in the stored list, but the list is not ' +
-            `trustworthy right now.${failureNote(freshness)}\n\n${dateline(freshness)}`);
-    }
-    return ('Every workbook Rockhopper has seen for this user is already in ' +
-        `Rockhopper.\n\n${dateline(freshness)}`);
-}
-/**
- * How old the answer is, on every branch that has an answer.
- *
- * Never omitted: the rows are stored, so a surface that does not date them
- * invites a user to conclude a file is absent from their drive when it is only
- * absent from our last refresh.
- */
-function dateline(freshness) {
-    const asOf = freshness.asOf
-        ? `As of ${freshness.asOf}`
-        : 'Never successfully refreshed';
-    const stale = freshness.stale ? ' (stale)' : '';
-    const refreshing = freshness.refreshing
-        ? ' A refresh is running; call again for a newer answer.'
-        : '';
-    return (`${asOf}${stale} — from Rockhopper's stored records, not a live ` +
-        `Microsoft read.${refreshing}${failureNote(freshness)}`);
-}
-function failureNote(freshness) {
-    if (!freshness.consecutiveFailures || !freshness.lastFailureAt)
-        return '';
-    return (` Refresh has failed ${freshness.consecutiveFailures} time(s) in a row, ` +
-        `last at ${freshness.lastFailureAt}` +
-        `${freshness.lastFailureReason ? ` (${freshness.lastFailureReason})` : ''}.`);
-}
 //# sourceMappingURL=list-unenrolled-files.js.map

package/dist/tools/list-unenrolled-files.js.map

@@ -1 +1 @@
-{"version":3,"file":"list-unenrolled-files.js","sourceRoot":"","sources":["../../src/tools/list-unenrolled-files.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EACL,uBAAuB,EACvB,oBAAoB,GACrB,MAAM,0BAA0B,CAAC;AAMlC;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,MAAM,UAAU,+BAA+B,CAC7C,MAAiB,EACjB,GAAc;IAEd,MAAM,CAAC,YAAY,CACjB,uBAAuB,EACvB;QACE,KAAK,EAAE,sCAAsC;QAC7C,WAAW,EACT,mEAAmE;YACnE,iEAAiE;YACjE,2DAA2D;YAC3D,oEAAoE;YACpE,uEAAuE;YACvE,qEAAqE;YACrE,8CAA8C;YAC9C,mEAAmE;YACnE,uEAAuE;YACvE,oEAAoE;YACpE,kEAAkE;YAClE,6DAA6D;YAC7D,kEAAkE;YAClE,kEAAkE;YAClE,sEAAsE;YACtE,qEAAqE;YACrE,qEAAqE;YACrE,2DAA2D;YAC3D,oDAAoD;YACpD,6BAA6B;YAC7B,sEAAsE;YACtE,oEAAoE;YACpE,gEAAgE;YAChE,kEAAkE;YAClE,sEAAsE;YACtE,gEAAgE;YAChE,uEAAuE;YACvE,0DAA0D;YAC1D,qEAAqE;YACrE,oEAAoE;YACpE,kEAAkE;YAClE,oEAAoE;YACpE,qEAAqE;YACrE,gEAAgE;YAChE,uEAAuE;QACzE,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC;YACpB,KAAK,EAAE,CAAC;iBACL,MAAM,EAAE;iBACR,GAAG,EAAE;iBACL,GAAG,CAAC,CAAC,CAAC;iBACN,GAAG,CAAC,GAAG,CAAC;iBACR,QAAQ,EAAE;iBACV,QAAQ,CACP,4DAA4D;gBAC1D,gEAAgE;gBAChE,kCAAkC,CACrC;YACH,MAAM,EAAE,CAAC;iBACN,MAAM,EAAE;iBACR,QAAQ,EAAE;iBACV,QAAQ,CACP,6DAA6D;gBAC3D,4DAA4D;gBAC5D,iEAAiE;gBACjE,4DAA4D,CAC/D;SACJ,CAAC;QACF,WAAW,EAAE;YACX,YAAY,EAAE,IAAI;YAClB,aAAa,EAAE,KAAK;SACrB;KACF,EACD,KAAK,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,EAAE,EAAE;QAC1B,IAAI,CAAC;YACH,MAAM,EAAE,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,CAAC,kBAAkB,CAAC;gBACpE,UAAU,EAAE,cAAc;gBAC1B,KAAK;gBACL,MAAM;aACP,CAAC,CAAC;YAEH,OAAO;gBACL,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,KAAK,CAAC,MAAM;4BAChB,CAAC,CAAC,WAAW,CAAC,KAAK,EAAE,SAAS,EAAE,UAAU,CAAC;4BAC3C,CAAC,CAAC,WAAW,CAAC,SAAS,EAAE,UAAU,CAAC;qBACvC;iBACF;aACF,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,uEAAuE;YACvE,OAAO;gBACL,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EACF,oDAAoD;4BACpD,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI;4BAC7D,6DAA6D;4BAC7D,oCAAoC;qBACvC;iBACF;gBACD,OAAO,EAAE,IAAI;aACd,CAAC;QACJ,CAAC;IACH,CAAC,CACF,CAAC;AACJ,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,UAAU,CAAC,UAAyB;IAC3C,IAAI,CAAC,UAAU;QAAE,OAAO,EAAE,CAAC;IAC3B,OAAO,CACL,2DAA2D;QAC3D,kDAAkD,UAAU,WAAW;QACvE,yEAAyE;QACzE,2EAA2E;QAC3E,yEAAyE;QACzE,8CAA8C,CAC/C,CAAC;AACJ,CAAC;AAED,SAAS,WAAW,CAClB,KAAoC,EACpC,SAAkC,EAClC,aAA4B,IAAI;IAEhC,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC9D,MAAM,QAAQ,GAAG,IAAI,CAAC,cAAc;YAClC,CAAC,CAAC,cAAc,IAAI,CAAC,cAAc,EAAE;YACrC,CAAC,CAAC,EAAE,CAAC;QACP,wEAAwE;QACxE,wEAAwE;QACxE,uEAAuE;QACvE,gCAAgC;QAChC,MAAM,UAAU,GACd,IAAI,CAAC,eAAe,KAAK,QAAQ,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,EAAE,CAAC;QACnE,OAAO,CACL,OAAO,IAAI,CAAC,IAAI,KAAK,KAAK,GAAG,QAAQ,GAAG,UAAU,IAAI;YACtD,WAAW,IAAI,CAAC,IAAI,gBAAgB,IAAI,CAAC,SAAS,EAAE,CACrD,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,8EAA8E;IAC9E,yEAAyE;IACzE,8CAA8C;IAC9C,MAAM,OAAO,GAAG,UAAU;QACxB,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,kDAAkD;QACnE,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,qCAAqC,CAAC;IAEzD,OAAO,CACL,GAAG,OAAO,MAAM;QAChB,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,QAAQ,CAAC,SAAS,CAAC,GAAG,UAAU,CAAC,UAAU,CAAC,EAAE,CACzE,CAAC;AACJ,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,mBAAmB,GAAG,qBAAqB,CAAC;AAElD;;;;;;;;;;;GAWG;AACH,MAAM,wBAAwB,GAC5B,yEAAyE;IACzE,yEAAyE;IACzE,2EAA2E;IAC3E,wEAAwE;IACxE,6EAA6E;IAC7E,uEAAuE;IACvE,6EAA6E;IAC7E,6BAA6B,CAAC;AAEhC,SAAS,WAAW,CAClB,SAAkC,EAClC,aAA4B,IAAI;IAEhC,wEAAwE;IACxE,4EAA4E;IAC5E,8EAA8E;IAC9E,qEAAqE;IACrE,EAAE;IACF,+EAA+E;IAC/E,8EAA8E;IAC9E,4EAA4E;IAC5E,4EAA4E;IAC5E,8EAA8E;IAC9E,2EAA2E;IAC3E,uDAAuD;IACvD,6EAA6E;IAC7E,EAAE;IACF,6EAA6E;IAC7E,8EAA8E;IAC9E,2EAA2E;IAC3E,sEAAsE;IACtE,2EAA2E;IAC3E,8EAA8E;IAC9E,EAAE;IACF,0EAA0E;IAC1E,6EAA6E;IAC7E,2DAA2D;IAC3D,4DAA4D;IAC5D,IAAI,SAAS,CAAC,kBAAkB,KAAK,mBAAmB,EAAE,CAAC;QACzD,OAAO,wBAAwB,CAAC;IAClC,CAAC;IAED,MAAM,WAAW,GAAG,oBAAoB,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;IACtE,IAAI,WAAW;QAAE,OAAO,uBAAuB,CAAC,WAAW,CAAC,CAAC;IAE7D,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC;QACpB,OAAO,CACL,yEAAyE;YACzE,GAAG,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,uBAAuB,GAAG;YAC5E,yEAAyE;YACzE,iBAAiB,WAAW,CAAC,SAAS,CAAC,EAAE,CAC1C,CAAC;IACJ,CAAC;IAED;;;;;;;;OAQG;IACH,IAAI,UAAU,EAAE,CAAC;QACf,OAAO,CACL,gEAAgE;YAChE,8DAA8D;YAC9D,GAAG,UAAU,CAAC,UAAU,CAAC,EAAE,CAC5B,CAAC;IACJ,CAAC;IAED,IAAI,SAAS,CAAC,mBAAmB,GAAG,CAAC,EAAE,CAAC;QACtC,OAAO,CACL,mEAAmE;YACnE,yBAAyB,WAAW,CAAC,SAAS,CAAC,OAAO,QAAQ,CAAC,SAAS,CAAC,EAAE,CAC5E,CAAC;IACJ,CAAC;IAED,OAAO,CACL,iEAAiE;QACjE,kBAAkB,QAAQ,CAAC,SAAS,CAAC,EAAE,CACxC,CAAC;AACJ,CAAC;AAED;;;;;;GAMG;AACH,SAAS,QAAQ,CAAC,SAAkC;IAClD,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI;QACzB,CAAC,CAAC,SAAS,SAAS,CAAC,IAAI,EAAE;QAC3B,CAAC,CAAC,8BAA8B,CAAC;IACnC,MAAM,KAAK,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC;IAChD,MAAM,UAAU,GAAG,SAAS,CAAC,UAAU;QACrC,CAAC,CAAC,uDAAuD;QACzD,CAAC,CAAC,EAAE,CAAC;IACP,OAAO,CACL,GAAG,IAAI,GAAG,KAAK,kDAAkD;QACjE,kBAAkB,UAAU,GAAG,WAAW,CAAC,SAAS,CAAC,EAAE,CACxD,CAAC;AACJ,CAAC;AAED,SAAS,WAAW,CAAC,SAAkC;IACrD,IAAI,CAAC,SAAS,CAAC,mBAAmB,IAAI,CAAC,SAAS,CAAC,aAAa;QAAE,OAAO,EAAE,CAAC;IAC1E,OAAO,CACL,uBAAuB,SAAS,CAAC,mBAAmB,qBAAqB;QACzE,WAAW,SAAS,CAAC,aAAa,EAAE;QACpC,GAAG,SAAS,CAAC,iBAAiB,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,CAAC,EAAE,GAAG,CAC7E,CAAC;AACJ,CAAC"}
+{"version":3,"file":"list-unenrolled-files.js","sourceRoot":"","sources":["../../src/tools/list-unenrolled-files.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,WAAW,EAAE,WAAW,EAAE,MAAM,6BAA6B,CAAC;AAEvE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,MAAM,UAAU,+BAA+B,CAC7C,MAAiB,EACjB,GAAc;IAEd,MAAM,CAAC,YAAY,CACjB,uBAAuB,EACvB;QACE,KAAK,EAAE,kCAAkC;QACzC,WAAW,EACT,gEAAgE;YAChE,uEAAuE;YACvE,sEAAsE;YACtE,qEAAqE;YACrE,kEAAkE;YAClE,iCAAiC;YACjC,qEAAqE;YACrE,oEAAoE;YACpE,sEAAsE;YACtE,sDAAsD;YACtD,sEAAsE;YACtE,qEAAqE;YACrE,uEAAuE;YACvE,wEAAwE;YACxE,uEAAuE;YACvE,+CAA+C;YAC/C,sEAAsE;YACtE,oEAAoE;YACpE,gEAAgE;YAChE,kEAAkE;YAClE,sEAAsE;YACtE,gEAAgE;YAChE,uEAAuE;YACvE,0DAA0D;YAC1D,qEAAqE;YACrE,oEAAoE;YACpE,kEAAkE;YAClE,oEAAoE;YACpE,qEAAqE;YACrE,gEAAgE;YAChE,uEAAuE;QACzE,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC;YACpB,KAAK,EAAE,CAAC;iBACL,MAAM,EAAE;iBACR,GAAG,EAAE;iBACL,GAAG,CAAC,CAAC,CAAC;iBACN,GAAG,CAAC,GAAG,CAAC;iBACR,QAAQ,EAAE;iBACV,QAAQ,CACP,4DAA4D;gBAC1D,gEAAgE;gBAChE,kCAAkC,CACrC;YACH,MAAM,EAAE,CAAC;iBACN,MAAM,EAAE;iBACR,QAAQ,EAAE;iBACV,QAAQ,CACP,6DAA6D;gBAC3D,4DAA4D;gBAC5D,iEAAiE;gBACjE,4DAA4D,CAC/D;SACJ,CAAC;QACF,WAAW,EAAE;YACX,YAAY,EAAE,IAAI;YAClB,aAAa,EAAE,KAAK;SACrB;KACF,EACD,KAAK,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,EAAE,EAAE;QAC1B,IAAI,CAAC;YACH,MAAM,EAAE,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,CAAC,kBAAkB,CAAC;gBACpE,UAAU,EAAE,cAAc;gBAC1B,KAAK;gBACL,MAAM;aACP,CAAC,CAAC;YAEH,OAAO;gBACL,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,KAAK,CAAC,MAAM;4BAChB,CAAC,CAAC,WAAW,CAAC,KAAK,EAAE,SAAS,EAAE,UAAU,CAAC;4BAC3C,CAAC,CAAC,WAAW,CAAC,SAAS,EAAE,UAAU,CAAC;qBACvC;iBACF;aACF,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,uEAAuE;YACvE,OAAO;gBACL,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EACF,oDAAoD;4BACpD,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI;4BAC7D,6DAA6D;4BAC7D,oCAAoC;qBACvC;iBACF;gBACD,OAAO,EAAE,IAAI;aACd,CAAC;QACJ,CAAC;IACH,CAAC,CACF,CAAC;AACJ,CAAC"}

package/dist/tools/list-unenrolled-render.d.ts

@@ -0,0 +1,4 @@
+import type { DriveInventoryFreshness, DriveInventoryItem } from '../types.js';
+export declare function renderFound(items: readonly DriveInventoryItem[], freshness: DriveInventoryFreshness, nextCursor?: string | null): string;
+export declare function renderEmpty(freshness: DriveInventoryFreshness, nextCursor?: string | null): string;
+//# sourceMappingURL=list-unenrolled-render.d.ts.map

package/dist/tools/list-unenrolled-render.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"list-unenrolled-render.d.ts","sourceRoot":"","sources":["../../src/tools/list-unenrolled-render.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EACV,uBAAuB,EACvB,kBAAkB,EAGnB,MAAM,aAAa,CAAC;AA4GrB,wBAAgB,WAAW,CACzB,KAAK,EAAE,SAAS,kBAAkB,EAAE,EACpC,SAAS,EAAE,uBAAuB,EAClC,UAAU,GAAE,MAAM,GAAG,IAAW,GAC/B,MAAM,CA0BR;AAmDD,wBAAgB,WAAW,CACzB,SAAS,EAAE,uBAAuB,EAClC,UAAU,GAAE,MAAM,GAAG,IAAW,GAC/B,MAAM,CA2CR"}

package/dist/tools/list-unenrolled-render.js

@@ -0,0 +1,219 @@
+import { GRAPH_LINK_FAILURE_TEXT, graphLinkFailureFrom, } from '../graph-link-failure.js';
+/**
+ * ENG-6410 — what `list_unenrolled_files` prints, derived from the backend's
+ * per-provider freshness rather than from a hardcoded provider.
+ *
+ * Moved out of `list-unenrolled-files.ts` so that file holds the tool's
+ * contract and this one holds its words. The Microsoft-only output is
+ * byte-identical to what shipped before the Google lane existed; that is
+ * pinned by `tools.list-unenrolled.google.test.ts`.
+ */
+/**
+ * ENG-4283 / ENG-6409 — the backend's codes for "this lane does not serve this
+ * account". Duplicated rather than imported: this package ships over npm and
+ * takes no build dependency on the API tree. EXACT match — an unknown value
+ * falls through rather than being guessed at (ENG-4311).
+ */
+const INAPPLICABLE = {
+    NO_MICROSOFT_TENANT: { covers: 'OneDrive and SharePoint', lacks: 'no Microsoft link' },
+    NO_GOOGLE_GRANT: { covers: 'Google Drive', lacks: 'no Google connection' },
+};
+/**
+ * Every lane the backend reported. A backend older than ENG-6409 sends no
+ * `providers` and served Microsoft alone, so its top level IS that one lane.
+ */
+function lanesOf(freshness) {
+    if (freshness.providers?.length)
+        return freshness.providers;
+    const { providers: _absent, ...top } = freshness;
+    return [{ ...top, provider: 'microsoft' }];
+}
+/**
+ * The lanes that serve this caller — the ones its rows can come from. A code
+ * this package does not know is NOT read as "unserved" (ENG-4311: match the
+ * known codes exactly and fall through on anything else).
+ */
+function servedLanes(freshness) {
+    return lanesOf(freshness).filter((lane) => !lane.inapplicableReason || !INAPPLICABLE[lane.inapplicableReason]);
+}
+/** Total over the provider union: a new provider fails to compile here. */
+function readerName(provider) {
+    switch (provider) {
+        case 'microsoft':
+            return 'Microsoft';
+        case 'google':
+            return 'Google';
+        default: {
+            const unhandled = provider;
+            return String(unhandled);
+        }
+    }
+}
+/**
+ * "workbook" while only Microsoft serves the caller, exactly as before; "file"
+ * once a Google lane does, because that lane lists Docs and Slides too.
+ */
+function noun(freshness) {
+    return servedLanes(freshness).some((lane) => lane.provider === 'google')
+        ? 'file'
+        : 'workbook';
+}
+/**
+ * ENG-2814 — the "there is more" line, and it is never optional when a cursor
+ * came back. Written to the MODEL: it names the exact next call, because a
+ * hint that only says "more available" gets summarised away.
+ */
+function moreToCome(nextCursor) {
+    if (!nextCursor)
+        return '';
+    return (`\n\nMORE FILES REMAIN — this is not the whole list. Call ` +
+        `\`list_unenrolled_files\` again with \`cursor="${nextCursor}"\`, and ` +
+        'keep going until a response comes back with no cursor. Do not tell the ' +
+        'user what they do or do not have until then. The cursor stops working 30 ' +
+        'minutes after the first page; if it is refused as expired, start again ' +
+        'from the first page rather than retrying it.');
+}
+/**
+ * The identity line `enroll_file` can act on, in the row's own provider's
+ * terms. A Microsoft row keeps its id pair. A Google row has no drive id and
+ * `enroll_file` takes a Google file only by its link, so the link is the id
+ * printed; with no link recorded the row says so rather than composing one.
+ */
+function identityLine(item) {
+    const provider = item.provider ?? 'microsoft';
+    switch (provider) {
+        case 'microsoft':
+            return `  msId: ${item.msId}, driveMsId: ${item.driveMsId}`;
+        case 'google':
+            return item.webUrl
+                ? `  provider: google, fileId: ${item.msId} — enroll with url: ${item.webUrl}`
+                : `  provider: google, fileId: ${item.msId} — no link recorded; ask ` +
+                    'the user for its link and pass that to `enroll_file` as `url`';
+        default: {
+            const unhandled = provider;
+            return `  provider: ${String(unhandled)}, fileId: ${item.msId}`;
+        }
+    }
+}
+export function renderFound(items, freshness, nextCursor = null) {
+    const lines = items.map((item) => {
+        const where = item.parentPath ? ` in ${item.parentPath}` : '';
+        const modified = item.lastModifiedAt
+            ? `, modified ${item.lastModifiedAt}`
+            : '';
+        // `hidden` means the user REMOVED this file before; enrolling restores
+        // rather than duplicates, the only thing distinguishing it from new.
+        const previously = item.enrollmentState === 'hidden' ? ' [previously removed]' : '';
+        return (`- **${item.name}**${where}${modified}${previously}\n` +
+            identityLine(item));
+    });
+    // "on this page", never a bare count: with a cursor outstanding the number
+    // is a page size, not a total.
+    const counted = nextCursor
+        ? `${items.length} ${noun(freshness)}(s) not yet in Rockhopper on this page:`
+        : `${items.length} ${noun(freshness)}(s) not yet in Rockhopper:`;
+    return (`${counted}\n\n` +
+        `${lines.join('\n')}\n\n${dateline(freshness)}${moreToCome(nextCursor)}`);
+}
+/**
+ * No lane serves the account. No retry language (nothing changes on a later
+ * call), no `connect_microsoft` (ENG-2614's loop), and the scope named from
+ * the lanes the backend reported rather than assumed.
+ */
+function inapplicableText(freshness) {
+    const known = lanesOf(freshness)
+        .map((lane) => INAPPLICABLE[lane.inapplicableReason ?? ''])
+        .filter((entry) => entry !== undefined);
+    return (`Rockhopper's drive inventory covers ${known.map((k) => k.covers).join(' and ')}, ` +
+        `and this account has ${known.map((k) => k.lacks).join(' and ')} — so ` +
+        'there is nothing for it to list and nothing to retry. This is NOT ' +
+        'evidence that every file is already in Rockhopper, and it does not mean ' +
+        'the user has no files: it means this particular list cannot see them. ' +
+        'Do not call `list_unenrolled_files` again for this user, and do not ' +
+        'report their drive as covered. A file the user can link to is still ' +
+        'added with `enroll_file` and that link.');
+}
+/** A Google lane whose refresh could not use the user's Google credential. */
+const GOOGLE_LINK_FAILURE_TEXT = 'Rockhopper cannot see this user\'s Google Drive files: their Google ' +
+    'account is not connected, or its connection has expired. Ask the user to ' +
+    'connect Google Drive in Rockhopper Settings; do not compose a sign-in link ' +
+    'yourself. This list will stay empty until that is done, which is NOT the ' +
+    'same as having nothing to add.';
+/** The first serving lane whose last refresh failed on its link, as text. */
+function linkFailureText(freshness) {
+    for (const lane of servedLanes(freshness)) {
+        const failure = graphLinkFailureFrom(lane.lastFailureReason);
+        if (!failure)
+            continue;
+        switch (lane.provider) {
+            case 'microsoft':
+                return GRAPH_LINK_FAILURE_TEXT[failure];
+            case 'google':
+                return GOOGLE_LINK_FAILURE_TEXT;
+            default: {
+                const unhandled = lane.provider;
+                return String(unhandled);
+            }
+        }
+    }
+    return null;
+}
+export function renderEmpty(freshness, nextCursor = null) {
+    // ORDER IS LOAD-BEARING (ENG-4283, ENG-4311, ENG-2814). An account no lane
+    // serves comes first: every later branch assumes a lane covers the caller,
+    // and its never-refreshed branch would otherwise claim a scan "has been
+    // started" forever. A broken link and an unfinished first refresh come next,
+    // because neither has anything to page THROUGH.
+    if (freshness.inapplicableReason && INAPPLICABLE[freshness.inapplicableReason]) {
+        return inapplicableText(freshness);
+    }
+    const linkFailure = linkFailureText(freshness);
+    if (linkFailure)
+        return linkFailure;
+    if (!freshness.asOf) {
+        return ('No answer yet — the first scan of this user\'s drive has not finished. ' +
+            `${freshness.refreshing ? 'One is running now.' : 'One has been started.'} ` +
+            'Try again shortly. This is NOT evidence that every workbook is already ' +
+            `in Rockhopper.${failureNote(freshness)}`);
+    }
+    // ENG-2814 — an empty page WITH a cursor is the middle of a walk: the
+    // backend filters after cutting a chunk and stops at a scan budget.
+    if (nextCursor) {
+        return ('No un-enrolled workbooks on this page — but the search is NOT ' +
+            'finished, and this says nothing yet about what the user has.' +
+            `${moreToCome(nextCursor)}`);
+    }
+    if (freshness.consecutiveFailures > 0) {
+        return ('No un-enrolled workbooks in the stored list, but the list is not ' +
+            `trustworthy right now.${failureNote(freshness)}\n\n${dateline(freshness)}`);
+    }
+    return (`Every ${noun(freshness)} Rockhopper has seen for this user is already in ` +
+        `Rockhopper.\n\n${dateline(freshness)}`);
+}
+/**
+ * How old the answer is, on every branch that has an answer, and which
+ * providers it was read from — never omitted, because the rows are stored.
+ */
+function dateline(freshness) {
+    const asOf = freshness.asOf
+        ? `As of ${freshness.asOf}`
+        : 'Never successfully refreshed';
+    const stale = freshness.stale ? ' (stale)' : '';
+    const refreshing = freshness.refreshing
+        ? ' A refresh is running; call again for a newer answer.'
+        : '';
+    const served = servedLanes(freshness);
+    const readers = (served.length ? served : lanesOf(freshness))
+        .map((lane) => readerName(lane.provider))
+        .join(' or ');
+    return (`${asOf}${stale} — from Rockhopper's stored records, not a live ` +
+        `${readers} read.${refreshing}${failureNote(freshness)}`);
+}
+function failureNote(freshness) {
+    if (!freshness.consecutiveFailures || !freshness.lastFailureAt)
+        return '';
+    return (` Refresh has failed ${freshness.consecutiveFailures} time(s) in a row, ` +
+        `last at ${freshness.lastFailureAt}` +
+        `${freshness.lastFailureReason ? ` (${freshness.lastFailureReason})` : ''}.`);
+}
+//# sourceMappingURL=list-unenrolled-render.js.map

package/dist/tools/list-unenrolled-render.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"list-unenrolled-render.js","sourceRoot":"","sources":["../../src/tools/list-unenrolled-render.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,uBAAuB,EACvB,oBAAoB,GACrB,MAAM,0BAA0B,CAAC;AAQlC;;;;;;;;GAQG;AAEH;;;;;GAKG;AACH,MAAM,YAAY,GAAgE;IAChF,mBAAmB,EAAE,EAAE,MAAM,EAAE,yBAAyB,EAAE,KAAK,EAAE,mBAAmB,EAAE;IACtF,eAAe,EAAE,EAAE,MAAM,EAAE,cAAc,EAAE,KAAK,EAAE,sBAAsB,EAAE;CAC3E,CAAC;AAEF;;;GAGG;AACH,SAAS,OAAO,CAAC,SAAkC;IACjD,IAAI,SAAS,CAAC,SAAS,EAAE,MAAM;QAAE,OAAO,SAAS,CAAC,SAAS,CAAC;IAC5D,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,GAAG,EAAE,GAAG,SAAS,CAAC;IACjD,OAAO,CAAC,EAAE,GAAG,GAAG,EAAE,QAAQ,EAAE,WAAW,EAAE,CAAC,CAAC;AAC7C,CAAC;AAED;;;;GAIG;AACH,SAAS,WAAW,CAAC,SAAkC;IACrD,OAAO,OAAO,CAAC,SAAS,CAAC,CAAC,MAAM,CAC9B,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,kBAAkB,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAC7E,CAAC;AACJ,CAAC;AAED,2EAA2E;AAC3E,SAAS,UAAU,CAAC,QAAsB;IACxC,QAAQ,QAAQ,EAAE,CAAC;QACjB,KAAK,WAAW;YACd,OAAO,WAAW,CAAC;QACrB,KAAK,QAAQ;YACX,OAAO,QAAQ,CAAC;QAClB,OAAO,CAAC,CAAC,CAAC;YACR,MAAM,SAAS,GAAU,QAAQ,CAAC;YAClC,OAAO,MAAM,CAAC,SAAS,CAAC,CAAC;QAC3B,CAAC;IACH,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,SAAS,IAAI,CAAC,SAAkC;IAC9C,OAAO,WAAW,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,KAAK,QAAQ,CAAC;QACtE,CAAC,CAAC,MAAM;QACR,CAAC,CAAC,UAAU,CAAC;AACjB,CAAC;AAED;;;;GAIG;AACH,SAAS,UAAU,CAAC,UAAyB;IAC3C,IAAI,CAAC,UAAU;QAAE,OAAO,EAAE,CAAC;IAC3B,OAAO,CACL,2DAA2D;QAC3D,kDAAkD,UAAU,WAAW;QACvE,yEAAyE;QACzE,2EAA2E;QAC3E,yEAAyE;QACzE,8CAA8C,CAC/C,CAAC;AACJ,CAAC;AAED;;;;;GAKG;AACH,SAAS,YAAY,CAAC,IAAwB;IAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,QAAQ,IAAI,WAAW,CAAC;IAC9C,QAAQ,QAAQ,EAAE,CAAC;QACjB,KAAK,WAAW;YACd,OAAO,WAAW,IAAI,CAAC,IAAI,gBAAgB,IAAI,CAAC,SAAS,EAAE,CAAC;QAC9D,KAAK,QAAQ;YACX,OAAO,IAAI,CAAC,MAAM;gBAChB,CAAC,CAAC,+BAA+B,IAAI,CAAC,IAAI,uBAAuB,IAAI,CAAC,MAAM,EAAE;gBAC9E,CAAC,CAAC,+BAA+B,IAAI,CAAC,IAAI,2BAA2B;oBACjE,+DAA+D,CAAC;QACxE,OAAO,CAAC,CAAC,CAAC;YACR,MAAM,SAAS,GAAU,QAAQ,CAAC;YAClC,OAAO,eAAe,MAAM,CAAC,SAAS,CAAC,aAAa,IAAI,CAAC,IAAI,EAAE,CAAC;QAClE,CAAC;IACH,CAAC;AACH,CAAC;AAED,MAAM,UAAU,WAAW,CACzB,KAAoC,EACpC,SAAkC,EAClC,aAA4B,IAAI;IAEhC,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC9D,MAAM,QAAQ,GAAG,IAAI,CAAC,cAAc;YAClC,CAAC,CAAC,cAAc,IAAI,CAAC,cAAc,EAAE;YACrC,CAAC,CAAC,EAAE,CAAC;QACP,uEAAuE;QACvE,qEAAqE;QACrE,MAAM,UAAU,GACd,IAAI,CAAC,eAAe,KAAK,QAAQ,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,EAAE,CAAC;QACnE,OAAO,CACL,OAAO,IAAI,CAAC,IAAI,KAAK,KAAK,GAAG,QAAQ,GAAG,UAAU,IAAI;YACtD,YAAY,CAAC,IAAI,CAAC,CACnB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,2EAA2E;IAC3E,+BAA+B;IAC/B,MAAM,OAAO,GAAG,UAAU;QACxB,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,IAAI,IAAI,CAAC,SAAS,CAAC,yCAAyC;QAC7E,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,IAAI,IAAI,CAAC,SAAS,CAAC,4BAA4B,CAAC;IAEnE,OAAO,CACL,GAAG,OAAO,MAAM;QAChB,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,QAAQ,CAAC,SAAS,CAAC,GAAG,UAAU,CAAC,UAAU,CAAC,EAAE,CACzE,CAAC;AACJ,CAAC;AAGD;;;;GAIG;AACH,SAAS,gBAAgB,CAAC,SAAkC;IAC1D,MAAM,KAAK,GAAG,OAAO,CAAC,SAAS,CAAC;SAC7B,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,kBAAkB,IAAI,EAAE,CAAC,CAAC;SAC1D,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC;IAC1C,OAAO,CACL,uCAAuC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI;QACnF,wBAAwB,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ;QACvE,oEAAoE;QACpE,0EAA0E;QAC1E,wEAAwE;QACxE,sEAAsE;QACtE,sEAAsE;QACtE,yCAAyC,CAC1C,CAAC;AACJ,CAAC;AAED,8EAA8E;AAC9E,MAAM,wBAAwB,GAC5B,sEAAsE;IACtE,2EAA2E;IAC3E,6EAA6E;IAC7E,2EAA2E;IAC3E,gCAAgC,CAAC;AAEnC,6EAA6E;AAC7E,SAAS,eAAe,CAAC,SAAkC;IACzD,KAAK,MAAM,IAAI,IAAI,WAAW,CAAC,SAAS,CAAC,EAAE,CAAC;QAC1C,MAAM,OAAO,GAAG,oBAAoB,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC7D,IAAI,CAAC,OAAO;YAAE,SAAS;QACvB,QAAQ,IAAI,CAAC,QAAQ,EAAE,CAAC;YACtB,KAAK,WAAW;gBACd,OAAO,uBAAuB,CAAC,OAAO,CAAC,CAAC;YAC1C,KAAK,QAAQ;gBACX,OAAO,wBAAwB,CAAC;YAClC,OAAO,CAAC,CAAC,CAAC;gBACR,MAAM,SAAS,GAAU,IAAI,CAAC,QAAQ,CAAC;gBACvC,OAAO,MAAM,CAAC,SAAS,CAAC,CAAC;YAC3B,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,WAAW,CACzB,SAAkC,EAClC,aAA4B,IAAI;IAEhC,2EAA2E;IAC3E,2EAA2E;IAC3E,wEAAwE;IACxE,6EAA6E;IAC7E,gDAAgD;IAChD,IAAI,SAAS,CAAC,kBAAkB,IAAI,YAAY,CAAC,SAAS,CAAC,kBAAkB,CAAC,EAAE,CAAC;QAC/E,OAAO,gBAAgB,CAAC,SAAS,CAAC,CAAC;IACrC,CAAC;IAED,MAAM,WAAW,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC;IAC/C,IAAI,WAAW;QAAE,OAAO,WAAW,CAAC;IAEpC,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC;QACpB,OAAO,CACL,yEAAyE;YACzE,GAAG,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,uBAAuB,GAAG;YAC5E,yEAAyE;YACzE,iBAAiB,WAAW,CAAC,SAAS,CAAC,EAAE,CAC1C,CAAC;IACJ,CAAC;IAED,sEAAsE;IACtE,oEAAoE;IACpE,IAAI,UAAU,EAAE,CAAC;QACf,OAAO,CACL,gEAAgE;YAChE,8DAA8D;YAC9D,GAAG,UAAU,CAAC,UAAU,CAAC,EAAE,CAC5B,CAAC;IACJ,CAAC;IAED,IAAI,SAAS,CAAC,mBAAmB,GAAG,CAAC,EAAE,CAAC;QACtC,OAAO,CACL,mEAAmE;YACnE,yBAAyB,WAAW,CAAC,SAAS,CAAC,OAAO,QAAQ,CAAC,SAAS,CAAC,EAAE,CAC5E,CAAC;IACJ,CAAC;IAED,OAAO,CACL,SAAS,IAAI,CAAC,SAAS,CAAC,mDAAmD;QAC3E,kBAAkB,QAAQ,CAAC,SAAS,CAAC,EAAE,CACxC,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,SAAS,QAAQ,CAAC,SAAkC;IAClD,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI;QACzB,CAAC,CAAC,SAAS,SAAS,CAAC,IAAI,EAAE;QAC3B,CAAC,CAAC,8BAA8B,CAAC;IACnC,MAAM,KAAK,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC;IAChD,MAAM,UAAU,GAAG,SAAS,CAAC,UAAU;QACrC,CAAC,CAAC,uDAAuD;QACzD,CAAC,CAAC,EAAE,CAAC;IACP,MAAM,MAAM,GAAG,WAAW,CAAC,SAAS,CAAC,CAAC;IACtC,MAAM,OAAO,GAAG,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;SAC1D,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;SACxC,IAAI,CAAC,MAAM,CAAC,CAAC;IAChB,OAAO,CACL,GAAG,IAAI,GAAG,KAAK,kDAAkD;QACjE,GAAG,OAAO,SAAS,UAAU,GAAG,WAAW,CAAC,SAAS,CAAC,EAAE,CACzD,CAAC;AACJ,CAAC;AAED,SAAS,WAAW,CAAC,SAAkC;IACrD,IAAI,CAAC,SAAS,CAAC,mBAAmB,IAAI,CAAC,SAAS,CAAC,aAAa;QAAE,OAAO,EAAE,CAAC;IAC1E,OAAO,CACL,uBAAuB,SAAS,CAAC,mBAAmB,qBAAqB;QACzE,WAAW,SAAS,CAAC,aAAa,EAAE;QACpC,GAAG,SAAS,CAAC,iBAAiB,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,CAAC,EAAE,GAAG,CAC7E,CAAC;AACJ,CAAC"}

package/dist/tools/search.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"search.d.ts","sourceRoot":"","sources":["../../src/tools/search.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAC;AAE9D,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAkBlD,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,SAAS,EACjB,GAAG,EAAE,SAAS,GACb,IAAI,CA8UN"}
+{"version":3,"file":"search.d.ts","sourceRoot":"","sources":["../../src/tools/search.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAC;AAE9D,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAMlD,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,SAAS,EACjB,GAAG,EAAE,SAAS,GACb,IAAI,CAmSN"}

package/dist/tools/search.js

@@ -1,8 +1,8 @@
 import { z } from 'zod';
-import { assertChangeHistoryComplete, assertEnrollmentComplete, isNotReady, notReadyToolResult, } from '../not-ready.js';
-import { changeModelForFileType, documentChangesUnavailableToolResult, } from '../document-changes.js';
-import { assertSheetExists, isUnknownSheet, unknownSheetToolResult, } from '../sheet-catalogue.js';
-import { readDocumentChanges } from './document-changes-read.js';
+import { isNotReady, notReadyToolResult } from '../not-ready.js';
+import { formatDocumentChanges } from '../document-changes.js';
+import { fileChangesUnavailableText, readFileChanges } from '../file-changes.js';
+import { isUnknownSheet, unknownSheetToolResult } from '../sheet-catalogue.js';
 export function registerSearchTool(server, api) {
     server.registerTool('search_files', {
         title: 'Search Files',
@@ -151,55 +151,31 @@ export function registerSearchTool(server, api) {
         },
     }, async ({ fileMsId, sheetName, cursor }) => {
         try {
-            // ENG-5397 — WHICH CHANGE MODEL, before which reader. The two lanes
-            // below are spreadsheet-only: `unattributed_change` requires a
-            // `sheetName` and the sheet route also filters `changeType = 'cell'`,
-            // so a Word document or a deck returned `[]` from them by
-            // construction and the tool printed "No unattributed changes found".
-            // Routing on the file's own type is what makes an absence mean
-            // something.
-            const file = await api.getEnrolledFile(fileMsId);
-            const model = changeModelForFileType(file.fileType);
-            if (model === null) {
-                return documentChangesUnavailableToolResult({
-                    reason: 'unknown_file_type',
-                    fileMsId,
-                    fileName: file.name,
-                });
+            // ENG-5397 — WHICH CHANGE MODEL, before which reader: the spreadsheet
+            // lanes cannot hold a document row, so a `.docx` read `[]` from them by
+            // construction. ENG-6431 — that dispatch (plus the fold, enrolment and
+            // sheet-existence gates) lives in `readFileChanges`, shared with the
+            // `…/changes` resource and the prompts; this tool only formats.
+            const read = await readFileChanges(api, fileMsId, { sheetName, cursor });
+            if (read.kind === 'unavailable') {
+                return {
+                    content: [{ type: 'text', text: fileChangesUnavailableText(read) }],
+                    isError: true,
+                };
             }
-            if (model !== 'spreadsheet') {
-                return await readDocumentChanges(api, {
-                    fileMsId,
-                    sheetName,
-                    fileType: file.fileType,
-                    fileName: file.name,
-                });
+            if (read.kind === 'document') {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: formatDocumentChanges(read.response, read.file.name),
+                        },
+                    ],
+                };
             }
-            // Plan 02 ruling 5 (STRICT): gate BOTH modes. The commit-diff fold
-            // retracts and rewrites the uncommitted window, so a pending fold
-            // means this list is mid-rewrite in either shape.
-            await assertChangeHistoryComplete(api, fileMsId);
-            if (sheetName) {
+            if (read.kind === 'spreadsheet_sheet') {
                 // Sheet-filtered mode: bounded by sheet size, no pagination needed.
-                // Audit (2026-05-23) measured: `Summary` sheet on a 12 MB file
-                // returned 9 rows clean. This is the practical workaround when the
-                // caller already knows which sheet to drill into.
-                const changes = await api.getUnattributedChangesBySheet(fileMsId, sheetName);
-                // ENG-2824: an empty answer is the only ambiguous one, so it is the
-                // only one that pays for the version read. A file still being read
-                // for the first time has no versions and no parsed cells, and the
-                // fold probe above cannot see that state — enrolment enqueues no
-                // fold. See `assertEnrollmentComplete`.
-                if (changes.length === 0) {
-                    assertEnrollmentComplete(fileMsId, await api.getFileVersions(fileMsId));
-                    // ENG-4347 — and the same trade one ambiguity over: only an empty
-                    // answer asserts something this tool has not checked. A row that
-                    // came back names its own sheet, so a non-empty answer has already
-                    // proved what this goes and asks. Enrolment is settled FIRST above
-                    // because a file whose initial read has not landed has no
-                    // catalogue either, and "still reading" is the truer answer.
-                    await assertSheetExists(api, file, sheetName);
-                }
+                const { changes } = read;
                 const body = formatChangeRows(changes);
                 return {
                     content: [
@@ -217,14 +193,8 @@ export function registerSearchTool(server, api) {
             // 25k-token client limit (audit measured one file at 12.5 MB / 28k
             // rows on the unpaginated route — KI-097).
             const MAX_DISPLAYED = 200;
-            const page = await api.getUnattributedChangesPaginated(fileMsId, cursor);
+            const { page } = read;
             if (page.changes.length === 0) {
-                // ENG-2824 — same rule as the sheet mode above. `totalCount === 0`
-                // over a file whose initial read has not landed is a premature
-                // absence, not a fact about the workbook.
-                if (page.totalCount === 0) {
-                    assertEnrollmentComplete(fileMsId, await api.getFileVersions(fileMsId));
-                }
                 return {
                     content: [
                         {