Add landing page with auto-generated screenshots#63
Open
Conversation
…lt types Domain services (HeatService, generateBracketForDivision) now return Promise<Result<T, E>> instead of throwing, making error paths explicit and type-safe. API handlers use unwrapOrThrow() at the oRPC boundary and result.isErr() checks in legacy REST routes. - Add neverthrow dependency and src/domain/result.ts re-export - Convert 6 HeatService methods and generateBracketForDivision to Result - Add HeatServiceError/BracketServiceError union types, TooManyParticipantsError - Create unwrapOrThrow() utility for oRPC handlers - Simplify domainErrorMapper middleware to infrastructure-error safety net - Fix heat-repository to return early/empty instead of throwing generic errors - Map HeatDoesNotExistError and ScoreNotFoundError to 404 (was 400) - Update all domain, integration, and middleware tests Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Covers page structure, Playwright screenshot generation, CI workflow, GitHub Pages deployment, and maintenance instructions for LLMs. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
11 tasks covering HTML page, Playwright setup, seed users, screenshot spec, CI workflow, README/CLAUDE.md updates, and GitHub Pages config. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Static landing page for GitHub Pages with ocean blue/teal gradient theme, sections for judges, head judges, and spectators, screenshot placeholders, and responsive layout using Tailwind CSS CDN. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add scripts/db/seed-users.ts that creates judge1, judge2, and headjudge test users without interactive prompts, for use in CI and local E2E setup. Skips users that already exist. Add db:seed:users npm script. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add isolated e2e/ directory with its own package.json, Playwright config, and TypeScript config for landing page screenshot generation. Update .gitignore to exclude e2e build artifacts, node_modules, and lock files. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Uses the modern Actions-based Pages deployment since the classic source.path only supports / or /docs, not /landing_page. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Run with: bun run test:e2e Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reads VITE_DEV_PORT and PORT from .env (bun auto-loads it) so E2E tests work in worktrees with non-default ports. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…env vars Simplify database configuration by removing support for the full connection string and requiring individual components: - POSTGRES_HOST (default: localhost) - POSTGRES_PORT (default: 5432) - POSTGRES_USER (required) - POSTGRES_PASSWORD (required) - POSTGRES_DB (required) Updated: - src/infrastructure/db/index.ts - drizzle.config.ts - scripts/db/reset.ts - docker-compose.yml - docker-compose.dev.yml - .github/workflows/screenshots.yml - .env.example - README.md - scripts/worktree-init.sh Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
| GitGuardian id | GitGuardian status | Secret | Commit | Filename | |
|---|---|---|---|---|---|
| 26794660 | Triggered | Generic Password | 31a218f | docker-compose.dev.yml | View secret |
| 26794660 | Triggered | Generic Password | 31a218f | docker-compose.dev.yml | View secret |
🛠 Guidelines to remediate hardcoded secrets
- Understand the implications of revoking this secret by investigating where it is used in your code.
- Replace and store your secrets safely. Learn here the best practices.
- Revoke and rotate these secrets.
- If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
- following these best practices for managing and storing secrets including API keys and other credentials
- install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
landing_page/index.html) with Tailwind CDN, ocean gradient themee2e/generate screenshots from seeded appNote: This branch is based on
neverthrow(PR #62). Merge that first, then this.Test plan
landing_page/index.htmlin browser - verify layout and stylingcd e2e && npx playwright testlocally with seeded app - verify all screenshots generated🤖 Generated with Claude Code