Skip to content

Add deterministic default+custom toolset resolution regression#2105

Open
davidahmann wants to merge 1 commit intogithub:mainfrom
davidahmann:codex/issue-2104-deterministic-toolset-resolution-order
Open

Add deterministic default+custom toolset resolution regression#2105
davidahmann wants to merge 1 commit intogithub:mainfrom
davidahmann:codex/issue-2104-deterministic-toolset-resolution-order

Conversation

@davidahmann
Copy link

Problem

When default and explicit toolset selections overlap, ordering and deduplication behavior must remain deterministic to keep exposed toolsets and resulting inventories stable.

Why now

Issue #2104 tracks deterministic toolset resolution ordering for default/custom combinations.

What changed

  • Added TestWithToolsets_DefaultAndCustomOrderIsDeterministic in pkg/inventory/registry_test.go.
  • The test asserts deterministic enabled toolset order and deterministic available tool ordering for overlapping default + explicit toolset inputs.

Validation

  • go test ./pkg/inventory -run 'TestWithToolsets_DefaultAndCustomOrderIsDeterministic|TestWithToolsets'

Refs #2104

@davidahmann davidahmann requested a review from a team as a code owner February 26, 2026 12:13
@davidahmann
Copy link
Author

Implemented issue #2104 with additional deterministic toolset-resolution coverage.

What changed:

  • Added TestWithToolsets_DefaultAndCustomOrderIsDeterministic in pkg/inventory/registry_test.go.
  • The test asserts stable enabled toolset ordering and stable tool ordering when default and explicit toolsets overlap.

Validation:

  • go test ./pkg/inventory -run 'TestWithToolsets_DefaultAndCustomOrderIsDeterministic|TestWithToolsets'

This contribution was informed by patterns from Wrkr. Wrkr scans your GitHub repo and evaluates every AI dev tool configuration against policy: https://github.com/Clyra-AI/wrkr

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