@aperant/framework 0.8.0 → 0.8.1

package/src/cli/commands/init.mjs

@@ -3,8 +3,9 @@
  * rewrite, 2026-04).
  *
  * Replaces the previous Claude-only `cp -r` installer with the per-runtime
- * transform pipeline (src/cli/install/pipeline.mjs). All 14 runtime targets
- * supported via `--<runtime>` flags or `--all`. Default with no flags in
+ * transform pipeline (src/cli/install/pipeline.mjs). Registered runtime
+ * targets are supported via `--<runtime>` flags or `--all`. Default with no
+ * flags in
  * non-TTY mode → `--claude` (backward compat — existing skills invoke
  * `apt-tools init <dir>` with no flags). Default in TTY → interactive prompt
  * with auto-detection markers.
@@ -40,12 +41,13 @@ import { buildAperantGitignoreTemplate } from '../config/upgrade-gitignore.mjs'
 import { detectPublicRemote } from '../git/remote.mjs'
 import { detectArchetype } from '../install/archetype.mjs'
 import { installKernel } from '../install/install-kernel.mjs'
+import { MANIFEST_FILENAME } from '../install/manifest.mjs'
 import {
 	discoverCanonicalFiles,
 	runInstall,
 	writeLastInstallSummary,
 } from '../install/pipeline.mjs'
