docs: describe the Vue 3 stack in AGENTS.md - #2945
Merged
Merged
Conversation
AGENTS.md still described the frontend as Vue 2.7 with Vue Router 3, which stopped being true when #2836 migrated the app to Vue 3. package.json now pins Vue 3.5 and Vue Router 4. The lifecycle hook names differ between the two majors, so an agent following the outdated description would write beforeDestroy, which Vue 3 never calls and which therefore leaks subscriptions without any error. Assisted-by: ClaudeCode:claude-opus-5 Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
enjeck
approved these changes
Sep 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #2836.
AGENTS.mdstill described the frontend as Vue 2.7 with Vue Router 3. That stopped being true when #2836 migrated the app to Vue 3 —package.jsonnow pinsvue@^3.5.13andvue-router@^4.5.1.Also notes that lifecycle hooks use the Vue 3 names. This is the part that bites: an agent following the outdated description writes
beforeDestroy, which Vue 3 simply never calls — no error, no warning, just a subscription that is never cleaned up. All 12 components insrc/already usebeforeUnmount.Docs only, no code change.
AI tool use
This change was prepared with Claude Code (
claude-opus-5) and is disclosed via theAssisted-bytrailer on the commit, per the AI Contribution Policy. It was reviewed and submitted by me.