mulmoclaude 0.6.0 → 0.6.2

package/src/plugins/_generated/server-bindings.ts

@@ -10,9 +10,11 @@ import chartDef from "../chart/definition";
 import manageSkillsDef from "../manageSkills/definition";
 import manageSourceDef from "../manageSource/definition";
 import markdownDef from "../markdown/definition";
+import photoLocationsDef from "../photoLocations/definition";
 import presentFormDef from "../presentForm/definition";
 import presentHtmlDef from "../presentHtml/definition";
 import presentMulmoScriptDef from "../presentMulmoScript/definition";
+import presentSVGDef from "../presentSVG/definition";
 import schedulerAutomationsDef from "../scheduler/automationsDefinition";
 import schedulerCalendarDef from "../scheduler/calendarDefinition";
 import spreadsheetDef from "../spreadsheet/definition";
@@ -22,9 +24,11 @@ import { META as chartMeta } from "../chart/meta";
 import { META as manageSkillsMeta } from "../manageSkills/meta";
 import { META as manageSourceMeta } from "../manageSource/meta";
 import { META as markdownMeta } from "../markdown/meta";
+import { META as photoLocationsMeta } from "../photoLocations/meta";
 import { META as presentFormMeta } from "../presentForm/meta";
 import { META as presentHtmlMeta } from "../presentHtml/meta";
 import { META as presentMulmoScriptMeta } from "../presentMulmoScript/meta";
+import { META as presentSVGMeta } from "../presentSVG/meta";
 import { META as schedulerCalendarMeta } from "../scheduler/calendarMeta";
 import { META as spreadsheetMeta } from "../spreadsheet/meta";
 
@@ -38,9 +42,11 @@ export const GENERATED_SERVER_BINDINGS: readonly ServerPluginBinding[] = [
   { def: manageSkillsDef, endpoint: mcpEndpoint(manageSkillsMeta) },
   { def: manageSourceDef, endpoint: mcpEndpoint(manageSourceMeta) },
   { def: markdownDef, endpoint: mcpEndpoint(markdownMeta) },
+  { def: photoLocationsDef, endpoint: mcpEndpoint(photoLocationsMeta) },
   { def: presentFormDef, endpoint: mcpEndpoint(presentFormMeta) },
   { def: presentHtmlDef, endpoint: mcpEndpoint(presentHtmlMeta) },
   { def: presentMulmoScriptDef, endpoint: mcpEndpoint(presentMulmoScriptMeta) },
+  { def: presentSVGDef, endpoint: mcpEndpoint(presentSVGMeta) },
   { def: schedulerAutomationsDef, endpoint: mcpEndpoint(schedulerCalendarMeta) },
   { def: schedulerCalendarDef, endpoint: mcpEndpoint(schedulerCalendarMeta) },
   { def: spreadsheetDef, endpoint: mcpEndpoint(spreadsheetMeta) },

package/src/plugins/accounting/Preview.vue

@@ -12,6 +12,7 @@
 <script setup lang="ts">
 import { computed } from "vue";
 import { useI18n } from "vue-i18n";
+import { formatAmountNumeric } from "./currencies";
 
 const { t } = useI18n();
 
@@ -34,10 +35,6 @@ interface BookLike {
   book?: { id?: string; name?: string };
 }
 
-function formatAmount(value: number): string {
-  return value.toLocaleString(undefined, { minimumFractionDigits: 2, maximumFractionDigits: 2 });
-}
-
 // Each summarise* helper returns null when its branch doesn't apply,
 // keeping the dispatch in `summary` linear (no nested if-trees).
 
@@ -65,7 +62,7 @@ function summarisePl(json: Record<string, unknown>): string | null {
   return t("pluginAccounting.preview.pl", {
     from: profitLoss.from ?? "?",
     to: profitLoss.to ?? "?",
-    net: formatAmount(profitLoss.netIncome),
+    net: formatAmountNumeric(profitLoss.netIncome),
   });
 }
 
@@ -75,7 +72,7 @@ function summariseBs(json: Record<string, unknown>): string | null {
   const assets = balanceSheet.sections.find((section) => section.type === "asset");
   return t("pluginAccounting.preview.bs", {
     date: balanceSheet.asOf,
-    assets: assets ? formatAmount(assets.total ?? 0) : "?",
+    assets: assets ? formatAmountNumeric(assets.total ?? 0) : "?",
   });
 }
 

package/src/plugins/accounting/View.vue

@@ -138,6 +138,7 @@ import BookSettings from "./components/BookSettings.vue";
 import { getOpeningBalances, getAccounts, getBooks, type Account, type BookSummary } from "./api";
 import { ACCOUNTING_ACTIONS } from "./actions";
 import { useAccountingChannel, useAccountingBooksChannel } from "../../composables/useAccountingChannel";
+import { errorMessage } from "../../utils/errors";
 
 const { t } = useI18n();
 
@@ -318,7 +319,7 @@ async function refetchBooks(): Promise<void> {
       showFirstRunForm.value = true;
     }
   } catch (err) {
-    bookLoadError.value = err instanceof Error ? err.message : String(err);
+    bookLoadError.value = errorMessage(err);
   } finally {
     loadingBooks.value = false;
   }

