@intentius/chant-lexicon-docker 0.32.0 → 0.33.1

package/dist/integrity.json

@@ -1,7 +1,7 @@
 {
   "algorithm": "sha256",
   "artifacts": {
-    "manifest.json": "308e3bacb2e82cad075a01f9ba73af540569ab9a1f1b722710319c81823e086b",
+    "manifest.json": "fcfdf631b9f065cc4d7f5e20090827ec78f3162d40539e6d407f9ec987074cbf",
     "meta.json": "ae27dc809e705d37a645ee83b9789c50051282f7891d21d6b9e6ce8d111624ff",
     "types/index.d.ts": "758f989ff86af54284d7f841a2db2259038d51ce1f3ed839a5d8e83fa20d95b8",
     "rules/no-latest-tag.ts": "efd060453d8ca3d5b85c3972906c6650b6f63eb5f67cb6cdb0f16a41cad91228",
@@ -15,5 +15,5 @@
     "skills/chant-docker.md": "3ff0708e3c76e245f202948949c768464563af2b60cc9aa2ca52f494ef8357f1",
     "skills/chant-docker-patterns.md": "ad1b0196d8150b2df579a699ff107f604340c799c04f3460ebf65003e287b12a"
   },
-  "composite": "118299174bb77206216858827cba459dbe5b53f055ab2d30231fb5fa1d9bd57a"
+  "composite": "a7bee24e3748edc559d615a1fad753c1bf5f80de3b164719d2e2c26a224bd7f3"
 }

package/dist/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "docker",
-  "version": "0.32.0",
+  "version": "0.33.1",
   "chantVersion": ">=0.1.0",
   "namespace": "Docker",
   "intrinsics": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@intentius/chant-lexicon-docker",
-  "version": "0.32.0",
+  "version": "0.33.1",
   "description": "Docker lexicon for chant — declarative IaC in TypeScript",
   "license": "Apache-2.0",
   "homepage": "https://intentius.io/chant",
@@ -67,7 +67,7 @@
     "typescript": "^5.9.3"
   },
   "peerDependencies": {
-    "@intentius/chant": "^0.32.0",
+    "@intentius/chant": "^0.33.1",
     "typescript": "^5.9.3"
   }
 }

package/src/lsp/completions.test.ts

@@ -2,33 +2,39 @@ import { describe, test, expect } from "vitest";
 import { dockerCompletions } from "./completions";
 import type { CompletionContext } from "@intentius/chant/lsp/types";
 
+// The context these tests build must match the real CompletionContext.
+// They previously used an older shape (word / isConstructorContext / prefix /
+// fileContent) the interface no longer has — vitest strips types without
+// checking them, so every assertion passed while exercising no completion
+// path at all.
+function ctx(partial: Partial<CompletionContext>): CompletionContext {
+  return {
+    uri: "file:///infra.ts",
+    content: "",
+    position: { line: 0, character: 0 },
+    wordAtCursor: "",
+    linePrefix: "",
+    ...partial,
+  };
+}
+
 describe("dockerCompletions", () => {
-  test("is a function", () => {
-    expect(typeof dockerCompletions).toBe("function");
+  test("suggests resource class names after `new `", () => {
+    const items = dockerCompletions(ctx({ linePrefix: "const s = new " }));
+
+    expect(items.length).toBeGreaterThan(0);
+    expect(items.map((i) => i.label)).toContain("Service");
+    expect(items.every((i) => i.kind === "resource")).toBe(true);
   });
 
-  test("returns array for non-constructor context", () => {
-    const ctx: CompletionContext = {
-      word: "Ser",
-      isConstructorContext: false,
-      prefix: "",
-      fileContent: "",
-      position: { line: 0, character: 3 },
-    };
-    // May return [] if generated index not present, but should not throw
-    const result = dockerCompletions(ctx);
-    expect(Array.isArray(result)).toBe(true);
+  test("narrows on a typed prefix", () => {
+    const items = dockerCompletions(ctx({ linePrefix: "new Serv", wordAtCursor: "Serv" }));
+
+    expect(items.map((i) => i.label)).toContain("Service");
+    expect(items.every((i) => i.label.toLowerCase().startsWith("serv"))).toBe(true);
   });
 
-  test("returns array for constructor context", () => {
-    const ctx: CompletionContext = {
-      word: "Service",
-      isConstructorContext: true,
-      prefix: "new ",
-      fileContent: "import { Service } from '@intentius/chant-lexicon-docker';",
-      position: { line: 0, character: 10 },
-    };
-    const result = dockerCompletions(ctx);
-    expect(Array.isArray(result)).toBe(true);
+  test("returns nothing in an unrelated position", () => {
+    expect(dockerCompletions(ctx({ linePrefix: "const x = 1" }))).toEqual([]);
   });
 });

package/src/lsp/hover.test.ts

@@ -2,33 +2,34 @@ import { describe, test, expect } from "vitest";
 import { dockerHover } from "./hover";
 import type { HoverContext } from "@intentius/chant/lsp/types";
 
+// Same correction as completions.test.ts: the old shape (word / fileContent /
+// position) no longer matches HoverContext, and the assertions were written to
+// pass either way ("undefined or an object", "if result !== undefined"), so a
+// hover that returned nothing at all still looked healthy.
+function ctx(word: string): HoverContext {
+  return {
+    uri: "file:///infra.ts",
+    content: "",
+    position: { line: 0, character: 0 },
+    word,
+    lineText: "",
+  };
+}
+
 describe("dockerHover", () => {
-  test("is a function", () => {
-    expect(typeof dockerHover).toBe("function");
+  test("returns undefined for an unknown word", () => {
+    expect(dockerHover(ctx("NotADockerThing"))).toBeUndefined();
+    expect(dockerHover(ctx(""))).toBeUndefined();
   });
 
-  test("returns undefined for unknown word", () => {
-    const ctx: HoverContext = {
-      word: "NotADockerThing",
-      fileContent: "",
-      position: { line: 0, character: 0 },
-    };
-    // Should not throw; returns undefined if not found
-    const result = dockerHover(ctx);
-    expect(result === undefined || typeof result === "object").toBe(true);
+  test("describes a known resource with its docker type", () => {
+    const info = dockerHover(ctx("Service"));
+
+    expect(info?.contents).toContain("**Service**");
+    expect(info?.contents).toContain("Docker::Compose::Service");
   });
 
-  test("returns HoverInfo with content for known resource", () => {
-    const ctx: HoverContext = {
-      word: "Service",
-      fileContent: "import { Service } from '@intentius/chant-lexicon-docker';",
-      position: { line: 0, character: 10 },
-    };
-    const result = dockerHover(ctx);
-    // If generated index exists, will return content; otherwise undefined
-    if (result !== undefined) {
-      expect(result.contents).toBeTruthy();
-      expect(typeof result.contents).toBe("string");
-    }
+  test("notes which artifact a resource serializes into", () => {
+    expect(dockerHover(ctx("Service"))?.contents).toContain("docker-compose.yml");
   });
 });