@mulmoclaude/core 5.7.0 → 5.7.1

package/assets/helps/error-recovery.md

@@ -584,6 +584,28 @@ guide it returns says `data/skills/`, this root does have a bridge and the
 problem is something else — check that `schema.json` passed validation (a
 schema that fails is silently skipped at discovery).
 
+## A `schema.json` change is not reflected — it was written with Bash or a script
+
+You changed `data/skills/<slug>/schema.json`, and `getSchema` shows the new
+content, but the collection still behaves as before — the old fields, a missing
+view, or a 404. No tool reported an error.
+
+### Why
+
+The server runs `.claude/skills/<slug>/schema.json`, not the staging copy. The
+skill-bridge hook copies staging files across only when they are written with
+the **Write / Edit** tools. A `schema.json` written through Bash (`cat >`,
+`sed -i`, `python`, `jq`) or by a script is never mirrored, so the two copies
+drift apart silently. When they differ, `getSchema` says so in a
+`manageCollection: NOTE` line above the JSON.
+
+### Fix
+
+Pass the schema to `manageCollection` `putSchema`. It validates the schema,
+writes the staging copy and mirrors it into `.claude/skills/<slug>/` in one
+step, and the collection reloads without a restart. Make every later schema
+change the same way — `getSchema`, edit, `putSchema` — never with Bash.
+
 ## dataSource (CSV) collection reads fail — "DuckDB is unavailable on this host"
 
 A collection whose schema declares `dataSource` (external CSV) reads its rows

package/dist/collection/registry/server/index.cjs

@@ -6,7 +6,7 @@ const require_relPath = require("../../../relPath-CTAjGdCL.cjs");
 const require_utils_fetch = require("../../../utils/fetch.cjs");
 const require_discovery = require("../../../discovery-CwBXvtyn.cjs");
 const require_templatePath = require("../../../templatePath-27vUZowm.cjs");
-const require_server = require("../../../server-ObUrsDs2.cjs");
+const require_server = require("../../../server-CLhww8NL.cjs");
 const require_skill_bridge_index = require("../../../skill-bridge/index.cjs");
 const require_types = require("../../../types-CNqkLT4M.cjs");
 let node_fs = require("node:fs");

package/dist/collection/registry/server/index.js

@@ -4,7 +4,7 @@ import { n as toPosixRelPath } from "../../../relPath-DW8MC8VO.js";
 import { fetchWithTimeout } from "../../../utils/fetch.js";
 import { K as safeRecordId, et as collectionsRegistriesConfigPath, ht as skillsStagingDir, lt as projectSkillsDir, r as loadCollection, rt as getWorkspaceRoot, s as CollectionSchemaZ, st as log, t as acceptParsedSchema } from "../../../discovery-XoYJI2Ot.js";
 import { t as isSafeActionTemplatePath } from "../../../templatePath-k_WNbL_Q.js";
-import { y as ONE_SECOND_MS } from "../../../server-CM5XNicg.js";
+import { y as ONE_SECOND_MS } from "../../../server-CTkTHqEL.js";
 import { claudeSkillDir, dataSkillDir, mirrorSkillWrite } from "../../../skill-bridge/index.js";
 import { n as parseRegistryIndex, r as isRecord, t as OFFICIAL_REGISTRY_NAME } from "../../../types-BKVZrtyc.js";
 import { readFileSync } from "node:fs";

package/dist/collection/server/index.cjs

@@ -3,7 +3,7 @@ const require_root = require("../../root-BvwBUcol.cjs");
 const require_itemId = require("../../itemId-CVrNibwT.cjs");
 const require_discovery = require("../../discovery-CwBXvtyn.cjs");
 const require_templatePath = require("../../templatePath-27vUZowm.cjs");
