Skip to content

Respect solver configuration precedence - #158

Draft
sapunyangkut wants to merge 1 commit into
FRED-2:masterfrom
sapunyangkut:codex/issue-157-solver-config-precedence
Draft

Respect solver configuration precedence#158
sapunyangkut wants to merge 1 commit into
FRED-2:masterfrom
sapunyangkut:codex/issue-157-solver-config-precedence

Conversation

@sapunyangkut

Copy link
Copy Markdown

Summary

  • stop Click's built-in default from unconditionally overwriting the solver loaded from config.ini
  • preserve the intended precedence: explicit CLI option, environment variable, configuration file, then the built-in glpk default
  • report the effective solver and add regression coverage for all precedence cases

Fixes #157.

Validation

  • focused regression: 1 passed
  • adjacent CLI tests: 12 passed
  • git diff --check, reverse patch validation, and a bounded manual security review passed

AI assistance disclosure

This change was prepared and tested with OpenAI Codex. It is intentionally opened as a Draft for maintainer review. No automated issue/PR comments or Ready transition will be made.

Only override the configured solver when the CLI option or environment variable is explicitly supplied, while preserving glpk as the built-in default.

AI-assisted by OpenAI Codex; submitted as a Draft for maintainer review.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

In 1.5.0, solver from config is ignored

1 participant