From 7ea2f7f40fa9de83b0699ab99b66cacebbc27af7 Mon Sep 17 00:00:00 2001
From: Katsuyuki Karasawa <4ranci0ne@gmail.com>
Date: Tue, 8 Apr 2025 22:44:52 +0900
Subject: [PATCH] refactor: remove unused props from components and improve
 error handling in about page (#385)

---
 src/components/ArchivePanel.astro        | 2 +-
 src/components/control/ButtonTag.astro   | 2 +-
 src/components/misc/License.astro        | 2 +-
 src/components/widget/SideBar.astro      | 2 --
 src/components/widget/WidgetLayout.astro | 1 -
 src/layouts/Layout.astro                 | 6 ++----
 src/pages/about.astro                    | 7 ++++++-
 src/pages/posts/[...slug].astro          | 1 -
 8 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/src/components/ArchivePanel.astro b/src/components/ArchivePanel.astro
index 6204bf21..102a53a3 100644
--- a/src/components/ArchivePanel.astro
+++ b/src/components/ArchivePanel.astro
@@ -10,7 +10,7 @@ interface Props {
   tags?: string[]
   categories?: string[]
 }
-const { keyword, tags, categories } = Astro.props
+const { tags, categories } = Astro.props
 
 let posts = await getSortedPosts()
 
diff --git a/src/components/control/ButtonTag.astro b/src/components/control/ButtonTag.astro
index bcafe657..50ac4b8a 100644
--- a/src/components/control/ButtonTag.astro
+++ b/src/components/control/ButtonTag.astro
@@ -5,7 +5,7 @@ interface Props {
   href?: string
   label?: string
 }
-const { size, dot, href, label }: Props = Astro.props
+const { dot, href, label }: Props = Astro.props
 ---
 <a href={href} aria-label={label} class="btn-regular h-8 text-sm px-3 rounded-lg">
     {dot && <div class="h-1 w-1 bg-[var(--btn-content)] dark:bg-[var(--card-bg)] transition rounded-md mr-2"></div>}
diff --git a/src/components/misc/License.astro b/src/components/misc/License.astro
index 750e659f..ebabd1b7 100644
--- a/src/components/misc/License.astro
+++ b/src/components/misc/License.astro
@@ -12,7 +12,7 @@ interface Props {
   class: string
 }
 
-const { title, slug, pubDate } = Astro.props
+const { title, pubDate } = Astro.props
 const className = Astro.props.class
 const profileConf = profileConfig
 const licenseConf = licenseConfig
diff --git a/src/components/widget/SideBar.astro b/src/components/widget/SideBar.astro
index 7c3242c1..4d0e0daf 100644
--- a/src/components/widget/SideBar.astro
+++ b/src/components/widget/SideBar.astro
@@ -3,7 +3,6 @@ import Profile from './Profile.astro'
 import Tag from './Tags.astro'
 import Categories from './Categories.astro'
 import type { MarkdownHeading } from 'astro'
-import TOC from './TOC.astro'
 
 interface Props {
     class? : string
@@ -11,7 +10,6 @@ interface Props {
 }
 
 const className = Astro.props.class
-const headings = Astro.props.headings
 
 ---
 <div id="sidebar" class:list={[className, "w-full"]}>
diff --git a/src/components/widget/WidgetLayout.astro b/src/components/widget/WidgetLayout.astro
index 3adb0150..9dde4573 100644
--- a/src/components/widget/WidgetLayout.astro
+++ b/src/components/widget/WidgetLayout.astro
@@ -10,7 +10,6 @@ interface Props {
   class?: string
   style?: string
 }
-const props = Astro.props
 const { id, name, isCollapsed, collapsedHeight, style } = Astro.props
 const className = Astro.props.class
 ---
diff --git a/src/layouts/Layout.astro b/src/layouts/Layout.astro
index 43a86f27..8a989ae6 100644
--- a/src/layouts/Layout.astro
+++ b/src/layouts/Layout.astro
@@ -208,9 +208,7 @@ import {
 	BANNER_HEIGHT,
 	BANNER_HEIGHT_HOME,
 	BANNER_HEIGHT_EXTEND,
-	MAIN_PANEL_OVERLAPS_BANNER_HEIGHT,
-	PAGE_WIDTH
-} from "../constants/constants";
+	MAIN_PANEL_OVERLAPS_BANNER_HEIGHT} from "../constants/constants";
 
 /* Preload fonts */
 // (async function() {
@@ -405,7 +403,7 @@ const setup = () => {
 			heightExtend.classList.remove('hidden')
 		}
 	});
-	window.swup.hooks.on('visit:end', (visit: {to: {url: string}}) => {
+	window.swup.hooks.on('visit:end', (_visit: {to: {url: string}}) => {
 		setTimeout(() => {
 			const heightExtend = document.getElementById('page-height-extend')
 			if (heightExtend) {
diff --git a/src/pages/about.astro b/src/pages/about.astro
index 7649dfcd..62d0864f 100644
--- a/src/pages/about.astro
+++ b/src/pages/about.astro
@@ -6,10 +6,15 @@ import { getEntry } from 'astro:content'
 import { i18n } from '../i18n/translation'
 import I18nKey from '../i18n/i18nKey'
 import Markdown from '@components/misc/Markdown.astro'
+import { render } from 'astro:content'
 
 const aboutPost = await getEntry('spec', 'about')
 
-const { Content } = await aboutPost.render()
+if (!aboutPost) {
+  throw new Error("About page content not found");
+}
+
+const { Content } = await render(aboutPost)
 ---
 <MainGridLayout title={i18n(I18nKey.about)} description={i18n(I18nKey.about)}>
     <div class="flex w-full rounded-[var(--radius-large)] overflow-hidden relative min-h-32">
diff --git a/src/pages/posts/[...slug].astro b/src/pages/posts/[...slug].astro
index 0585ddc4..7cb45228 100644
--- a/src/pages/posts/[...slug].astro
+++ b/src/pages/posts/[...slug].astro
@@ -1,6 +1,5 @@
 ---
 import path from 'node:path'
-import { getCollection } from 'astro:content'
 import License from '@components/misc/License.astro'
 import Markdown from '@components/misc/Markdown.astro'
 import I18nKey from '@i18n/i18nKey'