@tanagram/cli 0.1.9 → 0.1.11

package/commands/run.go

@@ -56,22 +56,21 @@ func Run() error {
 		return fmt.Errorf("failed to load cache: %w", err)
 	}
 
-	// Check each file for changes and auto-sync if needed
-	needsSync := false
+	// Check each file for changes and collect which files need syncing
+	var filesToSync []string
 	for _, file := range instructionFiles {
 		changed, err := cache.HasChanged(file)
 		if err != nil {
 			return fmt.Errorf("failed to check if %s changed: %w", file, err)
 		}
 		if changed {
-			needsSync = true
-			break
+			filesToSync = append(filesToSync, file)
 		}
 	}
 
-	// Auto-sync if any files changed
-	if needsSync {
-		fmt.Printf("\nSyncing policies with LLM (processing %d files in parallel)...\n", len(instructionFiles))
+	// Auto-sync only the files that changed
+	if len(filesToSync) > 0 {
+		fmt.Printf("\nSyncing policies with LLM (processing %d changed file(s) in parallel)...\n", len(filesToSync))
 
 		ctx := context.Background()
 
@@ -84,7 +83,7 @@ func Run() error {
 		}
 
 		// Channel to collect results
-		results := make(chan syncResult, len(instructionFiles))
+		results := make(chan syncResult, len(filesToSync))
 		var wg sync.WaitGroup
 
 		// Start spinner
@@ -103,15 +102,15 @@ func Run() error {
 					mu.Lock()
 					c := completed
 					mu.Unlock()
-					fmt.Printf("\r%s Processing files... (%d/%d completed)", chars[i%len(chars)], c, len(instructionFiles))
+					fmt.Printf("\r%s Processing files... (%d/%d completed)", chars[i%len(chars)], c, len(filesToSync))
 					i++
 					time.Sleep(100 * time.Millisecond)
 				}
 			}
 		}()
 
-		// Launch goroutines for each file
-		for _, file := range instructionFiles {
+		// Launch goroutines only for changed files
+		for _, file := range filesToSync {
 			wg.Add(1)
 			go func(file string) {
 				defer wg.Done()
@@ -162,7 +161,7 @@ func Run() error {
 			return fmt.Errorf("failed to save cache: %w", err)
 		}
 
-		fmt.Printf("\n✓ Synced %d policies from %d file(s)\n", totalPolicies, len(instructionFiles))
+		fmt.Printf("\n✓ Synced %d policies from %d changed file(s)\n", totalPolicies, len(filesToSync))
 	}
 
 	// Load all policies from cache

package/commands/sync.go

@@ -39,8 +39,26 @@ func Sync() error {
 		return fmt.Errorf("failed to load cache: %w", err)
 	}
 
-	// Parse and sync each file using LLM in parallel
-	fmt.Printf("\nSyncing policies with LLM (processing %d files in parallel)...\n", len(instructionFiles))
+	// Check which files have changed and need syncing
+	var filesToSync []string
+	for _, file := range instructionFiles {
+		changed, err := cache.HasChanged(file)
+		if err != nil {
+			return fmt.Errorf("failed to check if %s changed: %w", file, err)
+		}
+		if changed {
+			filesToSync = append(filesToSync, file)
+		}
+	}
+
+	// If no files changed, nothing to sync
+	if len(filesToSync) == 0 {
+		fmt.Println("✓ All instruction files are up to date")
+		return nil
+	}
+
+	// Parse and sync only changed files using LLM in parallel
+	fmt.Printf("\nSyncing policies with LLM (processing %d changed file(s) in parallel)...\n", len(filesToSync))
 
 	ctx := context.Background()
 
@@ -53,7 +71,7 @@ func Sync() error {
 	}
 
 	// Channel to collect results
-	results := make(chan syncResult, len(instructionFiles))
+	results := make(chan syncResult, len(filesToSync))
 	var wg sync.WaitGroup
 
 	// Start spinner
@@ -72,15 +90,15 @@ func Sync() error {
 				mu.Lock()
 				c := completed
 				mu.Unlock()
-				fmt.Printf("\r%s Processing files... (%d/%d completed)", chars[i%len(chars)], c, len(instructionFiles))
+				fmt.Printf("\r%s Processing files... (%d/%d completed)", chars[i%len(chars)], c, len(filesToSync))
 				i++
 				time.Sleep(100 * time.Millisecond)
 			}
 		}
 	}()
 
