dsh-config-manager 0.1.0 → 0.1.2

package/lib/ui/flow.d.ts

@@ -0,0 +1,17 @@
+/**
+ * 导入向导的 UI 流程阶段（覆盖在 wizard.step === 'preview' 之上）与导航。
+ *
+ * 注意：hasConflicts / hasPathIssues / hasSecrets 来自 Dry Run 的原始
+ * analysis/plan，流程中不会因用户已解决而重算——因此导航必须是「只前进」
+ * （按适用阶段有序列表推进），绝不能靠“当前阶段 != X 且 hasX”判定：
+ * 那会让已经过的阶段被重新命中，点「下一步」反而跳回上一步。
+ */
+/** 中间流程阶段（wizard.step 之外的 UI 层页面） */
+export type FlowPhase = 'preview' | 'conflicts' | 'path-mapping' | 'secrets' | 'confirm';
+/**
+ * 计算下一阶段：在 `list`（适用阶段的有序列表）中取 `from` 的下一项。
+ * - `from` 不在列表中（如来自 preview 页）→ 取第一项；
+ * - `from` 已是最后一项（confirm）→ 原地返回；
+ * - 绝不回退：已完成阶段不会因仍适用而被再次命中。
+ */
+export declare function nextFlowPhase(list: FlowPhase[], from: FlowPhase): FlowPhase;

package/lib/ui/flow.js

@@ -0,0 +1,22 @@
+/**
+ * 导入向导的 UI 流程阶段（覆盖在 wizard.step === 'preview' 之上）与导航。
+ *
+ * 注意：hasConflicts / hasPathIssues / hasSecrets 来自 Dry Run 的原始
+ * analysis/plan，流程中不会因用户已解决而重算——因此导航必须是「只前进」
+ * （按适用阶段有序列表推进），绝不能靠“当前阶段 != X 且 hasX”判定：
+ * 那会让已经过的阶段被重新命中，点「下一步」反而跳回上一步。
+ */
+/**
+ * 计算下一阶段：在 `list`（适用阶段的有序列表）中取 `from` 的下一项。
+ * - `from` 不在列表中（如来自 preview 页）→ 取第一项；
+ * - `from` 已是最后一项（confirm）→ 原地返回；
+ * - 绝不回退：已完成阶段不会因仍适用而被再次命中。
+ */
+export function nextFlowPhase(list, from) {
+    if (list.length === 0)
+        return 'confirm';
+    const idx = list.indexOf(from);
+    const next = list[Math.min(idx + 1, list.length - 1)];
+    return next ?? 'confirm';
+}
+//# sourceMappingURL=flow.js.map

package/lib/ui/flow.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"flow.js","sourceRoot":"","sources":["../../src/ui/flow.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAKH;;;;;GAKG;AACH,MAAM,UAAU,aAAa,CAAC,IAAiB,EAAE,IAAe;IAC9D,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,SAAS,CAAA;IACvC,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAA;IAC9B,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,GAAG,GAAG,CAAC,EAAE,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAA;IACrD,OAAO,IAAI,IAAI,SAAS,CAAA;AAC1B,CAAC"}

package/package.json

@@ -1,8 +1,12 @@
 {
   "name": "dsh-config-manager",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "DSH Config Manager — backup / export / import / migrate DSH configuration (dual-face Cordis plugin: host engine + web UI). Manifest shape mirrors @linxin666/dsh-ssh@0.1.12.",
   "type": "module",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/xiajiajun516/dsh-config-manager.git"
+  },
   "engines": {
     "node": "^22.19.0 || >=24.0.0"
   },

package/src/client/import/ImportWizardView.tsx

@@ -21,6 +21,7 @@ import { useRef, useState } from 'react'
 import type { ChangeEvent } from 'react'
 import { ImportWizard } from '../../ui/import-wizard.ts'
 import { ConflictCollector } from '../../ui/conflict-view.ts'
