nomen-lang 0.6.0 → 0.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nomen-lang",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "The CLI for the Nomen programming language.",
   "keywords": [],
   "license": "ISC",

package/src/test.ts

@@ -3,7 +3,7 @@ import fs from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 
-import build from "../../src/build.ts";
+import build, { default_platform } from "../../src/build.ts";
 import join from "../../src/join.ts";
 import { get_library } from "../../src/lib.ts";
 import parse from "../../src/parse.ts";
@@ -40,6 +40,12 @@ export interface TestFileResult {
 	// Set when the compiled binary crashed before finishing (e.g. a segfault
 	// in a test). We still surface whatever records it managed to emit.
 	crashed?: string;
+	/**
+	 * The pipeline stage that produced `crashed`, so the report can name it
+	 * accurately — a binary that BUILT fine and then segfaulted before
+	 * emitting any record is not a build failure (see report_file).
+	 */
+	phase?: "parse" | "build" | "setup" | "link" | "run";
 	ms: number;
 }
 
@@ -289,6 +295,7 @@ export function run_test_file(
 	const parsed = parse(source, library, resolved);
 	if (parsed.errors.length) {
 		result.ok = false;
+		result.phase = "parse";
 		result.crashed = parsed.errors
 			.map((e) => `${e.message} (${e.line ?? "?"}:${e.column ?? "?"})`)
 			.join("\n");
@@ -302,6 +309,7 @@ export function run_test_file(
 	});
 	if (buildResult.errors && buildResult.errors.length) {
 		result.ok = false;
+		result.phase = "build";
 		result.crashed = buildResult.errors
 			.map((e) => `${e.message} (${e.line ?? "?"}:${e.column ?? "?"})`)
 			.join("\n");
@@ -321,6 +329,7 @@ export function run_test_file(
 			: find_audit_runtime(path.dirname(resolved));
 		if (!runtime_src || !fs.existsSync(runtime_src)) {
 			result.ok = false;
+			result.phase = "setup";
 			result.crashed =
 				"Audit enabled but audit_runtime.c was not found. Pass --audit-runtime <path>.";
 			result.ms = performance.now() - start;
@@ -329,7 +338,12 @@ export function run_test_file(
 		audit_obj = path.join(buildDir, "audit_runtime.o");
 		execFileSync("clang", ["-c", runtime_src, "-o", audit_obj]);
 	}
-	const ext = arch === "aarch64" ? ".s" : ".c";
+	const platform = default_platform();
+	// Match the run/build command's emission: ObjC-bearing sources (GUI code
+	// pulled in through the library graph) must land in a `.m` TU on Apple
+	// platforms — the ObjC framework headers are Objective-C, and including
+	// them from a plain `.c` TU fails with `@class NSString` syntax errors.
+	const ext = arch === "aarch64" ? ".s" : platform === "macos" || platform === "ios" ? ".m" : ".c";
 	const codefile = path.join(buildDir, path.basename(entry_path, ".nm") + ext);
 	const outfile = path.join(buildDir, path.basename(entry_path, ".nm"));
 	// The C backend's generated source does `#include "main.h"`, so write the
@@ -337,10 +351,23 @@ export function run_test_file(
 	fs.writeFileSync(path.join(buildDir, "main.h"), buildResult.headers ?? "");
 	fs.writeFileSync(codefile, buildResult.code);
 
-	// Link the harness binary. The harness uses only Console/Time (printf,
-	// clock_gettime), so no platform frameworks are required.
+	// Link the harness binary. Frameworks are passed on Apple platforms for
+	// the same reason as the run/build command: the joined source may reference
+	// ObjC runtime symbols even when the test body doesn't, and unused
+	// frameworks cost nothing at link time.
 	const link_args = ["-o", outfile, codefile];
 	if (audit_obj) link_args.push(audit_obj);
+	if (platform === "macos" || platform === "ios") {
+		link_args.push(
+			"-framework",
+			"CoreGraphics",
+			"-framework",
+			"Foundation",
+			"-framework",
+			"AppKit",
+			"-lobjc",
+		);
+	}
 	try {
 		execFileSync("clang", link_args, {
 			encoding: "utf8",
@@ -349,6 +376,7 @@ export function run_test_file(
 	} catch (err: any) {
 		const stderr = err.stderr ? err.stderr.toString() : (err.message ?? "");
 		result.ok = false;
+		result.phase = "link";
 		result.crashed = `link failed: ${stderr.trim() || "clang error"}`;
 		result.ms = performance.now() - start;
 		return result;
@@ -381,6 +409,7 @@ export function run_test_file(
 	result.benches = records.benches;
 	result.other = records.other;
 	result.crashed = crashed;
+	if (crashed) result.phase = "run";
 	// A file "passes" when it reported no failed asserts and didn't crash.
 	const failed = result.fails.length > 0 || result.tests.some((t) => t.failed > 0);
 	result.ok = !failed && crashed === undefined;
@@ -417,7 +446,18 @@ const C = {
 export function report_file(result: TestFileResult): void {
 	const rel = path.relative(process.cwd(), result.file);
 	if (result.crashed && result.tests.length === 0 && result.fails.length === 0) {
-		console.log(` ${C.red("✗")} ${rel} ${C.red("(failed to build)")}`);
+		// Name the stage that actually failed. A binary that linked fine and
+		// then died before emitting any record (e.g. SIGSEGV) is NOT a build
+		// failure — conflating the two sent debugging to the wrong layer.
+		const label =
+			result.phase === "run"
+				? "(crashed before records)"
+				: result.phase === "link"
+					? "(link failed)"
+					: result.phase === "setup"
+						? "(setup failed)"
+						: "(failed to build)";
+		console.log(` ${C.red("✗")} ${rel} ${C.red(label)}`);
 		console.log(C.red(result.crashed));
 		return;
 	}