@prettier-ai/dsh-client-ui-message-feedback 0.1.2-alpha.1 → 0.1.2-alpha.3

package/lib/client.js

@@ -58,7 +58,7 @@ window.__ModuleLoader__.load({
 		* control in that Session, so a single list read seeds them all.
 		*/
 		var MessageFeedbackController = class {
-			remote;
+			ctx;
 			sessionId;
 			view = INITIAL_VIEW;
 			listeners = /* @__PURE__ */ new Set();
@@ -66,11 +66,11 @@ window.__ModuleLoader__.load({
 			operationTail = Promise.resolve();
 			disposed = false;
 			/**
-			* @param remote - the messageFeedback Remote namespace.
+			* @param ctx - the browser plugin context carrying the messageFeedback Remote namespace.
 			* @param sessionId - Session owning every addressed assistant message.
 			*/
-			constructor(remote, sessionId) {
-				this.remote = remote;
+			constructor(ctx, sessionId) {
+				this.ctx = ctx;
 				this.sessionId = sessionId;
 			}
 			/** Return the cached immutable view. */
@@ -185,7 +185,7 @@ window.__ModuleLoader__.load({
 			}
 			/** Commit one put against the observed version and reconcile a conflict. */
 			async putCommitted(messageId, rating, note, observed) {
-				const carried = await this.remote.put({
+				const carried = await this.ctx.remote.messageFeedback.put({
 					sessionId: this.sessionId,
 					messageId,
 					rating,
@@ -203,7 +203,7 @@ window.__ModuleLoader__.load({
 			}
 			/** Commit one delete against the observed version and reconcile a conflict. */
 			async deleteCommitted(messageId, observed) {
-				const carried = await this.remote.delete({
+				const carried = await this.ctx.remote.messageFeedback.delete({
 					sessionId: this.sessionId,
 					messageId,
 					ifVersion: observed.version
@@ -224,55 +224,37 @@ window.__ModuleLoader__.load({
 			}
 			/** Fetch the whole sidecar and publish it as the seeded view. */
 			async load() {
-				try {
-					const carried = await this.remote.list({ sessionId: this.sessionId });
-					if (this.disposed) return OK;
-					if (!carried.ok) {
-						this.publish({
-							status: "error",
-							items: this.view.items,
-							error: carried.error.message
-						});
-						return carrierFailure(carried.error);
-					}
-					const result = carried.value;
-					if (!result.ok) {
-						this.publish({
-							status: "error",
-							items: this.view.items,
-							error: describe(result.error.code)
-						});
-						return fail(result.error.code);
-					}
-					const items = /* @__PURE__ */ new Map();
-					for (const item of result.value.items) items.set(item.messageId, item);
+				const carried = await this.ctx.remote.messageFeedback.list({ sessionId: this.sessionId });
+				if (this.disposed) return OK;
+				if (!carried.ok) {
 					this.publish({
-						status: "ready",
-						items,
-						error: null
+						status: "error",
+						items: this.view.items,
+						error: carried.error.message
 					});
-					return OK;
-				} catch (error) {
-					if (this.disposed) return OK;
-					const message = error instanceof Error ? error.message : String(error);
+					return carrierFailure(carried.error);
+				}
+				const result = carried.value;
+				if (!result.ok) {
 					this.publish({
 						status: "error",
 						items: this.view.items,
-						error: message
+						error: describe(result.error.code)
 					});
-					return {
-						ok: false,
-						error: {
-							code: "transport",
-							message
-						}
-					};
+					return fail(result.error.code);
 				}
+				const items = /* @__PURE__ */ new Map();
+				for (const item of result.value.items) items.set(item.messageId, item);
+				this.publish({
+					status: "ready",
+					items,
+					error: null
+				});
+				return OK;
 			}
 			/**
 			* Serialize one mutation behind this Session's prior mutation so queued
-			* operations always compare against the committed version, and translate a
-			* transport throw into the same settled shape the controls already render.
+			* operations always compare against the committed version.
 			*/
 			mutate(operation, options = {}) {
 				const guarded = async () => {
@@ -282,17 +264,7 @@ window.__ModuleLoader__.load({
 						if (!loaded.ok) return loaded;
 						if (this.disposed) return DISPOSED;
 					}
-					try {
-						return await operation();
-					} catch (error) {
-						return {
-							ok: false,
-							error: {
-								code: "transport",
-								message: error instanceof Error ? error.message : String(error)
-							}
-						};
-					}
+					return await operation();
 				};
 				const result = this.operationTail.then(guarded, guarded);
 				this.operationTail = result.then(() => void 0);
@@ -326,7 +298,7 @@ window.__ModuleLoader__.load({
 		};
 		//#endregion
 		//#region \0dsh-css:/home/runner/work/dsh-publisher/dsh-publisher/upstream/packages/client/ui-message-feedback/src/client/MessageFeedbackActions.module.css.mjs
-		const css = ".-ssFWa_action{width:calc(28px + var(--dsh-content-font-delta,0px));height:calc(28px + var(--dsh-content-font-delta,0px));color:var(--dsw-alias-label-tertiary);cursor:pointer;background:0 0;border:none;border-radius:28px;justify-content:center;align-items:center;padding:6px;display:inline-flex}.-ssFWa_action svg{width:calc(16px + var(--dsh-content-font-delta,0px));height:calc(16px + var(--dsh-content-font-delta,0px))}.-ssFWa_action:hover{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_action:disabled{cursor:default;opacity:.4}.-ssFWa_action[data-active]{color:var(--dsw-alias-label-primary)}.-ssFWa_noteOpen{max-width:220px;color:var(--dsw-alias-label-tertiary);font-size:var(--dsh-content-font-size-secondary,13px);line-height:calc(28px + var(--dsh-content-font-delta,0px));white-space:nowrap;text-overflow:ellipsis;cursor:pointer;background:0 0;border:none;border-radius:14px;padding:0 8px;overflow:hidden}.-ssFWa_noteOpen:hover,.-ssFWa_noteOpen[aria-expanded=true]{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_notePanel{z-index:1100;box-sizing:border-box;border:1px solid var(--dsw-alias-border-inverted);background:var(--dsw-specific-menu);width:320px;max-width:min(360px,100vw - 24px);max-height:calc(100vh - 24px);box-shadow:var(--dsw-shadow-lv3);--dsh-scrollbar-thumb:var(--dsw-alias-scrollbar-bg-l2);--dsh-scrollbar-thumb-hover:var(--dsw-alias-scrollbar-hover-l2);border-radius:12px;flex-direction:column;gap:8px;padding:8px;display:flex;position:fixed;overflow-y:auto}.-ssFWa_noteInput{box-sizing:border-box;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);width:100%;color:var(--dsw-alias-label-primary);font:inherit;resize:vertical;border-radius:8px;padding:6px 8px;font-size:13px}.-ssFWa_noteActions{justify-content:flex-end;gap:6px;display:flex}.-ssFWa_noteSave,.-ssFWa_noteCancel{cursor:pointer;border:none;border-radius:14px;height:28px;padding:0 10px;font-size:13px}.-ssFWa_noteSave{background:var(--dsw-alias-button-primary-fill);color:var(--dsw-alias-label-primary-foreground)}.-ssFWa_noteSave:hover:not(:disabled){background:var(--dsw-alias-button-primary-hover)}.-ssFWa_noteSave:disabled{cursor:default;opacity:.4}.-ssFWa_noteCancel{color:var(--dsw-alias-label-tertiary);background:0 0}.-ssFWa_noteCancel:hover{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_failure{color:var(--dsw-alias-label-tertiary);padding-left:4px;font-size:13px;line-height:20px}";
+		const css = ".-ssFWa_action{width:calc(28px + var(--dsh-content-font-delta,0px));height:calc(28px + var(--dsh-content-font-delta,0px));color:var(--dsw-alias-label-tertiary);cursor:pointer;background:0 0;border:none;border-radius:28px;justify-content:center;align-items:center;padding:6px;display:inline-flex}.-ssFWa_action svg{width:calc(15px + var(--dsh-content-font-delta,0px));height:calc(15px + var(--dsh-content-font-delta,0px))}.-ssFWa_action:hover{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_action:disabled{cursor:default;opacity:.4}.-ssFWa_action[data-active]{color:var(--dsw-alias-label-primary)}.-ssFWa_noteOpen{max-width:220px;color:var(--dsw-alias-label-tertiary);font-size:var(--dsh-content-font-size-secondary,13px);line-height:calc(28px + var(--dsh-content-font-delta,0px));white-space:nowrap;text-overflow:ellipsis;cursor:pointer;background:0 0;border:none;border-radius:14px;padding:0 8px;overflow:hidden}.-ssFWa_noteOpen:hover,.-ssFWa_noteOpen[aria-expanded=true]{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_notePanel{z-index:1100;box-sizing:border-box;border:1px solid var(--dsw-alias-border-inverted);background:var(--dsw-specific-menu);width:320px;max-width:min(360px,100vw - 24px);max-height:calc(100vh - 24px);box-shadow:var(--dsw-shadow-lv3);--dsh-scrollbar-thumb:var(--dsw-alias-scrollbar-bg-l2);--dsh-scrollbar-thumb-hover:var(--dsw-alias-scrollbar-hover-l2);border-radius:12px;flex-direction:column;gap:8px;padding:8px;display:flex;position:fixed;overflow-y:auto}.-ssFWa_noteInput{box-sizing:border-box;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);width:100%;color:var(--dsw-alias-label-primary);font:inherit;resize:vertical;border-radius:8px;padding:6px 8px;font-size:13px}.-ssFWa_noteActions{justify-content:flex-end;gap:6px;display:flex}.-ssFWa_noteSave,.-ssFWa_noteCancel{cursor:pointer;border:none;border-radius:14px;height:28px;padding:0 10px;font-size:13px}.-ssFWa_noteSave{background:var(--dsw-alias-button-primary-fill);color:var(--dsw-alias-label-primary-foreground)}.-ssFWa_noteSave:hover:not(:disabled){background:var(--dsw-alias-button-primary-hover)}.-ssFWa_noteSave:disabled{cursor:default;opacity:.4}.-ssFWa_noteCancel{color:var(--dsw-alias-label-tertiary);background:0 0}.-ssFWa_noteCancel:hover{background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.-ssFWa_failure{color:var(--dsw-alias-label-tertiary);padding-left:4px;font-size:13px;line-height:20px}";
 		const tagId = "@prettier-ai/dsh-client-ui-message-feedback/MessageFeedbackActions.module.css";
 		if (typeof document !== "undefined" && document.querySelector("style[data-plugin-css=" + JSON.stringify(tagId) + "]") === null) {
 			const tag = document.createElement("style");
@@ -684,7 +656,7 @@ window.__ModuleLoader__.load({
 			const controllerFor = (sessionId) => {
 				let controller = controllers.get(sessionId);
 				if (controller === void 0) {
-					controller = new MessageFeedbackController(ctx.remote.messageFeedback, sessionId);
+					controller = new MessageFeedbackController(ctx, sessionId);
 					controllers.set(sessionId, controller);
 				}
 				return controller;

package/lib/types/client/controller.d.ts

@@ -6,34 +6,11 @@
  * instead of refetching the whole Session.
  * @module @prettier-ai/dsh-client-ui-message-feedback/client/controller
  */
-import type { RemoteResult } from '@prettier-ai/dsh-typert-protocol';
+import type { Context as ClientContext } from '@prettier-ai/cordis';
 import type { HostObservable } from '@prettier-ai/dsh-client-ui-slots';
-import type { MessageId } from '@prettier-ai/dsh-client-connection/client';
+import type { MessageId } from '@prettier-ai/dsh-api-remotes/client';
 import type { SessionId } from '@prettier-ai/dsh-session/types';
-import type { MessageFeedbackDeleteResult, MessageFeedbackItem, MessageFeedbackListResult, MessageFeedbackPutResult, MessageFeedbackRating } from '@prettier-ai/dsh-message-feedback/types';
-/**
- * The three Remote calls this controller needs. The generated face wraps every
- * business result in {@link RemoteResult}: a carrier failure arrives as the
- * `ok: false` branch rather than a rejection, so this controller reads one
- * envelope and never wraps a call to recover a transport error.
- */
-export interface MessageFeedbackRemote {
-    list: (request: {
-        sessionId: SessionId;
-    }) => Promise<RemoteResult<MessageFeedbackListResult>>;
-    put: (request: {
-        sessionId: SessionId;
-        messageId: MessageId;
-        rating: MessageFeedbackRating;
-        note?: string;
-        ifVersion: MessageFeedbackItem['version'] | null;
-    }) => Promise<RemoteResult<MessageFeedbackPutResult>>;
-    delete: (request: {
-        sessionId: SessionId;
-        messageId: MessageId;
-        ifVersion: MessageFeedbackItem['version'];
-    }) => Promise<RemoteResult<MessageFeedbackDeleteResult>>;
-}
+import type { MessageFeedbackItem, MessageFeedbackRating } from '@prettier-ai/dsh-message-feedback/types';
 /** Load state of the one list read that seeds every per-message control. */
 export type MessageFeedbackStatus = 'cold' | 'loading' | 'ready' | 'error';
 /** Immutable view published to every per-message control in one Session. */
@@ -59,7 +36,7 @@ export type MessageFeedbackActionResult = {
  * control in that Session, so a single list read seeds them all.
  */
 export declare class MessageFeedbackController implements HostObservable<MessageFeedbackView> {
-    private readonly remote;
+    private readonly ctx;
     private readonly sessionId;
     private view;
     private readonly listeners;
@@ -67,10 +44,10 @@ export declare class MessageFeedbackController implements HostObservable<Message
     private operationTail;
     private disposed;
     /**
-     * @param remote - the messageFeedback Remote namespace.
+     * @param ctx - the browser plugin context carrying the messageFeedback Remote namespace.
      * @param sessionId - Session owning every addressed assistant message.
      */
-    constructor(remote: MessageFeedbackRemote, sessionId: SessionId);
+    constructor(ctx: ClientContext, sessionId: SessionId);
     /** Return the cached immutable view. */
     getSnapshot: () => MessageFeedbackView;
     /** Subscribe to view replacement. */
@@ -146,8 +123,7 @@ export declare class MessageFeedbackController implements HostObservable<Message
     private load;
     /**
      * Serialize one mutation behind this Session's prior mutation so queued
-     * operations always compare against the committed version, and translate a
-     * transport throw into the same settled shape the controls already render.
+     * operations always compare against the committed version.
      */
     private mutate;
     /**

package/lib/types/client/index.d.ts

@@ -7,7 +7,7 @@
  * @module @prettier-ai/dsh-client-ui-message-feedback/client
  */
 import type { Context as ClientContext } from '@prettier-ai/cordis';
-export type { MessageFeedbackActionResult, MessageFeedbackStatus, MessageFeedbackView, MessageFeedbackRemote, } from './controller.ts';
+export type { MessageFeedbackActionResult, MessageFeedbackStatus, MessageFeedbackView, } from './controller.ts';
 export type { MessageFeedbackActionProps, MessageFeedbackInjected } from './slots.ts';
 export type { MessageFeedbackKey } from './locales.ts';
 /** Required services: the slot registry, the Remote namespace, and the copy. */

package/lib/types/client/slots.d.ts

@@ -8,7 +8,7 @@
  * @module @prettier-ai/dsh-client-ui-message-feedback/client/slots
  */
 import type { HostObservable, InjectFace, PropsLocale, PropsRuntime } from '@prettier-ai/dsh-client-ui-slots';
-import type { MessageId } from '@prettier-ai/dsh-client-connection/client';
+import type { MessageId } from '@prettier-ai/dsh-api-remotes/client';
 import type { MessageFeedbackRating } from '@prettier-ai/dsh-message-feedback/types';
 import type { MessageFeedbackActionResult, MessageFeedbackView } from './controller.ts';
 /** Injected business face of one assistant-message feedback entry. */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@prettier-ai/dsh-client-ui-message-feedback",
   "description": "Per-message feedback controls contributed to the assistant-message action strip, backed by the messageFeedback Host Remote",
-  "version": "0.1.2-alpha.1",
+  "version": "0.1.2-alpha.3",
   "publishConfig": {
     "access": "public"
   },
@@ -42,18 +42,7 @@
   },
   "license": "MIT",
   "peerDependencies": {
-    "@prettier-ai/dsh-api-remotes": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-connection": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-locale": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-renderer": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-conversation": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-message-feedback": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-session": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-invariants": "^0.1.2-alpha.1",
-    "@prettier-ai/cordis": "^4.0.1",
-    "@prettier-ai/dsh-typert-protocol": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-chat": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-session": "^0.1.2-alpha.1"
+    "@prettier-ai/cordis": "^4.0.2"
   },
   "devDependencies": {
     "@testing-library/react": "^16.1.0",
@@ -61,21 +50,21 @@
     "@types/react-dom": "~18.3.0",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
-    "@prettier-ai/dsh-api-remotes": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-connection": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-test-runtime": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-conversation": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-locale": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-primitives": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-renderer": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-slots": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-invariants": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-typert-protocol": "^0.1.2-alpha.1",
-    "@prettier-ai/cordis": "^4.0.1",
-    "@prettier-ai/dsh-client-ui-chat": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-message-feedback": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-session": "^0.1.2-alpha.1",
-    "@prettier-ai/dsh-client-ui-session": "^0.1.2-alpha.1"
+    "@prettier-ai/dsh-client-connection": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-locale": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-primitives": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-test-runtime": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-api-remotes": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-conversation": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-renderer": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-invariants": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-slots": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-message-feedback": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-typert-protocol": "^0.1.2-alpha.3",
+    "@prettier-ai/cordis": "^4.0.2",
+    "@prettier-ai/dsh-session": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-session": "^0.1.2-alpha.3",
+    "@prettier-ai/dsh-client-ui-chat": "^0.1.2-alpha.3"
   },
   "files": [
     "lib/index.js",