brainiac-fizzy 0.0.35 → 0.0.37

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f955d8bdd850598beb38a7779cee4018d05983e92fba4b331032915d4681716f
-  data.tar.gz: 7151d9a8f1b8ccdcfc2d1bb0d7a695ece787e279dbc88e7c567c7d8ea11cbe14
+  metadata.gz: d19bae8ece38c0924c091878a1bd2a5c6583089a0b6243ac69ff0cd9adaf1ab9
+  data.tar.gz: a149679a7117ca6312e9f854024bd9f51994e813df628f56a819b5e5d7610e5c
 SHA512:
-  metadata.gz: 8e9c9b4b9de9d348eda0279358b0353be2b40a281e59b33e2ba0b3d425113b69096e501dde2fa8b459b3bb5d4d7273ba05830189716740910ac0c25f1ffda1fe
-  data.tar.gz: 8ae711d9382289a266374f9a5291286c2710ec13e6f83cf601ffb81a907c2f41bfc3dc5330bb59ca7caf339a0e343bd5e9dac0e8f66af666b3d76655f3c0b6bd
+  metadata.gz: ca5ec9b110034ed0829014410cb50395b2ac23c2ee9d4ae43067ba8565626a1a148e002e9195fa07e605e9d520055df51230916780bf013097bb6a0b5ae98e5e
+  data.tar.gz: c2cdb76c0ab5622eef2edb97425b86329f68226bfbbab984ad2408fd365538f11fcc16ba4860e66d957e27bafb27f88d2c94e4b1a0088b155ccb99110b745564

data/lib/brainiac/plugins/fizzy/handlers/assignment.rb

@@ -269,12 +269,16 @@ def maybe_create_ephemeral_belt_env(worktree_path:, card_number:, project_key:,
   end
 
   LOG.info "[EphemeralEnv] Card ##{card_number} has deploy tag; #{env_name} not in worktree — creating from '#{parent_env}'"
+  # Manual setups (force: true) have a caller waiting on the result — there is
+  # no agent to unblock, so deploy synchronously and let failures surface.
+  # Agent-triggered deploys stay backgrounded so the agent isn't blocked.
   create_and_deploy_ephemeral_env(
     worktree_path: worktree_path, env_name: env_name, parent_env: parent_env,
-    card_number: card_number, project_key: project_key
+    card_number: card_number, project_key: project_key, synchronous: force
   )
 rescue StandardError => e
   LOG.error "[EphemeralEnv] Error creating ephemeral env: #{e.message}"
+  raise if force
 end
 
 def ensure_ephemeral_env_for_comment(ctx, card_number, worktree)
@@ -383,12 +387,14 @@ def track_ephemeral_env_if_needed(env_name, card_number, project_key, parent_env
                                  "worktree" => worktree_path)
 end
 
-def create_and_deploy_ephemeral_env(worktree_path:, env_name:, parent_env:, card_number:, project_key:)
+def create_and_deploy_ephemeral_env(worktree_path:, env_name:, parent_env:, card_number:, project_key:, synchronous: false)
   success = BeltEnvironment.create_environment(
     worktree: worktree_path, env_name: env_name, parent_env: parent_env
   )
   unless success
     LOG.error "[EphemeralEnv] Failed to create environment '#{env_name}'"
+    raise "Failed to create environment '#{env_name}'" if synchronous
+
     return
   end
 
@@ -397,19 +403,31 @@ def create_and_deploy_ephemeral_env(worktree_path:, env_name:, parent_env:, card
                                  "project" => project_key,
                                  "parent_env" => parent_env,
                                  "worktree" => worktree_path)
-  LOG.info "[EphemeralEnv] Configured #{env_name} in worktree, deploying in background"
 
-  Thread.new do
-    # First provisioning of a fresh env must be a full deploy — it stands up
-    # S3/CloudFront/backend. The frontend-only shortcut is only safe once the
-    # env is actually live (probe the derived public URL). Otherwise `belt
-    # deploy frontend` runs against a bucket that doesn't exist yet and fails
-    # with "Could not determine S3 bucket."
-    frontend_only =
-      ephemeral_env_live?(worktree_path, env_name) &&
-      BeltEnvironment.frontend_only_changes?(worktree: worktree_path)
-    BeltEnvironment.deploy(worktree: worktree_path, env_name: env_name, frontend_only: frontend_only)
-  rescue StandardError => e
-    LOG.error "[EphemeralEnv] Error deploying '#{env_name}': #{e.message}"
+  if synchronous
+    LOG.info "[EphemeralEnv] Configured #{env_name} in worktree, deploying synchronously"
+    deploy_ephemeral_env(worktree_path, env_name)
+  else
+    LOG.info "[EphemeralEnv] Configured #{env_name} in worktree, deploying in background"
+    Thread.new do
+      deploy_ephemeral_env(worktree_path, env_name)
+    rescue StandardError => e
+      LOG.error "[EphemeralEnv] Error deploying '#{env_name}': #{e.message}"
+    end
   end
 end
+
+# Run the actual `belt deploy` for a freshly configured ephemeral env.
+# Raises on failure so synchronous callers can surface the error; the
+# background path wraps this in its own rescue.
+def deploy_ephemeral_env(worktree_path, env_name)
+  # First provisioning of a fresh env must be a full deploy — it stands up
+  # S3/CloudFront/backend. The frontend-only shortcut is only safe once the
+  # env is actually live (probe the derived public URL). Otherwise `belt
+  # deploy frontend` runs against a bucket that doesn't exist yet and fails
+  # with "Could not determine S3 bucket."
+  frontend_only =
+    ephemeral_env_live?(worktree_path, env_name) &&
+    BeltEnvironment.frontend_only_changes?(worktree: worktree_path)
+  BeltEnvironment.deploy(worktree: worktree_path, env_name: env_name, frontend_only: frontend_only)
+end

data/lib/brainiac/plugins/fizzy/handlers/comments.rb

@@ -110,7 +110,7 @@ def build_comment_context(eventable:, plain_text:, tags:, card_internal_id:, car
   clean_text = tags[:clean_text]
 
   inline_cli = detect_cli_provider(text: plain_text, tags: card_tags)
-  inline_model, inline_model_explicit = detect_model_explicit(project_config, text: plain_text)
+  inline_model, inline_model_explicit = detect_model_explicit(project_config, text: plain_text, tags: card_tags)
   inline_effort = detect_effort(project_config, tags: card_tags, text: plain_text)
   inline_profile = detect_comment_profile(plain_text, card_tags, tags)
 

data/lib/brainiac/plugins/fizzy/version.rb

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Fizzy
-      VERSION = "0.0.35"
+      VERSION = "0.0.37"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainiac-fizzy
 version: !ruby/object:Gem::Version
-  version: 0.0.35
+  version: 0.0.37
 platform: ruby
 authors:
 - Andy Davis