@typespec/compiler 0.52.0-dev.11 → 0.52.0-dev.13

package/templates/__snapshots__/library-ts/src/testing/index.ts

@@ -0,0 +1,8 @@
+import { resolvePath } from "@typespec/compiler";
+import { createTestLibrary, TypeSpecTestLibrary } from "@typespec/compiler/testing";
+import { fileURLToPath } from "url";
+
+export const LibraryTsTestLibrary: TypeSpecTestLibrary = createTestLibrary({
+  name: "library-ts",
+  packageRoot: resolvePath(fileURLToPath(import.meta.url), "../../../../"),
+});

package/templates/__snapshots__/library-ts/test/decorators.test.ts

@@ -0,0 +1,48 @@
+import { strictEqual } from "node:assert";
+import { describe, it, beforeEach } from "node:test";
+import { Operation } from "@typespec/compiler";
+import { BasicTestRunner, expectDiagnostics, extractCursor } from "@typespec/compiler/testing";
+import { getAlternateName } from "../src/decorators.js";
+import { createLibraryTsTestRunner } from "./test-host.js";
+
+describe("decorators", () => {
+  let runner: BasicTestRunner;
+
+  beforeEach(async () => {
+    runner = await createLibraryTsTestRunner();
+  })
+
+  describe("@alternateName", () => {
+    it("set alternate name on operation", async () => {
+      const { test } = (await runner.compile(
+        `@alternateName("bar") @test op test(): void;`
+      )) as { test: Operation };
+      strictEqual(getAlternateName(runner.program, test), "bar");
+    });
+
+    it("emit diagnostic if not used on an operation", async () => {
+      const diagnostics = await runner.diagnose(
+        `@alternateName("bar") model Test {}`
+      );
+      expectDiagnostics(diagnostics, {
+        severity: "error",
+        code: "decorator-wrong-target",
+        message: "Cannot apply @alternateName decorator to Test since it is not assignable to Operation"
+      })
+    });
+
+
+    it("emit diagnostic if using banned name", async () => {
+      const {pos, source} = extractCursor(`@alternateName(┆"banned") op test(): void;`)
+      const diagnostics = await runner.diagnose(
+        source
+      );
+      expectDiagnostics(diagnostics, {
+        severity: "error",
+        code: "library-ts/banned-alternate-name",
+        message: `Banned alternate name "banned".`,
+        pos: pos + runner.autoCodeOffset
+      })
+    });
+  });
+});

package/templates/__snapshots__/library-ts/test/rules/no-interfaces.rule.test.ts

@@ -0,0 +1,29 @@
+import {
+  LinterRuleTester,
+  createLinterRuleTester,
+  createTestRunner,
+} from "@typespec/compiler/testing";
+import { beforeEach, describe, it } from "node:test";
+import { noInterfaceRule } from "../../src/rules/no-interfaces.rule.js";
+
+describe("noInterfaceRule", () => {
+  let ruleTester: LinterRuleTester;
+
+  beforeEach(async () => {
+    const runner = await createTestRunner();
+    ruleTester = createLinterRuleTester(runner, noInterfaceRule, "library-ts");
+  });
+
+  describe("models", () => {
+    it("emit diagnostics if using interfaces", async () => {
+      await ruleTester.expect(`interface Test {}`).toEmitDiagnostics({
+        code: "library-ts/no-interface",
+        message: "Interface shouldn't be used with this library. Keep operations at the root.",
+      });
+    });
+
+    it("should be valid if operation is at the root", async () => {
+      await ruleTester.expect(`op test(): void;`).toBeValid();
+    });
+  });
+});

package/templates/__snapshots__/library-ts/test/test-host.ts

@@ -0,0 +1,17 @@
+import { createTestHost, createTestWrapper } from "@typespec/compiler/testing";
+import { LibraryTsTestLibrary } from "../src/testing/index.js";
+
+export async function createLibraryTsTestHost() {
+  return createTestHost({
+    libraries: [LibraryTsTestLibrary],
+  });
+}
+
+export async function createLibraryTsTestRunner() {
+  const host = await createLibraryTsTestHost();
+
+  return createTestWrapper(host, {
+    autoUsings: ["LibraryTs"]
+  });
+}
+

