@itsmrtr/bunkoshelf 1.4.0 → 1.5.0

package/dist/src/components/profile/SubscriptionsTable.tsx

@@ -131,7 +131,7 @@ export default function SubscriptionsTable({ subscriptions: initial, intl }: Sub
                       onClick={() => !isLoading && handleDelete(sub.id, sub.endpoint)}
                       disabled={isLoading}
                       title={profile.unsubscribe as string}
-                      className="border border-stone-300 md:border-neutral-800 hover:text-danger-alt rounded-lg p-2 cursor-pointer transition-all duration-300 hover:border-danger-alt disabled:opacity-50 disabled:cursor-not-allowed"
+                      className="border border-neutral-800 hover:text-danger-alt rounded-lg p-2 cursor-pointer transition-all duration-300 hover:border-danger-alt disabled:opacity-50 disabled:cursor-not-allowed"
                     >
                       <Trash2Icon size={16} />
                     </button>

package/dist/src/components/settings/SettingsNav.tsx

@@ -4,6 +4,7 @@ import Link from "next/link";
 import { usePathname, useParams } from "next/navigation";
 import { useEffect, useState } from "react";
 import clsx from "clsx";
+import MobileSecondaryNav from "@/components/ui/MobileSecondaryNav";
 import { getSettingsNavLinks } from "@/lib/nav/settingsNav";
 import type { Dictionary } from "@/lib/types";
 
@@ -32,21 +33,23 @@ export default function SettingsNav({ intl }: SettingsNavProps) {
 
   return (
     <div className="mt-4 md:mt-16">
-      <div className="md:space-y-2 md:block flex gap-1">
+      <MobileSecondaryNav items={links} />
+
+      <div className="hidden md:space-y-2 md:block">
         {links.map(({ label, href, icon: Icon, isActive }) => (
           <Link
             key={href}
             href={href}
             className={clsx(
-              "flex items-center p-4 rounded-lg leading-none text-onix md:w-full justify-center md:justify-start flex-1 transition-all duration-300",
+              "flex items-center p-4 rounded-lg leading-none text-onix w-full justify-start transition-all duration-300",
               {
                 "bg-sand": isActive,
                 "hover:bg-sand": !isActive,
               }
             )}
           >
-            <Icon size={20} className="mr-0 md:mr-2" />
-            <span className="hidden md:inline">{label}</span>
+            <Icon size={20} className="mr-2" />
+            <span>{label}</span>
           </Link>
         ))}
       </div>

package/dist/src/components/settings/UsersTable.tsx

@@ -92,7 +92,7 @@ export default function UsersTable({
                       type="button"
                       onClick={() => handleEdit(user)}
                       title={intl.settings.edit as string}
-                      className="border border-stone-300 md:border-neutral-800 hover:text-pearl rounded-lg p-2 cursor-pointer transition-all duration-300 hover:border-lilah"
+                      className="border border-neutral-800 hover:text-pearl rounded-lg p-2 cursor-pointer transition-all duration-300 hover:border-lilah"
                     >
                       <UserRoundPenIcon size={16} />
                     </button>

package/dist/src/components/sidebar/MainNav.tsx

@@ -68,7 +68,7 @@ export default function MainNav({ intl, user }: MainNavProps) {
               </button>
 
               {openLibraryMenu && (
-                <div>
+                <div className="mt-1 md:mt-2 md:space-y-2">
                   {link.subItems?.map((subItem) => (
                     <Link
                       key={subItem.href}

package/dist/src/components/sidebar/sidebar.tsx

@@ -34,14 +34,11 @@ export default function Sidebar({
 
         {user && (
           <div className="flex-1">
-            <MainNav
-              intl={intl}
-              user={user}
-            />
+            <MainNav intl={intl} user={user} />
 
-            <div className="mt-16">
+            {/* <div className="mt-16">
               <SearchInput intl={intl} />
-            </div>
+            </div> */}
           </div>
         )}
 

package/dist/src/components/ui/MobileSecondaryNav.tsx

@@ -0,0 +1,126 @@
+"use client";
+
+import Link from "next/link";
+import { ChevronDownIcon, ChevronUpIcon, type LucideIcon } from "lucide-react";
+import clsx from "clsx";
+import { useEffect, useMemo, useRef, useState } from "react";
+
+export interface MobileSecondaryNavItem {
+  label: string;
+  href: string;
+  icon: LucideIcon;
+  isActive: boolean;
+  badge?: string | number | null;
+}
+
+interface MobileSecondaryNavProps {
+  items: MobileSecondaryNavItem[];
+}
+
+export default function MobileSecondaryNav({ items }: MobileSecondaryNavProps) {
+  const [open, setOpen] = useState(false);
+  const containerRef = useRef<HTMLDivElement | null>(null);
+
+  const activeItem = useMemo(
+    () => items.find((item) => item.isActive) ?? items[0],
+    [items],
+  );
+
+  useEffect(() => {
+    setOpen(false);
+  }, [activeItem?.href]);
+
+  useEffect(() => {
+    if (!open) {
+      return;
+    }
+
+    const handlePointerDown = (event: MouseEvent) => {
+      if (!containerRef.current?.contains(event.target as Node)) {
+        setOpen(false);
+      }
+    };
+
+    const handleEscape = (event: KeyboardEvent) => {
+      if (event.key === "Escape") {
+        setOpen(false);
+      }
+    };
+
+    document.addEventListener("mousedown", handlePointerDown);
+    document.addEventListener("keydown", handleEscape);
+
+    return () => {
+      document.removeEventListener("mousedown", handlePointerDown);
+      document.removeEventListener("keydown", handleEscape);
+    };
+  }, [open]);
+
+  if (!activeItem) {
+    return null;
+  }
+
+  const ActiveIcon = activeItem.icon;
+
+  return (
+    <div ref={containerRef} className="md:hidden mt-4 relative">
+      <button
+        type="button"
+        onClick={() => setOpen((prev) => !prev)}
+        aria-expanded={open}
+        className="w-full flex items-center justify-between gap-3 p-4 rounded-lg bg-sand text-onix leading-tight transition-all duration-300"
+      >
+        <span className="flex items-center min-w-0">
+          <ActiveIcon size={20} className="mr-2 shrink-0" />
+          <span className="truncate">{activeItem.label}</span>
+          {activeItem.badge !== undefined && activeItem.badge !== null && (
+            <span className="ml-2 text-xs px-2 py-0.5 rounded-sm bg-pearl shrink-0 uppercase">
+              {activeItem.badge}
+            </span>
+          )}
+        </span>
+        {open ? (
+          <ChevronUpIcon size={20} className="shrink-0" />
+        ) : (
+          <ChevronDownIcon size={20} className="shrink-0" />
+        )}
+      </button>
+
+      {open && (
+        <div className="absolute left-0 right-0 top-full mt-2 z-20 rounded-lg bg-blackamber p-2">
+          <nav className="space-y-1">
+            {items.map(({ href, icon: Icon, label, isActive, badge }) => (
+              <Link
+                key={href}
+                href={href}
+                prefetch={false}
+                onClick={() => setOpen(false)}
+                className={clsx(
+                  "flex items-center justify-between gap-3 rounded p-3 leading-tight transition-all duration-300",
+                  isActive
+                    ? "bg-sand text-onix"
+                    : "text-sand hover:bg-onix hover:text-pearl",
+                )}
+              >
+                <span className="flex items-center min-w-0">
+                  <Icon size={20} className="mr-2 shrink-0" />
+                  <span className="truncate">{label}</span>
+                </span>
+                {badge !== undefined && badge !== null && (
+                  <span
+                    className={clsx(
+                      "text-xs px-2 py-0.5 rounded-sm shrink-0 uppercase",
+                      isActive ? "bg-pearl text-onix" : "bg-sand text-onix",
+                    )}
+                  >
+                    {badge}
+                  </span>
+                )}
+              </Link>
+            ))}
+          </nav>
+        </div>
+      )}
+    </div>
+  );
+}

package/dist/src/lib/db/library.ts

@@ -17,6 +17,11 @@ export interface TagFilter {
   name: string;
 }
 
+export interface AuthorFilter {
+  id: string;
+  name: string;
+}
+
 export interface LibrarySeries {
   id: string;
   slug: string;
@@ -108,6 +113,7 @@ export interface LibrarySeriesWithVolumes extends LibrarySeries {
 interface SharedVolumeQueryOptions {
   includeGenres?: boolean;
   includeTags?: boolean;
+  authorNames?: string[];
   genreNames?: string[];
   tagNames?: string[];
   seriesIds?: string[];
@@ -122,6 +128,7 @@ interface VolumeQueryOptions extends SharedVolumeQueryOptions {
 
 interface SharedSeriesQueryOptions {
   userId?: string | null;
+  authorNames?: string[];
   genreNames?: string[];
   tagNames?: string[];
   seriesIds?: string[];
@@ -438,6 +445,7 @@ async function attachVolumeRelations(
 
 function buildVolumeFilterSql(
   options: {
+    authorNames?: string[];
     genreNames?: string[];
     tagNames?: string[];
     seriesIds?: string[];
@@ -468,6 +476,13 @@ function buildVolumeFilterSql(
     conditions.push(`mv.id = ANY($${params.length}::text[])`);
   }
 
+  if (options.authorNames && options.authorNames.length > 0) {
+    params.push(options.authorNames);
+    conditions.push(`
+      COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__') = ANY($${params.length}::text[])
+    `);
+  }
+
   if (options.genreNames && options.genreNames.length > 0) {
     for (const genreName of options.genreNames) {
       params.push(genreName);
@@ -664,15 +679,15 @@ export async function listPagedCatalogAuthors(
       `
         SELECT COUNT(*) AS total
         FROM (
-          SELECT vm.writer
+          SELECT COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__') AS author
           FROM manga_volumes mv
           LEFT JOIN volume_metadata vm ON vm.id = mv.metadata_id
-          GROUP BY vm.writer
+          GROUP BY COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__')
         ) AS authors
       `
     ),
     query<{
-      author: string | null;
+      author: string;
       works: string | number;
       avg_rating: number | null;
       read_count: string | number;
@@ -681,19 +696,28 @@ export async function listPagedCatalogAuthors(
     }>(
       `
         SELECT
-          vm.writer AS author,
-          COUNT(mv.id) AS works,
-          ROUND(AVG(utv.personal_rating)::numeric, 1)::float8 AS avg_rating,
-          COUNT(*) FILTER (WHERE utv.is_read = TRUE) AS read_count,
-          BOOL_OR(vm.manga_style IS DISTINCT FROM 'No') AS has_manga,
-          BOOL_OR(vm.manga_style = 'No') AS has_others
-        FROM manga_volumes mv
-        LEFT JOIN volume_metadata vm ON vm.id = mv.metadata_id
-        LEFT JOIN user_to_volumes utv
-          ON utv.volume_id = mv.id
-         AND utv.user_id = $1
-        GROUP BY vm.writer
-        ORDER BY COALESCE(vm.writer, '') ASC
+          grouped.author,
+          grouped.works,
+          grouped.avg_rating,
+          grouped.read_count,
+          grouped.has_manga,
+          grouped.has_others
+        FROM (
+          SELECT
+            COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__') AS author,
+            COUNT(mv.id) AS works,
+            ROUND(AVG(utv.personal_rating)::numeric, 1)::float8 AS avg_rating,
+            COUNT(*) FILTER (WHERE utv.is_read = TRUE) AS read_count,
+            BOOL_OR(vm.manga_style IS DISTINCT FROM 'No') AS has_manga,
+            BOOL_OR(vm.manga_style = 'No') AS has_others
+          FROM manga_volumes mv
+          LEFT JOIN volume_metadata vm ON vm.id = mv.metadata_id
+          LEFT JOIN user_to_volumes utv
+            ON utv.volume_id = mv.id
+           AND utv.user_id = $1
+          GROUP BY COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__')
+        ) AS grouped
+        ORDER BY grouped.author ASC
         LIMIT $2
         OFFSET $3
       `,
@@ -705,7 +729,7 @@ export async function listPagedCatalogAuthors(
 
   return mapPaginatedResult(
     rows.map((row) => ({
-      author: row.author,
+      author: row.author === "__unknown__" ? null : row.author,
       works: parseCount(row.works),
       avgRating: row.avg_rating,
       readCount: parseCount(row.read_count),
@@ -835,15 +859,30 @@ export async function listPagedCatalogTags(
 }
 
 async function listLibraryFiltersRaw(scope?: LibraryScope): Promise<{
+  authors: AuthorFilter[];
   genres: GenreFilter[];
   tags: TagFilter[];
 }> {
+  const authorParams: unknown[] = [];
   const genreParams: unknown[] = [];
   const tagParams: unknown[] = [];
+  const authorScopeWhere = buildLibraryFilterScopeConditions(authorParams, scope);
   const genreScopeWhere = buildLibraryFilterScopeConditions(genreParams, scope);
   const tagScopeWhere = buildLibraryFilterScopeConditions(tagParams, scope);
 
-  const [genres, tags] = await Promise.all([
+  const [authors, genres, tags] = await Promise.all([
+    query<AuthorFilter>(
+      `
+        SELECT DISTINCT
+          COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__') AS id,
+          COALESCE(NULLIF(BTRIM(vm.writer), ''), '__unknown__') AS name
+        FROM manga_volumes mv
+        LEFT JOIN volume_metadata vm ON vm.id = mv.metadata_id
+        ${authorScopeWhere}
+        ORDER BY name ASC
+      `,
+      authorParams
+    ),
     query<GenreFilter>(
       `
         SELECT DISTINCT g.id, g.name
@@ -870,7 +909,7 @@ async function listLibraryFiltersRaw(scope?: LibraryScope): Promise<{
     ),
   ]);
 
-  return { genres, tags };
+  return { authors, genres, tags };
 }
 
 async function listLibraryFiltersCached(scope?: LibraryScope) {
@@ -883,6 +922,7 @@ async function listLibraryFiltersCached(scope?: LibraryScope) {
 }
 
 export async function listLibraryFilters(scope?: LibraryScope): Promise<{
+  authors: AuthorFilter[];
   genres: GenreFilter[];
   tags: TagFilter[];
 }> {
@@ -910,6 +950,7 @@ async function listVolumesRaw(
 
   const conditions = buildVolumeFilterSql(
     {
+      authorNames: options?.authorNames,
       genreNames: options?.genreNames,
       tagNames: options?.tagNames,
       seriesIds: options?.seriesIds,
@@ -1022,6 +1063,7 @@ export async function listVolumes(
   return listVolumesCached({
     includeGenres: options?.includeGenres,
     includeTags: options?.includeTags,
+    authorNames: options?.authorNames,
     genreNames: options?.genreNames,
     tagNames: options?.tagNames,
     seriesIds: options?.seriesIds,
@@ -1037,6 +1079,7 @@ async function listPagedVolumeIdsRaw(
   const countParams: unknown[] = [];
   const countConditions = buildVolumeFilterSql(
     {
+      authorNames: options?.authorNames,
       genreNames: options?.genreNames,
       tagNames: options?.tagNames,
       seriesIds: options?.seriesIds,
@@ -1103,6 +1146,7 @@ export async function listPagedVolumes(
     : await listPagedVolumeIdsCached({
         includeGenres: options?.includeGenres,
         includeTags: options?.includeTags,
+        authorNames: options?.authorNames,
         genreNames: options?.genreNames,
         tagNames: options?.tagNames,
         seriesIds: options?.seriesIds,
@@ -1123,6 +1167,7 @@ export async function listPagedVolumes(
     volumeIds: pagedIds.items,
     includeGenres: options?.includeGenres,
     includeTags: options?.includeTags,
+    authorNames: options?.authorNames,
     scope: options?.scope,
   });
 
@@ -1205,6 +1250,7 @@ async function listSeriesWithVolumesRaw(
     userId: options?.userId,
     includeGenres: options?.includeGenres,
     includeTags: options?.includeTags,
+    authorNames: options?.authorNames,
     genreNames: options?.genreNames,
     tagNames: options?.tagNames,
     seriesIds: options?.seriesIds,
@@ -1245,6 +1291,7 @@ export async function listSeriesWithVolumes(
   options?: SharedSeriesQueryOptions
 ): Promise<LibrarySeriesWithVolumes[]> {
   return listSeriesWithVolumesCached({
+    authorNames: options?.authorNames,
     genreNames: options?.genreNames,
     tagNames: options?.tagNames,
     seriesIds: options?.seriesIds,
@@ -1264,6 +1311,7 @@ async function listPagedSeriesIdsRaw(
   const countParams: unknown[] = [];
   const countConditions = buildVolumeFilterSql(
     {
+      authorNames: options?.authorNames,
       genreNames: options?.genreNames,
       tagNames: options?.tagNames,
       seriesIds: options?.seriesIds,
@@ -1332,6 +1380,7 @@ export async function listPagedSeriesWithVolumes(
     }
 ): Promise<PaginatedResult<LibrarySeriesWithVolumes>> {
   const pagedIds = await listPagedSeriesIdsCached({
+    authorNames: options?.authorNames,
     genreNames: options?.genreNames,
     tagNames: options?.tagNames,
     seriesIds: options?.seriesIds,
@@ -1354,6 +1403,7 @@ export async function listPagedSeriesWithVolumes(
     seriesIds: pagedIds.items,
     includeGenres: options?.includeGenres,
     includeTags: options?.includeTags,
+    authorNames: options?.authorNames,
     scope: options?.scope,
   });
 

package/dist/src/lib/i18n/dictionaries/en/catalog.json

@@ -12,6 +12,7 @@
     "booksColumn": "Books",
     "totalColumn": "Total",
     "authorColumn": "Author",
+    "unknownAuthor": "Unknown",
     "worksColumn": "Works",
     "avgRatingColumn": "Avg",
     "progressColumn": "Progress",

package/dist/src/lib/i18n/dictionaries/en/favorites.json

@@ -9,6 +9,8 @@
     "ttSeriesOthers": "Series - Comic/Others",
     "ttVolumesOthers": "Volumes - Comic/Others",
     "sectionManga": "Manga",
+    "badgeSeries": "Series",
+    "badgeVolumes": "Volumes",
     "sectionMangaSeries": "Manga - Series",
     "sectionMangaVolumes": "Manga - Volumes",
     "sectionOthers": "Comic/Others",

package/dist/src/lib/i18n/dictionaries/en/filters.json

@@ -3,6 +3,7 @@
     "filter": "Filter",
     "filtering": "Filtering",
     "filters": "Filters",
+    "authors": "Authors",
     "genres": "Genres",
     "tags": "Tags",
     "apply": "Apply Filters",

package/dist/src/lib/i18n/dictionaries/es/catalog.json

@@ -12,6 +12,7 @@
     "booksColumn": "Libros",
     "totalColumn": "Total",
     "authorColumn": "Autor",
+    "unknownAuthor": "Desconocido",
     "worksColumn": "Obras",
     "avgRatingColumn": "Prom",
     "progressColumn": "Progreso",
@@ -22,7 +23,7 @@
     "yearColumn": "Año",
     "writerColumn": "Autor",
     "publisherColumn": "Editorial",
-    "statusColumn": "Estd",
+    "statusColumn": "Est",
     "languageColumn": "Idioma",
     "ageRatingColumn": "Clasf",
     "gtinColumn": "ISBN",

package/dist/src/lib/i18n/dictionaries/es/favorites.json

@@ -9,6 +9,8 @@
     "ttSeriesOthers": "Series - Cómic/Otros",
     "ttVolumesOthers": "Tomos - Cómic/Otros",
     "sectionManga": "Mangas",
+    "badgeSeries": "Series",
+    "badgeVolumes": "Tomos",
     "sectionMangaSeries": "Mangas - Series",
     "sectionMangaVolumes": "Mangas - Tomos",
     "sectionOthers": "Cómic/Otros",

package/dist/src/lib/i18n/dictionaries/es/filters.json

@@ -3,6 +3,7 @@
     "filter": "Filtrar",
     "filtering": "Filtrando",
     "filters": "Filtros",
+    "authors": "Autores",
     "genres": "Géneros",
     "tags": "Etiquetas",
     "apply": "Aplicar filtros",

package/dist/src/lib/nav/favoritesNav.ts

@@ -21,37 +21,39 @@ export function getFavoritesNavLinks({
   lang,
   pathname,
 }: GetFavoritesNavLinksParams): FavoritesNavLink[] {
+  const favorites = intl.favorites as DictionarySection;
+
   return [
     {
-      label: intl.favorites.sectionMangaSeries,
+      label: favorites.sectionManga,
       href: `/${lang}/favorites/manga`,
       icon: LibraryBigIcon,
       isActive: pathname === `/${lang}/favorites/manga`,
-      badge: "M",
+      badge: favorites.badgeSeries as string,
     },
     {
-      label: intl.favorites.sectionMangaVolumes,
+      label: favorites.sectionManga,
       href: `/${lang}/favorites/manga/volumes`,
       icon: BookCopyIcon,
       isActive: pathname === `/${lang}/favorites/manga/volumes`,
-      badge: "M",
+      badge: favorites.badgeVolumes as string,
     },
     {
-      label: intl.favorites.sectionOthersSeries,
+      label: favorites.sectionOthers,
       href: `/${lang}/favorites/others`,
       icon: LibraryBigIcon,
       isActive: pathname === `/${lang}/favorites/others`,
-      badge: "C",
+      badge: favorites.badgeSeries as string,
     },
     {
-      label: intl.favorites.sectionOthersVolumes,
+      label: favorites.sectionOthers,
       href: `/${lang}/favorites/others/volumes`,
       icon: BookCopyIcon,
       isActive: pathname === `/${lang}/favorites/others/volumes`,
-      badge: "C",
+      badge: favorites.badgeVolumes as string,
     },
     {
-      label: intl.favorites.sectionBooks,
+      label: favorites.sectionBooks,
       href: `/${lang}/favorites/books`,
       icon: BookHeartIcon,
       isActive: pathname === `/${lang}/favorites/books`,

package/dist/src/lib/utils.ts

@@ -39,6 +39,31 @@ export function ageRatingMap(ageRating: string | null | undefined): number | nul
   return mapping[ageRating] ?? null;
 }
 
+export function normalizeCommaSeparatedText(
+  value: string | string[] | null | undefined
+): string | null {
+  if (!value) return null;
+
+  if (Array.isArray(value)) {
+    const parts = value
+      .map((entry) => String(entry).trim())
+      .filter(Boolean);
+
+    return parts.length > 0 ? parts.join(", ") : null;
+  }
+
+  if (typeof value === "string") {
+    const parts = value
+      .split(",")
+      .map((entry) => entry.trim())
+      .filter(Boolean);
+
+    return parts.length > 0 ? parts.join(", ") : null;
+  }
+
+  return null;
+}
+
 function isTransientDatabaseConnectionError(error: unknown): boolean {
   if (!(error instanceof Error)) return false;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itsmrtr/bunkoshelf",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "Self-hosted personal manga library.",
   "keywords": [
     "bunko",