feat: add Scotiabank AI Engineer role and update researcher end date by Axeloooo · Pull Request #20 · Axeloooo/Portfolio

Axeloooo · 2026-08-02T21:37:00Z

Summary

Add Scotiabank AI Engineer work experience entry with logo
Update UCalgary Undergraduate Researcher end date to Apr. 2026

Test plan

Verified locally that resume renders correctly

Add Scotiabank work experience entry with logo, and set the UCalgary Undergraduate Researcher end date to Apr. 2026 now that it has a defined end.

vercel · 2026-08-02T21:37:05Z

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

Project	Deployment	Actions	Updated (UTC)
portfolio	Ready	Preview	Aug 2, 2026 9:44pm

qodo-code-review · 2026-08-02T21:37:48Z

PR Summary by Qodo

Add Scotiabank AI Engineer experience and set research end date

✨ Enhancement ⚙️ Configuration changes 📝 Documentation 🕐 10-20 Minutes

AI Description

• Add Scotiabank AI Engineer work experience entry with a new logo asset.
• Update UCalgary Undergraduate Researcher end date to Apr. 2026.
• Add Claude plugin settings, remove agent guidance doc, and normalize package-lock metadata.

Diagram

graph TD
  A["Resume data (src/data/resume.tsx)"] --> B["Resume UI (Next.js pages/components)"] --> C[("Static assets (public/)")]
  D["Claude settings (.claude/settings.json)"] --> E["Developer tooling"]
  F["package-lock.json"] --> E

High-Level Assessment

The following are alternative approaches to this PR:

1. Use an optimized SVG (or Next.js-optimized image pipeline) for the logo

➕ Better scaling/sharpness across devices
➕ Typically smaller payloads and improved Lighthouse scores
➕ Easier theming (e.g., dark mode variants) if needed
➖ May require sourcing/creating an SVG and validating brand guidelines
➖ Potential extra work to ensure consistent rendering vs PNG

2. Reduce lockfile churn by pinning Node/NPM versions (or using npm ci consistently)

➕ Avoids noisy diffs from metadata-only lockfile changes
➕ Improves reproducibility across contributors/CI
➖ Requires team/repo convention (e.g., .nvmrc, volta, or engine fields)
➖ Doesn't change runtime behavior; mostly workflow hygiene

Recommendation: The PR’s approach is appropriate for a content-driven portfolio (update the central resume data, add a public logo asset). If you want to tighten it further, consider switching the logo to an optimized SVG (or ensuring PNG optimization) and adopt a pinned Node/NPM setup to prevent future package-lock-only churn.

Files changed (4) +17 / -15 · 1 not counted

Enhancement (2) +12 / -1

scotiabank-logo.pngAdd Scotiabank logo asset for resume entry _{not counted}

Add Scotiabank logo asset for resume entry
• Introduces a new PNG logo under public/ so the new Scotiabank experience entry can render with branding.
public/scotiabank-logo.png

resume.tsxAdd Scotiabank AI Engineer role and update research end date +12/-1

Add Scotiabank AI Engineer role and update research end date

• Adds a new Scotiabank "AI Engineer" work experience item referencing the new logo asset. Updates the UCalgary Undergraduate Researcher role end date from "Present" to "Apr. 2026".

src/data/resume.tsx

Other (2) +5 / -14

settings.jsonEnable Claude plugin via repo settings +5/-0

Enable Claude plugin via repo settings
• Adds Claude configuration to enable the official "superpowers" plugin for agent-assisted workflows.
.claude/settings.json

package-lock.jsonNormalize lockfile metadata (peer flags removed) +0/-14

Normalize lockfile metadata (peer flags removed)

• Updates package-lock entries by removing several "peer": true metadata fields without changing dependency versions in the shown hunks. This is likely due to npm lockfile generation differences.

package-lock.json

Copilot

Pull request overview

This PR updates the portfolio’s single-source resume data (src/data/resume.tsx) to reflect new work experience and close out a research role. It also includes unrelated repo/tooling changes (lockfile churn, removal of agent docs, and new Claude plugin settings) that should be separated or explicitly justified.

Changes:

Add a new Scotiabank “AI Engineer” work entry (with logo) to the resume data.
Update the University of Calgary Undergraduate Researcher end date to Apr. 2026.
Include additional non-resume changes: lockfile metadata edits, removal of AGENTS.md, and addition of .claude/settings.json.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

File	Description
src/data/resume.tsx	Adds Scotiabank work entry and updates researcher end date in the central resume `DATA` object.
package-lock.json	Large lockfile churn (removal of many `"peer": true` markers) unrelated to resume content changes.
AGENTS.md	Deletes AI-agent guidance document (not mentioned in PR description).
.claude/settings.json	Adds Claude settings enabling a plugin by default (not mentioned in PR description).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+{
+  "enabledPlugins": {
+    "superpowers@claude-plugins-official": true
+  }


qodo-code-review · 2026-08-02T21:40:09Z

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📜 Skill insights (0)

Context used

✅ Compliance rules (platform): 3 rules

1. Scotiabank link blocked 🐞 Bug ≡ Correctness

Description

The new Scotiabank work entry sets a truthy description ("NDA"), which causes ResumeCard to call
preventDefault on click and toggle expansion instead of navigating, making the provided Scotiabank
href unreachable from the card.

Code

src/data/resume.tsx[R116-119]

+      start: "Jun. 2026",
+      end: "Present",
+      description: "NDA",
+    },