package/templates/__snapshots__/library-ts/tsconfig.json

@@ -0,0 +1,17 @@
+{
+  "compilerOptions": {
+    "target": "ES2022",
+    "useDefineForClassFields": true,
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "lib": ["ES2022"],
+    "rootDir": ".",
+    "outDir": "dist",
+    "sourceMap": true,
+    "declaration": true,
+
+    /* Linting */
+    "strict": true
+  },
+  "include": ["src", "test"]
+}

package/templates/emitter-ts/src/testing/index.ts.mu

@@ -0,0 +1,8 @@
+import { resolvePath } from "@typespec/compiler";
+import { createTestLibrary, TypeSpecTestLibrary } from "@typespec/compiler/testing";
+import { fileURLToPath } from "url";
+
+export const {{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary: TypeSpecTestLibrary = createTestLibrary({
+  name: "{{name}}",
+  packageRoot: resolvePath(fileURLToPath(import.meta.url), "../../../../"),
+});

package/templates/emitter-ts/test/test-host.ts.mu

@@ -4,11 +4,11 @@ import {
   createTestWrapper,
   expectDiagnosticEmpty,
 } from "@typespec/compiler/testing";
-import { TestLibrary } from "../src/testing/index.js";
+import { {{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary } from "../src/testing/index.js";
 
 export async function create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestHost() {
   return createTestHost({
-    libraries: [TestLibrary],
+    libraries: [{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary],
   });
 }
 

package/templates/library-ts/.eslintrc.yml

@@ -0,0 +1,14 @@
+root: true
+env:
+  es2021: true
+  node: true
+extends:
+  - eslint:recommended
+  - plugin:@typescript-eslint/recommended
+parser: "@typescript-eslint/parser"
+parserOptions:
+  ecmaVersion: latest
+  sourceType: module
+plugins:
+  - "@typescript-eslint"
+rules: {}

package/templates/library-ts/lib/decorators.tsp.mu

@@ -0,0 +1,12 @@
+import "../dist/src/decorators.js";
+
+using TypeSpec.Reflection;
+
+namespace {{#casing.pascalCase}}{{name}}{{/casing.pascalCase}};
+
+/**
+ * __Example Decorator__
+ * Provide an alternate name for an operation.
+ * @param name The alternate name.
+ */
+extern dec alternateName(target: Operation, name: valueof string);

package/templates/library-ts/lib/main.tsp

@@ -0,0 +1 @@
+import "./decorators.tsp";

package/templates/library-ts/package.json

@@ -0,0 +1,40 @@
+{
+  "name": "{{name}}",
+  "version": "0.1.0",
+  "type": "module",
+  "main": "dist/src/index.js",
+  "tspMain": "lib/main.tsp",
+  "exports": {
+    ".": {
+      "types": "./dist/src/index.d.ts",
+      "default": "./dist/src/index.js"
+    },
+    "./testing": {
+      "types": "./dist/src/testing/index.d.ts",
+      "default": "./dist/src/testing/index.js"
+    }
+  },
+  "dependencies": {
+    "@typespec/compiler": "latest"
+  },
+  "devDependencies": {
+    "@types/node": "latest",
+    "@typescript-eslint/eslint-plugin": "^6.0.0",
+    "@typescript-eslint/parser": "^6.0.0",
+    "@typespec/library-linter": "latest",
+    "eslint": "^8.45.0",
+    "prettier": "^3.0.3",
+    "typescript": "^5.3.3"
+  },
+  "scripts": {
+    "build": "tsc && npm run build:tsp",
+    "watch": "tsc --watch",
+    "build:tsp": "tsp compile . --warn-as-error --import @typespec/library-linter --no-emit",
+    "test": "node --test ./dist/test/",
+    "lint": "eslint src/ test/ --report-unused-disable-directives --max-warnings=0",
+    "lint:fix": "eslint . --report-unused-disable-directives --fix",
+    "format": "prettier .  --write",
+    "format:check": "prettier --check ."
+  },
+  "private": true
+}

package/templates/library-ts/prettierrc.yaml

@@ -0,0 +1,8 @@
+trailingComma: "all"
+printWidth: 120
+quoteProps: "consistent"
+endOfLine: lf
+arrowParens: always
+plugins:
+  - "./node_modules/@typespec/prettier-plugin-typespec/dist/index.js"
+overrides: [{ "files": "*.tsp", "options": { "parser": "typespec" } }]

package/templates/library-ts/src/decorators.ts

@@ -0,0 +1,33 @@
+import { DecoratorContext, Operation, Program } from "@typespec/compiler";
+import { StateKeys, reportDiagnostic } from "./lib.js";
+
+export const namespace = "{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}";
+
+/**
+ * __Example implementation of the `@alternateName` decorator.__
+ *
+ * @param context Decorator context.
+ * @param target Decorator target. Must be an operation.
+ * @param name Alternate name.
+ */
+export function $alternateName(context: DecoratorContext, target: Operation, name: string) {
+  if (name === "banned") {
+    reportDiagnostic(context.program, {
+      code: "banned-alternate-name",
+      target: context.getArgumentTarget(0)!,
+      format: { name },
+    });
+  }
+  context.program.stateMap(StateKeys.alternateName).set(target, name);
+}
+
+/**
+ * __Example accessor for  the `@alternateName` decorator.__
+ *
+ * @param program TypeSpec program.
+ * @param target Decorator target. Must be an operation.
+ * @returns Altenate name if provided on the given operation or undefined
+ */
+export function getAlternateName(program: Program, target: Operation): string | undefined {
+  return program.stateMap(StateKeys.alternateName).get(target);
+}

package/templates/library-ts/src/index.ts

@@ -0,0 +1,2 @@
+export { getAlternateName } from "./decorators.js";
+export { $lib } from "./lib.js";

package/templates/library-ts/src/lib.ts

@@ -0,0 +1,20 @@
+import { createTypeSpecLibrary, paramMessage } from "@typespec/compiler";
+
+export const $lib = createTypeSpecLibrary({
+  name: "{{name}}",
+  // Define diagnostics for the library. This will provide a typed API to report diagnostic as well as a auto doc generation.
+  diagnostics: {
+    "banned-alternate-name": {
+      severity: "error",
+      messages: {
+        default: paramMessage`Banned alternate name "${"name"}".`,
+      },
+    },
+  },
+  // Defined state keys for storing metadata in decorator.
+  state: {
+    alternateName: { description: "alternateName" },
+  },
+});
+
+export const { reportDiagnostic, createDiagnostic, stateKeys: StateKeys } = $lib;

package/templates/library-ts/src/linter.ts

@@ -0,0 +1,14 @@
+import { defineLinter } from "@typespec/compiler";
+import { noInterfaceRule } from "./rules/no-interfaces.rule.js";
+
+export const $linter = defineLinter({
+  rules: [noInterfaceRule],
+  ruleSets: {
+    recommended: {
+      enable: { [`{{name}}/${noInterfaceRule.name}`]: true },
+    },
+    all: {
+      enable: { [`{{name}}/${noInterfaceRule.name}`]: true },
+    },
+  },
+});

package/templates/library-ts/src/rules/no-interfaces.rule.ts

@@ -0,0 +1,19 @@
+import { createRule } from "@typespec/compiler";
+
+export const noInterfaceRule = createRule({
+  name: "no-interface",
+  severity: "warning",
+  description: "Make sure interface are not used.",
+  messages: {
+    default: "Interface shouldn't be used with this library. Keep operations at the root.",
+  },
+  create: (context) => {
+    return {
+      interface: (iface) => {
+        context.reportDiagnostic({
+          target: iface,
+        });
+      },
+    };
+  },
+});

package/templates/library-ts/src/testing/index.ts.mu

@@ -0,0 +1,8 @@
+import { resolvePath } from "@typespec/compiler";
+import { createTestLibrary, TypeSpecTestLibrary } from "@typespec/compiler/testing";
+import { fileURLToPath } from "url";
+
+export const {{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary: TypeSpecTestLibrary = createTestLibrary({
+  name: "{{name}}",
+  packageRoot: resolvePath(fileURLToPath(import.meta.url), "../../../../"),
+});

package/templates/library-ts/test/decorators.test.ts.mu

@@ -0,0 +1,48 @@
+import { strictEqual } from "node:assert";
+import { describe, it, beforeEach } from "node:test";
+import { Operation } from "@typespec/compiler";
+import { BasicTestRunner, expectDiagnostics, extractCursor } from "@typespec/compiler/testing";
+import { getAlternateName } from "../src/decorators.js";
+import { create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestRunner } from "./test-host.js";
+
+describe("decorators", () => {
+  let runner: BasicTestRunner;
+
+  beforeEach(async () => {
+    runner = await create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestRunner();
+  })
+
+  describe("@alternateName", () => {
+    it("set alternate name on operation", async () => {
+      const { test } = (await runner.compile(
+        `@alternateName("bar") @test op test(): void;`
+      )) as { test: Operation };
+      strictEqual(getAlternateName(runner.program, test), "bar");
+    });
+
+    it("emit diagnostic if not used on an operation", async () => {
+      const diagnostics = await runner.diagnose(
+        `@alternateName("bar") model Test {}`
+      );
+      expectDiagnostics(diagnostics, {
+        severity: "error",
+        code: "decorator-wrong-target",
+        message: "Cannot apply @alternateName decorator to Test since it is not assignable to Operation"
+      })
+    });
+
+
+    it("emit diagnostic if using banned name", async () => {
+      const {pos, source} = extractCursor(`@alternateName(┆"banned") op test(): void;`)
+      const diagnostics = await runner.diagnose(
+        source
+      );
+      expectDiagnostics(diagnostics, {
+        severity: "error",
+        code: "{{name}}/banned-alternate-name",
+        message: `Banned alternate name "banned".`,
+        pos: pos + runner.autoCodeOffset
+      })
+    });
+  });
+});

package/templates/library-ts/test/rules/no-interfaces.rule.test.ts

@@ -0,0 +1,29 @@
+import {
+  LinterRuleTester,
+  createLinterRuleTester,
+  createTestRunner,
+} from "@typespec/compiler/testing";
+import { beforeEach, describe, it } from "node:test";
+import { noInterfaceRule } from "../../src/rules/no-interfaces.rule.js";
+
+describe("noInterfaceRule", () => {
+  let ruleTester: LinterRuleTester;
+
+  beforeEach(async () => {
+    const runner = await createTestRunner();
+    ruleTester = createLinterRuleTester(runner, noInterfaceRule, "{{name}}");
+  });
+
+  describe("models", () => {
+    it("emit diagnostics if using interfaces", async () => {
+      await ruleTester.expect(`interface Test {}`).toEmitDiagnostics({
+        code: "{{name}}/no-interface",
+        message: "Interface shouldn't be used with this library. Keep operations at the root.",
+      });
+    });
+
+    it("should be valid if operation is at the root", async () => {
+      await ruleTester.expect(`op test(): void;`).toBeValid();
+    });
+  });
+});

package/templates/library-ts/test/test-host.ts.mu

@@ -0,0 +1,17 @@
+import { createTestHost, createTestWrapper } from "@typespec/compiler/testing";
+import { {{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary } from "../src/testing/index.js";
+
+export async function create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestHost() {
+  return createTestHost({
+    libraries: [{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestLibrary],
+  });
+}
+
+export async function create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestRunner() {
+  const host = await create{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}TestHost();
+
+  return createTestWrapper(host, {
+    autoUsings: ["{{#casing.pascalCase}}{{name}}{{/casing.pascalCase}}"]
+  });
+}
+

package/templates/library-ts/tsconfig.json

@@ -0,0 +1,17 @@
+{
+  "compilerOptions": {
+    "target": "ES2022",
+    "useDefineForClassFields": true,
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "lib": ["ES2022"],
+    "rootDir": ".",
+    "outDir": "dist",
+    "sourceMap": true,
+    "declaration": true,
+
+    /* Linting */
+    "strict": true
+  },
+  "include": ["src", "test"]
+}

package/templates/scaffolding.json

@@ -20,6 +20,82 @@
       ]
     }
   },
+  "library-ts": {
+    "title": "TypeSpec Library (With TypeScript)",
+    "description": "Create a new package to add decorators or linters to typespec.",
+    "compilerVersion": "0.51.0",
+    "libraries": [],
+    "files": [
+      {
+        "destination": "main.tsp",
+        "skipGeneration": true
+      },
+      {
+        "destination": "tspconfig.yaml",
+        "skipGeneration": true
+      },
+      {
+        "path": "library-ts/.eslintrc.yml",
+        "destination": ".eslintrc.yml"
+      },
+      {
+        "path": "library-ts/lib/decorators.tsp.mu",
+        "destination": "lib/decorators.tsp"
+      },
+      {
+        "path": "library-ts/lib/main.tsp",
+        "destination": "lib/main.tsp"
+      },
+      {
+        "path": "library-ts/package.json",
+        "destination": "package.json"
+      },
+      {
+        "path": "library-ts/prettierrc.yaml",
+        "destination": "prettierrc.yaml"
+      },
+      {
+        "path": "library-ts/src/decorators.ts",
+        "destination": "src/decorators.ts"
+      },
+      {
+        "path": "library-ts/src/index.ts",
+        "destination": "src/index.ts"
+      },
+      {
+        "path": "library-ts/src/lib.ts",
+        "destination": "src/lib.ts"
+      },
+      {
+        "path": "library-ts/src/linter.ts",
+        "destination": "src/linter.ts"
+      },
+      {
+        "path": "library-ts/src/rules/no-interfaces.rule.ts",
+        "destination": "src/rules/no-interfaces.rule.ts"
+      },
+      {
+        "path": "library-ts/src/testing/index.ts.mu",
+        "destination": "src/testing/index.ts"
+      },
+      {
+        "path": "library-ts/test/decorators.test.ts.mu",
+        "destination": "test/decorators.test.ts"
+      },
+      {
+        "path": "library-ts/test/rules/no-interfaces.rule.test.ts",
+        "destination": "test/rules/no-interfaces.rule.test.ts"
+      },
+      {
+        "path": "library-ts/test/test-host.ts.mu",
+        "destination": "test/test-host.ts"
+      },
+      {
+        "path": "library-ts/tsconfig.json",
+        "destination": "tsconfig.json"
+      }
+    ]
+  },
   "emitter-ts": {
     "title": "TypeSpec Emitter (With TypeScript)",
     "description": "Create a new package that will be emitting typespec",
@@ -59,7 +135,7 @@
         "destination": "src/lib.ts"
       },
       {
-        "path": "emitter-ts/src/testing/index.ts",
+        "path": "emitter-ts/src/testing/index.ts.mu",
         "destination": "src/testing/index.ts"
       },
       {

package/templates/emitter-ts/src/testing/index.ts

@@ -1,10 +0,0 @@
-import {
-  createTestLibrary,
-  findTestPackageRoot,
-  TypeSpecTestLibrary,
-} from "@typespec/compiler/testing";
-
-export const TestLibrary: TypeSpecTestLibrary = createTestLibrary({
-  name: "{{name}}",
-  packageRoot: await findTestPackageRoot(import.meta.url),
-});