Skip to content

Accept Swedish High reasoning label - #14

Draft
HenrikDanielsson wants to merge 1 commit into
DannyMac180:mainfrom
HenrikDanielsson:codex/accept-swedish-high-label
Draft

Accept Swedish High reasoning label#14
HenrikDanielsson wants to merge 1 commit into
DannyMac180:mainfrom
HenrikDanielsson:codex/accept-swedish-high-label

Conversation

@HenrikDanielsson

Copy link
Copy Markdown

What changed

  • Treat the user-facing reasoning labels High and Swedish Hög as the same semantic high level after trim/case normalization.
  • Keep model matching exact and machine/config evidence canonical lowercase high.
  • Apply the rule consistently in the orchestration skill, native role contract, and README.
  • Add verification assertions covering the localized-label contract.

Why

Codex Desktop localizes the High reasoning label as Hög in Swedish. Sol Advisor previously named only High, so a correctly configured Sol / high primary task could hit a false blocker solely because the UI label was localized.

The accepted-label set remains deliberately narrow: only high and hög normalize to the required semantic level. Different or unobservable effort evidence retains the existing stop/confirmation behavior.

Validation

  • sh plugins/sol-advisor/scripts/verify.sh — passed, including the new High/Hög assertion.
  • python3 .../skill-creator/scripts/quick_validate.py plugins/sol-advisor/skills/orchestrationSkill is valid!
  • git diff --check — passed.
  • Fresh Sol review — ship, no findings.

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.

1 participant