feat: uri encoded $refs (#617) (#618)

Author: DavideCanton

api.md

@@ -1846,6 +1846,13 @@ export type MyTuple = [string, number, boolean?];
 ```
 
 
+## [type-aliases-tuple-with-names](./test/programs/type-aliases-tuple-with-names)
+
+```ts
+export type MyTuple = [a: string, b: 123, c?: boolean, ...d: number[]];
+```
+
+
 ## [type-aliases-tuple-with-rest-element](./test/programs/type-aliases-tuple-with-rest-element)
 
 ```ts

test/programs/generic-anonymous/schema.json

@@ -37,10 +37,10 @@
     },
     "properties": {
         "value1": {
-            "$ref": "#/definitions/MyGeneric<string,number>"
+            "$ref": "#/definitions/MyGeneric%3Cstring%2Cnumber%3E"
         },
         "value2": {
-            "$ref": "#/definitions/MyGeneric<number,string>"
+            "$ref": "#/definitions/MyGeneric%3Cnumber%2Cstring%3E"
         }
     },
     "required": [

test/programs/generic-multiargs/schema.json

@@ -37,10 +37,10 @@
     },
     "properties": {
         "value1": {
-            "$ref": "#/definitions/MyGeneric<string,number>"
+            "$ref": "#/definitions/MyGeneric%3Cstring%2Cnumber%3E"
         },
         "value2": {
-            "$ref": "#/definitions/MyGeneric<number,string>"
+            "$ref": "#/definitions/MyGeneric%3Cnumber%2Cstring%3E"
         }
     },
     "required": [

test/programs/generic-multiple/schema.json

@@ -29,10 +29,10 @@
     },
     "properties": {
         "value1": {
-            "$ref": "#/definitions/MyGeneric<number>"
+            "$ref": "#/definitions/MyGeneric%3Cnumber%3E"
         },
         "value2": {
-            "$ref": "#/definitions/MyGeneric<string>"
+            "$ref": "#/definitions/MyGeneric%3Cstring%3E"
         }
     },
     "required": [

test/programs/generic-recursive/schema.json

@@ -6,7 +6,7 @@
             "additionalProperties": false,
             "properties": {
                 "field": {
-                    "$ref": "#/definitions/MyGeneric<string,number>"
+                    "$ref": "#/definitions/MyGeneric%3Cstring%2Cnumber%3E"
                 }
             },
             "required": [
@@ -18,7 +18,7 @@
             "additionalProperties": false,
             "properties": {
                 "field": {
-                    "$ref": "#/definitions/MyGeneric<number,string>"
+                    "$ref": "#/definitions/MyGeneric%3Cnumber%2Cstring%3E"
                 }
             },
             "required": [
@@ -30,7 +30,7 @@
             "additionalProperties": false,
             "properties": {
                 "value": {
-                    "$ref": "#/definitions/MyGeneric<string,number>"
+                    "$ref": "#/definitions/MyGeneric%3Cstring%2Cnumber%3E"
                 }
             },
             "required": [

test/programs/generic-simple/schema.json

@@ -17,7 +17,7 @@
     },
     "properties": {
         "value": {
-            "$ref": "#/definitions/MyGeneric<number>"
+            "$ref": "#/definitions/MyGeneric%3Cnumber%3E"
         }
     },
     "required": [

typescript-json-schema.ts

@@ -1414,7 +1414,7 @@ export class JsonSchemaGenerator {
             // We don't return the full definition, but we put it into
             // reffedDefinitions below.
             returnedDefinition = {
-                $ref: `${this.args.id}#/definitions/` + fullTypeName,
+                $ref: createRefURI(this.args.id, fullTypeName),
             };
         }
 
@@ -1519,7 +1519,7 @@ export class JsonSchemaGenerator {
                 }, {});
 
                 returnedDefinition = {
-                    $ref: `${this.args.id}#/definitions/` + fullTypeName,
+                    $ref: createRefURI(this.args.id, fullTypeName),
                     ...annotations,
                 };
             }
@@ -1640,6 +1640,22 @@ export class JsonSchemaGenerator {
     }
 }
 
+/**
+ * Generates the reference URI to the definition in the schema `id`
+ * containing the definition `name`.
+ *
+ * Encodes the `name` using `encodeURIComponent`.
+ *
+ * @see https://datatracker.ietf.org/doc/html/rfc3986
+ * @param id the id of the schema containing the definition
+ * @param name the name of the definition
+ * @returns the URI pointing to the definition in the schema
+ */
+function createRefURI(id: string, name: string): string {
+    const encoded = encodeURIComponent(name);
+    return `${id}#/definitions/${encoded}`;
+}
+
 export function getProgramFromFiles(
     files: string[],
     jsonCompilerOptions: any = {},