feat: Component best practices- async flags check

src/linter/LinterContext.ts

@@ -1,5 +1,6 @@
 import {AbstractAdapter, AbstractReader} from "@ui5/fs";
 import {createReader} from "@ui5/fs/resourceFactory";
+import {resolveLinks} from "../formatter/lib/resolveLinks.js";
 
 export type FilePath = string; // Platform-dependent path
 export type ResourcePath = string; // Always POSIX
@@ -156,6 +157,10 @@ export default class LinterContext {
 	}
 
 	addLintingMessage(resourcePath: ResourcePath, message: LintMessage) {
+		if (message.messageDetails) {
+			message.messageDetails = resolveLinks(message.messageDetails);
+		}
+
 		this.getLintingMessages(resourcePath).push(message);
 	}
 

src/linter/linter.ts

@@ -87,16 +87,19 @@ export async function lintProject({
 export async function lintFile({
 	rootDir, pathsToLint, namespace, reportCoverage, includeMessageDetails,
 }: LinterOptions): Promise<LintResult[]> {
-	const reader = createReader({
-		fsBasePath: rootDir,
-		virBasePath: "/",
-	});
 	let resolvedFilePaths;
+	let virBasePath = "";
 	if (pathsToLint?.length) {
 		const absoluteFilePaths = resolveFilePaths(rootDir, pathsToLint);
 		resolvedFilePaths = transformFilePathsToVirtualPaths(
 			absoluteFilePaths, rootDir, "/", rootDir);
+		// Extract the (virtual) path from the filename
+		virBasePath = resolvedFilePaths[0].split("/").slice(0, -1).join("/");
 	}
+	const reader = createReader({
+		fsBasePath: rootDir,
+		virBasePath: `${virBasePath}/`,
+	});
 
 	const res = await lint(reader, {
 		rootDir,

src/linter/ui5Types/SourceFileLinter.ts

@@ -1,6 +1,7 @@
 import ts, {Identifier} from "typescript";
 import SourceFileReporter from "./SourceFileReporter.js";
 import LinterContext, {ResourcePath, CoverageCategory, LintMessageSeverity} from "../LinterContext.js";
+import analyzeComponentJson from "./bestPractices.js";
 
 interface DeprecationInfo {
 	symbol: ts.Symbol;
@@ -16,11 +17,12 @@ export default class SourceFileLinter {
 	#boundVisitNode: (node: ts.Node) => void;
 	#reportCoverage: boolean;
 	#messageDetails: boolean;
+	#manifestContent: string | undefined;
 
 	constructor(
 		context: LinterContext, resourcePath: ResourcePath, sourceFile: ts.SourceFile, sourceMap: string | undefined,
 		checker: ts.TypeChecker, reportCoverage: boolean | undefined = false,
-		messageDetails: boolean | undefined = false
+		messageDetails: boolean | undefined = false, manifestContent?: string | undefined
 	) {
 		this.#resourcePath = resourcePath;
 		this.#sourceFile = sourceFile;
@@ -30,6 +32,7 @@ export default class SourceFileLinter {
 		this.#boundVisitNode = this.visitNode.bind(this);
 		this.#reportCoverage = reportCoverage;
 		this.#messageDetails = messageDetails;
+		this.#manifestContent = manifestContent;
 	}
 
 	// eslint-disable-next-line @typescript-eslint/require-await
@@ -67,6 +70,17 @@ export default class SourceFileLinter {
 				node as (ts.PropertyAccessExpression | ts.ElementAccessExpression)); // Check for deprecation
 		} else if (node.kind === ts.SyntaxKind.ImportDeclaration) {
 			this.analyzeImportDeclaration(node as ts.ImportDeclaration); // Check for deprecation
+		} else if (node.kind === ts.SyntaxKind.ExpressionWithTypeArguments &&
+		ts.isSourceFile(this.#sourceFile) &&
+		this.#sourceFile.fileName.endsWith("/Component.js")) {
+			analyzeComponentJson({
+				node: node as ts.ExpressionWithTypeArguments,
+				manifestContent: this.#manifestContent,
+				resourcePath: this.#resourcePath,
+				reporter: this.#reporter,
+				context: this.#context,
+				checker: this.#checker,
+			});
 		}
 
 		// Traverse the whole AST from top to bottom

src/linter/ui5Types/TypeLinter.ts

@@ -4,7 +4,7 @@ import SourceFileLinter from "./SourceFileLinter.js";
 import {taskStart} from "../../util/perf.js";
 import {getLogger} from "@ui5/logger";
 import LinterContext, {LinterParameters} from "../LinterContext.js";
-// import {Project} from "@ui5/project";
+import path from "node:path/posix";
 import {AbstractAdapter} from "@ui5/fs";
 import {createAdapter, createResource} from "@ui5/fs/resourceFactory";
 
@@ -103,12 +103,20 @@ export default class TypeChecker {
 				if (!sourceMap) {
 					log.verbose(`Failed to get source map for ${sourceFile.fileName}`);
 				}
+				let manifestContent;
+				if (sourceFile.fileName.endsWith("/Component.js")) {
+					const res = await this.#workspace.byPath(path.dirname(sourceFile.fileName) + "/manifest.json");
+					if (res) {
+						manifestContent = await res.getString();
+					}
+				}
 				const linterDone = taskStart("Type-check resource", sourceFile.fileName, true);
 				const linter = new SourceFileLinter(
 					this.#context,
 					sourceFile.fileName, sourceFile,
 					sourceMap,
-					checker, reportCoverage, messageDetails
+					checker, reportCoverage, messageDetails,
+					manifestContent
 				);
 				await linter.lint();
 				linterDone();