guardian-framework 0.1.80 → 0.1.82

package/dist/cli.js

@@ -3197,7 +3197,7 @@ __export(exports_package, {
   bin: () => bin,
   author: () => author
 });
-var name = "guardian-framework", version = "0.1.80", 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.82", 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

@@ -1982,7 +1982,7 @@ __export(exports_package, {
   bin: () => bin,
   author: () => author
 });
-var name = "guardian-framework", version = "0.1.80", 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.82", 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.80",
+	"version": "0.1.82",
 	"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

@@ -9,10 +9,59 @@
  * reach the agent. The agent makes them green via Red→Green→Refactor.
  */
 
-import { existsSync, mkdirSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import type { ModuleComponent } from "./types.ts";
 
+// ── Java package derivation (from project manifest) ──
+
+/** Read groupId from guardian-manifest.json (never from pom.xml). */
+function readGroupId(cwd: string): string {
+	try {
+		const raw = readFileSync(join(cwd, "guardian-manifest.json"), "utf-8");
+		const manifest = JSON.parse(raw);
+		if (manifest.groupId && typeof manifest.groupId === "string") {
+			return manifest.groupId;
+		}
+	} catch {}
+	return "com.example";
+}
+
+/**
+ * Read the module→Java-package mapping from guardian-manifest.json.
+ * The manifest should have a `javaModulePackages` field:
+ * \`\`\`json
+ * {
+ *   "javaModulePackages": {
+ *     "integration-acl": "acl",
+ *     "order-intake": "orderintake"
+ *   }
+ * }
+ * \`\`\`
+ * Missing modules throw — no silent fallback to bad paths.
+ */
+function readJavaModulePackages(cwd: string): Record<string, string> {
+	try {
+		const raw = readFileSync(join(cwd, "guardian-manifest.json"), "utf-8");
+		const manifest = JSON.parse(raw);
+		if (manifest.javaModulePackages && typeof manifest.javaModulePackages === "object") {
+			return manifest.javaModulePackages;
+		}
+	} catch {}
+	return {};
+}
+
+/** Resolve a module ID to its Java sub-package suffix. Throws if unknown. */
+function javaModulePackage(cwd: string, moduleId: string): string {
+	const mapping = readJavaModulePackages(cwd);
+	const pkg = mapping[moduleId];
+	if (pkg) return pkg;
+	throw new Error(
+		`Unknown Java package for module "${moduleId}". ` +
+		`Add it to the "javaModulePackages" field in guardian-manifest.json.`
+	);
+}
+
 // ── Supported Languages ──
 
 const TDD_LANGUAGES = new Set(["typescript", "rust", "python", "go", "java"]);
@@ -60,8 +109,14 @@ export function generateEpicTestFiles(options: TddGenerateOptions): string[] {
 	let modulePath: string;
 
 	if (language === "java") {
-		testBaseDir = testBaseDir ?? "src/test/java";
-		modulePath = (moduleName || moduleId).replace(/[^a-zA-Z0-9\/]/g, "-");
+		// Java: src/test/java/<groupId-path>/<module-package>/
+		// All components in the same module share the same package dir.
+		const groupId = readGroupId(cwd);
+		const groupPath = groupId.replace(/\./g, "/");
+		const modPkg = javaModulePackage(cwd, moduleName || moduleId);
+		const modPath = modPkg.replace(/\./g, "/");
+		testBaseDir = testBaseDir ?? join("src", "test", "java");
+		modulePath = join(groupPath, modPath);
 	} else {
 		testBaseDir = testBaseDir ?? "tests/unit";
 		modulePath = moduleName || moduleId;
@@ -72,7 +127,19 @@ export function generateEpicTestFiles(options: TddGenerateOptions): string[] {
 	for (const comp of components) {
 		const componentId = comp.name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/(^-|-$)/g, "");
 
-		const testDir = join(cwd, testBaseDir, modulePath, componentId);
+		let testDir: string;
+		let javaPackage: string | undefined;
+
+		if (language === "java") {
+			// No componentId subdir — all components in same package dir
+			testDir = join(cwd, testBaseDir, modulePath);
+			const groupId = readGroupId(cwd);
+			const modPkg = javaModulePackage(cwd, moduleName || moduleId);
+			javaPackage = `${groupId}.${modPkg}`;
+		} else {
+			testDir = join(cwd, testBaseDir, modulePath, componentId);
+		}
+
 		mkdirSync(testDir, { recursive: true });
 
 		const fileName = getTestFileName(componentId, language);
@@ -81,7 +148,7 @@ export function generateEpicTestFiles(options: TddGenerateOptions): string[] {
 		// Skip if file already exists (don't overwrite user edits)
 		if (existsSync(filePath)) continue;
 
-		const content = generateTestContent(comp, language, moduleId, componentId);
+		const content = generateTestContent(comp, language, moduleId, componentId, javaPackage);
 		writeFileSync(filePath, content, "utf-8");
 		generated.push(filePath);
 	}
@@ -115,6 +182,7 @@ function generateTestContent(
 	language: string,
 	moduleId: string,
 	componentId: string,
+	javaPackage?: string,
 ): string {
 	switch (language) {
 		case "rust":
@@ -124,7 +192,7 @@ function generateTestContent(
 		case "go":
 			return generateGo(component, moduleId, componentId);
 		case "java":
-			return generateJava(component, moduleId, componentId);
+			return generateJava(component, moduleId, componentId, javaPackage);
 		case "typescript":
 		default:
 			return generateTypeScript(component, moduleId, componentId);
@@ -136,7 +204,7 @@ function generateTestContent(
 function generateTypeScript(
 	component: ModuleComponent,
 	moduleId: string,
-	_componentId: string,
+	componentId: string,
 ): string {
 	const lines: string[] = [
 		`// Generated by Guardian TDD`,
@@ -310,6 +378,7 @@ function generateJava(
 	component: ModuleComponent,
 	moduleId: string,
 	componentId: string,
+	javaPackage?: string,
 ): string {
 	const clsName = toPascalCase(componentId);
 	const lines: string[] = [
@@ -317,6 +386,14 @@ function generateJava(
 		`// From: ${component.name} (${moduleId} module)`,
 		`// These tests will FAIL until ${component.name} is implemented.`,
 		``,
+	];
+
+	// Always include package declaration for Java
+	if (javaPackage) {
+		lines.push(`package ${javaPackage};`, ``);
+	}
+
+	lines.push(
 		`import org.junit.jupiter.api.Test;`,
 		`import static org.junit.jupiter.api.Assertions.*;`,
 		``,
@@ -329,10 +406,10 @@ function generateJava(
 		`        // assertNotNull(instance);`,
 		`        fail("Test not yet implemented");`,
 		`    }`,
-	];
+	);
 
 	for (const dep of component.dependencies) {
-		lines.push(...[
+		lines.push(
 			``,
 			`    @Test`,
 			`    void test${clsName}_InteractsWith_${capitalize(dep.replace(/[^a-zA-Z0-9]/g, ""))}() {`,
@@ -341,11 +418,10 @@ function generateJava(
 			`        // ${clsName} instance = new ${clsName}(mock);`,
 			`        fail("Dependency test not yet implemented");`,
 			`    }`,
-		]);
+		);
 	}
 
-	lines.push(`}`);
-	lines.push(``);
+	lines.push(`}`, ``);
 	return lines.join("\n");
 }
 

package/templates/pi/preflight_report.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2026-07-16T10:04:14Z",
+  "timestamp": "2026-07-16T13:04:53Z",
   "mode": "all",
   "stages_run": [],
   "summary": {

package/templates/pi/skills/java-spring-enterprise-codegen.md

@@ -29,7 +29,9 @@ module/
 │   ├── repository/             # Repository interfaces
 │   │   └── EntityRepository.java
 │   └── error/                  # Typed exception classes
-│       └── DomainError.java
+│       ├── DomainException.java   # Abstract base for all domain errors
+│       ├── DomainError.java      # Concrete domain error
+│       └── ErrorCode.java        # Machine-readable error codes
 ├── application/                # Service interfaces and DTOs
 │   ├── service/                # Service interfaces
 │   │   └── EntityService.java
@@ -177,25 +179,39 @@ public enum Status {
 ## 3. Domain Error Handling
 
 ```java
-// domain/error/DomainError.java
+// domain/error/DomainException.java
 
 package com.project.module.domain.error;
 
-public class DomainError extends RuntimeException {
-
+/**
+ * Base exception for all domain errors.
+ * Extending this (rather than RuntimeException directly) allows
+ * catching all domain-level errors in a single handler.
+ */
+public abstract class DomainException extends RuntimeException {
     private final ErrorCode code;
 
-    public DomainError(String message) {
-        super(message);
-        this.code = ErrorCode.VALIDATION;
-    }
-
-    public DomainError(ErrorCode code, String message) {
+    protected DomainException(ErrorCode code, String message) {
         super(message);
         this.code = code;
     }
 
     public ErrorCode getCode() { return code; }
+}
+
+// domain/error/DomainError.java
+
+package com.project.module.domain.error;
+
+public class DomainError extends DomainException {
+
+    public DomainError(String message) {
+        super(ErrorCode.VALIDATION, message);
+    }
+
+    public DomainError(ErrorCode code, String message) {
+        super(code, message);
+    }
 
     public static DomainError notFound(String id) {
         return new DomainError(ErrorCode.NOT_FOUND, "Resource " + id + " not found");
@@ -221,7 +237,8 @@ public enum ErrorCode {
 ```
 
 ### Rules
-- ✅ Use `RuntimeException` subclasses for domain errors
+- ✅ Use `DomainException` subclasses for domain errors (never `RuntimeException` directly)
+- ✅ Define `DomainException` as an abstract base in `domain/error/`
 - ✅ Use `ErrorCode` enum for machine-readable error types
 - ✅ Use `record` for value objects (Java 16+)
 - ❌ Don't use Spring annotations in domain layer