mulmoclaude 0.6.2 → 0.6.3

package/src/plugins/manageSkills/categories.ts

@@ -0,0 +1,125 @@
+// Pure helpers behind the /skills page sidebar. Lifted out of View.vue
+// so the section-collapse state and the provenance rule (mc- prefix
+// split, user/project source mapping) live in exactly one place and can
+// be unit-tested in node:test without a DOM or a Vue runtime.
+
+import type { SkillSummary } from "./index";
+
+// categorizeSkill / pickInitialSelection only care about name + source,
+// not description. Exposing a narrower input type lets unit tests build
+// fixtures without padding placeholder descriptions everywhere.
+export type SkillIdentity = Pick<SkillSummary, "name" | "source">;
+
+// `mc-` is the launcher-managed namespace (see
+// server/workspace/skills-preset.ts). Skills under this prefix ship
+// with mulmoclaude and are overwritten on every boot, so the UI treats
+// them as the read-only "system" provenance and gates editing
+// accordingly. This is NOT the sidebar grouping axis — provenance only
+// drives the per-row badge tooltip and the edit/delete gate. The
+// sidebar groups by section (active vs catalog), see SKILL_SECTION_KEYS.
+export const SYSTEM_SKILL_PREFIX = "mc-";
+export type SkillProvenance = "system" | "project" | "user";
+
+/** Map a skill to its provenance bucket (badge + edit-gate, not layout). */
+export function categorizeSkill(skill: SkillIdentity): SkillProvenance {
+  if (skill.source === "user") return "user";
+  if (skill.name.startsWith(SYSTEM_SKILL_PREFIX)) return "system";
+  return "project";
+}
+
+// Sidebar collapsible sections, aligned with the #1335 catalog/active
+// model: "active" = skills in `.claude/skills/` (discovered by Claude
+// Code, loaded into the system prompt); "catalog" = launcher-managed
+// presets the user can browse / ★ star / ▶ run once without bloating
+// the prompt. Provenance (system/project/user) is shown as a per-row
+// badge inside the Active section, not as its own collapsible group.
+export const SKILL_SECTION_KEYS = ["active", "catalog"] as const;
+export type SkillSectionKey = (typeof SKILL_SECTION_KEYS)[number];
+
+export const SECTION_LABEL_KEYS: Record<SkillSectionKey, string> = {
+  active: "pluginManageSkills.sectionActive",
+  catalog: "pluginManageSkills.sectionCatalog",
+};
+
+// Both sections open by default — #1335 shows Active and Catalog
+// expanded; the user collapses whichever they don't want to see.
+export const DEFAULT_CLOSED_SECTIONS: readonly SkillSectionKey[] = [];
+export const COLLAPSED_SECTIONS_STORAGE_KEY = "skills:sectionCollapsed";
+
+/**
+ * @internal exported only so the unit tests can target the type guard
+ * directly. Call sites should reach it via loadCollapsedSections.
+ */
+export function isSkillSectionKey(value: unknown): value is SkillSectionKey {
+  return typeof value === "string" && (SKILL_SECTION_KEYS as readonly string[]).includes(value);
+}
+
+/** Read the persisted collapse state, falling back to defaults on any error. */
+export function loadCollapsedSections(): Set<SkillSectionKey> {
+  const defaults = new Set<SkillSectionKey>(DEFAULT_CLOSED_SECTIONS);
+  if (typeof window === "undefined") return defaults;
+  try {
+    const raw = window.localStorage.getItem(COLLAPSED_SECTIONS_STORAGE_KEY);
+    if (raw === null) return defaults;
+    const parsed: unknown = JSON.parse(raw);
+    if (!Array.isArray(parsed)) return defaults;
+    return new Set<SkillSectionKey>(parsed.filter(isSkillSectionKey));
+  } catch {
+    return defaults;
+  }
+}
+
+/** Persist the collapse state. Failures (e.g. localStorage disabled) are swallowed. */
+export function persistCollapsedSections(state: ReadonlySet<SkillSectionKey>): void {
+  if (typeof window === "undefined") return;
+  try {
+    window.localStorage.setItem(COLLAPSED_SECTIONS_STORAGE_KEY, JSON.stringify([...state]));
+  } catch {
+    // localStorage may be unavailable (private mode) — swallow silently.
+  }
+}
+
+// Per-external-repo collapse state (#1383 PR-C2). Distinct storage key
+// from the section-level state above: the section axis is a fixed
+// 2-value union (active/catalog), whereas repo ids are open-ended
+// (one per installed external repo), so this set is validated as
+// plain strings rather than against a key union. Default: every repo
+// EXPANDED (absent = open) — a freshly installed repo should show its
+// skills without a click.
+export const REPO_COLLAPSED_STORAGE_KEY = "skills:repoCollapsed";
+
+/** Read the persisted per-repo collapse set, defaulting to empty
+ *  (all repos expanded) on any error. */
+export function loadRepoCollapsed(): Set<string> {
+  if (typeof window === "undefined") return new Set();
+  try {
+    const raw = window.localStorage.getItem(REPO_COLLAPSED_STORAGE_KEY);
+    if (raw === null) return new Set();
+    const parsed: unknown = JSON.parse(raw);
+    if (!Array.isArray(parsed)) return new Set();
+    return new Set(parsed.filter((entry): entry is string => typeof entry === "string"));
+  } catch {
+    return new Set();
+  }
+}
+
+/** Persist the per-repo collapse set. Failures are swallowed. */
+export function persistRepoCollapsed(state: ReadonlySet<string>): void {
+  if (typeof window === "undefined") return;
+  try {
+    window.localStorage.setItem(REPO_COLLAPSED_STORAGE_KEY, JSON.stringify([...state]));
+  } catch {
+    // localStorage may be unavailable (private mode) — swallow silently.
+  }
+}
+
+/**
+ * Auto-select the first active skill so the right pane isn't empty on
+ * open. Returns null when the Active section is collapsed (don't select
+ * a row the user can't see) or when there are no active skills.
+ */
+export function pickInitialSelection(skillList: readonly SkillIdentity[], collapsed: ReadonlySet<SkillSectionKey>): string | null {
+  if (skillList.length === 0) return null;
+  if (collapsed.has("active")) return null;
+  return skillList[0].name;
+}

