[StevScripts]

What

• Adds /stevin-george to Blade with a resume, portfolio, GitHub, and LinkedIn.
• Resume and Portfolio start collapsed. Opening either shows a preview that links to the full PDF or website.
• Lenny peeks out of the grass. On desktop, clicking him lets you move and jump through the tree. Landing on Resume or Portfolio opens that section.
• Uses a simpler layout on mobile without the Lenny game.

Test Plan

• Blade typecheck passed using the repo's pinned pnpm version.
• Checked the desktop and mobile layouts, both dropdowns, and Lenny's movement, jumping, and Escape control.

Checklist

• Database: No schema changes, OR I ran pnpm db:generate and committed the generated files in packages/db/drizzle/
• Environment Variables: No environment variables changed, OR I have contacted the Development Lead to modify them on Coolify BEFORE merging.

[coderabbitai]

Walkthrough

Adds the /stevin-george page with resume and portfolio previews, GitHub and LinkedIn links, and a desktop game controlled by keyboard input. The game updates Lenny’s movement and landing state. The page includes responsive layouts, reduced-motion styling, page metadata, and documented browser and static checks.

Priority: Normal

Merge Risk: Low · up to 85e3c

Keyboard users may struggle to locate focus on Lenny. Fix the indicator before merging, or accept this bounded accessibility risk.

Pre-merge checks: 6 · 2

Failed checks: 2 warnings

Check name	Status	Explanation	Resolution
Title check		The title describes the page change but does not start with an issue number in brackets.	Change the title to start with an issue number, for example: "[#627] Add Stevin George developer application". Keep it under 72 characters.
Docstring Coverage		Docstring coverage is 4.35% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 6 passed

Check name	Status	Explanation
Description check		The description explains the new page, its interactions, responsive behavior, and test plan.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
No Hardcoded Secrets		No hardcoded API keys, passwords, tokens, or secrets were found in the pull-request files. The literals are page URLs, asset paths, SVG data, and resume metadata.
Validated Env Access		No process.env access was added by this pull request. The changed TypeScript/TSX files contain no raw environment access, and the added documentation only mentions “process values,” not a direct `pr…
No Typescript Escape Hatches		No changed TypeScript source uses any, @ts-ignore, @ts-expect-error, or non-null assertions (!.).

Full details: Validated Env Access

Explanation

No process.env access was added by this pull request. The changed TypeScript/TSX files contain no raw environment access, and the added documentation only mentions “process values,” not a direct process.env usage.

Full details: Docstring Coverage

Explanation

Docstring coverage is 4.35% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 23 functions across 4 files. (5 skipped: 5 unsupported.)

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

⚠️ This pull request has been flagged as potential spam (promotional) by CodeRabbit slop detection and should be reviewed carefully.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Note

Quiet mode is enabled, so only the most important comments were posted inline. Other review comments are grouped below.

Other comments: 1

apps/blade/src/app/stevin-george/stevin-george.module.css: 1

389-403: Minor · Functional Correctness · Quick win

Replace the Lenny focus indicator with a visible one.

.lenny:focus-visible removes the outline. The only indicator is a 32×4px --leaf bar on --paper, which has about 2.5:1 contrast. The bar sits at the bottom edge of the button. The .grass layer (z-index: 4) can cover it, because .lenny is only z-index: 3. Keyboard users can lose track of focus.

Proposed fix

 .lenny:focus-visible {
-  outline: none;
-}
-
-.lenny:focus-visible::after {
-  content: "";
-  position: absolute;
-  bottom: 0;
-  left: 50%;
-  width: 32px;
-  height: 4px;
-  border-radius: 999px;
-  background: var(--leaf);
-  transform: translateX(-50%);
+  outline: 3px solid var(--terracotta);
+  outline-offset: 4px;
 }

Based on learnings: interactive elements need a visible keyboard focus indicator.

Source: Learnings

---

Review info

Run configuration

• Source mode: Auto
• Repository: KnightHacks/forge
• Branch: main
• File: .coderabbit.yml
• Commit: 5115574
• Layers: Repository YAML
• Review profile: QUIET
• Plan: Advanced
• Run ID: ce0d15b8-7ebe-47d2-9776-c822803fed20

Commits

Reviewing files that changed from the base of the PR and between 5115574 and 85e3c63.

Files ignored due to path filters: 8

• apps/blade/public/dev-team/stevin-george/chevron.svg is excluded by !**/*.svg
• apps/blade/public/dev-team/stevin-george/grass.png is excluded by !**/*.png
• apps/blade/public/dev-team/stevin-george/lenny.png is excluded by !**/*.png
• apps/blade/public/dev-team/stevin-george/pointer.svg is excluded by !**/*.svg
• apps/blade/public/dev-team/stevin-george/portfolio-preview.jpg is excluded by !**/*.jpg
• apps/blade/public/dev-team/stevin-george/stevin-george-resume-preview.jpg is excluded by !**/*.jpg
• apps/blade/public/dev-team/stevin-george/stevin-george-resume.pdf is excluded by !**/*.pdf
• apps/blade/public/dev-team/stevin-george/tree.png is excluded by !**/*.png

Files selected for processing: 10

• .forge/features/stevin-dev-team-application/spec.md
• .forge/features/stevin-dev-team-application/srd.md
• .forge/features/stevin-dev-team-application/status.md
• .forge/features/stevin-dev-team-application/test-cases.md
• apps/blade/public/dev-team/stevin-george/lenny-walk-sheet.webp
• apps/blade/src/app/stevin-george/application-scene.tsx
• apps/blade/src/app/stevin-george/lenny-game.ts
• apps/blade/src/app/stevin-george/lenny-walk.ts
• apps/blade/src/app/stevin-george/page.tsx
• apps/blade/src/app/stevin-george/stevin-george.module.css

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.