✨ Add support for set_variable.key_up_value (#131)

https://karabiner-elements.pqrs.org/docs/json/complex-modifications-manipulator-definition/to/set-variable/
evan-liu · Jul 9, 2024 · 32f15ce · 32f15ce
1 parent ee710bc
commit 32f15ce
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/src/config/manipulator.ts b/src/config/manipulator.ts
@@ -153,8 +153,12 @@ export class BasicManipulatorBuilder implements ManipulatorBuilder {
   }
 
   /** Map to setting a variable */
-  toVar(name: string, value: ToVariable['value'] = 1): this {
-    this.addToEvent(toSetVar(name, value))
+  toVar(
+    name: string,
+    value: ToVariable['value'] = 1,
+    key_up_value?: ToVariable['key_up_value'],
+  ): this {
+    this.addToEvent(toSetVar(name, value, key_up_value))
     return this
   }
 

diff --git a/src/config/to.test.ts b/src/config/to.test.ts
@@ -35,6 +35,9 @@ test('setVar()', () => {
   expect(toSetVar('test', true)).toEqual({
     set_variable: { name: 'test', value: true },
   })
+  expect(toSetVar('test', true, false)).toEqual({
+    set_variable: { name: 'test', value: true, key_up_value: false },
+  })
 })
 
 test('toPlaySound()', () => {

diff --git a/src/config/to.ts b/src/config/to.ts
@@ -156,8 +156,9 @@ export function toInputSource(inputSource: ToInputSource): ToEvent {
 export function toSetVar(
   name: string,
   value: ToVariable['value'] = 1,
+  key_up_value?: ToVariable['key_up_value'],
 ): ToEvent {
-  return { set_variable: { name, value } }
+  return { set_variable: { name, value, key_up_value } }
 }
 
 /** Create ToEvent with set_notification_message */

diff --git a/src/karabiner/karabiner-config.ts b/src/karabiner/karabiner-config.ts
@@ -78,6 +78,7 @@ export type ToInputSource = {
 export type ToVariable = {
   name: string
   value: number | boolean | string
+  key_up_value?: number | boolean | string
 }
 
 /** @see https://karabiner-elements.pqrs.org/docs/json/complex-modifications-manipulator-definition/to/mouse-key/ */