rmarkdown expects list of dependencies to be unnamed

[cderv]

This property is used in HTML or LaTeX deps resolution to know when to be recursive.

This is caused by changed in cc80516 which introduced the name

Quite specific case encountered with Quarto processing at quarto-dev/quarto-cli#5093. This PR solves quarto-dev/quarto-cli#5093 (comment) issue

You could also remove the name completely inside list_latex_dep()

Regarding the convention, this is here in rmarkdown
https://github.com/rstudio/rmarkdown/blame/main/R/html_dependencies.R#L382-L400

has_dependencies <- function(knit_meta, class) {
  if (inherits(knit_meta, class))
    return(TRUE)
  if (is.list(knit_meta)) {
    for (dep in knit_meta) {
      if (is.null(names(dep))) { # <<- if unnamed we considered we should recurse
        if (has_dependencies(dep, class))
          return(TRUE)
      } else { # <<- if named then we check if it inherits the class, as a html / latex dependency would
        if (inherits(dep, class))
          return(TRUE)
      }
    }
  }
  FALSE
}

[davidgohel]

Thank you @cderv, much appreciated - you saved me hours