@forgeailab/spark 0.1.3 → 0.3.0

package/README.md

@@ -36,13 +36,6 @@ Print everything the install would do — files touched, env vars added, deps in
 spark info payments-stripe
 ```
 
-For **hybrid** packs (those with a `[runtime_package]` block in their manifest), `info` also prints:
-
-- `Install mode: hybrid` (vs `copy` for the rest)
-- `Runtime helper: <package> (range <X>, resolved <Y>)` — the npm name + manifest version range + the version currently installed in the project's `package.json` (or `not installed`).
-
-For copy packs `info` prints `Install mode: copy` and omits the helper line.
-
 ### `add <pack...> [--dry-run]`
 
 Resolve the requested packs against the registry + installed set + active scaffold, then apply. Idempotent: running it twice with the same args is a no-op.
@@ -58,13 +51,6 @@ The resolver enforces:
 - **Exclusivity.** Capabilities classified exclusive (`db`, `auth`, `payments`, `ui-kit`, `sync`) reject double-installs. Non-exclusive caps (`ai-sdk`, `analytics`, `email`, `blob-storage`, `e2e`, `deploy-target`, `local-runtime`) coexist.
 - **Scaffold compatibility.** A pack with `compatible_scaffolds = ["nextjs"]` refuses to install on a `vite-react` project.
 
-**Hybrid pack install.** When a pack declares `[runtime_package]`, the CLI adds the helper package to the same `bun add` batch as the pack's explicit runtime deps. Two resolution modes:
-
-- **Dev mode** — `SPARK_ROOT` is set and `${SPARK_ROOT}/libs/<helper>/` exists. The helper is linked via `file:` to the workspace path. Used by the reference app and `/tmp/spark-validate` smoke installs.
-- **Published mode** — otherwise. The helper is installed by `<npm-name>@<range>` from the manifest's `[runtime_package].version`.
-
-Either way the pack's `[dependencies].runtime` array MUST NOT also list the helper — the CLI handles it implicitly (see Decision 6 in the runtime-packages design doc).
-
 ### `preset <name>`
 
 Apply a named bundle from `presets/<name>.toml`. Refuses if the active scaffold is not in the preset's `compatible_scaffolds`.
@@ -75,7 +61,7 @@ spark preset lean-saas
 
 ### `check`
 
-Audit `.spark/state.json` against the filesystem. Reports missing files, missing env vars in `.env.local`, deleted seeded tasks. For each installed **hybrid** pack, also verifies the helper package is still listed in the consumer's `package.json` (under `dependencies` or `devDependencies`) — surfaces a `drift: helper packages` section when a helper has been removed manually. **Does not repair.**
+Audit `.spark/state.json` against the filesystem. Reports missing files, missing env vars in `.env.local`, and deleted seeded tasks. **Does not repair.**
 
 ```bash
 spark check

package/package.json