-const require_server = require("../../server-ObUrsDs2.cjs");
+const require_server = require("../../server-CLhww8NL.cjs");
 exports.APP_MANIFEST_FILE = require_discovery.APP_MANIFEST_FILE;
 exports.BackendUnavailableError = require_discovery.BackendUnavailableError;
 exports.COLLECTION_ROOT_REQUIRED = require_discovery.COLLECTION_ROOT_REQUIRED;

package/dist/collection/server/index.js

@@ -2,5 +2,5 @@ import { t as canonicalRoot } from "../../root-BcS4-Kge.js";
 import { K as COMPUTED_TYPES } from "../../itemId-C9s-Jd5s.js";
 import { $ as collectionChangePayload, A as readItem, B as SCHEMA_FILE, C as CollectionQueryZ, D as generateItemId, E as deleteItem, F as loadAppManifest, G as resolveTemplatePath, H as isUnderRealRoot, I as parseAppManifest, J as BackendUnavailableError, K as safeRecordId, L as sharedItemsPath, M as writeItem, N as APP_MANIFEST_FILE, O as isRegularFile, P as appManifestReason, Q as collectionChangeKey, R as pageFromFullRead, S as compileJsonlQuery, T as MAX_QUERY_ROWS, U as itemFilePath, V as isContainedInRoot, W as resolveDataDir, X as COLLECTION_ROOT_REQUIRED, Y as isBackendUnavailable, _ as dedupeByRecordId, a as toDetail, ct as peekWorkspaceRoot, d as storeFor, dt as setCollectionChangePublisher, ft as setFirestoreAccessor, g as decodeCsvRecordId, h as csvRowToItem, i as resolvePrimaryField, it as hostSupportsSharedCollections, j as resolveCreateItemId, k as listItems, l as collectionWritable, mt as sharedCollectionChangePayload, n as discoverCollections, nt as firestoreHandle, o as toSummary, ot as localCollectionKey, p as MAX_CSV_ROWS, pt as setSharedCollectionsSupport, q as safeSlugName, r as loadCollection, rt as getWorkspaceRoot, s as CollectionSchemaZ, st as log, t as acceptParsedSchema, tt as configureCollectionHost, u as readOnlyRefusal, ut as publishCollectionChange, v as encodeCsvRecordId, w as DEFAULT_QUERY_ROWS, x as compileCsvQuery, y as normalizeCsvValue, z as projectItemFields } from "../../discovery-XoYJI2Ot.js";
 import { i as isSafeTemplatePath, n as isSafeCustomViewI18nPath, r as isSafeCustomViewPath, t as isSafeActionTemplatePath } from "../../templatePath-k_WNbL_Q.js";
