@dungsil/dsh-plugin-ui-model-picker 0.2.3 → 0.2.4

package/README.md

@@ -2,7 +2,7 @@
 
 ![모델 선택 팝오버의 라이트 모드와 다크 모드를 꼭짓점 대각선으로 나눈 미리보기](./assets/light-dark-preview.webp)
 
-DSH Web 입력창에서 모델과 추론 강도를 선택합니다. 하나의 선택 버튼을 누르면 추론 강도 팝오버가 먼저 열리며, 팝오버 안의 모델명을 눌러 모델 목록으로 이동합니다.
+DSH Web 입력창에서 모델과 추론 강도를 선택합니다. 하나의 선택 버튼을 누르면 추론 강도 팝오버가 먼저 열리며, 팝오버 안의 모델명을 눌러 모델 목록으로 이동합니다. 모델을 고르면 실제 모델은 바로 바뀌지 않고 추론 강도 슬라이더가 이어서 표시되며, 강도를 설정할 때 모델이 함께 적용됩니다.
 
 위 이미지는 플러그인 컴포넌트에 예시 모델 목록과 테마 색상을 제공한 검증 화면을 각각 캡처해 합성했습니다. 표시된 모델명과 추론 단계는 예시이며, 실제 목록은 연결한 제공자가 결정합니다.
 
@@ -10,14 +10,14 @@ DSH Web 입력창에서 모델과 추론 강도를 선택합니다. 하나의
 
 1. DSH 플러그인 설치 화면에서 `@dungsil/dsh-plugin-ui-model-picker`를 설치하고 활성화합니다.
 2. 입력창 오른쪽 선택 버튼을 눌러 추론 강도 슬라이더를 엽니다. 방향키와 Home/End 키도 사용할 수 있습니다.
-3. 모델을 바꾸려면 팝오버 상단의 모델명 부분을 누르고 제공자별 모델을 선택합니다. 오른쪽 위 재설정 아이콘은 기본 추론 강도를 복원합니다.
+3. 모델을 바꾸려면 팝오버 상단의 모델명 부분을 누르고 제공자별 모델을 선택합니다. 추론 강도를 지원하는 모델을 고르면 슬라이더가 이어서 나타나며, 강도를 설정해야 모델이 적용됩니다. 오른쪽 위 재설정 아이콘은 기본 추론 강도를 복원합니다.
 
 ## 동작과 호환성
 
 - DSH `0.2.1-rc.5` 이상 `0.3.0` 미만의 Web UI를 지원합니다. 기본 `ui-model-selection` 기능이 활성화되어 있어야 합니다.
 - 공용 모델 디렉터리와 선택 API를 그대로 사용하므로 `/model` 명령과 선택 상태를 공유합니다. 별도 모델 설정을 저장하거나 API 키를 다루지 않습니다.
-- 실제 제공자 목록과 모델별 추론 메타데이터만 사용합니다. 별도 모델 목록, Fast 모드, 고정 추론 단계는 추가하지 않습니다. 추론을 지원하지 않는 모델에는 슬라이더를 표시하지 않으며, 팝오버 안에서 모델은 계속 바꿀 수 있습니다.
-- 다른 모델로 바꾸면 새 모델의 기본 추론 강도를 사용합니다. 기본값 재설정은 별도 강도 지정을 해제합니다.
+- 실제 제공자 목록과 모델별 추론 메타데이터만 사용합니다. 별도 모델 목록, Fast 모드, 고정 추론 단계는 추가하지 않습니다. 추론을 지원하지 않는 모델에는 슬라이더를 표시하지 않으며, 그런 모델은 목록에서 고르는 즉시 적용됩니다.
+- 다른 모델로 바꾸면 그 모델이 지원하는 추론 강도에서 슬라이더가 시작되고, 강도를 설정해야 모델 선택이 적용됩니다. 슬라이더를 건드리지 않고 팝오버를 닫으면 모델은 바뀌지 않습니다. 기본값 재설정은 별도 강도 지정을 해제합니다.
 - Escape, 바깥 클릭, 스크롤, 창 크기 변경으로 메뉴를 닫습니다. 메뉴의 방향키와 Tab 탐색을 지원합니다. 한국어·영어·중국어(간체)와 DSH 테마 색상을 따릅니다.
 - 기본 모델 슬롯만 우선순위로 대체합니다. 제거하면 기본 선택 UI가 돌아오며 모델 선택은 유지됩니다. 같은 슬롯을 대체하는 다른 플러그인과는 함께 사용하지 마세요.
 

