@typespec/http-client-python 0.37.0 → 0.37.2

package/emitter/test/emitter.test.ts

@@ -0,0 +1,32 @@
+import { expectDiagnostics, t } from "@typespec/compiler/testing";
+import { expect, it } from "vitest";
+import { emitCodeModel } from "./test-host.js";
+
+it("targets the service namespace when no SDK clients are found", async () => {
+  const { diagnostics } = await emitCodeModel(t.code`
+    #suppress "@typespec/http-client-python/no-sdk-clients" "This service intentionally has no client."
+    @service namespace ${t.namespace("Service")} {}
+  `);
+
+  expectDiagnostics(diagnostics, []);
+});
+
+it("generates models when no service exists", async () => {
+  const { codeModel, diagnostics } = await emitCodeModel(`
+    import "@azure-tools/typespec-client-generator-core";
+    using Azure.ClientGenerator.Core;
+
+    #suppress "@typespec/http-client-python/no-sdk-clients" "This model-only package intentionally has no client."
+    @access(Access.public)
+    @usage(Usage.input | Usage.output)
+    @clientNamespace("Models")
+    model Widget {}
+  `);
+
+  expectDiagnostics(diagnostics, []);
+  // A model-only package has no clients but must still emit its models into the code model.
+  expect(codeModel.clients).toHaveLength(0);
+  expect(codeModel.types.some((type) => type.type === "model" && type.name === "Widget")).toBe(
+    true,
+  );
+});

package/emitter/test/test-host.ts

@@ -0,0 +1,83 @@
+import type { Diagnostic } from "@typespec/compiler";
+import { resolvePath } from "@typespec/compiler";
+import { createTester, mockFile } from "@typespec/compiler/testing";
+import { mkdtemp, readdir, readFile, rm } from "fs/promises";
+import jsyaml from "js-yaml";
+import { tmpdir } from "os";
+import { join } from "path";
+import { $onEmit } from "../src/emitter.js";
+
+const PythonTester = createTester(resolvePath(import.meta.dirname, "../.."), {
+  libraries: ["@azure-tools/typespec-client-generator-core"],
+});
+
+export const Tester = PythonTester;
+
+// `@typespec/http-client-python` is intentionally excluded from the pnpm workspace, so it isn't
+// linked into its own `node_modules` and `.emit()` can't resolve it by name. Register a minimal
+// virtual package whose entrypoint re-exports the real `$onEmit`, so the emitter under test runs.
+export const EmitterTester = PythonTester.files({
+  "node_modules/@typespec/http-client-python/package.json": JSON.stringify({
+    name: "@typespec/http-client-python",
+    version: "0.0.0",
+    exports: { ".": "./index.js" },
+  }),
+  "node_modules/@typespec/http-client-python/index.js": mockFile.js({
+    $onEmit,
+  }),
+}).emit("@typespec/http-client-python", {
+  "generate-packaging-files": false,
+});
+
+/** A type entry in the emitted code model. */
+export interface CodeModelType {
+  type: string;
+  name?: string;
+}
+
+/** The subset of the emitted code model that the emitter tests assert against. */
+export interface CodeModel {
+  clients: unknown[];
+  types: CodeModelType[];
+}
+
+/**
+ * Run the emitter's TypeScript step only (via `emit-yaml-only`) and return the code model it
+ * produced along with any diagnostics. This stops right after `emitCodeModel`, so it exercises the
+ * real emitter logic that decides what gets generated without booting the Python/Pyodide generator.
+ */
+export async function emitCodeModel(
+  code: string,
+): Promise<{ codeModel: CodeModel; diagnostics: readonly Diagnostic[] }> {
+  const outputDir = await mkdtemp(join(tmpdir(), "typespec-python-"));
+  let yamlPath: string | undefined;
+  try {
+    const [, diagnostics] = await EmitterTester.compileAndDiagnose(code, {
+      compilerOptions: {
+        options: {
+          "@typespec/http-client-python": {
+            "emit-yaml-only": true,
+            "emitter-output-dir": outputDir,
+          },
+        },
+      },
+    });
+
+    // `emit-yaml-only` writes a `.tsp-codegen-*.json` pointer into the output dir that references
+    // the serialized code model YAML written to the OS temp dir.
+    const pointerName = (await readdir(outputDir)).find(
+      (name) => name.startsWith(".tsp-codegen-") && name.endsWith(".json"),
+    );
+    if (!pointerName) {
+      throw new Error("Emitter did not produce a code model.");
+    }
+    ({ yamlPath } = JSON.parse(await readFile(join(outputDir, pointerName), "utf-8")));
+    const codeModel = jsyaml.load(await readFile(yamlPath!, "utf-8")) as CodeModel;
+    return { codeModel, diagnostics };
+  } finally {
+    await rm(outputDir, { recursive: true, force: true });
+    if (yamlPath) {
+      await rm(yamlPath, { force: true });
+    }
+  }
+}

