Do not use index.ts

[aeschli]

https://github.com/Microsoft/vscode/blob/8a221b0bf4fb7d53703e7a45982d229c19fcadd0/src/vs/workbench/contrib/chat/common/promptSyntax/contributions/languageFeatures/index.ts
https://github.com/Microsoft/vscode/blob/8a221b0bf4fb7d53703e7a45982d229c19fcadd0/src/vs/workbench/contrib/chat/common/promptSyntax/contributions/index.ts

In the VS Code code base we have avoided using index.ts

• it will lead to a large number of 'index.ts' in the project, making the navigation and understanding more difficult
• if you want an entry point to a contribution or package, name it as your package. e.g. folding.contribution.ts
• index.ts. is more common in CommonJS where you can require a folder which then is resolved to the index file in that folder.
  The VS Code code base is ESM, so this might be confusing

[bpasero]

@legomushroom friendly ping, this is similar to other issues I had filed where it seems against typical patterns we use elsewhere.

We typically put the "API" shape of a component into a <component name>.ts file. As such, a good example for this is src/vs/workbench/contrib/files/common/files.ts:

vscode/src/vs/workbench/contrib/files/common/files.ts

Lines 26 to 29 in 2be96ab

	/**
	* Explorer viewlet id.
	*/
	export const VIEWLET_ID = 'workbench.view.explorer';

This exports things other components can use as needed.

[aeschli]

Fixed