package/src/plugins/accounting/components/AccountsModal.vue

@@ -73,6 +73,7 @@ import AccountEditor from "./AccountEditor.vue";
 import type { AccountDraft } from "./accountDraft";
 import { validateAccountDraft, type AccountValidationError } from "./accountValidation";
 import { suggestNextCode } from "./accountNumbering";
+import { errorMessage } from "../../../utils/errors";
 
 const { t } = useI18n();
 
@@ -222,7 +223,7 @@ async function onSave(next: AccountDraft): Promise<void> {
     // result.ok=false, so this is a belt-and-braces guard against
     // a runtime failure that would otherwise leave the Save button
     // stuck on "Saving…".
-    error.value = err instanceof Error ? err.message : String(err);
+    error.value = errorMessage(err);
   } finally {
     saving.value = false;
   }
@@ -268,7 +269,7 @@ async function onToggleActive(account: Account): Promise<void> {
     }
     emit("changed");
   } catch (err) {
-    toggleError.value = err instanceof Error ? err.message : String(err);
+    toggleError.value = errorMessage(err);
   } finally {
     toggleSaving.value = false;
   }

package/src/plugins/accounting/components/JournalEntryForm.vue

@@ -182,6 +182,7 @@ import { localDateString } from "../dates";
 import { countryHasFeature, type SupportedCountryCode } from "../countries";
 import { isTaxAccountCode } from "./accountNumbering";
 import AccountsModal from "./AccountsModal.vue";
+import { errorMessage } from "../../../utils/errors";
 
 const { t } = useI18n();
 
@@ -413,7 +414,7 @@ async function onSubmit(): Promise<void> {
     // belt-and-braces guard against a runtime failure leaving the
     // submit button stuck — the user gets a visible error
     // instead of an unhandled rejection.
-    error.value = err instanceof Error ? err.message : String(err);
+    error.value = errorMessage(err);
   } finally {
     submitting.value = false;
   }

package/src/plugins/accounting/components/JournalList.vue

@@ -207,6 +207,7 @@ import type { SupportedCountryCode } from "../countries";
 import { useLatestRequest } from "./useLatestRequest";
 import DateRangePicker from "./DateRangePicker.vue";
 import JournalEntryForm from "./JournalEntryForm.vue";
+import { errorMessage } from "../../../utils/errors";
 
 const { t } = useI18n();
 
@@ -460,7 +461,7 @@ async function onVoid(entry: JournalEntry): Promise<void> {
     const result = await voidEntry({ entryId: entry.id, reason: reason || undefined, bookId: props.bookId });
     if (!result.ok) error.value = result.error;
   } catch (err) {
-    error.value = err instanceof Error ? err.message : String(err);
+    error.value = errorMessage(err);
   }
 }
 

package/src/plugins/accounting/components/OpeningBalancesForm.vue

