fix: escape double quotes in generated string literals and enum values

fix: type-checkエラーをすべて解消

- tsconfig.json: test/code/をexcludeに追加（生成済みフィクスチャは型チェック対象外）
- factory/index.ts: string enum値とLiteralTypeNodeのダブルクォートをエスケープ処理
- test/__tests__/**/*.ts: vitestのdescribe/test/expectを明示的にimport
- examples/readme-sample/*.ts: Params APIの変更(functionName→convertedParams.functionName, comment→operationParams.comment)に対応
- scripts/tools/shell.ts: ExecaChildProcessをexeca v9のResultPromiseに置換
- .github/workflows/build.yml: pnpm type-checkをCIに追加

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Himenon

.github/workflows/build.yml

@@ -33,6 +33,7 @@ jobs:
       - name: Test & Build
         run: |
           pnpm build
+          pnpm type-check
           pnpm test:code:gen
           pnpm test
         env:

examples/readme-sample/ast-code-template.ts

@@ -12,7 +12,7 @@ const generator: Types.CodeGenerator.GenerateFunction<Option> = (
   return payload.map(params => {
     return factory.InterfaceDeclaration.create({
       export: true,
-      name: params.functionName,
+      name: params.convertedParams.functionName,
       members: [],
     });
   });

examples/readme-sample/generator-template.ts

@@ -4,6 +4,7 @@ import { CodeGenerator } from "@himenon/openapi-typescript-code-generator";
 import type * as Types from "@himenon/openapi-typescript-code-generator/dist/types";
 
 /** ここにCode Templateの定義を記述してください  */
+// @ts-ignore -- placeholder example, generator is intentionally omitted
 const customGenerator: Types.CodeGenerator.CustomGenerator<{}> = {
   /** .... */
 };

examples/readme-sample/use-extract-schema-params.ts

@@ -4,7 +4,7 @@ type Option = {};
 
 const generator: Types.CodeGenerator.GenerateFunction<Option> = (payload: Types.CodeGenerator.Params[]): string[] => {
   return payload.map(params => {
-    return `function ${params.operationId}() { console.log("${params.comment}") }`;
+    return `function ${params.operationId}() { console.log("${params.operationParams.comment}") }`;
   });
 };
 

package.json

@@ -73,6 +73,7 @@
     "test:snapshot": "vitest run --config ./vitest.snapshot.config.ts",
     "test:vitest": "vitest run --config ./vitest.config.ts --coverage",
     "ts": "node --no-warnings=ExperimentalWarning --experimental-specifier-resolution=node --loader ts-node/esm",
+    "type-check": "tsc --noEmit",
     "update:snapshot": "vitest run --config ./vitest.snapshot.config.ts --update",
     "validate": "pnpm ts ./scripts/validate.ts",
     "watch": "pnpm ts ./scripts/watch.ts"

scripts/tools/shell.ts

@@ -1,6 +1,6 @@
-import { ExecaChildProcess, execa } from "execa";
+import { type ResultPromise, execa } from "execa";
 
