bug: hallucinated project_id

The agent created a hallucinated project_id when deploying for the syncable platform.
Instead of relying context, the project_id is now ALWAYS derrived directly from the session, where users attach them selves to a project.

Author: Alex Holmberg

src/agent/tools/platform/create_deployment_config.rs

@@ -13,13 +13,12 @@ use crate::platform::api::types::{
     build_cloud_runner_config_v2,
 };
 use crate::platform::api::{PlatformApiClient, PlatformApiError};
+use crate::platform::session::PlatformSession;
 use std::str::FromStr;
 
 /// Arguments for the create deployment config tool
 #[derive(Debug, Deserialize)]
 pub struct CreateDeploymentConfigArgs {
-    /// The project UUID
-    pub project_id: String,
     /// Service name for the deployment
     pub service_name: String,
     /// Repository ID from GitHub integration
@@ -102,7 +101,6 @@ A deployment config defines how to build and deploy a service, including:
 - Auto-deploy settings
 
 **Required Parameters:**
-- project_id: The project UUID
 - service_name: Name for the service (lowercase, hyphens allowed)
 - repository_id: GitHub repository ID (from platform GitHub integration)
 - repository_full_name: Full repo name like "owner/repo"
@@ -138,10 +136,6 @@ A deployment config defines how to build and deploy a service, including:
             parameters: json!({
                 "type": "object",
                 "properties": {
-                    "project_id": {
-                        "type": "string",
-                        "description": "The UUID of the project"
-                    },
                     "service_name": {
                         "type": "string",
                         "description": "Name for the service (lowercase, hyphens allowed)"
@@ -218,27 +212,38 @@ A deployment config defines how to build and deploy a service, including:
                     }
                 },
                 "required": [
-                    "project_id", "service_name", "repository_id", "repository_full_name",
+                    "service_name", "repository_id", "repository_full_name",
                     "port", "branch", "target_type", "provider", "environment_id"
                 ]
             }),
         }
     }
 
     async fn call(&self, args: Self::Args) -> Result<Self::Output, Self::Error> {
-        // Validate required fields
-        if args.project_id.trim().is_empty() {
+        // Load project_id from session (authoritative source — prevents stale IDs from LLM context)
+        let session = match PlatformSession::load() {
+            Ok(s) => s,
+            Err(_) => {
+                return Ok(format_error_for_llm(
+                    "create_deployment_config",
+                    ErrorCategory::InternalError,
+                    "Failed to load platform session",
+                    Some(vec!["Try authenticating with `sync-ctl auth login`"]),
+                ));
+            }
+        };
+
+        if !session.is_project_selected() {
             return Ok(format_error_for_llm(
                 "create_deployment_config",
                 ErrorCategory::ValidationFailed,
-                "project_id cannot be empty",
-                Some(vec![
-                    "Use list_projects to find valid project IDs",
-                    "Use current_context to get the selected project",
-                ]),
+                "No project selected",
+                Some(vec!["Use select_project to choose a project first"]),
             ));
         }
 
+        let project_id = session.project_id.clone().unwrap_or_default();
+
         if args.service_name.trim().is_empty() {
             return Ok(format_error_for_llm(
                 "create_deployment_config",
@@ -316,7 +321,7 @@ A deployment config defines how to build and deploy a service, including:
             if let Some(ref provider) = provider_enum {
                 if matches!(provider, CloudProvider::Gcp | CloudProvider::Azure) {
                     if let Ok(credential) = client
-                        .check_provider_connection(provider, &args.project_id)
+                        .check_provider_connection(provider, &project_id)
                         .await
                     {
                         if let Some(cred) = credential {
@@ -351,7 +356,7 @@ A deployment config defines how to build and deploy a service, including:
         // Note: Send both field name variants (dockerfile/dockerfilePath, context/buildContext)
         // for backend compatibility - different endpoints may expect different field names
         let request = CreateDeploymentConfigRequest {
-            project_id: args.project_id.clone(),
+            project_id,
             service_name: args.service_name.clone(),
             repository_id: args.repository_id,
             repository_full_name: args.repository_full_name.clone(),

src/agent/tools/platform/trigger_deployment.rs

@@ -9,12 +9,11 @@ use serde_json::json;
 
 use crate::agent::tools::error::{ErrorCategory, format_error_for_llm};
 use crate::platform::api::{PlatformApiClient, PlatformApiError, TriggerDeploymentRequest};
+use crate::platform::session::PlatformSession;
 
 /// Arguments for the trigger deployment tool
 #[derive(Debug, Deserialize)]
 pub struct TriggerDeploymentArgs {
-    /// The project ID for the deployment
-    pub project_id: String,
     /// The deployment config ID to use
     pub config_id: String,
     /// Optional specific commit SHA to deploy
@@ -56,13 +55,12 @@ Starts a new deployment for the specified config. Returns a task ID that can be
 used to monitor deployment progress with `get_deployment_status`.
 
 **Parameters:**
-- project_id: The project UUID
-- config_id: The deployment config ID (get from list_deployment_configs)
+- config_id: The deployment config ID (get from list_deployment_configs or create_deployment_config)
 - commit_sha: Optional specific commit to deploy (defaults to latest on branch)
 
 **Prerequisites:**
 - User must be authenticated via `sync-ctl auth login`
-- A deployment config must exist for the project
+- A deployment config must exist (use create_deployment_config first if needed)
 
 **Use Cases:**
 - Deploy the latest code from a branch
@@ -77,50 +75,57 @@ used to monitor deployment progress with `get_deployment_status`.
             parameters: json!({
                 "type": "object",
                 "properties": {
-                    "project_id": {
-                        "type": "string",
-                        "description": "The UUID of the project"
-                    },
                     "config_id": {
                         "type": "string",
-                        "description": "The deployment config ID (from list_deployment_configs)"
+                        "description": "The deployment config ID (from list_deployment_configs or create_deployment_config)"
                     },
                     "commit_sha": {
                         "type": "string",
                         "description": "Optional: specific commit SHA to deploy (defaults to latest)"
                     }
                 },
-                "required": ["project_id", "config_id"]
+                "required": ["config_id"]
             }),
         }
     }
 
     async fn call(&self, args: Self::Args) -> Result<Self::Output, Self::Error> {
-        // Validate project_id
-        if args.project_id.trim().is_empty() {
+        // Validate config_id
+        if args.config_id.trim().is_empty() {
             return Ok(format_error_for_llm(
                 "trigger_deployment",
                 ErrorCategory::ValidationFailed,
-                "project_id cannot be empty",
+                "config_id cannot be empty",
                 Some(vec![
-                    "Use list_projects to find valid project IDs",
-                    "Use select_project to set the current project context",
+                    "Use list_deployment_configs to find available deployment configs",
                 ]),
             ));
         }
 
-        // Validate config_id
-        if args.config_id.trim().is_empty() {
+        // Load project_id from session (authoritative source)
+        let session = match PlatformSession::load() {
+            Ok(s) => s,
+            Err(_) => {
+                return Ok(format_error_for_llm(
+                    "trigger_deployment",
+                    ErrorCategory::InternalError,
+                    "Failed to load platform session",
+                    Some(vec!["Try authenticating with `sync-ctl auth login`"]),
+                ));
+            }
+        };
+
+        if !session.is_project_selected() {
             return Ok(format_error_for_llm(
                 "trigger_deployment",
                 ErrorCategory::ValidationFailed,
-                "config_id cannot be empty",
-                Some(vec![
-                    "Use list_deployment_configs to find available deployment configs",
-                ]),
+                "No project selected",
+                Some(vec!["Use select_project to choose a project first"]),
             ));
         }
 
+        let project_id = session.project_id.clone().unwrap_or_default();
+
         // Create the API client
         let client = match PlatformApiClient::new() {
             Ok(c) => c,
@@ -131,7 +136,7 @@ used to monitor deployment progress with `get_deployment_status`.
 
         // Build the request
         let request = TriggerDeploymentRequest {
-            project_id: args.project_id.clone(),
+            project_id,
             config_id: args.config_id.clone(),
             commit_sha: args.commit_sha.clone(),
         };

src/platform/api/client.rs

@@ -722,7 +722,7 @@ impl PlatformApiClient {
         request: &TriggerDeploymentRequest,
     ) -> Result<TriggerDeploymentResponse> {
         log::debug!(
-            "Triggering deployment: POST /api/deployment-configs/deploy with projectId={}, configId={}",
+            "Triggering deployment: POST /api/deployment-configs/deploy with projectId={} configId={}",
             request.project_id,
             request.config_id
         );

src/platform/api/types.rs

@@ -335,7 +335,7 @@ pub struct CreateDeploymentConfigResponse {
 #[derive(Debug, Clone, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct TriggerDeploymentRequest {
-    /// Project ID for the deployment
+    /// The project UUID this config belongs to
     pub project_id: String,
     /// Deployment config ID to use
     pub config_id: String,

src/wizard/orchestrator.rs

@@ -525,8 +525,7 @@ pub async fn run_wizard(
 
     // Debug: Show trigger request
     log::debug!(
-        "Trigger request: projectId={}, configId={}",
-        trigger_request.project_id,
+        "Trigger request: configId={}",
         trigger_request.config_id
     );