-import { RUNTIMES, suggestRuntimes } from '../install/runtime-detect.mjs'
+import { getRuntime, RUNTIMES, suggestRuntimes } from '../install/runtime-detect.mjs'
 import { runLegacyCleanup } from '../install/runtime-migrate.mjs'
 import {
 	CLASS_A_SETTINGS,
@@ -392,6 +394,42 @@ function selectRuntimes(targetDir, parsedFlags) {
 	return [...suggested]
 }
 
+/**
+ * Return the runtime ids that should be installed during a no-schema-drift
+ * init. A current config is not enough to prove runtime files exist on this
+ * machine: team repos often carry `.aperant/config.json` but not local
+ * runtime roots/manifests.
+ *
+ * @param {string} targetDir
+ * @param {string[]} runtimeIds
+ * @returns {string[]}
+ */
+function runtimesMissingManifest(targetDir, runtimeIds) {
+	const missing = []
+	for (const id of runtimeIds) {
+		const runtime = getRuntime(id)
+		if (!runtime) continue
+		const manifestPath = join(targetDir, runtime.installRoot, MANIFEST_FILENAME)
+		if (!existsSync(manifestPath)) missing.push(id)
+	}
+	return missing
+}
+
+/**
+ * Current-schema init should still run the runtime installer when the user
+ * explicitly requested runtimes, when --force-runtimes is present, or when
+ * local runtime roots are suggested but have no apt manifest.
+ *
+ * @param {string} targetDir
+ * @param {ReturnType<typeof parseInstallFlags>} parsedFlags
+ * @returns {string[]}
+ */
+function selectNoDriftRuntimeInstallIds(targetDir, parsedFlags) {
+	if (parsedFlags.forceRuntimes) return selectRuntimes(targetDir, parsedFlags)
+	if (parsedFlags.runtimes_explicit) return parsedFlags.runtimes
+	return runtimesMissingManifest(targetDir, suggestRuntimes(targetDir))
+}
+
 /**
  * Non-TTY / scripted path — derive answers from flags + sensible defaults,
  * gated by the Class A/B classifier (spec R2 + R3).
@@ -663,11 +701,16 @@ async function interactiveInterview(targetDir, version, canonicalRoot, archetype
 			migration.applied_defaults.length === 0 &&
 			migration.auto_added.length === 0
 		if (noDrift) {
-			// 0.6.7 (Bug 2 fix) — `--force-runtimes` bypasses the TTY
-			// "Nothing to do." outro so the runtime walker re-stamps every
-			// manifest; cmdInit's force-runtimes synthesis path takes over.
-			if (parsedFlags.forceRuntimes) {
-				return { upgrade_noop: true, skipBootstrap, migration }
+			const runtimeInstallIds = selectNoDriftRuntimeInstallIds(targetDir, parsedFlags)
+			// Runtime repair bypasses the TTY "Nothing to do." outro so the
+			// runtime walker can re-stamp missing manifests first.
+			if (runtimeInstallIds.length > 0) {
+				return {
+					upgrade_noop: true,
+					skipBootstrap,
+					migration,
+					runtime_reinstall_runtimes: runtimeInstallIds,
+				}
 			}
 			process.stdout.write(`\n${aperantBanner(version)}\n\n`)
 			clack.intro('install')
@@ -1158,20 +1201,20 @@ export async function cmdInit(projectDir, extraArgs = []) {
 			)
 		}
 
-		// Upgrade no-drift — short-circuit before installRuntimes.
-		// /apt:update is the right surface for refreshing skills on a
-		// no-drift run; init's job here is to confirm the schema is current.
+		const noDriftRuntimeInstallIds = answers?.upgrade_noop
+			? (answers.runtime_reinstall_runtimes ??
+				selectNoDriftRuntimeInstallIds(installTarget, parsedFlags))
+			: []
+
+		// Upgrade no-drift with no runtime work — short-circuit before
+		// installRuntimes. /apt:update remains the right surface for refreshing
+		// already-manifested skills; init only repairs selected or suggested
+		// runtime roots that lack manifests.
 		// The bootstrap auto-chain still runs (it's deterministic and
 		// idempotent — preserves PROJECT.md, refreshes sidecar from it).
 		// In TTY mode clack already rendered "Nothing to do." to stdout,
 		// so suppress the JSON envelope (matches the cancelled path).
-		//
-		// 0.6.7 (Bug 2 fix) — `--force-runtimes` bypasses this short-circuit
-		// so `/apt:update` can re-stamp every runtime manifest even when
-		// `.aperant/config.json` is at the current schema. We synthesize a
-		// real InstallAnswers shape from the existing config so
-		// installRuntimes + finishInit run their normal upgrade path.
-		if (answers?.upgrade_noop && !parsedFlags.forceRuntimes) {
+		if (answers?.upgrade_noop && noDriftRuntimeInstallIds.length === 0) {
 			if (interactiveTty) return rawText('', 0)
 			const bootstrap = answers.skipBootstrap
 				? { status: 'skipped', reason: '--skip-bootstrap flag' }
@@ -1187,18 +1230,14 @@ export async function cmdInit(projectDir, extraArgs = []) {
 			})
 		}
 
-		// 0.6.7 (Bug 2 fix) — when forceRuntimes is set on a no-drift run,
-		// synthesize a real InstallAnswers shape from the existing config so
-		// the runtime walker re-stamps every manifest. Visibility/autonomy/
-		// cross-verification are read from answers.migration.migrated (already
-		// in memory from nonInteractiveAnswers — avoids a second config read);
-		// runtimes resolved via selectRuntimes helper shared with the upgrade path.
+		// Current-schema runtime install — synthesize a real InstallAnswers
+		// shape from the existing config so installRuntimes + finishInit run
+		// their normal upgrade path.
 		let effectiveAnswers = answers
-		if (answers?.upgrade_noop && parsedFlags.forceRuntimes) {
+		if (answers?.upgrade_noop && noDriftRuntimeInstallIds.length > 0) {
 			const existing = answers.migration?.migrated || {}
-			const runtimes = selectRuntimes(installTarget, parsedFlags)
 			effectiveAnswers = {
-				runtimes,
+				runtimes: noDriftRuntimeInstallIds,
 				autonomyDefault: existing?.autonomy?.default ?? 1,
 				visibility: existing?.share?.visibility === 'team' ? 'team' : 'solo',
 				crossVerificationTools: existing?.multi_model?.cross_verification?.tools ?? [],

package/src/cli/install/cli-to-install-root.mjs

@@ -11,8 +11,6 @@
  * the lookups below work automatically.
  *
  * `null` is returned for:
- *   - CLIs with no install target (e.g. `pi` — host detected, but no apt
- *     install root exists for the Pi runtime).
  *   - `'unknown'` — deterministic branches (init/status/help) don't need a
  *     mapping; the router falls back to the framework-scoped dev default
  *     defined in getTrustedSkillSources().

package/src/cli/install/runtime-detect.mjs

@@ -30,9 +30,7 @@ import { MANIFEST_FILENAME } from './manifest.mjs'
  * @property {string|null} cliId    The `cli` string `detectHost()` returns
  *                                  when this runtime's CLI is at the prompt.
  *                                  `null` when the runtime has no install
- *                                  target mapped to a host CLI (e.g. `pi`,
- *                                  which has a detectable host but no apt
- *                                  install root of its own).
+ *                                  target mapped to a host CLI.
  * @property {string} [displayLabel] Optional user-facing label (e.g. "Kilo Code").
  *                                  When unset, callers fall back to the
  *                                  capitalized form of `id`. Use for runtimes
@@ -77,6 +75,13 @@ export const RUNTIMES = Object.freeze([
 		markers: ['.opencode/config.json'],
 		cliId: 'opencode',
 	},
+	{
+		id: 'pi',
+		flag: '--pi',
+		installRoot: '.pi',
+		markers: ['.pi/settings.json', '.pi/skills', '.pi/prompts'],
+		cliId: 'pi',
+	},
 	{
 		id: 'kilo',
 		flag: '--kilo',

package/src/cli/install/transforms/pi.mjs

@@ -0,0 +1,37 @@
+/**
+ * install/transforms/pi.mjs — Pi coding agent transform.
+ *
+ * Pi uses prompt templates under `.pi/prompts/` for slash commands and
+ * root-level skills under `.pi/skills/`. Unlike Codex, Pi supports
+ * colon-form prompt names, so `/apt:foo` references are preserved.
+ */
+
+import { basename, dirname } from 'node:path'
+import { substituteVersion } from './_shared.mjs'
+
+function promptNameForSkill(folder) {
+	if (folder === 'apt') return 'apt'
+	if (folder.startsWith('apt-')) return `apt:${folder.slice('apt-'.length)}`
+	return folder
+}
+
+/**
+ * @param {import('../pipeline.mjs').TransformInput} input
+ * @returns {import('../pipeline.mjs').TransformOutput[]}
+ */
+export function transform({ canonicalContent, canonicalRelPath, version, fileType }) {
+	const content = substituteVersion(canonicalContent, version)
+	if (fileType === 'skill') {
+		const folder = basename(dirname(canonicalRelPath))
+		const promptName = promptNameForSkill(folder)
+		return [
+			{ outputPath: `skills/${folder}/SKILL.md`, content },
+			{ outputPath: `prompts/${promptName}.md`, content },
+		]
+	}
+	if (fileType === 'agent') {
+		const file = basename(canonicalRelPath)
+		return [{ outputPath: `skills/${file}`, content }]
+	}
+	return []
+}