[JUM-906][Fix] Breadcrumb names the module and tab and links to real locations - #540
Merged
Merged
Conversation
…locations JUM-906. AppBreadcrumb rendered route.matched[].path — the route pattern — so a module page read "Home / Home" and linked to /#/m/:moduleId/:tab?. Crumbs now resolve to real locations and the Module route shows the module title and the active tab. Also fix the frontend typecheck, which failed on dev with findLast (ES2023) under an ES2022 lib. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_54578d8f-7b3f-430f-aa9a-eec571709a3c) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_98b114d8-48b5-498d-ad75-24696c7be196) |
32 tasks
…mb fix The frontend seed changed (AppBreadcrumb.vue, canaSchema.ts), so the packaged templates were stale and `cli:check-template-freshness` failed the branch gate (CircleCI job 2630). Refs: JUM-906 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_c37edb23-5a9a-4f31-bd81-7943d143f41c) |
web2solutions
deleted the
claude/fix/JUM-906-breadcrumb-route-pattern
branch
September 26, 2026 21:36
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.
Summary
The frontend breadcrumb now reads
Home / Users / Dashboardand links to real locations instead ofHome / Homepointing at/#/m/:moduleId/:tab?. Also fixes the frontend typecheck, which failed ondev.Problem Statement
AppBreadcrumb.vuerenderedroute.matched[].path— route patterns — and the Module route reusestitleKey: 'nav.home'. Measured on/#/m/users/dashboard: links/#/and/#/m/:moduleId/:tab?, both labelled Home.bun run typecheck(apps/frontend) →src/data/canaSchema.ts(35,57): error TS2550: Property 'findLast' does not exist on type 'string[]'(lib ES2022).Project Tracking (Required)
Jumentix(https://linear.app/jumentix)Start date->End date): 2026-09-26 -> 2026-09-29P0/P1/P2): P2Branch Promotion Path (Required)
[JUM-XXXX][Nature] <concise outcome>JUM-XXXXmatches the single Linear Issue declared above.dev.main, it is a release promotion sourced fromdev, references the task PRs/issues already merged intodev, and introduces no unreviewed changes.main.Bidirectional Traceability (Required)
JUM-906: 3aa086eScope of Change
Domain / Business Rules
Application / Use Cases
Adapters / Infrastructure
apps/frontend/src/components/AppBreadcrumb.vue: crumbs from resolved locations; Module route → module title + active tab label.apps/frontend/src/data/canaSchema.ts:.at(-1).apps/frontend/test/component/AppBreadcrumb.test.ts: 2 tests (labels; no:moduleIdhref).API / Contracts (OpenAPI, DTOs, handlers, controllers)
Detailed Technical Changes
Not fixed here (recorded on the Issue): the dashboard shows organization UUIDs instead of names.
Architecture and Design Alignment
Security Impact
Data and Migration Impact
Breaking Changes
None.
Acceptance Criteria
Test Plan (Evidence)
bun test apps/frontend/test— 313 pass, 0 failbun run typecheck (apps/frontend)— cleanbun run test:unit— task gate: Tests: 1158 passed, 1158 totalExtra evidence:
Feedback Resolution (Required)
resolved, a SHA from this PR.Coverage
SonarQube Cloud
Performance / Reliability Impact
No measurable impact.
Observability
Deployment and Rollout
dev, promote with the next release.Risks and Mitigations
Documentation Updates
mainmergeReviewer Checklist
🤖 Generated with Claude Code
Note
Low Risk
Navigation UI and a one-line schema helper change only; behavior is covered by new component tests with no auth or data impact.
Overview
Fixes JUM-906 by rebuilding
AppBreadcrumbso crumbs use resolved labels andhrefs instead of rawroute.matched[].pathpatterns. On module URLs, the trail is now Home / module title / tab (from the module manifest ornav.dashboard), with links like/m/usersrather than/#/m/:moduleId/:tab?. Breadcrumbs are driven by acomputedonuseRoute()instead ofref+afterEach/onMounted.Also replaces
findLastwith.at(-1)incanaSchema.tsso frontend typecheck passes on the current TS lib target. Adds component tests for labels and hrefs, and syncs cli-init templates and test-map registration.Reviewed by Cursor Bugbot for commit ba5e95e. Bugbot is set up for automated code reviews on this repo. Configure here.