fix(shell): Add shellcheck SC2329 disable directives

- Add SC2329 to bash-preexec.sh __bp_adjust_histcontrol function
- Add SC2329 disable before override function in shell integration generator
- Add cfg_attr to suppress dead_code warning on Windows for list_profiles_blocking
- Regenerate test snapshots with updated shellcheck directives

SC2329 warns about functions defined after being called, which is
intentional in our bash-preexec override pattern.

🤖 Assisted by Amazon Q Developer

Author: kkashilk

crates/chat-cli/src/cli/chat/context.rs

@@ -251,6 +251,7 @@ impl ContextManager {
     ///
     /// # Returns
     /// A Result containing a vector of profile names, with "default" always first
+    #[cfg_attr(target_os = "windows", allow(dead_code))]
     pub fn list_profiles_blocking(&self, os: &Os) -> Result<Vec<String>> {
         let _ = self;
 

crates/fig_integrations/src/shell/mod.rs

@@ -198,6 +198,7 @@ impl ShellExt for Shell {
                     include_str!("scripts/bash-preexec.sh"),
                     "}\n",
                     "__fig_source_bash_preexec\n",
+                    "# shellcheck disable=SC2329\n",
                     "function __bp_adjust_histcontrol() { :; }\n",
                     include_str!("scripts/pre.sh")
                 )
@@ -208,6 +209,7 @@ impl ShellExt for Shell {
                     include_str!("scripts/bash-preexec.sh"),
                     "}\n",
                     "__fig_source_bash_preexec\n",
+                    "# shellcheck disable=SC2329\n",
                     "function __bp_adjust_histcontrol() { :; }\n",
                     include_str!("scripts/post.bash")
                 )

crates/fig_integrations/src/shell/scripts/bash-preexec.sh

@@ -88,7 +88,7 @@ __bp_require_not_readonly() {
 # Remove ignorespace and or replace ignoreboth from HISTCONTROL
 # so we can accurately invoke preexec with a command from our
 # history even if it starts with a space.
-# shellcheck disable=SC2317
+# shellcheck disable=SC2317,SC2329
 __bp_adjust_histcontrol() {
     local histcontrol
     histcontrol="${HISTCONTROL:-}"

crates/q_cli/tests/snapshots/init__init_snapshot_bash_post_bash_profile.snap

@@ -1,5 +1,5 @@
 ---
-source: q_cli/tests/init.rs
+source: crates/q_cli/tests/init.rs
 expression: init
 ---
 Q_SHELL="/bin/zsh"
@@ -94,7 +94,7 @@ __bp_require_not_readonly() {
 # Remove ignorespace and or replace ignoreboth from HISTCONTROL
 # so we can accurately invoke preexec with a command from our
 # history even if it starts with a space.
-# shellcheck disable=SC2317
+# shellcheck disable=SC2317,SC2329
 __bp_adjust_histcontrol() {
     local histcontrol
     histcontrol="${HISTCONTROL:-}"
@@ -387,6 +387,7 @@ if [[ -z "${__bp_delay_install:-}" ]]; then
 fi;
 }
 __fig_source_bash_preexec
+# shellcheck disable=SC2329
 function __bp_adjust_histcontrol() { :; }
 if [[ -n "${BASH:-}" ]]; then
 

crates/q_cli/tests/snapshots/init__init_snapshot_bash_post_bashrc.snap

@@ -1,5 +1,5 @@
 ---
-source: q_cli/tests/init.rs
+source: crates/q_cli/tests/init.rs
 expression: init
 ---
 Q_SHELL="/bin/zsh"
@@ -94,7 +94,7 @@ __bp_require_not_readonly() {
 # Remove ignorespace and or replace ignoreboth from HISTCONTROL
 # so we can accurately invoke preexec with a command from our
 # history even if it starts with a space.
-# shellcheck disable=SC2317
+# shellcheck disable=SC2317,SC2329
 __bp_adjust_histcontrol() {
     local histcontrol
     histcontrol="${HISTCONTROL:-}"
@@ -387,6 +387,7 @@ if [[ -z "${__bp_delay_install:-}" ]]; then
 fi;
 }
 __fig_source_bash_preexec
+# shellcheck disable=SC2329
 function __bp_adjust_histcontrol() { :; }
 if [[ -n "${BASH:-}" ]]; then
 

crates/q_cli/tests/snapshots/init__init_snapshot_bash_pre_bash_profile.snap

@@ -1,5 +1,5 @@
 ---
-source: q_cli/tests/init.rs
+source: crates/q_cli/tests/init.rs
 expression: init
 ---
 Q_SHELL="/bin/zsh"
@@ -95,7 +95,7 @@ __bp_require_not_readonly() {
 # Remove ignorespace and or replace ignoreboth from HISTCONTROL
 # so we can accurately invoke preexec with a command from our
 # history even if it starts with a space.
-# shellcheck disable=SC2317
+# shellcheck disable=SC2317,SC2329
 __bp_adjust_histcontrol() {
     local histcontrol
     histcontrol="${HISTCONTROL:-}"
@@ -388,6 +388,7 @@ if [[ -z "${__bp_delay_install:-}" ]]; then
 fi;
 }
 __fig_source_bash_preexec
+# shellcheck disable=SC2329
 function __bp_adjust_histcontrol() { :; }
 #!/usr/bin/env bash
 

crates/q_cli/tests/snapshots/init__init_snapshot_bash_pre_bashrc.snap

@@ -1,5 +1,5 @@
 ---
-source: q_cli/tests/init.rs
+source: crates/q_cli/tests/init.rs
 expression: init
 ---
 Q_SHELL="/bin/zsh"
@@ -95,7 +95,7 @@ __bp_require_not_readonly() {
 # Remove ignorespace and or replace ignoreboth from HISTCONTROL
 # so we can accurately invoke preexec with a command from our
 # history even if it starts with a space.
-# shellcheck disable=SC2317
+# shellcheck disable=SC2317,SC2329
 __bp_adjust_histcontrol() {
     local histcontrol
     histcontrol="${HISTCONTROL:-}"
@@ -388,6 +388,7 @@ if [[ -z "${__bp_delay_install:-}" ]]; then
 fi;
 }
 __fig_source_bash_preexec
+# shellcheck disable=SC2329
 function __bp_adjust_histcontrol() { :; }
 #!/usr/bin/env bash