package/eng/scripts/ci/config/oxlint.json

@@ -0,0 +1,6 @@
+{
+  "ignorePatterns": ["**/dist/**/*", "**/node_modules/**/*"],
+  "rules": {
+    "unicorn/no-useless-spread": "off"
+  }
+}

package/eng/scripts/ci/lint.ts

@@ -41,7 +41,7 @@ ${pc.bold("Description:")}
 
 ${pc.bold("Options:")}
   ${pc.cyan("-e, --emitter")}
-      Run ESLint on TypeScript emitter code.
+      Run Oxlint on TypeScript emitter code.
 
   ${pc.cyan("-g, --generator")}
       Run pylint on Python generator source code (pygen).
@@ -85,7 +85,7 @@ function runCommand(
   const workingDir = cwd || root;
   const name = displayName || command;
 
-  // Add node_modules/.bin directories to PATH so commands like eslint can be found
+  // Add node_modules/.bin directories to PATH so commands like oxlint can be found
   // Also set NODE_PATH so that config files can resolve packages from monorepo's node_modules
   const pathSep = process.platform === "win32" ? ";" : ":";
   const binPaths = [
@@ -134,20 +134,17 @@ function runCommand(
 
 async function lintEmitter(): Promise<boolean> {
   console.log(`\n${pc.bold("=== Linting TypeScript ===")}\n`);
-  // Run eslint with local config to avoid dependency on monorepo's eslint.config.js
-  // This ensures the package can be linted in CI without full monorepo dependencies
-  // Lint both emitter/ and eng/scripts/ directories
   return runCommand(
-    "eslint",
+    "oxlint",
     [
       "emitter/",
       "eng/scripts/",
       "--config",
-      "eng/scripts/ci/config/eslint-ci.config.mjs",
-      "--max-warnings=0",
+      "eng/scripts/ci/config/oxlint.json",
+      "--deny-warnings",
     ],
     root,
-    "eslint emitter/ eng/scripts/ --max-warnings=0",
+    "oxlint emitter/ eng/scripts/ --deny-warnings",
   );
 }
 

package/generator/pygen.egg-info/PKG-INFO

@@ -21,7 +21,7 @@ Requires-Dist: docutils>=0.20.1
 Requires-Dist: Jinja2==3.1.6
 Requires-Dist: PyYAML==6.0.1
 Requires-Dist: tomli==2.0.1
-Requires-Dist: setuptools==75.3.2
+Requires-Dist: setuptools==83.0.0
 Dynamic: author
 Dynamic: author-email
 Dynamic: classifier

package/generator/pygen.egg-info/requires.txt

@@ -3,4 +3,4 @@ docutils>=0.20.1
 Jinja2==3.1.6
 PyYAML==6.0.1
 tomli==2.0.1
-setuptools==75.3.2
+setuptools==83.0.0

package/generator/setup.py

@@ -52,6 +52,6 @@ setup(
         "Jinja2==3.1.6",
         "PyYAML==6.0.1",
         "tomli==2.0.1",
-        "setuptools==75.3.2",
+        "setuptools==83.0.0",
     ],
 )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typespec/http-client-python",
-  "version": "0.37.0",
+  "version": "0.37.2",
   "author": "Microsoft Corporation",
   "description": "TypeSpec emitter for Python SDKs",
   "homepage": "https://typespec.io",
@@ -66,20 +66,20 @@
     "emitter"
   ],
   "peerDependencies": {
-    "@azure-tools/typespec-autorest": ">=0.71.0 <1.0.0",
-    "@azure-tools/typespec-azure-core": ">=0.71.0 <1.0.0",
-    "@azure-tools/typespec-azure-resource-manager": ">=0.71.0 <1.0.0",
-    "@azure-tools/typespec-azure-rulesets": ">=0.71.0 <1.0.0",
-    "@azure-tools/typespec-client-generator-core": ">=0.71.0 <1.0.0",
-    "@typespec/compiler": "^1.15.0",
-    "@typespec/http": "^1.15.0",
-    "@typespec/openapi": "^1.15.0",
-    "@typespec/rest": ">=0.85.0 <1.0.0",
-    "@typespec/versioning": ">=0.85.0 <1.0.0",
-    "@typespec/events": ">=0.85.0 <1.0.0",
-    "@typespec/sse": ">=0.85.0 <1.0.0",
-    "@typespec/streams": ">=0.85.0 <1.0.0",
-    "@typespec/xml": ">=0.85.0 <1.0.0"
+    "@azure-tools/typespec-autorest": ">=0.72.0 <1.0.0",
+    "@azure-tools/typespec-azure-core": ">=0.72.0 <1.0.0",
+    "@azure-tools/typespec-azure-resource-manager": ">=0.72.0 <1.0.0",
+    "@azure-tools/typespec-azure-rulesets": ">=0.72.0 <1.0.0",
+    "@azure-tools/typespec-client-generator-core": ">=0.72.0 <1.0.0",
+    "@typespec/compiler": "^1.16.0",
+    "@typespec/http": "^1.16.0",
+    "@typespec/openapi": "^1.16.0",
+    "@typespec/rest": ">=0.86.0 <1.0.0",
+    "@typespec/versioning": ">=0.86.0 <1.0.0",
+    "@typespec/events": ">=0.86.0 <1.0.0",
+    "@typespec/sse": ">=0.86.0 <1.0.0",
+    "@typespec/streams": ">=0.86.0 <1.0.0",
+    "@typespec/xml": ">=0.86.0 <1.0.0"
   },
   "playgroundConfig": {
     "assets": [
@@ -97,39 +97,39 @@
     ]
   },
   "dependencies": {
-    "js-yaml": "^4.2.0",
+    "js-yaml": "^4.3.1",
     "marked": "^15.0.6",
     "pyodide": "0.26.2",
     "semver": "~7.6.2",
-    "tsx": "^4.21.0"
+    "tsx": "^4.23.13"
   },
   "devDependencies": {
-    "@azure-tools/typespec-autorest": "~0.71.0",
-    "@azure-tools/typespec-azure-core": "~0.71.0",
-    "@azure-tools/typespec-azure-resource-manager": "~0.71.0",
-    "@azure-tools/typespec-azure-rulesets": "~0.71.0",
-    "@azure-tools/typespec-client-generator-core": "~0.71.0",
-    "@azure-tools/azure-http-specs": "0.1.0-alpha.44",
-    "@typespec/compiler": "^1.15.0",
-    "@typespec/http": "^1.15.0",
-    "@typespec/openapi": "^1.15.0",
-    "@typespec/rest": "~0.85.0",
-    "@typespec/versioning": "~0.85.0",
-    "@typespec/events": "~0.85.0",
-    "@typespec/spector": "0.1.0-alpha.29-dev.0",
-    "@typespec/spec-api": "0.1.0-alpha.16",
-    "@typespec/sse": "~0.85.0",
-    "@typespec/streams": "~0.85.0",
-    "@typespec/xml": "~0.85.0",
-    "@typespec/http-specs": "0.1.0-alpha.42-dev.2",
+    "@azure-tools/typespec-autorest": "~0.72.0",
+    "@azure-tools/typespec-azure-core": "~0.72.0",
+    "@azure-tools/typespec-azure-resource-manager": "~0.72.0",
+    "@azure-tools/typespec-azure-rulesets": "~0.72.0",
+    "@azure-tools/typespec-client-generator-core": "~0.72.0",
+    "@azure-tools/azure-http-specs": "0.1.0-alpha.45",
+    "@typespec/compiler": "^1.16.0",
+    "@typespec/http": "^1.16.0",
+    "@typespec/openapi": "^1.16.0",
+    "@typespec/rest": "~0.86.0",
+    "@typespec/versioning": "~0.86.0",
+    "@typespec/events": "~0.86.0",
+    "@typespec/spector": "0.1.0-alpha.29",
+    "@typespec/spec-api": "0.1.0-alpha.17",
+    "@typespec/sse": "~0.86.0",
+    "@typespec/streams": "~0.86.0",
+    "@typespec/xml": "~0.86.0",
+    "@typespec/http-specs": "0.1.0-alpha.42",
     "@types/js-yaml": "~4.0.5",
     "@types/node": "~25.0.2",
     "@types/semver": "7.5.8",
     "c8": "^10.1.3",
+    "oxlint": "^1.69.0",
     "picocolors": "~1.1.1",
     "rimraf": "~6.1.2",
     "typescript": "~5.9.2",
-    "typescript-eslint": "^8.49.0",
     "vitest": "^4.0.15",
     "prettier": "^3.9.5"
   }

package/eng/scripts/ci/config/eslint-ci.config.mjs

@@ -1,64 +0,0 @@
-// @ts-check
-// Standalone eslint config for http-client-python package
-// This config is used in CI where monorepo dependencies may not be available
-import eslint from "@eslint/js";
-import { dirname } from "path";
-import tsEslint from "typescript-eslint";
-import { fileURLToPath } from "url";
-
-const root = dirname(dirname(dirname(dirname(fileURLToPath(import.meta.url)))));
-
-export default [
-  {
-    ignores: ["**/dist/**/*", "**/node_modules/**/*"],
-  },
-  eslint.configs.recommended,
-  ...tsEslint.configs.recommended,
-  {
-    languageOptions: {
-      parserOptions: {
-        tsconfigRootDir: root,
-      },
-    },
-    rules: {
-      // TypeScript plugin overrides
-      "@typescript-eslint/no-non-null-assertion": "off",
-      "@typescript-eslint/no-explicit-any": "off",
-      "@typescript-eslint/no-inferrable-types": "off",
-      "@typescript-eslint/no-empty-function": "off",
-      "@typescript-eslint/no-empty-interface": "off",
-      "@typescript-eslint/no-empty-object-type": "off",
-      "@typescript-eslint/no-unused-vars": [
-        "warn",
-        {
-          varsIgnorePattern: "^_",
-          argsIgnorePattern: ".*",
-          ignoreRestSiblings: true,
-          caughtErrorsIgnorePattern: ".*",
-        },
-      ],
-      "@typescript-eslint/no-unused-expressions": [
-        "warn",
-        { allowShortCircuit: true, allowTernary: true },
-      ],
-
-      // Core rules
-      "no-inner-declarations": "off",
-      "no-empty": "off",
-      "no-constant-condition": "off",
-      "no-case-declarations": "off",
-      "no-ex-assign": "off",
-      "no-undef": "off",
-      "no-useless-assignment": "error",
-      "prefer-const": [
-        "warn",
-        {
-          destructuring: "all",
-        },
-      ],
-      eqeqeq: ["warn", "always", { null: "ignore" }],
-      "no-console": "warn",
-      "symbol-description": "warn",
-    },
-  },
-];