version 8.0.0 has incorrect typescript config

[jefbarn]

Could not find the type definitions:

import OdataQuery, { QueryOptions } from "odata-query";

TS7016: Could not find a declaration file for module odata-query.

Also I had to do this before:

const buildQuery = OdataQuery.default;

Is that fixed now?

[techniq]

Hey @jefbarn, @enure just reworked the build so OdataQuery.default should no longer be required, but looks like there is an issue with building types.

I'm a bit covered up but if someone can submit a PR I'll review / release.

[techniq]

I haven't been able to track down the root cause of this yet as building locally produces the index.d.ts file but it's no longer working when built by the release pipeline.

[enure]

@techniq this is my first guess: #134. & backgrounds the previous command, && only runs the following command if the previous succeeds. Even if it doesn't fix this issue, sequentially seems like the right way to run these commands.

[techniq]

Thanks @enure. Merged and released 8.0.2 but sadly it didn't include index.d.ts

Interesting, I'm not seeing it recreated locally now either when I test.

[techniq]

@enure awe, might be related to the outdated typescript dependency.

If you have any cycles, could you look at updating/resolving the issues (I know you mentioned you ran into some issues when you attempted in the vite PR).

[enure]

What do you think about trying #136? I mentioned env variables on the PR description, but testing it more locally, those seem unrelated. It does repro a fix for me locally.

[techniq]

@enure That was it, thanks!

@jefbarn fixed in 8.0.3

[jefbarn]

@techniq Thanks the types are working as they did before. Still have the problem with the default export (you're using
export { W as default } instead of export default W in the js code which aren't the same.) but that's a minor inconvenience.

[techniq]

@techniq Thanks the types are working as they did before. Still have the problem with the default export (you're using export { W as default } instead of export default W in the js code which aren't the same.) but that's a minor inconvenience.

Thanks for confirming. Interesting about the default export, as it's using export default function... in the source. I'm open for a PR if you see the solution.

[jefbarn]

@techniq
The only way I can get this to work currently is using

import OdataQuery, { QueryOptions } from "odata-query";
const buildQuery = OdataQuery as unknown as typeof OdataQuery.default;

but looking at the code, yeah I agree it should be working as is, so this is a bit of a head scratcher. I'll try to play around with the code and send a PR.

[satispunk]

Hello, has this issue been addressed?

I have also faced

TS2349: This expression is not callable.
Type
typeof import("C:/dev/npm/forsta/node_modules/odata-query/dist/index")
has no call signatures.