Merge pull request 'chore(deps): update all non-major dependencies' (#125) from renovate/all-minor-patch into main

Reviewed-on: #125

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "fotochallenge",
-	"version": "0.0.7",
+	"version": "0.0.8",
 	"private": true,
 	"scripts": {
 		"dev": "vite dev",
@@ -34,7 +34,7 @@
 		"sass": "^1.77.5",
 		"simple-svelte-autocomplete": "^2.5.2",
 		"svelte": "^4.2.7",
-		"svelte-check": "^3.6.0",
+		"svelte-check": "^4.0.0",
 		"tslib": "^2.4.1",
 		"typescript": "^5.0.0",
 		"typescript-eslint": "^8.0.0-alpha.20",

renovate.json5

@@ -1,4 +1,12 @@
 {
 	$schema: 'https://docs.renovatebot.com/renovate-schema.json',
 	extends: ['local>renovate-bot/renovate-config'],
+	packageRules: [
+		{
+			groupName: 'all non-major dependencies',
+			groupSlug: 'all-minor-patch',
+			matchPackageNames: ['*'],
+			matchUpdateTypes: ['minor', 'patch'],
+		},
+	],
 }

src/hooks.server.ts

@@ -24,9 +24,9 @@ export const handle: Handle = async ({ event, resolve }) => {
 	// make requestId available to handlers
 	event.locals.requestId = requestId;
 
-	const { executionTime, result: response } = await timedExecution(async () => {
-		return await resolve(event);
-	});
+	const { executionTime, result: response } = await timedExecution(
+		async () => await resolve(event)
+	);
 	response.headers.set(requestIdHeader, requestId);
 
 	log.info(

src/lib/index.test.ts

@@ -1,4 +1,4 @@
-import safePath from '$lib';
+import safePath, { timedExecution } from '$lib';
 import { describe, it, expect } from 'vitest';
 
 describe('safe path', () => {
@@ -30,3 +30,21 @@ describe('safe path', () => {
 		expect(safePath('./uplodas', '..foobar..')).toBe(true);
 	});
 });
+
+describe('timedExecution', () => {
+	const asyncIdentity = async <T>(v: T): Promise<T> => v;
+	const identity = <T>(v: T): T => v;
+
+	it('works with async', async () => {
+		const { executionTime, result } = await timedExecution(() => asyncIdentity(5));
+		expect(result).toBe(5);
+		// execution time is always positive
+		expect(executionTime).toBeGreaterThanOrEqual(0);
+	});
+	it('works with sync', async () => {
+		const { executionTime, result } = await timedExecution(() => identity(5));
+		expect(result).toBe(5);
+		// execution time is always positive
+		expect(executionTime).toBeGreaterThanOrEqual(0);
+	});
+});

src/lib/index.ts

@@ -1,7 +1,6 @@
 // place files you want to import through the `$lib` alias in this folder.
 import path from 'path';
 import bunyan from 'bunyan';
-import type { MaybePromise } from '@sveltejs/kit';
 
 export const log = bunyan.createLogger({
 	name: 'fotochallenge',
@@ -9,7 +8,7 @@ export const log = bunyan.createLogger({
 	src: true,
 });
 
-function safePath(basePath: string, name: string): boolean {
+const safePath = (basePath: string, name: string): boolean => {
 	const fullPath = `${basePath}/${name}`;
 	const relative = path.relative(basePath, fullPath);
 	return (
@@ -21,7 +20,9 @@ function safePath(basePath: string, name: string): boolean {
 		// result is not an absolute path
 		!path.isAbsolute(relative)
 	);
-}
+};
+
+export default safePath;
 
 const defaultPath: string = './uploads';
 if (!('STORAGE_PATH' in process.env)) {
@@ -31,7 +32,7 @@ export const storagePath: string = process.env.STORAGE_PATH ?? defaultPath;
 
 export const requestIdHeader = 'x-request-id';
 
-export default safePath;
+export type MaybePromise<T> = T | Promise<T>;
 
 export async function timedExecution<T>(
 	fn: () => MaybePromise<T>