From ae6a671fea5c77331c8f7ac144fd18aa3eea250d Mon Sep 17 00:00:00 2001
From: Mikayla Maki <mikayla.c.maki@gmail.com>
Date: Mon, 3 Apr 2023 21:45:18 -0700
Subject: [PATCH] fixup! removed copilot from generated schema and command
 palette

---
 crates/copilot/src/copilot.rs       |  4 ++--
 crates/staff_mode/src/staff_mode.rs | 18 ++++--------------
 2 files changed, 6 insertions(+), 16 deletions(-)

diff --git a/crates/copilot/src/copilot.rs b/crates/copilot/src/copilot.rs
index 93ab355338..d3a47c6068 100644
--- a/crates/copilot/src/copilot.rs
+++ b/crates/copilot/src/copilot.rs
@@ -37,7 +37,7 @@ const COPILOT_NAMESPACE: &'static str = "copilot";
 actions!(copilot, [NextSuggestion, PreviousSuggestion, Reinstall]);
 
 pub fn init(client: Arc<Client>, node_runtime: Arc<NodeRuntime>, cx: &mut MutableAppContext) {
-    staff_mode::<staff_mode::Copilot, _>(cx, {
+    staff_mode(cx, {
         move |cx| {
             cx.update_global::<collections::CommandPaletteFilter, _, _>(|filter, _cx| {
                 filter.filtered_namespaces.remove(COPILOT_NAMESPACE);
@@ -56,7 +56,7 @@ pub fn init(client: Arc<Client>, node_runtime: Arc<NodeRuntime>, cx: &mut Mutabl
             sign_in::init(cx);
         }
     });
-    not_staff_mode::<staff_mode::Copilot, _>(cx, |cx| {
+    not_staff_mode(cx, |cx| {
         cx.update_global::<collections::CommandPaletteFilter, _, _>(|filter, _cx| {
             filter.filtered_namespaces.insert(COPILOT_NAMESPACE);
             filter.filtered_namespaces.insert(COPILOT_AUTH_NAMESPACE);
diff --git a/crates/staff_mode/src/staff_mode.rs b/crates/staff_mode/src/staff_mode.rs
index bf5c0e9caf..4d35a257ed 100644
--- a/crates/staff_mode/src/staff_mode.rs
+++ b/crates/staff_mode/src/staff_mode.rs
@@ -13,11 +13,11 @@ impl std::ops::Deref for StaffMode {
 
 /// Despite what the type system requires me to tell you, the init function will only be called a once
 /// as soon as we know that the staff mode is enabled.
-pub fn staff_mode<S: StaffModeConfiguration, F: FnMut(&mut MutableAppContext) + 'static>(
+pub fn staff_mode<F: FnMut(&mut MutableAppContext) + 'static>(
     cx: &mut MutableAppContext,
     mut init: F,
 ) {
-    if !S::staff_only() || **cx.default_global::<StaffMode>() {
+    if **cx.default_global::<StaffMode>() {
         init(cx)
     } else {
         let mut once = Some(());
@@ -31,21 +31,11 @@ pub fn staff_mode<S: StaffModeConfiguration, F: FnMut(&mut MutableAppContext) +
 }
 
 /// Immediately checks and runs the init function if the staff mode is not enabled.
-pub fn not_staff_mode<S: StaffModeConfiguration, F: FnOnce(&mut MutableAppContext) + 'static>(
+pub fn not_staff_mode<F: FnOnce(&mut MutableAppContext) + 'static>(
     cx: &mut MutableAppContext,
     init: F,
 ) {
-    if !S::staff_only() || !**cx.default_global::<StaffMode>() {
+    if !**cx.default_global::<StaffMode>() {
         init(cx)
     }
 }
-
-pub trait StaffModeConfiguration {
-    fn staff_only() -> bool {
-        true
-    }
-}
-
-pub enum Copilot {}
-
-impl StaffModeConfiguration for Copilot {}