Skip to content

Fix ReflectionSliceBuilder by removing store name - #51

Merged
engineeringmadness merged 1 commit into
masterfrom
fix/reflection-slice-test
Jul 18, 2026
Merged

Fix ReflectionSliceBuilder by removing store name#51
engineeringmadness merged 1 commit into
masterfrom
fix/reflection-slice-test

Conversation

@engineeringmadness

Copy link
Copy Markdown
Owner

No description provided.

…package

The reflection-based slice test was failing because the @autostore annotation lost its store-name filter and the test package contained four reducers sharing the setName action type. Moving DifferentStoreReducer and OtherStoreReducer into a reflection.other sub-package leaves only SetNameReducer and DuplicateNameReducer competing for setName, matching the test's 'duplicate reducers should resolve' assertion.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@engineeringmadness
engineeringmadness merged commit 6ebb825 into master Jul 18, 2026
1 check passed
@engineeringmadness
engineeringmadness deleted the fix/reflection-slice-test branch August 1, 2026 05:54
engineeringmadness added a commit that referenced this pull request Aug 1, 2026
…package (#51)

The reflection-based slice test was failing because the @autostore annotation lost its store-name filter and the test package contained four reducers sharing the setName action type. Moving DifferentStoreReducer and OtherStoreReducer into a reflection.other sub-package leaves only SetNameReducer and DuplicateNameReducer competing for setName, matching the test's 'duplicate reducers should resolve' assertion.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
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