Skip to content

test: organize recipe sync cases without changing discovery - #1577

Merged
risu729 merged 3 commits into
mainfrom
claude/project-thread-ei232v
Nov 12, 2026
Merged

risu729 merged 3 commits into
mainfrom
claude/project-thread-ei232v

Conversation

@risu729

@risu729 risu729 commented Nov 12, 2026

Copy link
Copy Markdown
Owner

This replaces #1569. It keeps Codex's commits unchanged and adds one fix commit and a merge of main.

The fix commit:

  • Imports: in sync-layouts.ts and sync-recovery.ts, the value import and the separate type import from @pateat/contracts are now one statement, as in the original file.
  • Context type: the RecipeSyncContext = RecipeSyncFixtures alias is gone. The case modules now use the entry's RecipeSyncFixtures type directly.
  • Docs: the layout description gets its own section after the other test organization sections. It now describes the layout rather than the change. It also no longer calls recipes.test.ts the sole recipe sync entry, because recipe-fences.test.ts tests createRecipeSync too.

The merge of main resolves the docs conflict and keeps every sibling section.

Verification

  • Test discovery: the collected test IDs on main and on this branch are identical, in the same order: 109 tests, all passing.
  • Independent review: a reviewer pasted each registrar's body back in place of its call. Apart from the exported fixture type, the result is identical to origin/main. The test files have no hooks or mocks, and only consts and functions are passed in.
  • Checks: oxlint, oxfmt, rumdl and the extension type check pass.

Generated by Claude Code

risu729 and others added 3 commits November 12, 2026 21:41
Merge the split @pateat/contracts imports, drop the context alias, and give the
layout its own docs section beside the other test organization notes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011PUTx3w1HVYZp73fQasU3g
@risu729 risu729 self-assigned this Nov 12, 2026
@risu729
risu729 merged commit ac61e82 into main Nov 12, 2026
14 checks passed
@risu729
risu729 deleted the claude/project-thread-ei232v branch November 12, 2026 13:19
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.

2 participants