@cueloop/client 0.1.0-alpha.37 → 0.1.0-alpha.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/client",
-  "version": "0.1.0-alpha.37",
+  "version": "0.1.0-alpha.39",
   "description": "The cueloop review UI: projection renderer, keyboard grammar, review rail",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -18,16 +18,17 @@
   ],
   "type": "module",
   "exports": {
-    ".": "./src/index.ts"
+    ".": "./src/index.ts",
+    "./config": "./src/config.ts"
   },
   "publishConfig": {
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/daemon": "0.1.0-alpha.37",
-    "@cueloop/extension-api": "0.1.0-alpha.37",
-    "@cueloop/integration-obsidian": "0.1.0-alpha.37",
-    "@cueloop/schema": "0.1.0-alpha.37",
+    "@cueloop/daemon": "0.1.0-alpha.39",
+    "@cueloop/extension-api": "0.1.0-alpha.39",
+    "@cueloop/integration-obsidian": "0.1.0-alpha.39",
+    "@cueloop/schema": "0.1.0-alpha.39",
     "@opentui/core": "^0.5.2",
     "@opentui/keymap": "0.5.2",
     "@opentui/react": "^0.5.2",

package/src/App.plan-review.test.tsx

@@ -9,6 +9,7 @@ import { testRender } from "@opentui/react/test-utils";
 import { DaemonServer } from "@cueloop/daemon";
 import type { ReviewSession } from "@cueloop/schema";
 import { App } from "./App";
+import { DEFAULT_QUICK_ACTIONS, quickActionBody } from "./config";
 import { DARK } from "./theme";
 import {
   isolateUserConfig,
@@ -599,6 +600,6 @@ describe("marker-actions popover", () => {
     await waitForState(setup, () => server.core.sessionGet(session.id).annotations.length === 1);
     const stored = server.core.sessionGet(session.id);
     expect(stored.annotations[0]!.kind).toBe("comment");
-    expect(stored.annotations[0]!.body).toBe("Restate simplified");
+    expect(stored.annotations[0]!.body).toBe(quickActionBody(DEFAULT_QUICK_ACTIONS[1]!));
   });
 });

package/src/components/story.ts

@@ -53,13 +53,14 @@ export async function loadStories(): Promise<LoadedStory[]> {
   return loaded;
 }
 
-/** Component files that must carry stories: every .tsx that is not a story/test. */
+/** Component files that must carry stories: every .tsx that is not a story/test/prototype. */
 export function componentFilesMissingStories(): string[] {
   const componentGlob = new Bun.Glob("**/*.tsx");
   const files = [...componentGlob.scanSync({ cwd: import.meta.dir })];
   const missing: string[] = [];
   for (const file of files) {
     if (file.endsWith(".stories.tsx") || file.endsWith(".test.tsx")) continue;
+    if (file.endsWith(".prototype.tsx")) continue;
     if (file === "stories-app.tsx") continue;
     const storiesSibling = file.replace(/\.tsx$/, ".stories.tsx");
     if (!files.includes(storiesSibling)) missing.push(file);

package/src/config.test.ts

@@ -11,6 +11,8 @@ import {
   persistReviewState,
   persistReviewWidth,
   persistTheme,
+  quickActionBody,
+  resolveQuickAction,
 } from "./config";
 import { REVIEW_DEFAULT_WIDTH, REVIEW_MAX_WIDTH } from "./review-panel";
 import { DARK } from "./theme";
@@ -418,6 +420,33 @@ describe("quick actions ([[actions]])", () => {
   });
 });
 
