style: включил type-check для tslint

Author: standy

package.json

@@ -24,7 +24,7 @@
     "release-minor": "npm run release -- --release-as minor",
     "release-major": "npm run release -- --release-as major",
     "lint": "eslint **/*.js --fix",
-    "tslint": "tslint source/**/*.ts -c tslint.json --fix",
+    "tslint": "tslint source/**/*.ts --fix --type-check -p tsconfig.json",
     "build-chrome": "webpack --config webpack.chrome.config.js",
     "build-userscript": "webpack --config webpack.userscript.config.js",
     "build": "npm run build-chrome && npm run build-userscript",

source/js/auto/Auto.ts

@@ -157,36 +157,36 @@ export default class Auto {
 						Auto.CHOICES.hasOwnProperty(reward) &&
 						Auto.CHOICES[reward].includes(choiceName)
 					) {
-						chooseQuest(option_uid, choiceName);
+						this.chooseQuest(option_uid, choiceName, settingsValues);
 					}
 				}
 			}
 		}
+	}
 
-		function chooseQuest(uid: string, name: string) {
-			if (settingsValues.autoquestNotify && this.lastquest !== name) {
-				this.lastquest = name;
-				sendNotify(`The Tale Extended - ${storage.heroName}`, {
-					tag: 'autoquest',
-					body: `Сделан выбор! \n— ${name}`,
-					addTime: true,
-					icon: `${window.extPath}img/quest/caravan.png`,
-				});
-			}
-
-			if (!settingsValues.autoquest) {
-				return;
-			}
-			// const csrf = document.head.innerHTML.match(/("X-CSRFToken")(.*)(".*")/, 'g')[3].replace(/"/g, '');
-			$.ajax({
-				url: `/game/quests/api/choose?api_version=1.0&api_client=${window.API_CLIENT}&option_uid=${encodeURIComponent(uid)}`,
-				dataType: 'json',
-				type: 'post',
-				// beforeSend: function(xhr) {
-				// 	xhr.setRequestHeader('X-CSRFToken', csrf);
-				// },
-				data: {},
+	private chooseQuest(uid: string, name: string, settingsValues: SettingsValues) {
+		if (settingsValues.autoquestNotify && this.lastquest !== name) {
+			this.lastquest = name;
+			sendNotify(`The Tale Extended - ${storage.heroName}`, {
+				tag: 'autoquest',
+				body: `Сделан выбор! \n— ${name}`,
+				addTime: true,
+				icon: `${window.extPath}img/quest/caravan.png`,
 			});
 		}
+
+		if (!settingsValues.autoquest) {
+			return;
+		}
+		// const csrf = document.head.innerHTML.match(/("X-CSRFToken")(.*)(".*")/, 'g')[3].replace(/"/g, '');
+		$.ajax({
+			url: `/game/quests/api/choose?api_version=1.0&api_client=${window.API_CLIENT}&option_uid=${encodeURIComponent(uid)}`,
+			dataType: 'json',
+			type: 'post',
+			// beforeSend: function(xhr) {
+			// 	xhr.setRequestHeader('X-CSRFToken', csrf);
+			// },
+			data: {},
+		});
 	}
 }

source/js/notifications/sendNotify.ts

@@ -1,11 +1,11 @@
 $('body').one('click', request);
 
 function request() {
-	if ((Notification as any).permission.toLowerCase() !== 'granted') {
-		Notification.requestPermission(permission => {
-			if (permission !== 'granted') return false;
-			const notify = new Notification('Thanks for letting notify you');
-			return !!notify;
+	const Notify = Notification as any;
+	if (Notify.permission.toLowerCase() !== 'granted') {
+		Notify.requestPermission().then((permission: string) => {
+			if (permission !== 'granted') return;
+			Notify('Thanks for letting notify you');
 		});
 	}
 }

source/js/tabs/settings/Settings.ts

@@ -221,7 +221,7 @@ export default class Settings {
 			const name = st.name as void|keyof SettingsValues;
 			if (name) {
 				childs.push(name);
-				const storedValue = this.settingsValues[name];
+				const storedValue = this.settingsValues[name] as string | boolean | undefined;
 				this.settingsValues[name] = typeof storedValue === 'undefined' ? st.value : storedValue;
 			}
 

source/js/tabs/stats/Stats.ts

@@ -6,6 +6,35 @@ import {PHRASE_ID_TO_TEXT} from '../../utils/const/texts';
 import {PHRASE_NUMBER_KEYS} from '../../utils/const/vars';
 import ICONS from '../../utils/const/icons';
 
+const statsList = [
+	1, // 'Журнал: Стан'
+	11, // Журнал: Периодический урон огнём
+	12, // Журнал: Периодический урон ядом
+	280000, // Журнал: Пиромания
+	280001, // Журнал: Пиромания (промах)
+	280002, // Журнал: Контроль
+	280003, // Журнал: Удар
+	280004, // Журнал: Удар (Промах)
+	280005, // Журнал: Последний шанс
+	280006, // Журнал: Ярость
+	280007, // Журнал: Ядовитость
+	280008, // Журнал: Регенерация
+	280009, // Журнал: Ошеломление
+	280010, // Журнал: Ошеломление (промах)
+	280011, // Журнал: Дезориентация
+	280012, // Журнал: Ускорение
+	280013, // Журнал: Сильный удар
+	280014, // Журнал: Сильный удар (промах)
+	280015, // Журнал: Вампиризм
+	280016, // Журнал: Вампиризм (промах)
+	280017, // Журнал: Герой лечит спутника
+	280018, // Журнал: Безрассудная атака
+	280019, // Журнал: Безрассудная атака (промах)
+	580003, // Журнал: спутник защитил своего владельца от удара
+	580004, // Журнал: спутник защитил своего владельца от удара, но получил рану
+];
+
+
 type StatsByPhraseAndOwner = PhraseMeta & {
 	phraseId: number,
 	count: number,
@@ -160,31 +189,3 @@ export default class Stats {
 	}
 }
 
-
-const statsList = [
-	1, // 'Журнал: Стан'
-	11, // Журнал: Периодический урон огнём
-	12, // Журнал: Периодический урон ядом
-	280000, // Журнал: Пиромания
-	280001, // Журнал: Пиромания (промах)
-	280002, // Журнал: Контроль
-	280003, // Журнал: Удар
-	280004, // Журнал: Удар (Промах)
-	280005, // Журнал: Последний шанс
-	280006, // Журнал: Ярость
-	280007, // Журнал: Ядовитость
-	280008, // Журнал: Регенерация
-	280009, // Журнал: Ошеломление
-	280010, // Журнал: Ошеломление (промах)
-	280011, // Журнал: Дезориентация
-	280012, // Журнал: Ускорение
-	280013, // Журнал: Сильный удар
-	280014, // Журнал: Сильный удар (промах)
-	280015, // Журнал: Вампиризм
-	280016, // Журнал: Вампиризм (промах)
-	280017, // Журнал: Герой лечит спутника
-	280018, // Журнал: Безрассудная атака
-	280019, // Журнал: Безрассудная атака (промах)
-	580003, // Журнал: спутник защитил своего владельца от удара
-	580004, // Журнал: спутник защитил своего владельца от удара, но получил рану
-];

source/js/tracking/Tracking.ts

@@ -44,16 +44,17 @@ export default class Tracking {
 			messageRaw[MSG.TimeSting],
 			messageRaw[MSG.PhraseSting],
 			messageRaw[MSG.PhraseId],
-			Tracking.convertPhraseDataFromRaw( messageRaw[MSG.PhraseData]), /*phraseData*/
+			Tracking.convertPhraseDataFromRaw(messageRaw[MSG.PhraseData]), /*phraseData*/
 			Tracking.getMeta(messageRaw), /*phraseMeta*/
 		] as Message;
 	}
 
-	static convertPhraseDataFromRaw(phraseDataRaw: PhraseDataRaw): PhraseData {
+	static convertPhraseDataFromRaw(phraseDataRaw: Partial<PhraseDataRaw>): PhraseData {
 		const phraseData = Object.assign({}, phraseDataRaw) as PhraseData;
 		for (let i = 0; i < PHRASE_NUMBER_KEYS.length; i++) {
 			const key = PHRASE_NUMBER_KEYS[i];
-			if (typeof phraseDataRaw[key] === 'string') phraseData[key] = +phraseDataRaw[key];
+			const value = phraseDataRaw[key];
+			if (typeof value === 'string') phraseData[key] = +value;
 		}
 		return phraseData;
 	}

tsconfig.json

@@ -5,14 +5,13 @@
     "noImplicitAny": true,
     "rootDir": "source",
     "moduleResolution": "node",
-    "module": "es6",
-    "allowJs": true
+    "module": "es6"
   },
   "files": [
     "source/js/typings.d.ts"
   ],
   "include": [
-    "src/**/*.ts"
+    "source/**/*.ts"
   ],
   "exclude": [
     "node_modules/**"