-import { A as recordFieldProblem, B as stagedSkillDir, C as applyMutateAction, D as validateCollectionRecords, E as STORE_UNREADABLE, F as buildCollectionActionSeedPrompt, I as promptPathsFor, L as readCustomViewHtml, M as enrichItems, N as runQueryOverRows, O as validateRecordObject, P as buildActionSeedPrompt, R as readCustomViewI18n, S as schemaRelations, T as MAX_RECORD_ISSUES, _ as resolveEvery, a as MAX_UNSELECTIVE_ITEMS, b as computeCollectionIcon, c as deleteCollection, d as computeSuccessor, f as daysInMonth, g as parseCivil, h as maybeSpawnSuccessor, i as MAX_SCHEMA_ISSUES, j as runCollectionQuery, k as compileRecordZ, l as deleteCollectionRefusalMessage, m as isTriggerDue, n as MAX_PUT_ITEMS, o as makeManageCollectionTool, p as formatCivil, r as MAX_PUT_LINT, s as deleteCustomView, t as MAX_ITEMS_FILE_BYTES, u as advanceTriggerDate, v as successorId, w as firstMutateParamProblem, x as buildWorkspaceOntology, z as readSkillTemplate } from "../../server-CM5XNicg.js";
+import { A as recordFieldProblem, B as stagedSkillDir, C as applyMutateAction, D as validateCollectionRecords, E as STORE_UNREADABLE, F as buildCollectionActionSeedPrompt, I as promptPathsFor, L as readCustomViewHtml, M as enrichItems, N as runQueryOverRows, O as validateRecordObject, P as buildActionSeedPrompt, R as readCustomViewI18n, S as schemaRelations, T as MAX_RECORD_ISSUES, _ as resolveEvery, a as MAX_UNSELECTIVE_ITEMS, b as computeCollectionIcon, c as deleteCollection, d as computeSuccessor, f as daysInMonth, g as parseCivil, h as maybeSpawnSuccessor, i as MAX_SCHEMA_ISSUES, j as runCollectionQuery, k as compileRecordZ, l as deleteCollectionRefusalMessage, m as isTriggerDue, n as MAX_PUT_ITEMS, o as makeManageCollectionTool, p as formatCivil, r as MAX_PUT_LINT, s as deleteCustomView, t as MAX_ITEMS_FILE_BYTES, u as advanceTriggerDate, v as successorId, w as firstMutateParamProblem, x as buildWorkspaceOntology, z as readSkillTemplate } from "../../server-CTkTHqEL.js";
 export { APP_MANIFEST_FILE, BackendUnavailableError, COLLECTION_ROOT_REQUIRED, COMPUTED_TYPES, CollectionQueryZ, CollectionSchemaZ, DEFAULT_QUERY_ROWS, MAX_CSV_ROWS, MAX_ITEMS_FILE_BYTES, MAX_PUT_ITEMS, MAX_PUT_LINT, MAX_QUERY_ROWS, MAX_RECORD_ISSUES, MAX_SCHEMA_ISSUES, MAX_UNSELECTIVE_ITEMS, SCHEMA_FILE, STORE_UNREADABLE, acceptParsedSchema, advanceTriggerDate, appManifestReason, applyMutateAction, buildActionSeedPrompt, buildCollectionActionSeedPrompt, buildWorkspaceOntology, canonicalRoot, collectionChangeKey, collectionChangePayload, collectionWritable, compileCsvQuery, compileJsonlQuery, compileRecordZ, computeCollectionIcon, computeSuccessor, configureCollectionHost, csvRowToItem, daysInMonth, decodeCsvRecordId, dedupeByRecordId, deleteCollection, deleteCollectionRefusalMessage, deleteCustomView, deleteItem, discoverCollections, encodeCsvRecordId, enrichItems, firestoreHandle, firstMutateParamProblem, formatCivil, generateItemId, getWorkspaceRoot, hostSupportsSharedCollections, isBackendUnavailable, isContainedInRoot, isRegularFile, isSafeActionTemplatePath, isSafeCustomViewI18nPath, isSafeCustomViewPath, isSafeTemplatePath, isTriggerDue, isUnderRealRoot, itemFilePath, listItems, loadAppManifest, loadCollection, localCollectionKey, log, makeManageCollectionTool, maybeSpawnSuccessor, normalizeCsvValue, pageFromFullRead, parseAppManifest, parseCivil, peekWorkspaceRoot, projectItemFields, promptPathsFor, publishCollectionChange, readCustomViewHtml, readCustomViewI18n, readItem, readOnlyRefusal, readSkillTemplate, recordFieldProblem, resolveCreateItemId, resolveDataDir, resolveEvery, resolvePrimaryField, resolveTemplatePath, runCollectionQuery, runQueryOverRows, safeRecordId, safeSlugName, schemaRelations, setCollectionChangePublisher, setFirestoreAccessor, setSharedCollectionsSupport, sharedCollectionChangePayload, sharedItemsPath, stagedSkillDir, storeFor, successorId, toDetail, toSummary, validateCollectionRecords, validateRecordObject, writeItem };

package/dist/collection/server/schemaReadReply.d.ts

@@ -0,0 +1,4 @@
+/** The reply for `getSchema`: the staging copy when there is one, else the active
+ *  copy; prefixed with a notice when both exist and differ. `null` when neither
+ *  copy could be read. */
+export declare function schemaReadReply(slug: string, staging: string | null, active: string | null): string | null;

