Skip to content

Commit 72ca49d

Browse files
committed
fix(showcase/shell-docs): layout reserved-slug collision logs in production too
1 parent 0bab312 commit 72ca49d

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

showcase/shell-docs/src/app/layout.tsx

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,13 @@ export default function RootLayout({
5555
.map((i) => i.slug)
5656
.filter((slug) => {
5757
if (reserved.has(slug)) {
58-
if (process.env.NODE_ENV !== "production") {
59-
// eslint-disable-next-line no-console
60-
console.error(
61-
`[layout] integration slug "${slug}" collides with a reserved top-level route and was dropped from knownFrameworks. Rename the integration slug or update RESERVED_ROUTE_SLUGS.`,
62-
);
63-
}
58+
// Always log — a registry integration slug colliding with a
59+
// reserved top-level route is a hard wiring bug that production
60+
// operators need to see in logs, not a dev-only warning.
61+
// eslint-disable-next-line no-console
62+
console.error(
63+
`[layout] integration slug "${slug}" collides with a reserved top-level route and was dropped from knownFrameworks. Rename the integration slug or update RESERVED_ROUTE_SLUGS.`,
64+
);
6465
return false;
6566
}
6667
return true;

0 commit comments

Comments
 (0)