package/src/plugins/manageSkills/meta.ts

@@ -16,6 +16,36 @@ export const META = definePluginMeta({
     update: { method: "PUT", path: "/:name" },
     /** DELETE /api/skills/:name — delete a project-scope skill. */
     remove: { method: "DELETE", path: "/:name" },
+    /** GET /api/skills/catalog — list catalog entries (preset for
+     *  now; anthropic / community land in #1335 PR-C). Catalog
+     *  entries are NOT in `.claude/skills/` and don't enter the
+     *  Claude Code system prompt until the user ★ Stars them. */
+    catalogList: { method: "GET", path: "/catalog" },
+    /** POST /api/skills/catalog/star — body `{ source, slug }`.
+     *  Copies the catalog entry into `.claude/skills/<slug>/` so
+     *  Claude Code's slash-command resolver picks it up. */
+    catalogStar: { method: "POST", path: "/catalog/star" },
+    /** GET /api/skills/catalog/preview?source=&slug= — returns one
+     *  catalog entry's description + body. Used by the 📖 Preview
+     *  modal and the ▶ Run once action (which feeds `body` into a
+     *  fresh chat as user input). For external source, pass `repoId`
+     *  + `skillFolder` instead of `slug`. */
+    catalogPreview: { method: "GET", path: "/catalog/preview" },
+    /** GET /api/skills/external/suggestions — bundled list of repo
+     *  URLs the launcher recommends as starting points (Anthropic
+     *  skills + community picks). */
+    externalSuggestions: { method: "GET", path: "/external/suggestions" },
+    /** GET /api/skills/external/repos — list installed external
+     *  repos with their recorded URL / subpath / SHA. */
+    externalReposList: { method: "GET", path: "/external/repos" },
+    /** POST /api/skills/external/repos — body `{ url, subpath?, ref? }`.
+     *  Clone (or refresh) the repo, copy each discovered SKILL.md
+     *  into the catalog under `<external>/<repoId>/`. */
+    externalReposInstall: { method: "POST", path: "/external/repos" },
+    /** DELETE /api/skills/external/repos/:repoId — remove the
+     *  catalog dir + scratch clone for one external repo. Active
+     *  copies under `.claude/skills/` are NOT touched (star = fork). */
+    externalReposRemove: { method: "DELETE", path: "/external/repos/:repoId" },
   },
   mcpDispatch: "create",
 });

package/src/plugins/markdown/definition.ts

