From f756deb4f55138b3af250b81d4a33f345d366a2f Mon Sep 17 00:00:00 2001 From: Jack Amadeo Date: Fri, 3 Apr 2026 10:14:19 -0400 Subject: [PATCH] some redundant comments --- crates/goose/src/config/base.rs | 2 -- ui/desktop/src/components/AnnouncementModal.tsx | 1 - .../src/components/settings/extensions/bundled-extensions.ts | 3 +-- ui/desktop/src/distro.ts | 3 --- ui/desktop/src/goosed.ts | 1 - ui/desktop/src/preload.ts | 1 - 6 files changed, 1 insertion(+), 10 deletions(-) diff --git a/crates/goose/src/config/base.rs b/crates/goose/src/config/base.rs index 61f89a8c6e..42187ba679 100644 --- a/crates/goose/src/config/base.rs +++ b/crates/goose/src/config/base.rs @@ -1086,7 +1086,6 @@ fn find_workspace_or_exe_root() -> Option { } pub fn load_init_config_from_workspace() -> Result { - // Check for distro-provided init config first if let Ok(init_config_path) = std::env::var("GOOSE_INIT_CONFIG") { let path = PathBuf::from(&init_config_path); if path.exists() { @@ -1095,7 +1094,6 @@ pub fn load_init_config_from_workspace() -> Result { } } - // Fall through to existing workspace root logic let root = find_workspace_or_exe_root().ok_or_else(|| { ConfigError::FileError(std::io::Error::new( std::io::ErrorKind::NotFound, diff --git a/ui/desktop/src/components/AnnouncementModal.tsx b/ui/desktop/src/components/AnnouncementModal.tsx index 4b2669b66d..340378ccb5 100644 --- a/ui/desktop/src/components/AnnouncementModal.tsx +++ b/ui/desktop/src/components/AnnouncementModal.tsx @@ -57,7 +57,6 @@ export default function AnnouncementModal() { } try { - // Try distro announcements first, then fall back to bundled let announcements: AnnouncementMeta[]; try { const distroAnnouncements = await window.electron.getDistroAnnouncements(); diff --git a/ui/desktop/src/components/settings/extensions/bundled-extensions.ts b/ui/desktop/src/components/settings/extensions/bundled-extensions.ts index 6c440184e7..3b03d489ea 100644 --- a/ui/desktop/src/components/settings/extensions/bundled-extensions.ts +++ b/ui/desktop/src/components/settings/extensions/bundled-extensions.ts @@ -32,7 +32,7 @@ async function loadBundledExtensions(): Promise { return distroExtensions; } } catch { - // Fall through to default + // use default } return defaultBundledExtensionsData as BundledExtension[]; } @@ -75,7 +75,6 @@ export async function syncBundledExtensions( addExtensionFn: (name: string, config: ExtensionConfig, enabled: boolean) => Promise ): Promise { try { - // Load bundled extensions (may be overridden by distro overlay) const bundledExtensions = await loadBundledExtensions(); // Process each bundled extension diff --git a/ui/desktop/src/distro.ts b/ui/desktop/src/distro.ts index 2fc4fd265a..173b650125 100644 --- a/ui/desktop/src/distro.ts +++ b/ui/desktop/src/distro.ts @@ -11,13 +11,11 @@ let distroDir: string | null = null; let distroConfig: DistroConfig = {}; function findDistroDir(): string | null { - // 1. Check external path (no re-signing needed) const externalPath = path.join(app.getPath('userData'), 'distro'); if (fs.existsSync(path.join(externalPath, 'distro.json'))) { return externalPath; } - // 2. Check in-bundle path const bundlePath = path.join(process.resourcesPath, 'distro'); if (fs.existsSync(path.join(bundlePath, 'distro.json'))) { return bundlePath; @@ -32,7 +30,6 @@ if (distroDir) { const configPath = path.join(distroDir, 'distro.json'); distroConfig = JSON.parse(fs.readFileSync(configPath, 'utf-8')); - // Apply env var overrides immediately if (distroConfig.env) { for (const [key, value] of Object.entries(distroConfig.env)) { process.env[key] = value; diff --git a/ui/desktop/src/goosed.ts b/ui/desktop/src/goosed.ts index 51439fa0fd..f8e6244d3d 100644 --- a/ui/desktop/src/goosed.ts +++ b/ui/desktop/src/goosed.ts @@ -257,7 +257,6 @@ export const startGoosed = async (options: StartGoosedOptions): Promise Promise; closeApp: (appName: string) => Promise; addRecentDir: (dir: string) => Promise; - // Distro overlay functions getDistroBundledExtensions: () => Promise; getDistroAnnouncements: () => Promise; };