-export const shell = (command: string, cwd: string = process.cwd()): ExecaChildProcess<string> => {
+export const shell = (command: string, cwd: string = process.cwd()): ResultPromise => {
   return execa(command, {
     stdio: ["pipe", "pipe", "inherit"],
     shell: true,

src/internal/TsGenerator/factory/index.ts

@@ -228,7 +228,8 @@ export const create = (): Type => {
       create(p) {
         const node = (() => {
           if (typeof p.value === "string") {
-            return `"${p.value}"`;
+            const escaped = p.value.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
+            return `"${escaped}"`;
           }
           if (typeof p.value === "number") {
             return `${p.value}`;
@@ -244,7 +245,7 @@ export const create = (): Type => {
         switch (p.type) {
           case "string":
             if (p.enum) {
-              return p.enum.map(v => `"${v}"`).join(" | ");
+              return p.enum.map(v => `"${v.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"`).join(" | ");
             }
             return "string";
           case "number":

test/__tests__/class/__snapshots__/cloudflare-test.ts.snap

@@ -10459,7 +10459,7 @@ mode?: string;
 port?: number;
 }
     /** The session_affinity specifies the type of session affinity the load balancer should use unless specified as "none" or ""(default). The supported types are "cookie" and "ip_cookie". "cookie" - On the first request to a proxied load balancer, a cookie is generated, encoding information of which origin the request will be forwarded to. Subsequent requests, by the same client to the same load balancer, will be sent to the origin server the cookie encodes, for the duration of the cookie and as long as the origin server remains healthy. If the cookie has expired or the origin server is unhealthy then a new origin server is calculated and used. "ip_cookie" behaves the same as "cookie" except the initial origin selection is stable and based on the client’s ip address. */
-export type legacy$jhs_session_affinity = "none" | "cookie" | "ip_cookie" | """";
+export type legacy$jhs_session_affinity = "none" | "cookie" | "ip_cookie" | "\\"\\"";
     /** Configures cookie attributes for session affinity cookie. */
 export interface legacy$jhs_session_affinity_attributes {
     /** Configures the drain duration in seconds. This field is only used when session affinity is enabled on the load balancer. */
@@ -10611,7 +10611,7 @@ export type legacy$jhs_std_dev_rtt_ms = number;
  * - \`"proximity"\`: Use the pools' latitude and longitude to select the closest pool using the Cloudflare PoP location for proxied requests or the location determined by \`location_strategy\` for non-proxied requests.
  * - \`""\`: Will map to \`"geo"\` if you use \`region_pools\`/\`country_pools\`/\`pop_pools\` otherwise \`"off"\`.
  */
-export type legacy$jhs_steering_policy = "off" | "geo" | "random" | "dynamic_latency" | "proximity" | """";
+export type legacy$jhs_steering_policy = "off" | "geo" | "random" | "dynamic_latency" | "proximity" | "\\"\\"";
     /** STIX 2.1 identifier: https://docs.oasis-open.org/cti/stix/v2.1/cs02/stix-v2.1-cs02.html#_64yvzeku5a5c */
 export type legacy$jhs_stix_identifier = string;
     export type legacy$jhs_subdomain$response = Schemas.legacy$jhs_api$response$common & {
@@ -11427,7 +11427,7 @@ policy?: "random" | "hash" | "least_outstanding_requests" | "least_connections";
     /** The list of origins within this pool. Traffic directed at this pool is balanced across all currently healthy origins, provided the pool itself is healthy. */
 export type load$balancing_origins = Schemas.load$balancing_origin[];
     /** The email address to send health status notifications to. This field is now deprecated in favor of Cloudflare Notifications for Load Balancing, so only resetting this field with an empty string \`""\` is accepted. */
-export type load$balancing_patch_pools_notification_email = """";
+export type load$balancing_patch_pools_notification_email = "\\"\\"";
     /** The endpoint path you want to conduct a health check against. This parameter is only valid for HTTP and HTTPS monitors. */
 export type load$balancing_path = string;
     export interface load$balancing_pool {
@@ -11617,7 +11617,7 @@ references?: "" | "*" | "referral" | "referrer";
  * - \`"ip_cookie"\`: Behaves the same as \`"cookie"\` except the initial origin selection is stable and based on the client's ip address.
  * - \`"header"\`: On the first request to a proxied load balancer, a session key based on the configured HTTP headers (see \`session_affinity_attributes.headers\`) is generated, encoding the request headers used for storing in the load balancer session state which origin the request will be forwarded to. Subsequent requests to the load balancer with the same headers will be sent to the same origin server, for the duration of the session and as long as the origin server remains healthy. If the session has been idle for the duration of \`session_affinity_ttl\` seconds or the origin server is unhealthy, then a new origin server is calculated and used. See \`headers\` in \`session_affinity_attributes\` for additional required configuration.
  */
-export type load$balancing_session_affinity = "none" | "cookie" | "ip_cookie" | "header" | """";
+export type load$balancing_session_affinity = "none" | "cookie" | "ip_cookie" | "header" | "\\"\\"";
     /** Configures attributes for session affinity. */
 export interface load$balancing_session_affinity_attributes {
     /** Configures the drain duration in seconds. This field is only used when session affinity is enabled on the load balancer. */
@@ -11659,7 +11659,7 @@ export type load$balancing_session_affinity_ttl = number;
  * - \`"least_connections"\`: Select a pool by taking into consideration \`random_steering\` weights, as well as each pool's number of open connections. Pools with more open connections are weighted proportionately less relative to others. Supported for HTTP/1 and HTTP/2 connections.
  * - \`""\`: Will map to \`"geo"\` if you use \`region_pools\`/\`country_pools\`/\`pop_pools\` otherwise \`"off"\`.
  */
-export type load$balancing_steering_policy = "off" | "geo" | "random" | "dynamic_latency" | "proximity" | "least_outstanding_requests" | "least_connections" | """";
+export type load$balancing_steering_policy = "off" | "geo" | "random" | "dynamic_latency" | "proximity" | "least_outstanding_requests" | "least_connections" | "\\"\\"";
     /** Two-letter subdivision code followed in ISO 3166-2. */
 export type load$balancing_subdivision_code_a2 = string;
     /** The timeout (in seconds) before marking the health check as failed. */

test/__tests__/class/argo-rollout-test.ts

@@ -1,3 +1,5 @@
+import { describe, expect, test } from "vitest";
+
 import * as fs from "fs";
 
 import * as Utils from "../../utils";

test/__tests__/class/cloudflare-test.ts

@@ -1,3 +1,5 @@
+import { describe, expect, test } from "vitest";
+
 import * as fs from "fs";
 
 import * as Utils from "../../utils";