fix(auto): *Options.from now resolves repeated options to the last occurrence

[lbialy]

*Options.from now resolves repeated options to the last occurrence

[Copilot]

Pull request overview

Updates automation option merging so repeated options use the last occurrence.

Changes:

• Refactors option reducers to left-fold with last-value precedence.
• Preserves accumulation for environment maps.
• Reorders workspace shell options and adds regression tests.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
ShellTest.scala	Tests repeated environment keys.
OptionsTest.scala	Tests last-occurrence behavior across option types.
Workspace.scala	Reorders shell options by precedence.
Stack.scala	Updates stack option reducers.
shell.scala	Updates shell option merging.
LocalWorkspace.scala	Updates workspace option merging.
auto.scala	Updates login/logout option merging.

Suppressed comments (5)

auto/src/main/scala/besom/auto/internal/auto.scala:245

• This conversion removes the prior invalid-option guard, so LogoutOptions.from(List(null)) now leaks MatchError rather than AutoError. Add an explicit null case to preserve the existing failure behavior.

      opt match

auto/src/main/scala/besom/auto/internal/Stack.scala:1018

• The previous fallback converted an invalid/null up option to AutoError; this exhaustive match instead throws MatchError for UpOptions.from(List(null)). Retain the former error contract with an explicit null case.

      opt match

auto/src/main/scala/besom/auto/internal/Stack.scala:1175

• The previous fallback converted an invalid/null refresh option to AutoError; this exhaustive match instead throws MatchError for RefreshOptions.from(List(null)). Retain the former error contract with an explicit null case.

      opt match

auto/src/main/scala/besom/auto/internal/Stack.scala:1324

• The previous fallback converted an invalid/null destroy option to AutoError; this exhaustive match instead throws MatchError for DestroyOptions.from(List(null)). Retain the former error contract with an explicit null case.

      opt match

auto/src/main/scala/besom/auto/internal/Stack.scala:1377

• This removes the explicit null handling that previously raised AutoError("Unexpected null history option"); HistoryOptions.from(List(null)) now throws MatchError. Preserve the deliberate error type and message with a null case.

      opt match

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.