Skip to content

add website landing page#55

Merged
PropzSaladaz merged 1 commit into
masterfrom
enhancement/add-website
Apr 12, 2026
Merged

add website landing page#55
PropzSaladaz merged 1 commit into
masterfrom
enhancement/add-website

Conversation

@PropzSaladaz
Copy link
Copy Markdown
Owner

No description provided.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@PropzSaladaz PropzSaladaz self-assigned this Apr 12, 2026
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Add Laze website landing page with Next.js and Tailwind CSS

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Create complete Next.js website landing page for Laze project
• Implement dark-themed design with custom CSS variables and Tailwind integration
• Build responsive hero section with feature showcase and comparison table
• Add download section, screenshots gallery, and project story narrative
Diagram
flowchart LR
  A["Next.js Setup"] --> B["Design System"]
  B --> C["Landing Page"]
  C --> D["Hero & Features"]
  D --> E["Comparison & Downloads"]
  E --> F["Footer & Navigation"]
Loading

Grey Divider

File Changes

1. website/next.config.ts ⚙️ Configuration changes +7/-0

Next.js configuration with image optimization

website/next.config.ts


2. website/app/globals.css ✨ Enhancement +88/-0

Dark theme design tokens and Tailwind integration

website/app/globals.css


3. website/app/page.tsx ✨ Enhancement +586/-0

Complete landing page with hero, features, and sections

website/app/page.tsx


View more (6)
4. website/app/layout.tsx ✨ Enhancement +17/-0

Root layout with metadata and styling imports

website/app/layout.tsx


5. website/package.json Dependencies +25/-0

Project dependencies for Next.js and Tailwind

website/package.json


6. website/tsconfig.json ⚙️ Configuration changes +21/-0

TypeScript configuration for Next.js project

website/tsconfig.json


7. website/postcss.config.mjs ⚙️ Configuration changes +7/-0

PostCSS configuration for Tailwind CSS

website/postcss.config.mjs


8. README.md Additional files +0/-4

...

README.md


9. desktop_app/src-tauri/icons/icon.icns Additional files +0/-0

...

desktop_app/src-tauri/icons/icon.icns


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Apr 12, 2026

Code Review by Qodo

🐞 Bugs (2)   📘 Rule violations (0)   📎 Requirement gaps (0)   🖥 UI issues (0)   🎨 UX Issues (0)
🐞\ ≡ Correctness (1) ☼ Reliability (1)

Grey Divider


Action required

1. Invalid TS path alias 🐞
Description
website/tsconfig.json defines compilerOptions.paths without a baseUrl, which TypeScript treats as an
invalid configuration and can break builds or make the @/* alias unusable. This will surface as
module resolution/config errors once the website is type-checked/built.
Code

website/tsconfig.json[R2-18]

+  "compilerOptions": {
+    "target": "ES2017",
+    "lib": ["dom", "dom.iterable", "esnext"],
+    "allowJs": true,
+    "skipLibCheck": true,
+    "strict": true,
+    "noEmit": true,
+    "esModuleInterop": true,
+    "module": "esnext",
+    "moduleResolution": "bundler",
+    "resolveJsonModule": true,
+    "isolatedModules": true,
+    "jsx": "preserve",
+    "incremental": true,
+    "plugins": [{ "name": "next" }],
+    "paths": { "@/*": ["./*"] }
+  },
Evidence
The website tsconfig introduces a path alias ("@/*") but does not define compilerOptions.baseUrl,
which is required for paths mapping to work correctly.

website/tsconfig.json[1-21]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`website/tsconfig.json` sets `compilerOptions.paths` without setting `compilerOptions.baseUrl`, which makes the TS path alias config invalid / non-functional.

### Issue Context
This repo uses Next.js + TypeScript in multiple subprojects (e.g., `desktop_app`). The new `website` app should have a valid TS config so `next build` / type-checking works reliably.

### Fix Focus Areas
- Add `"baseUrl": "."` under `compilerOptions` (keep the existing `paths` mapping).

- website/tsconfig.json[2-18]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Remediation recommended

2. Website not built in CI 🐞
Description
No GitHub Actions workflow runs build/typecheck for the new website/ project, so landing-page
regressions can merge without any automated signal. Current workflows only cover controller_server,
desktop_app/src-tauri, and mobile_client paths.
Code

website/package.json[R6-24]

+  "scripts": {
+    "dev": "next dev",
+    "build": "next build",
+    "start": "next start"
+  },
+  "dependencies": {
+    "next": "^15.3.3",
+    "react": "^18.3.1",
+    "react-dom": "^18.3.1"
+  },
+  "devDependencies": {
+    "@tailwindcss/postcss": "^4.1.12",
+    "@types/node": "22.15.29",
+    "@types/react": "^18.3.1",
+    "@types/react-dom": "^18.3.1",
+    "postcss": "^8.5.6",
+    "tailwindcss": "^4.1.12",
+    "typescript": "~5.6.2"
+  }
Evidence
The PR adds a new Next.js project under website/, but existing CI workflows are scoped via
paths: filters to other subprojects and never run when only website/** changes.

website/package.json[1-25]
.github/workflows/rust-ci.yml[3-16]
.github/workflows/flutter-ci.yml[3-14]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
The new `website/` Next.js app has no CI coverage, so broken builds/type errors can be merged without detection.

### Issue Context
Current GitHub Actions workflows use `paths:` filters and do not include `website/**`.

### Fix Focus Areas
- Add a new workflow (e.g. `.github/workflows/website-ci.yml`) that triggers on PRs/commits touching `website/**`.
- In that workflow, run at least:
 - `npm ci`
 - `npm run build`
 - (optional) `npm run lint` / `next lint` if you add linting

- .github/workflows/website-ci.yml[1-160]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Comment thread website/tsconfig.json
Comment on lines +2 to +18
"compilerOptions": {
"target": "ES2017",
"lib": ["dom", "dom.iterable", "esnext"],
"allowJs": true,
"skipLibCheck": true,
"strict": true,
"noEmit": true,
"esModuleInterop": true,
"module": "esnext",
"moduleResolution": "bundler",
"resolveJsonModule": true,
"isolatedModules": true,
"jsx": "preserve",
"incremental": true,
"plugins": [{ "name": "next" }],
"paths": { "@/*": ["./*"] }
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Action required

1. Invalid ts path alias 🐞 Bug ≡ Correctness

website/tsconfig.json defines compilerOptions.paths without a baseUrl, which TypeScript treats as an
invalid configuration and can break builds or make the @/* alias unusable. This will surface as
module resolution/config errors once the website is type-checked/built.
Agent Prompt
### Issue description
`website/tsconfig.json` sets `compilerOptions.paths` without setting `compilerOptions.baseUrl`, which makes the TS path alias config invalid / non-functional.

### Issue Context
This repo uses Next.js + TypeScript in multiple subprojects (e.g., `desktop_app`). The new `website` app should have a valid TS config so `next build` / type-checking works reliably.

### Fix Focus Areas
- Add `"baseUrl": "."` under `compilerOptions` (keep the existing `paths` mapping).

- website/tsconfig.json[2-18]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

@PropzSaladaz PropzSaladaz merged commit 5dd2b9c into master Apr 12, 2026
4 checks passed
@PropzSaladaz PropzSaladaz deleted the enhancement/add-website branch April 12, 2026 21:54
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.

1 participant