fix(contract) Remove unused UserDto import from recipes.ts #3

Merged
araemer merged 1 commit from fix-recipes-contract-unused-import into main 2026-09-19 17:30:41 +00:00
Owner

Summary

  • api-contract/src/recipes.ts imported UserDto from ./shared.js but never used it.
  • The contract ships raw .ts with no build step, so each consuming repo type-checks it under its own tsconfig. recipe-app's noUnusedLocals: true rejects the build over this dead import, discovered while implementing recipe-app's onboarding-integration plan (recipe-app PR #35).

Test plan

  • One-line removal, no shape change to the contract.
## Summary - `api-contract/src/recipes.ts` imported `UserDto` from `./shared.js` but never used it. - The contract ships raw `.ts` with no build step, so each consuming repo type-checks it under its own `tsconfig`. `recipe-app`'s `noUnusedLocals: true` rejects the build over this dead import, discovered while implementing recipe-app's onboarding-integration plan (recipe-app PR #35). ## Test plan - [x] One-line removal, no shape change to the contract.
fix(contract) Remove unused UserDto import from recipes.ts
All checks were successful
/ typecheck-contract (pull_request) Successful in 10s
d638d216a4
The contract ships raw .ts with no build step, so each consuming repo
type-checks it under its own tsconfig. recipe-app's noUnusedLocals: true
rejected the build over this dead import.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
araemer/recipe-platform!3
No description provided.