package/dist/collection-watchers/index.cjs

@@ -4,7 +4,7 @@ const require_itemId = require("../itemId-CVrNibwT.cjs");
 const require_promptSafety = require("../promptSafety-BGV1wrE6.cjs");
 require("../collection/index.cjs");
 const require_discovery = require("../discovery-CwBXvtyn.cjs");
-const require_server = require("../server-ObUrsDs2.cjs");
+const require_server = require("../server-CLhww8NL.cjs");
 const require_notifier = require("../notifier-B65Mvngb.cjs");
 let node_fs_promises = require("node:fs/promises");
 //#region src/collection-watchers/config.ts

package/dist/collection-watchers/index.js

@@ -3,7 +3,7 @@ import { $ as fieldText, P as isValidCollectionName } from "../itemId-C9s-Jd5s.j
 import { E as whenMatches, f as itemIsDone } from "../promptSafety-B5uGroik.js";
 import "../collection/index.js";
 import { $ as collectionChangePayload, U as itemFilePath, ct as peekWorkspaceRoot, d as storeFor, mt as sharedCollectionChangePayload, n as discoverCollections, nt as firestoreHandle, r as loadCollection, ut as publishCollectionChange } from "../discovery-XoYJI2Ot.js";
-import { h as maybeSpawnSuccessor, m as isTriggerDue } from "../server-CM5XNicg.js";
+import { h as maybeSpawnSuccessor, m as isTriggerDue } from "../server-CTkTHqEL.js";
 import { d as publish, m as updateForPlugin, n as clear, s as listAll } from "../notifier-vAp5t6wL.js";
 import { access } from "node:fs/promises";
 //#region src/collection-watchers/config.ts

package/dist/feeds/server/index.cjs

@@ -5,7 +5,7 @@ const require_root = require("../../root-BvwBUcol.cjs");
 const require_itemId = require("../../itemId-CVrNibwT.cjs");
 const require_project = require("../../project-C1ep9pvo.cjs");
 const require_discovery = require("../../discovery-CwBXvtyn.cjs");
-const require_server = require("../../server-ObUrsDs2.cjs");
+const require_server = require("../../server-CLhww8NL.cjs");
 const require_feeds_paths = require("../paths.cjs");
 const require_ingestTypes = require("../../ingestTypes-DS9HiR2I.cjs");
 const require_notifier = require("../../notifier-B65Mvngb.cjs");

package/dist/feeds/server/index.js

@@ -3,7 +3,7 @@ import { t as canonicalRoot } from "../../root-BcS4-Kge.js";
 import { J as INGEST_KINDS, q as FEED_SCHEDULES } from "../../itemId-C9s-Jd5s.js";
 import { t as mergeIntoExisting } from "../../project-B0j_Vf_Q.js";
 import { W as resolveDataDir, _t as createForwardingLogger, d as storeFor, k as listItems, n as discoverCollections, q as safeSlugName, vt as createHostSlot } from "../../discovery-XoYJI2Ot.js";
-import { F as buildCollectionActionSeedPrompt, I as promptPathsFor, z as readSkillTemplate } from "../../server-CM5XNicg.js";
+import { F as buildCollectionActionSeedPrompt, I as promptPathsFor, z as readSkillTemplate } from "../../server-CTkTHqEL.js";
 import { FEEDS_DIR, feedDir, feedStatePath, feedsRoot, ingestStateDir, ingestStatePath } from "../paths.js";
 import { n as DEFAULT_FEED_MAX_ITEMS, r as isFeedSchedule, t as AGENT_INGEST_KIND } from "../../ingestTypes-Bs31Xt5c.js";
 import { d as publish, n as clear } from "../../notifier-vAp5t6wL.js";

package/dist/{server-ObUrsDs2.cjs → server-CLhww8NL.cjs}

