Merge pull request #2 from diogoazevedos/test/polish-sample

Author: diogoazevedos

.github/workflows/main.yml

@@ -16,8 +16,8 @@ jobs:
           - 22
           - 24
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v5
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install

main.ts

@@ -46,6 +46,7 @@ export function pipe<Output, C, B, A, Input>(
 	s3: Pipe<C, B>,
 	s4: Pipe<Output, C>
 ): Pipe<Output, Input>;
+/* eslint-disable max-params */
 export function pipe<Output, D, C, B, A, Input>(
 	s1: Pipe<A, Input>,
 	s2: Pipe<B, A>,
@@ -70,6 +71,7 @@ export function pipe<Output, F, E, D, C, B, A, Input>(
 	s6: Pipe<F, E>,
 	s7: Pipe<Output, F>
 ): Pipe<Output, Input>;
+/* eslint-enable max-params */
 export function pipe<Input>(
 	init: Pipe<unknown, Input>,
 	...steps: Array<Pipe<unknown, unknown>>
@@ -107,6 +109,7 @@ export function parallel<H, G, F, E, D, C, B, A>(
 	s3: Pipe<F, E>,
 	s4: Pipe<H, G>
 ): Pipe<[B, D, F, H], A & C & E & G>;
+/* eslint-disable max-params */
 export function parallel<J, I, H, G, F, E, D, C, B, A>(
 	s1: Pipe<B, A>,
 	s2: Pipe<D, C>,
@@ -131,6 +134,7 @@ export function parallel<N, M, L, K, J, I, H, G, F, E, D, C, B, A>(
 	s6: Pipe<L, K>,
 	s7: Pipe<N, M>
 ): Pipe<[B, D, F, H, J, L, N], A & C & E & G & I & K & M>;
+/* eslint-enable max-params */
 export function parallel<Input>(...steps: Array<Pipe<unknown, Input>>): Pipe<unknown[], Input> {
 	return async input => Promise.all(steps.map(step => step(input)));
 }
@@ -153,9 +157,7 @@ type Assign<T extends unknown[]> = T extends [infer Head, ...infer Tail]
  * //=> {name: 'John Smith', hobby: 'Coding'}
  * ```
  */
-export function assign<Output extends unknown[], Input>(
-	step: Step<Output, Input>,
-): Pipe<Assign<Output>, Input> {
+export function assign<Output extends unknown[], Input>(step: Step<Output, Input>): Pipe<Assign<Output>, Input> {
 	// eslint-disable-next-line @typescript-eslint/no-unsafe-return
 	return async input => Object.assign({}, ...(await step(input)));
 }

package.json

@@ -36,17 +36,17 @@
 		"typescript"
 	],
 	"devDependencies": {
-		"@commitlint/cli": "^19.7.1",
-		"@commitlint/config-conventional": "^19.7.1",
-		"@sindresorhus/tsconfig": "^7.0.0",
-		"@types/node": "^22.13.4",
-		"ava": "^6.2.0",
+		"@commitlint/cli": "^20.0.0",
+		"@commitlint/config-conventional": "^20.0.0",
+		"@sindresorhus/tsconfig": "^8.0.1",
+		"@types/node": "^24.5.2",
+		"ava": "^6.4.1",
 		"c8": "^10.1.3",
 		"husky": "^9.1.7",
-		"lint-staged": "^15.4.3",
-		"tsx": "^4.19.2",
-		"typescript": "^5.7.3",
-		"xo": "^0.60.0"
+		"lint-staged": "^16.2.1",
+		"tsx": "^4.20.6",
+		"typescript": "^5.9.2",
+		"xo": "^1.2.2"
 	},
 	"commitlint": {
 		"extends": [

sample/user/middleware.ts

@@ -9,7 +9,7 @@ export type Logged = {
 
 export function makeWithUser(userService: UserService) {
 	return middleware(async ({claim}: Authorized) => {
-		const user = await userService.getByUserEmail(claim.email);
+		const user = await userService.getByEmail(claim.email);
 
 		return {user};
 	});

sample/user/service.ts

@@ -3,8 +3,8 @@ import type {User} from './schema.js';
 import type {UserRepository} from './repository.js';
 
 export type UserService = {
-	create(name: string, email: string): Promise<void>;
-	getByUserEmail(email: string): Promise<User>;
+	create(name: string, email: string): Promise<User>;
+	getByEmail(email: string): Promise<User>;
 };
 
 type UserServiceDependency = {
@@ -28,9 +28,11 @@ export function makeUserService({userRepository}: UserServiceDependency): UserSe
 		};
 
 		await userRepository.create(user);
+
+		return user;
 	}
 
-	async function getByUserEmail(email: string) {
+	async function getByEmail(email: string) {
 		const users = await userRepository.getAll();
 
 		const user = users.find(user => user.email === email);
@@ -42,5 +44,5 @@ export function makeUserService({userRepository}: UserServiceDependency): UserSe
 		throw new Error('User not found');
 	}
 
-	return {create, getByUserEmail};
+	return {create, getByEmail};
 }