@oisincoveney/pipeline 1.8.0 → 1.8.1

package/README.md

@@ -37,11 +37,17 @@ Momokaya remote endpoint
 
 The default GitHub MCP registration uses GitHub's official container in
 read-only mode and reads `GITHUB_PERSONAL_ACCESS_TOKEN` from the environment.
-The Momokaya Qdrant endpoint is protected by Traefik HTTP Basic auth. Set either
-`MOMOKAYA_MCP_AUTHORIZATION` to the complete Authorization header value
-(`Basic <base64-user-colon-password>`) or `MEMORY_MCP_BASIC_AUTH` to only the
-base64 payload. `pipe init` resolves that value before registering the remote
-server with MCPM.
+The Momokaya Qdrant endpoint is protected by Traefik HTTP Basic auth. Set
+`MEMORY_MCP_BASIC_AUTH` to the base64 `user:password` payload before running
+`pipe init` if you want init to register that remote server with MCPM:
+
+```shell
+export MEMORY_MCP_BASIC_AUTH="$(printf '%s' 'user:password' | base64)"
+```
+
+When `MEMORY_MCP_BASIC_AUTH` is not set, `pipe init` still writes the default
+scaffold and keeps the generated `qdrant` MCP entry, but skips immediate MCPM
+registration for that private endpoint.
 
 Check local prerequisites and config health:
 

package/defaults/install-manifest.json

@@ -78,12 +78,10 @@
       "name": "oisin-pipeline-qdrant",
       "transport": "remote",
       "url": "https://memory-mcp.momokaya.ee/mcp/",
+      "optionalRegistration": true,
       "headers": {
         "Authorization": {
-          "sources": [
-            { "env": "MOMOKAYA_MCP_AUTHORIZATION" },
-            { "env": "MEMORY_MCP_BASIC_AUTH", "prefix": "Basic " }
-          ]
+          "sources": [{ "env": "MEMORY_MCP_BASIC_AUTH", "prefix": "Basic " }]
         }
       }
     },

package/dist/pipeline-init.js

@@ -29,6 +29,18 @@ var PipelineMcpInstallError = class extends Error {
 		this.name = "PipelineMcpInstallError";
 	}
 };
