improve types for CssLocations

projectwallace · Oct 29, 2023 · cc4b60f · cc4b60f
1 parent d01bb75
commit cc4b60f
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/src/collection.js b/src/collection.js
@@ -1,6 +1,6 @@
 export class Collection {
 	constructor({ useLocations = false }) {
-		/** @type {Map<string, Array<Number>>} */
+		/** @type {Map<string, number[]>} */
 		this._items = new Map()
 		this._total = 0
 		/** @type {number[]} */
@@ -44,6 +44,15 @@ export class Collection {
 		return this._total
 	}
 
+	/**
+	 * @typedef CssLocation
+	 * @property {number} line
+	 * @property {number} column
+	 * @property {number} offset
+	 * @property {number} length
+	 *
+	 * @returns {{ total: number; totalUnique: number; uniquenessRatio: number; unique: Record<string, number>; __unstable__uniqueWithLocations: Record<string, CssLocation[]>}}
+	 */
 	count() {
 		let uniqueWithLocations = new Map()
 		let unique = {}

diff --git a/src/context-collection.js b/src/context-collection.js
@@ -30,7 +30,7 @@ class ContextCollection {
      * @type {Map<string, {
      * total: number,
      * totalUnique: number,
-     * unique: {[k: string]: number},
+     * unique: Record<string, number>,
      * uniquenessRatio: number
      * }>}
      */

diff --git a/src/index.js b/src/index.js
@@ -84,7 +84,7 @@ export function analyze(css, options = {}) {
 
   // Atrules
   let totalAtRules = 0
-  /** @type {{[property: string]: string}[]} */
+  /** @type {Record<string: string>}[]} */
   let fontfaces = []
   let layers = new Collection({ useLocations })
   let imports = new Collection({ useLocations })