dsh-prompt-rewriter 1.6.1 → 1.6.3

package/CHANGELOG.md

@@ -2,6 +2,28 @@
 
 记录每个**已发布**版本对使用者可见的变化。中英双语并列，中文在前。
 
+## 1.6.3
+
+**中文**
+
+- 快捷键服务不可用时插件不再整体失效。此前宿主的快捷键服务若没有注册（例如与另一个锁定旧版客户端的插件共存），整个插件会保持休眠，输入框旁的按钮也随之消失；现在只有快捷键降级为不可用，改写按钮照常工作。
+- 快捷键服务晚于插件注册时也能读到当前绑定，不再把它显示成「未设置」或「不可用」。
+
+**English**
+
+- The plugin no longer goes dormant when the host's shortcut service is missing. Previously, if that service never registered — for example when another plugin pinned the client runtime to an older range — the whole plugin stayed inactive and the composer button disappeared with it. Only the shortcut degrades now; the Enhance button keeps working.
+- A shortcut service that registers after the plugin now resolves correctly, instead of being reported as unset or unavailable.
+## 1.6.2
+
+**中文**
+
+- 增强进行中的鱼尾动效改用品牌蓝。此前它跟随按钮的文字色，深色主题下接近白色，与旁边的鱼形图标不同色。
+- 输入框里的图片读取更可靠：宿主服务取不到时改从输入框自身的通道取图；两处都无法回答时不再静默跳过，文字改写照常完成，状态区会告诉你少读了几张。
+
+**English**
+
+- The tail animation shown while an enhancement runs now uses the brand blue. It previously followed the button's text colour, which is near white in the dark theme and did not match the fish icon beside it.
+- Composer images are read more reliably: when the host service cannot answer, the composer's own channel is used instead; when neither can, the images are no longer skipped silently — the text-only rewrite still completes and the status area reports how many were missed.
 ## 1.6.0
 
 **中文**
@@ -86,4 +108,4 @@
 
 **English**
 
-- Added the bilingual README and screenshots; fixed images and the Chinese guide missing from the npm package.
+- Added the bilingual README and screenshots; fixed images and the Chinese guide missing from the npm package.

package/README.md

@@ -43,7 +43,7 @@ An enhancement is a real model call, and **how fast the result comes back depend
 
 If it feels slow, switch the session to a model that is more stable and responds faster — the same draft can take a few seconds on one model and well over ten on another. The button does not favour any particular model; it uses whichever one you picked.
 
-> Want a fixed model just for enhancement, without affecting normal chat? Open `Settings → Prompt rewriter` and turn on **Use a fixed model**.
+> Want a fixed model just for enhancement, without affecting normal chat? Open `Settings → Prompt Enhance` and turn on **Use a fixed model**.
 
 ### You can navigate away mid-rewrite
 
@@ -55,7 +55,7 @@ That wasn't true in 1.0.x: switching sessions aborted the task, and coming back
 
 ### Keyboard shortcut (off by default)
 
-Turn it on at **Settings → Prompt rewriter → Shortcut**. Desktop uses `Ctrl+E` (`Cmd+E` on macOS); the browser uses `Ctrl+Alt+E` (`Cmd+Alt+E`).
+Turn it on at **Settings → Prompt Enhance → Shortcut**. Desktop uses `Ctrl+E` (`Cmd+E` on macOS); the browser uses `Ctrl+Alt+E` (`Cmd+Alt+E`).
 
 - Press once to enhance; press again while it runs to pause. It acts on the composer of the current conversation.
 - **Same as clicking the icon** — the shortcut runs the icon's own click handler, including its undo state, so the two paths cannot drift. Restoring a draft stays on the button or `Ctrl+Z`.
@@ -95,7 +95,7 @@ Only images **currently sitting in the composer** — **never images from earlie
 
 ## Settings
 
