Made Git City: Pulse for Neovim (Plugin) (New PR Updated some things) #69
Made Git City: Pulse for Neovim (Plugin) (New PR Updated some things) #69Woeter69 wants to merge 12 commits intosrizzon:mainfrom
Conversation
|
@Woeter69 is attempting to deploy a commit to the Samuel's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hey @Woeter69, the Neovim plugin idea is solid and the API/UI changes look reasonable. However there are two blockers:
Can you start a fresh branch from current main, apply just your changes (without the lock file corruption), and drop the dotenv dependency? The actual feature code looks good, just need a clean PR. Happy to merge once that's sorted. |
|
@srizzon have fixed the things mentioned. |
|
@srizzon hey just checking in to see if the latest changes look good to you. |
What does this PR do?
This PR introduces support for Neovim by updating the heartbeats API and adding frontend instructions for the upcoming
gitcity.nvimplugin.Specifically:
"neovim"toALLOWED_EDITORSin src/app/api/heartbeats/route.ts so Neovim heartbeats are correctly tracked./api/neovim-keyto handle generating Neovim API keys independently. Like GitHub tokens, keys are prefixed (nvim_), returned once, and only their secure hash is stored invscode_api_key_hash.gitcity.nvimalongside the existing VS Code extension steps. It also makes the privacy settings note editor-agnostic.Related issue
N/A
Screenshots
Testing Locally (Curl)
You can verify the backend API accepts
"neovim"without needing to install the plugin by firing a rawcurlrequest to your local dev server:Expected Result:
{"accepted":1,"rejected":0}And the user will appear with a green "Live" badge on the frontend.
Checklist
npm run lintpasses (Note: Some pre-existing linting errors exist in other files likeuseStreakCheckin.ts, but the touched files are clean)