Skip to content

Commit 91fc5ce

Browse files
authored
Merge pull request #19 from llbbl/deps/update-packages
chore(deps): update dependencies and fix code review findings
2 parents fc0e255 + a0ad78b commit 91fc5ce

11 files changed

Lines changed: 489 additions & 2898 deletions

File tree

‎.dockerignore‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,14 @@ out
3030
*.sublime-workspace
3131

3232
# Ignore testing files
33+
tests
3334
coverage
3435
.nyc_output
3536

37+
# Ignore CI and documentation
38+
.github
39+
*.md
40+
3641
# Ignore logs and temporary files
3742
logs
3843
*.log

‎.github/workflows/lighthouse.yml‎

Lines changed: 0 additions & 67 deletions
This file was deleted.

‎Dockerfile‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ FROM node:24-alpine AS deps
33
WORKDIR /app
44

55
# Install pnpm
6-
RUN corepack enable && corepack prepare pnpm@latest --activate
6+
RUN corepack enable && corepack prepare pnpm@10 --activate
77

88
# Copy package files
99
COPY package.json pnpm-lock.yaml ./
@@ -18,7 +18,7 @@ FROM node:24-alpine AS builder
1818
WORKDIR /app
1919

2020
# Install pnpm
21-
RUN corepack enable && corepack prepare pnpm@latest --activate
21+
RUN corepack enable && corepack prepare pnpm@10 --activate
2222

2323
# Copy dependencies from deps stage
2424
COPY --from=deps /app/node_modules ./node_modules

‎biome.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"$schema": "https://biomejs.dev/schemas/2.3.13/schema.json",
2+
"$schema": "https://biomejs.dev/schemas/2.4.4/schema.json",
33
"vcs": {
44
"enabled": true,
55
"clientKind": "git",

‎components/layout.tsx‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
// components/layout.tsx
22

3-
import { Inter } from 'next/font/google';
43
import type React from 'react';
54
import Copyright from '@/components/Copyright';
65
import { ErrorBoundary } from '@/components/ErrorBoundary';
@@ -9,7 +8,6 @@ import SEO from '@/components/SEO';
98
interface LayoutProps {
109
children: React.ReactNode;
1110
}
12-
const _inter = Inter({ subsets: ['latin'] });
1311

1412
const seoData = {
1513
title: '10 hours of Ludwig doing the Luddy! 🕺',

‎lighthouserc.js‎

Lines changed: 0 additions & 55 deletions
This file was deleted.

‎package.json‎

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -17,41 +17,35 @@
1717
"biome:lint": "biome lint .",
1818
"biome:format": "biome format . --write",
1919
"biome:fix": "biome check . --write",
20-
"analyze": "ANALYZE=true next build",
21-
"lighthouse": "lhci autorun",
22-
"lighthouse:collect": "lhci collect",
23-
"lighthouse:assert": "lhci assert"
20+
"analyze": "ANALYZE=true next build"
2421
},
2522
"dependencies": {
26-
"logan-logger": "^1.1.13",
23+
"logan-logger": "^1.1.14",
2724
"next": "16.1.6",
2825
"react": "^19.2.4",
2926
"react-dom": "^19.2.4"
3027
},
3128
"devDependencies": {
32-
"@biomejs/biome": "^2.3.13",
33-
"@lhci/cli": "^0.15.1",
29+
"@biomejs/biome": "^2.4.4",
3430
"@next/bundle-analyzer": "^16.1.6",
35-
"@tailwindcss/postcss": "^4.1.18",
31+
"@tailwindcss/postcss": "^4.2.1",
3632
"@testing-library/jest-dom": "^6.9.1",
3733
"@testing-library/react": "^16.3.2",
38-
"@types/node": "^24.10.1",
39-
"@types/react": "^19.2.10",
34+
"@types/node": "^25.3.0",
35+
"@types/react": "^19.2.14",
4036
"@types/react-dom": "^19.2.3",
41-
"@vitejs/plugin-react": "^5.1.2",
37+
"@vitejs/plugin-react": "^5.1.4",
4238
"@vitest/ui": "^4.0.18",
43-
"autoprefixer": "^10.4.24",
44-
"jsdom": "^27.4.0",
39+
"jsdom": "^28.1.0",
4540
"postcss": "^8.5.6",
4641
"sharp": "^0.34.5",
47-
"tailwindcss": "^4.1.18",
42+
"tailwindcss": "^4.2.1",
4843
"typescript": "^5.9.3",
4944
"vitest": "^4.0.18"
5045
},
5146
"pnpm": {
5247
"overrides": {
5348
"js-yaml": ">=4.1.1",
54-
"qs": ">=6.14.1",
5549
"lodash": ">=4.17.23",
5650
"lodash-es": ">=4.17.23"
5751
}

‎pages/_app.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// pages/_app.tsx
22
import type { AppProps } from 'next/app';
33
import '../assets/styles/globals.css';
4-
import Layout from '@/components/layout'; // Adjust the path as needed
4+
import Layout from '@/components/layout';
55

66
function MyApp({ Component, pageProps }: AppProps) {
77
return (

‎pages/index.tsx‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export default function Component() {
99

1010
useEffect(() => {
1111
// Set user context on page load
12-
const sessionId = `session_${Date.now()}_${Math.random().toString(36).substr(2, 9)}`;
12+
const sessionId = `session_${Date.now()}_${crypto.randomUUID().slice(0, 9)}`;
1313
setUserContext(undefined, sessionId, {
1414
page: 'home',
1515
userAgent: navigator.userAgent,
@@ -52,7 +52,7 @@ export default function Component() {
5252
</header>
5353
<main className="flex justify-center w-full mt-6 px-4">
5454
<section
55-
className="aspect-youtube w-full flex justify-center relative"
55+
className="aspect-video w-full flex justify-center relative"
5656
style={{ maxWidth: '80vw' }}
5757
aria-label="Ludwig Luddy dance video player"
5858
>
@@ -68,10 +68,9 @@ export default function Component() {
6868
)}
6969
<iframe
7070
ref={iframeRef}
71-
className="absolute top-0 left-0 w-full h-auto"
71+
className="absolute top-0 left-0 w-full h-auto border-0"
7272
src={shouldLoadIframe ? 'https://www.youtube.com/embed/L3Ucukzbp6k' : undefined}
7373
title="10 hours of Ludwig doing the Luddy dance - YouTube video player"
74-
frameBorder="0"
7574
allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture"
7675
allowFullScreen
7776
loading="lazy"

0 commit comments

Comments
 (0)