@agent-facets/adapter-claude-code 0.4.3 → 0.4.5

package/dist/index.mjs

@@ -6,22 +6,6 @@ import { mkdir, readFile, rm, writeFile } from "node:fs/promises";
 var __commonJSMin = (cb, mod) => () => (mod || (cb((mod = { exports: {} }).exports, mod), cb = null), mod.exports);
 var __require = /* @__PURE__ */ createRequire(import.meta.url);
 //#endregion
-//#region ../../common/src/text.ts
-/**
-* Small text-normalization helpers used across packages that parse
-* user-authored files. Lives in `common` so core front-matter parsing and
-* the adapter SDK's asset-fs helpers can share one definition — Windows
-* line endings or a stray BOM shouldn't cause divergent behavior across
-* readers.
-*/
-/**
-* Strip a leading UTF-8 BOM and convert any `\r\n` / `\r` line endings to
-* `\n`. Idempotent. Returns the string unchanged if already normalized.
-*/
-function normalizeLineEndings(raw) {
-	return raw.replace(/^\uFEFF/, "").replace(/\r\n/g, "\n").replace(/\r/g, "\n");
-}
-//#endregion
 //#region ../../../node_modules/.bun/yaml@2.8.3/node_modules/yaml/dist/nodes/identity.js
 var require_identity = /* @__PURE__ */ __commonJSMin(((exports) => {
 	const ALIAS = Symbol.for("yaml.alias");
@@ -6570,7 +6554,7 @@ var require_public_api = /* @__PURE__ */ __commonJSMin(((exports) => {
 	exports.stringify = stringify;
 }));
 //#endregion
-//#region ../../adapter/src/asset-fs.ts
+//#region ../../common/src/text.ts
 var import_dist = (/* @__PURE__ */ __commonJSMin(((exports) => {
 	var composer = require_composer();
 	var Document = require_Document();
@@ -6618,6 +6602,69 @@ var import_dist = (/* @__PURE__ */ __commonJSMin(((exports) => {
 	exports.visitAsync = visit.visitAsync;
 })))();
 /**
+* Small text-normalization helpers used across packages that parse
+* user-authored files. Lives in `common` so core front-matter parsing and
+* the adapter SDK's asset-fs helpers can share one definition — Windows
+* line endings or a stray BOM shouldn't cause divergent behavior across
+* readers.
+*/
+/**
+* Strip a leading UTF-8 BOM and convert any `\r\n` / `\r` line endings to
+* `\n`. Idempotent. Returns the string unchanged if already normalized.
+*/
+function normalizeLineEndings(raw) {
+	return raw.replace(/^\uFEFF/, "").replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+}
+//#endregion
+//#region ../../common/src/front-matter.ts
+/**
+* Split a string into a body without front matter and the parsed YAML
+* front-matter metadata, when present.
+*
+* Used by:
+*  - the adapter SDK's `splitAssetContent` (and via it, every adapter's
+*    read path) to recover the body + metadata from on-disk asset files.
+*  - core's materialize skip-if-identical compare to normalize the
+*    candidate write content the same way the read path would, so the
+*    body comparison is symmetric.
+*
+* Lives in `common` because both the adapter SDK and `core` need
+* identical splitting semantics. A core-side implementation that
+* imported `splitAssetContent` from the adapter SDK directly would
+* pull `yaml` and friends into `core`'s runtime graph, which collides
+* with `Bun.build` when the CLI's adapter integration tests bundle the
+* same source files in the same process. Putting the primitive here
+* means both packages get the same code without a cross-package value
+* import.
+*
+* Returns `{ content: <normalized raw> }` when no front matter is
+* detected or the YAML is malformed.
+*
+* Known limitation: the regex below is non-greedy, so a body that
+* legitimately contains a literal `---` terminator line will be split
+* at the first match. Skills and commands rarely contain frontmatter-
+* shaped content inside their bodies, so in practice this is benign;
+* a parser swap (e.g., `gray-matter`) is the right fix when needed.
+*/
+const FRONT_MATTER_RE = /^---\n([\s\S]*?)\n---\n?([\s\S]*)$/;
+function splitFrontMatter(raw) {
+	const normalized = normalizeLineEndings(raw);
+	const match = normalized.match(FRONT_MATTER_RE);
+	if (!match) return { content: normalized };
+	try {
+		const parsed = (0, import_dist.parse)(match[1] ?? "");
+		if (parsed && typeof parsed === "object" && !Array.isArray(parsed)) return {
+			content: match[2] ?? "",
+			metadata: parsed
+		};
+		return { content: normalized };
+	} catch {
+		return { content: normalized };
+	}
+}
+//#endregion
+//#region ../../adapter/src/asset-fs.ts
+/**
 * Write `body` to `path.file`, prepending YAML front-matter assembled from
 * `metadata` when non-empty. Creates parent directories. Overwrites
 * unconditionally (idempotent by contract — see Adjustment B).
@@ -6650,11 +6697,21 @@ async function deleteAssetFile(path) {
 	await rm(path.file, { force: true });
 	await rm(`${path.file}.meta.json`, { force: true });
 }
-const FRONT_MATTER_RE = /^---\n([\s\S]*?)\n---\n?([\s\S]*)$/;
 /**
 * Assemble a full file string from optional front-matter metadata + body.
 * When `body` already contains a front-matter block, the two are merged
 * (`metadata` wins on key collision) so external keys on the body survive.
+*
+* The output is exactly `---\n<yaml>\n---\n<body>` — no separator newline
+* between the closing fence and the body. This is the inverse of
+* `splitAssetContent`'s regex (which consumes one `\n` after `---`), so
+* `assemble → write → read → split` is byte-stable.
+*
+* Earlier versions inserted a cosmetic blank line between the fence and
+* the body; that asymmetry caused materialize's skip-if-identical check
+* to see phantom drift on every re-install (`# body` going in, `\n# body`
+* coming out). Tests in this file's "round-trip" describe block enforce
+* the inverse contract explicitly.
 */
 function assembleAssetContent(body, metadata) {
 	const existing = splitAssetContent(body);
@@ -6664,35 +6721,21 @@ function assembleAssetContent(body, metadata) {
 	};
 	const bodyOnly = existing.content;
 	if (Object.keys(merged).length === 0) return bodyOnly;
-	return `---\n${(0, import_dist.stringify)(merged).trimEnd()}\n---\n${bodyOnly.length === 0 || bodyOnly.startsWith("\n") ? "" : "\n"}${bodyOnly}`;
+	return `---\n${(0, import_dist.stringify)(merged).trimEnd()}\n---\n${bodyOnly}`;
 }
 /**
 * Parse a file string into its body + parsed front-matter metadata. Returns
 * the raw string as `content` when no front-matter is detected. Malformed
 * YAML falls back to "no front-matter."
 *
-* Known limitation (tracked as a post-alpha TODO — see plan F10): the regex
-* below is non-greedy, so if a body legitimately contains a literal line of
-* exactly `---` followed by content and another `---` line, the first
-* terminator wins and the body will be split in the wrong place. Skills and
-* commands rarely include frontmatter-shaped content inside their bodies,
-* so in practice this is benign — but replacing the regex with `gray-matter`
-* or an equivalent well-tested parser is the right fix.
+* Thin re-export over `@agent-facets/common`'s `splitFrontMatter` so the
+* adapter SDK and `core` share one canonical implementation. Kept as a
+* named export here because adapter authors may import it directly when
+* they customize their read path; `common` is bundled into the published
+* adapter SDK so this stays a leaf import for consumers.
 */
 function splitAssetContent(raw) {
-	const normalized = normalizeLineEndings(raw);
-	const match = normalized.match(FRONT_MATTER_RE);
-	if (!match) return { content: normalized };
-	try {
-		const parsed = (0, import_dist.parse)(match[1] ?? "");
-		if (parsed && typeof parsed === "object" && !Array.isArray(parsed)) return {
-			content: match[2] ?? "",
-			metadata: parsed
-		};
-		return { content: normalized };
-	} catch {
-		return { content: normalized };
-	}
+	return splitFrontMatter(raw);
 }
 //#endregion
 //#region ../../adapter/src/define-adapter.ts

package/package.json

@@ -5,7 +5,7 @@
     "url": "https://github.com/agent-facets/facets",
     "directory": "packages/adapters/claude-code"
   },
-  "version": "0.4.3",
+  "version": "0.4.5",
   "type": "module",
   "files": [
     "dist"