@@ -1,17 +1,19 @@
 {
   "name": "@forgeailab/spark",
-  "version": "0.1.3",
+  "version": "0.3.0",
   "description": "CLI for managing feature packs in an spark-scaffolded project.",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/ForgeAILab/spark.git",
+    "directory": "packages/spark"
+  },
   "type": "module",
   "publishConfig": {
     "access": "public"
   },
   "files": [
     "src",
-    "README.md",
-    "packs",
-    "presets",
-    "templates"
+    "README.md"
   ],
   "bin": {
     "spark": "./src/cli.ts"
@@ -21,10 +23,7 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@forgeailab/spark-schema": "^0.1.3",
-    "@forgeailab/spark-state": "^0.1.3",
-    "@forgeailab/spark-skill-utils": "^0.1.3",
-    "@forgeailab/spark-board": "^0.1.3",
+    "@forgeailab/spark-schema": "0.3.0",
     "@clack/prompts": "latest",
     "citty": "latest",
     "picocolors": "latest",

package/src/commands/add.ts

@@ -17,7 +17,6 @@ import { readRegistry, type Registry } from '../io/registry.ts';
 import { addInstalledPack, installedPackNames, readState, writeState } from '../io/state.ts';
 import { seedBoardTasks } from '../io/board.ts';
 import { copyPackSkills } from '../io/skills.ts';
-import { assertRuntimeHelperNotRedeclared, resolveRuntimeHelper } from '../runtime-package.ts';
 
 type AddOutput = Pick<Console, 'log' | 'error'>;
 
@@ -66,20 +65,8 @@ function formatResolverError(error: ResolverError): string {
   return `Unknown pack: ${error.pack}`;
 }
 
-async function packRuntimeDependencies(plan: InstallPlan): Promise<string[]> {
-  const dependencies: string[] = [];
-
-  for (const pack of plan.packs) {
-    assertRuntimeHelperNotRedeclared(pack.name, pack.manifest);
-    dependencies.push(...(pack.manifest.dependencies?.runtime ?? []));
-
-    const helper = await resolveRuntimeHelper(pack.manifest);
-    if (helper) {
-      dependencies.push(helper);
-    }
-  }
-
-  return dependencies;
+function packRuntimeDependencies(plan: InstallPlan): string[] {
+  return plan.packs.flatMap((pack) => pack.manifest.dependencies?.runtime ?? []);
 }
 
 function packDevDependencies(plan: InstallPlan): string[] {
@@ -182,11 +169,13 @@ function stateEntryForPack(
   skillFiles: readonly string[],
   touchedEnvFiles: readonly string[],
 ): StateInstalledPack {
+  const now = new Date();
+  const packInstallChange = `pack-install-${now.getUTCFullYear()}-${String(now.getUTCMonth() + 1).padStart(2, '0')}-${String(now.getUTCDate()).padStart(2, '0')}`;
   const fileTargets = [
     ...fileRecords.map((record) => record.to),
     ...skillFiles,
     ...touchedEnvFiles,
-    ...(taskIds.length > 0 ? ['.ai/board.md'] : []),
+    ...(taskIds.length > 0 ? [`docs/spark/changes/${packInstallChange}/tasks.md`] : []),
   ];
 
   return {
@@ -235,7 +224,7 @@ export async function runAdd(requestedPacks: readonly string[], options: AddOpti
     };
   }
 
-  const runtimeDependencies = await packRuntimeDependencies(plan);
+  const runtimeDependencies = packRuntimeDependencies(plan);
   const devDependencies = packDevDependencies(plan);
 
   if (options.dryRun) {

package/src/commands/check.ts

@@ -2,11 +2,10 @@ import { readFile, stat } from 'node:fs/promises';
 import { join } from 'node:path';
 import { defineCommand } from 'citty';
 import pc from 'picocolors';
-import { readConfig, type AppSkillsConfig } from '../config.ts';
+import type { AppSkillsConfig } from '../config.ts';
 import { missingBoardTasks } from '../io/board.ts';
-import { readRegistry, type Registry } from '../io/registry.ts';
+import type { Registry } from '../io/registry.ts';
 import { readState } from '../io/state.ts';
-import { installedRuntimeHelperSpecifier } from '../runtime-package.ts';
 
 type CheckOutput = Pick<Console, 'log' | 'error'>;
 
@@ -14,7 +13,6 @@ export type DriftReport = {
   missingFiles: string[];
   missingEnv: string[];
   missingTasks: string[];
-  missingHelpers: string[];
 };
 
 export type CheckOptions = {
@@ -56,13 +54,9 @@ async function readEnvLocal(projectRoot: string): Promise<string> {
 export async function runCheck(
   projectRoot = process.cwd(),
   output: CheckOutput = console,
-  options: CheckOptions = {},
+  _options: CheckOptions = {},
 ): Promise<DriftReport> {
-  const [, registry, state] = await Promise.all([
-    options.config ? Promise.resolve(options.config) : readConfig(projectRoot),
-    options.registry ? Promise.resolve(options.registry) : readRegistry(projectRoot),
-    readState(projectRoot),
-  ]);
+  const state = await readState(projectRoot);
   const recordedFiles = [
     ...new Set(state.installed_packs.flatMap((pack) => pack.files)),
   ].sort();
@@ -79,33 +73,17 @@ export async function runCheck(
   const envLocal = await readEnvLocal(projectRoot);
   const missingEnv = recordedEnv.filter((key) => !hasEnvVar(envLocal, key));
   const missingTasks = await missingBoardTasks(projectRoot, recordedTasks);
-  const missingHelpers: string[] = [];
-
-  for (const pack of state.installed_packs) {
-    const runtimePackage = registry.packs.get(pack.name)?.manifest.runtime_package;
-    if (!runtimePackage) {
-      continue;
-    }
-
-    if (!(await installedRuntimeHelperSpecifier(projectRoot, runtimePackage))) {
-      missingHelpers.push(
-        `${pack.name}: helper package ${runtimePackage.package} missing from package.json`,
-      );
-    }
-  }
 
   if (
     missingFiles.length === 0 &&
     missingEnv.length === 0 &&
-    missingTasks.length === 0 &&
-    missingHelpers.length === 0
+    missingTasks.length === 0
   ) {
     output.log(pc.green('OK: spark state matches the project filesystem.'));
     return {
       missingFiles,
       missingEnv,
       missingTasks,
-      missingHelpers,
     };
   }
 
@@ -132,18 +110,10 @@ export async function runCheck(
     }
   }
 
-  if (missingHelpers.length > 0) {
-    output.error('drift: helper packages');
-    for (const helper of missingHelpers) {
-      output.error(`  ${helper}`);
-    }
-  }
-
   return {
     missingFiles,
     missingEnv,
     missingTasks,
-    missingHelpers,
   };
 }
 
@@ -157,8 +127,7 @@ export const checkCommand = defineCommand({
     if (
       report.missingFiles.length > 0 ||
       report.missingEnv.length > 0 ||
-      report.missingTasks.length > 0 ||
-      report.missingHelpers.length > 0
+      report.missingTasks.length > 0
     ) {
       process.exitCode = 1;
     }

package/src/commands/info.ts

@@ -3,7 +3,6 @@ import pc from 'picocolors';
 import { readConfig, type AppSkillsConfig } from '../config.ts';
 import { readRegistry, type Registry } from '../io/registry.ts';
 import { installedPackNames, readState } from '../io/state.ts';
-import { formatResolvedRuntimeHelper } from '../runtime-package.ts';
 
 type InfoOutput = Pick<Console, 'log'>;
 
@@ -50,13 +49,7 @@ export async function runInfo(
   output.log(pc.bold(`${manifest.name}@${manifest.version}`));
   output.log(manifest.description ?? '');
   output.log(`status: ${installed ? 'installed' : 'available'}`);
-  output.log(`Install mode: ${manifest.runtime_package ? 'hybrid' : 'copy'}`);
-  if (manifest.runtime_package) {
-    const resolved = await formatResolvedRuntimeHelper(projectRoot, manifest.runtime_package);
-    output.log(
-      `Runtime helper: ${manifest.runtime_package.package} (range ${manifest.runtime_package.version}, resolved ${resolved})`,
-    );
-  }
+  output.log('Install mode: copy');
   output.log(`category: ${manifest.category}`);
   output.log(`provides: ${formatList(manifest.provides)}`);
   output.log(`requires: ${formatList(manifest.requires)}`);

package/src/commands/preset.ts

@@ -27,6 +27,7 @@ export async function runPreset(
     projectRoot,
     registry,
     config,
+    yes: true,
   });
 }