@telorun/assert 0.1.12 → 0.3.0

package/README.md

@@ -1,7 +1,3 @@
----
-description: "Telo: YAML-driven execution engine for declarative backends with micro-kernel architecture and language-agnostic design"
----
-
 # ⚡ Telo
 
 Runtime for declarative backends.

package/dist/manifest.d.ts

@@ -11,6 +11,7 @@ interface ManifestAssertManifest {
     source: string;
     expect: {
         errors?: ExpectError[];
+        warnings?: ExpectError[];
         loadError?: string;
     };
 }

package/dist/manifest.js

@@ -1,7 +1,7 @@
 import { DEFAULT_MANIFEST_FILENAME, Loader, StaticAnalyzer } from "@telorun/analyzer";
 import * as fs from "fs/promises";
 import * as path from "path";
-class LocalFileAdapter {
+class LocalFileSource {
     supports(p) {
         return (p.startsWith("file://") ||
             p.startsWith("/") ||
@@ -52,7 +52,7 @@ export async function create(manifest, ctx) {
             const green = (t) => c("32", t);
             const dim = (t) => c("2", t);
             const name = manifest.metadata.name;
-            const loader = new Loader([new LocalFileAdapter()]);
+            const loader = new Loader([new LocalFileSource()]);
             const analyzer = new StaticAnalyzer();
             const resolvedUrl = new URL(manifest.source, ctx.moduleContext.source).toString();
             let manifests;
@@ -87,7 +87,9 @@ export async function create(manifest, ctx) {
             }
             const diagnostics = analyzer.analyze(manifests);
             const errors = diagnostics.filter((d) => d.severity === 1); // DiagnosticSeverity.Error = 1
+            const warnings = diagnostics.filter((d) => d.severity === 2); // DiagnosticSeverity.Warning = 2
             const expectedErrors = manifest.expect.errors ?? [];
+            const expectedWarnings = manifest.expect.warnings ?? [];
             const failures = [];
             const matched = [];
             if (expectedErrors.length === 0) {
@@ -108,17 +110,34 @@ export async function create(manifest, ctx) {
                         matched.push(`${expected.code ?? "*"}${expected.message ? ` (${expected.message})` : ""}`);
                     }
                     else {
-                        failures.push(`expected ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`);
+                        failures.push(`expected error ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`);
+                    }
+                }
+            }
+            // Warnings are checked only when the caller declares expect.warnings. Unexpected
+            // warnings are not failures (unlike errors) — warnings are advisory and may exist
+            // on manifests that are otherwise valid. When expect.warnings is present, every
+            // listed warning must be found; extras are ignored.
+            if (expectedWarnings.length > 0) {
+                for (const expected of expectedWarnings) {
+                    const match = warnings.find((d) => matchesDiagnostic(d, expected));
+                    if (match) {
+                        matched.push(`warning ${expected.code ?? "*"}${expected.message ? ` (${expected.message})` : ""}`);
+                    }
+                    else {
+                        failures.push(`expected warning ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`);
                     }
                 }
             }
             const passedLines = matched.map((m) => `  ${green("✓")} ${dim(m)}\n`).join("");
             if (failures.length > 0) {
                 const failedLines = failures.map((f) => `  ${red("✗")} ${f}\n`).join("");
-                const actualLines = errors.length > 0
-                    ? `  ${dim("actual errors:")}\n` +
-                        errors.map((d) => `    ${dim(`[${d.code}] ${d.message}`)}\n`).join("")
-                    : `  ${dim("no errors produced")}\n`;
+                const actualLines = errors.length > 0 || warnings.length > 0
+                    ? `  ${dim("actual diagnostics:")}\n` +
+                        [...errors, ...warnings]
+                            .map((d) => `    ${dim(`[${d.code}] ${d.message}`)}\n`)
+                            .join("")
+                    : `  ${dim("no diagnostics produced")}\n`;
                 ctx.stderr.write(bold(red(`Assert.Manifest.${name}: assertion failed`)) + "\n" +
                     passedLines + failedLines + actualLines);
                 ctx.requestExit(1);

package/dist/module-context.js

@@ -44,14 +44,13 @@ export async function create(manifest, ctx) {
     return {
         run: async () => {
             const { bold, red, green, yellow, dim } = createColors(ctx.stderr);
-            const declaringModule = manifest.metadata.module ?? "default";
             const resourcesToCheck = manifest.resources ?? {};
             const failures = [];
             const passed = [];
             const { resources } = ctx.moduleContext;
             for (const [alias, expected] of Object.entries(resourcesToCheck)) {
-                if (!ctx.resolveModuleAlias(declaringModule, alias)) {
-                    failures.push(`Import alias '${alias}' not found in module '${declaringModule}'`);
+                if (!ctx.moduleContext.hasImport(alias)) {
+                    failures.push(`Import alias '${alias}' not found in declaring module`);
                     continue;
                 }
                 const snap = resources[alias] ?? {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@telorun/assert",
-  "version": "0.1.12",
+  "version": "0.3.0",
   "description": "Telo Assert module - Assertion resource kinds for Telo manifests.",
   "keywords": [
     "telo",
@@ -31,8 +31,8 @@
   ],
   "dependencies": {
     "@sinclair/typebox": "^0.34.48",
-    "@telorun/analyzer": "0.3.0",
-    "@telorun/sdk": "0.3.2"
+    "@telorun/analyzer": "0.5.0",
+    "@telorun/sdk": "0.6.0"
   },
   "devDependencies": {
     "@types/node": "^20.0.0",

package/src/manifest.ts

@@ -1,4 +1,4 @@
-import { DEFAULT_MANIFEST_FILENAME, Loader, StaticAnalyzer, type AnalysisDiagnostic, type ManifestAdapter } from "@telorun/analyzer";
+import { DEFAULT_MANIFEST_FILENAME, Loader, StaticAnalyzer, type AnalysisDiagnostic, type ManifestSource } from "@telorun/analyzer";
 import type { ResourceContext, Runnable } from "@telorun/sdk";
 import * as fs from "fs/promises";
 import * as path from "path";
@@ -13,11 +13,12 @@ interface ManifestAssertManifest {
   source: string;
   expect: {
     errors?: ExpectError[];
+    warnings?: ExpectError[];
     loadError?: string;
   };
 }
 
-class LocalFileAdapter implements ManifestAdapter {
+class LocalFileSource implements ManifestSource {
   supports(p: string): boolean {
     return (
       p.startsWith("file://") ||
@@ -77,7 +78,7 @@ export async function create(
       const dim = (t: string) => c("2", t);
 
       const name = manifest.metadata.name;
-      const loader = new Loader([new LocalFileAdapter()]);
+      const loader = new Loader([new LocalFileSource()]);
       const analyzer = new StaticAnalyzer();
 
       const resolvedUrl = new URL(manifest.source, ctx.moduleContext.source).toString();
@@ -121,7 +122,9 @@ export async function create(
 
       const diagnostics = analyzer.analyze(manifests);
       const errors = diagnostics.filter((d) => d.severity === 1); // DiagnosticSeverity.Error = 1
+      const warnings = diagnostics.filter((d) => d.severity === 2); // DiagnosticSeverity.Warning = 2
       const expectedErrors = manifest.expect.errors ?? [];
+      const expectedWarnings = manifest.expect.warnings ?? [];
       const failures: string[] = [];
       const matched: string[] = [];
 
@@ -143,7 +146,26 @@ export async function create(
             );
           } else {
             failures.push(
-              `expected ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`,
+              `expected error ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`,
+            );
+          }
+        }
+      }
+
+      // Warnings are checked only when the caller declares expect.warnings. Unexpected
+      // warnings are not failures (unlike errors) — warnings are advisory and may exist
+      // on manifests that are otherwise valid. When expect.warnings is present, every
+      // listed warning must be found; extras are ignored.
+      if (expectedWarnings.length > 0) {
+        for (const expected of expectedWarnings) {
+          const match = warnings.find((d) => matchesDiagnostic(d, expected));
+          if (match) {
+            matched.push(
+              `warning ${expected.code ?? "*"}${expected.message ? ` (${expected.message})` : ""}`,
+            );
+          } else {
+            failures.push(
+              `expected warning ${expected.code ?? "*"}${expected.message ? ` containing "${expected.message}"` : ""} — not found`,
             );
           }
         }
@@ -152,10 +174,13 @@ export async function create(
       const passedLines = matched.map((m) => `  ${green("✓")} ${dim(m)}\n`).join("");
       if (failures.length > 0) {
         const failedLines = failures.map((f) => `  ${red("✗")} ${f}\n`).join("");
-        const actualLines = errors.length > 0
-          ? `  ${dim("actual errors:")}\n` +
-            errors.map((d) => `    ${dim(`[${d.code}] ${d.message}`)}\n`).join("")
-          : `  ${dim("no errors produced")}\n`;
+        const actualLines =
+          errors.length > 0 || warnings.length > 0
+            ? `  ${dim("actual diagnostics:")}\n` +
+              [...errors, ...warnings]
+                .map((d) => `    ${dim(`[${d.code}] ${d.message}`)}\n`)
+                .join("")
+            : `  ${dim("no diagnostics produced")}\n`;
         ctx.stderr.write(
           bold(red(`Assert.Manifest.${name}: assertion failed`)) + "\n" +
             passedLines + failedLines + actualLines,

package/src/module-context.ts

@@ -50,14 +50,13 @@ export async function create(
   return {
     run: async () => {
       const { bold, red, green, yellow, dim } = createColors(ctx.stderr);
-      const declaringModule = manifest.metadata.module ?? "default";
       const resourcesToCheck = manifest.resources ?? {};
       const failures: string[] = [];
       const passed: string[] = [];
       const { resources } = ctx.moduleContext;
       for (const [alias, expected] of Object.entries(resourcesToCheck)) {
-        if (!(ctx as any).resolveModuleAlias(declaringModule, alias)) {
-          failures.push(`Import alias '${alias}' not found in module '${declaringModule}'`);
+        if (!ctx.moduleContext.hasImport(alias)) {
+          failures.push(`Import alias '${alias}' not found in declaring module`);
           continue;
         }