+describe("quickActionBody", () => {
+  test("appends the system prompt below the prompt when set", () => {
+    // Assert
+    expect(quickActionBody({ prompt: "Prototype this", metadata: "skip tests" })).toBe(
+      "Prototype this\n\nskip tests",
+    );
+    expect(quickActionBody({ prompt: "Out of scope" })).toBe("Out of scope");
+  });
+});
+
+describe("resolveQuickAction", () => {
+  test("resolves by 1-based index and by case-insensitive prompt", () => {
+    // Assert
+    expect(resolveQuickAction(DEFAULT_QUICK_ACTIONS, "1")).toEqual(DEFAULT_QUICK_ACTIONS[0]!);
+    expect(resolveQuickAction(DEFAULT_QUICK_ACTIONS, "out of scope")).toEqual(
+      DEFAULT_QUICK_ACTIONS[2]!,
+    );
+  });
+
+  test("returns undefined for an out-of-range index or an unknown name", () => {
+    // Assert
+    expect(resolveQuickAction(DEFAULT_QUICK_ACTIONS, "0")).toBeUndefined();
+    expect(resolveQuickAction(DEFAULT_QUICK_ACTIONS, "99")).toBeUndefined();
+    expect(resolveQuickAction(DEFAULT_QUICK_ACTIONS, "no such action")).toBeUndefined();
+  });
+});
+
 describe("actionFor", () => {
   test("resolves keys and shifted keys to actions", () => {
     // Assert

package/src/config.ts

@@ -35,15 +35,56 @@ export interface QuickAction {
 
 /** The built-in quick-actions for the marker popover when no `[[actions]]` are configured. */
 export const DEFAULT_QUICK_ACTIONS: QuickAction[] = [
-  { prompt: "Zoom out, research in depth" },
-  { prompt: "Restate simplified" },
-  { prompt: "Out of scope" },
-  { prompt: "Let's chat about this" },
-  { prompt: "Prototype this" },
-  { prompt: "Ensure 0 regressions" },
-  { prompt: "Consider existing repo patterns" },
+  {
+    prompt: "Zoom out, research in depth",
+    metadata: "Research the broader problem, prior art, and alternatives before changing anything.",
+  },
+  {
+    prompt: "Restate simplified",
+    metadata: "Restate this as the simplest thing that works and cut the incidental complexity.",
+  },
+  {
+    prompt: "Out of scope",
+    metadata:
+      "This is out of scope for the task, so capture it as a follow-up instead of doing it now.",
+  },
+  {
+    prompt: "Let's chat about this",
+    metadata:
+      "Do not implement yet, and surface the open questions and trade-offs so we decide together.",
+  },
+  {
+    prompt: "Prototype this",
+    metadata: "Build a throwaway prototype to answer the question, skipping tests and polish.",
+  },
+  {
+    prompt: "Ensure 0 regressions",
+    metadata:
+      "Characterize the current behavior with tests first, then keep them green through the change.",
+  },
+  {
+    prompt: "Consider existing repo patterns",
+    metadata:
+      "Follow the nearest existing pattern in this codebase rather than introducing a new one.",
+  },
 ];
 
+/** The comment body a quick action expands to: the prompt, then its system prompt when set. */
+export function quickActionBody(action: QuickAction): string {
+  return action.metadata ? `${action.prompt}\n\n${action.metadata}` : action.prompt;
+}
+
+/** Resolve a quick action by 1-based index or case-insensitive prompt; undefined when no match. */
+export function resolveQuickAction(
+  actions: QuickAction[],
+  actionRef: string,
+): QuickAction | undefined {
+  const index = Number(actionRef);
+  if (Number.isInteger(index) && index >= 1 && index <= actions.length) return actions[index - 1];
+  const wanted = actionRef.trim().toLowerCase();
+  return actions.find((action) => action.prompt.toLowerCase() === wanted);
+}
+
 export interface CueloopConfig {
   keys: Record<string, string[]>;
   theme: Theme;

package/src/intent-dispatch.ts

@@ -13,7 +13,7 @@ import { displayText, spanKey, startSpan, type DisplayBlock, type SpanState } fr
 import type { DiffRow } from "./view-diff";
 import type { ReviewController } from "./session-controller";
 import type { Intent } from "./keymap";
-import type { QuickAction } from "./config";
+import { quickActionBody, type QuickAction } from "./config";
 import type { PlanSheetHandle } from "./components/PlanSheet";
 import { VERDICTS } from "./components/ConfirmCard";
 import {
@@ -251,7 +251,7 @@ export function createIntentDispatch(deps: IntentDispatchDeps): (intent: Intent)
         if (mode.type !== "spanActions") return;
         const action = quickActions[intent.index ?? mode.index];
         if (session && action) {
-          const body = action.metadata ? `${action.prompt}\n\n${action.metadata}` : action.prompt;
+          const body = quickActionBody(action);
           const annotationId = controller.annotate(
             "comment",
             mode.span.displayIndex,