@banana-peeljj12/dsh-trellis 0.3.1 → 0.3.2

package/lib/client.js

@@ -27,6 +27,14 @@ window.__ModuleLoader__.load({
 
     let react = require('react');
 
+    /**
+     * Module-level client root context, captured in apply(ctx). Components (slot
+     * registrations) do not receive ctx directly, but need host services such as
+     * `sidebarRight` (native document preview) at click time — this is the
+     * bridge. Null until apply runs, which happens before any component mounts.
+     */
+    let clientCtx = null;
+
     // #region lib/types/client/trellis-settings-tab.js
 
     /** Settings namespace registered by the Host half (see lib/meta.js). */
@@ -36,7 +44,7 @@ window.__ModuleLoader__.load({
     const LOCALE_NS = 'settings.trellisWorkflow';
 
     /** Services required by this client half. */
-    const inject = ['slots', 'locale', 'settingsScope'];
+    const inject = ['slots', 'locale', 'settingsScope', 'sidebarRight'];
 
     const zh = {
       tab: 'Trellis 工作流',
@@ -94,6 +102,7 @@ window.__ModuleLoader__.load({
       sendToChat: '推进任务',
       sendToChatSuccess: '已填入输入框，回车即可发送',
       sendToChatFailed: '自动填入失败，已复制到剪贴板',
+      openFailed: '打开失败，文件引用已复制到剪贴板',
       pendingVerification: '待验证',
       blocked: '阻塞',
       open: '打开',
@@ -171,6 +180,7 @@ window.__ModuleLoader__.load({
       sendToChat: 'Push to chat',
       sendToChatSuccess: 'Filled into the input — press Enter to send',
       sendToChatFailed: 'Auto-fill failed, copied to clipboard',
+      openFailed: 'Failed to open — file reference copied to clipboard',
       pendingVerification: 'Verifying',
       blocked: 'Blocked',
       open: 'Open',
@@ -942,8 +952,72 @@ window.__ModuleLoader__.load({
       return '@.trellis/tasks/' + task.slug + '/' + name;
     }
 
+    /**
+     * DSH-native file address for an artifact token. Mirrors the official
+     * `fileAddressFor` / `sessionFileAddress` contract
+     * (@deepseek-ai/dsh-client-ui-sidebar-files): the token's path (minus the
+     * leading `@`) is a workspace-relative path, so it maps to a session-scoped
+     * `dsh-resource://file/session/<sessionId>/<path>` address the right-sidebar
+     * document preview resolves against the session's workspace. Path segments
+     * are component-encoded, keeping `:` literal for drive letters.
+     * @param {string} sessionId the session whose workspace resolves the path.
+     * @param {string} token artifact token from {@link artifactToken}.
+     * @returns {string} `dsh-resource://file/session/...` address.
+     */
+    function artifactResourceUrl(sessionId, token) {
+      const path = (token || '').replace(/^@/, '');
+      const encodeSegment = (segment) => encodeURIComponent(segment).replace(/%3A/gi, ':');
+      const encodedPath = path
+        .replace(/\\/g, '/')
+        .split('/')
+        .map(encodeSegment)
+        .join('/');
+      return 'dsh-resource://file/session/' + encodeSegment(sessionId) + '/' + encodedPath;
+    }
+
+    /**
+     * Open an artifact file in the DSH-native document preview (right sidebar),
+     * mirroring how the official chat client opens `@file` references
+     * (`ctx.sidebarRight.openResource(...)`). Never touches the composer input.
+     * Falls back to copying the token to the clipboard when the right-sidebar
+     * service is unavailable or the open fails, so the click still yields
+     * something useful without polluting the input box.
+     * @param {object} ctx client root context (with `sidebarRight` injected).
+     * @param {string} sessionId the session owning the artifact's workspace.
+     * @param {string} token artifact token from {@link artifactToken}.
+     * @returns {boolean} true when opened natively; false when only copied.
+     */
+    function openArtifactFile(ctx, sessionId, token) {
+      try {
+        const sidebar = ctx && ctx.sidebarRight;
+        if (sidebar && typeof sidebar.openResource === 'function') {
+          sidebar.openResource(artifactResourceUrl(sessionId, token));
+          return true;
+        }
+      } catch {
+        /* fall through to clipboard copy */
+      }
+      try {
+        if (navigator.clipboard && navigator.clipboard.writeText) {
+          navigator.clipboard.writeText(token);
+        } else {
+          const ta = document.createElement('textarea');
+          ta.value = token;
+          ta.style.position = 'fixed';
+          ta.style.opacity = '0';
+          document.body.appendChild(ta);
+          ta.select();
+          document.execCommand('copy');
+          document.body.removeChild(ta);
+        }
+      } catch {
+        /* nothing more we can do */
+      }
+      return false;
+    }
+
     function KanbanDetails(props) {
-      const { task, t, active, busy, onActivate, onDeactivate, tracks } = props;
+      const { task, t, active, busy, onActivate, onDeactivate, tracks, onOpenArtifact } = props;
       // One-shot injection notice: 'ok' | 'fail' | null, auto-clears.
       const [notice, setNotice] = react.useState(null);
       react.useEffect(() => {
@@ -1276,7 +1350,11 @@ window.__ModuleLoader__.load({
                     textAlign: 'center',
                   },
                 },
-                notice === 'ok' ? t('sendToChatSuccess') : t('sendToChatFailed'),
+                notice === 'ok'
+                  ? t('sendToChatSuccess')
+                  : notice === 'open-fail'
+                    ? t('openFailed')
+                    : t('sendToChatFailed'),
               )
             : null,
         ),
@@ -1336,7 +1414,13 @@ window.__ModuleLoader__.load({
                       'button',
                       {
                         type: 'button',
-                        onClick: () => push(artifactToken(task, name)),
+                        onClick: () => {
+                          const token = artifactToken(task, name);
+                          const opened = onOpenArtifact ? onOpenArtifact(token) : false;
+                          // Notice only on failure: a successful native open is
+                          // visible in the right sidebar already, so no toast.
+                          setNotice(opened ? null : 'open-fail');
+                        },
                         title: t('open') + ' ' + artifactToken(task, name),
                         style: {
                           display: 'inline-flex',
@@ -1814,6 +1898,7 @@ window.__ModuleLoader__.load({
       const {
         board,
         t,
+        sessionId,
         selected,
         onSelect,
         busy,
@@ -1823,6 +1908,7 @@ window.__ModuleLoader__.load({
         onFilterChange,
         onClose,
         initialViewMode,
+        onOpenArtifact,
       } = props;
       const [query, setQuery] = react.useState('');
       const [viewMode, setViewMode] = react.useState(initialViewMode === 'list' ? 'list' : 'lanes'); // 'lanes' | 'list'
@@ -2424,6 +2510,7 @@ window.__ModuleLoader__.load({
               onActivate: () => onActivate(selectedTask.slug),
               onDeactivate,
               tracks,
+              onOpenArtifact: (token) => openArtifactFile(clientCtx, sessionId, token),
             }),
           ),
         ),
@@ -2812,6 +2899,7 @@ window.__ModuleLoader__.load({
         ? react.createElement(KanbanExpandedModal, {
             board,
             t,
+            sessionId,
             selected,
             onSelect: setSelected,
             busy,
@@ -2820,6 +2908,7 @@ window.__ModuleLoader__.load({
             filter,
             onFilterChange: setFilter,
             onClose: closeExpanded,
+            onOpenArtifact: (token) => openArtifactFile(clientCtx, sessionId, token),
           })
         : null;
 
@@ -2877,6 +2966,7 @@ window.__ModuleLoader__.load({
     // #endregion
 
     function apply(ctx) {
+      clientCtx = ctx;
       ctx.effect(() => ctx.locale.register(LOCALE_NS, { zh, en }), 'trellis-workflow: dictionaries');
       const t = ctx.locale.bind(LOCALE_NS);
       // Bound once on this plugin's fiber: the scope's disposer follows the fiber.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@banana-peeljj12/dsh-trellis",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Self-contained Trellis workflow trigger for DeepSeek Harness: per-turn breadcrumb injection via agent/pre-step, a bundled skill provider, and DSH-native task/phase state resolution. No Trellis AGPL code is vendored; workflow semantics are rewritten and licensed MIT.",
   "type": "module",
   "main": "lib/index.js",