@kylecheng3146/agent-ops 0.1.18 → 0.1.20

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/codex-loop.js

@@ -134,12 +134,16 @@ export function planLoopContribution(options) {
         return { artifacts: [], blocks: [] };
     }
     const harnesses = selectedLoopHarnesses(options.harnesses);
-    if (options.scope !== "project" || harnesses.length === 0) {
-        throw new AgentOpsError("LOOP_PROFILE_UNSUPPORTED", "The loop profile requires project scope and the Codex or Claude harness.");
+    if (options.scope !== "project" ||
+        (harnesses.length === 0 && !options.harnesses.includes("agy"))) {
+        throw new AgentOpsError("LOOP_PROFILE_UNSUPPORTED", "The loop profile requires project scope and the agy, Codex, or Claude harness.");
     }
     if (options.hookRuntimePath === undefined) {
         throw new AgentOpsError("LOOP_RUNTIME_REQUIRED", "The loop profile requires the installed hook runtime path.");
     }
+    if (harnesses.length === 0) {
+        return { artifacts: [], blocks: [] };
+    }
     const artifacts = harnesses.flatMap((harness) => [
         {
             id: loopLauncherArtifactId(harness),

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

@@ -309,6 +309,14 @@ function checkLifecycleSummary(manifest, config) {
     }
     return check("lifecycle-summary", "PASS", "Lifecycle summary is reachable for every selected harness.");
 }
+function checkProjectLoop(manifest, config) {
+    if (manifest === undefined || config === undefined || !config.profiles.includes("loop")) {
+        return undefined;
+    }
+    return manifest.harness.includes("agy")
+        ? check("project-loop", "DEGRADED", "agy loop uses only PreInvocation and PreToolUse(run_command); prompt, permission, compact, and subagent events are unavailable.")
+        : check("project-loop", "PASS", "Project loop events are fully registered.");
+}
 async function checkSurfaceInventory(root, manifest, config) {
     if (manifest === undefined || config === undefined) {
         return {
@@ -420,6 +428,13 @@ export async function doctorInstallation(options) {
         await checkRegistrationDrift(options.root, manifest.manifest, config.config),
         await checkProbe("hook-registration", options.probes?.hookRegistration),
         checkLifecycleSummary(manifest.manifest, config.config),
+        ...(() => {
+            const projectLoop = checkProjectLoop(manifest.manifest, config.config);
+            return projectLoop === undefined ? [] : [projectLoop];
+        })(),
+        ...(manifest.manifest?.harness.includes("agy") === true
+            ? [await checkProbe("agy-runtime", options.probes?.agyRuntime)]
+            : []),
         await checkProbe("repository-trust", options.probes?.repositoryTrust),
         await checkProbe("smoke-availability", options.probes?.smokeAvailability),
         await checkReviewTargets(config.config, options.probes?.reviewTarget, options.checkReviewTargetAuth === true)

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

@@ -1,3 +1,8 @@
+import { buildAgyHookSettings, isAgyHookRegistered, isAgyManagedHook, mergeAgyHooks, stripAgyHooks } from "../adapters/agy/config.js";
+import { AGY_CAPABILITY_REGISTRATIONS } from "../adapters/agy/events.js";
+import { normalizeAgyHookInput } from "../adapters/agy/input.js";
+import { agyHookOutput } from "../adapters/agy/output.js";
+import { agySurfaces } from "../adapters/agy/surfaces.js";
 import { buildClaudeHookSettings, isClaudeManagedHandler, mergeClaudeSettings, stripClaudeManagedHooks } from "../adapters/claude/config.js";
 import { CLAUDE_CAPABILITY_REGISTRATIONS } from "../adapters/claude/events.js";
 import { normalizeClaudeHookInput } from "../adapters/claude/input.js";
@@ -16,6 +21,7 @@ import { opencodeSurfaces } from "../adapters/opencode/surfaces.js";
 import { AgentOpsError } from "../fs/paths.js";
 import { findSurfaceById, findSurfaceByPath, isWritableSurface } from "./surfaces.js";
 export const HARNESS_IDS = [
+    "agy",
     "codex",
     "claude",
     "opencode"
@@ -108,7 +114,52 @@ const CLAUDE_ROUTING = {
         "## Loop Engineering\n\nUse `.agent-ops/CLAUDE.md` as the canonical Loop Engineering specification for this project.\n"
     ]
 };
+const AGY_ROUTING = {
+    desired: "## Loop Engineering\n\nLoad `.agent-ops/GEMINI.md` as the agent-ops managed baseline.\nProject-specific instructions in this file remain authoritative.\n",
+    legacy: []
+};
 const DESCRIPTORS = {
+    agy: {
+        id: "agy",
+        control: {
+            instructionFile: "GEMINI.md",
+            routing: AGY_ROUTING,
+            hookPath: ".agents/hooks.json",
+            hookPathForScope: (scope) => scope === "project" ? ".agents/hooks.json" : ".gemini/config/hooks.json",
+            surfaces: agySurfaces,
+            ownSettingsKeys: [],
+            buildHooks: buildAgyHookSettings,
+            mergeHooks: mergeAgyHooks,
+            stripHooks: stripAgyHooks,
+            isManagedHandler: isAgyManagedHook,
+            registrations: AGY_CAPABILITY_REGISTRATIONS,
+            hookRegistered: (source, capabilities) => isAgyHookRegistered(parseJsonSource(source), capabilities),
+            plan: async (context) => {
+                if (context.scope === "project") {
+                    return await planCommonHarnessContribution("codex", context);
+                }
+                const descriptor = DESCRIPTORS.agy;
+                return {
+                    artifacts: [{
+                            id: "gemini-rules",
+                            path: ".agent-ops/GEMINI.md",
+                            content: managedRules(descriptor, context)
+                        }],
+                    blocks: [{
+                            id: "agy-routing",
+                            path: ".gemini/GEMINI.md",
+                            version: 1,
+                            content: AGY_ROUTING.desired
+                        }]
+                };
+            }
+        },
+        runtime: {
+            normalizeInput: normalizeAgyHookInput,
+            formatOutput: agyHookOutput,
+            formatRuntimeFailure: (event, capability, remedy) => agyHookOutput(event, runtimeFailureResult(capability, AGY_CAPABILITY_REGISTRATIONS, remedy))
+        }
+    },
     codex: createJsonDescriptor({
         id: "codex",
         instructionFile: "AGENTS.md",
@@ -244,7 +295,7 @@ export function managedRules(descriptor, context) {
         ""
     ];
     if (context.capabilities.includes("rules")) {
-        lines.push("For every change:", "", "1. Define two to five mechanically verifiable acceptance criteria.", "2. Inspect the smallest relevant scope and preserve unrelated changes.", "3. Apply the smallest safe change.", "4. Run evidence-producing verification for every criterion.", "5. Obtain independent review before claiming completion, via", "   `agent-ops review --yes` (or the CLI's equivalent invocation). Never call a", "   review-target CLI (agy, codex, claude) directly — direct calls skip", "   the enforced read-only sandbox flags and can hang or fail on command", "   permission prompts.", "", "Treat `.agent-ops/config.json` as verifier authority. Discovery output is", "only a proposal until a user confirms it. Repository commands require an", "exact matching trust record. Confirmed project init/update grants it", "automatically when verification commands are configured.", "");
+        lines.push("For every change:", "", "1. Define two to five mechanically verifiable acceptance criteria.", "2. Inspect the smallest relevant scope and preserve unrelated changes.", "3. Apply the smallest safe change.", "4. Run evidence-producing verification for every criterion.", "5. Obtain independent review before claiming completion, via", "   `agent-ops review --yes` (or the CLI's equivalent invocation). Never call a", "   review-target CLI (agy, codex, claude) directly — direct calls skip", "   the enforced read-only sandbox flags and can hang or fail on command", "   permission prompts.", "   Set `AGENT_OPS_HOST` to the current CLI id when invoking review so", "   agent-ops tries a different CLI first and uses isolated self-review", "   only when no other configured reviewer is usable.", "", "Treat `.agent-ops/config.json` as verifier authority. Discovery output is", "only a proposal until a user confirms it. Repository commands require an", "exact matching trust record. Confirmed project init/update grants it", "automatically when verification commands are configured.", "");
     }
     if (context.capabilities.includes("task")) {
         lines.push("Split a change that exceeds five acceptance criteria into subtasks:", "`agent-ops task create --parent <task-id>` records one, and", "`agent-ops task status --parent <task-id>` lists them. Each subtask", "carries its own criteria, verification, and independent review;", "completing one never completes its parent.", "");

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

@@ -35,7 +35,9 @@ export function planHookRegistration(options) {
     }
     const path = options.path ?? hookRegistrationPath(options.harness, options.scope);
     const managed = descriptor.control.buildHooks(options.capabilities, options.runtimePath, options.platform);
-    const events = Object.keys(managed.hooks);
+    const events = Object.keys(managed.hooks).map((event) => options.harness === "agy" && event === "PreInvocation"
+        ? "SessionStart"
+        : event);
     if (events.length === 0) {
         return null;
     }
@@ -54,6 +56,9 @@ export function planHookRegistration(options) {
     };
 }
 function onlyManagedRemains(harness, value) {
+    if (harness === "agy") {
+        return Object.keys(value).length === 0;
+    }
     const ownKeys = new Set(harnessDescriptor(harness).control.ownSettingsKeys ?? []);
     const hooks = value.hooks;
     return (Object.keys(value).every((key) => ownKeys.has(key)) &&

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

@@ -1,16 +1,18 @@
-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";
 import { LOOP_MARKER_ID, LOOP_MARKER_VERSION, loopIgnoreContent, loopLauncherArtifactId, loopWindowsLauncherArtifactId, loopWindowsLauncherPath, loopLauncherPath, selectedLoopHarnesses } from "./codex-loop.js";
 function expectedMarker(manifest, id, markerId) {
-    const descriptor = harnessDescriptor(id);
+    const descriptor = harnessDescriptor(id === "agy" && manifest.scope === "project" ? "codex" : id);
     const markers = managedBlockMarkers(markerId, 1, "html");
     return {
         id: markerId,
         path: manifest.scope === "project"
             ? descriptor.control.instructionFile
-            : `.${id}/${descriptor.control.instructionFile}`,
+            : id === "agy"
+                ? ".gemini/GEMINI.md"
+                : `.${id}/${descriptor.control.instructionFile}`,
         startMarker: markers.start,
         endMarker: markers.end,
         markerStyle: "html",
@@ -96,7 +98,7 @@ export function assertSupportedManifestOwnership(manifest, root) {
         throw manifestOwnershipError();
     }
     for (const id of harnesses) {
-        const descriptor = harnessDescriptor(id);
+        const descriptor = harnessDescriptor(id === "agy" && manifest.scope === "project" ? "codex" : id);
         const artifactPath = `.agent-ops/${descriptor.control.instructionFile}`;
         const artifactKey = pathKey(artifactPath);
         const artifactEntry = expectedArtifactPaths.get(artifactKey);
@@ -112,7 +114,9 @@ export function assertSupportedManifestOwnership(manifest, root) {
         requiredArtifactPaths.add(artifactKey);
         const markerPath = manifest.scope === "project"
             ? descriptor.control.instructionFile
-            : `.${id}/${descriptor.control.instructionFile}`;
+            : id === "agy"
+                ? ".gemini/GEMINI.md"
+                : `.${id}/${descriptor.control.instructionFile}`;
         const markerKey = pathKey(markerPath);
         expectedMarkerPaths.add(markerKey);
         const currentId = routingBlockId(id, manifest.scope, descriptor);
@@ -211,18 +215,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/dist/runtime/src/install/plan.js

@@ -280,8 +280,8 @@ function assertLoopProfileSupport(scope, harness, capabilities) {
         return;
     }
     if (scope !== "project" ||
-        !harness.some((id) => id === "codex" || id === "claude")) {
-        throw new AgentOpsError("LOOP_PROFILE_UNSUPPORTED", "The loop profile requires project scope and the Codex or Claude harness.");
+        !harness.some((id) => id === "agy" || id === "codex" || id === "claude")) {
+        throw new AgentOpsError("LOOP_PROFILE_UNSUPPORTED", "The loop profile requires project scope and the agy, Codex, or Claude harness.");
     }
 }
 async function assertCodexLoopConfiguration(root, harness, capabilities) {

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

@@ -1,5 +1,66 @@
 import { harnessDescriptor } from "./harness.js";
 import { resolveCapabilities } from "./profiles.js";
+const MINIMUM_AGY_VERSION = [1, 1, 12];
+export function agyVersionSupported(versionOutput) {
+    const match = /\b(\d+)\.(\d+)\.(\d+)\b/u.exec(versionOutput);
+    const version = match?.slice(1).map(Number);
+    return version !== undefined && !version.some((part, index) => part < MINIMUM_AGY_VERSION[index] &&
+        version.slice(0, index).every((prior, priorIndex) => prior === MINIMUM_AGY_VERSION[priorIndex]));
+}
+export function agyRuntimeStatus(versionOutput, hooksOutput, expectedEvents = []) {
+    const match = /\b(\d+)\.(\d+)\.(\d+)\b/u.exec(versionOutput);
+    if (!agyVersionSupported(versionOutput)) {
+        return {
+            status: "FAIL",
+            message: "agy 1.1.12 or newer is required.",
+            remediation: "Update agy, then run `agent-ops doctor` again."
+        };
+    }
+    try {
+        const parsed = JSON.parse(hooksOutput);
+        const hooks = parsed.command?.data?.hooks;
+        const loaded = Array.isArray(hooks) && hooks.some((hook) => {
+            if (typeof hook !== "object" || hook === null || Array.isArray(hook))
+                return false;
+            const value = hook;
+            const actions = value.actions;
+            if (!(value.name === "agent-ops" &&
+                value.enabled === true &&
+                Array.isArray(actions) &&
+                actions.length > 0 &&
+                actions.every((action) => typeof action === "object" && action !== null && !Array.isArray(action))))
+                return false;
+            return expectedEvents.every((expected) => actions.some((action) => {
+                const nativeEvent = expected === "SessionStart" ? "PreInvocation" : expected;
+                return (typeof action === "object" && action !== null && !Array.isArray(action) &&
+                    action.event === nativeEvent &&
+                    typeof action.command === "string" &&
+                    action.command.endsWith(` agy ${expected} --managed-by=agent-ops`));
+            }));
+        });
+        if (!Array.isArray(hooks) || (expectedEvents.length > 0 && !loaded)) {
+            return {
+                status: "FAIL",
+                message: "agy is installed, but its loaded hook list does not include agent-ops.",
+                code: "UPDATE_REQUIRED",
+                remediation: "Run `agent-ops update`, restart agy, then run doctor again."
+            };
+        }
+        return {
+            status: "PASS",
+            message: expectedEvents.length > 0
+                ? `agy ${match?.[0]} loaded the agent-ops hook.`
+                : `agy ${match?.[0]} meets the minimum supported version.`
+        };
+    }
+    catch {
+        return {
+            status: "FAIL",
+            message: "agy returned an unreadable /hooks response.",
+            remediation: "Run `agy -p \"/hooks\" --output-format json` and inspect the result."
+        };
+    }
+}
 /**
  * Returns the harness ids missing an agent-ops owned handler for the hook
  * events implied by the installed profiles. Empty when installations without

package/dist/runtime/src/install/surface-inspection.js

@@ -90,6 +90,9 @@ function managedJsonCount(source, isManagedHandler) {
     }
     return jsonHandlerCounts(source, isManagedHandler)?.managed ?? 0;
 }
+function desiredHookEvents(harness, events) {
+    return events.map((event) => harness === "agy" && event === "PreInvocation" ? "SessionStart" : event);
+}
 export async function inspectHarnessRegistrations(options) {
     const capabilities = desiredCapabilities(options.config);
     const statuses = [];
@@ -100,7 +103,7 @@ export async function inspectHarnessRegistrations(options) {
         const recordedEvents = hookRecord?.events ?? [];
         const desiredEvents = control.buildHooks === undefined
             ? []
-            : Object.keys(control.buildHooks(capabilities, "probe").hooks);
+            : desiredHookEvents(harness, Object.keys(control.buildHooks(capabilities, "probe").hooks));
         if (control.buildHooks !== undefined) {
             const surfaces = harnessSurfaces(harness, options.manifest.scope, options.root);
             const writableJsonSurfaces = surfaces.filter((surface) => isWritableSurface(surface) && surface.representation === "json");
@@ -186,6 +189,12 @@ function jsonHandlerCounts(source, isManagedHandler) {
     if (!isRecord(parsed)) {
         return null;
     }
+    if (isManagedHandler?.(parsed) === true) {
+        return {
+            managed: 1,
+            foreign: Math.max(0, Object.keys(parsed).length - 1)
+        };
+    }
     const hooks = parsed.hooks;
     if (hooks === undefined) {
         return { managed: 0, foreign: 0 };