Skip to content

Conversation

@alexechoi
Copy link
Owner

@alexechoi alexechoi commented Nov 15, 2025

Add GitHub Format Check

High-level PR Summary

This PR adds a GitHub Actions workflow for Prettier formatting checks and implements a complete redesign of the landing page with new component architecture. The changes include setting up automated code formatting validation, adding the lucide-react icon library, creating a new themed navigation system with the AppChrome wrapper component, and building out a full landing page with Hero, Demo, TrustScore, and Footer sections. The PR also includes minor improvements to authentication flow (redirect logic for logged-in users) and error handling, plus gitignore updates for development files.

⏱️ Estimated Review Time: 15-30 minutes

💡 Review Order Suggestion
Order File Path
1 .github/workflows/format-check.yml
2 .gitignore
3 junction-app/.gitignore
4 junction-app/package.json
5 junction-app/bun.lock
6 junction-app/components/AppChrome.tsx
7 junction-app/components/Nav.tsx
8 junction-app/components/landing/Hero.tsx
9 junction-app/components/landing/Demo.tsx
10 junction-app/components/landing/TrustScore.tsx
11 junction-app/components/landing/Footer.tsx
12 junction-app/app/layout.tsx
13 junction-app/app/page.tsx
14 junction-app/app/auth/page.tsx
15 junction-app/app/dashboard/page.tsx
⚠️ Inconsistent Changes Detected
File Path Warning
junction-app/app/auth/page.tsx Authentication improvements (redirect logic and error handling) appear unrelated to the main PR purpose of adding format checks and landing page redesign
junction-app/app/dashboard/page.tsx Minor formatting changes to the dashboard component seem unrelated to the main PR focus on format checks and landing page

Need help? Join our Discord

@vercel
Copy link

vercel bot commented Nov 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aegis Ready Ready Preview Comment Nov 15, 2025 10:09am

@alexechoi alexechoi changed the base branch from main to alex/add-homepage November 15, 2025 10:00
Base automatically changed from alex/add-homepage to main November 15, 2025 10:02
Copy link

@recurseml recurseml bot left a comment

Choose a reason for hiding this comment

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

Review by RecurseML

🔍 Review performed on 0bb1f8d..2cd274b

  Severity     Location     Issue     Delete  
High .github/workflows/format-check.yml:10 GitHub workflow wrong directory
High .github/workflows/format-check.yml:21 Missing npm script reference
✅ Files analyzed, no issues (14)

.gitignore
junction-app/.gitignore
junction-app/app/auth/page.tsx
junction-app/app/dashboard/page.tsx
junction-app/app/layout.tsx
junction-app/app/page.tsx
junction-app/bun.lock
junction-app/components/AppChrome.tsx
junction-app/components/Nav.tsx
junction-app/components/landing/Demo.tsx
junction-app/components/landing/Footer.tsx
junction-app/components/landing/Hero.tsx
junction-app/components/landing/TrustScore.tsx
junction-app/package.json

@alexechoi alexechoi merged commit 13ab4f5 into main Nov 15, 2025
4 checks passed
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.

2 participants