@@ -17,9 +17,9 @@ export interface MarkdownToolData {
 /** True when the `markdown` field is a workspace-relative file path
  *  rather than inline content. Accepts only the canonical
  *  `artifacts/documents/*.md` prefix now that server-side
- *  `isMarkdownPath` agrees. Any legacy `markdowns/*.md` references
- *  in old session JSONL must be migrated via
- *  `scripts/migrate-legacy-artifact-paths.ts` (#773). */
+ *  `isMarkdownPath` agrees. Legacy `markdowns/*.md` references in
+ *  old session JSONL are no longer auto-resolved — those sessions
+ *  render their markdown content as plain text. */
 export function isFilePath(value: string): boolean {
   if (!value.endsWith(".md")) return false;
   return value.startsWith("artifacts/documents/");

package/src/plugins/meta-types.ts

@@ -80,6 +80,11 @@ export interface PluginMeta {
    *  live as separate named exports in the plugin's `meta.ts`
    *  because their signatures are plugin-specific. */
   readonly staticChannels?: Readonly<Record<string, string>>;
+  /** Optional host binaries this plugin's features need (ids from
+   *  the optional-deps registry, e.g. `["ffmpeg"]`). When any is
+   *  missing the host warns the user once and the plugin's
+   *  dependency-bound features degrade gracefully — see #1385. */
+  readonly requires?: readonly string[];
 }
 
 /** Substitute `:param` placeholders in a route URL with caller-

package/src/plugins/presentMulmoScript/Preview.vue

@@ -1,9 +1,9 @@
 <template>
-  <div class="p-2 text-sm">
-    <div class="font-medium text-gray-700 truncate mb-1">
+  <div class="p-2 text-sm" data-testid="mulmo-script-preview">
+    <div class="font-medium text-gray-700 truncate mb-1" data-testid="mulmo-script-preview-title">
       {{ title }}
     </div>
-    <div v-if="description" class="text-xs text-gray-500 leading-relaxed">
+    <div v-if="description" class="text-xs text-gray-500 leading-relaxed" data-testid="mulmo-script-preview-description">
       {{ description }}
     </div>
   </div>

package/src/plugins/presentMulmoScript/View.vue

@@ -3,10 +3,10 @@
     <!-- Header -->
     <div class="flex items-start justify-between px-6 py-4 border-b border-gray-100 shrink-0">
       <div class="min-w-0 flex-1">
-        <h2 class="text-lg font-semibold text-gray-800 truncate">
+        <h2 class="text-lg font-semibold text-gray-800 truncate" data-testid="mulmo-script-title">
           {{ script.title || "Untitled Script" }}
         </h2>
-        <p v-if="script.description" class="text-sm text-gray-500 mt-0.5 truncate">
+        <p v-if="script.description" class="text-sm text-gray-500 mt-0.5 truncate" data-testid="mulmo-script-description">
           {{ script.description }}
         </p>
         <div class="flex items-center gap-3 mt-1 text-xs text-gray-400">
@@ -89,6 +89,33 @@
       </div>
     </div>
 
+    <!--
+      Inline error chip for movie-generation failures (#1197).
+      Previously the catch arm of `generateMovie` raised an `alert()` —
+      blocking, no retry path, and many users just dismissed the modal
+      and saw a stalled spinner with no explanation. The chip stays
+      visible until the next generate attempt clears it.
+    -->
+    <div
+      v-if="movieError"
+      data-testid="mulmo-script-movie-error-chip"
+      class="bg-red-50 border border-red-200 text-red-800 text-xs px-3 py-2 mx-4 mt-3 mb-1 rounded flex items-start gap-2"
+    >
+      <span class="material-icons text-base shrink-0 mt-px">error_outline</span>
+      <div class="flex-1 min-w-0">
+        <div class="font-medium">{{ t("pluginMulmoScript.movieGenerationFailed") }}</div>
+        <div class="break-words whitespace-pre-wrap mt-0.5">{{ movieError }}</div>
+      </div>
+      <button
+        class="shrink-0 h-7 px-2 text-xs rounded border border-red-300 text-red-700 hover:bg-red-100 disabled:opacity-50"
+        :disabled="movieGenerating"
+        data-testid="mulmo-script-movie-retry-button"
+        @click="generateMovie"
+      >
+        {{ t("pluginMulmoScript.retry") }}
+      </button>
+    </div>
+
     <!-- Characters section -->
     <div v-if="characterKeys.length > 0" class="border-b border-gray-100 shrink-0 px-4 py-3">
       <div class="flex items-center justify-between mb-2">
@@ -459,6 +486,11 @@ interface Beat {
   id?: string;
   imagePrompt?: string;
   image?: { type: string; [key: string]: unknown };
+  /** Beat duration in seconds. The mulmocast schema notes this is
+   *  "Used only when the text is empty" — when there's no TTS audio
+   *  to drive playback, the Play loop uses this as the auto-advance
+   *  timer (#1073). */
+  duration?: number;
 }
 
 interface ImageEntry {
@@ -510,11 +542,27 @@ const localOverrides = reactive<Record<number, Beat>>({});
 const movieGenerating = ref(false);
 const movieDownloading = ref(false);
 const moviePath = ref<string | null>(null);
+// Persists the most-recent movie-generation failure so the spinner
+// area can surface it inline with a retry button (#1197). Cleared
+// at the start of every generate / regenerate attempt.
+const movieError = ref<string | null>(null);
 const beatAudios = reactive<Record<number, string>>({});
 const audioState = reactive<Record<number, "generating" | "done" | "error">>({});
 const audioErrors = reactive<Record<number, string>>({});
 const playingAudio = ref<{ index: number; audio: HTMLAudioElement } | null>(null);
+// Tracks the auto-advance timer running on a silent beat
+// (`beat.text === ""`). Beats without text generate no audio, so the
+// Play loop falls back to a `setTimeout(beat.duration)` for cues —
+// without this, Play would stall on the first silent beat (#1073).
+const silentPlaybackTimer = ref<{ index: number; timer: ReturnType<typeof setTimeout> } | null>(null);
 const audioProgress = ref(0);
+
+// Default duration (seconds) for a silent beat whose script doesn't
+// set `duration` either. Picked to roughly match the time it takes a
+// reader to scan a `textSlide` — long enough to read, short enough
+// not to feel stuck. The script's own `duration` always wins.
+const SILENT_BEAT_DEFAULT_SEC = 3;
+const MS_PER_SECOND = 1000;
 const beatListEl = ref<HTMLElement | null>(null);
 const lightbox = ref<{
   src: string;
@@ -564,10 +612,11 @@ function characterPrompt(key: string): string {
 }
 
 function stopPlayingAudio() {
-  if (!playingAudio.value) return;
-  playingAudio.value.audio.pause();
-  playingAudio.value = null;
-  audioProgress.value = 0;
+  // Single helper that clears both the audio path and the silent
+  // auto-advance timer — callers (lightbox open / arrow nav / Stop
+  // button) get consistent behaviour without remembering which
+  // playback mode the current beat was using (#1073).
+  stopAllPlayback();
 }
 
 function openLightbox(index: number) {
@@ -614,7 +663,76 @@ const isPlayReady = computed<boolean>(() => {
 function playPresentation() {
   if (!isPlayReady.value) return;
   openLightbox(0);
-  if (beatAudios[0]) playAudio(0);
+  playBeat(0);
+}
+
+// Stop whichever playback handle is active. Idempotent. Called by
+// openLightbox, manual stop / pause buttons, and by `playBeat`
+// before kicking off a new beat so we never double-schedule. (#1073)
+function stopAllPlayback(): void {
+  if (playingAudio.value) {
+    playingAudio.value.audio.pause();
+    playingAudio.value = null;
+    audioProgress.value = 0;
+  }
+  if (silentPlaybackTimer.value) {
+    clearTimeout(silentPlaybackTimer.value.timer);
+    silentPlaybackTimer.value = null;
+  }
+}
+
+// Single entry point for "start playback at beat <index>". Routes
+// on what the script DECLARED, not on what's currently hydrated:
+//
+//   - `text` empty  → silent path (`scheduleSilentAdvance`). The
+//     schema says no audio is generated for empty-text beats, so
+//     `duration` drives auto-advance.
+//   - `text` present + audio loaded → audio path. `audio.ended`
+//     chains via `advanceFromBeat`.
+//   - `text` present + audio NOT loaded → stop. The Play button's
+//     `isPlayReady` gate prevented this for beat 0, but mid-stream
+//     a transient fetch miss must not silently skip the narration
+//     by falling through to the silent timer (Codex review on
+//     #1073 — gating on `beatAudios[index]` would do exactly that).
+//
+// Either path chains to the next beat via `advanceFromBeat`, so a
+// run of silent beats — or audio / silent / audio sequences —
+// plays through without manual interaction.
+function playBeat(index: number): void {
+  stopAllPlayback();
+  const hasText = Boolean(effectiveBeat(index).text);
+  if (!hasText) {
+    scheduleSilentAdvance(index);
+    return;
+  }
+  if (beatAudios[index]) {
+    playAudio(index);
+  }
+  // Text beat with no audio yet → stop. The user can re-click Play
+  // once the audio finishes hydrating.
+}
+
+function scheduleSilentAdvance(index: number): void {
+  // Defensively narrow the script-supplied duration. A bad value
+  // (zero, negative, NaN, non-number) would otherwise collapse to
+  // an immediate timeout and the Play loop would race through every
+  // silent beat in a single tick (Codex review iter-5 on #1365).
+  // Falling back to the default keeps the presentation watchable.
+  const raw = effectiveBeat(index).duration;
+  const seconds = typeof raw === "number" && Number.isFinite(raw) && raw > 0 ? raw : SILENT_BEAT_DEFAULT_SEC;
+  const timer = setTimeout(() => {
+    if (silentPlaybackTimer.value?.index !== index) return;
+    silentPlaybackTimer.value = null;
+    if (lightbox.value?.index === index) advanceFromBeat(index);
+  }, seconds * MS_PER_SECOND);
+  silentPlaybackTimer.value = { index, timer };
+}
+
+function advanceFromBeat(fromIndex: number): void {
+  lightboxMove(1);
+  const nextIndex = lightbox.value?.index;
+  if (nextIndex === undefined || nextIndex === fromIndex) return;
+  playBeat(nextIndex);
 }
 
 const hasPrev = computed(() => {
@@ -637,11 +755,12 @@ function jumpToBeat(index: number) {
   if (!lightbox.value) return;
   if (index === lightbox.value.index) return;
   if (!renderedImages[index]) return;
-  const wasPlaying = playingAudio.value !== null;
+  // Carry the playback mode forward (audio OR silent timer) so a
+  // user clicking the beat-strip thumbnail mid-playback keeps the
+  // presentation rolling (#1073).
+  const wasPlaying = playingAudio.value !== null || silentPlaybackTimer.value !== null;
   openLightbox(index);
-  if (wasPlaying && beatAudios[index]) {
-    playAudio(index);
-  }
+  if (wasPlaying) playBeat(index);
 }
 
 function beatTooltip(index: number): string {
@@ -652,20 +771,19 @@ function beatTooltip(index: number): string {
 function lightboxMove(delta: number) {
   if (!lightbox.value) return;
   const total = beats.value.length;
-  // If audio was playing when the user clicked the arrow, carry the
-  // playback over to the next beat that has audio. openLightbox()
-  // unconditionally stops any active audio, so we capture the flag
-  // BEFORE that and replay AFTER. The on-ended auto-advance path
-  // already nulls playingAudio before calling lightboxMove, so this
-  // branch won't double-fire there.
-  const wasPlaying = playingAudio.value !== null;
+  // If a playback was in progress when the user clicked the arrow,
+  // carry it forward to whichever beat we land on — `playBeat`
+  // picks audio vs silent automatically. `openLightbox` stops the
+  // current playback, so capture the flag BEFORE that and chain
+  // AFTER. The on-ended / silent-advance paths already null their
+  // own state before calling `lightboxMove`, so this branch won't
+  // double-fire there.
+  const wasPlaying = playingAudio.value !== null || silentPlaybackTimer.value !== null;
   let i = lightbox.value.index + delta;
   while (i >= 0 && i < total) {
     if (renderedImages[i]) {
       openLightbox(i);
-      if (wasPlaying && beatAudios[i]) {
-        playAudio(i);
-      }
+      if (wasPlaying) playBeat(i);
       return;
     }
     i += delta;
@@ -911,13 +1029,7 @@ function playAudio(index: number) {
     if (playingAudio.value?.index !== index) return;
     playingAudio.value = null;
     audioProgress.value = 0;
-    if (lightbox.value?.index === index) {
-      lightboxMove(1);
-      const nextIndex = lightbox.value?.index;
-      if (nextIndex !== undefined && nextIndex !== index && beatAudios[nextIndex]) {
-        playAudio(nextIndex);
-      }
-    }
+    if (lightbox.value?.index === index) advanceFromBeat(index);
   });
   audio.play();
 }
@@ -1019,10 +1131,9 @@ async function onCharDrop(event: DragEvent, key: string) {
 }
 
 function openCharacterLightbox(key: string) {
-  if (playingAudio.value) {
-    playingAudio.value.audio.pause();
-    playingAudio.value = null;
-  }
+  // Stop both audio and silent timer — character lightbox is
+  // outside the play loop (#1073).
+  stopAllPlayback();
   lightbox.value = {
     src: charImages[key],
     text: key,
@@ -1135,6 +1246,15 @@ async function refreshScriptFromDisk(): Promise<void> {
 }
 
 async function initializeScript() {
+  // Stop any in-flight playback BEFORE we tear down per-script state
+  // — a pending `silentPlaybackTimer` or running audio from the
+  // previous script would otherwise fire `advanceFromBeat()` against
+  // the new script's lightbox / beat list and either crash or
+  // silently jump the new presentation forward. Also close any open
+  // lightbox so the user lands on the clean View for the new result
+  // (Codex review iter-4 on #1365).
+  stopAllPlayback();
+  lightbox.value = null;
   // Reset scroll position so new results start at the top
   if (beatListEl.value) beatListEl.value.scrollTop = 0;
   // Reset per-script state
@@ -1275,6 +1395,7 @@ async function refreshMoviePath(): Promise<void> {
 
 async function generateMovie() {
   movieGenerating.value = true;
+  movieError.value = null;
   try {
     const res = await apiFetchRaw(endpoints.generateMovie.url, {
       method: "POST",
@@ -1296,7 +1417,10 @@ async function generateMovie() {
       },
     });
   } catch (err) {
-    alert(extractErrorMessage(err));
+    // Surface inline (instead of `alert()` which blocks + has no
+    // retry affordance). The error chip with a retry button lives
+    // next to the generate button in the template (#1197).
+    movieError.value = extractErrorMessage(err);
   } finally {
     movieGenerating.value = false;
   }

package/src/plugins/presentMulmoScript/meta.ts

@@ -49,4 +49,8 @@ export const META = definePluginMeta({
     downloadMovie: { method: "GET", path: "/download-movie" },
   },
   mcpDispatch: "save",
+  // mulmocast shells out to ffmpeg for movie/beat rendering. Without
+  // it the editor still works but render/generate-movie degrades —
+  // the host warns once at boot (#1385).
+  requires: ["ffmpeg"],
 });

package/src/plugins/scheduler/View.vue

@@ -127,12 +127,13 @@
               <div
                 v-for="item in itemsForDay(day)"
                 :key="item.id"
-                class="text-xs px-1.5 py-0.5 rounded cursor-pointer truncate"
-                :class="selectedId === item.id ? 'bg-blue-500 text-white' : 'bg-blue-100 text-blue-800 hover:bg-blue-200'"
-                :title="item.title"
+                class="text-xs px-1.5 py-0.5 cursor-pointer truncate"
+                :class="[segmentClasses(item, day), selectedId === item.id ? 'bg-blue-500 text-white' : chipColorClasses(item)]"
+                :title="chipTitle(item)"
                 @click="selectItem(item)"
               >
-                <span v-if="itemTime(item)" class="font-medium">{{ itemTime(item) }} </span>{{ item.title }}
+                <span v-if="isBrokenChip(item)" class="font-medium">⚠ </span><span v-else-if="itemTime(item)" class="font-medium">{{ itemTime(item) }} </span
+                >{{ item.title }}
               </div>
             </div>
           </div>
@@ -173,12 +174,12 @@
               <div
                 v-for="item in itemsForDay(day).slice(0, MAX_MONTH_ITEMS)"
                 :key="item.id"
-                class="text-[10px] leading-tight px-1 py-0.5 rounded cursor-pointer truncate"
-                :class="selectedId === item.id ? 'bg-blue-500 text-white' : 'bg-blue-100 text-blue-800 hover:bg-blue-200'"
-                :title="item.title"
+                class="text-[10px] leading-tight px-1 py-0.5 cursor-pointer truncate"
+                :class="[segmentClasses(item, day), selectedId === item.id ? 'bg-blue-500 text-white' : chipColorClasses(item)]"
+                :title="chipTitle(item)"
                 @click="selectItem(item)"
               >
-                {{ item.title }}
+                <span v-if="isBrokenChip(item)" class="font-medium">⚠ </span>{{ item.title }}
               </div>
               <div v-if="itemsForDay(day).length > MAX_MONTH_ITEMS" class="text-[10px] text-gray-400 px-1">
                 {{ t("pluginScheduler.moreCount", { count: itemsForDay(day).length - MAX_MONTH_ITEMS }) }}
@@ -261,6 +262,7 @@ import TasksTab from "./TasksTab.vue";
 import { isToday, formatShortDate, formatMonthYear } from "../../utils/format/date";
 import { errorMessage } from "../../utils/errors";
 import { SCHEDULER_VIEW, SCHEDULER_VIEW_MODES as VIEW_MODES, SCHEDULER_TAB, type SchedulerViewMode as ViewMode, type SchedulerTab } from "./viewModes";
+import { coversDay, eventColorClasses, isMalformedRange, segmentPosition, type SegmentPosition } from "./multiDayHelpers";
 
 const { t } = useI18n();
 
@@ -367,7 +369,41 @@ function toDateString(date: Date): string {
 
 function itemsForDay(day: Date): ScheduledItem[] {
   const dateStr = toDateString(day);
-  return items.value.filter((item) => String(item.props.date) === dateStr);
+  return items.value.filter((item) => coversDay(item, dateStr));
+}
+
+const SEGMENT_BASE: Record<SegmentPosition, string> = {
+  only: "rounded",
+  start: "rounded-l",
+  middle: "",
+  end: "rounded-r",
+};
+
+function segmentClasses(item: ScheduledItem, day: Date): string {
+  const pos = segmentPosition(item, toDateString(day));
+  return pos ? SEGMENT_BASE[pos] : "rounded";
+}
+
+// Red dashed outline + warning-amber background screams "this is
+// wrong, click and fix it." Returns a class string when the event
+// has a broken range; empty when the event is well-formed and the
+// per-event palette colour should apply.
+const BROKEN_CLASSES = "bg-red-50 text-red-900 hover:bg-red-100 border border-dashed border-red-400";
+
+function chipColorClasses(item: ScheduledItem): string {
+  if (isMalformedRange(item)) return BROKEN_CLASSES;
+  return eventColorClasses(item.id);
+}
+
+function chipTitle(item: ScheduledItem): string {
+  if (isMalformedRange(item)) {
+    return `⚠ ${t("pluginScheduler.invalidRange", { endDate: String(item.props.endDate) })} — ${item.title}`;
+  }
+  return item.title;
+}
+
+function isBrokenChip(item: ScheduledItem): boolean {
+  return isMalformedRange(item);
 }
 
 const unscheduledItems = computed(() => items.value.filter((item) => !item.props.date));

package/src/plugins/scheduler/calendarDefinition.ts

@@ -11,9 +11,10 @@ const toolDefinition: ToolDefinition = {
   prompt:
     "When users mention calendar events, appointments, meetings, or one-off reminders that have a date/time, use manageCalendar. " +
     "Use show to display the calendar, add to create an event, update to edit one, delete to remove one. " +
+    "Multi-day events (trips, conferences, vacations) set both `date` (start, inclusive) and `endDate` (end, inclusive) in `props`, both as `YYYY-MM-DD`. " +
     "For recurring automated tasks driven by a schedule (e.g. 'every morning at 8 fetch news'), use manageAutomations instead.",
   description:
-    "Manage the user's calendar — show / add / update / delete dated calendar items. Calendar items have a title and free-form properties (date, time, location, …).",
+    "Manage the user's calendar — show / add / update / delete dated calendar items. Calendar items have a title and free-form properties (date, time, location, …); multi-day events also set endDate.",
   parameters: {
     type: "object",
     properties: {
@@ -32,7 +33,10 @@ const toolDefinition: ToolDefinition = {
       },
       props: {
         type: "object",
-        description: "For 'add': initial properties (e.g. { date, time, location }). For 'update': properties to merge in; set a key to null to remove it.",
+        description:
+          "For 'add': initial properties (e.g. { date, time, location, endDate }). " +
+          "`date` and `endDate` are ISO `YYYY-MM-DD`; `endDate` is the inclusive last day of a multi-day event (omit for single-day events). " +
+          "For 'update': properties to merge in; set a key to null to remove it.",
         additionalProperties: true,
       },
     },