@skaile/workspaces 0.41.3 → 0.41.4

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.41.4
+
+### Patch Changes
+
+- [#177](https://github.com/skaile-ai/workspaces/pull/177) [`2aff69b`](https://github.com/skaile-ai/workspaces/commit/2aff69b1e155795da3a28ed4a16197eb33125cd5) Thanks [@peteralbert](https://github.com/peteralbert)! - fix(sharepoint): implement `resolveWebUrl` so "Open in SharePoint" works for Office files
+
+  The SharePoint connector now answers the `resolveWebUrl` custom op (Microsoft Graph `webUrl` lookup) that the platform's `resolveViewerUrl` path already calls. Previously no connector implemented it and the mount resource handler had no `_customOp` branch, so Office files (xlsx/docx/pptx) on a SharePoint mount only offered a download instead of opening in SharePoint. Fleet-managed mounts return null (no in-container token) and keep the download fallback.
+
 ## 0.41.3
 
 ### Patch Changes

package/dist/asset-manager/index.js

@@ -1,4 +1,4 @@
-export { AssetManager, parseCanonicalLockKey } from '../chunk-BNBHHVZ5.js';
+export { AssetManager, parseCanonicalLockKey } from '../chunk-CQ7OEJQT.js';
 export { createScaffold, deployAll, installAgent, removeAsset } from '../chunk-TB342H55.js';
 import '../chunk-MVXXAXDW.js';
 import '../chunk-ZAP2BDHG.js';

package/dist/{chunk-Z6DAVEDZ.js → chunk-3JTWGXL6.js}

@@ -3464,6 +3464,12 @@ function cacheBase2() {
 var SharePointConnector = class extends AbstractConnector {
   name = "sharepoint";
   rclone = new RcloneProcessManager();
+  /**
+   * Captured on a successful non-fleet `connect()` so `resolveWebUrl` can mint a
+   * fresh Graph token later. Null until connected, and stays null for
+   * fleet-managed mounts (the host owns the token; no in-container resolution).
+   */
+  webUrlCtx = null;
   filesystem = {
     deferMount: true,
     sync: (handle, options) => {
@@ -3578,6 +3584,7 @@ var SharePointConnector = class extends AbstractConnector {
     this.log.info("mount ready", { pid: rcloneHandle.pid });
     handleRef.current = rcloneHandle;
     refresh?.schedule(cred.expiresAt);
+    this.webUrlCtx = { declaration, secrets, tokenMediator, driveId, folderPath, basePath };
     const state = { handleRef, refresh };
     return this.makeHandle(declaration, state, targetDir);
   }
@@ -3621,6 +3628,42 @@ var SharePointConnector = class extends AbstractConnector {
     await this._sync(handle);
     return "flushed";
   }
+  /**
+   * Resolve the SharePoint/OneDrive browser URL (Microsoft Graph `webUrl`) for
+   * the file at `relPath` (relative to the mount root). The platform's
+   * `resolveViewerUrl` path calls this so Office files that can't render inline
+   * (xlsx/docx/pptx) offer "Open in SharePoint" instead of only a download.
+   *
+   * Best-effort: returns null for fleet-managed mounts (no in-container token)
+   * or on any Graph failure, so the caller falls back to download.
+   */
+  async resolveWebUrl(relPath) {
+    const ctx = this.webUrlCtx;
+    if (!ctx) return null;
+    try {
+      const cred = await resolveMountCredential(
+        ctx.declaration,
+        ctx.secrets,
+        ctx.tokenMediator,
+        "refresh"
+      );
+      const token = parseAuthBlob2(cred.token, this.log).accessToken;
+      if (!token) return null;
+      const endpoint = buildGraphWebUrlEndpoint(ctx.driveId, ctx.folderPath, ctx.basePath, relPath);
+      const res = await fetch(endpoint, { headers: { Authorization: `Bearer ${token}` } });
+      if (!res.ok) {
+        this.log.warn("resolveWebUrl: Graph request failed", { status: res.status });
+        return null;
+      }
+      const body = await res.json();
+      return body.webUrl ?? null;
+    } catch (err) {
+      this.log.warn("resolveWebUrl failed", {
+        error: err instanceof Error ? err.message : String(err)
+      });
+      return null;
+    }
+  }
 };
 function createConnector13() {
   return new SharePointConnector();
@@ -3655,6 +3698,15 @@ function parseAuthBlob2(raw, log) {
     return { ...empty, accessToken: raw };
   }
 }
+function buildGraphWebUrlEndpoint(driveId, folderPath, basePath, relPath) {
+  const drive = encodeURIComponent(driveId);
+  const itemPath = [folderPath, basePath, relPath].map((p) => p.replace(/^\/+|\/+$/g, "")).filter((p) => p.length > 0).join("/");
+  if (!itemPath) {
+    return `https://graph.microsoft.com/v1.0/drives/${drive}/root?$select=webUrl`;
+  }
+  const encoded = itemPath.split("/").map((seg) => encodeURIComponent(seg)).join("/");
+  return `https://graph.microsoft.com/v1.0/drives/${drive}/root:/${encoded}:?$select=webUrl`;
+}
 function buildRemotePath(folderPath, basePath) {
   const folder = folderPath.replace(/^\/+|\/+$/g, "");
   const base = basePath.replace(/^\/+|\/+$/g, "");
@@ -8457,6 +8509,6 @@ function createConnector() {
   return new FlowAdapter();
 }
 
-export { AbstractConnector, BUILTIN_CONNECTOR_CATALOG, ConnectorFieldMissingError, ConnectorManager, ConnectorStartupError, DEFAULT_REACTIVE_COOLDOWN_MS, DeployConnector, DevServerConnector, FleetHealthGate, FlowAdapter, GitConnector, GmailConnector, GoogleDriveConnector, LocalConnector, LogBuffer, ManagedGitconfigCollisionError, MattermostConnector, MemoryConnector, MinIOConnector, NOT_READY_SENTINEL, PROACTIVE_RCLONE_REFRESH_MARGIN_MS, PostgresConnector, RoMountPlaceholderBarrier, S3Connector, SQLiteConnector, SharePointConnector, StaticServerConnector, TunnelConnector, WebDAVConnector, XStateConnector, XStateStoreConnector, __test, atomicReplaceCredential, buildConnectorPromptSection, buildSdkConnectorTools, createConnector, createConnector10, createConnector11, createConnector12, createConnector13, createConnector14, createConnector15, createConnector16, createConnector17, createConnector18, createConnector19, createConnector2, createConnector3, createConnector4, createConnector5, createConnector6, createConnector7, createConnector8, createConnector9, createRcloneTokenRefresh, createWorktree, defaultMountPlaceholderBarrier, defaultSpawn, ensureFleetMounted, findGitRoot, findMissingPackages, getConnector, installNpmPackages, isFleetManaged, isPackageResolvable, listConnectors, listWorktrees, probeFleetHealth, registerBuiltinConnectors, removeMountBlock, renderCredentialHelperScript, tryGetConnector, worktreeRegistry, writeHelperScript, writeMountBlock };
-//# sourceMappingURL=chunk-Z6DAVEDZ.js.map
-//# sourceMappingURL=chunk-Z6DAVEDZ.js.map
+export { AbstractConnector, BUILTIN_CONNECTOR_CATALOG, ConnectorFieldMissingError, ConnectorManager, ConnectorStartupError, DEFAULT_REACTIVE_COOLDOWN_MS, DeployConnector, DevServerConnector, FleetHealthGate, FlowAdapter, GitConnector, GmailConnector, GoogleDriveConnector, LocalConnector, LogBuffer, ManagedGitconfigCollisionError, MattermostConnector, MemoryConnector, MinIOConnector, NOT_READY_SENTINEL, PROACTIVE_RCLONE_REFRESH_MARGIN_MS, PostgresConnector, RoMountPlaceholderBarrier, S3Connector, SQLiteConnector, SharePointConnector, StaticServerConnector, TunnelConnector, WebDAVConnector, XStateConnector, XStateStoreConnector, __test, atomicReplaceCredential, buildConnectorPromptSection, buildGraphWebUrlEndpoint, buildSdkConnectorTools, createConnector, createConnector10, createConnector11, createConnector12, createConnector13, createConnector14, createConnector15, createConnector16, createConnector17, createConnector18, createConnector19, createConnector2, createConnector3, createConnector4, createConnector5, createConnector6, createConnector7, createConnector8, createConnector9, createRcloneTokenRefresh, createWorktree, defaultMountPlaceholderBarrier, defaultSpawn, ensureFleetMounted, findGitRoot, findMissingPackages, getConnector, installNpmPackages, isFleetManaged, isPackageResolvable, listConnectors, listWorktrees, probeFleetHealth, registerBuiltinConnectors, removeMountBlock, renderCredentialHelperScript, tryGetConnector, worktreeRegistry, writeHelperScript, writeMountBlock };
+//# sourceMappingURL=chunk-3JTWGXL6.js.map
+//# sourceMappingURL=chunk-3JTWGXL6.js.map