Fix types

Author: wooorm

index.js

@@ -2,14 +2,28 @@
  * @typedef {import('unist').Point} Point
  * @typedef {import('unist').Node} Node
  * @typedef {import('unist').Position} Position
- * @typedef {object & {type: string, position?: Position|undefined}} NodeLike
+ */
+
+/**
+ * @typedef NodeLike
+ * @property {string} type
+ * @property {PositionLike | null | undefined} [position]
+ *
+ * @typedef PositionLike
+ * @property {PointLike | null | undefined} [start]
+ * @property {PointLike | null | undefined} [end]
+ *
+ * @typedef PointLike
+ * @property {number | null | undefined} [line]
+ * @property {number | null | undefined} [column]
+ * @property {number | null | undefined} [offset]
  */
 
 /**
  * Stringify one point, a position (start and end points), or a node’s
  * positional information.
  *
- * @param {Node|NodeLike|Position|Point|null} [value]
+ * @param {Node | NodeLike | Position | PositionLike | Point | PointLike | null | undefined} [value]
  * @returns {string}
  */
 export function stringifyPosition(value) {
@@ -38,23 +52,23 @@ export function stringifyPosition(value) {
 }
 
 /**
- * @param {Point|undefined} point
+ * @param {Point | PointLike | null | undefined} point
  * @returns {string}
  */
 function point(point) {
   return index(point && point.line) + ':' + index(point && point.column)
 }
 
 /**
- * @param {Position|undefined} pos
+ * @param {Position | PositionLike | null | undefined} pos
  * @returns {string}
  */
 function position(pos) {
   return point(pos && pos.start) + '-' + point(pos && pos.end)
 }
 
 /**
- * @param {number|undefined} value
+ * @param {number | null | undefined} value
  * @returns {number}
  */
 function index(value) {

test.js

@@ -24,12 +24,7 @@ test('stringifyPosition', function (t) {
     '',
     'should return empty `string` with `number`'
   )
-  t.equal(
-    // @ts-expect-error runtime.
-    stringifyPosition({}),
-    '',
-    'should return empty `string` with `{}`'
-  )
+  t.equal(stringifyPosition({}), '', 'should return empty `string` with `{}`')
 
   t.equal(
     stringifyPosition({type: 'text'}),
@@ -47,7 +42,6 @@ test('stringifyPosition', function (t) {
   t.equal(
     stringifyPosition({
       type: 'text',
-      // @ts-expect-error runtime.
       position: {start: {}, end: {}}
     }),
     '1:1-1:1',
@@ -58,9 +52,7 @@ test('stringifyPosition', function (t) {
     stringifyPosition({
       type: 'text',
       position: {
-        // @ts-expect-error runtime.
         start: {line: null, column: null},
-        // @ts-expect-error runtime.
         end: {line: null, column: null}
       }
     }),
@@ -96,7 +88,6 @@ test('stringifyPosition', function (t) {
   )
 
   t.equal(
-    // @ts-expect-error runtime.
     stringifyPosition({start: null, end: null}),
     '1:1-1:1',
     'should return a range for a `position` without `point`s'
@@ -110,17 +101,14 @@ test('stringifyPosition', function (t) {
   )
 
   t.equal(
-    // @ts-expect-error runtime.
     stringifyPosition({start: {}, end: {}}),
     '1:1-1:1',
     'should return range for `position` with invalid `point`s #1'
   )
 
   t.equal(
     stringifyPosition({
-      // @ts-expect-error runtime.
       start: {line: null, column: null},
-      // @ts-expect-error runtime.
       end: {line: null, column: null}
     }),
     '1:1-1:1',
@@ -137,7 +125,6 @@ test('stringifyPosition', function (t) {
   )
 
   t.equal(
-    // @ts-expect-error runtime.
     stringifyPosition({line: null, column: null}),
     '1:1',
     'should return a point for a `point` without indices'
@@ -151,14 +138,12 @@ test('stringifyPosition', function (t) {
   )
 
   t.equal(
-    // @ts-expect-error runtime.
     stringifyPosition({line: 4}),
     '4:1',
     'should return a point for a partially valid `point` #1'
   )
 
   t.equal(
-    // @ts-expect-error runtime.
     stringifyPosition({column: 12}),
     '1:12',
     'should return a point for a partially valid `point` #1'