Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { ExampleEndpointCall, HttpHeader, InlinedRequestBodyProperty, QueryParameter } from "@fern-fern/ir-sdk/api";
import { GetReferenceOpts } from "@fern-typescript/commons";
import { GetReferenceOpts, isValidIdentifier } from "@fern-typescript/commons";
import { GeneratedRequestWrapper, SdkContext } from "@fern-typescript/contexts";
import { ts } from "ts-morph";

Expand Down Expand Up @@ -111,9 +111,13 @@ export class FileUploadRequestParameter extends AbstractRequestParameter {
}

private getReferenceToProperty(propertyName: string): ts.Expression {
return ts.factory.createPropertyAccessExpression(
ts.factory.createIdentifier(this.getRequestParameterName()),
propertyName
);
const requestIdentifier = ts.factory.createIdentifier(this.getRequestParameterName());
if (!isValidIdentifier(propertyName)) {
return ts.factory.createElementAccessExpression(
requestIdentifier,
ts.factory.createStringLiteral(propertyName)
);
}
return ts.factory.createPropertyAccessExpression(requestIdentifier, propertyName);
}
}
10 changes: 10 additions & 0 deletions generators/typescript/sdk/versions.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,14 @@
# yaml-language-server: $schema=../../../fern-versions-yml.schema.json
- version: 3.38.4
changelogEntry:
- summary: |
Fix file upload request parameters to use bracket notation for property names that are not valid JavaScript identifiers (e.g., header names with hyphens like `X-Custom-Header`).
Previously, the generator produced invalid JavaScript like `request.X - Custom - Header` which was interpreted as subtraction operations.
Now it correctly generates `request["X-Custom-Header"]`.
type: fix
createdAt: "2025-12-10"
irVersion: 62

- version: 3.38.3
changelogEntry:
- summary: |
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
import esutils from "esutils";

/**
* Checks if a string is a valid JavaScript/TypeScript identifier name.
* This can be used to determine whether to use dot notation (obj.prop) or bracket notation (obj["prop"]).
*/
export function isValidIdentifier(name: string): boolean {
return esutils.keyword.isIdentifierNameES6(name);
}

export function getPropertyKey(key: string): string {
// [key: string]: any; => [key: string]: any;
if (key.startsWith("[") && key.endsWith("]")) {
Expand Down
2 changes: 1 addition & 1 deletion generators/typescript/utils/commons/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export {
getParameterNameForRootExamplePathParameter,
getParameterNameForRootPathParameter
} from "./codegen-utils/getParameterNameForPathParameter";
export { getPropertyKey } from "./codegen-utils/getPropertyKey";
export { getPropertyKey, isValidIdentifier } from "./codegen-utils/getPropertyKey";
export { getSchemaOptions } from "./codegen-utils/getSchemaOptions";
export { getSdkParameterPropertyName } from "./codegen-utils/getSdkParameterPropertyName";
export { getTextOfTsKeyword } from "./codegen-utils/getTextOfTsKeyword";
Expand Down
Loading