@kylecheng3146/agent-ops 0.1.18 → 0.1.19

package/dist/runtime/src/fs/managed-block.js

@@ -2,6 +2,13 @@ import { AgentOpsError } from "./paths.js";
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
+export function normalizeToLF(source) {
+    return source.replace(/\r\n/g, "\n");
+}
+/** Newline sequence to use for newly rendered content, inferred from the file. */
+function detectNewline(source) {
+    return source.includes("\r\n") ? "\r\n" : "\n";
+}
 function assertBlockId(id) {
     if (!/^[a-z][a-z0-9-]{0,127}$/.test(id)) {
         throw new AgentOpsError("INVALID_BLOCK_ID", `Invalid block ID: ${id}`);
@@ -70,25 +77,30 @@ function locateMarkers(source, id, version, markerStyle = "html") {
     }
     return { start, end, startIndex, endIndex };
 }
-function renderBlock(options) {
+function renderBlock(options, nl) {
     if (/(?:<!--|#)\s*agent-ops:/.test(options.content)) {
         throw new AgentOpsError("AMBIGUOUS_MANAGED_CONTENT", "Managed content must not contain agent-ops marker boundaries.");
     }
     const { start, end } = managedBlockMarkers(options.id, options.version, options.markerStyle);
-    const content = options.content.replace(/\r\n/g, "\n").replace(/\n+$/g, "");
-    return `${start}\n${content}\n${end}`;
+    const content = normalizeToLF(options.content)
+        .replace(/\n+$/g, "")
+        .split("\n")
+        .join(nl);
+    return `${start}${nl}${content}${nl}${end}`;
 }
 export function applyManagedBlock(source, options) {
-    const block = renderBlock(options);
+    const nl = detectNewline(source);
+    const block = renderBlock(options, nl);
     const located = locateMarkers(source, options.id, options.version, options.markerStyle);
     if (located === null) {
         if (source.length === 0) {
-            return `${block}\n`;
+            return `${block}${nl}`;
         }
-        return `${source.replace(/\n*$/, "\n\n")}${block}\n`;
+        return `${source.replace(/(?:\r\n|\n)*$/, `${nl}${nl}`)}${block}${nl}`;
     }
     return `${source.slice(0, located.startIndex)}${block}${source.slice(located.endIndex + located.end.length)}`;
 }
+const NEWLINE_STYLES = ["\r\n", "\n"];
 export function removeManagedBlock(source, id, markerStyle = "html") {
     const located = locateMarkers(source, id, undefined, markerStyle);
     if (located === null) {
@@ -96,12 +108,13 @@ export function removeManagedBlock(source, id, markerStyle = "html") {
     }
     let before = source.slice(0, located.startIndex);
     let after = source.slice(located.endIndex + located.end.length);
-    if (before.length === 0 && after === "\n") {
+    if (before.length === 0 && NEWLINE_STYLES.some((nl) => after === nl)) {
         return "";
     }
-    if (before.endsWith("\n\n") && after.startsWith("\n")) {
-        before = before.slice(0, -1);
-        after = after.slice(1);
+    const collapseNl = NEWLINE_STYLES.find((nl) => before.endsWith(`${nl}${nl}`) && after.startsWith(nl));
+    if (collapseNl !== undefined) {
+        before = before.slice(0, -collapseNl.length);
+        after = after.slice(collapseNl.length);
     }
     return `${before}${after}`;
 }

package/dist/runtime/src/install/ownership.js

@@ -1,4 +1,4 @@
-import { applyManagedBlock, managedBlockMarkers } from "../fs/managed-block.js";
+import { applyManagedBlock, managedBlockMarkers, normalizeToLF } from "../fs/managed-block.js";
 import { AgentOpsError } from "../fs/paths.js";
 import { harnessDescriptor, harnessHookPath, routingBlockId, selectHarnessHookSurface, rulesArtifactId } from "./harness.js";
 import { isOpencodePluginPath } from "../adapters/opencode/config.js";
@@ -211,18 +211,18 @@ export function assertExpectedManagedBlock(source, marker, expected) {
         startIndex >= endIndex) {
         throw new AgentOpsError("MANAGED_BLOCK_CHANGED", `Managed block boundaries changed after installation: ${marker.path}`);
     }
-    const currentBlock = source.slice(startIndex, endIndex + marker.endMarker.length);
+    const currentBlock = normalizeToLF(source.slice(startIndex, endIndex + marker.endMarker.length));
     const candidates = [
         ["desired", expected.content],
         ...expected.legacyContent.map((content) => ["legacy", content])
     ];
     for (const [kind, content] of candidates) {
-        const expectedBlock = applyManagedBlock("", {
+        const expectedBlock = normalizeToLF(applyManagedBlock("", {
             id: expected.id,
             version: 1,
             content,
             markerStyle: expected.markerStyle
-        }).replace(/\n$/u, "");
+        }).replace(/\n$/u, ""));
         if (currentBlock === expectedBlock) {
             return kind;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kylecheng3146/agent-ops",
-  "version": "0.1.18",
+  "version": "0.1.19",
   "description": "Evidence-driven development loops for Codex, Claude Code, and opencode",
   "type": "module",
   "license": "MIT",