From 0db75293836a80db0f4daa51150e22dac6155101 Mon Sep 17 00:00:00 2001 From: awesomerobot Date: Tue, 15 Jul 2025 12:10:36 -0400 Subject: [PATCH 1/2] FIX: prevent crash in "all" filter on features page --- .../discourse/components/ai-features-list.gjs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/assets/javascripts/discourse/components/ai-features-list.gjs b/assets/javascripts/discourse/components/ai-features-list.gjs index 27dd66b6f..e5e3d8713 100644 --- a/assets/javascripts/discourse/components/ai-features-list.gjs +++ b/assets/javascripts/discourse/components/ai-features-list.gjs @@ -66,12 +66,9 @@ class ExpandableList extends Component { export default class AiFeaturesList extends Component { get sortedModules() { - if (!this.args.modules || !this.args.modules.length) { - return []; - } - - return this.args.modules.toSorted((a, b) => - a.module_name.localeCompare(b.module_name) + const modules = this.args.modules?.toArray?.() || this.args.modules; + return [...(modules || [])].toSorted((a, b) => + (a?.module_name || "").localeCompare(b?.module_name || "") ); } From b99f6798286e5f57f5b3e54bda4477fa34a589e4 Mon Sep 17 00:00:00 2001 From: awesomerobot Date: Tue, 15 Jul 2025 12:14:52 -0400 Subject: [PATCH 2/2] simplify --- assets/javascripts/discourse/components/ai-features-list.gjs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/assets/javascripts/discourse/components/ai-features-list.gjs b/assets/javascripts/discourse/components/ai-features-list.gjs index e5e3d8713..4cd52499e 100644 --- a/assets/javascripts/discourse/components/ai-features-list.gjs +++ b/assets/javascripts/discourse/components/ai-features-list.gjs @@ -66,9 +66,9 @@ class ExpandableList extends Component { export default class AiFeaturesList extends Component { get sortedModules() { - const modules = this.args.modules?.toArray?.() || this.args.modules; + const modules = this.args.modules?.toArray() || this.args.modules; return [...(modules || [])].toSorted((a, b) => - (a?.module_name || "").localeCompare(b?.module_name || "") + (a.module_name || "").localeCompare(b.module_name || "") ); }