@tiuxo/opencode-openrgb 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.1] - 2026-11-10
+
+### Fixed
+
+- Opened one OpenRGB connection per location rather than one per machine, so a
+  single opencode server could run several controllers writing to the same
+  devices at once. The shared controller registry was module-level state, but
+  opencode gives each location its own module instance, making that scope
+  per-location rather than per-process. It now lives on a `globalThis` symbol,
+  which is the scope the hardware actually has.
+
+0.1.0 was staged but never released, so no published version carries this bug.
+
 ## [0.1.0] - 2026-11-10
 
 Initial release, targeting the opencode 2 plugin API.
@@ -39,5 +52,6 @@ Initial release, targeting the opencode 2 plugin API.
   plugin's own status colors, which indicates a previous run exited without
   restoring.
 
-[Unreleased]: https://git.tiuxo.com/brian/opencode-openrgb/compare/v0.1.0...HEAD
+[Unreleased]: https://git.tiuxo.com/brian/opencode-openrgb/compare/v0.1.1...HEAD
+[0.1.1]: https://git.tiuxo.com/brian/opencode-openrgb/releases/tag/v0.1.1
 [0.1.0]: https://git.tiuxo.com/brian/opencode-openrgb/releases/tag/v0.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiuxo/opencode-openrgb",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Ambient status lighting for opencode via OpenRGB. Mirrors session state onto your RGB hardware.",
   "type": "module",
   "license": "BSD-3-Clause",

package/src/index.ts

@@ -26,13 +26,27 @@ const LOG_PREFIX = "[opencode-openrgb]"
  * it. Instances share one controller and refcount it: the first caller builds
  * it, the last one to leave shuts it down. Shutting down on the first unload
  * instead would darken the lights for every location still running.
+ *
+ * The registry lives on `globalThis`, not in a module-level variable. opencode
+ * loads the plugin once per location and each load gets its own module
+ * instance, so module scope is per-location, not per-process: every location
+ * would build its own controller and open its own OpenRGB connection.
  */
-let shared: { controller: Controller; refs: number } | null = null
+const SHARED_KEY = Symbol.for("opencode-openrgb.shared")
+
+interface Shared {
+  controller: Controller
+  refs: number
+}
+
+type SharedHost = typeof globalThis & { [SHARED_KEY]?: Shared | null }
 
 function acquire(options: unknown): Controller | null {
-  if (shared) {
-    shared.refs++
-    return shared.controller
+  const host = globalThis as SharedHost
+  const existing = host[SHARED_KEY]
+  if (existing) {
+    existing.refs++
+    return existing.controller
   }
 
   const resolved = resolveOptions(options, (message) =>
@@ -41,17 +55,19 @@ function acquire(options: unknown): Controller | null {
   if (!resolved.enabled) return null
 
   const controller = new Controller(resolved)
-  shared = { controller, refs: 1 }
+  host[SHARED_KEY] = { controller, refs: 1 }
   controller.start()
   return controller
 }
 
 function release(): void {
+  const host = globalThis as SharedHost
+  const shared = host[SHARED_KEY]
   if (!shared) return
   shared.refs--
   if (shared.refs > 0) return
   shared.controller.shutdown()
-  shared = null
+  host[SHARED_KEY] = null
 }
 
 /*