Jesica Bermudes dev team application

[Cherriuu]

Why

Dev team application for Knights Hack.

What

Added my webpage page to Blade at /jesica-bermudes.

No other changes made.

Test Plan

I ran Blade locally and verified /jesica-bermudes loads correctly, I also tested the navigation links and verified that everything opens correctly.

Checklist

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

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds the /jesica-bermudes portfolio page. The page renders navigation, hero content, resume and section links, three project cards, and an about section. CSS defines the page layout, typography, buttons, cards, hover states, and pink-and-white theme.

Merge Risk: 🔵 Low · up to 53dbe

The new portfolio page may be difficult to read for users needing sufficient color contrast, and its navigation can overflow on narrow mobile screens. Addressing these styling issues will make the page ready to merge.

🚥 Pre-merge checks | ✅ 6 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title describes the added Jesica Bermudes page but does not start with an issue number in brackets.	Prefix the title with an issue number, for example: "[#123] Add Jesica Bermudes page".
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description is related to the changeset and identifies the new /jesica-bermudes page and local test plan.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
No Hardcoded Secrets	✅ Passed	No hardcoded API keys, passwords, tokens, or secrets are present in the three added files. Source scans found no secret-like assignments or token prefixes. The added PDF contained no credential labels…
Validated Env Access	✅ Passed	The pull request adds only the Jesica Bermudes page, its CSS module, and a PDF. No changed file contains process.env; the page imports only its CSS module. Existing environment access is confined to…
No Typescript Escape Hatches	✅ Passed	No TypeScript escape hatches were added. The only changed TypeScript file is the new page.tsx; searches found no any, @ts-ignore, @ts-expect-error, or !. usage, and the file contains no excl…

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

apps/blade/src/app/jesica-bermudes/page.module.css

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

apps/blade/src/app/jesica-bermudes/page.tsx

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).

---

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 (2)

apps/blade/src/app/jesica-bermudes/page.module.css-37-37 (1)

37-37: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Increase contrast for text and the primary button.

The #d97fa2, #e995b5, #81777f, and #d584a2 combinations are too light against the page or white backgrounds. Darken the text colors or the primary button background to meet WCAG AA contrast requirements.

As per path instructions, Blade UI changes require accessibility review.

Also applies to: 48-48, 52-52, 69-70, 115-115, 122-122

Source: Path instructions

apps/blade/src/app/jesica-bermudes/page.module.css-12-15 (1)

12-15: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Prevent navigation overflow on narrow screens.

At a 320px viewport, the four links, 20px gaps, and 40px horizontal padding can exceed the available width. Add a narrow-screen rule that reduces spacing and enables wrapping.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: QUIET

Plan: Advanced

Run ID: 6d9b25df-29a8-499c-b588-4197182ef246

📥 Commits

Reviewing files that changed from the base of the PR and between 4b19ad0 and 53dbea9.

⛔ Files ignored due to path filters (1)

• apps/blade/public/jesica-bermudes-resume.pdf is excluded by !**/*.pdf

📒 Files selected for processing (2)

• apps/blade/src/app/jesica-bermudes/page.module.css
• apps/blade/src/app/jesica-bermudes/page.tsx

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