@intentius/chant 0.0.18 → 0.0.24

package/src/utils.test.ts

@@ -14,15 +14,17 @@ describe("LOGICAL_NAME_SYMBOL", () => {
   });
 
   test("uses Symbol.for for global registry", () => {
-    expect(LOGICAL_NAME_SYMBOL).toBe(Symbol.for("chant.logicalName"));
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    expect(LOGICAL_NAME_SYMBOL).toBe(Symbol.for("chant.logicalName") as any);
   });
 });
 
 describe("getAttributes", () => {
   test("returns empty array for entity with no AttrRef properties", () => {
-    const entity: Declarable = {
+    const entity = {
+      lexicon: "test",
       entityType: "Test",
-      [DECLARABLE_MARKER]: true,
+      [DECLARABLE_MARKER]: true as const,
       prop1: "value",
       prop2: 123,
     };
@@ -34,6 +36,7 @@ describe("getAttributes", () => {
   test("returns property names with AttrRef values", () => {
     const parent = {};
     const entity: Declarable & { arn: AttrRef; name: AttrRef } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       arn: new AttrRef(parent, "Arn"),
@@ -53,6 +56,7 @@ describe("getAttributes", () => {
       regularProp: string;
       numberProp: number;
     } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       arn: new AttrRef(parent, "Arn"),
@@ -71,6 +75,7 @@ describe("getAttributes", () => {
       second: AttrRef;
       third: AttrRef;
     } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       first: new AttrRef(parent, "First"),
@@ -92,6 +97,7 @@ describe("getAttributes", () => {
       object: object;
       anotherAttrRef: AttrRef;
     } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       attrRef: new AttrRef(parent, "Attr1"),
@@ -112,6 +118,7 @@ describe("getAttributes", () => {
 describe("getLogicalName", () => {
   test("returns logical name when set", () => {
     const entity: Declarable & Record<symbol, unknown> = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       [LOGICAL_NAME_SYMBOL]: "MyResource",
@@ -123,6 +130,7 @@ describe("getLogicalName", () => {
 
   test("throws when logical name is not set", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "TestEntity",
       [DECLARABLE_MARKER]: true,
     };
@@ -134,6 +142,7 @@ describe("getLogicalName", () => {
 
   test("throws when logical name is not a string", () => {
     const entity: Declarable & Record<symbol, unknown> = {
+      lexicon: "test",
       entityType: "TestEntity",
       [DECLARABLE_MARKER]: true,
       [LOGICAL_NAME_SYMBOL]: 123,
@@ -149,6 +158,7 @@ describe("getLogicalName", () => {
 
     for (const name of names) {
       const entity: Declarable & Record<symbol, unknown> = {
+        lexicon: "test",
         entityType: "Test",
         [DECLARABLE_MARKER]: true,
         [LOGICAL_NAME_SYMBOL]: name,
@@ -160,6 +170,7 @@ describe("getLogicalName", () => {
 
   test("includes entity type in error message", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "MyCustomType",
       [DECLARABLE_MARKER]: true,
     };
@@ -171,6 +182,7 @@ describe("getLogicalName", () => {
 
   test("throws when logical name is undefined", () => {
     const entity: Declarable & Record<symbol, unknown> = {
+      lexicon: "test",
       entityType: "TestEntity",
       [DECLARABLE_MARKER]: true,
       [LOGICAL_NAME_SYMBOL]: undefined,
@@ -183,6 +195,7 @@ describe("getLogicalName", () => {
 
   test("throws when logical name is null", () => {
     const entity: Declarable & Record<symbol, unknown> = {
+      lexicon: "test",
       entityType: "TestEntity",
       [DECLARABLE_MARKER]: true,
       [LOGICAL_NAME_SYMBOL]: null,

package/src/utils.ts

@@ -6,6 +6,36 @@ import { AttrRef } from "./attrref";
  */
 export const LOGICAL_NAME_SYMBOL = Symbol.for("chant.logicalName");
 
+/**
+ * Duck-type check for AttrRef that survives the dual-package hazard.
+ * When npm installs separate copies of @intentius/chant (one for the CLI,
+ * one for the lexicon), `instanceof AttrRef` fails across package boundaries.
+ */
+export function isAttrRefLike(value: unknown): value is AttrRef {
+  if (value instanceof AttrRef) return true;
+  return (
+    typeof value === "object" &&
+    value !== null &&
+    "parent" in value &&
+    "attribute" in value &&
+    "_setLogicalName" in value &&
+    typeof (value as any).parent === "object" &&
+    typeof (value as any).attribute === "string"
+  );
+}
+
+/**
+ * JSON.stringify replacer that sorts object keys for deterministic output.
+ */
+export function sortedJsonReplacer(_key: string, value: unknown): unknown {
+  if (value && typeof value === "object" && !Array.isArray(value)) {
+    return Object.fromEntries(
+      Object.entries(value as Record<string, unknown>).sort(([a], [b]) => a.localeCompare(b))
+    );
+  }
+  return value;
+}
+
 /**
  * Get all property names that have AttrRef values
  * @param entity - The declarable entity to inspect
@@ -18,7 +48,7 @@ export function getAttributes(entity: Declarable): string[] {
   for (const key in obj) {
     if (Object.prototype.hasOwnProperty.call(obj, key)) {
       const value = obj[key];
-      if (value instanceof AttrRef) {
+      if (isAttrRefLike(value)) {
         attributes.push(key);
       }
     }

package/src/validation.test.ts

@@ -50,6 +50,7 @@ describe("ValidationRule", () => {
 
   test("validate function receives entity and returns result", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -71,6 +72,7 @@ describe("ValidationRule", () => {
 describe("validate", () => {
   test("returns empty array when no rules provided", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -81,6 +83,7 @@ describe("validate", () => {
 
   test("applies single rule and returns result", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -98,6 +101,7 @@ describe("validate", () => {
 
   test("applies multiple rules and returns all results", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -130,6 +134,7 @@ describe("validate", () => {
 
   test("is non-blocking - does not throw on validation failure", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -149,6 +154,7 @@ describe("validate", () => {
 
   test("passes entity to each rule", () => {
     const entity: Declarable & { name: string } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       name: "MyEntity",
@@ -172,6 +178,7 @@ describe("validate", () => {
 
   test("preserves rule execution order", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -211,6 +218,7 @@ describe("validate", () => {
 
   test("handles rules with context in results", () => {
     const entity: Declarable & { count: number } = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
       count: 5,
@@ -240,6 +248,7 @@ describe("validate", () => {
 
   test("handles entity type validation", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Bucket",
       [DECLARABLE_MARKER]: true,
     };
@@ -263,6 +272,7 @@ describe("validate", () => {
 
   test("validation is opt-in - only runs rules provided", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };
@@ -274,6 +284,7 @@ describe("validate", () => {
 
   test("returns results in same order as rules", () => {
     const entity: Declarable = {
+      lexicon: "test",
       entityType: "Test",
       [DECLARABLE_MARKER]: true,
     };

package/src/cli/commands/__fixtures__/init-lexicon-output/docs/src/content/docs/getting-started.mdx

@@ -1,6 +0,0 @@
----
-title: Getting Started
-description: Get started with the Fixture lexicon
----
-
-TODO: Document how to set up a project using the Fixture lexicon.

package/src/cli/commands/__fixtures__/init-lexicon-output/docs/src/content/docs/lint-rules.mdx

@@ -1,6 +0,0 @@
----
-title: Lint Rules
-description: Fixture lint rules reference
----
-
-TODO: Document the lint rules provided by the Fixture lexicon.

package/src/cli/commands/__fixtures__/init-lexicon-output/docs/src/content/docs/serialization.mdx

@@ -1,6 +0,0 @@
----
-title: Serialization
-description: Fixture output format
----
-
-TODO: Document the Fixture serialization format and output structure.

package/src/cli/commands/__fixtures__/init-lexicon-output/src/coverage.ts

@@ -1,11 +0,0 @@
-/**
- * Coverage analysis for the fixture lexicon.
- *
- * TODO: Implement coverage analysis that checks how much of the
- * upstream spec is covered by the generated types.
- */
-export async function analyzeCoverage(options?: { verbose?: boolean }): Promise<void> {
-  console.error("Coverage analysis not yet implemented");
-  // TODO: Read generated lexicon JSON, compare against upstream spec,
-  // and report coverage metrics.
-}

package/src/cli/commands/__fixtures__/init-lexicon-output/src/import/generator.ts

@@ -1,10 +0,0 @@
-import type { TypeScriptGenerator } from "@intentius/chant/import/generator";
-
-/**
- * TypeScript generator for converting imported fixture templates.
- *
- * TODO: Implement the TypeScriptGenerator interface for your format.
- */
-// export class FixtureGenerator implements TypeScriptGenerator {
-//   generate(ir: IR): string { ... }
-// }

package/src/cli/commands/__fixtures__/init-lexicon-output/src/import/parser.ts

@@ -1,10 +0,0 @@
-import type { TemplateParser } from "@intentius/chant/import/parser";
-
-/**
- * Template parser for importing external fixture templates.
- *
- * TODO: Implement the TemplateParser interface for your format.
- */
-// export class FixtureParser implements TemplateParser {
-//   parse(data: unknown): IR { ... }
-// }