Evidence

The Scotiabank entry includes both an external href and a non-empty description. The homepage passes
that description into ResumeCard, whose click handler prevents default navigation whenever
description is present, so clicking the card cannot follow the href.

src/data/resume.tsx[108-119]
src/app/page.tsx[79-95]
src/components/resume-card.tsx[34-46]

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 Scotiabank ResumeCard is configured with an external `href`, but because `description` is non-empty, the ResumeCard click handler prevents default navigation and toggles expansion instead. This makes the Scotiabank link non-functional from the UI.

### Issue Context
`ResumeCard` is wrapped in a `Link` and attaches `onClick={handleClick}`. `handleClick` calls `e.preventDefault()` whenever `description` is truthy.

### Fix options
- **Simplest (data-only):** For the Scotiabank entry, omit `description` (or set it to an empty string) so the card click navigates to the href.
- **Behavioral (component):** Change `ResumeCard` so navigation is not prevented when `href` is provided (e.g., only toggle expansion via a dedicated control like the chevron).

### Fix Focus Areas
- src/data/resume.tsx[108-119]
- src/components/resume-card.tsx[34-46]

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

_{To customize comments, go to the Qodo configuration screen, or learn more in the docs.}

qodo-code-review · 2026-08-02T21:40:10Z

+      start: "Jun. 2026",
+      end: "Present",
+      description: "NDA",
+    },


1. Scotiabank link blocked 🐞 Bug ≡ Correctness

The new Scotiabank work entry sets a truthy description ("NDA"), which causes ResumeCard to call preventDefault on click and toggle expansion instead of navigating, making the provided Scotiabank href unreachable from the card.

Agent Prompt

### Issue description The Scotiabank ResumeCard is configured with an external `href`, but because `description` is non-empty, the ResumeCard click handler prevents default navigation and toggles expansion instead. This makes the Scotiabank link non-functional from the UI. ### Issue Context `ResumeCard` is wrapped in a `Link` and attaches `onClick={handleClick}`. `handleClick` calls `e.preventDefault()` whenever `description` is truthy. ### Fix options - **Simplest (data-only):** For the Scotiabank entry, omit `description` (or set it to an empty string) so the card click navigates to the href. - **Behavioral (component):** Change `ResumeCard` so navigation is not prevented when `href` is provided (e.g., only toggle expansion via a dedicated control like the chevron). ### Fix Focus Areas - src/data/resume.tsx[108-119] - src/components/resume-card.tsx[34-46]

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

Add contributing/workflow sections to README.md and CLAUDE.md explaining the devel -> main branch flow, the requirement to always squash-merge PRs, and the Conventional Commits format semantic-release relies on.

github-actions · 2026-08-02T21:53:00Z

🎉 This PR is included in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

feat: add Scotiabank AI Engineer role and update researcher end date

16f415d

Add Scotiabank work experience entry with logo, and set the UCalgary Undergraduate Researcher end date to Apr. 2026 now that it has a defined end.

Copilot AI review requested due to automatic review settings August 2, 2026 21:37

Copilot started reviewing on behalf of Axeloooo August 2, 2026 21:37 View session

vercel Bot deployed to Preview August 2, 2026 21:37 View deployment

Copilot AI reviewed Aug 2, 2026

View reviewed changes

Comment thread .claude/settings.json

Comment on lines +1 to +4

{

"enabledPlugins": {

"superpowers@claude-plugins-official": true

}

qodo-code-review Bot reviewed Aug 2, 2026

View reviewed changes

docs: document squash-merge and conventional commit workflow

56483e7

Add contributing/workflow sections to README.md and CLAUDE.md explaining the devel -> main branch flow, the requirement to always squash-merge PRs, and the Conventional Commits format semantic-release relies on.

vercel Bot deployed to Preview August 2, 2026 21:44 View deployment

Axeloooo merged commit d806ece into devel Aug 2, 2026
3 checks passed

Axeloooo deleted the feature/add-scotiabank-experience branch August 2, 2026 21:47

github-actions Bot added the released label Aug 2, 2026

Navigation Menu

feat: add Scotiabank AI Engineer role and update researcher end date - #20

feat: add Scotiabank AI Engineer role and update researcher end date#20
Axeloooo merged 2 commits into
develfrom
feature/add-scotiabank-experience

Axeloooo commented Aug 2, 2026

Uh oh!

vercel Bot commented Aug 2, 2026 •

edited

Loading

Uh oh!

qodo-code-review Bot commented Aug 2, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

qodo-code-review Bot commented Aug 2, 2026

Uh oh!

qodo-code-review Bot Aug 2, 2026

Uh oh!

Uh oh!

github-actions Bot commented Aug 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Axeloooo commented Aug 2, 2026

Summary

Test plan

Uh oh!

vercel Bot commented Aug 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

qodo-code-review Bot commented Aug 2, 2026

PR Summary by Qodo

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

qodo-code-review Bot commented Aug 2, 2026

Code Review by Qodo

Uh oh!

qodo-code-review Bot Aug 2, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Aug 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel Bot commented Aug 2, 2026 •

edited

Loading