-`Settings → Prompt rewriter`. Three switches and one dropdown, affecting **only this button, never normal sending**.
+`Settings → Prompt Enhance`. Four switches and one dropdown, affecting **only this button, never normal sending**.
 
 ![Settings page](https://raw.githubusercontent.com/huaguoxingkong/dsh-prompt-rewriter/main/docs/images/03-settings-en.png)
 
@@ -104,6 +104,7 @@ Only images **currently sitting in the composer** — **never images from earlie
 | Use a fixed model | Off | Off: follow the model of the current conversation. On: always use the model picked below. |
 | Reference the conversation | **On** | Read earlier turns. More accurate, but extra tokens and slower on long chats. |
 | Reference composer images | Off | Images in the composer take part. Earlier messages' images are never read. |
+| Shortcut | Off | Lets the keyboard trigger a rewrite. Off by default, so no global key fires unasked. |
 
 - The model dropdown lists **the providers and models you actually have** — you can't type in one that doesn't exist.
 - Turning the model switch off does not clear your selection; flipping it back on shows the same choice.
@@ -137,7 +138,7 @@ Download the zip from [Releases](https://github.com/huaguoxingkong/dsh-prompt-re
 
 ## FAQ
 
-**Clicked it and nothing happened.** The composer is empty, or *Use a fixed model* is on with no model picked.
+**Clicked it and nothing happened.** The composer is empty, or *Use a fixed model* is on with no model picked. If you used the shortcut instead, check that the Shortcut switch is on and that no conflict is reported.
 
 **"No credentials configured for the current model."** That model has no key set. Switch models, or pin one that works in settings.
 
@@ -147,7 +148,7 @@ Download the zip from [Releases](https://github.com/huaguoxingkong/dsh-prompt-re
 
 ## Configuration
 
-Every `config` key on the profile plugin line is optional. The first five are managed by the settings page; the last seven are operational ceilings and are not exposed in the UI.
+Every `config` key on the profile plugin line is optional. The first six are managed by the settings page; the last seven are operational ceilings and are not exposed in the UI.
 
 ```yaml
 - id: dsh-prompt-rewriter
@@ -159,6 +160,7 @@ Every `config` key on the profile plugin line is optional. The first five are ma
     enhanceModel: ""
     referenceImages: false     # read composer images
     referenceContext: true     # read earlier turns
+    shortcutEnabled: false     # keyboard shortcut (off by default)
 
     # operational ceilings (not exposed in the UI)
     maxDraftChars: 8000        # draft length cap

package/README.zh.md

@@ -44,7 +44,7 @@
 
 如果觉得等得久，把会话用的模型换成一个更稳定、响应更快的——同一个草稿，换个模型往往就是几秒和十几秒的差别。这个按钮不挑模型，你选哪个它就用哪个。
 
-> 想固定用某个模型专门做增强（不影响正常对话）？去 `设置 → 提示词改写` 打开「用哪个模型改写」。
+> 想固定用某个模型专门做增强（不影响正常对话）？去 `设置 → 提示词增强` 打开「用哪个模型改写」。
 
 ### 增强过程中可以切走
 
@@ -56,7 +56,7 @@
 
 ### 键盘快捷键（默认关闭）
 
-到 **设置 → 提示词改写 → 快捷键** 打开开关。桌面端为 `Ctrl+E`（macOS `Cmd+E`），浏览器端为 `Ctrl+Alt+E`（macOS `Cmd+Alt+E`）。
+到 **设置 → 提示词增强 → 快捷键** 打开开关。桌面端为 `Ctrl+E`（macOS `Cmd+E`），浏览器端为 `Ctrl+Alt+E`（macOS `Cmd+Alt+E`）。
 
 - 按一次增强，运行中再按一次暂停。作用对象是当前会话的输入框。
 - **与点击图标完全等价**：走的是图标自身的点击入口，含「撤销」态，不存在第二套逻辑。回退仍由按钮或 `Ctrl+Z` 负责。
@@ -96,7 +96,7 @@
 
 ## 设置
 
-`设置 → 提示词改写`。三个开关一个下拉，**只影响这个按钮，不影响正常发送**。
+`设置 → 提示词增强`。四个开关一个下拉，**只影响这个按钮，不影响正常发送**。
 
 ![设置页](https://raw.githubusercontent.com/huaguoxingkong/dsh-prompt-rewriter/main/docs/images/03-settings.png)
 
@@ -105,6 +105,7 @@
 | 用哪个模型改写 | 关 | 关：跟随当前对话的模型。开：固定用下拉选定的模型。 |
 | 参考之前的对话 | **开** | 读前面的内容。更准，但多耗 token、长对话更慢。 |
 | 参考输入框里的图片 | 关 | 输入框里的图一起参与。历史消息里的图片始终不读。 |
+| 快捷键 | 关 | 打开后可用键盘触发改写。默认关闭，不会在你没要求时抢占按键。 |
 
 - 模型下拉列出你**实际拥有**的渠道与模型，填不进不存在的模型。
 - 关掉模型开关不会清空你的选择，重新打开还是同一个。
@@ -138,7 +139,7 @@ dsh plugin --profile web add github:huaguoxingkong/dsh-prompt-rewriter
 
 ## 常见问题
 
-**点了没反应？** 输入框是空的，或者「用哪个模型改写」打开了但没选模型。
+**点了没反应？** 输入框是空的，或者「用哪个模型改写」打开了但没选模型。如果用的是快捷键，先确认「快捷键」开关已打开、且没有提示冲突。
 
 **报「当前模型尚未配置可用的凭据」？** 这个模型没配 key。换个模型，或在设置里固定一个能用的。
 
@@ -148,7 +149,7 @@ dsh plugin --profile web add github:huaguoxingkong/dsh-prompt-rewriter
 
 ## 配置
 
-profile 插件行的 `config` 全部可选。前五项由设置页托管，后七项是运维上限，不暴露在界面上：
+profile 插件行的 `config` 全部可选。前六项由设置页托管，后七项是运维上限，不暴露在界面上：
 
 ```yaml
 - id: dsh-prompt-rewriter
@@ -160,6 +161,7 @@ profile 插件行的 `config` 全部可选。前五项由设置页托管，后
     enhanceModel: ""
     referenceImages: false     # 是否参考输入框图片
     referenceContext: true     # 是否参考前文
+    shortcutEnabled: false     # 键盘快捷键（默认关闭）
 
     # 运维上限（不暴露在设置页）
     maxDraftChars: 8000        # 草稿字符上限

package/client.js

@@ -75,7 +75,7 @@ window.__ModuleLoader__.load({
 			// Colour: the official brand blue, not `currentColor`. The tail is a graphic,
 			// not text, so inheriting the button's label colour made it render white on
 			// the dark composer — visually detached from the fish mark it replaces.
-			'.dsh-pe_twist{display:block;width:20px;height:16px;overflow:visible;color:inherit}',
+			'.dsh-pe_twist{display:block;width:20px;height:16px;overflow:visible;color:var(--dsw-alias-brand-primary-new-colorprimary-new-color,#4176e6)}',
 			'.dsh-pe_twist path{fill:currentColor;fill-rule:evenodd}',
 			'.dsh-pe_icon{display:block;width:15px;height:15px}',
 			// Context-mode mark: a dot on the seat that reports an EXPECTATION — "if you
@@ -460,6 +460,7 @@ window.__ModuleLoader__.load({
 		 */
 		let conversationResolver = () => undefined;
 
+
 		/**
 		 * Resolve the client `conversation` service AT CALL TIME.
 		 *
@@ -708,10 +709,6 @@ window.__ModuleLoader__.load({
 			// feature is opt-in, so a global key never fires on a user who did not ask for
 			// it. Read from the same settings response as the other switches.
 			const [shortcutOn, setShortcutOn] = useState(false);
-			// Exposed on the seat as data-ctx-debug so a missing dot can be diagnosed
-			// without guessing: it records which of the two facts was false, and whether
-			// either request failed. Not shown to users; harmless in the DOM.
-			const [ctxProbe, setCtxProbe] = useState('pending');
 			useEffect(() => {
 				let cancelled = false;
 				const read = async () => {
@@ -730,16 +727,11 @@ window.__ModuleLoader__.load({
 						const allowed = settings?.ok === true && settings.values?.shortcutEnabled === true;
 						setShortcutOn(allowed);
 						taskStore.setShortcutEnabled(allowed);
-							setCtxProbe(
-							`sw=${switchOn ? 1 : 0},hc=${hasContext ? 1 : 0},`
-							+ `sHttp=${settingsResponse.status},cHttp=${statusResponse.status}`,
-						);
 					} catch (error) {
 						// An unreachable route means "we cannot promise context"; showing no dot
 						// is the honest default rather than promising a context-aware run.
 						if (!cancelled) {
 							setWillUseContext(false);
-							setCtxProbe(`error:${error?.message ?? 'unknown'}`);
 						}
 					}
 				};
@@ -819,19 +811,52 @@ window.__ModuleLoader__.load({
 		 * @param signal - cancellation.
 		 * @returns `{ images, skipped }`.
 		 */
-		async function readComposerImages(conversation, ids, signal) {
-			const images = [];
-			let skipped = 0;
-			if (!Array.isArray(ids) || ids.length === 0) return { images, skipped };
-			if (conversation === undefined || typeof conversation.resolveDraftAttachments !== 'function') {
-				return { images, skipped: ids.length };
+		/**
+		 * Resolve the composer draft descriptors for these ids.
+		 *
+		 * Two owners can answer, and which one is reachable depends on the shell:
+		 *
+		 * 1. the `conversation` client service, which owns the draft map;
+		 * 2. the composer action face handed to the slot, which exposes the same
+		 *    lookup when the host chooses to publish it there.
+		 *
+		 * Trying both removes the single point of failure that made every image come
+		 * back unreadable when the first owner was absent.
+		 *
+		 * @param inputActions - the composer action face from slot props.
+		 * @param ids - draft attachment ids.
+		 * @returns `{ descriptors, source }` where source is the owner that answered.
+		 */
+		function resolveDraftDescriptors(inputActions, ids) {
+			const conversation = conversationResolver();
+			if (conversation !== undefined && typeof conversation.resolveDraftAttachments === 'function') {
+				try {
+					return { descriptors: conversation.resolveDraftAttachments(ids) ?? [], source: 'service' };
+				} catch {
+					// Fall through to the action face.
+				}
 			}
-			let descriptors;
-			try {
-				descriptors = conversation.resolveDraftAttachments(ids) ?? [];
-			} catch {
-				return { images, skipped: ids.length };
+			if (inputActions !== undefined && typeof inputActions.resolveDraftAttachments === 'function') {
+				try {
+					return { descriptors: inputActions.resolveDraftAttachments(ids) ?? [], source: 'actions' };
+				} catch {
+					// Neither owner could answer.
+				}
 			}
+			return { descriptors: [], source: 'none' };
+		}
+
+		/**
+		 * Turn resolved draft descriptors into request images.
+		 *
+		 * @param descriptors - draft descriptors carrying the original browser `File`.
+		 * @param signal - cancellation.
+		 * @returns `{ images, skipped }`.
+		 */
+		async function readComposerImages(descriptors, signal) {
+			const images = [];
+			let skipped = 0;
+			if (!Array.isArray(descriptors) || descriptors.length === 0) return { images, skipped };
 			for (const descriptor of descriptors.slice(0, 4)) {
 				if (signal.aborted) return { images, skipped };
 				if (descriptor?.kind !== 'image') continue;
@@ -900,10 +925,17 @@ window.__ModuleLoader__.load({
 						// race entirely and keeps one authority for the switch.
 						// Resolved here, not at activation: the service mounts with the
 						// conversation plugin, which can come up after this plugin.
-						const conversation = attachmentIds.length > 0 ? conversationResolver() : undefined;
+						const drafts = attachmentIds.length > 0
+							? resolveDraftDescriptors(inputActions, attachmentIds)
+							: { descriptors: [], source: 'none' };
 						const collected = attachmentIds.length > 0
-							? await readComposerImages(conversation, attachmentIds, controller.signal)
+							? await readComposerImages(drafts.descriptors, controller.signal)
 							: { images: [], skipped: 0 };
+						if (attachmentIds.length > 0 && drafts.source === 'none') {
+							// Nothing could read the attachments. Report it through the same notice
+							// the Host uses for a skipped image, instead of dropping them silently.
+							collected.skipped = attachmentIds.length;
+						}
 							if (controller.signal.aborted) return;
 						const response = await fetch('/api/prompt-enhance', {
 							method: 'POST',
@@ -1112,7 +1144,6 @@ window.__ModuleLoader__.load({
 					className: 'dsh-pe_root',
 					'data-prompt-enhance': '1',
 					'data-context': showContextMark ? 'true' : 'false',
-					'data-ctx-debug': ctxProbe,
 					'data-notice': failure !== null ? 'true' : 'false',
 				},
 				h(
@@ -1158,6 +1189,61 @@ window.__ModuleLoader__.load({
 		/** The command id this plugin registers with the host shortcuts service. */
 		const SHORTCUT_ID = 'prompt-enhance.run';
 
+		/**
+		 * Whether a value behaves like the host `shortcuts` service.
+		 *
+		 * The service is OPTIONAL: the host builds it only when the desktop keyboard
+		 * bridge exists, and a third-party shortcut plugin that pins the client runtime
+		 * to an older range can keep it from registering at all. Losing it must cost
+		 * only the shortcut settings row — never the composer button.
+		 *
+		 * @param candidate - value from `ctx.get('shortcuts')`.
+		 * @returns whether the face can be used.
+		 */
+		function usableShortcuts(candidate) {
+			if (candidate === undefined || candidate === null) return false;
+			if (typeof candidate !== 'object' && typeof candidate !== 'function') return false;
+			return typeof candidate.register === 'function';
+		}
+
+		/**
+		 * Read a snapshot from a face that may expose it as a method or as a plain value.
+		 *
+		 * Host builds differ here: one publishes `catalog` as an object with
+		 * `getSnapshot()`, another as the array itself. Both shapes are accepted, and
+		 * anything unexpected collapses to an empty list so callers stay simple.
+		 *
+		 * @param face - `shortcuts.catalog` or `shortcuts.config`.
+		 * @param key - array field to use when the value is a wrapper object.
+		 * @returns the snapshot array, or an empty array.
+		 */
+		function readSnapshot(face, key) {
+			if (face === undefined || face === null) return [];
+			try {
+				const raw = typeof face.getSnapshot === 'function' ? face.getSnapshot() : face;
+				if (Array.isArray(raw)) return raw;
+				if (Array.isArray(raw?.[key])) return raw[key];
+			} catch {
+				// A throwing getter means "not ready"; callers treat that as empty.
+			}
+			return [];
+		}
+
+		/**
+		 * Read the binding revision, tolerating both shapes and any throw.
+		 *
+		 * @param shortcuts - the service face.
+		 * @returns the revision, or undefined.
+		 */
+		function readRevision(shortcuts) {
+			try {
+				const config = shortcuts?.config;
+				const raw = typeof config?.getSnapshot === 'function' ? config.getSnapshot() : config;
+				return raw?.revision;
+			} catch {
+				return undefined;
+			}
+		}
 		/**
 		 * Read and edit this plugin's keyboard binding through the host shortcuts service.
 		 *
@@ -1170,7 +1256,11 @@ window.__ModuleLoader__.load({
 		 * @param t - localized translator, for host-reported failure states.
 		 * @returns `{ keys, isDefault, assign, reset, refresh }`.
 		 */
-		function useShortcutBinding(shortcuts, t) {
+		function useShortcutBinding(resolveShortcuts, t) {
+			// Resolved on every read, never captured: the host may register the service
+			// after this section mounts, and a value captured during render would stay
+			// undefined for the life of the component.
+			const shortcuts = typeof resolveShortcuts === 'function' ? resolveShortcuts() : resolveShortcuts;
 			const [state, setState] = useState({ keys: '', isDefault: true, revision: undefined });
 			const alive = useRef(false);
 			alive.current = true;
@@ -1179,10 +1269,11 @@ window.__ModuleLoader__.load({
 			const refresh = useCallback(() => {
 				if (shortcuts === undefined) return;
 				try {
-					const snapshot = shortcuts.catalog?.getSnapshot?.() ?? shortcuts.catalog ?? [];
-					const row = Array.isArray(snapshot) ? snapshot.find((entry) => entry.id === SHORTCUT_ID) : undefined;
-					const config = shortcuts.config?.getSnapshot?.() ?? shortcuts.config;
-					const revision = config?.revision;
+					const snapshot = readSnapshot(shortcuts?.catalog, 'commands');
+					const row = snapshot.find((entry) => entry?.id === SHORTCUT_ID);
+					const revision = readRevision(shortcuts);
+					// The command is not in the host catalog (yet, or at all). Leaving the row
+					// as it stands is correct: nothing is bound, so nothing can be shown.
 					if (row === undefined) return;
 					setState({
 						// `keys` is the host's own display form of the resolved binding.
@@ -1206,10 +1297,7 @@ window.__ModuleLoader__.load({
 			 * refuses it with `if (revision !== this.snapshot.revision) return result("stale")`.
 			 * That is why 恢复默认 did nothing and the old override kept showing.
 			 */
-			const revisionOf = useCallback(() => {
-				const config = shortcuts?.config?.getSnapshot?.() ?? shortcuts?.config;
-				return config?.revision;
-			}, [shortcuts]);
+			const revisionOf = useCallback(() => readRevision(shortcuts), [shortcuts]);
 
 			/**
 			 * Submit one edit against the live revision, retrying once on "stale".
@@ -1220,23 +1308,48 @@ window.__ModuleLoader__.load({
 			 * and then trying to reset it.
 			 */
 			const applyEdit = useCallback(async (operation) => {
-				if (shortcuts === undefined) return { status: 'unavailable' };
-				let result = await shortcuts.edit(operation, revisionOf());
-				if (result?.status === 'stale') {
-					refresh();
-					result = await shortcuts.edit(operation, revisionOf());
+				if (!usableShortcuts(shortcuts)) return { status: 'unavailable' };
+				if (typeof shortcuts.edit !== 'function') return { status: 'unavailable' };
+				try {
+					let result = await shortcuts.edit(operation, revisionOf());
+					if (result?.status === 'stale') {
+						refresh();
+						result = await shortcuts.edit(operation, revisionOf());
+					}
+					if (result?.status === 'saved') refresh();
+					return result;
+				} catch (error) {
+					// A throwing edit is a broken host, not a user mistake: report it as
+					// failed so the row shows a reason instead of crashing the page.
+					return { status: 'failed', reason: error?.message ?? 'exception' };
 				}
-				if (result?.status === 'saved') refresh();
-				return result;
 			}, [refresh, revisionOf, shortcuts]);
 
 			useEffect(() => {
 				refresh();
+				// The host may register the service a beat after this section mounts. A
+				// captured-at-render lookup would then leave the row permanently blank,
+				// which is exactly the empty box users saw. Re-reading a few times covers
+				// that window without polling forever.
+				let tries = 0;
+				const retry = setInterval(() => {
+					tries += 1;
+					refresh();
+					if (tries >= 10) clearInterval(retry);
+				}, 120);
 				// The host publishes catalog changes; following them keeps this row honest
 				// when a binding is changed from the general Settings page instead.
-				const unsubscribe = shortcuts?.catalog?.subscribe?.(refresh);
+				let unsubscribe;
+				try {
+					if (typeof shortcuts?.catalog?.subscribe === 'function') {
+						unsubscribe = shortcuts.catalog.subscribe(refresh);
+					}
+				} catch {
+					// No subscription means the row does not live-update; it still works.
+				}
 				return () => {
 					alive.current = false;
+					clearInterval(retry);
 					if (typeof unsubscribe === 'function') unsubscribe();
 				};
 			}, [refresh, shortcuts]);
@@ -1251,19 +1364,20 @@ window.__ModuleLoader__.load({
 			 * @returns a promise resolving to an error message, or null on success.
 			 */
 			const assign = useCallback(async (binding) => {
-				if (shortcuts === undefined) return t('settings.shortcut.unavailable');
+				if (!usableShortcuts(shortcuts)) return t('settings.shortcut.unavailable');
 				try {
 					// The host exposes the collision report for a candidate without applying
 					// it; naming the colliding commands is what makes the failure readable.
-					const described = shortcuts.describeBinding?.(binding);
+					const described = typeof shortcuts.describeBinding === 'function' ? shortcuts.describeBinding(binding) : undefined;
 					if (described?.issue !== null && described?.issue !== undefined) {
 						return t('settings.shortcut.rejected', { reason: described.issue });
 					}
 					const conflicts = (described?.conflicts ?? [])
 						.filter((id) => id !== SHORTCUT_ID)
 						.map((id) => {
-							const snapshot = shortcuts.catalog?.getSnapshot?.() ?? [];
-							const row = Array.isArray(snapshot) ? snapshot.find((entry) => entry.id === id) : undefined;
+							// A broken catalog only costs the friendly names in the message.
+							const snapshot = readSnapshot(shortcuts?.catalog, 'commands');
+							const row = snapshot.find((entry) => entry?.id === id);
 							const label = typeof row?.label === 'function' ? row.label() : row?.label;
 							return label ?? id;
 						});
@@ -1289,7 +1403,7 @@ window.__ModuleLoader__.load({
 			 * @returns a promise resolving to an error message, or null on success.
 			 */
 			const reset = useCallback(async () => {
-				if (shortcuts === undefined) return t('settings.shortcut.unavailable');
+				if (!usableShortcuts(shortcuts)) return t('settings.shortcut.unavailable');
 				try {
 					const result = await applyEdit({ type: 'reset', id: SHORTCUT_ID });
 					if (result !== undefined && result?.status !== 'saved') {
@@ -1315,7 +1429,7 @@ window.__ModuleLoader__.load({
 		 * @param props.t - localized translator.
 		 * @param props.getRemote - thunk returning the injected `remote.session` face.
 		 */
-		function SettingsSection({ t, getRemote, shortcuts }) {
+		function SettingsSection({ t, getRemote, getShortcuts }) {
 			const [values, setValues] = useState(null);
 			// `{ version, url }` reported by the Host from the package manifest. Never
 			// hard-coded here: a release only bumps package.json.
@@ -1371,7 +1485,6 @@ window.__ModuleLoader__.load({
 						// Keep the reason: an empty picker with no explanation is
 						// indistinguishable from a slow load, and the message is shown to the
 						// user instead of being swallowed.
-						if (typeof console !== 'undefined') console.warn('[dsh-prompt-rewriter] model catalog failed:', failure?.message ?? failure);
 						setCatalog({ status: 'error', groups: [] });
 					}
 				})();
@@ -1436,7 +1549,9 @@ window.__ModuleLoader__.load({
 			// it after that return meant the loading render used fewer hooks than the
 			// loaded render, which React rejects with "Rendered more hooks than during the
 			// previous render" — and the settings page went completely blank.
-			const shortcut = useShortcutBinding(shortcuts, t);
+			// The hook resolves the face on every read: the host may register it after
+			// this section mounts, and it may never register it at all.
+			const shortcut = useShortcutBinding(getShortcuts, t);
 
 			if (values === null) {
 				return h(
@@ -1717,7 +1832,16 @@ window.__ModuleLoader__.load({
 			// detection and the user's binding overrides. Registering through it — instead
 			// of adding our own keydown listener — is what makes a custom binding persist
 			// across restarts and collide visibly with a host command.
-			inject: ['slots', 'locale', 'remote', 'remote.session', 'shortcuts'],
+			//
+			// It is deliberately NOT in this list. Cordis activates a plugin only when
+			// every injected service exists, so listing it made a missing keyboard
+			// bridge fatal: the whole plugin stayed dormant and the composer button
+			// vanished along with the shortcut row. The host builds that service
+			// conditionally (`window.dshDesktop.keyboard`), and a third-party shortcut
+			// plugin that pins the client runtime to an older range can keep it from
+			// ever registering. Resolving it through `ctx.get` at call time makes the
+			// keyboard binding optional while the rewrite itself stays unconditional.
+			inject: ['slots', 'locale', 'remote', 'remote.session'],
 			apply(ctx) {
 				ctx.effect(installStyles, 'dsh-prompt-rewriter: styles');
 				ctx.effect(
@@ -1835,7 +1959,12 @@ window.__ModuleLoader__.load({
 				// The registered component is the section itself, bound to this context.
 				// Keeping ONE stable component identity (no per-render wrapper) means the
 				// owner mounts a stable type and the section owns its own hook slots.
-				const Section = (props) => h(SettingsSection, { ...props, t, getRemote: getSessionRemote, shortcuts: ctx.shortcuts });
+								// Resolved through `ctx.get` instead of `ctx.shortcuts`: once `shortcuts`
+				// leaves the inject list the ctx proxy no longer exposes it as a property,
+				// describing then throws. A thunk also lets the section re-read the face
+				// after the host finishes mounting it.
+				const getShortcuts = () => (typeof ctx.get === 'function' ? ctx.get('shortcuts') : ctx.shortcuts);
+				const Section = (props) => h(SettingsSection, { ...props, t, getRemote: getSessionRemote, getShortcuts });
 				ctx.slots.inject('settings.section', () =>
 					ctx.slots.register(
 						{ name: 'settings.section', id: 'prompt-enhance', order: 50, label: () => t('nav'), locale: NS },
@@ -1864,7 +1993,17 @@ window.__ModuleLoader__.load({
 				//     three specific punctuation chords,
 				// so the combination must carry three modifiers AND include Alt.
 				// Ctrl+Shift+Alt+Enter satisfies both and is unused by the host.
-				ctx.effect(() => {
+				// Registration waits for the host shortcut service WITHOUT gating activation.
+				//
+				// `ctx.get(name)` does not bypass the inject gate: a service outside `inject`
+				// resolves to an Error object, so the previous guard tripped and the command
+				// was never registered (the host catalog listed its own commands, none ours).
+				// Every name in `inject` is REQUIRED for activation, so listing `shortcuts`
+				// there would keep the whole plugin dormant without a keyboard bridge.
+				// `ctx.inject([...], cb)` is the middle path: it waits for the service and
+				// leaves the plugin active either way.
+				ctx.inject(['shortcuts'], (scope) => {
+					scope.effect(() => {
 					try {
 						// One implementation, reached from two places: `resolve` is what dispatch
 						// invokes, `run` is kept for any caller that uses the command object directly.
@@ -1884,7 +2023,11 @@ window.__ModuleLoader__.load({
 								trigger();
 							}
 						};
-						const dispose = ctx.shortcuts.register({
+						// No service means no host-owned binding: the fallback listener below
+						// still runs the command, so the key keeps working where it can.
+						const shortcutsFace = scope.shortcuts;
+						if (typeof shortcutsFace?.register !== 'function') return undefined;
+						const dispose = shortcutsFace.register({
 							id: 'prompt-enhance.run',
 							label: () => t('shortcut.run'),
 							aliases: ['prompt enhance', 'enhance prompt', 'rewrite prompt'],
@@ -1925,12 +2068,13 @@ window.__ModuleLoader__.load({
 							modals: [],
 						});
 						return dispose;
-					} catch {
+					} catch (error) {
 						// A throw here is swallowed by the effect wrapper; the keydown fallback
 						// below still serves the shortcut, so it must not fail activation.
 						return undefined;
 					}
 				}, 'dsh-prompt-rewriter: shortcut');
+				});
 
 				// Keyboard fallback.
 				//
@@ -1951,7 +2095,10 @@ window.__ModuleLoader__.load({
 					/** The binding the host currently resolves for our command, if any. */
 					const resolvedBinding = () => {
 						try {
-							const snapshot = ctx.shortcuts?.catalog?.getSnapshot?.() ?? [];
+							// A missing service just means "no host binding to compare against";
+							// the fallback listener then relies on its own defaults.
+							const face = typeof ctx.get === 'function' ? ctx.get('shortcuts') : ctx.shortcuts;
+							const snapshot = readSnapshot(face?.catalog, 'commands');
 							const row = Array.isArray(snapshot)
 								? snapshot.find((entry) => entry.id === 'prompt-enhance.run')
 								: undefined;
@@ -2017,7 +2164,7 @@ window.__ModuleLoader__.load({
 
 						let candidate = fromEvent(event);
 						try {
-							const described = ctx.shortcuts?.describeBinding?.(candidate);
+							const described = typeof face?.describeBinding === 'function' ? face.describeBinding(candidate) : undefined;
 							if (described?.binding !== undefined && described?.binding !== null) candidate = described.binding;
 						} catch {
 							// Keep the symbolic form; `same` compares modifier sets.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-prompt-rewriter",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "One-click prompt enhancement beside the DeepSeek Harness composer: rewrites a draft in place without sending it.",
   "type": "module",
   "icon": "icon.svg",