@oisincoveney/pipeline 3.8.0 → 3.9.0

package/dist/install-commands/shared.js

@@ -9,8 +9,6 @@ const OWNER_TS_MARKER_PREFIX = "// @oisincoveney/pipeline:";
 const OWNER_YAML_MARKER_PREFIX = "# @oisincoveney/pipeline:";
 const AGENTS_MD_START = "<!-- @oisincoveney/pipeline:agents:start -->";
 const AGENTS_MD_END = "<!-- @oisincoveney/pipeline:agents:end -->";
-const INSTRUCTIONS_START = "<!-- @oisincoveney/pipeline:instructions:start -->";
-const INSTRUCTIONS_END = "<!-- @oisincoveney/pipeline:instructions:end -->";
 const SINGLE_OPENCODE_PLUGIN_ARRAY_RE = /\n {2}"plugin": \[\n {4}("[^"]+")\n {2}\]/;
 const OPENCODE_PROJECT_CONFIG_PATH = ".opencode/opencode.json";
 const CLAUDE_PROJECT_CONFIG_PATH = ".claude/settings.json";
@@ -94,4 +92,4 @@ function commandIdForHost(host, entrypointId) {
 	return entrypointId;
 }
 //#endregion
-export { AGENTS_MD_END, AGENTS_MD_START, CLAUDE_PROJECT_CONFIG_PATH, COMMAND_HOSTS, DEFAULT_HARNESS_SCOPE, ENTRYPOINT_PATH_PATTERNS, GENERATED_MARKER, GENERATED_TS_MARKER, GENERATED_YAML_MARKER, INSTRUCTIONS_END, INSTRUCTIONS_START, OPENCODE_PROJECT_CONFIG_PATH, OWNER_MARKER_PREFIX, OWNER_TS_MARKER_PREFIX, OWNER_YAML_MARKER_PREFIX, SINGLE_OPENCODE_PLUGIN_ARRAY_RE, commandIdForHost, compactLines, entrypointDescription, entrypointEntries, instructionsPointer, invocationForHost, profileEntries, resolveHarnessTarget };
+export { AGENTS_MD_END, AGENTS_MD_START, CLAUDE_PROJECT_CONFIG_PATH, COMMAND_HOSTS, DEFAULT_HARNESS_SCOPE, ENTRYPOINT_PATH_PATTERNS, GENERATED_MARKER, GENERATED_TS_MARKER, GENERATED_YAML_MARKER, OPENCODE_PROJECT_CONFIG_PATH, OWNER_MARKER_PREFIX, OWNER_TS_MARKER_PREFIX, OWNER_YAML_MARKER_PREFIX, SINGLE_OPENCODE_PLUGIN_ARRAY_RE, commandIdForHost, compactLines, entrypointDescription, entrypointEntries, instructionsPointer, invocationForHost, profileEntries, resolveHarnessTarget };

package/dist/install-commands.js

@@ -3,7 +3,6 @@ import "./config.js";
 import { COMMAND_HOSTS, ENTRYPOINT_PATH_PATTERNS, invocationForHost, resolveHarnessTarget } from "./install-commands/shared.js";
 import { opencodeAdapter } from "./install-commands/opencode.js";
 import { claudeCodeAdapter } from "./install-commands/claude-code.js";
-import { INSTRUCTION_PATHS, globalInstructionDefinitions } from "./install-commands/instructions.js";
 import { existsSync, readFileSync, statSync } from "node:fs";
 import { dirname, join, relative } from "node:path";
 import { mkdir, readdir, rm, writeFile } from "node:fs/promises";
@@ -156,18 +155,15 @@ function commandInstallPlanItem(definition, action) {
 	};
 }
 const PROJECT_ONLY_PATHS = new Set(["AGENTS.md"]);
