Skip to content

Conversation

Copy link

Copilot AI commented Dec 13, 2025

User requested verification that frontend endpoints match the backend Axum rewrite PR. Successfully accessed and analyzed backend PR #115.

Verification Results

✅ 27/28 endpoints verified matching:

  • Authentication (4): /auth/login, /auth/register, /auth/change-username, /auth/change-password
  • Users (7): /users/@me, /users/@me/leaderboards, /users/@me/delete, /users/@me/activity/data, /users/@me/activity/summary, /users/{username}/activity/current, /users/{username}/activity/data
  • Friends (4): /friends/list, /friends/add, /friends/remove, /friends/regenerate
  • Leaderboards (9): /leaderboards/create, /leaderboards/join, /leaderboards/{name}, /leaderboard/{name} (DELETE), /leaderboards/{name}/leave, /leaderboards/{name}/promote, /leaderboards/{name}/demote, /leaderboards/{name}/kick, /leaderboards/{name}/regenerate
  • Activity (1): /activity/rename
  • Account (1): /account/settings
  • Search (1): /search/users

Key Findings:

  • The /leaderboard/{name} (singular) DELETE endpoint is correct - backend intentionally uses singular form
  • Backend properly supports @me as a special username parameter for activity endpoints
  • Previous review comment about singular/plural inconsistency is not an issue

❌ Critical Issue Identified:

  • /auth/regenerate endpoint was removed from backend but frontend still uses it in:
    • src/app/[locale]/profile/actions.ts:16 (regenerateToken function)
    • src/app/[locale]/profile/AuthTokenField.tsx:20 (UI component)
  • This is a breaking change requiring decision on whether to remove the feature or wait for replacement endpoint

Awaiting direction on how to handle the removed /auth/regenerate endpoint.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@vercel
Copy link

vercel bot commented Dec 13, 2025

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

Project Deployment Review Updated (UTC)
testaustime Ready Ready Preview, Comment Dec 13, 2025 0:18am
testaustime-storybook Ready Ready Preview, Comment Dec 13, 2025 0:18am

Copy link
Author

Copilot AI commented Dec 13, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/Testaustime/testaustime-backend/pulls/115/files
    • Triggering command: /usr/bin/curl curl -s -H Accept: application/vnd.github+json REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix changes for axum rewrite and refactor request handling Respond to endpoint consistency verification request Dec 13, 2025
Copilot AI requested a review from Eldemarkki December 13, 2025 00:19
@Eldemarkki
Copy link
Member

@copilot You now have access to the URL https://api.github.com/repos/Testaustime/testaustime-backend/pulls/115/files, please verify again

Copy link
Author

Copilot AI commented Dec 13, 2025

@copilot You now have access to the URL https://api.github.com/repos/Testaustime/testaustime-backend/pulls/115/files, please verify again

Verification complete!

Good news: 27/28 endpoints match correctly. The /leaderboard/{name} (singular) DELETE endpoint is actually CORRECT - the backend also uses the singular form.

Critical Issue Found: /auth/regenerate endpoint was removed from the backend (lines with -#[post("/auth/regenerate")]) but the frontend still uses it in:

  • src/app/[locale]/profile/actions.ts:16 (regenerateToken function)
  • src/app/[locale]/profile/AuthTokenField.tsx:20 (UI component)

This is a breaking change. Should I:

  1. Remove the regenerate auth token feature from the frontend, OR
  2. Wait for a replacement endpoint in the backend?

All other endpoints including /users/@me/activity/* are confirmed working (backend supports @me as a special username).

@Eldemarkki Eldemarkki closed this Dec 13, 2025
Copilot AI changed the title Respond to endpoint consistency verification request Verify endpoint consistency with backend Axum rewrite PR Dec 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants