@open-webapp/drive-sync 0.5.3 → 0.5.4

package/dist/errors.d.ts

@@ -46,6 +46,15 @@ export declare class NotFoundError extends DriveSyncError {
     fileId: string;
     constructor(fileId: string, opts?: DriveSyncErrorOptions);
 }
+/**
+ * Thrown when Drive refuses to serve a file's raw bytes via `alt=media`
+ * because it isn't binary content (e.g. a native Google Docs/Sheets/Slides
+ * file, which only supports Export, not Download).
+ */
+export declare class NotDownloadableError extends DriveSyncError {
+    fileId: string;
+    constructor(fileId: string, opts?: DriveSyncErrorOptions);
+}
 export interface RateLimitedErrorOptions extends DriveSyncErrorOptions {
     retryAfter?: number;
 }

package/dist/errors.js

@@ -59,6 +59,19 @@ export class NotFoundError extends DriveSyncError {
         this.fileId = fileId;
     }
 }
+/**
+ * Thrown when Drive refuses to serve a file's raw bytes via `alt=media`
+ * because it isn't binary content (e.g. a native Google Docs/Sheets/Slides
+ * file, which only supports Export, not Download).
+ */
+export class NotDownloadableError extends DriveSyncError {
+    fileId;
+    constructor(fileId, opts) {
+        super(`File not downloadable as raw content: ${fileId}`, { status: 403, ...opts });
+        this.name = 'NotDownloadableError';
+        this.fileId = fileId;
+    }
+}
 /** Thrown on a 429 response. */
 export class RateLimitedError extends DriveSyncError {
     retryAfter;

package/dist/files.d.ts

@@ -18,7 +18,9 @@ export interface ReadOptions extends BaseCallOptions {
  * since Drive 404s both for a genuinely wrong id and for a file the
  * connected account cannot see — this ambiguity is documented in the public
  * API and callers are expected to treat `null` as "not available" rather
- * than distinguishing the two cases.
+ * than distinguishing the two cases. Also returns `null` for a native Google
+ * Workspace file (Docs/Sheets/Slides/...), since those only support Export,
+ * not the raw `alt=media` download this function issues.
  */
 export declare function read(opts: ReadOptions): Promise<string | Blob | null>;
 export interface RemoveOptions extends BaseCallOptions {

package/dist/files.js

@@ -1,6 +1,6 @@
 import { driveFetch } from './http.js';
 import { escapeQ } from './query.js';
-import { NotFoundError, RemoteChangedError } from './errors.js';
+import { NotFoundError, NotDownloadableError, RemoteChangedError } from './errors.js';
 import { getFileState, setFileState, clearFileState } from './storage.js';
 const DRIVE_BASE = 'https://www.googleapis.com/drive/v3';
 const UPLOAD_BASE = 'https://www.googleapis.com/upload/drive/v3';
@@ -53,7 +53,9 @@ async function recordBaseline(opts, fileId, version) {
  * since Drive 404s both for a genuinely wrong id and for a file the
  * connected account cannot see — this ambiguity is documented in the public
  * API and callers are expected to treat `null` as "not available" rather
- * than distinguishing the two cases.
+ * than distinguishing the two cases. Also returns `null` for a native Google
+ * Workspace file (Docs/Sheets/Slides/...), since those only support Export,
+ * not the raw `alt=media` download this function issues.
  */
 export async function read(opts) {
     try {
@@ -88,7 +90,7 @@ export async function read(opts) {
         return content;
     }
     catch (err) {
-        if (err instanceof NotFoundError) {
+        if (err instanceof NotFoundError || err instanceof NotDownloadableError) {
             return null;
         }
         throw err;

package/dist/http.js

@@ -1,7 +1,7 @@
 import { acquireToken } from './token.js';
 import { getConnection, refreshSilently } from './connection.js';
 import { clearToken, getToken } from './storage.js';
-import { DriveSyncError, NeedsReauthError, ScopeInsufficientError, NotFoundError, RateLimitedError, TransientError, WrongAccountError, } from './errors.js';
+import { DriveSyncError, NeedsReauthError, ScopeInsufficientError, NotFoundError, NotDownloadableError, RateLimitedError, TransientError, WrongAccountError, } from './errors.js';
 const MAX_ATTEMPTS = 3;
 const BASE_DELAY_MS = 500;
 /** Mirrors connection.ts's own buffer: a cached token this close to expiry is treated as unusable. */
@@ -182,6 +182,12 @@ async function performFetch(opts, accessToken, isRetryAfter401) {
                 reason: lastBodyText,
             });
         }
+        if (lastBodyText.includes('fileNotDownloadable')) {
+            // No fileId is reliably available at this layer (see the 404 case
+            // below) — files.ts re-catches this and returns null instead of
+            // rethrowing with the concrete fileId, so the empty default is fine.
+            throw new NotDownloadableError('', { status: 403, reason: lastBodyText });
+        }
         throw new DriveSyncError(`Drive request forbidden (403): ${lastBodyText}`, {
             status: 403,
             reason: lastBodyText,

package/dist/index.js

@@ -227,7 +227,11 @@ export function createDriveSync(options) {
                 });
                 const results = [];
                 for (const p of picked) {
-                    const content = await filesImpl.read({ ...base, fileId: p.fileId, interactive: true });
+                    // Not interactive: the token above was already acquired
+                    // interactively moments ago for the Picker itself and is now
+                    // cached, so each file read reuses it (or silently refreshes)
+                    // instead of forcing its own consent prompt per file.
+                    const content = await filesImpl.read({ ...base, fileId: p.fileId, interactive: false });
                     results.push({ fileId: p.fileId, name: p.name, mimeType: p.mimeType, content });
                 }
                 return results;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-webapp/drive-sync",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "type": "module",
   "repository": {
     "type": "git",