@intentius/chant-lexicon-aws 0.0.5 → 0.0.8

package/src/serializer.ts

@@ -3,7 +3,6 @@ import { isPropertyDeclarable } from "@intentius/chant/declarable";
 import type { Serializer, SerializerResult } from "@intentius/chant/serializer";
 import type { LexiconOutput } from "@intentius/chant/lexicon-output";
 import { walkValue, type SerializerVisitor } from "@intentius/chant/serializer-walker";
-import { toPascalCase } from "@intentius/chant/codegen/case";
 import { isChildProject, type ChildProjectInstance } from "@intentius/chant/child-project";
 import { isStackOutput, type StackOutput } from "@intentius/chant/stack-output";
 
@@ -68,7 +67,7 @@ interface CFOutput {
  */
 function cfnVisitor(entityNames: Map<Declarable, string>): SerializerVisitor {
   return {
-    attrRef: (name, attr) => ({ "Fn::GetAttr": [name, attr] }),
+    attrRef: (name, attr) => ({ "Fn::GetAtt": [name, attr] }),
     resourceRef: (name) => ({ Ref: name }),
     propertyDeclarable: (entity, walk) => {
       if (!("props" in entity) || typeof entity.props !== "object" || entity.props === null) {
@@ -78,26 +77,19 @@ function cfnVisitor(entityNames: Map<Declarable, string>): SerializerVisitor {
       const cfProps: Record<string, unknown> = {};
       for (const [key, value] of Object.entries(props)) {
         if (value !== undefined) {
-          const cfKey = toPascalCase(key);
-          cfProps[cfKey] = walk(value);
+          cfProps[key] = walk(value);
         }
       }
       return Object.keys(cfProps).length > 0 ? cfProps : undefined;
     },
-    transformKey: toPascalCase,
   };
 }
 
 /**
  * Convert a value to CF-compatible JSON using the generic walker.
  */
-function toCFValue(value: unknown, entityNames: Map<Declarable, string>, convertKeys = false): unknown {
-  const visitor = cfnVisitor(entityNames);
-  if (!convertKeys) {
-    // When not converting keys, use a visitor without transformKey
-    return walkValue(value, entityNames, { ...visitor, transformKey: undefined });
-  }
-  return walkValue(value, entityNames, visitor);
+function toCFValue(value: unknown, entityNames: Map<Declarable, string>): unknown {
+  return walkValue(value, entityNames, cfnVisitor(entityNames));
 }
 
 /**
@@ -116,8 +108,7 @@ function toProperties(
 
   for (const [key, value] of Object.entries(props)) {
     if (value !== undefined) {
-      const cfKey = toPascalCase(key);
-      cfProps[cfKey] = toCFValue(value, entityNames, true);
+      cfProps[key] = toCFValue(value, entityNames);
     }
   }
 
@@ -231,7 +222,7 @@ function serializeToTemplate(
       const logicalName = ref.getLogicalName();
       if (logicalName) {
         const output: CFOutput = {
-          Value: { "Fn::GetAttr": [logicalName, ref.attribute] },
+          Value: { "Fn::GetAtt": [logicalName, ref.attribute] },
         };
         if (stackOutput.description) {
           output.Description = stackOutput.description;
@@ -247,7 +238,7 @@ function serializeToTemplate(
     for (const output of outputs) {
       template.Outputs[output.outputName] = {
         Value: {
-          "Fn::GetAttr": [output.sourceEntity, output.sourceAttribute],
+          "Fn::GetAtt": [output.sourceEntity, output.sourceAttribute],
         },
       };
     }

package/src/spec/parse.ts

@@ -36,7 +36,7 @@ export interface ParsedAttribute {
 
 export interface ParsedPropertyType {
   name: string;
-  cfnType: string;
+  specType: string;
   properties: ParsedProperty[];
 }
 
@@ -130,7 +130,7 @@ export function parseCFNSchema(data: string | Buffer): SchemaParseResult {
         }
         propertyTypes.push({
           name: `${shortName}_${defName}`,
-          cfnType: defName,
+          specType: defName,
           properties: defProps,
         });
       }

package/dist/skills/aws-cloudformation.md

@@ -1,41 +0,0 @@
----
-name: aws-cloudformation
-description: AWS CloudFormation best practices and common patterns
----
-
-# AWS CloudFormation with Chant
-
-## Common Resource Types
-
-- `AWS::S3::Bucket` — Object storage
-- `AWS::Lambda::Function` — Serverless compute
-- `AWS::DynamoDB::Table` — NoSQL database
-- `AWS::IAM::Role` — Identity and access management
-- `AWS::SNS::Topic` — Pub/sub messaging
-- `AWS::SQS::Queue` — Message queue
-- `AWS::EC2::SecurityGroup` — Network firewall rules
-
-## Intrinsic Functions
-
-- `Sub` — String interpolation with `${}` syntax
-- `Ref` — Reference a resource or parameter
-- `GetAtt` — Get a resource attribute (e.g. ARN)
-- `If` — Conditional value based on a condition
-- `Join` — Join strings with a delimiter
-- `Select` — Pick an item from a list by index
-
-## Pseudo Parameters
-
-- `AWS::StackName` — Name of the current stack
-- `AWS::Region` — Current deployment region
-- `AWS::AccountId` — Current AWS account ID
-- `AWS::Partition` — Partition (aws, aws-cn, aws-us-gov)
-
-## Best Practices
-
-1. **Always enable encryption** — Use `BucketEncryption` for S3, `SSESpecification` for DynamoDB
-2. **Block public access** — Set `PublicAccessBlockConfiguration` on all S3 buckets
-3. **Use least-privilege IAM** — Avoid `*` in IAM policy actions and resources
-4. **Enable versioning** — Turn on `VersioningConfiguration` for data buckets
-5. **Use Sub for dynamic names** — `Sub\`\${AWS::StackName}-suffix\`` for unique naming
-6. **Share config via barrel files** — Put common settings in `_.ts` and import as `* as _`

package/src/codegen/rollback.test.ts

@@ -1,80 +0,0 @@
-import { describe, test, expect } from "bun:test";
-import { mkdirSync, writeFileSync, rmSync, readFileSync } from "fs";
-import { join } from "path";
-import { tmpdir } from "os";
-import { snapshotArtifacts, saveSnapshot, restoreSnapshot, listSnapshots } from "./rollback";
-
-function makeTempDir(): string {
-  const dir = join(tmpdir(), `chant-rollback-test-${Date.now()}-${Math.random().toString(36).slice(2)}`);
-  mkdirSync(dir, { recursive: true });
-  return dir;
-}
-
-describe("rollback", () => {
-  test("snapshot captures generated files", () => {
-    const dir = makeTempDir();
-    const genDir = join(dir, "generated");
-    mkdirSync(genDir, { recursive: true });
-
-    writeFileSync(join(genDir, "lexicon-aws.json"), '{"Bucket":{"kind":"resource"}}');
-    writeFileSync(join(genDir, "index.d.ts"), "declare class Bucket {}");
-    writeFileSync(join(genDir, "index.ts"), "export const Bucket = {};");
-
-    const snapshot = snapshotArtifacts(genDir);
-    expect(snapshot.files["lexicon-aws.json"]).toBeDefined();
-    expect(snapshot.files["index.d.ts"]).toBeDefined();
-    expect(snapshot.files["index.ts"]).toBeDefined();
-    expect(snapshot.hashes["lexicon-aws.json"]).toBeDefined();
-    expect(snapshot.resourceCount).toBe(1);
-
-    rmSync(dir, { recursive: true, force: true });
-  });
-
-  test("save and list snapshots", () => {
-    const dir = makeTempDir();
-    const snapshotsDir = join(dir, ".snapshots");
-
-    const snapshot = {
-      timestamp: "2025-01-01T00:00:00.000Z",
-      files: { "test.json": "{}" },
-      hashes: { "test.json": "abc123" },
-      resourceCount: 0,
-    };
-
-    saveSnapshot(snapshot, snapshotsDir);
-    const list = listSnapshots(snapshotsDir);
-    expect(list.length).toBe(1);
-    expect(list[0].timestamp).toBe("2025-01-01T00:00:00.000Z");
-
-    rmSync(dir, { recursive: true, force: true });
-  });
-
-  test("restore snapshot overwrites generated files", () => {
-    const dir = makeTempDir();
-    const genDir = join(dir, "generated");
-    const snapshotsDir = join(dir, ".snapshots");
-    mkdirSync(genDir, { recursive: true });
-
-    // Write original files
-    writeFileSync(join(genDir, "lexicon-aws.json"), '{"original":true}');
-
-    // Snapshot them
-    const snapshot = snapshotArtifacts(genDir);
-    const snapshotPath = saveSnapshot(snapshot, snapshotsDir);
-
-    // Overwrite with new content
-    writeFileSync(join(genDir, "lexicon-aws.json"), '{"modified":true}');
-    expect(readFileSync(join(genDir, "lexicon-aws.json"), "utf-8")).toBe('{"modified":true}');
-
-    // Restore
-    restoreSnapshot(snapshotPath, genDir);
-    expect(readFileSync(join(genDir, "lexicon-aws.json"), "utf-8")).toBe('{"original":true}');
-
-    rmSync(dir, { recursive: true, force: true });
-  });
-
-  test("listSnapshots returns empty for nonexistent dir", () => {
-    const list = listSnapshots("/nonexistent/path");
-    expect(list).toHaveLength(0);
-  });
-});

package/src/codegen/rollback.ts

@@ -1,20 +0,0 @@
-/**
- * Re-export from core with AWS-specific artifact names.
- */
-import {
-  snapshotArtifacts as _snapshotArtifacts,
-  saveSnapshot,
-  restoreSnapshot,
-  listSnapshots,
-} from "@intentius/chant/codegen/rollback";
-export type { ArtifactSnapshot, SnapshotInfo } from "@intentius/chant/codegen/rollback";
-export { saveSnapshot, restoreSnapshot, listSnapshots };
-
-const AWS_ARTIFACT_NAMES = ["lexicon-aws.json", "index.d.ts", "index.ts"];
-
-/**
- * Snapshot AWS lexicon artifacts.
- */
-export function snapshotArtifacts(generatedDir: string) {
-  return _snapshotArtifacts(generatedDir, AWS_ARTIFACT_NAMES);
-}