+import { nextFlowPhase, type FlowPhase } from '../../ui/flow.ts'
 import type { ImportPreviewSummary, ProgressEvent, WizardSnapshot } from '../../ui/types.ts'
 import type { PathMapping } from '../../core/types.ts'
 import type { ConfigManagerApi, UploadResponse } from '../api.ts'
@@ -38,8 +39,7 @@ export interface ImportWizardViewProps {
   t: TranslateNS<'config-manager'>
 }
 
-/** 中间流程阶段（wizard.step 之外的 UI 层页面） */
-type FlowPhase = 'preview' | 'conflicts' | 'path-mapping' | 'secrets' | 'confirm'
+/** 中间流程阶段（wizard.step 之外的 UI 层页面）——定义见 src/ui/flow.ts */
 
 const SCORE_LABEL: Record<string, string> = {
   excellent: 'Excellent',
@@ -118,14 +118,24 @@ export function ImportWizardView({ api, t }: ImportWizardViewProps) {
   const hasPathIssues = (snapshot.analysis?.pathIssues.length ?? 0) > 0
   const hasSecrets = (snapshot.plan?.missingSecrets.length ?? 0) > 0
 
-  /** 从某阶段完成后进入的下一个阶段（跳过空阶段；from 之前的阶段不再进入） */
-  const nextPhase = (from: FlowPhase): FlowPhase => {
-    if (from !== 'conflicts' && hasConflicts) return 'conflicts'
-    if (from !== 'path-mapping' && hasPathIssues) return 'path-mapping'
-    if (from !== 'secrets' && hasSecrets) return 'secrets'
-    return 'confirm'
+  /**
+   * 适用阶段的有序列表（仅含需要用户处理 + 确认页）。
+   * hasConflicts/hasPathIssues/hasSecrets 基于原始 analysis/plan（Dry Run 产物），
+   * 在流程中不会因已解决而重算——所以导航必须只前进（见 nextFlowPhase），
+   * 而不是靠"当前阶段 != X"判定（那会让已完成阶段被重新命中、跳回上一步）。
+   */
+  const applicablePhases = (): FlowPhase[] => {
+    const list: FlowPhase[] = []
+    if (hasConflicts) list.push('conflicts')
+    if (hasPathIssues) list.push('path-mapping')
+    if (hasSecrets) list.push('secrets')
+    list.push('confirm')
+    return list
   }
 
+  /** 从某阶段完成后进入的下一个阶段：只前进（from 不在列表时取第一项） */
+  const nextPhase = (from: FlowPhase): FlowPhase => nextFlowPhase(applicablePhases(), from)
+
   /* ---------- 动作 ---------- */
 
   /** 选择并上传 ZIP → wizard.selectZip（analyzing → compatibility） */

package/src/core/smoke.test.ts

@@ -37,12 +37,15 @@ import type {
 
 class MemFs implements FileSystemFacade {
   files = new Map<string, Uint8Array>();
-  private readonly homeDir: string;
+  private readonly homeDir: string; // 规范化后的 home（纯字符串前缀，不依赖宿主 path 语义）
   constructor(homeDir: string) {
-    this.homeDir = homeDir;
+    this.homeDir = normalizePath(homeDir);
   }
+  /** 内存 fs 的 key：与宿主平台解耦——避免 POSIX 上 path.resolve('C:\\...', p) 注入 cwd 前缀，
+   * 也避免 win32 上 path.resolve('/home/...', p) 补盘符，导致 listRecursive 切片错位。 */
   private key(p: string): string {
-    return normalizePath(path.resolve(this.homeDir, p));
+    const rel = normalizePath(p).replace(/^\/+/, '');
+    return rel === '' ? this.homeDir : `${this.homeDir}/${rel}`;
   }
   async readFile(relPath: string): Promise<Uint8Array> {
     const v = this.files.get(this.key(relPath));
@@ -64,10 +67,10 @@ class MemFs implements FileSystemFacade {
     this.files.delete(this.key(relPath));
   }
   async listRecursive(dir: string): Promise<string[]> {
-    const prefix = normalizePath(path.resolve(this.homeDir, dir)) + '/';
+    const prefix = this.key(dir) + '/';
     const out: string[] = [];
     for (const k of this.files.keys()) {
-      if (k.startsWith(prefix)) out.push(k.slice(this.homeDir.length).replace(/^[\\/]+/, ''));
+      if (k.startsWith(prefix)) out.push(k.slice(this.homeDir.length + 1)); // 去掉 home + '/'
     }
     return out.sort();
   }

package/src/ui/flow.test.ts

@@ -0,0 +1,37 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import { nextFlowPhase, type FlowPhase } from './flow.ts'
+
+const ALL: FlowPhase[] = ['conflicts', 'path-mapping', 'secrets', 'confirm']
+
+test('来自 preview（不在列表中）→ 取第一项', () => {
+  assert.equal(nextFlowPhase(['conflicts', 'path-mapping', 'confirm'], 'preview'), 'conflicts')
+  assert.equal(nextFlowPhase(['secrets', 'confirm'], 'preview'), 'secrets')
+  assert.equal(nextFlowPhase(['confirm'], 'preview'), 'confirm')
+})
+
+test('顺序前进：conflicts → path-mapping → secrets → confirm', () => {
+  assert.equal(nextFlowPhase(ALL, 'conflicts'), 'path-mapping')
+  assert.equal(nextFlowPhase(ALL, 'path-mapping'), 'secrets')
+  assert.equal(nextFlowPhase(ALL, 'secrets'), 'confirm')
+})
+
+test('回归：path-mapping 完成后即使 conflicts 仍适用（hasX 不变）也只前进，不回跳', () => {
+  // 模拟“既有冲突又有路径问题”：list 含 conflicts，但从 path-mapping 出发必须到 secrets/confirm
+  const list: FlowPhase[] = ['conflicts', 'path-mapping', 'confirm']
+  assert.equal(nextFlowPhase(list, 'path-mapping'), 'confirm')
+  assert.equal(nextFlowPhase(list, 'conflicts'), 'path-mapping')
+})
+
+test('跳过不适用的阶段', () => {
+  assert.equal(nextFlowPhase(['path-mapping', 'confirm'], 'preview'), 'path-mapping')
+  assert.equal(nextFlowPhase(['confirm'], 'conflicts'), 'confirm')
+})
+
+test('最后一项原地返回（confirm 无下一步）', () => {
+  assert.equal(nextFlowPhase(ALL, 'confirm'), 'confirm')
+})
+
+test('空列表兜底返回 confirm', () => {
+  assert.equal(nextFlowPhase([], 'preview'), 'confirm')
+})

package/src/ui/flow.ts

@@ -0,0 +1,24 @@
+/**
+ * 导入向导的 UI 流程阶段（覆盖在 wizard.step === 'preview' 之上）与导航。
+ *
+ * 注意：hasConflicts / hasPathIssues / hasSecrets 来自 Dry Run 的原始
+ * analysis/plan，流程中不会因用户已解决而重算——因此导航必须是「只前进」
+ * （按适用阶段有序列表推进），绝不能靠“当前阶段 != X 且 hasX”判定：
+ * 那会让已经过的阶段被重新命中，点「下一步」反而跳回上一步。
+ */
+
+/** 中间流程阶段（wizard.step 之外的 UI 层页面） */
+export type FlowPhase = 'preview' | 'conflicts' | 'path-mapping' | 'secrets' | 'confirm'
+
+/**
+ * 计算下一阶段：在 `list`（适用阶段的有序列表）中取 `from` 的下一项。
+ * - `from` 不在列表中（如来自 preview 页）→ 取第一项；
+ * - `from` 已是最后一项（confirm）→ 原地返回；
+ * - 绝不回退：已完成阶段不会因仍适用而被再次命中。
+ */
+export function nextFlowPhase(list: FlowPhase[], from: FlowPhase): FlowPhase {
+  if (list.length === 0) return 'confirm'
+  const idx = list.indexOf(from)
+  const next = list[Math.min(idx + 1, list.length - 1)]
+  return next ?? 'confirm'
+}