+var PipelineMcpMissingCredentialError = class extends PipelineMcpInstallError {
+	headerName;
+	missingEnv;
+	serverName;
+	constructor(serverName, headerName, missingEnv) {
+		super([`MCP server ${serverName} requires ${headerName} credentials before it can be registered.`, `Set ${missingEnv.join(" or ")} and re-run pipeline init.`].join("\n"));
+		this.name = "PipelineMcpMissingCredentialError";
+		this.serverName = serverName;
+		this.headerName = headerName;
+		this.missingEnv = missingEnv;
+	}
+};
 var PipelineDefaultManifestError = class extends Error {
 	constructor(message) {
 		super(message);
@@ -58,6 +70,7 @@ const pipelineMcpInstallSpecSchema = z.object({
 	env: z.record(z.string(), z.string()).optional(),
 	headers: z.record(z.string(), pipelineMcpHeaderValueSchema).optional(),
 	name: z.string().min(1),
+	optionalRegistration: z.boolean().optional(),
 	transport: z.enum(["remote", "stdio"]),
 	url: z.string().url().optional()
 }).strict().superRefine((spec, ctx) => {
@@ -824,8 +837,13 @@ async function installDefaultSkillsWithCli(specs, cwd) {
 	}
 }
 async function installDefaultMcpsWithCli(specs, cwd) {
+	const skipped = [];
 	for (const spec of specs) {
 		const install = mcpInstallArgs(spec);
+		if ("skipped" in install) {
+			skipped.push(install.skipped);
+			continue;
+		}
 		try {
 			await execa("uvx", [...DEFAULT_MCPM_ARGS, ...install.args], {
 				cwd,
@@ -848,6 +866,7 @@ async function installDefaultMcpsWithCli(specs, cwd) {
 			].filter(Boolean).join("\n"));
 		}
 	}
+	return { skipped };
 }
 function mcpInstallArgs(spec) {
 	if (spec.catalog) return {
@@ -870,19 +889,29 @@ function mcpInstallArgs(spec) {
 	if (spec.transport === "remote") {
 		if (!spec.url) throw new PipelineMcpInstallError(`MCP server ${spec.name} is remote but has no url.`);
 		const redactions = [];
-		return {
-			args: [
-				...args,
-				"--url",
-				spec.url,
-				...Object.entries(spec.headers ?? {}).flatMap(([key, value]) => {
-					const headerValue = resolveMcpHeaderValue(spec.name, key, value);
-					redactions.push(headerValue);
-					return ["--headers", `${key}=${headerValue}`];
-				})
-			],
-			redactions
-		};
+		try {
+			const headers = Object.entries(spec.headers ?? {}).flatMap(([key, value]) => {
+				const headerValue = resolveMcpHeaderValue(spec.name, key, value);
+				redactions.push(headerValue);
+				return ["--headers", `${key}=${headerValue}`];
+			});
+			return {
+				args: [
+					...args,
+					"--url",
+					spec.url,
+					...headers
+				],
+				redactions
+			};
+		} catch (err) {
+			if (spec.optionalRegistration && err instanceof PipelineMcpMissingCredentialError) return { skipped: {
+				missingEnv: err.missingEnv,
+				name: spec.name,
+				reason: `missing ${err.headerName} credentials`
+			} };
+			throw err;
+		}
 	}
 	if (!spec.command) throw new PipelineMcpInstallError(`MCP server ${spec.name} is stdio but has no command.`);
 	return {
@@ -914,8 +943,7 @@ function resolveMcpHeaderValue(serverName, headerName, header) {
 		const rawValue = process.env[source.env];
 		if (rawValue && rawValue.trim().length > 0) return `${source.prefix ?? ""}${rawValue}${source.suffix ?? ""}`;
 	}
-	const envNames = header.sources.map((source) => source.env).join(" or ");
-	throw new PipelineMcpInstallError([`MCP server ${serverName} requires ${headerName} credentials before it can be registered.`, `Set ${envNames} and re-run pipeline init.`].join("\n"));
+	throw new PipelineMcpMissingCredentialError(serverName, headerName, header.sources.map((source) => source.env));
 }
 function assertDefaultSkillsInstalled(cwd) {
 	const paths = defaultSkillPaths();
@@ -935,7 +963,7 @@ async function initPipelineProject(options = {}) {
 	const paths = Object.keys(files);
 	const conflicts = paths.filter((path) => existsSync(join(cwd, path)));
 	if (conflicts.length > 0 && !options.overwrite) throw new PipelineInitError(conflicts);
-	await (options.mcpInstaller ?? installDefaultMcpsWithCli)(DEFAULT_MCP_INSTALLS, cwd);
+	const mcpInstallResult = await (options.mcpInstaller ?? installDefaultMcpsWithCli)(DEFAULT_MCP_INSTALLS, cwd);
 	await (options.skillInstaller ?? installDefaultSkillsWithCli)(DEFAULT_SKILL_INSTALLS, cwd);
 	const skillPaths = assertDefaultSkillsInstalled(cwd);
 	for (const [path, content] of Object.entries(files)) {
@@ -949,10 +977,18 @@ async function initPipelineProject(options = {}) {
 		...existsSync(join(cwd, "skills-lock.json")) ? ["skills-lock.json"] : []
 	];
 	loadPipelineConfig(cwd);
-	return { files: generatedPaths };
+	return {
+		files: generatedPaths,
+		skippedMcps: mcpInstallResult?.skipped ?? []
+	};
 }
 function formatPipelineInitResult(result) {
-	return ["Initialized pipeline scaffold:", ...result.files.map((path) => `create ${path}`)].join("\n");
+	const skippedMcps = result.skippedMcps ?? [];
+	return [
+		"Initialized pipeline scaffold:",
+		...result.files.map((path) => `create ${path}`),
+		...skippedMcps.flatMap((skip) => [`Skipped MCPM registration for ${skip.name}: ${skip.reason}.`, `Set ${skip.missingEnv.join(" or ")} before retrying MCPM registration. The generated MCP entry remains in .mcp.json.`])
+	].join("\n");
 }
 function hostResourceInput(host) {
 	return [

package/package.json

@@ -82,7 +82,7 @@
     "prepack": "bun run build:cli"
   },
   "type": "module",
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "Config-driven multi-agent pipeline runner for repository work",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",