-const GLOBAL_ONLY_PATHS = new Set(INSTRUCTION_PATHS);
 function scopedDefinitions(definitions, scope) {
 	if (scope === "global") return definitions.filter((definition) => !PROJECT_ONLY_PATHS.has(definition.path));
-	return definitions.filter((definition) => !GLOBAL_ONLY_PATHS.has(definition.path));
+	return definitions;
 }
 function installCommandsContext(options) {
 	const cwd = options.cwd ?? process.cwd();
 	const host = options.host ?? "all";
 	const scope = options.scope ?? "global";
-	const config = loadPipelineConfig(cwd, { allowMissingLintFileReferences: true });
-	const instructionDefinitions = globalInstructionDefinitions().filter((definition) => host === "all" || definition.host === host);
-	const definitions = scopedDefinitions([...definitionsFor(host, config, cwd), ...instructionDefinitions], scope);
+	const definitions = scopedDefinitions(definitionsFor(host, loadPipelineConfig(cwd, { allowMissingLintFileReferences: true }), cwd), scope);
 	return {
 		cwd,
 		definitions,

package/dist/install-rules.js

@@ -0,0 +1,127 @@
+import "./install-commands/shared.js";
+import { execa } from "execa";
+import { dirname, join } from "node:path";
+import { homedir, tmpdir } from "node:os";
+import { fileURLToPath } from "node:url";
+import { mkdir, mkdtemp, readFile, readdir, rm, writeFile } from "node:fs/promises";
+//#region src/install-rules.ts
+const DEFAULT_RULES_INSTALL_SOURCE = "oisin-ee/rules";
+const RULESYNC_TARGETS = [
+	"claudecode",
+	"codexcli",
+	"geminicli",
+	"opencode"
+];
+function packageRoot() {
+	return join(dirname(fileURLToPath(import.meta.url)), "..");
+}
+const PACKAGE_ROOT = packageRoot();
+async function cloneRulesRepository(targetDir) {
+	await execa("gh", [
+		"repo",
+		"clone",
+		DEFAULT_RULES_INSTALL_SOURCE,
+		targetDir,
+		"--",
+		"--depth=1"
+	], { stdio: "inherit" });
+}
+async function withRulesSource(sourceOverride, useSource) {
+	if (sourceOverride !== void 0) return useSource(sourceOverride);
+	const parent = await mkdtemp(join(tmpdir(), "moka-rules-"));
+	const source = join(parent, "rules");
+	try {
+		await cloneRulesRepository(source);
+		return await useSource(source);
+	} finally {
+		await rm(parent, {
+			force: true,
+			recursive: true
+		});
+	}
+}
+async function defaultRulesyncRunner(args, opts) {
+	try {
+		await execa("rulesync", args, {
+			cwd: opts.cwd,
+			env: opts.env,
+			localDir: PACKAGE_ROOT,
+			preferLocal: true,
+			stdio: "inherit"
+		});
+	} catch (error) {
+		const cause = error instanceof Error ? `: ${error.message}` : "";
+		throw new Error(`Failed to generate global rules from ${DEFAULT_RULES_INSTALL_SOURCE}${cause}. If this is a private repository, authenticate GitHub access with \`gh auth login\` and rerun \`moka refresh-harnesses --scope global\`.`);
+	}
+}
+async function buildRootRule(source) {
+	const rulesDir = join(source, "rules");
+	let entries = [];
+	try {
+		entries = (await readdir(rulesDir, { withFileTypes: true })).filter((d) => d.isFile() && d.name.endsWith(".md")).map((d) => d.name).sort((a, b) => a.localeCompare(b));
+	} catch {}
+	const rootContent = `---
+root: true
+targets:
+  - "*"
+---
+
+${(await Promise.all(entries.map(async (name) => {
+		return (await readFile(join(rulesDir, name), "utf8")).trimEnd();
+	}))).join("\n\n")}\n`;
+	const rulesyncRulesDir = join(source, ".rulesync", "rules");
+	await mkdir(rulesyncRulesDir, { recursive: true });
+	await writeFile(join(rulesyncRulesDir, "_root.md"), rootContent);
+}
+function installRules(options = {}) {
+	if ((options.scope ?? "global") !== "global") return Promise.resolve({
+		items: [],
+		source: DEFAULT_RULES_INSTALL_SOURCE
+	});
+	const runner = options.rulesyncRunner ?? defaultRulesyncRunner;
+	return withRulesSource(options.sourceOverride, async (source) => {
+		await buildRootRule(source);
+		const home = process.env.HOME_DIR ?? homedir();
+		const args = [
+			"generate",
+			"-t",
+			RULESYNC_TARGETS.join(","),
+			"-f",
+			"rules",
+			"--delete"
+		];
+		if (options.dryRun) args.push("--dry-run");
+		if (options.check) args.push("--check");
+		await runner(args, {
+			cwd: source,
+			env: {
+				...process.env,
+				HOME_DIR: home
+			}
+		});
+		const action = options.dryRun ?? options.check ? "skip" : "generate";
+		return {
+			items: [
+				{
+					action,
+					path: join(home, ".claude/CLAUDE.md")
+				},
+				{
+					action,
+					path: join(home, ".codex/AGENTS.md")
+				},
+				{
+					action,
+					path: join(home, ".gemini/GEMINI.md")
+				},
+				{
+					action,
+					path: join(home, ".config/opencode/AGENTS.md")
+				}
+			],
+			source: DEFAULT_RULES_INSTALL_SOURCE
+		};
+	});
+}
+//#endregion
+export { installRules };

package/dist/moka-submit.d.ts

@@ -5,13 +5,13 @@ import { z } from "zod";
 //#region src/moka-submit.d.ts
 declare const mokaSubmitDirectHooksSchema: z.ZodRecord<z.ZodEnum<{
   "workflow.start": "workflow.start";
-  "node.finish": "node.finish";
-  "node.start": "node.start";
   "workflow.success": "workflow.success";
   "workflow.failure": "workflow.failure";
   "workflow.complete": "workflow.complete";
+  "node.start": "node.start";
   "node.success": "node.success";
   "node.error": "node.error";
+  "node.finish": "node.finish";
   "gate.failure": "gate.failure";
 }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
   failure: z.ZodDefault<z.ZodEnum<{
@@ -94,13 +94,13 @@ declare const mokaSubmitOptionsSchema: z.ZodDiscriminatedUnion<[z.ZodObject<{
   }, z.core.$strict>>;
   hooks: z.ZodOptional<z.ZodRecord<z.ZodEnum<{
     "workflow.start": "workflow.start";
-    "node.finish": "node.finish";
-    "node.start": "node.start";
     "workflow.success": "workflow.success";
     "workflow.failure": "workflow.failure";
     "workflow.complete": "workflow.complete";
+    "node.start": "node.start";
     "node.success": "node.success";
     "node.error": "node.error";
+    "node.finish": "node.finish";
     "gate.failure": "gate.failure";
   }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
     failure: z.ZodDefault<z.ZodEnum<{
@@ -207,13 +207,13 @@ declare const mokaSubmitOptionsSchema: z.ZodDiscriminatedUnion<[z.ZodObject<{
   }, z.core.$strict>>;
   hooks: z.ZodOptional<z.ZodRecord<z.ZodEnum<{
     "workflow.start": "workflow.start";
-    "node.finish": "node.finish";
-    "node.start": "node.start";
     "workflow.success": "workflow.success";
     "workflow.failure": "workflow.failure";
     "workflow.complete": "workflow.complete";
+    "node.start": "node.start";
     "node.success": "node.success";
     "node.error": "node.error";
+    "node.finish": "node.finish";
     "gate.failure": "gate.failure";
   }> & z.core.$partial, z.ZodDiscriminatedUnion<[z.ZodObject<{
     failure: z.ZodDefault<z.ZodEnum<{

package/dist/pipeline-init.js

@@ -1,6 +1,7 @@
 import "./install-commands/shared.js";
 import { installCommands } from "./install-commands.js";
 import { installHooks } from "./install-hooks.js";
+import { installRules } from "./install-rules.js";
 import { existsSync } from "node:fs";
 import { execa } from "execa";
 import { join } from "node:path";
@@ -76,6 +77,10 @@ async function initPipelineProject(options = {}) {
 	const scope = options.scope ?? "global";
 	const skillInstaller = options.skillInstaller ?? ((target) => installDefaultSkills(target, skillScopeFor(scope)));
 	const hookInstaller = options.hookInstaller ?? installDefaultHooks;
+	const rulesInstaller = options.rulesInstaller ?? ((target, s) => installRules({
+		cwd: target,
+		scope: s
+	}));
 	await skillInstaller(cwd);
 	const result = await installCommands({
 		cwd,
@@ -84,8 +89,13 @@ async function initPipelineProject(options = {}) {
 		scope
 	});
 	const hooks = await hookInstaller(cwd, scope);
+	const rulesResult = await rulesInstaller(cwd, scope);
 	return {
-		files: [...result.items.map((item) => item.path), ...hookInstallerFiles(hooks)],
+		files: [
+			...result.items.map((item) => item.path),
+			...hookInstallerFiles(hooks),
+			...rulesResult.items.map((item) => item.path)
+		],
 		scope
 	};
 }
@@ -94,6 +104,7 @@ async function refreshAgentHarnesses(options = {}) {
 	const init = await initPipelineProject({
 		cwd: context.cwd,
 		hookInstaller: options.hookInstaller,
+		rulesInstaller: options.rulesInstaller,
 		scope: options.scope,
 		skillInstaller: options.skillInstaller
 	});
@@ -118,7 +129,7 @@ async function refreshAgentHarnessesCommitResult(context) {
 function formatPipelineInitResult(result) {
 	return [
 		"Initialized package-owned pipeline support:",
-		result.scope === "global" ? "installed the per-machine harness globally (user/global skills + ~/.claude, ~/.config/opencode, ~/.codex); inherited by every repo with no per-repo copy" : "installed default skills and host config repo-local",
+		result.scope === "global" ? "installed the per-machine harness globally (user/global skills + ~/.claude, ~/.config/opencode, ~/.codex); global instruction files generated via rulesync from oisin-ee/rules; inherited by every repo with no per-repo copy" : "installed default skills and host config repo-local",
 		...result.files.map((path) => `generated ${path}`),
 		"no repo-local pipeline config files were created"
 	].join("\n");

package/dist/runner-event-schema.d.ts

@@ -11,8 +11,8 @@ declare const runnerEventRecordSchema: z.ZodUnion<readonly [z.ZodObject<{
   runId: z.ZodString;
   sequence: z.ZodNumber;
   type: z.ZodEnum<{
-    "workflow.planned": "workflow.planned";
     "workflow.start": "workflow.start";
+    "workflow.planned": "workflow.planned";
   }>;
   workflowPlan: z.ZodObject<{
     edges: z.ZodOptional<z.ZodArray<z.ZodObject<{
@@ -58,10 +58,10 @@ declare const runnerEventRecordSchema: z.ZodUnion<readonly [z.ZodObject<{
     }>;
   }, z.core.$strip>;
   type: z.ZodEnum<{
+    "node.start": "node.start";
+    "node.finish": "node.finish";
     "agent.finish": "agent.finish";
     "agent.start": "agent.start";
-    "node.finish": "node.finish";
-    "node.start": "node.start";
   }>;
 }, z.core.$strip>, z.ZodObject<{
   at: z.ZodOptional<z.ZodString>;
@@ -189,8 +189,8 @@ declare const runnerEventBatchSchema: z.ZodObject<{
     runId: z.ZodString;
     sequence: z.ZodNumber;
     type: z.ZodEnum<{
-      "workflow.planned": "workflow.planned";
       "workflow.start": "workflow.start";
+      "workflow.planned": "workflow.planned";
     }>;
     workflowPlan: z.ZodObject<{
       edges: z.ZodOptional<z.ZodArray<z.ZodObject<{
@@ -236,10 +236,10 @@ declare const runnerEventBatchSchema: z.ZodObject<{
       }>;
     }, z.core.$strip>;
     type: z.ZodEnum<{
+      "node.start": "node.start";
+      "node.finish": "node.finish";
       "agent.finish": "agent.finish";
       "agent.start": "agent.start";
-      "node.finish": "node.finish";
-      "node.start": "node.start";
     }>;
   }, z.core.$strip>, z.ZodObject<{
     at: z.ZodOptional<z.ZodString>;

package/package.json

@@ -1,6 +1,7 @@
 {
   "dependencies": {
     "@dagrejs/graphlib": "^4.0.1",
+    "rulesync": "8.30.1",
     "@kubernetes/client-node": "^1.4.0",
     "@opencode-ai/sdk": "1.17.4",
     "ajv": "^8.20.0",
@@ -126,7 +127,7 @@
     "prepack": "bun run build:cli"
   },
   "type": "module",
-  "version": "3.8.0",
+  "version": "3.9.0",
   "description": "Config-driven multi-agent pipeline runner for repository work",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

package/defaults/instructions/global.md

@@ -1,68 +0,0 @@
-# Caveman Mode (default ON)
-
-Operate in caveman mode by default on every response: terse, smart-caveman phrasing — drop filler, keep ALL technical substance, code, commands, paths, and accuracy. This is active every response and does not drift off over a long session. Defer to the `caveman` skill for the exact compression rules and intensity levels. Turn off only when the user says "stop caveman" or "normal mode".
-
-# Global Behavior
-
-- Answer fully and stop. Never end with "Want me to do X?" or "Should I implement this?" — if the user wants more, they'll ask.
-- Research before acting — know how components and libraries work before making changes.
-- When uncertain, ask — don't guess.
-- The answer to "Why is X an improvement?" should never be "I'm not sure."
-
-## Before writing code
-
-- Search for existing implementations before creating new code.
-- Check for existing utilities before adding helpers.
-- Don't add dependencies without checking if functionality already exists in current deps.
-- Reuse patterns from similar files in the codebase.
-
-## Problem-solving
-
-- Is this a real problem? Reject over-engineering.
-- Is there a simpler way? Always seek the simplest solution.
-- Will it break anything? Backward compatibility matters.
-
-# Anti-Patterns
-
-## Code quality
-
-- NEVER manually edit auto-generated files — regenerate them instead.
-- NEVER suppress type errors (`as any`, `@ts-ignore`, `@ts-expect-error`).
-- NEVER use bandaids or hacks — proper fixes only.
-- NEVER create new Zod schemas when generated ones exist.
-- NEVER use `var` declarations.
-- NEVER add `.unwrap()` calls in Rust code.
-
-## Error handling
-
-- Silent error handling is NEVER permitted.
-- Every fallback and default value MUST have specific business-logic reasoning.
-- Unexpected errors MUST be logged, not swallowed.
-- Errors affecting user flow MUST surface to the user — never hide failures.
-
-## Testing
-
-- NEVER commit code without tests for new functionality.
-- NEVER skip tests or mark them skipped to make CI pass.
-- NEVER disable or delete existing tests — fix the code, not the tests.
-- Test both success and error cases.
-
-# Verification
-
-- Run tests, lint, and typecheck after every change.
-- Self-assessment is unreliable — use external signals (build output, test results) as ground truth.
-- Don't claim something works without running it.
-- If a test suite exists, run it. Don't skip it because "the change is small."
-
-# Coding Style
-
-- 120 char line width.
-- Trailing commas everywhere.
-
-## Git
-
-- NEVER commit/push directly to main.
-- NEVER amend commits or rewrite history after pushing.
-- NEVER use `--force` without explicit approval.
-- Always create new commits — never amend, squash, or rebase unless explicitly asked.
-- Conventional commit format: `feat|fix|chore|docs|test|refactor(scope): description`.

package/dist/install-commands/instructions.js

@@ -1,62 +0,0 @@
-import { resolvePackageAssetPath } from "../package-assets.js";
-import { GENERATED_MARKER, INSTRUCTIONS_END, INSTRUCTIONS_START, OWNER_MARKER_PREFIX } from "./shared.js";
-import { readFileSync } from "node:fs";
-//#region src/install-commands/instructions.ts
-/**
-* The canonical global agent instruction body, package-owned so every machine,
-* k8s job, and host renders the same behavior. Shipped in `defaults/` (see
-* package.json `files`).
-*/
-const INSTRUCTION_BODY = readFileSync(resolvePackageAssetPath("defaults/instructions/global.md"), "utf8").trimEnd();
-/**
-* The global instruction memory file each host reads, and the repo-relative
-* path that `resolveHarnessTarget` rebases onto its per-machine config dir.
-* Distinct from the bare `AGENTS.md` project guidance file, so these coexist
-* with (and never trip) the PROJECT_ONLY AGENTS.md handling.
-*/
-const INSTRUCTION_TARGETS = [
-	{
-		host: "claude-code",
-		path: ".claude/CLAUDE.md"
-	},
-	{
-		host: "codex",
-		path: ".codex/AGENTS.md"
-	},
-	{
-		host: "gemini",
-		path: ".gemini/GEMINI.md"
-	}
-];
-/** Repo-relative paths of every generated instruction file (global scope). */
-const INSTRUCTION_PATHS = INSTRUCTION_TARGETS.map((target) => target.path);
-function instructionContent(host) {
-	return `${[
-		INSTRUCTIONS_START,
-		GENERATED_MARKER,
-		`${OWNER_MARKER_PREFIX}host=${host} -->`,
-		"",
-		INSTRUCTION_BODY,
-		"",
-		INSTRUCTIONS_END
-	].join("\n")}\n`;
-}
-/**
-* Per-host global instruction definitions. Each is upserted as a marker block
-* so any user-authored content outside the markers in the target file is
-* preserved. Emitted in global scope only (see GLOBAL_ONLY_PATHS).
-*/
-function globalInstructionDefinitions() {
-	return INSTRUCTION_TARGETS.map(({ host, path }) => ({
-		block: {
-			end: INSTRUCTIONS_END,
-			start: INSTRUCTIONS_START
-		},
-		content: instructionContent(host),
-		host,
-		invocation: "(global instructions)",
-		path
-	}));
-}
-//#endregion
-export { INSTRUCTION_PATHS, globalInstructionDefinitions };