Refactor CommandMenu to prevent duplicate scripts across categories
This commit is contained in:
parent
5ad9323944
commit
319cf3167e
@ -134,9 +134,27 @@ export default function CommandMenu() {
|
|||||||
<CommandInput placeholder="Search for a script..." />
|
<CommandInput placeholder="Search for a script..." />
|
||||||
<CommandList>
|
<CommandList>
|
||||||
<CommandEmpty>{isLoading ? "Loading..." : "No scripts found."}</CommandEmpty>
|
<CommandEmpty>{isLoading ? "Loading..." : "No scripts found."}</CommandEmpty>
|
||||||
{links.map((category) => (
|
{(() => {
|
||||||
|
// Track seen scripts globally to avoid duplicates across all categories
|
||||||
|
const globalSeenScripts = new Set<string>();
|
||||||
|
|
||||||
|
return links.map((category) => {
|
||||||
|
const uniqueScripts = category.scripts.filter((script) => {
|
||||||
|
if (globalSeenScripts.has(script.slug)) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
globalSeenScripts.add(script.slug);
|
||||||
|
return true;
|
||||||
|
});
|
||||||
|
|
||||||
|
// Only render category if it has unique scripts
|
||||||
|
if (uniqueScripts.length === 0) {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
<CommandGroup key={`category:${category.name}`} heading={category.name}>
|
<CommandGroup key={`category:${category.name}`} heading={category.name}>
|
||||||
{category.scripts.map((script) => (
|
{uniqueScripts.map((script) => (
|
||||||
<CommandItem
|
<CommandItem
|
||||||
key={`script:${script.slug}`}
|
key={`script:${script.slug}`}
|
||||||
value={`${script.slug}-${script.name}`}
|
value={`${script.slug}-${script.name}`}
|
||||||
@ -161,7 +179,9 @@ export default function CommandMenu() {
|
|||||||
</CommandItem>
|
</CommandItem>
|
||||||
))}
|
))}
|
||||||
</CommandGroup>
|
</CommandGroup>
|
||||||
))}
|
);
|
||||||
|
});
|
||||||
|
})()}
|
||||||
</CommandList>
|
</CommandList>
|
||||||
</CommandDialog>
|
</CommandDialog>
|
||||||
</>
|
</>
|
||||||
|
Loading…
x
Reference in New Issue
Block a user