guardian-framework 0.1.157 → 0.1.158

package/dist/cli.js

@@ -3206,7 +3206,7 @@ __export(exports_package, {
   bin: () => bin,
   author: () => author
 });
-var name = "guardian-framework", version = "0.1.157", description = "Token-optimized agentic framework scaffolder with pi-first architecture", type = "module", main = "dist/exports.js", exports, types = "dist/exports.d.ts", bin, files, engines, scripts, publishConfig, pi, repository, homepage = "https://github.com/arman-jalili/guardian-framework#readme", bugs, dependencies, devDependencies, keywords, author = "Arman Wolkensteiner-Jalili", license = "MIT", package_default;
+var name = "guardian-framework", version = "0.1.158", description = "Token-optimized agentic framework scaffolder with pi-first architecture", type = "module", main = "dist/exports.js", exports, types = "dist/exports.d.ts", bin, files, engines, scripts, publishConfig, pi, repository, homepage = "https://github.com/arman-jalili/guardian-framework#readme", bugs, dependencies, devDependencies, keywords, author = "Arman Wolkensteiner-Jalili", license = "MIT", package_default;
 var init_package = __esm(() => {
   exports = {
     ".": {

package/dist/exports.js

@@ -1991,7 +1991,7 @@ __export(exports_package, {
   bin: () => bin,
   author: () => author
 });
-var name = "guardian-framework", version = "0.1.157", description = "Token-optimized agentic framework scaffolder with pi-first architecture", type = "module", main = "dist/exports.js", exports, types = "dist/exports.d.ts", bin, files, engines, scripts, publishConfig, pi, repository, homepage = "https://github.com/arman-jalili/guardian-framework#readme", bugs, dependencies, devDependencies, keywords, author = "Arman Wolkensteiner-Jalili", license = "MIT", package_default;
+var name = "guardian-framework", version = "0.1.158", description = "Token-optimized agentic framework scaffolder with pi-first architecture", type = "module", main = "dist/exports.js", exports, types = "dist/exports.d.ts", bin, files, engines, scripts, publishConfig, pi, repository, homepage = "https://github.com/arman-jalili/guardian-framework#readme", bugs, dependencies, devDependencies, keywords, author = "Arman Wolkensteiner-Jalili", license = "MIT", package_default;
 var init_package = __esm(() => {
   exports = {
     ".": {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "guardian-framework",
-	"version": "0.1.157",
+	"version": "0.1.158",
 	"description": "Token-optimized agentic framework scaffolder with pi-first architecture",
 	"type": "module",
 	"main": "dist/exports.js",

package/templates/pi/extensions/architect-lib/tdd-generator.ts

@@ -98,6 +98,61 @@ export interface TddGenerateOptions {
  * Generate failing test files for all components in an epic slice.
  * Returns list of generated file paths.
  */
+/**
+ * Identifier builders shared by every language generator.
+ *
+ * Component and module ids are human labels ("aggregate-household",
+ * "payee review & digest", "--bankdigest-"), not identifiers. Interpolating
+ * them raw produced invalid code:
+ *   Python  def test_aggregate-household_is_defined():   -> SyntaxError
+ *   Rust    fn test_aggregate-household_is_defined()     -> will not compile
+ *   Go/Java/CamelCase(...)  names that start with a digit or keep spaces
+ *
+ * Every generated identifier now goes through one of these.
+ */
+
+/** Split a human label into alphanumeric words (never empty). */
+function identWords(raw: string): string[] {
+	const words = (raw ?? "")
+		.replace(/([a-z0-9])([A-Z])/g, "$1 $2")
+		.split(/[^A-Za-z0-9]+/)
+		.filter((w) => w.length > 0);
+	return words.length > 0 ? words : ["component"];
+}
+
+/** `aggregate-household` -> `aggregate_household` (python, rust). */
+export function snakeIdent(raw: string): string {
+	const joined = identWords(raw)
+		.map((w) => w.toLowerCase())
+		.join("_");
+	// Identifiers may not start with a digit.
+	return /^[0-9]/.test(joined) ? `n_${joined}` : joined;
+}
+
+/** `aggregate-household` -> `AggregateHousehold` (java, go, ts types). */
+export function pascalIdent(raw: string): string {
+	const joined = identWords(raw)
+		.map((w) => w.charAt(0).toUpperCase() + w.slice(1).toLowerCase())
+		.join("");
+	return /^[0-9]/.test(joined) ? `N${joined}` : joined;
+}
+
+/** `aggregate-household` -> `aggregateHousehold` (ts values). */
+export function camelIdent(raw: string): string {
+	const p = pascalIdent(raw);
+	return p.charAt(0).toLowerCase() + p.slice(1);
+}
+
+/** Kept for callers/tests that ask for a Python identifier specifically. */
+export function pyIdent(raw: string): string {
+	return snakeIdent(raw);
+}
+
+/** Components are not required to declare dependencies. */
+function depsOf(component: { dependencies?: string[] }): string[] {
+	return component.dependencies ?? [];
+}
+
 export function generateEpicTestFiles(options: TddGenerateOptions): string[] {
 	const {
 		components,
@@ -176,7 +231,7 @@ function getTestFileName(componentId: string, language: string): string {
 		case "go":
 			return `${componentId}_test.go`;
 		case "java":
-			return `${toPascalCase(componentId)}Test.java`;
+			return `${pascalIdent(componentId)}Test.java`;
 		default:
 			return `${componentId}.test.ts`;
 	}
@@ -228,19 +283,19 @@ function generateTypeScript(
 		`it("should be defined", () => {`,
 		`  expect(false).toBe(true);`,
 		`  // TODO: uncomment when ${component.name} is implemented`,
-		`  // const instance = new ${component.name.replace(/[^a-zA-Z0-9]/g, "")}();`,
+		`  // const instance = new ${pascalIdent(component.name)}();`,
 		`  // expect(instance).toBeDefined();`,
 		`});`,
 	], 1));
 
 	// Dependency interaction tests
-	for (const dep of component.dependencies) {
+	for (const dep of depsOf(component)) {
 		lines.push(...indent([
 			`it("should interact with ${dep}", () => {`,
 			`  expect(false).toBe(true);`,
 			`  // TODO: implement dependency test for ${dep}`,
 			`  // const mock = { /* mock ${dep} interface */ };`,
-			`  // const result = new ${component.name.replace(/[^a-zA-Z0-9]/g, "")}(mock);`,
+			`  // const result = new ${pascalIdent(component.name)}(mock);`,
 			`  // expect(result).toBeDefined();`,
 			`});`,
 		], 1));
@@ -268,21 +323,21 @@ function generateRust(
 		`    use super::*;`,
 		``,
 		`    #[test]`,
-		`    fn test_${componentId}_is_defined() {`,
+		`    fn test_${snakeIdent(componentId)}_is_defined() {`,
 		`        // TODO: uncomment when ${component.name} is implemented`,
-		`        // let _instance = ${toPascalCase(componentId)}::new();`,
+		`        // let _instance = ${pascalIdent(componentId)}::new();`,
 		`        assert!(false, "Test not yet implemented");`,
 		`    }`,
 	];
 
-	for (const dep of component.dependencies) {
+	for (const dep of depsOf(component)) {
 		lines.push(...[
 			``,
 			`    #[test]`,
-			`    fn test_${componentId}_interacts_with_${dep.replace(/[^a-z0-9]/gi, "_").toLowerCase()}() {`,
+			`    fn test_${snakeIdent(componentId)}_interacts_with_${snakeIdent(dep)}() {`,
 			`        // TODO: implement dependency test for ${dep}`,
 			`        // let mock = Mock${dep.replace(/[^a-zA-Z0-9]/g, "")}::new();`,
-			`        // let result = ${toPascalCase(componentId)}::new(mock);`,
+			`        // let result = ${pascalIdent(componentId)}::new(mock);`,
 			`        assert!(false, "Dependency test not yet implemented");`,
 			`    }`,
 		]);
@@ -300,7 +355,7 @@ function generatePython(
 	moduleId: string,
 	componentId: string,
 ): string {
-	const clsName = toPascalCase(componentId);
+	const clsName = pascalIdent(componentId);
 	const lines: string[] = [
 		`# Generated by Guardian TDD`,
 		`# From: ${component.name} (${moduleId} module)`,
@@ -312,17 +367,17 @@ function generatePython(
 		`class Test${clsName}:`,
 		`    """Tests for ${component.name}."""`,
 		``,
-		`    def test_${componentId}_is_defined(self):`,
+		`    def test_${snakeIdent(componentId)}_is_defined(self):`,
 		`        """${component.name} should be instantiable."""`,
 		`        assert False, "TODO: uncomment when ${component.name} is implemented"`,
 		`        # instance = ${clsName}()`,
 		`        # assert instance is not None`,
 	];
 
-	for (const dep of component.dependencies) {
+	for (const dep of depsOf(component)) {
 		lines.push(...[
 			``,
-			`    def test_${componentId}_interacts_with_${dep.replace(/[^a-z0-9]/gi, "_").toLowerCase()}(self):`,
+			`    def test_${snakeIdent(componentId)}_interacts_with_${snakeIdent(dep)}(self):`,
 			`        """${component.name} should interact with ${dep}."""`,
 			`        assert False, "TODO: implement dependency test for ${dep}"`,
 			`        # mock = MagicMock(spec=${dep.replace(/[^a-zA-Z0-9]/g, "")})`,
@@ -348,13 +403,13 @@ function generateGo(
 		`// From: ${component.name} (${moduleId} module)`,
 		`// These tests will FAIL until ${component.name} is implemented.`,
 		``,
-		`package ${moduleId.replace(/[^a-zA-Z0-9]/g, "").toLowerCase()}`,
+		`package ${snakeIdent(moduleId).replace(/_/g, "")}`,
 		``,
 		`import "testing"`,
 		``,
-		`func Test${toPascalCase(componentId)}_IsDefined(t *testing.T) {`,
+		`func Test${pascalIdent(componentId)}_IsDefined(t *testing.T) {`,
 		`	// TODO: uncomment when ${component.name} is implemented`,
-		`	// instance := New${toPascalCase(componentId)}()`,
+		`	// instance := New${pascalIdent(componentId)}()`,
 		`	// if instance == nil {`,
 		`	// 	t.Fail()`,
 		`	// }`,
@@ -362,13 +417,13 @@ function generateGo(
 		`}`,
 	];
 
-	for (const dep of component.dependencies) {
+	for (const dep of depsOf(component)) {
 		lines.push(...[
 			``,
-			`func Test${toPascalCase(componentId)}_InteractsWith_${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))}(t *testing.T) {`,
+			`func Test${pascalIdent(componentId)}_InteractsWith_${pascalIdent(dep)}(t *testing.T) {`,
 			`	// TODO: implement dependency test for ${dep}`,
-			`	// mock := new(Mock${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))})`,
-			`	// instance := New${toPascalCase(componentId)}(mock)`,
+			`	// mock := new(Mock${pascalIdent(dep)})`,
+			`	// instance := New${pascalIdent(componentId)}(mock)`,
 			`	// result := instance.Execute()`,
 			`	t.Fatal("Dependency test not yet implemented")`,
 			`}`,
@@ -387,7 +442,7 @@ function generateJava(
 	componentId: string,
 	javaPackage?: string,
 ): string {
-	const clsName = toPascalCase(componentId);
+	const clsName = pascalIdent(componentId);
 	const lines: string[] = [
 		`// Generated by Guardian TDD`,
 		`// From: ${component.name} (${moduleId} module)`,
@@ -415,13 +470,13 @@ function generateJava(
 		`    }`,
 	);
 
-	for (const dep of component.dependencies) {
+	for (const dep of depsOf(component)) {
 		lines.push(
 			``,
 			`    @Test`,
-			`    void test${clsName}_InteractsWith_${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))}() {`,
+			`    void test${clsName}_InteractsWith_${pascalIdent(dep)}() {`,
 			`        // TODO: implement dependency test for ${dep}`,
-			`        // ${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))} mock = mock(${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))}.class);`,
+			`        // ${pascalIdent(dep)} mock = mock(${pascalIdent(dep)}.class);`,
 			`        // ${clsName} instance = new ${clsName}(mock);`,
 			`        fail("Dependency test not yet implemented");`,
 			`    }`,
@@ -434,14 +489,6 @@ function generateJava(
 
 // ── Utility ──
 
-function capitalize(s: string): string {
-	return s.charAt(0).toUpperCase() + s.slice(1);
-}
-
-function toPascalCase(s: string): string {
-	return s.split("-").map(capitalize).join("");
-}
-
 function indent(lines: string[], level: number): string[] {
 	const pad = "  ".repeat(level);
 	return lines.map((l) => `${pad}${l}`);

package/templates/pi/preflight_report.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2026-11-13T17:52:57Z",
+  "timestamp": "2026-11-13T19:00:09Z",
   "mode": "all",
   "stages_run": [],
   "summary": {

package/templates/pi/scripts/epic-runner.mjs

@@ -62,7 +62,10 @@ const args = ["--mode", "rpc"];
 if (opts.session) args.push("--session-id", opts.session);
 else args.push("--no-session");
 
-const child = spawn("pi", args, {
+// Explicit binary wins: tests and CI set GUARDIAN_PI_BIN so resolution never
+// depends on PATH (a project's node_modules/.bin can contain another `pi`).
+const piBin = process.env.GUARDIAN_PI_BIN || "pi";
+const child = spawn(piBin, args, {
 	cwd: opts.cwd,
 	stdio: ["pipe", "pipe", "pipe"],
 	env: process.env,