@@ -98,6 +98,7 @@ import { formatAmount, inputStepFor } from "../currencies";
 import { localDateString } from "../dates";
 import { useLatestRequest } from "./useLatestRequest";
 import AccountsModal from "./AccountsModal.vue";
+import { errorMessage } from "../../../utils/errors";
 
 const { t } = useI18n();
 
@@ -238,7 +239,7 @@ async function onSubmit(): Promise<void> {
     successMessage.value = t("pluginAccounting.openingForm.success");
     emit("submitted");
   } catch (err) {
-    error.value = err instanceof Error ? err.message : String(err);
+    error.value = errorMessage(err);
   } finally {
     submitting.value = false;
   }

package/src/plugins/accounting/currencies.ts

@@ -62,3 +62,16 @@ export function formatAmount(value: number, currency: string, locale?: string):
     return value.toFixed(fractionDigitsFor(currency));
   }
 }
+
+/** Currency-agnostic amount formatter — "1,130.00" — for places that
+ *  don't carry the currency code on the data path (compact preview
+ *  envelopes etc.). Use `formatAmount(value, currency)` whenever the
+ *  currency IS available — the currency-aware path picks the right
+ *  fraction-digit count automatically (JPY = 0, USD = 2).
+ *
+ *  `locale` mirrors `formatAmount`'s signature: pass an explicit BCP-47
+ *  locale (`"en-US"`, `"ja-JP"`, …) when the caller knows the desired
+ *  grouping / digit-shape; omit to fall back to the runtime default. */
+export function formatAmountNumeric(value: number, decimals = 2, locale?: string): string {
+  return value.toLocaleString(locale, { minimumFractionDigits: decimals, maximumFractionDigits: decimals });
+}

package/src/plugins/manageRoles/View.vue

