refactor: Replace _impl alias with explicit wrapper in R/layout.R

[Copilot]

Replaced direct _impl function aliasing with an explicit wrapper function to improve code clarity and maintainability.

Changes

• Converted align_layout <- layout_align_impl to an explicit wrapper function that forwards arguments individually using arg = arg notation

Before

align_layout <- layout_align_impl

After

align_layout <- function(graph, layout) {
  layout_align_impl(
    graph = graph,
    layout = layout
  )
}

This makes the function signature explicit and follows the pattern established elsewhere in the codebase for wrapping implementation functions.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Replace _impl aliases in R/layout.R</issue_title>
<issue_description>Goal
Review the code in R/layout.R and replace all direct _impl function aliases with explicit wrapper functions.

Functions to update:

• align_layout <- layout_align_impl

Instructions

1. Review the code to understand the function signatures
2. Create explicit wrapper functions that forward to the _impl functions
3. Forward each argument individually and explicitly using arg = arg notation
4. Do not use ... unless the _impl function itself uses ...

Pattern
Replace direct aliasing like:

func_name <- func_name_impl

With an explicit wrapper:

func_name <- function(arg1, arg2, ...) {
  func_name_impl(
    arg1 = arg1,
    arg2 = arg2,
    ...
  )
}

File
R/layout.R</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Replace _impl aliases in R/layout.R #2351

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.