Fix kaleido version detection for CI compatibility

cpsievert · claude · cpsievert · commit 275422bba62e · 2026-01-23T11:24:20.000-06:00
Check for kaleido v1 by looking for the v1-specific `write_fig_sync`
function rather than checking for absence of `scopes`. This handles
edge cases where kaleido v0.2.1 may have broken `scopes` attribute
on certain Python versions (e.g., Python 3.14).

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/R/kaleido.R b/R/kaleido.R
@@ -97,10 +97,11 @@ kaleido <- function(...) {
     }
   )
   
-  res <- if (reticulate::py_has_attr(kaleido, "scopes")) {
-    legacyKaleidoScope(kaleido)
-  } else {
+  # Check for kaleido v1 by looking for v1-specific function
+  res <- if (reticulate::py_has_attr(kaleido, "write_fig_sync")) {
     newKaleidoScope(kaleido)
+  } else {
+    legacyKaleidoScope(kaleido)
   }
   
   class(res) <- "kaleidoScope"

Original file line number	Diff line number	Diff line change
`@@ -97,10 +97,11 @@ kaleido <- function(...) {`
`97`	`97`	`}`
`98`	`98`	`)`
`99`	`99`
`100`		`- res <- if (reticulate::py_has_attr(kaleido, "scopes")) {`
`101`		`- legacyKaleidoScope(kaleido)`
`102`		`- } else {`
	`100`	`+ # Check for kaleido v1 by looking for v1-specific function`
	`101`	`+ res <- if (reticulate::py_has_attr(kaleido, "write_fig_sync")) {`
`103`	`102`	`newKaleidoScope(kaleido)`
	`103`	`+ } else {`
	`104`	`+ legacyKaleidoScope(kaleido)`
`104`	`105`	`}`
`105`	`106`
`106`	`107`	`class(res) <- "kaleidoScope"`