@@ -297,18 +297,29 @@ interface PluginEntry {
 
 // Plugins the user can assign — exclude internal/auto-managed ones
 const EXCLUDED = new Set(["text-response"]);
-const guiPlugins: PluginEntry[] = pluginAllPluginNames()
-  .filter((name) => !EXCLUDED.has(name))
-  .map((name) => ({ name, enabled: true, requiredEnv: [] }));
+// `pluginAllPluginNames()` reads from the reactive `runtimeRegistry`
+// (src/tools/runtimeLoader.ts). Wrapping in `computed` makes this
+// re-evaluate when `loadRuntimePlugins` populates the registry
+// post-mount — runtime plugins (e.g. server-only `edgar`) only
+// become visible to the role editor through this reactive read.
+// Snapshotting once into a non-reactive const meant a setup() that
+// ran before the loader resolved would never see them.
+const guiPlugins = computed<PluginEntry[]>(() =>
+  pluginAllPluginNames()
+    .filter((name) => !EXCLUDED.has(name))
+    .map((name) => ({ name, enabled: true, requiredEnv: [] })),
+);
+
+const mcpTools = ref<PluginEntry[]>([]);
 
-const availablePlugins = ref<PluginEntry[]>(guiPlugins);
+const availablePlugins = computed<PluginEntry[]>(() => [...guiPlugins.value, ...mcpTools.value]);
 
 const mcpEndpoints = pluginEndpoints<{ list: string }>("mcpTools");
 
 onMounted(async () => {
   const result = await apiGet<PluginEntry[]>(mcpEndpoints.list);
   if (result.ok) {
-    availablePlugins.value = [...guiPlugins, ...result.data];
+    mcpTools.value = result.data;
   }
   // Non-critical: MCP tools enrich the plugin palette for role editing
   // but the view works fine with GUI plugins alone. No error banner needed.

package/src/plugins/manageSkills/View.vue

@@ -129,8 +129,15 @@
             </div>
           </div>
           <!-- View mode -->
-          <!-- eslint-disable-next-line vue/no-v-html -- sanitized via DOMPurify -->
-          <div v-else-if="detail && renderedBody" class="markdown-content text-gray-700" data-testid="skill-body-rendered" v-html="renderedBody"></div>
+          <!-- eslint-disable vue/no-v-html -- sanitized via DOMPurify; multi-line element so disable/enable pair (CLAUDE.md UI rule) instead of -next-line -->
+          <div
+            v-else-if="detail && renderedBody"
+            class="markdown-content text-gray-700"
+            data-testid="skill-body-rendered"
+            @click="handleExternalLinkClick"
+            v-html="renderedBody"
+          ></div>
+          <!-- eslint-enable vue/no-v-html -->
           <p v-else-if="detail" class="text-sm text-gray-400 italic">{{ t("pluginManageSkills.emptyBody") }}</p>
         </div>
       </div>
@@ -142,11 +149,12 @@
 import { computed, onMounted, ref, watch } from "vue";
 import { useI18n } from "vue-i18n";
 import { marked } from "marked";
-import DOMPurify from "dompurify";
 import type { ToolResultComplete } from "gui-chat-protocol/vue";
 import type { ManageSkillsData, SkillSummary } from "./index";
 import { useAppApi } from "../../composables/useAppApi";
 import { apiGet, apiPut, apiDelete } from "../../utils/api";
+import { handleExternalLinkClick } from "../../utils/dom/externalLink";
+import { sanitizeMarkdownHtml } from "../../utils/markdown/sanitize";
 import { pluginEndpoints } from "../api";
 import { buildRouteUrl } from "../meta-types";
 import type { SkillsEndpoints } from "./definition";
@@ -184,7 +192,7 @@ const selected = computed(() => skills.value.find((skill) => skill.name === sele
 const renderedBody = computed(() => {
   const body = detail.value?.body;
   if (!body) return "";
-  return DOMPurify.sanitize(marked(body) as string);
+  return sanitizeMarkdownHtml(marked(body) as string);
 });
 
 // Reset the selection when the tool result is replaced (e.g. the

package/src/plugins/markdown/View.vue

@@ -87,6 +87,7 @@ import { rewriteMarkdownImageRefs } from "../../utils/image/rewriteMarkdownImage
 import { findTaskLines, makeTasksInteractive, toggleTaskAt } from "../../utils/markdown/taskList";
 import { usePdfDownload } from "../../composables/usePdfDownload";
 import { apiGet, apiPut } from "../../utils/api";
+import { handleExternalLinkClick } from "../../utils/dom/externalLink";
 import { pluginEndpoints } from "../api";
 import { useClipboardCopy } from "../../composables/useClipboardCopy";
 import { buildPdfFilename } from "../../utils/files/filename";
@@ -354,6 +355,11 @@ async function persistTaskMarkdown(relativePath: string, markdown: string): Prom
 }
 
 function onMarkdownClick(event: MouseEvent): void {
+  // External http(s) links: open in a new tab instead of letting the
+  // SPA navigate away. Same handler the wiki / textResponse renders
+  // use; without it, clicking an external link from a markdown file
+  // tore the user out of MulmoClaude (#1221).
+  if (handleExternalLinkClick(event)) return;
   const { target } = event;
   if (!(target instanceof HTMLInputElement)) return;
   if (target.type !== "checkbox") return;

package/src/plugins/photoLocations/View.vue

@@ -0,0 +1,231 @@
+<script setup lang="ts">
+// Photo-locations View (#1222 PR-B). A list of every captured
+// sidecar — photo path, lat/lng, takenAt, camera. The map handoff
+// is by chat (the LLM calls `mapControl({action: "addMarker", lat,
+// lng})` once asked) rather than embedded here, so this view stays
+// small + always-loadable regardless of whether the user has a
+// Google Maps API key set.
+
+import { computed, onMounted, ref } from "vue";
+import { useI18n } from "vue-i18n";
+import { apiPost } from "../../utils/api";
+import { pluginEndpoints } from "../api";
+import type { ResolvedRoute } from "../meta-types";
+import { errorMessage as toErrorMessage } from "../../utils/errors";
+import { formatDate } from "../../utils/format/date";
+
+interface Sidecar {
+  version: 1;
+  photo: { relativePath: string; mimeType: string };
+  exif: {
+    lat?: number;
+    lng?: number;
+    altitude?: number;
+    takenAt?: string;
+    make?: string;
+    model?: string;
+    lens?: string;
+  };
+  capturedAt: string;
+}
+
+interface ListedSidecar {
+  id: string;
+  relativePath: string;
+  sidecar: Sidecar;
+}
+
+interface ListResult {
+  message?: string;
+  data?: { locations: ListedSidecar[]; total: number };
+}
+
+interface PhotoLocationsEndpoints {
+  dispatch: ResolvedRoute;
+}
+
+const { t } = useI18n();
+const endpoints = pluginEndpoints<PhotoLocationsEndpoints>("photoLocations");
+
+const locations = ref<ListedSidecar[]>([]);
+const loading = ref(true);
+const errorMessage = ref<string>("");
+
+async function refresh(): Promise<void> {
+  loading.value = true;
+  errorMessage.value = "";
+  try {
+    const result = await apiPost<ListResult>(endpoints.dispatch.url, { kind: "list" });
+    if (!result.ok) throw new Error(result.error);
+    locations.value = result.data.data?.locations ?? [];
+  } catch (err) {
+    errorMessage.value = toErrorMessage(err);
+  } finally {
+    loading.value = false;
+  }
+}
+
+const withGps = computed(() => locations.value.filter((row) => hasFiniteCoords(row.sidecar.exif)));
+
+onMounted(() => {
+  void refresh();
+});
+
+function fmtCoord(value: unknown): string {
+  // The handler validates lat/lng on write, but a hand-edited
+  // sidecar can still ship a string / null past the type — guard
+  // before calling `toFixed` so one bad row doesn't crash the View.
+  // (Codex review on PR #1250.)
+  return typeof value === "number" && Number.isFinite(value) ? value.toFixed(5) : "—";
+}
+
+function fmtAltitude(value: unknown): string | null {
+  return typeof value === "number" && Number.isFinite(value) ? value.toFixed(0) : null;
+}
+
+function fmtDate(iso: string | undefined): string {
+  if (!iso) return "—";
+  return formatDate(iso);
+}
+
+function hasFiniteCoords(exif: { lat?: unknown; lng?: unknown }): boolean {
+  return typeof exif.lat === "number" && Number.isFinite(exif.lat) && typeof exif.lng === "number" && Number.isFinite(exif.lng);
+}
+</script>
+
+<template>
+  <div class="photo-locations-view">
+    <header>
+      <h2>{{ t("photoLocations.title") }}</h2>
+      <span class="count" data-testid="photo-locations-count">{{ t("photoLocations.summary", { total: locations.length, withGps: withGps.length }) }}</span>
+    </header>
+
+    <p class="hint">{{ t("photoLocations.mapHint") }}</p>
+
+    <div v-if="loading" class="message loading">{{ t("photoLocations.loading") }}</div>
+    <div v-else-if="errorMessage" class="message error">⚠ {{ errorMessage }}</div>
+    <div v-else-if="locations.length === 0" class="message empty">{{ t("photoLocations.empty") }}</div>
+
+    <ul v-else class="rows">
+      <li v-for="row in locations" :key="row.id" class="row" :data-testid="`photo-locations-row-${row.id}`">
+        <div class="row-main">
+          <code class="path">{{ row.sidecar.photo.relativePath }}</code>
+          <span class="taken">{{ fmtDate(row.sidecar.exif.takenAt) }}</span>
+        </div>
+        <div class="row-meta">
+          <span v-if="hasFiniteCoords(row.sidecar.exif)" class="coords">
+            <!-- eslint-disable @intlify/vue-i18n/no-raw-text -- coordinates emoji + decimal pair + altitude unit are language-neutral numeric formatters, not user-facing prose -->
+            📍 {{ fmtCoord(row.sidecar.exif.lat) }}, {{ fmtCoord(row.sidecar.exif.lng) }}
+            <span v-if="fmtAltitude(row.sidecar.exif.altitude)" class="altitude">({{ fmtAltitude(row.sidecar.exif.altitude) }}m)</span>
+            <!-- eslint-enable @intlify/vue-i18n/no-raw-text -->
+          </span>
+          <span v-else class="no-gps">{{ t("photoLocations.noGps") }}</span>
+          <span v-if="row.sidecar.exif.model" class="camera">{{ row.sidecar.exif.model }}</span>
+        </div>
+      </li>
+    </ul>
+  </div>
+</template>
+
+<style scoped>
+.photo-locations-view {
+  height: 100%;
+  display: flex;
+  flex-direction: column;
+  gap: 0.5rem;
+  padding: 0.75rem 1rem;
+  font-family: system-ui, sans-serif;
+  overflow-y: auto;
+}
+header {
+  display: flex;
+  align-items: baseline;
+  gap: 0.75rem;
+}
+h2 {
+  margin: 0;
+  font-size: 1rem;
+  font-weight: 600;
+  color: #1f2937;
+}
+.count {
+  font-size: 0.75rem;
+  color: #6b7280;
+}
+.hint {
+  margin: 0;
+  font-size: 0.75rem;
+  color: #6b7280;
+  font-style: italic;
+}
+.message {
+  padding: 0.75rem 1rem;
+  border-radius: 0.5rem;
+  font-size: 0.875rem;
+}
+.message.loading,
+.message.empty {
+  color: #6b7280;
+}
+.message.error {
+  background: #fee2e2;
+  border: 1px solid #fecaca;
+  color: #991b1b;
+}
+.rows {
+  list-style: none;
+  margin: 0;
+  padding: 0;
+  display: flex;
+  flex-direction: column;
+  gap: 0.375rem;
+}
+.row {
+  padding: 0.5rem 0.75rem;
+  border-radius: 0.375rem;
+  background: #f9fafb;
+  border: 1px solid #e5e7eb;
+}
+.row-main {
+  display: flex;
+  justify-content: space-between;
+  gap: 0.5rem;
+  align-items: baseline;
+}
+.path {
+  font-family: monospace;
+  font-size: 0.75rem;
+  color: #374151;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+  flex: 1;
+  min-width: 0;
+}
+.taken {
+  font-size: 0.75rem;
+  color: #6b7280;
+  flex-shrink: 0;
+}
+.row-meta {
+  margin-top: 0.25rem;
+  display: flex;
+  flex-wrap: wrap;
+  gap: 0.5rem;
+  font-size: 0.75rem;
+  color: #4b5563;
+}
+.coords {
+  font-family: monospace;
+}
+.altitude {
+  color: #9ca3af;
+}
+.no-gps {
+  color: #d97706;
+  font-style: italic;
+}
+.camera {
+  color: #6b7280;
+}
+</style>

package/src/plugins/photoLocations/definition.ts

@@ -0,0 +1,47 @@
+// MCP tool definition for `managePhotoLocations` (#1222 PR-B).
+//
+// Two kinds for v1:
+//   - `list`  — every sidecar, newest first.
+//   - `count` — quick scalar for the View's badge.
+//
+// `extractExif` (on-demand re-read for one photo) and `rescan`
+// (backfill sidecars for old uploads) ride a follow-up PR.
+
+import type { ToolDefinition } from "gui-chat-protocol";
+import { META } from "./meta";
+
+export const TOOL_NAME = META.toolName;
+
+export const PHOTO_LOCATIONS_KINDS = {
+  list: "list",
+  count: "count",
+} as const;
+export type PhotoLocationsKind = (typeof PHOTO_LOCATIONS_KINDS)[keyof typeof PHOTO_LOCATIONS_KINDS];
+
+export interface PhotoLocationsArgs {
+  kind: PhotoLocationsKind;
+}
+
+/** Single tool, kind-discriminated. Same shape as
+ *  manageAccounting / manageScheduler so the MCP bridge plugs in
+ *  unchanged. */
+const TOOL_DEFINITION: ToolDefinition = {
+  type: "function",
+  name: TOOL_NAME,
+  description:
+    "Read the photo-location sidecars produced by the EXIF auto-capture hook. " +
+    "Use `list` to fetch every captured location (lat/lng/altitude/takenAt/camera) " +
+    "for queries like 'show last week's photos on a map' — the lat/lng pair is " +
+    'shape-compatible with `mapControl({action: "addMarker"})` so you can hand ' +
+    "the result straight to the Google Map plugin without reshape. " +
+    "Use `count` for a quick scalar.",
+  parameters: {
+    type: "object",
+    properties: {
+      kind: { type: "string", enum: Object.values(PHOTO_LOCATIONS_KINDS) },
+    },
+    required: ["kind"],
+  },
+};
+
+export default TOOL_DEFINITION;

package/src/plugins/photoLocations/index.ts

@@ -0,0 +1,38 @@
+import type { PluginRegistration, ToolPlugin } from "../../tools/types";
+import type { ToolResult } from "gui-chat-protocol";
+import View from "./View.vue";
+import toolDefinition from "./definition";
+import { META } from "./meta";
+import { wrapWithScope } from "../scope";
+import { apiCall } from "../../utils/api";
+import { makeUuid } from "../../utils/id";
+
+// Same one-line pass-through pattern as accounting / scheduler:
+// production tool calls flow Claude → MCP → REST, never through
+// `execute()`. The body satisfies the gui-chat-protocol shape and
+// supports any host that does call it.
+export type PhotoLocationsActionData = Record<string, unknown>;
+
+const photoLocationsPlugin: ToolPlugin<PhotoLocationsActionData> = {
+  toolDefinition,
+
+  async execute(_context, args) {
+    const { method, path } = META.apiRoutes.dispatch;
+    const result = await apiCall<ToolResult<PhotoLocationsActionData>>(`/api/${META.apiNamespace}${path}`, { method, body: args });
+    if (!result.ok) {
+      return { toolName: toolDefinition.name, uuid: makeUuid(), message: result.error };
+    }
+    return { ...result.data, toolName: toolDefinition.name, uuid: result.data.uuid ?? makeUuid() };
+  },
+
+  isEnabled: () => true,
+  generatingMessage: "Reading photo locations…",
+  viewComponent: wrapWithScope("photoLocations", View),
+};
+
+export default photoLocationsPlugin;
+
+export const REGISTRATION: PluginRegistration = {
+  toolName: META.toolName,
+  entry: photoLocationsPlugin,
+};

package/src/plugins/photoLocations/meta.ts

@@ -0,0 +1,35 @@
+// Central-registry metadata for the photo-locations plugin
+// (#1222 PR-B). Owns one tool — `managePhotoLocations` — exposing
+// the sidecars produced by the post-save EXIF hook (PR-A) so the
+// LLM can answer "where were my photos taken" / "show last week's
+// shots on a map" without poking at `data/locations/` directly.
+//
+// `mapControl` (`@gui-chat-plugin/google-map`) is the natural
+// downstream consumer: a sidecar's `lat` / `lng` flow into
+// `mapControl({ action: "addMarker", lat, lng })` without reshape.
+//
+// Browser-safe: no Vue imports, no server-only imports.
+
+import { definePluginMeta } from "../meta-types";
+
+export const META = definePluginMeta({
+  toolName: "managePhotoLocations",
+  apiNamespace: "photoLocations",
+  apiRoutes: {
+    /** POST /api/photoLocations — single dispatch with action
+     *  discriminator. Mirrors the accounting / scheduler
+     *  convention so the MCP bridge plugs in unchanged. */
+    dispatch: { method: "POST", path: "" },
+  },
+  mcpDispatch: "dispatch",
+  // Sidecar storage already lives at the host-level
+  // `WORKSPACE_DIRS.locations` (declared by the host because the
+  // hook runs server-side on every saved attachment). No new
+  // directories belong to this plugin — it's a read surface over
+  // host data. This META therefore omits `workspaceDirs`.
+  staticChannels: {
+    /** Published whenever a sidecar is added / removed so an open
+     *  View refreshes without polling. */
+    locationsChanged: "photoLocations:locations-changed",
+  },
+});