Apps-MCP: minor changes (#4057)

## Changes
- fix maps-related prompt;
- fix inconsistent working dir; 
- fix confusing bits in tool descriptions.

Author: arsenyinfo

experimental/apps-mcp/cmd/init_template.go

@@ -309,25 +309,27 @@ After initialization:
 		tmpl.Writer.LogTelemetry(ctx)
 
 		// Determine actual output directory (template writes to subdirectory with project name)
-		actualOutputDir := outputDir
-		if actualOutputDir == "" {
-			actualOutputDir = name
+		actualOutputDir := name
+		if outputDir != "" {
+			actualOutputDir = filepath.Join(outputDir, name)
 		}
 
-		// Count files if we can
+		// Count files and get absolute path
 		fileCount := 0
-		if absPath, err := filepath.Abs(actualOutputDir); err == nil {
-			_ = filepath.Walk(absPath, func(path string, info os.FileInfo, err error) error {
-				if err == nil && !info.IsDir() {
-					fileCount++
-				}
-				return nil
-			})
+		absOutputDir, err := filepath.Abs(actualOutputDir)
+		if err != nil {
+			absOutputDir = actualOutputDir
 		}
-		cmdio.LogString(ctx, common.FormatScaffoldSuccess("appkit", actualOutputDir, fileCount))
+		_ = filepath.Walk(absOutputDir, func(path string, info os.FileInfo, err error) error {
+			if err == nil && !info.IsDir() {
+				fileCount++
+			}
+			return nil
+		})
+		cmdio.LogString(ctx, common.FormatScaffoldSuccess("appkit", absOutputDir, fileCount))
 
 		// Generate and print file tree structure
-		fileTree, err := generateFileTree(actualOutputDir)
+		fileTree, err := generateFileTree(absOutputDir)
 		if err == nil && fileTree != "" {
 			cmdio.LogString(ctx, "\nFile structure:")
 			cmdio.LogString(ctx, fileTree)

experimental/apps-mcp/lib/prompts/apps.tmpl

@@ -30,6 +30,8 @@ When building or planning a new Databricks app, use the following steps:
 
 invoke_databricks_cli 'experimental apps-mcp tools init-template --name my-app-name --description "My app description"'
 
+This creates a `my-app-name/` directory in the current working directory containing the app files.
+
 # Validation
 
 ⚠️ Always validate your app before deploying to production:

experimental/apps-mcp/lib/providers/clitools/invoke_databricks_cli.go

@@ -12,9 +12,9 @@ import (
 )
 
 // InvokeDatabricksCLI runs a Databricks CLI command and returns the output.
-func InvokeDatabricksCLI(ctx context.Context, command []string, workingDirectory string) (string, error) {
-	if len(command) == 0 {
-		return "", errors.New("command is required")
+func InvokeDatabricksCLI(ctx context.Context, args []string, workingDirectory string) (string, error) {
+	if len(args) == 0 {
+		return "", errors.New("args is required")
 	}
 
 	workspaceClient, err := middlewares.GetDatabricksClient(ctx)
@@ -26,7 +26,7 @@ func InvokeDatabricksCLI(ctx context.Context, command []string, workingDirectory
 
 	// GetCLIPath returns the path to the current CLI executable
 	cliPath := common.GetCLIPath()
-	cmd := exec.CommandContext(ctx, cliPath, command...)
+	cmd := exec.CommandContext(ctx, cliPath, args...)
 	cmd.Dir = workingDirectory
 	env := os.Environ()
 	env = append(env, "DATABRICKS_HOST="+host)

experimental/apps-mcp/lib/providers/clitools/provider.go

@@ -52,7 +52,7 @@ func (p *Provider) RegisterTools(server *mcpsdk.Server) error {
 	mcpsdk.AddTool(server,
 		&mcpsdk.Tool{
 			Name:        "databricks_configure_auth",
-			Description: "Configure authentication for Databricks. Only call when Databricks authentication has has failed to authenticate automatically or when the user explicitly asks for using a specific host or profile. Validates credentials and stores the authenticated client in the session.",
+			Description: "Configure authentication for Databricks. Only call when Databricks authentication has failed to authenticate automatically or when the user explicitly asks for using a specific host or profile. Validates credentials and stores the authenticated client in the session.",
 		},
 		func(ctx context.Context, req *mcpsdk.CallToolRequest, args ConfigureAuthInput) (*mcpsdk.CallToolResult, any, error) {
 			log.Debug(ctx, "databricks_configure_auth called")
@@ -89,7 +89,7 @@ func (p *Provider) RegisterTools(server *mcpsdk.Server) error {
 	mcpsdk.AddTool(server,
 		&mcpsdk.Tool{
 			Name:        "explore",
-			Description: "**REQUIRED DURING PLAN MODE** - Call this FIRST when planning ANY Databricks work. Use this to discover available workspaces, warehouses, and get workflow recommendations for your specific task. Even if you're just reading an assignment document, call this first. Especially important when task involves: creating Databricks projects/apps/pipelines/jobs, SQL pipelines or data transformation workflows, deploying code to multiple environments (dev/prod), or working with databricks.yml files. You DON'T need a workspace name - call this when starting ANY Databricks planning to understand workspace capabilities and recommended tooling before you create your plan.",
+			Description: "Discover available Databricks workspaces, warehouses, and get workflow recommendations. Call this FIRST when planning ANY Databricks work involving apps, pipelines, jobs, bundles, or SQL workflows. Returns workspace capabilities and recommended tooling.",
 		},
 		func(ctx context.Context, req *mcpsdk.CallToolRequest, args struct{}) (*mcpsdk.CallToolResult, any, error) {
 			log.Debug(ctx, "explore called")
@@ -104,16 +104,16 @@ func (p *Provider) RegisterTools(server *mcpsdk.Server) error {
 	// Register invoke_databricks_cli tool
 	type InvokeDatabricksCLIInput struct {
 		WorkingDirectory string   `json:"working_directory" jsonschema:"required" jsonschema_description:"The directory to run the command in."`
-		Args             []string `json:"args" jsonschema:"required" jsonschema_description:"The arguments to pass to the Databricks CLI command e.g. ['bundle', 'deploy'] or ['bundle', 'validate']. Do not include the 'databricks' prefix."`
+		Args             []string `json:"args" jsonschema:"required" jsonschema_description:"CLI arguments as array, e.g. [\"bundle\", \"deploy\"] or [\"bundle\", \"validate\", \"--target\", \"dev\"]. Do not include 'databricks' prefix."`
 	}
 
 	mcpsdk.AddTool(server,
 		&mcpsdk.Tool{
 			Name:        "invoke_databricks_cli",
-			Description: "Execute Databricks CLI command. Pass all arguments as a single string.",
+			Description: "Execute Databricks CLI command. Pass arguments as an array of strings.",
 		},
 		func(ctx context.Context, req *mcpsdk.CallToolRequest, args InvokeDatabricksCLIInput) (*mcpsdk.CallToolResult, any, error) {
-			log.Debugf(ctx, "invoke_databricks_cli called: args=%s, working_directory=%s", args.Args, args.WorkingDirectory)
+			log.Debugf(ctx, "invoke_databricks_cli called: args=%v, working_directory=%s", args.Args, args.WorkingDirectory)
 			result, err := InvokeDatabricksCLI(ctx, args.Args, args.WorkingDirectory)
 			if err != nil {
 				return nil, nil, err

experimental/apps-mcp/templates/appkit/template/{{.project_name}}/CLAUDE.md

@@ -503,6 +503,21 @@ npm run test:e2e:ui     # Run with Playwright UI
 
 - `SelectItem` cannot have `value=""`. Use sentinel value like `"all"` for "show all" options.
 
+### Map Libraries (react-leaflet)
+
+For maps with React 19, use react-leaflet v5:
+
+```bash
+npm install react-leaflet@^5.0.0 leaflet @types/leaflet
+```
+
+Import CSS in your component:
+```typescript
+import 'leaflet/dist/leaflet.css';
+```
+
+Note: react-leaflet v4.x requires React 18. Use v5 for React 19 compatibility.
+
 ### Best Practices:
 
 - Use shadcn/radix components (Button, Input, Card, etc.) for consistent UI