@@ -1605,6 +1605,30 @@ function renderSchemaDocs(rawDoc, topic, variant = "staged") {
 	if (sections.length === 0) return doc;
 	return clip(requested ? renderTopic(lines, sections, requested) : renderDefault(lines, sections), SCHEMA_DOCS_RESULT_BUDGET);
 }
+//#endregion
+//#region src/collection/server/schemaReadReply.ts
+/** The note prefixed when the two copies differ. Kept separate from the JSON by a
+*  blank line so the schema below it is still copyable as-is. */
+function divergenceNotice(slug) {
+	return [
+		`manageCollection: NOTE — the schema below is data/skills/${slug}/schema.json, and it DIFFERS from .claude/skills/${slug}/schema.json, the copy the server is running.`,
+		"The staging copy was changed without being mirrored (Write / Edit are mirrored; Bash and scripts are not).",
+		`To apply it, pass the JSON below to putSchema. To keep the running schema instead, Read .claude/skills/${slug}/schema.json and putSchema that.`
+	].join("\n");
+}
+/** The reply for `getSchema`: the staging copy when there is one, else the active
+*  copy; prefixed with a notice when both exist and differ. `null` when neither
+*  copy could be read. */
+function schemaReadReply(slug, staging, active) {
+	const primary = staging ?? active;
+	if (primary === null) return null;
+	if (staging === null || active === null || staging === active) return primary;
+	return [
+		divergenceNotice(slug),
+		"",
+		staging
+	].join("\n");
+}
 /** True for a launcher-managed preset slug (`mc-<something>`). The sync logic uses
 *  this to bound which active skills it may refresh/prune, so user-authored skills are
 *  never touched. */
@@ -2222,19 +2246,24 @@ async function handleSchemaDocs(deps, topic) {
 	} catch {}
 	return `manageCollection: could not read the collection-authoring reference (${SCHEMA_DOCS_FILE}).`;
 }
+async function readTextOrNull(filePath) {
+	try {
+		return await (0, node_fs_promises.readFile)(filePath, "utf-8");
+	} catch {
+		return null;
+	}
+}
 /** Return the raw schema.json of an existing collection, for editing.
-*  Staging (the canonical writable copy) first, the active mirror as a
-*  fallback for user-scope skills that have no staging copy. Raw text —
+*  A project collection's staging copy (the canonical writable one) first,
+*  else the discovered skill dir's copy. Raw text —
 *  not the parsed schema — so the agent edits the true on-disk source. */
 async function handleGetSchema(slug, deps) {
 	const collection = await require_discovery.loadCollection(slug, deps);
 	if (!collection) return unknownCollection(slug);
 	const { stagingDir } = authoringTarget(deps, collection.slug);
-	const candidates = [...stagingDir === null ? [] : [node_path.default.join(stagingDir, SCHEMA_FILE)], node_path.default.join(collection.skillDir, SCHEMA_FILE)];
-	for (const candidate of candidates) try {
-		return await (0, node_fs_promises.readFile)(candidate, "utf-8");
-	} catch {}
-	return `manageCollection: '${require_promptSafety.defangForPrompt(slug)}' has no readable ${SCHEMA_FILE}.`;
+	const staging = stagingDir !== null && collection.source === "project" ? await readTextOrNull(node_path.default.join(stagingDir, SCHEMA_FILE)) : null;
+	const active = await readTextOrNull(node_path.default.join(collection.skillDir, SCHEMA_FILE));
+	return schemaReadReply(collection.slug, staging, active) ?? `manageCollection: '${require_promptSafety.defangForPrompt(slug)}' has no readable ${SCHEMA_FILE}.`;
 }
 /** Where the agent should CREATE a collection skill in this root, named in the
 *  messages that tell it to. Naming `data/skills/` under a root that has no
@@ -2689,4 +2718,4 @@ Object.defineProperty(exports, "validateRecordObject", {
 	}
 });
 
-//# sourceMappingURL=server-ObUrsDs2.cjs.map
+//# sourceMappingURL=server-CLhww8NL.cjs.map