package/dist/client.js

@@ -108,6 +108,7 @@ const styles = `${composerStyles}
 .dsh-codex-slider input::-moz-range-track { height: 28px; background: transparent; }
 .dsh-codex-slider input::-moz-range-thumb { width: 26px; height: 26px; border: 1px solid #0000000a; border-radius: 50%; background: #fff; box-shadow: 0 1px 3px #0002; }
 .dsh-codex-notice { padding: 6px 10px; margin: 0; font-size: 12px; overflow-wrap: anywhere; }
+.dsh-codex-status { position: absolute; width: 1px; height: 1px; padding: 0; margin: 0; overflow: hidden; clip-path: inset(50%); white-space: nowrap; border: 0; }
 .dsh-codex-popover [role=alert] { color: var(--dsw-alias-state-error-primary, #b54424); }
 .dsh-codex-popover :focus-visible { outline: 2px solid var(--dsw-focus-ring-color, var(--dsw-alias-state-business-primary)); outline-offset: -2px; }
 .dsh-codex-popover :disabled { color: var(--dsw-alias-label-dimmed); cursor: default; }
@@ -127,7 +128,6 @@ const copy = {
 		model: "모델 선택",
 		effort: "추론 강도",
 		choose: "추론 수준 선택",
-		default: "제공자 기본값",
 		reset: "기본값으로 재설정",
 		loading: "모델을 불러오는 중입니다…",
 		empty: "사용할 수 있는 모델이 없습니다.",
@@ -141,7 +141,6 @@ const copy = {
 		model: "Select model",
 		effort: "Reasoning effort",
 		choose: "Select reasoning effort",
-		default: "Provider default",
 		reset: "Reset to default",
 		loading: "Loading models…",
 		empty: "No models available.",
@@ -155,7 +154,6 @@ const copy = {
 		model: "选择模型",
 		effort: "推理强度",
 		choose: "选择推理级别",
-		default: "提供商默认值",
 		reset: "重置为默认值",
 		loading: "正在加载模型…",
 		empty: "没有可用的模型。",
@@ -174,25 +172,28 @@ function apply(ctx) {
 		const { active: locale } = useLocaleSnapshot(ctx.locale);
 		const text = copy[locale.startsWith("ko") ? "ko" : locale.startsWith("zh") ? "zh" : "en"];
 		const [requestedPanel, setPanel] = (0, react.useState)(null);
+		const [staged, setStaged] = (0, react.useState)(null);
 		const [draft, setDraft] = (0, react.useState)(0);
 		const [failed, setFailed] = (0, react.useState)(false);
 		const trigger = (0, react.useRef)(null);
 		const anchor = (0, react.useRef)(null);
 		const popup = (0, react.useRef)(null);
 		const pending = (0, react.useRef)(false);
+		const sliderSelection = (0, react.useRef)(null);
 		const mounted = (0, react.useRef)(true);
 		const id = (0, react.useId)();
 		const current = state.current;
 		const model = state.groups.find((item) => item.id === current?.provider)?.models.find((item) => item.id === current?.model);
 		const reasoning = model?.reasoning;
-		const efforts = reasoning?.efforts.length ? [...reasoning.defaultEffort === void 0 ? [{
-			id: void 0,
-			name: text.default
-		}] : [], ...reasoning.efforts] : [];
-		const panel = requestedPanel === "effort" && !efforts.length ? "model" : requestedPanel;
+		const efforts = reasoning?.efforts ?? [];
+		const stagedGroup = staged ? state.groups.find((item) => item.id === staged.provider) : void 0;
+		const stagedModel = staged ? stagedGroup?.models.find((item) => item.id === staged.model) : model;
+		const stagedEfforts = (stagedModel?.reasoning)?.efforts ?? [];
+		const panel = requestedPanel === "effort" && !stagedEfforts.length ? "model" : requestedPanel;
 		const effective = current?.reasoningEffort ?? reasoning?.defaultEffort;
 		const effortIndex = efforts.findIndex((item) => item.id === effective);
 		const effortLabel = efforts.length ? efforts[effortIndex]?.name ?? state.retainedEffort : void 0;
+		const stagedName = stagedModel?.name ?? (staged ? staged.model : current?.model) ?? text.model;
 		const busy = state.status === "selecting";
 		const disabled = locked || busy;
 		(0, react.useEffect)(() => {
@@ -203,10 +204,16 @@ function apply(ctx) {
 			};
 		}, [directory]);
 		const close = () => {
+			sliderSelection.current = null;
 			setPanel(null);
+			setStaged(null);
 			if (trigger.current) (0, _deepseek_ai_dsh_client_ui_primitives.focusWithoutRing)(trigger.current);
 		};
-		(0, _deepseek_ai_dsh_client_ui_primitives.useDismissOnOutsidePointer)(anchor, panel !== null, () => setPanel(null), popup);
+		(0, _deepseek_ai_dsh_client_ui_primitives.useDismissOnOutsidePointer)(anchor, panel !== null, () => {
+			sliderSelection.current = null;
+			setPanel(null);
+			setStaged(null);
+		}, popup);
 		(0, react.useLayoutEffect)(() => {
 			if (!panel || !popup.current || !anchor.current) return;
 			const element = popup.current;
@@ -252,25 +259,42 @@ function apply(ctx) {
 			};
 		}, [panel]);
 		(0, react.useEffect)(() => {
-			if (locked) setPanel(null);
+			if (locked) {
+				sliderSelection.current = null;
+				setPanel(null);
+				setStaged(null);
+			}
 		}, [locked]);
 		(0, react.useEffect)(() => {
+			const selection = sliderSelection.current;
+			sliderSelection.current = null;
+			if (selection && current?.provider === selection.provider && current?.model === selection.model && current?.reasoningEffort === selection.reasoningEffort) {
+				setStaged(null);
+				return;
+			}
 			setPanel(null);
+			setStaged(null);
 		}, [
 			current?.provider,
 			current?.model,
 			current?.reasoningEffort
 		]);
-		const select = async (selection) => {
+		const select = async (selection, keepSlider = false) => {
 			if (disabled || pending.current) return;
 			pending.current = true;
+			sliderSelection.current = keepSlider ? selection : null;
 			setFailed(false);
 			try {
 				const result = await directory.select(selection);
 				if (!mounted.current) return;
-				if (result.ok) close();
-				else setFailed(true);
+				if (result.ok) {
+					if (!keepSlider) close();
+				} else {
+					sliderSelection.current = null;
+					setFailed(true);
+				}
 			} catch {
+				sliderSelection.current = null;
 				if (mounted.current) setFailed(true);
 			} finally {
 				pending.current = false;
@@ -282,19 +306,50 @@ function apply(ctx) {
 				close();
 				return;
 			}
+			setStaged(null);
 			setDraft(Math.max(0, effortIndex));
 			setFailed(false);
 			setPanel("effort");
 			directory.load().catch(() => {});
 		};
 		const selectEffort = (index) => {
-			if (!current || !efforts[index]) return;
-			const effort = efforts[index].id;
-			select({
+			const target = staged ?? (current ? {
 				provider: current.provider,
-				model: current.model,
-				...effort === void 0 ? {} : { reasoningEffort: effort }
+				model: current.model
+			} : null);
+			const effort = stagedEfforts[index];
+			if (!target || !effort) return;
+			select({
+				provider: target.provider,
+				model: target.model,
+				reasoningEffort: effort.id
+			}, true);
+		};
+		const stageModel = (provider, option) => {
+			const same = provider === current?.provider && option.id === current?.model;
+			const optionReasoning = option.reasoning;
+			const optionEfforts = optionReasoning?.efforts ?? [];
+			if (optionEfforts.length < 2) {
+				if (same) {
+					close();
+					return;
+				}
+				const effort = optionReasoning?.defaultEffort;
+				select({
+					provider,
+					model: option.id,
+					...effort === void 0 ? {} : { reasoningEffort: effort }
+				});
+				return;
+			}
+			const preferredEffort = same ? current?.reasoningEffort ?? optionReasoning?.defaultEffort : optionReasoning?.defaultEffort ?? current?.reasoningEffort;
+			setStaged({
+				provider,
+				model: option.id
 			});
+			setDraft(Math.max(0, optionEfforts.findIndex((item) => item.id === preferredEffort)));
+			setFailed(false);
+			setPanel("effort");
 		};
 		return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)(react_jsx_runtime.Fragment, { children: [/* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 			className: "dsh-codex-picker",
@@ -305,13 +360,16 @@ function apply(ctx) {
 				icon: /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_deepseek_ai_dsh_client_ui_primitives.IconDataOutlineRegular, { size: 16 }),
 				label: model?.name ?? current?.model ?? text.model,
 				detail: effortLabel,
-				disabled,
+				disabled: locked,
+				"aria-disabled": disabled,
 				"aria-label": `${efforts.length ? text.choose : text.model}: ${model?.name ?? current?.model ?? text.model}${effortLabel ? `, ${effortLabel}` : ""}`,
 				title: `${model?.name ?? current?.model ?? text.model}${effortLabel ? ` ${effortLabel}` : ""}`,
 				"aria-haspopup": "dialog",
 				"aria-expanded": panel !== null,
 				"aria-controls": panel ? id : void 0,
-				onClick: (event) => show(event.currentTarget)
+				onClick: (event) => {
+					if (!disabled) show(event.currentTarget);
+				}
 			})
 		}), panel && (0, react_dom.createPortal)(/* @__PURE__ */ (0, react_jsx_runtime.jsxs)(_deepseek_ai_dsh_client_ui_primitives.MenuSurface, {
 			id,
@@ -320,7 +378,7 @@ function apply(ctx) {
 			"aria-label": panel === "model" ? text.model : text.effort,
 			className: "dsh-codex-popover",
 			children: [
-				panel === "model" ? efforts.length > 0 && /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("button", {
+				panel === "model" ? stagedEfforts.length > 0 && /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("button", {
 					type: "button",
 					className: "dsh-codex-back",
 					onClick: () => setPanel("effort"),
@@ -334,25 +392,30 @@ function apply(ctx) {
 					children: [/* @__PURE__ */ (0, react_jsx_runtime.jsxs)("button", {
 						type: "button",
 						className: "dsh-codex-model",
-						disabled,
-						"aria-label": `${text.model}: ${model?.name ?? current?.model ?? text.model}`,
-						onClick: () => setPanel("model"),
-						children: [/* @__PURE__ */ (0, react_jsx_runtime.jsx)("strong", { children: efforts[draft]?.name ?? effortLabel ?? text.default }), /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("span", { children: [
-							model?.name ?? current?.model ?? text.model,
+						disabled: locked,
+						"aria-disabled": disabled,
+						"aria-label": `${text.model}: ${stagedName}`,
+						onClick: () => {
+							if (!disabled) setPanel("model");
+						},
+						children: [/* @__PURE__ */ (0, react_jsx_runtime.jsx)("strong", { children: stagedEfforts[draft]?.name }), /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("span", { children: [
+							stagedName,
 							" ",
 							/* @__PURE__ */ (0, react_jsx_runtime.jsx)(_deepseek_ai_dsh_client_ui_primitives.IconChevronRightOutlineRegular, { size: 14 })
 						] })]
 					}), /* @__PURE__ */ (0, react_jsx_runtime.jsx)("button", {
 						type: "button",
 						className: "dsh-codex-reset",
-						disabled: disabled || !current,
+						disabled: locked || !(staged ?? current),
+						"aria-disabled": disabled || !(staged ?? current),
 						"aria-label": text.reset,
 						title: text.reset,
 						onClick: () => {
-							if (current) select({
+							const target = staged ?? (current ? {
 								provider: current.provider,
 								model: current.model
-							});
+							} : null);
+							if (target) select(target);
 						},
 						children: /* @__PURE__ */ (0, react_jsx_runtime.jsx)(IconResetOutlineRegular, {})
 					})]
@@ -363,7 +426,7 @@ function apply(ctx) {
 					children: text.loading
 				}),
 				busy && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("p", {
-					className: "dsh-codex-notice",
+					className: panel === "effort" ? "dsh-codex-status" : "dsh-codex-notice",
 					role: "status",
 					children: text.busy
 				}),
@@ -397,14 +460,7 @@ function apply(ctx) {
 								"aria-checked": provider.id === current?.provider && option.id === current?.model,
 								className: "dsh-codex-option",
 								disabled,
-								onClick: () => {
-									const effort = provider.id === current?.provider && option.id === current?.model ? current.reasoningEffort : option.reasoning?.defaultEffort;
-									select({
-										provider: provider.id,
-										model: option.id,
-										...effort === void 0 ? {} : { reasoningEffort: effort }
-									});
-								},
+								onClick: () => stageModel(provider.id, option),
 								children: [/* @__PURE__ */ (0, react_jsx_runtime.jsxs)("span", { children: [option.name, option.description && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("small", { children: option.description })] }), /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
 									className: "dsh-codex-check",
 									"aria-hidden": "true",
@@ -428,26 +484,29 @@ function apply(ctx) {
 						},
 						children: text.retry
 					})
-				] }) : efforts.length > 0 && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
+				] }) : stagedEfforts.length > 0 && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 					className: "dsh-codex-slider",
 					children: /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
 						className: "dsh-codex-rail",
-						style: { "--dsh-codex-progress": `${efforts.length > 1 ? draft / (efforts.length - 1) * 100 : 0}%` },
+						style: { "--dsh-codex-progress": `${stagedEfforts.length > 1 ? draft / (stagedEfforts.length - 1) * 100 : 0}%` },
 						children: [/* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 							className: "dsh-codex-track",
 							"aria-hidden": "true",
-							children: efforts.map((effort, index) => /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", { "data-filled": index <= draft }, effort.id ?? "default"))
+							children: stagedEfforts.map((effort, index) => /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", { "data-filled": index <= draft }, effort.id))
 						}), /* @__PURE__ */ (0, react_jsx_runtime.jsx)("input", {
 							id: `${id}-effort`,
 							type: "range",
 							min: 0,
-							max: efforts.length - 1,
+							max: stagedEfforts.length - 1,
 							step: 1,
 							value: draft,
-							disabled: disabled || efforts.length < 2,
+							disabled: locked || stagedEfforts.length < 2,
+							"aria-disabled": disabled || stagedEfforts.length < 2,
 							"aria-label": text.effort,
-							"aria-valuetext": efforts[draft]?.name,
-							onChange: (event) => setDraft(Number(event.target.value)),
+							"aria-valuetext": stagedEfforts[draft]?.name,
+							onChange: (event) => {
+								if (!disabled) setDraft(Number(event.target.value));
+							},
 							onPointerUp: (event) => selectEffort(Number(event.currentTarget.value)),
 							onKeyUp: (event) => {
 								if ([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dungsil/dsh-plugin-ui-model-picker",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "Model and reasoning effort picker for the DeepSeek Harness composer.",
   "license": "MIT",
   "repository": {
@@ -40,6 +40,7 @@
     "@dungsil/dsh-ui": "0.0.0",
     "@types/react": "18.3.31",
     "@types/react-dom": "18.3.7",
+    "happy-dom": "21.0.4",
     "react": "18.3.1",
     "react-dom": "18.3.1",
     "tsdown": "0.23.4"
@@ -88,6 +89,7 @@
   },
   "scripts": {
     "build": "tsdown",
-    "typecheck": "tsc --noEmit"
+    "typecheck": "tsc --noEmit",
+    "test": "node --test test/*.test.mjs"
   }
 }