Feat/me by taterhead247 · Pull Request #163 · F3-Nation/f3-nation

taterhead247 · 2026-03-05T17:29:41Z

This pull request introduces the initial infrastructure and documentation for the new f3-me app, including deployment automation, environment configuration, code ownership, and foundational backend tests. The changes are primarily focused on enabling smooth development, deployment, and testing of the f3-me app within the monorepo, while also providing clear documentation for both human and AI contributors.

Deployment & Infrastructure

Added a GitHub Actions workflow (.github/workflows/deploy-me.yml) to automate building, staging, and production deployment of the f3-me app to Google Cloud Run, including concurrency controls and environment-specific authentication.
Added a multi-stage Dockerfile (apps/me/Dockerfile) for efficient, production-ready builds of the f3-me Next.js app using Turbo and pnpm.
Added example environment variable files for both local development (apps/me/.env.local.example) and Cloud Run deployment (apps/me/.env.cloud-run.example). [1] [2]

Documentation & Ownership

Added an AGENTS.md guide in apps/me to document architecture, API usage, file structure, and common enhancement tasks, specifically aimed at onboarding AI coding agents and new developers.
Updated .github/CODEOWNERS to assign ownership of the apps/me directory to @taterhead247.

Testing Foundation

Introduced initial Vitest-based API route tests for avatar uploads (apps/me/__tests__/api/avatar.test.ts) and position removal (apps/me/__tests__/api/positions.test.ts), including mocking of authentication and API client dependencies. [1] [2]

Dependency Management

Updated dependencies in apps/api/package.json and apps/map/package.json to use the latest workspace version of @acme/api for improved compatibility. [1] [2]

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Pull request overview

Copilot reviewed 70 out of 74 changed files in this pull request and generated 7 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

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

apps/me/src/app/api/profile/roles/route.ts

apps/me/src/components/ui/avatar.tsx

apps/me/src/components/avatar-upload.tsx

apps/me/src/components/ui/toast.tsx

apps/me/__tests__/lib/auth/session.test.ts

apps/me/README.md

apps/me/src/app/api/profile/route.ts

taterhead247 · 2026-03-05T22:35:00Z

@copilot open a new pull request to apply changes based on the remaining comments in this thread

Copilot · 2026-03-05T22:35:07Z

@taterhead247 I've opened a new pull request, #164, to work on those changes. Once the pull request is ready, I'll request review from you.