-	// Launch goroutines for each file
-	for _, file := range instructionFiles {
+	// Launch goroutines only for changed files
+	for _, file := range filesToSync {
 		wg.Add(1)
 		go func(file string) {
 			defer wg.Done()
@@ -132,7 +150,7 @@ func Sync() error {
 		return fmt.Errorf("failed to save cache: %w", err)
 	}
 
-	fmt.Printf("\n✓ Synced %d policies from %d file(s)\n", totalPolicies, len(instructionFiles))
+	fmt.Printf("\n✓ Synced %d policies from %d changed file(s)\n", totalPolicies, len(filesToSync))
 	return nil
 }
 

package/extractor/extractor.go

@@ -60,28 +60,33 @@ func ExtractPolicies(ctx context.Context, content string) ([]parser.Policy, erro
 
 // buildExtractionPrompt creates the prompt for policy extraction
 func buildExtractionPrompt(content string) string {
-	return fmt.Sprintf(`You are a code policy extraction system. Extract ONLY coding policies that apply when developers write and edit code.
-
-Analyze the following instruction file and extract policies that define:
-- Code structure and organization rules
-- Coding standards and style guidelines
-- Language-specific patterns and practices
-- File naming and directory conventions
-- Error handling requirements
-- Security practices in code
-- Testing requirements
-- Performance optimization rules
-- Import/dependency management
-
-DO NOT extract:
+	return fmt.Sprintf(`You are a code policy extraction system. Extract ONLY policies that define anti-patterns or bad practices that can be detected by reading code.
+
+IMPORTANT: Extract policies about what the CODE should or shouldn't look like. DO NOT extract policies that require running commands, executing tools, or performing actions.
+
+Extract policies that define CODE ANTI-PATTERNS:
+- Code structure and organization anti-patterns (e.g., "Don't use deeply nested functions")
+- Coding standards violations (e.g., "Don't use var, use let/const")
+- Language-specific anti-patterns (e.g., "Don't use 'any' type in TypeScript")
+- Prohibited code patterns (e.g., "Don't hardcode API keys")
+- Security anti-patterns in code (e.g., "Don't use eval()")
+- Error handling anti-patterns (e.g., "Don't swallow exceptions silently")
+- Import/dependency anti-patterns (e.g., "Don't import entire lodash library")
+
+DO NOT extract policies that require ACTIONS or COMMANDS:
+- Running tests (e.g., "Run pytest before committing")
+- Running linters or formatters (e.g., "Run black on all Python files")
+- Building or compiling (e.g., "Build the project to verify")
+- Running scripts or commands (e.g., "Execute database migrations")
+- Using specific tools (e.g., "Use ruff format" - this requires running ruff)
 - Agent behavior instructions (how AI agents should operate)
 - Workflow or process instructions
-- Tool usage instructions
 - Communication style guidelines
 - Project management policies
-- Non-code related guidelines
 
-For each code policy, provide:
+CRITICAL: If a policy would require the agent to run a command or tool to fix a violation, DO NOT extract it. Only extract policies about code patterns that can be fixed by editing code directly.
+
+For each code anti-pattern policy, provide:
 1. Name: Short descriptive name (2-5 words)
 2. Message: The full policy text or a clear summary
 3. OriginalText: The exact text from the file

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tanagram/cli",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "Tanagram - Catch sloppy code before it ships",
   "main": "index.js",
   "bin": {