Merge pull request #462 from stasm/ts-langneg

Migrate @fluent/langneg to TypeScript

Author: stasm

eslint_ts.json

@@ -16,6 +16,7 @@
         "./eslint_src.json"
     ],
     "rules": {
+        "complexity": "off",
         "prefer-const": "off",
         "no-unused-vars": ["error", {"args": "none"}],
         "@typescript-eslint/no-inferrable-types": "off",

fluent-langneg/.esdoc.json

@@ -1,2 +1,3 @@
+esm/
 /index.js
 /compat.js

fluent-langneg/.gitignore

@@ -1,5 +1,7 @@
 .nyc_output
 coverage
-docs
+esm/.compiled
+src
 test
 makefile
+tsconfig.json

fluent-langneg/.npmignore

@@ -3,7 +3,21 @@ GLOBAL  := FluentLangNeg
 
 include ../common.mk
 
-test:
+lint:
+	@eslint --config $(ROOT)/eslint_ts.json --max-warnings 0 src/*.ts
+	@eslint --config $(ROOT)/eslint_test.json --max-warnings 0 test/
+	@echo -e " $(OK) lint"
+
+.PHONY: compile
+compile: esm/.compiled
+
+esm/.compiled: $(SOURCES)
+	@tsc
+	@touch $@
+	@echo -e " $(OK) esm/ compiled"
+
+.PHONY: test
+test: esm/.compiled
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
 	    --require esm \
@@ -13,7 +27,7 @@ test:
 build: index.js compat.js
 
 index.js: $(SOURCES)
-	@rollup $(CURDIR)/src/index.js \
+	@rollup $(CURDIR)/esm/index.js \
 	    --config $(ROOT)/bundle_config.js \
 	    --banner "/* $(PACKAGE)@$(VERSION) */" \
 	    --amd.id $(PACKAGE) \
@@ -22,14 +36,26 @@ index.js: $(SOURCES)
 	@echo -e " $(OK) $@ built"
 
 compat.js: $(SOURCES)
-	@rollup $(CURDIR)/src/index.js \
+	@rollup $(CURDIR)/esm/index.js \
 	    --config $(ROOT)/compat_config.js \
 	    --banner "/* $(PACKAGE)@$(VERSION) */" \
 	    --amd.id $(PACKAGE) \
 	    --name $(GLOBAL) \
 	    --output.file $@
 	@echo -e " $(OK) $@ built"
 
-lint: _lint
-html: _html
-clean: _clean
+html:
+	@typedoc src \
+	    --out ../html/bundle \
+	    --mode file \
+	    --excludeNotExported \
+	    --excludePrivate \
+	    --logger none \
+	    --hideGenerator
+	@echo -e " $(OK) html built"
+
+clean:
+	@rm -f esm/*.js esm/*.d.ts esm/.compiled
+	@rm -f index.js compat.js
+	@rm -rf .nyc_output coverage
+	@echo -e " $(OK) clean"

fluent-langneg/makefile

@@ -0,0 +1,7 @@
+export function acceptedLanguages(str: string = ""): Array<string> {
+  if (typeof str !== "string") {
+    throw new TypeError("Argument must be a string");
+  }
+  const tokens = str.split(",").map(t => t.trim());
+  return tokens.filter(t => t !== "").map(t => t.split(";")[0]);
+}

fluent-langneg/src/accepted_languages.js

@@ -7,5 +7,5 @@
  *
  */
 
-export { default as negotiateLanguages } from "./negotiate_languages";
-export { default as acceptedLanguages } from "./accepted_languages";
+export {negotiateLanguages} from "./negotiate_languages";
+export {acceptedLanguages} from "./accepted_languages";

fluent-langneg/src/accepted_languages.ts

@@ -1,6 +1,6 @@
 /* eslint no-magic-numbers: 0 */
 
-import { getLikelySubtagsMin } from "./subtags";
+import {getLikelySubtagsMin} from "./subtags";
 
 const languageCodeRe = "([a-z]{2,3}|\\*)";
 const scriptCodeRe = "(?:-([a-z]{4}|\\*))";
@@ -22,9 +22,13 @@ const variantCodeRe = "(?:-(([0-9][a-z0-9]{3}|[a-z0-9]{5,8})|\\*))";
 const localeRe = new RegExp(
   `^${languageCodeRe}${scriptCodeRe}?${regionCodeRe}?${variantCodeRe}?$`, "i");
 
-export const localeParts = ["language", "script", "region", "variant"];
+export class Locale {
+  isWellFormed: boolean;
+  language?: string;
+  script?: string;
+  region?: string;
+  variant?: string;
 
-export default class Locale {
   /**
    * Parses a locale id using the localeRe into an array with four elements.
    *
@@ -34,7 +38,7 @@ export default class Locale {
    * It also allows skipping the script section of the id, so `en-US` is
    * properly parsed as `en-*-US-*`.
    */
-  constructor(locale) {
+  constructor(locale: string) {
     const result = localeRe.exec(locale.replace(/_/g, "-"));
     if (!result) {
       this.isWellFormed = false;
@@ -56,38 +60,49 @@ export default class Locale {
     this.isWellFormed = true;
   }
 
-  isEqual(locale) {
-    return localeParts.every(part => this[part] === locale[part]);
+  isEqual(other: Locale): boolean {
+    return this.language === other.language
+      && this.script === other.script
+      && this.region === other.region
+      && this.variant === other.variant;
   }
 
-  matches(locale, thisRange = false, otherRange = false) {
-    return localeParts.every(part => {
-      return ((thisRange && this[part] === undefined) ||
-              (otherRange && locale[part] === undefined) ||
-              this[part] === locale[part]);
-    });
+  matches(other: Locale, thisRange = false, otherRange = false): boolean {
+    return (this.language === other.language
+        || thisRange && this.language === undefined
+        || otherRange && other.language === undefined)
+      && (this.script === other.script
+        || thisRange && this.script === undefined
+        || otherRange && other.script === undefined)
+      && (this.region === other.region
+        || thisRange && this.region === undefined
+        || otherRange && other.region === undefined)
+      && (this.variant === other.variant
+        || thisRange && this.variant === undefined
+        || otherRange && other.variant === undefined);
   }
 
-  toString() {
-    return localeParts
-      .map(part => this[part])
+  toString(): string {
+    return [this.language, this.script, this.region, this.variant]
       .filter(part => part !== undefined)
       .join("-");
   }
 
-  clearVariants() {
+  clearVariants(): void {
     this.variant = undefined;
   }
 
-  clearRegion() {
+  clearRegion(): void {
     this.region = undefined;
   }
 
-  addLikelySubtags() {
+  addLikelySubtags(): boolean {
     const newLocale = getLikelySubtagsMin(this.toString().toLowerCase());
-
     if (newLocale) {
-      localeParts.forEach(part => this[part] = newLocale[part]);
+      this.language = newLocale.language;
+      this.script = newLocale.script;
+      this.region = newLocale.region;
+      this.variant = newLocale.variant;
       return true;
     }
     return false;

fluent-langneg/src/index.js renamed to fluent-langneg/src/index.ts

@@ -1,7 +1,6 @@
 /* eslint no-magic-numbers: 0 */
-/* eslint complexity: ["error", { "max": 27 }] */
 
-import Locale from "./locale";
+import {Locale} from "./locale";
 
 /**
  * Negotiates the languages between the list of requested locales against
@@ -73,13 +72,13 @@ import Locale from "./locale";
  *    ignoring script ranges. That means that `sr-Cyrl` will never match
  *    against `sr-Latn`.
  */
-export default function filterMatches(
-  requestedLocales, availableLocales, strategy
-) {
-  /* eslint complexity: ["error", 31]*/
-  const supportedLocales = new Set();
-
-  const availableLocalesMap = new Map();
+export function filterMatches(
+  requestedLocales: Array<string>,
+  availableLocales: Array<string>,
+  strategy: string
+): Array<string> {
+  const supportedLocales: Set<string> = new Set();
+  const availableLocalesMap: Map<string, Locale> = new Map();
 
   for (let locale of availableLocales) {
     let newLocale = new Locale(locale);