…and session expiry test (#164) * Initial plan * Fix all remaining open PR review comments Co-authored-by: taterhead247 <26818765+taterhead247@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: taterhead247 <26818765+taterhead247@users.noreply.github.com>

taterhead247 · 2026-03-13T14:00:10Z

@copilot open a new pull request to apply changes based on the remaining unresolved comments in this thread

Copilot · 2026-03-13T14:00:17Z

@taterhead247 I've opened a new pull request, #178, to work on those changes. Once the pull request is ready, I'll request review from you.

#177) * Initial plan * Fix validation, type safety, and add missing config files in apps/me Co-authored-by: taterhead247 <26818765+taterhead247@users.noreply.github.com> * improved test that this sub-pr broke --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: taterhead247 <26818765+taterhead247@users.noreply.github.com> Co-authored-by: Tackle <damon.vinciguerra@gmail.com>

Copilot

Pull request overview

Copilot reviewed 72 out of 76 changed files in this pull request and generated 7 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

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

You can also share your feedback on Copilot code review. Take the survey.

package.json

apps/me/src/components/avatar-upload.tsx

apps/me/src/components/ui/avatar.tsx

apps/me/src/components/auth-card.tsx

apps/me/src/app/api/auth/callback/route.ts

apps/me/src/lib/api/client.ts

apps/me/src/app/api/profile/positions/route.ts

postion put response

Copilot

Pull request overview

Copilot reviewed 65 out of 70 changed files in this pull request and generated 3 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

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

You can also share your feedback on Copilot code review. Take the survey.

apps/me/src/components/profile-form.tsx

apps/me/src/app/api/profile/route.ts

apps/me/scripts/cloud-run-env.sh

github-project-automation bot added this to F3 Nation Tech Pull Requests Mar 5, 2026

github-project-automation bot moved this to Draft in F3 Nation Tech Pull Requests Mar 5, 2026

taterhead247 requested a review from Copilot March 5, 2026 17:30

Copilot AI reviewed Mar 5, 2026

View reviewed changes

taterhead247 requested a review from Copilot March 5, 2026 21:21

Copilot started reviewing on behalf of taterhead247 March 5, 2026 21:21 View session

Copilot AI reviewed Mar 5, 2026

View reviewed changes

Copilot AI mentioned this pull request Mar 5, 2026

Fix bugs in roles API, avatar state, toast a11y, meta serialization, and session expiry test #164

Merged

taterhead247 and others added 9 commits March 5, 2026 20:49

f3 me initial

bfec820

it's alive!

72b708a

mostly working local

603a810

branding

6773a6a

fixing ci checks

16c4cac

fix build by downgrading react to 18

05f41aa

fixed orange change bar

382fe64

remove gif as allowable type

66e3dc2

taterhead247 force-pushed the feat/me branch from ca2ea4f to d606c19 Compare March 6, 2026 02:49

saving seed file

fcb6fcd

taterhead247 requested a review from Copilot March 6, 2026 03:15

taterhead247 marked this pull request as ready for review March 6, 2026 03:15

taterhead247 requested review from dnishiyama, evanpetzoldt and pstaylor-patrick as code owners March 6, 2026 03:15

Copilot started reviewing on behalf of taterhead247 March 6, 2026 03:16 View session

Add CODEOWNER for /apps/me directory

6236cba

This comment was marked as outdated.

Sign in to view

converted from firebase to cloud run

f506030

Copilot AI mentioned this pull request Mar 13, 2026

Fix input validation, type safety, and missing config files in apps/me #177

Merged

Copilot AI mentioned this pull request Mar 13, 2026

Fix input validation and meta type alignment in apps/me API routes #178

Closed

taterhead247 requested a review from Copilot March 13, 2026 19:17

Copilot started reviewing on behalf of taterhead247 March 13, 2026 19:17 View session

Copilot AI reviewed Mar 13, 2026

View reviewed changes

taterhead247 added 9 commits March 13, 2026 14:47

removed firebase residue and cleanup

e964052

matching versions and removing gifs

c4710ee

open-redirect vector

4268d4e

postion put response

useful auth fail responses

0cd7029

consolidate references

bc90844

html fix

9c06f89

toast timers

928fdda

add server-only package and improve environment variable handling

942e15b

handling server side security changes

b63a4de

taterhead247 requested a review from Copilot March 13, 2026 21:28

Copilot started reviewing on behalf of taterhead247 March 13, 2026 21:29 View session

Copilot AI reviewed Mar 13, 2026

View reviewed changes

apps/me/src/components/profile-form.tsx Outdated Show resolved Hide resolved

apps/me/src/app/api/profile/route.ts Outdated Show resolved Hide resolved

apps/me/scripts/cloud-run-env.sh Show resolved Hide resolved

taterhead247 added 9 commits March 13, 2026 16:43

type validation

9c8b531

meta validaton

1164175

wasn't saved yet

bff1929

Merge branch 'dev' into feat/me

66299b0

new api

3f8df9f

missed some

014ec00

a ton of changes on user reckoning

bd99bd6

stopping to take a look at api security

8941cdb

Merge branch 'dev' into feat/me

0fef9ae

taterhead247 moved this from Draft to Initiated in F3 Nation Tech Pull Requests Mar 20, 2026

Conversation

taterhead247 commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

taterhead247 commented Mar 5, 2026

Uh oh!

Copilot AI commented Mar 5, 2026

Uh oh!

This comment was marked as outdated.

Uh oh!

taterhead247 commented Mar 13, 2026

Uh oh!

Copilot AI commented Mar 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

taterhead247 commented Mar 5, 2026 •

edited

Loading