Treat indirect ifdef names as uncached

Finding: [P2] Track env vars behind indirect ifdef names — /Users/tomasz/github/t-kalinowski/quickr/R/aaa-utils.R:799-800

Resolution: indirect ifdef and ifndef names now match the dynamic-input detector, so probes guarded by those conditionals bypass successful-result caching. Added a regression test for direct FC assignments behind an indirect ifdef name.

Author: t-kalinowski

R/aaa-utils.R

@@ -768,12 +768,14 @@ quickr_text_has_makevars_uncached_function <- function(text) {
   substitution_pattern <- "\\$\\([A-Za-z_][A-Za-z0-9_.-]*:[^)]*\\)|\\$\\{[A-Za-z_][A-Za-z0-9_.-]*:[^}]*\\}"
   shell_assignment_pattern <- "^[[:space:]]*(?:(?:export|override|private)[[:space:]]+)*[A-Za-z_][A-Za-z0-9_.-]*[[:space:]]*!="
   indirect_ref_pattern <- "\\$[({][^)}]*\\$[({]"
+  indirect_conditional_pattern <- "^[[:space:]]*(?:else[[:space:]]+)?ifn?def[[:space:]]+\\$[({]"
   any(grepl(
     paste(
       function_pattern,
       substitution_pattern,
       shell_assignment_pattern,
       indirect_ref_pattern,
+      indirect_conditional_pattern,
       sep = "|"
     ),
     text,

tests/testthat/test-compiler-cache.R

@@ -1157,6 +1157,46 @@ test_that("quickr_cached_r_cmd_config_value expands ifdef variable names", {
   expect_equal(calls, 2L)
 })
 
+test_that("quickr_cached_r_cmd_config_value retries direct assignments behind indirect ifdef names", {
+  cache <- new.env(parent = emptyenv())
+  makevars <- withr::local_tempfile()
+  writeLines(
+    c(
+      "FLAG_NAME = USE_A",
+      "ifdef $(FLAG_NAME)",
+      "FC=gfortran",
+      "else",
+      "FC=flang",
+      "endif"
+    ),
+    makevars
+  )
+  calls <- 0L
+  local_mocked_bindings(
+    quickr_r_cmd_config_probe = function(name) {
+      calls <<- calls + 1L
+      list(value = paste0("value-", calls), ok = TRUE)
+    },
+    .package = "quickr"
+  )
+  withr::local_envvar(c(
+    R_MAKEVARS_USER = makevars,
+    USE_A = "1"
+  ))
+
+  expect_identical(
+    quickr_cached_r_cmd_config_value("FC", cache = cache),
+    "value-1"
+  )
+
+  withr::local_envvar(USE_A = NA)
+  expect_identical(
+    quickr_cached_r_cmd_config_value("FC", cache = cache),
+    "value-2"
+  )
+  expect_equal(calls, 2L)
+})
+
 test_that("quickr_cached_r_cmd_config_value retries Makevars with wildcard assignments", {
   cache <- new.env(parent = emptyenv())
   root <- withr::local_tempdir()