Address review comments

Author: sethmlarson

output/schema/schema.json

@@ -39,7 +39,11 @@ import { Sort, SortResults } from '@_types/sort'
 
 export class Hit<TDocument> {
   _index: IndexName
-  _id?: Id
+  /**
+   * @es_quirk '_id' is not available when using 'stored_fields: _none_'
+   * on a search request. Otherwise the field is always present on hits.
+   */
+  _id: Id
   _score?: double | null
   _explanation?: Explanation
   fields?: Dictionary<string, UserDefinedValue>

output/typescript/types.ts

@@ -49,7 +49,7 @@ export class ErrorCause
 
 export class ShardFailure {
   index?: IndexName
-  node?: string | null
+  node?: string
   reason: ErrorCause
   shard: integer
   status?: string

specification/_global/search/_types/hits.ts

@@ -98,12 +98,7 @@ export type SortCombinations = Field | SortOptions
 
 export type Sort = SortCombinations | SortCombinations[]
 
-/**
- * Only used when sorting by "_tsid"
- */
-export type SortTsid = Dictionary<Field, FieldValue>
-
-export type SortResults = Array<long | double | string | null | SortTsid>
+export type SortResults = FieldValue[]
 
 /**
  * Defines what values to pick in the case a document contains multiple values for a particular field.