open-wc / custom-elements-manifest

Custom Elements Manifest is a file format that describes custom elements in your project.
https://custom-elements-manifest.open-wc.org/
226 stars 37 forks source link

[Analyzer] `InitializeParams` type missing from `index.d.ts` #237

Closed xenobytezero closed 4 months ago

xenobytezero commented 4 months ago

Checklist

I'm importing the @custom-elements-manifest/analyzer types to help writing a plugin, and tsc is complaining the following

../../../node_modules/.pnpm/@custom-elements-manifest+analyzer@0.9.2/node_modules/@custom-elements-manifest/analyzer/index.d.ts:95:23 - error TS2304: Cannot find name 'InitializeParams'.

95   initialize?(params: InitializeParams): void;
                         ~~~~~~~~~~~~~~~~

Found 1 error in ../../../node_modules/.pnpm/@custom-elements-manifest+analyzer@0.9.2/node_modules/@custom-elements-manifest/analyzer/index.d.ts:95

Sure enough, this type is missing, and is flagged as an error when opening the file up in VSCode.

I can work around the issue using skipLibCheck but I am assuming this omission is a mistake, and thought I should flag it.

thepassle commented 4 months ago

Ah yes, we recently added this hook but I guess forgot to update the .d.ts file. Would you mind creating a PR? :)

xenobytezero commented 4 months ago

Will do.

xenobytezero commented 4 months ago

Done!

thepassle commented 4 months ago

Released in 0.9.3