Build: Introduce stricter typechecking (#868)

Author: jackw

Diff for: packages/create-plugin/src/utils/utils.packageManager.ts

@@ -97,6 +97,7 @@ function getPackageManagerFromLockFile(): PackageManager | undefined {
     if (error instanceof Error) {
       throw error;
     }
+    return undefined;
   }
 }
 

Diff for: packages/create-plugin/src/utils/utils.templates.ts

@@ -3,7 +3,7 @@ import path from 'node:path';
 import fs from 'node:fs';
 import { mkdirp } from 'mkdirp';
 import createDebug from 'debug';
-import { directoryExists, filterOutCommonFiles, isFile, isFileStartingWith } from './utils.files.js';
+import { filterOutCommonFiles, isFile, isFileStartingWith } from './utils.files.js';
 import { renderHandlebarsTemplate } from './utils.handlebars.js';
 import { getPluginJson } from './utils.plugin.js';
 import {
@@ -17,7 +17,7 @@ import { TemplateData } from '../types.js';
 import { getPackageManagerInstallCmd, getPackageManagerWithFallback } from './utils.packageManager.js';
 import { getExportFileName } from '../utils/utils.files.js';
 import { getVersion } from './utils.version.js';
-import { FeatureFlags, getConfig } from './utils.config.js';
+import { getConfig } from './utils.config.js';
 
 const debug = createDebug('templates');
 

Diff for: packages/plugin-e2e/src/fixtures/isFeatureToggleEnabled.ts

@@ -3,7 +3,7 @@ import { PlaywrightArgs } from '../types';
 
 type FeatureToggleFixture = TestFixture<<T = object>(featureToggle: keyof T) => Promise<boolean>, PlaywrightArgs>;
 
-export const isFeatureToggleEnabled: FeatureToggleFixture = async ({ page, grafanaVersion }, use) => {
+export const isFeatureToggleEnabled: FeatureToggleFixture = async ({ page }, use) => {
   await use(async <T = object>(featureToggle: keyof T) => {
     const featureToggles: T = await page.evaluate('window.grafanaBootData.settings.featureToggles');
     return Boolean(featureToggles[featureToggle]);

Diff for: packages/plugin-e2e/src/models/pages/ExplorePage.ts

@@ -1,5 +1,5 @@
 import * as semver from 'semver';
-import { Locator, expect } from '@playwright/test';
+import { Locator } from '@playwright/test';
 import { NavigateOptions, PluginTestCtx, RequestOptions } from '../../types';
 import { DataSourcePicker } from '../components/DataSourcePicker';
 import { GrafanaPage } from './GrafanaPage';
@@ -8,10 +8,8 @@ import { Panel } from '../components/Panel';
 
 const TIME_SERIES_PANEL_SELECTOR_SUFFIX = 'Graph';
 const TABLE_PANEL_SELECTOR_SUFFIX = 'Table';
-const LOGS_PANEL_SELECTOR_SUFFIX = 'Logs';
 const TIME_SERIES_PANEL_TEXT = 'Graph Lines Bars Points';
 const TABLE_PANEL_TEXT = 'Table';
-const LOGS_PANEL_TEXT = 'Logs';
 
 export class ExplorePage extends GrafanaPage {
   datasource: DataSourcePicker;

Diff for: packages/plugin-e2e/src/models/pages/PluginConfigPage.ts

@@ -1,4 +1,3 @@
-import { Response as PlaywrightResponse } from '@playwright/test';
 import { PluginPageArgs, NavigateOptions, PluginTestCtx } from '../../types';
 import { GrafanaPage } from './GrafanaPage';
 

Diff for: tsconfig.base.json

@@ -2,6 +2,12 @@
   "$schema": "https://json.schemastore.org/tsconfig",
   "extends": ["@tsconfig/recommended/tsconfig.json", "@tsconfig/node20/tsconfig.json"],
   "compilerOptions": {
+    "allowUnusedLabels": false,
+    "allowUnreachableCode": false,
+    "noFallthroughCasesInSwitch": true,
+    "noImplicitReturns": true,
+    "noUnusedLocals": true,
+    "noUnusedParameters": true,
     "removeComments": true,
     "types": ["vitest/globals"]
   }