Build error fixed (TS)

[siddiqss]

Error: "Property 'INDEX_NOT_EXIST' does not exist on type 'typeof ErrorCode'. Did you mean 'IndexNotExist'?ts(2551)
error.d.ts(39, 5): 'IndexNotExist' is declared here."
in "packages/components/nodes/vectorstores/Milvus/Milvus_Upsert.ts".

Fixed: Replaced INDEX_NOT_EXIST with IndexNotExist

issue described here: #1135

[automaton82]

Yea I've got the same build error, this change appears to fix it.

[HenryHengZJ]

Its showing error on my side:

[automaton82]

@HenryHengZJ I think your packages may be out of date or something else is going wrong. Even with this fixed the build is broken and I'm not sure why. I suspect dependency issue.

[automaton82]

Also if you look at the Milvus SDK source it has changed:

https://github.com/milvus-io/milvus-sdk-node/blob/main/milvus/const/error.ts

Yours shouldn't build with that const value.

[siddiqss]

@automaton82 What error do you get? My build was successful after this fix.

[automaton82]

@siddiqss #1139 this one

[siddiqss]

@automaton82 did you install all dependencies with yarn?

[automaton82]

@siddiqss yes, fresh clone and followed dev steps including yarn install.

I've had this working for weeks, it's only yesterday fresh installs started failing. I tried Windows, Ubuntu 22 and the Docker build. I will try again this morning, maybe a package was broken last night and not today.

[siddiqss]

so maybe in the last pull requests someone installed a new package which conflicted with existing one.

[automaton82]

I don't think so because I reset the head back to the last public release (18937de), cleaned, deleted node_modules from all folders and re-installed and the same error occurred. I think a dependency has a problem, because I know I built that version 2 weeks ago without issue.

[automaton82]

@HenryHengZJ the automated tests are now failing with this same TS error: https://github.com/FlowiseAI/Flowise/actions/runs/6667186174/job/18120163971

[automaton82]

I closed my #1139 issue as it's no longer happening this morning. But this TS issue remains, and this pull request does fix it, so hopefully it can be merged.

[Initonedeb]

Hey [automaton82] I guess you were doing the change on Flowise\node_modules\flowise-components\dist\nodes\vectorstores\Milvus instead of packages/components/nodes/vectorstores/Milvus/Milvus_Upsert.ts?

[Initonedeb]

Allow me to elaborate how i made it work, i had the same issue you have [automaton82], 1) rename flowise folder to anything, 2) git clone repository, 3) cd to flowise, 4) go to packages/components/nodes/vectorstores/Milvus/Milvus_Upsert.ts? Replaced INDEX_NOT_EXIST with IndexNotExist on line 255, 5)Yarn Install 6)Yarn Build. Thats it now it will work.

[automaton82]

@Initonedeb Yes, I fixed the issue this PR fixes manually. But I had a separate build error which is now gone.

[HenryHengZJ]

ah okay, perfect, a fresh new install can see the error:

[HenryHengZJ]

thanks everyone!

[automaton82]

Thanks!