[nfebe]

Load optional pages and translations from installed plugin layers. Hide integration links when their pages are absent.

[cloudflare-workers-and-pages]

Deploying trakli-dev with    Cloudflare Pages

Latest commit:	312126d
Status:	✅  Deploy successful!
Preview URL:	https://f5aa4d1b.trakli-dev.pages.dev
Branch Preview URL:	https://feat-optional-ui-plugin-laye.trakli-dev.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying webui with    Cloudflare Pages

Latest commit:	312126d
Status:	✅  Deploy successful!
Preview URL:	https://d2a743aa.webui-9fh.pages.dev
Branch Preview URL:	https://feat-optional-ui-plugin-laye.webui-9fh.pages.dev

View logs

[sourceant]

Code Review Summary

Adds support for optional UI plugin layers via the TRAKLI_UI_LAYERS environment variable, wired into nuxt.config.ts and vitest.config.ts so layer pages, translations, tests, and aliases load alongside the host app. Extension slot contributions now skip links pointing at local routes that do not exist, and a new check:plugins script lints and format-checks configured layers. Documentation is added in PLUGINS.md.

🚀 Key Improvements

• TRAKLI_UI_LAYERS lets installed Nuxt layers extend routes, translations, and components without changing host code.
• useExtensionSlots hides sidebar and settings links whose local route is not installed, with test coverage.
• npm run check:plugins lints and format-checks configured plugin layers using the host's ESLint and Prettier config.

💡 Minor Suggestions

• Review coverage is incomplete. Inspect the execution details before approving.

[sourceant]

Review complete. See the overview comment for a summary.

[sourceant]

|| takes the first present href, so a contribution is gated on onboarding.href even when the card's own card.href route exists (and vice versa). A working link then disappears from a slot. Check every local href and skip only when none resolve.

Suggested change

	const href = ui.onboarding?.href || ui.card?.href;
	const localHrefs = [ui.onboarding?.href, ui.card?.href].filter(
	(href): href is string => typeof href === 'string' && href.startsWith('/')
	);
	if (localHrefs.length && localHrefs.every((href) => !router.resolve(href).matched.length)) {
	continue;
	}

[sourceant]

Review complete. See the overview comment for a summary.