@posthog/convex 2.1.2 → 2.1.3

package/dist/component/version.d.ts

@@ -1,2 +1,2 @@
-export declare const version = "2.1.2";
+export declare const version = "2.1.3";
 //# sourceMappingURL=version.d.ts.map

package/dist/component/version.js

@@ -1,2 +1,2 @@
-export const version = '2.1.2';
+export const version = '2.1.3';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -11,7 +11,7 @@
     "url": "https://github.com/PostHog/posthog-js/issues"
   },
   "author": "PostHog Inc.",
-  "version": "2.1.2",
+  "version": "2.1.3",
   "license": "MIT",
   "keywords": [
     "convex",
@@ -48,8 +48,8 @@
   "types": "./dist/client/index.d.ts",
   "module": "./dist/client/index.js",
   "dependencies": {
-    "@posthog/core": "^1.48.11",
-    "posthog-node": "^5.51.2"
+    "@posthog/core": "^1.48.12",
+    "posthog-node": "^5.51.3"
   },
   "devDependencies": {
     "@edge-runtime/vm": "^5.0.0",

package/src/client/feature-flags/evaluator.test.ts

@@ -200,7 +200,7 @@ describe('LocalFeatureFlagEvaluator', () => {
     expect(await evaluator.getFeatureFlag('not-pro', 'user', {}, { plan: 'free' })).toBe(true)
   })
 
-  test('is_not_set condition resolves locally without forcing inconclusive', async () => {
+  test('is_not_set condition is inconclusive when the property is omitted', async () => {
     const evaluator = new LocalFeatureFlagEvaluator(
       definitions([
         makeFlag('only-anon', {
@@ -215,9 +215,7 @@ describe('LocalFeatureFlagEvaluator', () => {
         }),
       ])
     )
-    // Key absent → property is_not_set is true → flag matches.
-    expect(await evaluator.getFeatureFlag('only-anon', 'user', {}, {})).toBe(true)
-    // Key present → property IS set → flag does not match.
+    expect(await evaluator.getFeatureFlag('only-anon', 'user', {}, {})).toBeUndefined()
     expect(await evaluator.getFeatureFlag('only-anon', 'user', {}, { email: 'a@b.com' })).toBe(false)
   })
 

package/src/client/feature-flags/match-property.test.ts

@@ -37,34 +37,22 @@ describe('matchProperty — numeric comparisons', () => {
   })
 })
 
-describe('matchProperty — is_not_set', () => {
-  test('returns true when the property is absent', () => {
-    expect(matchProperty({ key: 'missing', value: 'whatever', operator: 'is_not_set' }, {})).toBe(true)
-  })
-
-  test('returns false when the property is present', () => {
-    expect(matchProperty({ key: 'plan', value: 'whatever', operator: 'is_not_set' }, { plan: 'pro' })).toBe(false)
-  })
-
-  test('treats null-valued property as still set (returns false)', () => {
-    // `null` counts as present in propertyValues; only genuinely missing keys read as "not set".
-    expect(matchProperty({ key: 'plan', value: 'whatever', operator: 'is_not_set' }, { plan: null })).toBe(false)
-  })
-})
-
-describe('matchProperty — is_set', () => {
-  test('returns true when the property is present with a non-null value', () => {
-    expect(matchProperty({ key: 'plan', value: '', operator: 'is_set' }, { plan: 'pro' })).toBe(true)
-  })
-
-  test('returns true when the property is present with a null value', () => {
-    // `is_set` is about key presence, not value. Pre-fix, the null guard short-circuited and
-    // returned false here.
-    expect(matchProperty({ key: 'plan', value: '', operator: 'is_set' }, { plan: null })).toBe(true)
+describe('matchProperty — presence operators', () => {
+  test.each([
+    ['null', null],
+    ['undefined', undefined],
+    ['false', false],
+    ['zero', 0],
+    ['empty string', ''],
+    ['empty array', []],
+    ['empty object', {}],
+  ])('treats present %s as set', (_, value) => {
+    expect(matchProperty({ key: 'key', value: '', operator: 'is_set' }, { key: value })).toBe(true)
+    expect(matchProperty({ key: 'key', value: '', operator: 'is_not_set' }, { key: value })).toBe(false)
   })
 
-  test('throws InconclusiveMatchError when the property is absent', () => {
-    expect(() => matchProperty({ key: 'plan', value: '', operator: 'is_set' }, {})).toThrow(InconclusiveMatchError)
+  test.each(['is_set', 'is_not_set'])('%s is inconclusive when the property is omitted', (operator) => {
+    expect(() => matchProperty({ key: 'key', value: '', operator }, {})).toThrow(InconclusiveMatchError)
   })
 })
 
@@ -152,7 +140,7 @@ describe('matchProperty — SemVer compatibility policy', () => {
 })
 
 describe('matchProperty — error cases', () => {
-  test('throws InconclusiveMatchError when key is absent for non-is_not_set operators', () => {
+  test('throws InconclusiveMatchError when key is absent for other operators', () => {
     expect(() => matchProperty(prop('exact', 'x'), {})).toThrow(InconclusiveMatchError)
   })
 })

package/src/component/version.ts

@@ -1 +1 @@
-export const version = '2.1.2'
+export const version = '2.1.3'