fix: do not return false immediately with negative operators

[thucpn]

I think != operator should able to check whether the key is existed or not
Eg:

{
  "key": "private",
  "value": "true",
  "operator": "!="
}
-> Should return true when metadata[private] = "false" or metadata[private] = undefined.

So I think we can change from "if the metadata key is not present, always return false"

to "for all operators except != and nin, if the metadata key is not present, return false"

I also add some tests when filtering nodes have optional key

[changeset-bot]

⚠️ No Changeset found

Latest commit: c1e82bd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
llama-index-ts-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 19, 2024 5:22am

[pkg-pr-new]

Run & review this pull request in StackBlitz Codeflow.

commit: c1e82bd

@llamaindex/autotool

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/autotool@1058

@llamaindex/cloud

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/cloud@1058

@llamaindex/community

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/community@1058

@llamaindex/core

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/core@1058

@llamaindex/env

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/env@1058

@llamaindex/experimental

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/experimental@1058

llamaindex

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/llamaindex@1058

@llamaindex/wasm-tools

npm i https://pkg.pr.new/run-llama/LlamaIndexTS/@llamaindex/wasm-tools@1058

---

templates

• default

[thucpn]

@marcusschiesser I think we can close this PR because is_empty has already added in PR #1107