@btst/stack 1.4.1 → 1.5.0

package/src/plugins/blog/api/plugin.ts

@@ -442,11 +442,22 @@ export const blogBackendPlugin = (hooks?: BlogBackendHooks) =>
 						const { tags, ...postData } = ctx.body;
 						const tagNames = tags || [];
 
+						// Always slugify to ensure URL-safe slug, whether provided or generated from title
+						const slug = slugify(postData.slug || postData.title);
+
+						// Validate that slugification produced a non-empty result
+						if (!slug) {
+							throw ctx.error(400, {
+								message:
+									"Invalid slug: must contain at least one alphanumeric character",
+							});
+						}
+
 						const newPost = await adapter.create<Post>({
 							model: "post",
 							data: {
 								...postData,
-								slug: postData.slug ? postData.slug : slugify(postData.title),
+								slug,
 								tags: [],
 								createdAt: new Date(),
 								updatedAt: new Date(),
@@ -513,9 +524,25 @@ export const blogBackendPlugin = (hooks?: BlogBackendHooks) =>
 							}
 						}
 
-						const { tags, ...postData } = ctx.body;
+						const { tags, slug: rawSlug, ...restPostData } = ctx.body;
 						const tagNames = tags || [];
 
+						// Sanitize slug if provided to ensure it's URL-safe
+						const slugified = rawSlug ? slugify(rawSlug) : undefined;
+
+						// Validate that slugification produced a non-empty result (if slug was provided)
+						if (rawSlug && !slugified) {
+							throw ctx.error(400, {
+								message:
+									"Invalid slug: must contain at least one alphanumeric character",
+							});
+						}
+
+						const postData = {
+							...restPostData,
+							...(slugified ? { slug: slugified } : {}),
+						};
+
 						const updated = await adapter.transaction(async (tx) => {
 							const existingPostTags = await tx.findMany<{
 								postId: string;

package/src/plugins/blog/client/components/loading/form-page-skeleton.tsx

@@ -1,5 +1,5 @@
 import { PageHeaderSkeleton } from "./page-header-skeleton";
-import { PageLayout } from "../shared/page-layout";
+import { PageLayout } from "@workspace/ui/components/page-layout";
 import { Skeleton } from "@workspace/ui/components/skeleton";
 
 export function FormPageSkeleton() {

package/src/plugins/blog/client/components/loading/list-page-skeleton.tsx

@@ -1,5 +1,5 @@
 import { PageHeaderSkeleton } from "./page-header-skeleton";
-import { PageLayout } from "../shared/page-layout";
+import { PageLayout } from "@workspace/ui/components/page-layout";
 import { PostCardSkeleton } from "./post-card-skeleton";
 import { Skeleton } from "@workspace/ui/components/skeleton";
 

package/src/plugins/blog/client/components/loading/post-page-skeleton.tsx

@@ -1,5 +1,5 @@
 import { PageHeaderSkeleton } from "./page-header-skeleton";
-import { PageLayout } from "../shared/page-layout";
+import { PageLayout } from "@workspace/ui/components/page-layout";
 import { Skeleton } from "@workspace/ui/components/skeleton";
 
 export function PostPageSkeleton() {

package/src/plugins/blog/client/components/shared/page-wrapper.tsx

@@ -1,8 +1,7 @@
 "use client";
 
 import { usePluginOverrides } from "@btst/stack/context";
-import { BetterStackAttribution } from "@workspace/ui/components/better-stack-attribution";
-import { PageLayout } from "./page-layout";
+import { PageWrapper as SharedPageWrapper } from "@workspace/ui/components/page-wrapper";
 import type { BlogPluginOverrides } from "../../overrides";
 
 export function PageWrapper({
@@ -20,13 +19,14 @@ export function PageWrapper({
 	>("blog", {
 		showAttribution: true,
 	});
-	return (
-		<>
-			<PageLayout className={className} data-testid={testId}>
-				{children}
-			</PageLayout>
 
-			{showAttribution && <BetterStackAttribution />}
-		</>
+	return (
+		<SharedPageWrapper
+			className={className}
+			testId={testId}
+			showAttribution={showAttribution}
+		>
+			{children}
+		</SharedPageWrapper>
 	);
 }