From 45d98289254495fae08c6b764138319247ce1261 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 27 Apr 2026 20:24:05 +0000
Subject: [PATCH 1/2] Initial plan


From 68c92783fde043d44ba1ee5e6fbd1389e2d01a31 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 27 Apr 2026 20:27:56 +0000
Subject: [PATCH 2/2] fix: correct tryCatch handler signature and make
 flow-typed install non-fatal

- R/shinyModule.R: fix `error = function()` -> `error = function(e)` in
  `test_shiny_version()`. The handler must accept the condition argument
  passed by tryCatch, otherwise R throws "unused argument (cond)" when
  shiny is unavailable (e.g. in check-depends-only mode).

- bin/postinstall.sh: append `|| true` to `flow-typed install` so that
  transient GitHub API failures during cache fetch don't abort the
  postinstall script and block the routine CI job.

Agent-Logs-Url: https://github.com/rstudio/reactlog/sessions/b3ba1985-0e25-4de9-9e09-23b064266fe3

Co-authored-by: karangattu <4220325+karangattu@users.noreply.github.com>
---
 R/shinyModule.R    | 2 +-
 bin/postinstall.sh | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/R/shinyModule.R b/R/shinyModule.R
index 830dd1f..2e9af29 100644
--- a/R/shinyModule.R
+++ b/R/shinyModule.R
@@ -135,7 +135,7 @@ shiny_version_required <- function() {
 test_shiny_version <- function() {
   tryCatch({
     utils::packageVersion("shiny") >= shiny_version_required()
-  }, error = function() {
+  }, error = function(e) {
     # package not found
     FALSE
   })
diff --git a/bin/postinstall.sh b/bin/postinstall.sh
index 6c14877..37dab33 100755
--- a/bin/postinstall.sh
+++ b/bin/postinstall.sh
@@ -2,6 +2,6 @@
 
 # do not create stubs from missing libraries
 # ignore dev dependencies
-yarn flow-typed install --skip true --ignoreDeps dev
+yarn flow-typed install --skip true --ignoreDeps dev || true
 
 yarn build-only