Unused type imports Parsable and ParsableFactory are causing build fail

[sher]

Kiota generated request builder imports several types from abstractions package.

import { type BaseRequestBuilder, type Parsable, type ParsableFactory, type RequestConfiguration, type RequestInformation, type RequestsMetadata } from '@microsoft/kiota-abstractions';

Among them Parsable and ParsableFactory are never referenced in the generated code.
This is failing to build in projects where tsconfig.json has noUnusedLocals: true such as default Vitejs tsconfig.json.

[baywet]

Hi @sher
Thanks for using kiota and for reporting this.
Kiota already emits tslint and eslint disable "comments" so linter do not run on the generated code, reducing noise.
Now, because this is a TypeScript check (doesn't require any additional linter), it's still triggering.
I suggest we add an additional @ts-nocheck comment atop the files. documentation

The doc comments are being written here

kiota/src/Kiota.Builder/Writers/TypeScript/TypeScriptConventionService.cs

Line 16 in 39b3cde

writer.WriteLine("/* tslint:disable */");

And things are tested at the following places

kiota/tests/Kiota.Builder.Tests/Writers/TypeScript/CodeFileWriterTests.cs

Line 60 in 39b3cde

Assert.Contains("/* tslint:disable */", result);

kiota/tests/Kiota.Builder.Tests/Writers/TypeScript/CodeFunctionWriterTests.cs

Line 48 in 39b3cde

Assert.DoesNotContain("/* tslint:disable */", result);

kiota/tests/Kiota.Builder.Tests/Writers/TypeScript/CodeClassDeclarationWriterTests.cs

Line 44 in 39b3cde

Assert.Contains("/* eslint-disable */", result);

kiota/tests/Kiota.Builder.Tests/Writers/TypeScript/CodeInterfaceDeclarationWriterTests .cs

Line 38 in 39b3cde

Assert.Contains("/* eslint-disable */", result);

With all that information in mind, would you be willing to submit a pull request to address this?

[baywet]

Actually, let me rephrase that suggestion, this is probably too big of a suppression because semantic checks will also be disbaled. This will make it much harder to catch any regression at build time.
Instead adding a // @ts-ignore directive before import statements would probably be safer

kiota/src/Kiota.Builder/Writers/TypeScript/CodeUsingWriter.cs

Line 29 in 39b3cde

writer.WriteLine($"import {{ {codeUsing.Select(static x => GetAliasedSymbol(x.Symbol, x.Alias, x.ShouldUseTypeImport)).Distinct().OrderBy(static x => x).Aggregate(static (x, y) => x + ", " + y)} }} from '{codeUsing.Key}';");

[sher]

Hi @baywet
Thank you for the feedback. Submitted a PR #4398 PTAL.