.gitea/workflows/auto-complete-todo.yml

@@ -0,0 +1,37 @@
+name: 'Auto-complete TODO'
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  move-todo:
+    runs-on: ubuntu-gitea
+    steps:
+      - name: 'Clone PR branch, move TODO, push update'
+        env:
+          PAT_TOKEN: ${{ secrets.PAT_TOKEN }}
+          SERVER_URL: ${{ github.server_url }}
+          REPO_OWNER: ${{ github.repository_owner }}
+          REPO_NAME: ${{ github.event.pull_request.head.repo.name }}
+          PR_BRANCH: ${{ github.head_ref }}
+        run: |
+          git clone https://${PAT_TOKEN}@${SERVER_URL}/${REPO_OWNER}/${REPO_NAME}.git pr-temp || exit 1
+          cd pr-temp
+          git checkout ${PR_BRANCH}
+          BRANCH=$(git rev-parse --abbrev-ref HEAD)
+          TODO_NAME="${BRANCH#feature/}.md"
+          if [[ "${BRANCH}" == feature/* && -f todo/queued/${TODO_NAME} ]]; then
+            mkdir -p todo/completed
+            mv todo/queued/${TODO_NAME} todo/completed/
+            git config user.name 'Gitea Actions Bot'
+            git config user.email 'actions@noreply.local'
+            git add todo/
+            git commit -m "chore: auto-complete ${TODO_NAME} via Gitea Actions"
+            git push https://${PAT_TOKEN}@${SERVER_URL}/${REPO_OWNER}/${REPO_NAME}.git ${PR_BRANCH}
+            echo "✅ Moved todo/queued/${TODO_NAME} → completed/"
+          else
+            echo "ℹ️ No action: branch='${BRANCH}', expected 'feature/*' with todo/queued/${TODO_NAME}"
+          fi
+          cd ..
+          rm -rf pr-temp

.gitea/workflows/ci.yml

@@ -40,10 +40,10 @@ jobs:
         run: |
           go tool cover -func=coverage.out | tail -1
 
-#      - name: Enforce coverage threshold
-#        run: |
-#          PCT=$(go tool cover -func=coverage.out | tail -1 | awk '{print $3}' | tr -d '%')
-#          awk "BEGIN { exit ($PCT < 65) }" || (echo "Coverage ${PCT}% is below 65%" && exit 1)
+      - name: Enforce coverage threshold
+        run: |
+          PCT=$(go tool cover -func=coverage.out | tail -1 | awk '{print $3}' | tr -d '%')
+          awk "BEGIN { exit ($PCT < 65) }" || (echo "Coverage ${PCT}% is below 65%" && exit 1)
 
   lint:
     name: Lint

.golangci.yml

@@ -1,14 +1,18 @@
 version: "2"
+
 linters:
   default: standard
   enable:
     - misspell
+
   settings:
     errcheck:
       check-type-assertions: true
       check-blank: false
+
 formatters:
   enable:
     - gofmt
+
 run:
   timeout: 5m

.grok/settings.json

@@ -1,3 +0,0 @@
-{
-  "model": "grok-code-fast-1"
-}

CHANGELOG.md

@@ -1,53 +0,0 @@
-## [v0.1.9] - 2026-03-04
-
-Grokkit gets a quick-query upgrade—because who has time for chit-chat?
-
-### Added
-- Implement `query` command in cmd/query.go for non-interactive Grok queries focused on programming.
-- Add wordy flag for detailed responses in query command.
-- Add .grok/settings.json with fast model configuration.
-- Set default model for query in config.go.
-- Add entry for query command to commands list in README.
-- Create new section in README with query command usage examples and features.
-- Add spec for grokkit non-interactive query tool in TODO.
-- Add detailed description, examples, and ROI for `query` feature in TODO.
-- Introduce initial CHANGELOG.md with v0.1.8 entries.
-
-### Changed
-- Update root.go to include queryCmd.
-- Reorder and update queued task list in TODO with new entries like non-interactive-query.md.
-- Move changelog.md to completed tasks in TODO with version note.
-- Standardize link formats and list markers in README.md.
-- Rename TODO entry from "grokkit query Go tools integration" to "grokkit query Simple Query Tool".
-- Revise TODO description to focus on one-shot prompt/answer tool for concise queries.
-# Changelog
-
-All notable changes to this project will be documented in this file.
-
-## [v0.1.8] - 2026-03-03
-
-Another step towards automated bliss, with AI changelogs and Git taking the safety wheel.
-
-### Added
-- Add release.sh script for automating releases with version validation, tagging, changelog generation, committing, and pushing.
-- Add grokkit changelog command with flags for version, since, stdout, and commit reminder.
-- Add git helpers for retrieving latest/previous tags and formatted logs since a reference.
-- Add tests for changelog message building and full construction.
-- Add todo/README.md as a table of contents for queued and completed tasks.
-- Add section in README.md documenting grokkit changelog command with usage examples.
-- Add detailed Git workflow for managing changes in README.md.
-- Add detail in changelog.md about using generated CHANGELOG.md for Gitea release notes.
-
-### Changed
-- Change version flag shorthand from "v" to "V" in changelog command and update test assertion.
-- Disable coverage threshold enforcement in CI workflow by commenting out the 65% minimum step.
-- Update README.md to include grokkit changelog in the command features list.
-- Reprioritize queued tasks in README.md to place changelog first, followed by interactive-agent, make, tea, and gotools.
-- Update priorities in individual queued/*.md files to match new order.
-- Adjust priorities for features including audit, changelog, gotools, interactive-agent, and rg in todo docs.
-- Replace file backups with Git-based change management in safety features.
-- Remove mentions of backups from README.md, cmd/lint.go, ARCHITECTURE.md, and TROUBLESHOOTING.md.
-- Update troubleshooting guide with Git rollback instructions.
-- Modify feature lists and safety descriptions to emphasize Git integration.
-- Remove automatic .bak backup creation from edit, docs, lint, testgen, and agent commands.
-- Update README.md, architecture docs, troubleshooting, TODOs, and tests to reflect removal of backups.

README.md

@@ -38,13 +38,11 @@ grokkit version
 
 - [Commands](#commands)
   - [chat](#-grokkit-chat)
-  - [query](#-grokkit-query)
   - [edit](#-grokkit-edit-file-instruction)
   - [commit / commitmsg](#-grokkit-commitmsg)
   - [review](#-grokkit-review)
   - [pr-describe](#-grokkit-pr-describe)
   - [history](#-grokkit-history)
-  - [changelog](#-grokkit-changelog)
   - [lint](#-grokkit-lint-file)
   - [docs](#-grokkit-docs-file)
   - [testgen](#-grokkit-testgen)
@@ -73,27 +71,8 @@ grokkit chat --debug            # Enable debug logging
 - History is saved automatically between sessions
 - Use `--debug` to see API request timing
 
-### 🤖 `grokkit query`
-
-One-shot technical question answering. Perfect for quick programming or engineering questions.
-
-```bash
-# Basic usage
-grokkit query "How do I sort a slice of structs by a field in Go?"
-
-# Longer, more detailed answer
-grokkit query --wordy "Explain how Go's context package works with cancellation"
-```
-Features:
-
-Default mode is concise, factual, and actionable
---wordy flag gives longer, more explanatory answers
-Uses the fast non-reasoning model by default for speed
-No persistent history or interactive chat UI
-
-
 ### ✏️ `grokkit edit FILE "instruction"`
-AI-powered file editing with preview.
+AI-powered file editing with preview and automatic backups.
 
 ```bash
 # Basic usage
@@ -110,6 +89,7 @@ grokkit edit utils.go "add detailed docstrings to all exported functions"
 ```
 
 **Safety features:**
+- Creates `.bak` backup before any changes
 - Shows preview with diff-style output
 - Requires confirmation before applying
 - Uses silent streaming (no console spam)
@@ -178,24 +158,6 @@ Summarize recent git commits.
 grokkit history                 # Last 10 commits
 ```
 
-### 🗒️ `grokkit changelog`
-
-Generate a clean `CHANGELOG.md` section from git history, designed specifically so the output can be pasted directly into Gitea release notes.
-
-```bash
-# 1. Create your version tag first
-git tag v0.2.0
-
-# 2. Generate preview + write (with safety confirmation)
-grokkit changelog
-
-# 3. Output ONLY the new section (perfect for Gitea "Release notes")
-grokkit changelog --stdout
-
-# 4. Write file + get commit reminder
-grokkit changelog --commit
-```
-
 ### 📖 `grokkit docs <file> [file...]`
 Generate language-appropriate documentation comments using Grok AI.
 
@@ -227,6 +189,7 @@ grokkit docs app.py -m grok-4
 | Shell | Shell comments (`# function: desc, # Args: ...`) |
 
 **Safety features:**
+- Creates `.bak` backup before any changes
 - Shows first 50 lines of documented code as preview
 - Requires confirmation (unless `--auto-apply`)
 	
@@ -239,7 +202,7 @@ grokkit docs app.py -m grok-4
 - Python: Pytest with `@parametrize`.
 - C: Check framework suites.
 - C++: Google Test `EXPECT_*`.
-- Boosts coverage; safe preview.
+- Boosts coverage; safe preview/backup.
 	
 **CLI examples**:
 ```bash
@@ -250,6 +213,7 @@ grokkit testgen foo.c bar.cpp
 	
 **Safety features**:
 - Lang detection via `internal/linter`.
+- Creates `test_*.bak` backups.
 - Unified diff preview.
 - Y/N (--yes auto).
 	
@@ -289,42 +253,11 @@ grokkit lint script.rb -m grok-4
 - **Shell** (shellcheck)
 
 **Safety features:**
+- Creates `.bak` backup before changes
 - Shows preview of fixes
 - Verifies fixes by re-running linter
 - Requires confirmation (unless `--auto-fix`)
 
-## Safety & Change Management
-
-Grokkit is designed to work seamlessly with Git. Rather than creating redundant `.bak` files, we lean on Git's powerful version control to manage changes and rollbacks.
-
-### The Safety Workflow
-
-1.  **Preview**: Every command that modifies files (like `edit`, `lint`, `docs`) shows a diff-style preview first.
-2.  **Confirm**: You must explicitly confirm (`y/N`) before any changes are written to disk.
-3.  **Git Integration**: Use Git to manage the "pre-staged," "staged," and "committed" degrees of change.
-
-### Managing Undesired Changes
-
-If you've applied a change that you don't like, Git makes it easy to roll back:
-
-- **Unstaged changes**: If you haven't `git add`-ed the changes yet:
-  ```bash
-  git restore <file>
-  ```
-- **Staged changes**: If you've already staged the changes:
-  ```bash
-  git restore --staged <file>
-  git restore <file>
-  ```
-- **Committed changes**: If you've already committed the changes:
-  ```bash
-  git revert HEAD
-  # or to reset to a previous state:
-  git reset --hard HEAD~1
-  ```
-
-By using Git, you have a complete audit trail and multiple levels of undo, ensuring your codebase remains stable even when experimenting with AI-driven refactors.
-
 ## Configuration
 
 ### Environment Variables
@@ -519,7 +452,7 @@ grokkit review -v
 - ✅ **Persistent chat history** - Never lose your conversations
 - ✅ **Configurable parameters** - Temperature, timeout, model selection
 - ✅ **Shell completions** - Bash, Zsh, Fish, PowerShell
-- ✅ **Safe file editing** - Preview and confirmation, leverage git for rollbacks
+- ✅ **Safe file editing** - Automatic backups, preview, confirmation
 - ✅ **Git workflow integration** - Commit messages, reviews, PR descriptions
 - ✅ **Multi-language linting** - 9 languages supported with AI-powered fixes
 - ✅ **AI documentation generation** - 8 doc styles (godoc, PEP 257, Doxygen, JSDoc, rustdoc, YARD, Javadoc, shell)

cmd/agent.go

@@ -73,13 +73,15 @@ var agentCmd = &cobra.Command{
 		for i, file := range files {
 			color.Yellow("[%d/%d] → %s", i+1, len(files), file)
 
-			// nolint:gosec // intentional file read from user input
 			original, err := os.ReadFile(file)
 			if err != nil {
 				color.Red("Could not read %s", file)
 				continue
 			}
 
+			backupPath := file + ".bak"
+			_ = os.WriteFile(backupPath, original, 0644)
+
 			messages := []map[string]string{
 				{"role": "system", "content": "You are an expert programmer. Remove all unnecessary comments including last modified timestamps and ownership comments. Return clean, complete file content with no explanations, markdown, diffs, or extra text."},
 				{"role": "user", "content": fmt.Sprintf("File: %s\n\nOriginal content:\n%s\n\nTask: %s", filepath.Base(file), original, instruction)},
@@ -105,12 +107,12 @@ var agentCmd = &cobra.Command{
 				if answer == "a" {
 					applyAll = true
 				} else if answer != "y" {
-					color.Yellow("Skipped %s", file)
+					color.Yellow("Skipped %s (backup kept)", file)
 					continue
 				}
 			}
 
-			_ = os.WriteFile(file, []byte(newContent), 0600)
+			_ = os.WriteFile(file, []byte(newContent), 0644)
 			color.Green("✅ Applied %s", file)
 		}
 

cmd/changelog.go

@@ -1,143 +0,0 @@
-package cmd
-
-import (
-	"fmt"
-	"os"
-	"strings"
-	"time"
-
-	"github.com/fatih/color"
-	"github.com/spf13/cobra"
-	"gmgauthier.com/grokkit/config"
-	"gmgauthier.com/grokkit/internal/git"
-)
-
-var changelogCmd = &cobra.Command{
-	Use:   "changelog",
-	Short: "Generate CHANGELOG.md section from git history for Gitea releases",
-	Long:  `AI-generated changelog using only Added/Changed/Fixed. Designed so the output can be pasted directly into Gitea release notes.`,
-	Run:   runChangelog,
-}
-
-func init() {
-	changelogCmd.Flags().String("since", "", "Start from this tag/ref (default: previous tag)")
-	changelogCmd.Flags().StringP("version", "V", "", "Override version for header (default: latest git tag)")
-	changelogCmd.Flags().Bool("stdout", false, "Print ONLY the new section (ideal for Gitea release notes)")
-	changelogCmd.Flags().Bool("commit", false, "After writing, remind to run grokkit commit")
-	rootCmd.AddCommand(changelogCmd)
-}
-
-func runChangelog(cmd *cobra.Command, _ []string) {
-	stdout, _ := cmd.Flags().GetBool("stdout")
-	doCommit, _ := cmd.Flags().GetBool("commit")
-	version, _ := cmd.Flags().GetString("version")
-	since, _ := cmd.Flags().GetString("since")
-
-	if !git.IsRepo() {
-		color.Red("Not inside a git repository")
-		return
-	}
-
-	// Version from tag you set (or override)
-	if version == "" {
-		var err error
-		version, err = git.LatestTag()
-		if err != nil {
-			color.Red("No git tags found. Create one first: git tag vX.Y.Z")
-			return
-		}
-	}
-
-	// Since ref (defaults to previous tag)
-	if since == "" {
-		if prev, err := git.PreviousTag(version); err == nil {
-			since = prev
-		} else {
-			since = "HEAD~100" // safe first-release fallback
-		}
-	}
-
-	logOutput, err := git.LogSince(since)
-	if err != nil || strings.TrimSpace(logOutput) == "" {
-		color.Yellow("No new commits since last tag.")
-		return
-	}
-
-	// Grok generation (strong prompt for your exact format)
-	client := newGrokClient()
-	messages := buildChangelogMessages(logOutput, version)
-	model := config.GetModel("changelog", "")
-
-	color.Yellow("Asking Grok to categorize changes...")
-	section := client.Stream(messages, model)
-
-	date := time.Now().Format("2006-01-02")
-	newSection := fmt.Sprintf("## [%s] - %s\n\n%s\n", version, date, section)
-
-	if stdout {
-		fmt.Print(newSection)
-		return
-	}
-
-	// Build full file (prepend or create)
-	content := buildFullChangelog(newSection)
-
-	// Preview + safety confirm (exactly like commit/review)
-	color.Cyan("\n--- Proposed CHANGELOG.md update ---\n%s\n--------------------------------", content)
-	var confirm string
-	color.Yellow("Write this to CHANGELOG.md? (y/n): ")
-	_, _ = fmt.Scanln(&confirm) // we don't care about scan errors here
-
-	if confirm != "y" && confirm != "Y" {
-		color.Yellow("Aborted.")
-		return
-	}
-
-	if err := os.WriteFile("CHANGELOG.md", []byte(content), 0600); err != nil {
-		color.Red("Failed to write CHANGELOG.md")
-		return
-	}
-
-	color.Green("✅ CHANGELOG.md updated with version %s", version)
-
-	if doCommit {
-		color.Yellow("Run `grokkit commit` (or `git add CHANGELOG.md && git commit`) to stage it.")
-	}
-}
-
-func buildChangelogMessages(log, version string) []map[string]string {
-	return []map[string]string{
-		{
-			"role": "system",
-			"content": `You are an expert technical writer.
-Generate a changelog section using **only** these headings (include only if content exists):
-### Added
-### Changed
-### Fixed
-
-Rules:
-- Start directly with the section headings (no extra header — we add it).
-- One optional short summary sentence at the very top (light humour OK here only).
-- Every bullet must be factual, imperative, one clear action per line.
-- Be concise. No marketing language or explanations.
-Output ONLY clean markdown.`,
-		},
-		{
-			"role":    "user",
-			"content": fmt.Sprintf("Version: %s\n\nCommit history:\n%s", version, log),
-		},
-	}
-}
-
-func buildFullChangelog(newSection string) string {
-	existing, err := os.ReadFile("CHANGELOG.md")
-	if err != nil {
-		// File doesn't exist (or unreadable) → create a new changelog with a header
-		return `# Changelog
-
-All notable changes to this project will be documented in this file.
-
-` + newSection
-	}
-	return newSection + string(existing)
-}

cmd/changelog_test.go

@@ -1,110 +0,0 @@
-package cmd
-
-import (
-	"os"
-	"strings"
-	"testing"
-
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-)
-
-func TestBuildChangelogMessages(t *testing.T) {
-	t.Parallel()
-
-	log := `feat: add changelog command
-
-Implements #1
----
-fix: typo in docs
----`
-	version := "v0.2.0"
-
-	messages := buildChangelogMessages(log, version)
-
-	require.Len(t, messages, 2)
-	assert.Equal(t, "system", messages[0]["role"])
-	assert.Contains(t, messages[0]["content"], "Generate a changelog section")
-	assert.Contains(t, messages[0]["content"], "### Added")
-	assert.Contains(t, messages[0]["content"], "### Changed")
-	assert.Contains(t, messages[0]["content"], "### Fixed")
-	assert.Equal(t, "user", messages[1]["role"])
-	assert.Contains(t, messages[1]["content"], "Version: v0.2.0")
-	assert.Contains(t, messages[1]["content"], log)
-}
-
-func TestBuildFullChangelog(t *testing.T) {
-	t.Parallel()
-
-	newSection := "## [v0.2.0] - 2026-03-03\n\n### Added\n- changelog command\n"
-
-	t.Run("creates new file with header", func(t *testing.T) {
-		t.Parallel()
-		// nolint:tparallel // os.Chdir affects the entire process
-		tmpDir := t.TempDir()
-
-		originalWd, err := os.Getwd()
-		require.NoError(t, err)
-		t.Cleanup(func() {
-			require.NoError(t, os.Chdir(originalWd))
-		})
-		require.NoError(t, os.Chdir(tmpDir))
-
-		result := buildFullChangelog(newSection)
-
-		assert.Contains(t, result, "# Changelog")
-		assert.Contains(t, result, "All notable changes to this project will be documented in this file.")
-		assert.Contains(t, result, newSection)
-	})
-
-	t.Run("prepends to existing file", func(t *testing.T) {
-		t.Parallel()
-		// nolint:tparallel // os.Chdir affects the entire process
-		tmpDir := t.TempDir()
-
-		originalWd, err := os.Getwd()
-		require.NoError(t, err)
-		t.Cleanup(func() {
-			require.NoError(t, os.Chdir(originalWd))
-		})
-		require.NoError(t, os.Chdir(tmpDir))
-
-		// Simulate existing changelog
-		existing := `# Changelog
-
-All notable changes to this project will be documented in this file.
-
-## [v0.1.0] - 2025-01-01
-
-### Fixed
-- old bug
-`
-		require.NoError(t, os.WriteFile("CHANGELOG.md", []byte(existing), 0600))
-
-		result := buildFullChangelog(newSection)
-
-		assert.True(t, strings.HasPrefix(result, newSection), "new version section must be prepended at the top")
-		assert.Contains(t, result, "old bug")
-	})
-}
-
-func TestChangelogCmd_Flags(t *testing.T) {
-	t.Parallel()
-
-	require.NotNil(t, changelogCmd)
-
-	stdoutFlag := changelogCmd.Flags().Lookup("stdout")
-	require.NotNil(t, stdoutFlag)
-	assert.Equal(t, "Print ONLY the new section (ideal for Gitea release notes)", stdoutFlag.Usage)
-
-	versionFlag := changelogCmd.Flags().Lookup("version")
-	require.NotNil(t, versionFlag)
-	assert.Equal(t, "V", versionFlag.Shorthand)
-
-	commitFlag := changelogCmd.Flags().Lookup("commit")
-	require.NotNil(t, commitFlag)
-	assert.Equal(t, "After writing, remind to run grokkit commit", commitFlag.Usage)
-
-	sinceFlag := changelogCmd.Flags().Lookup("since")
-	require.NotNil(t, sinceFlag)
-}

cmd/chat.go

@@ -21,7 +21,6 @@ type ChatHistory struct {
 
 func loadChatHistory() []map[string]string {
 	histFile := getChatHistoryFile()
-	// nolint:gosec // intentional file read from config/home
 	data, err := os.ReadFile(histFile)
 	if err != nil {
 		return nil
@@ -41,7 +40,7 @@ func saveChatHistory(messages []map[string]string) error {
 	if err != nil {
 		return err
 	}
-	return os.WriteFile(histFile, data, 0600)
+	return os.WriteFile(histFile, data, 0644)
 }
 
 func getChatHistoryFile() string {
@@ -55,7 +54,7 @@ func getChatHistoryFile() string {
 		home = "."
 	}
 	histDir := filepath.Join(home, ".config", "grokkit")
-	_ = os.MkdirAll(histDir, 0750) // Ignore error, WriteFile will catch it
+	_ = os.MkdirAll(histDir, 0755) // Ignore error, WriteFile will catch it
 	return filepath.Join(histDir, "chat_history.json")
 }
 

cmd/chat_test.go

@@ -91,11 +91,11 @@ func TestLoadChatHistory_InvalidJSON(t *testing.T) {
 
 	// Create invalid JSON file
 	histDir := filepath.Join(tmpDir, ".config", "grokkit")
-	if err := os.MkdirAll(histDir, 0750); err != nil {
+	if err := os.MkdirAll(histDir, 0755); err != nil {
 		t.Fatalf("MkdirAll() error: %v", err)
 	}
 	histFile := filepath.Join(histDir, "chat_history.json")
-	if err := os.WriteFile(histFile, []byte("invalid json{{{"), 0600); err != nil {
+	if err := os.WriteFile(histFile, []byte("invalid json{{{"), 0644); err != nil {
 		t.Fatalf("WriteFile() error: %v", err)
 	}
 

cmd/commit.go

@@ -15,7 +15,7 @@ var commitCmd = &cobra.Command{
 	Run:   runCommit,
 }
 
-func runCommit(cmd *cobra.Command, _ []string) {
+func runCommit(cmd *cobra.Command, args []string) {
 	diff, err := gitRun([]string{"diff", "--cached", "--no-color"})
 	if err != nil {
 		color.Red("Failed to get staged changes: %v", err)
@@ -45,7 +45,6 @@ func runCommit(cmd *cobra.Command, _ []string) {
 		return
 	}
 
-	// nolint:gosec // intentional subprocess for git operation
 	if err := exec.Command("git", "commit", "-m", msg).Run(); err != nil {
 		color.Red("Git commit failed")
 	} else {

cmd/completion.go

@@ -49,7 +49,6 @@ PowerShell:
 	ValidArgs:             []string{"bash", "zsh", "fish", "powershell"},
 	Args:                  cobra.MatchAll(cobra.ExactArgs(1), cobra.OnlyValidArgs),
 	Run: func(cmd *cobra.Command, args []string) {
-		// nolint:gosec // intentional subprocess for shell completion
 		var err error
 		switch args[0] {
 		case "bash":

cmd/docs.go

@@ -32,6 +32,7 @@ Supported doc styles:
   Shell       Shell comments (# function: ...)
 
 Safety features:
+- Creates .bak backup before modifying files
 - Shows preview of changes before applying
 - Requires confirmation unless --auto-apply is used`,
 	Args: cobra.MinimumNArgs(1),
@@ -68,7 +69,6 @@ func processDocsFile(client grok.AIClient, model, filePath string) {
 		return
 	}
 
-	// nolint:gosec // intentional file read from user input
 	originalContent, err := os.ReadFile(filePath)
 	if err != nil {
 		logger.Error("failed to read file", "file", filePath, "error", err)
@@ -100,7 +100,7 @@ func processDocsFile(client grok.AIClient, model, filePath string) {
 	if len(lines) < previewLines {
 		previewLines = len(lines)
 	}
-	for i := range previewLines {
+	for i := 0; i < previewLines; i++ {
 		fmt.Println(lines[i])
 	}
 	if len(lines) > previewLines {
@@ -124,7 +124,16 @@ func processDocsFile(client grok.AIClient, model, filePath string) {
 		}
 	}
 
-	if err := os.WriteFile(filePath, []byte(documented), 0600); err != nil {
+	// Create backup
+	backupPath := filePath + ".bak"
+	if err := os.WriteFile(backupPath, originalContent, 0644); err != nil {
+		logger.Error("failed to create backup", "file", filePath, "error", err)
+		color.Red("❌ Failed to create backup: %v", err)
+		return
+	}
+	logger.Info("backup created", "backup", backupPath)
+
+	if err := os.WriteFile(filePath, []byte(documented), 0644); err != nil {
 		logger.Error("failed to write documented file", "file", filePath, "error", err)
 		color.Red("❌ Failed to write file: %v", err)
 		return
@@ -132,6 +141,7 @@ func processDocsFile(client grok.AIClient, model, filePath string) {
 
 	logger.Info("documentation applied successfully", "file", filePath)
 	color.Green("✅ Documentation applied: %s", filePath)
+	color.Cyan("💾 Original saved to: %s", backupPath)
 }
 
 func buildDocsMessages(language, code string) []map[string]string {

cmd/edit.go

@@ -15,7 +15,7 @@ import (
 
 var editCmd = &cobra.Command{
 	Use:   "edit FILE INSTRUCTION",
-	Short: "Edit a file in-place with Grok (safe preview)",
+	Short: "Edit a file in-place with Grok (safe preview + backup)",
 	Args:  cobra.ExactArgs(2),
 	Run: func(cmd *cobra.Command, args []string) {
 		filePath := args[0]
@@ -35,7 +35,6 @@ var editCmd = &cobra.Command{
 			os.Exit(1)
 		}
 
-		// nolint:gosec // intentional file read from user input
 		original, err := os.ReadFile(filePath)
 		if err != nil {
 			logger.Error("failed to read file", "file", filePath, "error", err)
@@ -45,6 +44,15 @@ var editCmd = &cobra.Command{
 		logger.Debug("file read successfully", "file", filePath, "size_bytes", len(original))
 		cleanedOriginal := removeLastModifiedComments(string(original))
 
+		backupPath := filePath + ".bak"
+		logger.Debug("creating backup", "backup_path", backupPath)
+		if err := os.WriteFile(backupPath, original, 0644); err != nil {
+			logger.Error("failed to create backup", "backup_path", backupPath, "error", err)
+			color.Red("Failed to create backup: %v", err)
+			os.Exit(1)
+		}
+		logger.Info("backup created", "backup_path", backupPath)
+
 		client := grok.NewClient()
 		messages := []map[string]string{
 			{"role": "system", "content": "You are an expert programmer. Remove all unnecessary comments including last modified timestamps and ownership comments. Return only the cleaned code with no explanations, no markdown, no extra text."},
@@ -65,31 +73,32 @@ var editCmd = &cobra.Command{
 		var confirm string
 		if _, err := fmt.Scanln(&confirm); err != nil {
 			color.Red("Failed to read input: %v", err)
-			color.Yellow("Changes discarded.")
+			color.Yellow("Changes discarded. Backup saved as %s", backupPath)
 			return
 		}
 		if confirm != "y" && confirm != "Y" {
-			color.Yellow("Changes discarded.")
+			color.Yellow("Changes discarded. Backup saved as %s", backupPath)
 			return
 		}
 
 		logger.Debug("applying changes", "file", filePath, "new_size_bytes", len(newContent))
-		if err := os.WriteFile(filePath, []byte(newContent), 0600); err != nil {
+		if err := os.WriteFile(filePath, []byte(newContent), 0644); err != nil {
 			logger.Error("failed to write file", "file", filePath, "error", err)
 			color.Red("Failed to write file: %v", err)
 			os.Exit(1)
 		}
 		logger.Info("changes applied successfully",
 			"file", filePath,
+			"backup", backupPath,
 			"original_size", len(original),
 			"new_size", len(newContent))
-		color.Green("✅ Applied successfully!")
+		color.Green("✅ Applied successfully! Backup: %s", backupPath)
 	},
 }
 
 func removeLastModifiedComments(content string) string {
 	lines := strings.Split(content, "\n")
-	cleanedLines := make([]string, 0, len(lines))
+	var cleanedLines []string
 
 	for _, line := range lines {
 		if strings.Contains(line, "Last modified") {

cmd/edit_test.go

@@ -17,7 +17,7 @@ func TestEditCommand(t *testing.T) {
 	defer func() { _ = os.Remove(tmpfile.Name()) }()
 
 	original := []byte("package main\n\nfunc hello() {}\n")
-	if err := os.WriteFile(tmpfile.Name(), original, 0600); err != nil {
+	if err := os.WriteFile(tmpfile.Name(), original, 0644); err != nil {
 		t.Fatal(err)
 	}
 
@@ -34,7 +34,7 @@ func TestEditCommand(t *testing.T) {
 	newContent := grok.CleanCodeResponse(raw)
 
 	// Apply the result (this is what the real command does after confirmation)
-	if err := os.WriteFile(tmpfile.Name(), []byte(newContent), 0600); err != nil {
+	if err := os.WriteFile(tmpfile.Name(), []byte(newContent), 0644); err != nil {
 		t.Fatal(err)
 	}
 

cmd/history.go

@@ -12,7 +12,7 @@ var historyCmd = &cobra.Command{
 	Run:   runHistory,
 }
 
-func runHistory(cmd *cobra.Command, _ []string) {
+func runHistory(cmd *cobra.Command, args []string) {
 	log, err := gitRun([]string{"log", "--oneline", "-10"})
 	if err != nil {
 		color.Red("Failed to get git log: %v", err)

cmd/lint.go

@@ -32,6 +32,7 @@ The command will:
 4. (Optional) Use Grok AI to generate and apply fixes
 
 Safety features:
+- Creates backup before modifying files
 - Shows preview of changes before applying
 - Requires confirmation unless --auto-fix is used`,
 	Args: cobra.ExactArgs(1),
@@ -103,7 +104,6 @@ func runLint(cmd *cobra.Command, args []string) {
 	}
 
 	// Read original file content
-	// nolint:gosec // intentional file read from user input
 	originalContent, err := os.ReadFile(absPath)
 	if err != nil {
 		logger.Error("failed to read file", "file", absPath, "error", err)
@@ -132,6 +132,16 @@ func runLint(cmd *cobra.Command, args []string) {
 	fixedCode := grok.CleanCodeResponse(response)
 	logger.Info("received AI fixes", "file", absPath, "fixed_size", len(fixedCode))
 
+	// Create backup
+	backupPath := absPath + ".bak"
+	if err := os.WriteFile(backupPath, originalContent, 0644); err != nil {
+		logger.Error("failed to create backup", "file", absPath, "error", err)
+		color.Red("❌ Failed to create backup: %v", err)
+		return
+	}
+	logger.Info("backup created", "backup", backupPath)
+	color.Green("💾 Backup created: %s", backupPath)
+
 	// Show preview if not auto-fix
 	if !autoFix {
 		fmt.Println()
@@ -143,7 +153,7 @@ func runLint(cmd *cobra.Command, args []string) {
 		if len(lines) < previewLines {
 			previewLines = len(lines)
 		}
-		for i := range previewLines {
+		for i := 0; i < previewLines; i++ {
 			fmt.Println(lines[i])
 		}
 		if len(lines) > previewLines {
@@ -157,20 +167,20 @@ func runLint(cmd *cobra.Command, args []string) {
 		var response string
 		if _, err := fmt.Scanln(&response); err != nil {
 			logger.Info("failed to read user input", "file", absPath, "error", err)
-			color.Yellow("❌ Cancelled. No changes made.")
+			color.Yellow("❌ Cancelled. Backup preserved at: %s", backupPath)
 			return
 		}
 		response = strings.ToLower(strings.TrimSpace(response))
 
 		if response != "y" && response != "yes" {
 			logger.Info("user cancelled fix application", "file", absPath)
-			color.Yellow("❌ Cancelled. No changes made.")
+			color.Yellow("❌ Cancelled. Backup preserved at: %s", backupPath)
 			return
 		}
 	}
 
 	// Apply fixes
-	if err := os.WriteFile(absPath, []byte(fixedCode), 0600); err != nil {
+	if err := os.WriteFile(absPath, []byte(fixedCode), 0644); err != nil {
 		logger.Error("failed to write fixed file", "file", absPath, "error", err)
 		color.Red("❌ Failed to write file: %v", err)
 		return
@@ -178,6 +188,7 @@ func runLint(cmd *cobra.Command, args []string) {
 
 	logger.Info("fixes applied successfully", "file", absPath)
 	color.Green("✅ Fixes applied successfully!")
+	color.Cyan("💾 Original saved to: %s", backupPath)
 
 	// Optionally run linter again to verify
 	color.Cyan("\n🔍 Re-running linter to verify fixes...")

cmd/prdescribe.go

@@ -14,23 +14,17 @@ var prDescribeCmd = &cobra.Command{
 	Run:   runPRDescribe,
 }
 
-func init() {
-	prDescribeCmd.Flags().StringP("base", "b", "master", "Base branch to compare against")
-}
-
-func runPRDescribe(cmd *cobra.Command, _ []string) {
-	base, _ := cmd.Flags().GetString("base")
-
-	diff, err := gitRun([]string{"diff", fmt.Sprintf("%s..HEAD", base), "--no-color"})
+func runPRDescribe(cmd *cobra.Command, args []string) {
+	diff, err := gitRun([]string{"diff", "main..HEAD", "--no-color"})
 	if err != nil || diff == "" {
-		diff, err = gitRun([]string{"diff", fmt.Sprintf("origin/%s..HEAD", base), "--no-color"})
+		diff, err = gitRun([]string{"diff", "origin/main..HEAD", "--no-color"})
 		if err != nil {
 			color.Red("Failed to get branch diff: %v", err)
 			return
 		}
 	}
 	if diff == "" {
-		color.Yellow("No changes on this branch compared to %s/origin/%s.", base, base)
+		color.Yellow("No changes on this branch compared to main/origin/main.")
 		return
 	}
 	modelFlag, _ := cmd.Flags().GetString("model")

cmd/query.go

@@ -1,53 +0,0 @@
-package cmd
-
-import (
-	"github.com/fatih/color"
-	"github.com/spf13/cobra"
-	"gmgauthier.com/grokkit/config"
-	"gmgauthier.com/grokkit/internal/grok"
-)
-
-var queryCmd = &cobra.Command{
-	Use:   "query [question]",
-	Short: "One-shot non-interactive query to Grok (programming focused)",
-	Long: `Ask Grok a single technical question and get a concise, actionable answer.
-
-Default mode is factual and brief. Use --wordy for longer, more explanatory answers.`,
-	Args: cobra.MinimumNArgs(1),
-	Run:  runQuery,
-}
-
-func init() {
-	queryCmd.Flags().Bool("wordy", false, "Give a longer, more detailed answer")
-	rootCmd.AddCommand(queryCmd)
-}
-
-func runQuery(cmd *cobra.Command, args []string) {
-	wordy, _ := cmd.Flags().GetBool("wordy")
-	question := args[0]
-
-	// Use fast model by default for quick queries
-	model := config.GetModel("query", "")
-
-	client := grok.NewClient()
-
-	systemPrompt := `You are Grok, a helpful and truthful AI built by xAI.
-Focus on programming, software engineering, and technical questions.
-Be concise, factual, and actionable. Include code snippets when helpful.
-Do not add unnecessary fluff.`
-
-	if wordy {
-		systemPrompt = `You are Grok, a helpful and truthful AI built by xAI.
-Give thorough, detailed, textbook-style answers to technical questions.
-Explain concepts clearly, include examples, and allow light humour where appropriate.
-Be comprehensive but still clear and well-structured.`
-	}
-
-	messages := []map[string]string{
-		{"role": "system", "content": systemPrompt},
-		{"role": "user", "content": question},
-	}
-
-	color.Yellow("Asking Grok...")
-	client.Stream(messages, model)
-}

cmd/review.go

@@ -14,7 +14,7 @@ var reviewCmd = &cobra.Command{
 	Run:   runReview,
 }
 
-func runReview(cmd *cobra.Command, _ []string) {
+func runReview(cmd *cobra.Command, args []string) {
 	modelFlag, _ := cmd.Flags().GetString("model")
 	model := config.GetModel("review", modelFlag)
 

cmd/root.go

@@ -4,7 +4,6 @@ import (
 	"os"
 
 	"fmt"
-
 	"github.com/spf13/cobra"
 	"gmgauthier.com/grokkit/config"
 	"gmgauthier.com/grokkit/internal/logger"
@@ -59,8 +58,6 @@ func init() {
 	rootCmd.AddCommand(versionCmd)
 	rootCmd.AddCommand(docsCmd)
 	rootCmd.AddCommand(testgenCmd)
-	rootCmd.AddCommand(scaffoldCmd)
-	rootCmd.AddCommand(queryCmd)
 
 	// Add model flag to all commands
 	rootCmd.PersistentFlags().StringP("model", "m", "", "Grok model to use (overrides config)")

cmd/run_test.go

@@ -47,11 +47,10 @@ func withMockGit(fn func([]string) (string, error)) func() {
 	return func() { gitRun = orig }
 }
 
-// testCmd returns a minimal cobra command with common flags registered.
+// testCmd returns a minimal cobra command with the model flag registered.
 func testCmd() *cobra.Command {
 	c := &cobra.Command{}
 	c.Flags().String("model", "", "")
-	c.Flags().String("base", "master", "")
 	return c
 }
 
@@ -309,62 +308,22 @@ func TestRunPRDescribe(t *testing.T) {
 		}
 	})
 
-	t.Run("uses custom base branch", func(t *testing.T) {
-		mock := &mockStreamer{response: "PR description"}
+	t.Run("second diff error — skips AI", func(t *testing.T) {
+		mock := &mockStreamer{}
 		defer withMockClient(mock)()
-		var capturedArgs []string
+		callCount := 0
 		defer withMockGit(func(args []string) (string, error) {
-			capturedArgs = args
-			return "diff content", nil
-		})()
-
-		cmd := testCmd()
-		if err := cmd.Flags().Set("base", "develop"); err != nil {
-			t.Fatal(err)
-		}
-		runPRDescribe(cmd, nil)
-
-		if mock.calls != 1 {
-			t.Errorf("expected 1 AI call, got %d", mock.calls)
-		}
-		// Expect "diff", "develop..HEAD", "--no-color"
-		expectedArg := "develop..HEAD"
-		found := false
-		for _, arg := range capturedArgs {
-			if arg == expectedArg {
-				found = true
-				break
+			callCount++
+			if callCount == 2 {
+				return "", errors.New("no remote")
 			}
-		}
-		if !found {
-			t.Errorf("expected arg %q not found in %v", expectedArg, capturedArgs)
-		}
-	})
-
-	t.Run("defaults to master", func(t *testing.T) {
-		mock := &mockStreamer{response: "PR description"}
-		defer withMockClient(mock)()
-		var capturedArgs []string
-		defer withMockGit(func(args []string) (string, error) {
-			capturedArgs = args
-			return "diff content", nil
+			return "", nil
 		})()
 
 		runPRDescribe(testCmd(), nil)
 
-		if mock.calls != 1 {
-			t.Errorf("expected 1 AI call, got %d", mock.calls)
-		}
-		expectedArg := "master..HEAD"
-		found := false
-		for _, arg := range capturedArgs {
-			if arg == expectedArg {
-				found = true
-				break
-			}
-		}
-		if !found {
-			t.Errorf("expected arg %q not found in %v", expectedArg, capturedArgs)
+		if mock.calls != 0 {
+			t.Errorf("expected 0 AI calls, got %d", mock.calls)
 		}
 	})
 }
@@ -427,6 +386,7 @@ func TestProcessDocsFilePreviewAndCancel(t *testing.T) {
 		t.Fatal(err)
 	}
 	defer func() { _ = os.Remove(f.Name()) }()
+	defer func() { _ = os.Remove(f.Name() + ".bak") }()
 
 	mock := &mockStreamer{response: "package main\n\n// Foo does nothing.\nfunc Foo() {}\n"}
 
@@ -466,6 +426,7 @@ func TestProcessDocsFileAutoApply(t *testing.T) {
 		t.Fatal(err)
 	}
 	defer func() { _ = os.Remove(f.Name()) }()
+	defer func() { _ = os.Remove(f.Name() + ".bak") }()
 
 	// CleanCodeResponse will trim the trailing newline from the AI response.
 	aiResponse := "package main\n\n// Bar does nothing.\nfunc Bar() {}\n"
@@ -486,6 +447,11 @@ func TestProcessDocsFileAutoApply(t *testing.T) {
 	if string(content) != documented {
 		t.Errorf("file content = %q, want %q", string(content), documented)
 	}
+	// Verify backup was created with original content.
+	backup, _ := os.ReadFile(f.Name() + ".bak")
+	if string(backup) != original {
+		t.Errorf("backup content = %q, want %q", string(backup), original)
+	}
 }
 
 func TestRunDocs(t *testing.T) {

cmd/scaffold.go

@@ -1,187 +0,0 @@
-package cmd
-
-import (
-	"fmt"
-	"os"
-	"path/filepath"
-	"strings"
-
-	"github.com/fatih/color"
-	"github.com/spf13/cobra"
-	"gmgauthier.com/grokkit/config"
-	"gmgauthier.com/grokkit/internal/grok"
-	"gmgauthier.com/grokkit/internal/logger"
-)
-
-var scaffoldCmd = &cobra.Command{
-	Use:   "scaffold FILE DESCRIPTION",
-	Short: "Scaffold a new file with Grok (safe preview + confirmation)",
-	Args:  cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		filePath := args[0]
-		description := args[1]
-
-		withTests, _ := cmd.Flags().GetBool("with-tests")
-		dryRun, _ := cmd.Flags().GetBool("dry-run")
-		force, _ := cmd.Flags().GetBool("force")
-		yes, _ := cmd.Flags().GetBool("yes")
-		langOverride, _ := cmd.Flags().GetString("lang")
-
-		modelFlag, _ := cmd.Flags().GetString("model")
-		model := config.GetModel("scaffold", modelFlag)
-
-		logger.Info("scaffold command started",
-			"file", filePath,
-			"description", description,
-			"with_tests", withTests,
-			"model", model)
-
-		// Safety: don't overwrite existing file unless --force
-		if _, err := os.Stat(filePath); err == nil && !force {
-			color.Red("File already exists: %s (use --force to overwrite)", filePath)
-			os.Exit(1)
-		}
-
-		dir := filepath.Dir(filePath)
-		if dir != "." && dir != "" {
-			if err := os.MkdirAll(dir, 0750); err != nil {
-				logger.Error("failed to create directory", "dir", dir, "error", err)
-				color.Red("Failed to create directory: %v", err)
-				os.Exit(1)
-			}
-		}
-
-		// Detect language and harvest context
-		lang := detectLanguage(filePath, langOverride)
-		context := harvestContext(filePath, lang)
-
-		// Build system prompt with style enforcement
-		systemPrompt := fmt.Sprintf(`You are an expert %s programmer.
-Match the exact style, naming conventions, error handling, logging, and package structure of this project.
-Return ONLY the complete code file. No explanations, no markdown, no backticks.`, lang)
-
-		if withTests {
-			systemPrompt += "\nAlso generate a basic _test.go file with at least one test when --with-tests is used."
-		}
-
-		messages := []map[string]string{
-			{"role": "system", "content": systemPrompt},
-			{"role": "user", "content": fmt.Sprintf(
-				"File path: %s\n\nProject context:\n%s\n\nCreate this new file:\n%s",
-				filePath, context, description)},
-		}
-
-		color.Yellow("Asking Grok to scaffold %s...\n", filepath.Base(filePath))
-		client := grok.NewClient()
-		raw := client.StreamSilent(messages, model)
-		newContent := grok.CleanCodeResponse(raw)
-		color.Green("Response received")
-
-		// Preview
-		color.Cyan("\nProposed new file:")
-		fmt.Printf("--- /dev/null\n")
-		fmt.Printf("+++ b/%s\n", filepath.Base(filePath))
-		fmt.Println(newContent)
-
-		if dryRun {
-			color.Yellow("\n--dry-run: file not written")
-			return
-		}
-
-		if !yes {
-			fmt.Print("\n\nCreate this file? (y/n): ")
-			var confirm string
-			if _, err := fmt.Scanln(&confirm); err != nil || (confirm != "y" && confirm != "Y") {
-				color.Yellow("Scaffold cancelled.")
-				return
-			}
-		}
-
-		// Write main file
-		if err := os.WriteFile(filePath, []byte(newContent), 0600); err != nil {
-			logger.Error("failed to write file", "file", filePath, "error", err)
-			color.Red("Failed to write file: %v", err)
-			os.Exit(1)
-		}
-		color.Green("✓ Created: %s", filePath)
-
-		// Optional test file
-		if withTests {
-			testPath := strings.TrimSuffix(filePath, filepath.Ext(filePath)) + "_test.go"
-			testMessages := []map[string]string{
-				{"role": "system", "content": systemPrompt},
-				{"role": "user", "content": fmt.Sprintf("Generate a basic test file for %s using the same style.", filepath.Base(filePath))},
-			}
-			testRaw := client.StreamSilent(testMessages, model)
-			testContent := grok.CleanCodeResponse(testRaw)
-
-			if err := os.WriteFile(testPath, []byte(testContent), 0600); err == nil {
-				color.Green("✓ Created test: %s", filepath.Base(testPath))
-			}
-		}
-
-		logger.Info("scaffold completed successfully", "file", filePath, "with_tests", withTests)
-	},
-}
-
-// Simple language detector (can be moved to internal/linter later)
-func detectLanguage(path, override string) string {
-	if override != "" {
-		return override
-	}
-	ext := strings.ToLower(filepath.Ext(path))
-	switch ext {
-	case ".go":
-		return "Go"
-	case ".py":
-		return "Python"
-	case ".js", ".ts":
-		return "TypeScript"
-	case ".c":
-		return "C"
-	case ".cpp":
-		return "C++"
-	case ".java":
-		return "Java"
-	// add more as needed
-	default:
-		return "code"
-	}
-}
-
-// Basic context harvester (~4000 token cap)
-func harvestContext(filePath, _ string) string {
-	var sb strings.Builder
-	dir := filepath.Dir(filePath)
-
-	// Siblings
-	files, _ := os.ReadDir(dir)
-	for _, f := range files {
-		if f.IsDir() || strings.HasPrefix(f.Name(), ".") {
-			continue
-		}
-		if filepath.Ext(f.Name()) == filepath.Ext(filePath) && f.Name() != filepath.Base(filePath) {
-			// nolint:gosec // intentional file read from project directory
-			content, _ := os.ReadFile(filepath.Join(dir, f.Name()))
-			if len(content) > 2000 {
-				content = content[:2000]
-			}
-			// Fixed: use Fprintf instead of WriteString + Sprintf
-			fmt.Fprintf(&sb, "=== %s ===\n%s\n\n", f.Name(), string(content))
-		}
-	}
-
-	// Rough token cap
-	if sb.Len() > 4000 {
-		return sb.String()[:4000] + "\n... (truncated)"
-	}
-	return sb.String()
-}
-
-func init() {
-	scaffoldCmd.Flags().Bool("with-tests", false, "Also scaffold a basic _test.go file")
-	scaffoldCmd.Flags().Bool("dry-run", false, "Preview only, do not write")
-	scaffoldCmd.Flags().BoolP("yes", "y", false, "Skip confirmation prompt")
-	scaffoldCmd.Flags().Bool("force", false, "Overwrite existing file")
-	scaffoldCmd.Flags().String("lang", "", "Force language for prompt (Go, Python, etc.)")
-}

cmd/scaffold_test.go

@@ -1,78 +0,0 @@
-// cmd/scaffold_test.go
-package cmd
-
-import (
-	"os"
-	"testing"
-
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-)
-
-// TestScaffoldCmd — FAST DEFAULT TEST (runs on make test / go test ./cmd)
-func TestScaffoldCmd(t *testing.T) {
-	// Minimal fast unit test — no API calls, no prompts, no cost
-	t.Log("✓ Fast scaffold unit test (no Grok API call)")
-	assert.True(t, true, "command is registered and basic structure is intact")
-}
-
-// TestScaffoldCmd_Live — LIVE INTEGRATION TEST (only runs when you ask)
-func TestScaffoldCmd_Live(t *testing.T) {
-	if !testing.Short() {
-		t.Skip("skipping live Grok integration test. Run with:\n  go test ./cmd -run TestScaffoldCmd_Live -short -v")
-	}
-
-	t.Log("Running live scaffold integration tests...")
-
-	tests := []struct {
-		name      string
-		args      []string
-		expectErr bool
-	}{
-		{
-			name:      "basic scaffold happy path",
-			args:      []string{"scaffold", "newfile.go", "A simple test struct for configuration"},
-			expectErr: false,
-		},
-		{
-			name:      "scaffold with --with-tests flag",
-			args:      []string{"scaffold", "newfile.go", "A simple test struct", "--with-tests"},
-			expectErr: false,
-		},
-		{
-			name:      "dry-run does not fail",
-			args:      []string{"scaffold", "dry.go", "dry run test", "--dry-run"},
-			expectErr: false,
-		},
-		{
-			name:      "force flag works",
-			args:      []string{"scaffold", "exists.go", "overwrite me", "--force"},
-			expectErr: false,
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			t.Logf("Live test: %s", tt.name)
-
-			tmpDir := t.TempDir()
-			origDir, _ := os.Getwd()
-			require.NoError(t, os.Chdir(tmpDir))
-			defer func() {
-				if err := os.Chdir(origDir); err != nil {
-					t.Logf("warning: failed to restore original directory: %v", err)
-				}
-			}()
-
-			rootCmd.SetArgs(tt.args)
-			err := rootCmd.Execute()
-
-			if tt.expectErr {
-				assert.Error(t, err)
-				return
-			}
-			assert.NoError(t, err)
-			t.Log("✓ Live test passed")
-		})
-	}
-}

cmd/testgen.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"errors"
 	"fmt"
 	"os"
 	"path/filepath"
@@ -33,8 +32,6 @@ Examples:
 		modelFlag, _ := cmd.Flags().GetString("model")
 		model := config.GetModel("testgen", modelFlag)
 
-		color.Cyan("🔧 testgen using model: %s (override with -m flag)", model)
-
 		logger.Info("testgen started", "num_files", len(args), "model", model, "auto_apply", yesFlag)
 
 		client := grok.NewClient()
@@ -93,15 +90,29 @@ func processTestgenFile(client *grok.Client, filePath, lang, systemPrompt, model
 	testPath := getTestFilePath(filePath, lang)
 
 	// Handle existing test file
+	var origTest []byte
 	testExists := true
 	testInfo, err := os.Stat(testPath)
-	switch {
-	case errors.Is(err, os.ErrNotExist):
+	if os.IsNotExist(err) {
 		testExists = false
-	case err != nil:
+	} else if err != nil {
 		return fmt.Errorf("stat test file: %w", err)
-	case testInfo.IsDir():
+	} else if testInfo.IsDir() {
 		return fmt.Errorf("test path is dir: %s", testPath)
+	} else {
+		origTest, err = os.ReadFile(testPath)
+		if err != nil {
+			return fmt.Errorf("read existing test: %w", err)
+		}
+	}
+
+	// Backup existing test
+	backupPath := testPath + ".bak"
+	if testExists {
+		if err := os.WriteFile(backupPath, origTest, 0644); err != nil {
+			return fmt.Errorf("backup test file: %w", err)
+		}
+		color.Yellow("💾 Backup: %s", backupPath)
 	}
 
 	// Generate tests
@@ -113,24 +124,15 @@ func processTestgenFile(client *grok.Client, filePath, lang, systemPrompt, model
 		},
 		{
 			"role":    "user",
-			"content": fmt.Sprintf("Source file %s (%s):\n```%s\n%s\n```\n\nGenerate the test file using the new Unit + Live integration pattern described in the system prompt.", filepath.Base(filePath), lang, codeLang, cleanSrc)},
+			"content": fmt.Sprintf("Source file %s (%s):\n```%s\n%s\n```\n\nGenerate the language-appropriate test file.", filepath.Base(filePath), lang, codeLang, cleanSrc),
+		},
 	}
 
 	color.Yellow("🤖 Generating tests for %s → %s...", filepath.Base(filePath), filepath.Base(testPath))
 	rawResponse := client.StreamSilent(messages, model)
-
-	// ← NEW: detailed debugging when the model returns nothing useful
 	newTestCode := grok.CleanCodeResponse(rawResponse)
+
 	if len(newTestCode) == 0 || strings.TrimSpace(newTestCode) == "" {
-		color.Red("❌ Cleaned response is empty")
-		color.Red("   Raw AI response length : %d characters", len(rawResponse))
-		if len(rawResponse) > 0 {
-			previewLen := 300
-			if len(rawResponse) < previewLen {
-				previewLen = len(rawResponse)
-			}
-			color.Red("   Raw preview (first %d chars):\n%s", previewLen, rawResponse[:previewLen])
-		}
 		return fmt.Errorf("empty generation response")
 	}
 
@@ -154,13 +156,13 @@ func processTestgenFile(client *grok.Client, filePath, lang, systemPrompt, model
 		}
 		confirm = strings.TrimSpace(strings.ToLower(confirm))
 		if confirm != "y" && confirm != "yes" {
-			color.Yellow("⏭️  Skipped %s", testPath)
+			color.Yellow("⏭️  Skipped %s (backup: %s)", testPath, backupPath)
 			return nil
 		}
 	}
 
 	// Apply
-	if err := os.WriteFile(testPath, []byte(newTestCode), 0600); err != nil {
+	if err := os.WriteFile(testPath, []byte(newTestCode), 0644); err != nil {
 		return fmt.Errorf("write test file: %w", err)
 	}
 
@@ -171,7 +173,7 @@ func processTestgenFile(client *grok.Client, filePath, lang, systemPrompt, model
 
 func removeSourceComments(content, lang string) string {
 	lines := strings.Split(content, "\n")
-	cleanedLines := make([]string, 0, len(lines))
+	var cleanedLines []string
 	for _, line := range lines {
 		if strings.Contains(line, "Last modified") || strings.Contains(line, "Generated by") ||
 			strings.Contains(line, "Generated by testgen") {
@@ -191,42 +193,21 @@ func removeSourceComments(content, lang string) string {
 func getTestPrompt(lang string) string {
 	switch lang {
 	case "Go":
-		return `You are an expert Go test writer.
+		return `You are an expert Go testing specialist. Generate COMPLETE, production-ready unit tests for the provided Go source:
 
-Generate a COMPLETE, production-ready *_test.go file using ONLY idiomatic Go.
+- Table-driven with t.Run(subtest, func(t *testing.T)) and t.Parallel()
+- Match exact style of gmgauthier/grokkit/internal/version/version_test.go
+- Use t.Context() for ctx-aware tests
+- Modern Go 1.24+: slices.Contains/IndexFunc, maps.Keys, errors.Is/Join, any, etc.
+- Cover ALL public funcs/methods/fields: happy path, edges, errors, panics, zero values
+- Realistic inputs, no external deps/mocks unless code requires
+- func TestXxx(t *testing.T) { ... } only
 
-STRICT RULES — NEVER VIOLATE THESE:
-- NO monkey-patching: never assign to runtime.GOOS, exec.Command, os.Stdout, os.Exit, or ANY package-level func/var
-- NO global variable reassignment
-- NO reflect tricks for mocking
-- Use ONLY real function calls + table-driven tests
-- For os.Exit or stdout testing, use simple smoke tests or pipes (no reassignment)
-- Prefer simple happy-path + error-path tests
-
-1. Fast unit test:
-   func TestXXX_Unit(t *testing.T) {
-       t.Parallel()
-       t.Log("✓ Fast XXX unit test")
-       // table-driven, real calls only
-   }
-
-2. Optional live test:
-   func TestXXX_Live(t *testing.T) {
-       if !testing.Short() {
-           t.Skip("skipping live integration test. Run with:\n      go test -run TestXXX_Live -short -v")
-       }
-       t.Log("🧪 Running live integration test...")
-   }
-
-Exact rules:
-- Derive test names from functions (e.g. isInstalled → TestIsInstalled_Unit)
-- The XXX in t.Skip MUST exactly match the live function name
-- t.Parallel() on unit tests only
-- NO unused imports
-- Keep tests simple and idiomatic Go
-- Return ONLY the full test file. No explanations, no markdown, no backticks.`
-
-	// Python/C/C++ prompts unchanged
+Respond with ONLY the full *_test.go file:
+- Correct package name (infer from code)
+- Necessary imports
+- No benchmarks unless obvious perf func
+- NO prose, explanations, markdown, code blocks, or extra text. Pure Go test file.`
 	case "Python":
 		return `You are a pytest expert. Generate COMPLETE pytest unit tests for the Python source.
 
@@ -243,7 +224,7 @@ Respond ONLY with full test_*.py file: imports, fixtures, tests. Pure Python tes
 
 - Use Check suite: Suite, tcase_begin/end, ck_assert_* macros
 - Cover ALL functions: happy/edge/error cases
-- Include #include <check.h>
+- Include #include <check.h> <minunit.h>? Use Check std.
 - main() runner if needed.
 
 Respond ONLY full test_*.c: headers, suite funcs, main. Pure C.`

cmd/testgen_test.go

@@ -5,21 +5,6 @@ import (
 	"testing"
 )
 
-// New scaffold-style dual tests (fast unit + optional live)
-// These are exactly the pattern used in scaffold_test.go
-func TestTestgenCmd(t *testing.T) {
-	t.Parallel()
-	t.Log("✓ Fast testgen unit test (no Grok API call)")
-}
-
-func TestTestgenCmd_Live(t *testing.T) {
-	if !testing.Short() {
-		t.Skip("skipping live Grok integration test. Run with:\n      go test ./cmd -run TestTestgenCmd_Live -short -v")
-	}
-	t.Log("🧪 Running live testgen integration test with real Grok API...")
-	// TODO: expand later (e.g. create temp source + call processTestgenFile + verify output)
-}
-
 func TestRemoveSourceComments(t *testing.T) {
 	t.Parallel()
 
@@ -93,7 +78,6 @@ int foo() {}`,
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			t.Parallel()
 			got := removeSourceComments(tt.input, tt.lang)
 			if got != tt.want {
 				t.Errorf("removeSourceComments() =\n%q\nwant\n%q", got, tt.want)
@@ -109,7 +93,7 @@ func TestGetTestPrompt(t *testing.T) {
 		lang       string
 		wantPrefix string
 	}{
-		{"Go", "You are an expert Go test writer."}, // ← updated to match the new generalized prompt
+		{"Go", "You are an expert Go testing specialist."},
 		{"Python", "You are a pytest expert."},
 		{"C", "You are a C unit testing expert using Check framework."},
 		{"C++", "You are a Google Test expert."},
@@ -118,7 +102,6 @@ func TestGetTestPrompt(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.lang, func(t *testing.T) {
-			t.Parallel()
 			got := getTestPrompt(tt.lang)
 			if tt.wantPrefix != "" && !strings.HasPrefix(got, tt.wantPrefix) {
 				t.Errorf("getTestPrompt(%q) prefix =\n%q\nwant %q", tt.lang, got[:100], tt.wantPrefix)
@@ -146,7 +129,6 @@ func TestGetTestFilePath(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.filePath+"_"+tt.lang, func(t *testing.T) {
-			t.Parallel()
 			got := getTestFilePath(tt.filePath, tt.lang)
 			if got != tt.want {
 				t.Errorf("getTestFilePath(%q, %q) = %q, want %q", tt.filePath, tt.lang, got, tt.want)
@@ -170,7 +152,6 @@ func TestGetCodeLang(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.lang, func(t *testing.T) {
-			t.Parallel()
 			got := getCodeLang(tt.lang)
 			if got != tt.want {
 				t.Errorf("getCodeLang(%q) = %q, want %q", tt.lang, got, tt.want)

config/config.go

@@ -7,7 +7,6 @@ import (
 	"github.com/spf13/viper"
 )
 
-// Load initializes the configuration from Viper
 func Load() {
 	home, err := os.UserHomeDir()
 	if err != nil {
@@ -37,13 +36,11 @@ func Load() {
 	viper.SetDefault("commands.prdescribe.model", "grok-4")
 	viper.SetDefault("commands.review.model", "grok-4")
 	viper.SetDefault("commands.docs.model", "grok-4")
-	viper.SetDefault("commands.query.model", "grok-4-1-fast-non-reasoning")
 
 	// Config file is optional, so we ignore read errors
 	_ = viper.ReadInConfig()
 }
 
-// GetModel returns the model to use for a specific command, considering flags and aliases
 func GetModel(commandName string, flagModel string) string {
 	if flagModel != "" {
 		if alias := viper.GetString("aliases." + flagModel); alias != "" {
@@ -58,12 +55,10 @@ func GetModel(commandName string, flagModel string) string {
 	return viper.GetString("default_model")
 }
 
-// GetTemperature returns the temperature from the configuration
 func GetTemperature() float64 {
 	return viper.GetFloat64("temperature")
 }
 
-// GetTimeout returns the timeout from the configuration
 func GetTimeout() int {
 	timeout := viper.GetInt("timeout")
 	if timeout <= 0 {
@@ -72,7 +67,6 @@ func GetTimeout() int {
 	return timeout
 }
 
-// GetLogLevel returns the log level from the configuration
 func GetLogLevel() string {
 	return viper.GetString("log_level")
 }

docs/ARCHITECTURE.md

@@ -23,7 +23,7 @@ Grokkit follows these core principles:
 - Single responsibility per package
 
 ### 2. **Safety by Default**
-- Git-based version control for change management
+- File backups before any modification
 - Confirmation prompts for destructive actions
 - Comprehensive error handling
 
@@ -334,6 +334,7 @@ User: grokkit edit file.go "instruction"
 editCmd.Run()
     ├─→ Validate file exists
     ├─→ Read original content
+    ├─→ Create backup (.bak)
     ├─→ Clean "Last modified" comments
     ↓
 grok.Client.StreamSilent()  # Get AI response
@@ -346,7 +347,8 @@ Display preview             # Show diff-style output
 Prompt user (y/n)
     ↓
 Write file (if confirmed)
-    └─→ logger.Info()       # Log changes
+    ├─→ logger.Info()       # Log changes
+    └─→ Keep backup
 ```
 
 ### Commit Command Flow
@@ -392,6 +394,8 @@ grok.Client.StreamSilent()       # Get fixed code
     ↓
 grok.CleanCodeResponse()         # Remove markdown
     ↓
+Create backup (.bak)
+    ↓
 (if not --auto-fix) → Show preview + prompt
     ↓
 Write fixed file
@@ -567,7 +571,7 @@ func Run(args []string) (string, error) {
 **Current measures:**
 - API key via environment variable
 - No credential storage
-- Git-based rollbacks for safety
+- Backup files for safety
 
 **Future considerations:**
 - Encrypted config storage

docs/TROUBLESHOOTING.md

@@ -322,27 +322,27 @@ pwd
 ls
 ```
 
-### Rolling back AI changes
+### Permission denied on backup creation
 
 **Symptom:**
-AI suggested changes are undesired or introduced bugs.
-
-**Solution:**
-Use Git to roll back:
-
-```bash
-# If not yet added/staged:
-git restore <file>
-
-# If staged:
-git restore --staged <file>
-git restore <file>
-
-# If committed:
-git revert HEAD
+```
+Failed to create backup: permission denied
 ```
 
-Always review changes with `git diff` before and after applying AI suggestions.
+**Solution:**
+```bash
+# Check file permissions
+ls -la main.go
+
+# Check directory permissions
+ls -ld .
+
+# Fix permissions
+chmod 644 main.go    # File
+chmod 755 .          # Directory
+
+# Or run from writable directory
+```
 
 ### Failed to write file
 
@@ -581,8 +581,8 @@ After applying AI fixes, re-running the linter still reports issues.
 **Solution:**
 ```bash
 # Some issues may require manual intervention
-# Use git diff to compare changes
-git diff file.py
+# Check the backup file to compare
+diff file.py file.py.bak
 
 # Apply fixes iteratively
 grokkit lint file.py         # Apply fixes

go.mod

@@ -3,29 +3,46 @@ module gmgauthier.com/grokkit
 go 1.24.2
 
 require (
+	github.com/charmbracelet/bubbles v1.0.0
+	github.com/charmbracelet/bubbletea v1.3.10
+	github.com/charmbracelet/lipgloss v1.1.0
 	github.com/fatih/color v1.18.0
 	github.com/spf13/cobra v1.9.1
 	github.com/spf13/viper v1.21.0
-	github.com/stretchr/testify v1.11.1
 )
 
 require (
-	github.com/davecgh/go-spew v1.1.1 // indirect
+	github.com/atotto/clipboard v0.1.4 // indirect
+	github.com/aymanbagabas/go-osc52/v2 v2.0.1 // indirect
+	github.com/charmbracelet/colorprofile v0.4.1 // indirect
+	github.com/charmbracelet/x/ansi v0.11.6 // indirect
+	github.com/charmbracelet/x/cellbuf v0.0.15 // indirect
+	github.com/charmbracelet/x/term v0.2.2 // indirect
+	github.com/clipperhouse/displaywidth v0.9.0 // indirect
+	github.com/clipperhouse/stringish v0.1.1 // indirect
+	github.com/clipperhouse/uax29/v2 v2.5.0 // indirect
+	github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f // indirect
 	github.com/fsnotify/fsnotify v1.9.0 // indirect
 	github.com/go-viper/mapstructure/v2 v2.4.0 // indirect
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
+	github.com/lucasb-eyer/go-colorful v1.3.0 // indirect
 	github.com/mattn/go-colorable v0.1.13 // indirect
 	github.com/mattn/go-isatty v0.0.20 // indirect
+	github.com/mattn/go-localereader v0.0.1 // indirect
+	github.com/mattn/go-runewidth v0.0.19 // indirect
+	github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6 // indirect
+	github.com/muesli/cancelreader v0.2.2 // indirect
+	github.com/muesli/termenv v0.16.0 // indirect
 	github.com/pelletier/go-toml/v2 v2.2.4 // indirect
-	github.com/pmezard/go-difflib v1.0.0 // indirect
+	github.com/rivo/uniseg v0.4.7 // indirect
 	github.com/sagikazarmark/locafero v0.11.0 // indirect
 	github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect
 	github.com/spf13/afero v1.15.0 // indirect
 	github.com/spf13/cast v1.10.0 // indirect
 	github.com/spf13/pflag v1.0.10 // indirect
 	github.com/subosito/gotenv v1.6.0 // indirect
+	github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e // indirect
 	go.yaml.in/yaml/v3 v3.0.4 // indirect
 	golang.org/x/sys v0.38.0 // indirect
 	golang.org/x/text v0.28.0 // indirect
-	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

go.sum

@@ -1,6 +1,36 @@
+github.com/MakeNowJust/heredoc v1.0.0 h1:cXCdzVdstXyiTqTvfqk9SDHpKNjxuom+DOlyEeQ4pzQ=
+github.com/MakeNowJust/heredoc v1.0.0/go.mod h1:mG5amYoWBHf8vpLOuehzbGGw0EHxpZZ6lCpQ4fNJ8LE=
+github.com/atotto/clipboard v0.1.4 h1:EH0zSVneZPSuFR11BlR9YppQTVDbh5+16AmcJi4g1z4=
+github.com/atotto/clipboard v0.1.4/go.mod h1:ZY9tmq7sm5xIbd9bOK4onWV4S6X0u6GY7Vn0Yu86PYI=
+github.com/aymanbagabas/go-osc52/v2 v2.0.1 h1:HwpRHbFMcZLEVr42D4p7XBqjyuxQH5SMiErDT4WkJ2k=
+github.com/aymanbagabas/go-osc52/v2 v2.0.1/go.mod h1:uYgXzlJ7ZpABp8OJ+exZzJJhRNQ2ASbcXHWsFqH8hp8=
+github.com/aymanbagabas/go-udiff v0.3.1 h1:LV+qyBQ2pqe0u42ZsUEtPiCaUoqgA9gYRDs3vj1nolY=
+github.com/aymanbagabas/go-udiff v0.3.1/go.mod h1:G0fsKmG+P6ylD0r6N/KgQD/nWzgfnl8ZBcNLgcbrw8E=
+github.com/charmbracelet/bubbles v1.0.0 h1:12J8/ak/uCZEMQ6KU7pcfwceyjLlWsDLAxB5fXonfvc=
+github.com/charmbracelet/bubbles v1.0.0/go.mod h1:9d/Zd5GdnauMI5ivUIVisuEm3ave1XwXtD1ckyV6r3E=
+github.com/charmbracelet/bubbletea v1.3.10 h1:otUDHWMMzQSB0Pkc87rm691KZ3SWa4KUlvF9nRvCICw=
+github.com/charmbracelet/bubbletea v1.3.10/go.mod h1:ORQfo0fk8U+po9VaNvnV95UPWA1BitP1E0N6xJPlHr4=
+github.com/charmbracelet/colorprofile v0.4.1 h1:a1lO03qTrSIRaK8c3JRxJDZOvhvIeSco3ej+ngLk1kk=
+github.com/charmbracelet/colorprofile v0.4.1/go.mod h1:U1d9Dljmdf9DLegaJ0nGZNJvoXAhayhmidOdcBwAvKk=
+github.com/charmbracelet/lipgloss v1.1.0 h1:vYXsiLHVkK7fp74RkV7b2kq9+zDLoEU4MZoFqR/noCY=
+github.com/charmbracelet/lipgloss v1.1.0/go.mod h1:/6Q8FR2o+kj8rz4Dq0zQc3vYf7X+B0binUUBwA0aL30=
+github.com/charmbracelet/x/ansi v0.11.6 h1:GhV21SiDz/45W9AnV2R61xZMRri5NlLnl6CVF7ihZW8=
+github.com/charmbracelet/x/ansi v0.11.6/go.mod h1:2JNYLgQUsyqaiLovhU2Rv/pb8r6ydXKS3NIttu3VGZQ=
+github.com/charmbracelet/x/cellbuf v0.0.15 h1:ur3pZy0o6z/R7EylET877CBxaiE1Sp1GMxoFPAIztPI=
+github.com/charmbracelet/x/cellbuf v0.0.15/go.mod h1:J1YVbR7MUuEGIFPCaaZ96KDl5NoS0DAWkskup+mOY+Q=
+github.com/charmbracelet/x/term v0.2.2 h1:xVRT/S2ZcKdhhOuSP4t5cLi5o+JxklsoEObBSgfgZRk=
+github.com/charmbracelet/x/term v0.2.2/go.mod h1:kF8CY5RddLWrsgVwpw4kAa6TESp6EB5y3uxGLeCqzAI=
+github.com/clipperhouse/displaywidth v0.9.0 h1:Qb4KOhYwRiN3viMv1v/3cTBlz3AcAZX3+y9OLhMtAtA=
+github.com/clipperhouse/displaywidth v0.9.0/go.mod h1:aCAAqTlh4GIVkhQnJpbL0T/WfcrJXHcj8C0yjYcjOZA=
+github.com/clipperhouse/stringish v0.1.1 h1:+NSqMOr3GR6k1FdRhhnXrLfztGzuG+VuFDfatpWHKCs=
+github.com/clipperhouse/stringish v0.1.1/go.mod h1:v/WhFtE1q0ovMta2+m+UbpZ+2/HEXNWYXQgCt4hdOzA=
+github.com/clipperhouse/uax29/v2 v2.5.0 h1:x7T0T4eTHDONxFJsL94uKNKPHrclyFI0lm7+w94cO8U=
+github.com/clipperhouse/uax29/v2 v2.5.0/go.mod h1:Wn1g7MK6OoeDT0vL+Q0SQLDz/KpfsVRgg6W7ihQeh4g=
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f h1:Y/CXytFA4m6baUTXGLOoWe4PQhGxaX0KpnayAqC48p4=
+github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f/go.mod h1:vw97MGsxSvLiUE2X8qFplwetxpGLQrlU1Q9AUEIzCaM=
 github.com/fatih/color v1.18.0 h1:S8gINlzdQ840/4pfAwic/ZE0djQEH3wM94VfqLTZcOM=
 github.com/fatih/color v1.18.0/go.mod h1:4FelSpRwEGDpQ12mAdzqdOukCy4u8WUtOY6lkT/6HfU=
 github.com/frankban/quicktest v1.14.6 h1:7Xjx+VpznH+oBnejlPUj8oUpdxnVs4f8XU8WnHkI4W8=
@@ -17,15 +47,29 @@ github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
 github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
 github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
 github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
+github.com/lucasb-eyer/go-colorful v1.3.0 h1:2/yBRLdWBZKrf7gB40FoiKfAWYQ0lqNcbuQwVHXptag=
+github.com/lucasb-eyer/go-colorful v1.3.0/go.mod h1:R4dSotOR9KMtayYi1e77YzuveK+i7ruzyGqttikkLy0=
 github.com/mattn/go-colorable v0.1.13 h1:fFA4WZxdEF4tXPZVKMLwD8oUnCTTo08duU7wxecdEvA=
 github.com/mattn/go-colorable v0.1.13/go.mod h1:7S9/ev0klgBDR4GtXTXX8a3vIGJpMovkB8vQcUbaXHg=
 github.com/mattn/go-isatty v0.0.16/go.mod h1:kYGgaQfpe5nmfYZH+SKPsOc2e4SrIfOl2e/yFXSvRLM=
 github.com/mattn/go-isatty v0.0.20 h1:xfD0iDuEKnDkl03q4limB+vH+GxLEtL/jb4xVJSWWEY=
 github.com/mattn/go-isatty v0.0.20/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
+github.com/mattn/go-localereader v0.0.1 h1:ygSAOl7ZXTx4RdPYinUpg6W99U8jWvWi9Ye2JC/oIi4=
+github.com/mattn/go-localereader v0.0.1/go.mod h1:8fBrzywKY7BI3czFoHkuzRoWE9C+EiG4R1k4Cjx5p88=
+github.com/mattn/go-runewidth v0.0.19 h1:v++JhqYnZuu5jSKrk9RbgF5v4CGUjqRfBm05byFGLdw=
+github.com/mattn/go-runewidth v0.0.19/go.mod h1:XBkDxAl56ILZc9knddidhrOlY5R/pDhgLpndooCuJAs=
+github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6 h1:ZK8zHtRHOkbHy6Mmr5D264iyp3TiX5OmNcI5cIARiQI=
+github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6/go.mod h1:CJlz5H+gyd6CUWT45Oy4q24RdLyn7Md9Vj2/ldJBSIo=
+github.com/muesli/cancelreader v0.2.2 h1:3I4Kt4BQjOR54NavqnDogx/MIoWBFa0StPA8ELUXHmA=
+github.com/muesli/cancelreader v0.2.2/go.mod h1:3XuTXfFS2VjM+HTLZY9Ak0l6eUKfijIfMUZ4EgX0QYo=
+github.com/muesli/termenv v0.16.0 h1:S5AlUN9dENB57rsbnkPyfdGuWIlkmzJjbFf0Tf5FWUc=
+github.com/muesli/termenv v0.16.0/go.mod h1:ZRfOIKPFDYQoDFF4Olj7/QJbW60Ol/kL1pU3VfY/Cnk=
 github.com/pelletier/go-toml/v2 v2.2.4 h1:mye9XuhQ6gvn5h28+VilKrrPoQVanw5PMw/TB0t5Ec4=
 github.com/pelletier/go-toml/v2 v2.2.4/go.mod h1:2gIqNv+qfxSVS7cM2xJQKtLSTLUE9V8t9Stt+h56mCY=
 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/rivo/uniseg v0.4.7 h1:WUdvkW8uEhrYfLC4ZzdpI2ztxP1I582+49Oc5Mq64VQ=
+github.com/rivo/uniseg v0.4.7/go.mod h1:FN3SvrM+Zdj16jyLfmOkMNblXMcoc8DfTHruCPUcx88=
 github.com/rogpeppe/go-internal v1.9.0 h1:73kH8U+JUqXU8lRuOHeVHaa/SZPifC7BkcraZVejAe8=
 github.com/rogpeppe/go-internal v1.9.0/go.mod h1:WtVeX8xhTBvf0smdhujwtBcq4Qrzq/fJaraNFVN+nFs=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
@@ -48,8 +92,13 @@ github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu
 github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
 github.com/subosito/gotenv v1.6.0 h1:9NlTDc1FTs4qu0DDq7AEtTPNw6SVm7uBMsUCUjABIf8=
 github.com/subosito/gotenv v1.6.0/go.mod h1:Dk4QP5c2W3ibzajGcXpNraDfq2IrhjMIvMSWPKKo0FU=
+github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e h1:JVG44RsyaB9T2KIHavMF/ppJZNG9ZpyihvCd0w101no=
+github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e/go.mod h1:RbqR21r5mrJuqunuUZ/Dhy/avygyECGrLceyNeo4LiM=
 go.yaml.in/yaml/v3 v3.0.4 h1:tfq32ie2Jv2UxXFdLJdh3jXuOzWiL1fo0bu/FbuKpbc=
 go.yaml.in/yaml/v3 v3.0.4/go.mod h1:DhzuOOF2ATzADvBadXxruRBLzYTpT36CKvDb3+aBEFg=
+golang.org/x/exp v0.0.0-20231006140011-7918f672742d h1:jtJma62tbqLibJ5sFQz8bKtEM8rJBtfilJ2qTU199MI=
+golang.org/x/exp v0.0.0-20231006140011-7918f672742d/go.mod h1:ldy0pHrwJyGW56pPQzzkH36rKxoZW1tw7ZJpeKx+hdo=
+golang.org/x/sys v0.0.0-20210809222454-d867a43fc93e/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220811171246-fbc7d0a398ab/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.38.0 h1:3yZWxaJjBmCWXqhN1qh02AkOnCQ1poK6oF+a7xWL6Gc=

internal/errors/errors.go

@@ -9,7 +9,6 @@ type GitError struct {
 }
 
 func (e *GitError) Error() string {
-	_ = e.Err // keep field used for error message
 	return fmt.Sprintf("git %s failed: %v", e.Command, e.Err)
 }
 

internal/errors/errors_test.go

@@ -17,8 +17,8 @@ func TestGitError(t *testing.T) {
 		t.Errorf("GitError.Error() = %q, want %q", gitErr.Error(), expected)
 	}
 
-	if !errors.Is(gitErr, baseErr) {
-		t.Errorf("GitError did not wrap base error")
+	if gitErr.Unwrap() != baseErr {
+		t.Errorf("GitError.Unwrap() did not return base error")
 	}
 }
 
@@ -68,8 +68,8 @@ func TestFileError(t *testing.T) {
 		t.Errorf("FileError.Error() = %q, want %q", fileErr.Error(), expected)
 	}
 
-	if !errors.Is(fileErr, baseErr) {
-		t.Errorf("FileError did not wrap base error")
+	if fileErr.Unwrap() != baseErr {
+		t.Errorf("FileError.Unwrap() did not return base error")
 	}
 }
 
@@ -80,7 +80,7 @@ func TestAPIErrorUnwrap(t *testing.T) {
 		Message:    "internal error",
 		Err:        baseErr,
 	}
-	if !errors.Is(apiErr, baseErr) {
-		t.Errorf("APIError.Unwrap() = %v, want %v", apiErr.Unwrap(), baseErr)
+	if unwrap := apiErr.Unwrap(); unwrap != baseErr {
+		t.Errorf("APIError.Unwrap() = %v, want %v", unwrap, baseErr)
 	}
 }

internal/git/git.go

@@ -12,7 +12,6 @@ func Run(args []string) (string, error) {
 	cmdStr := "git " + strings.Join(args, " ")
 	logger.Debug("executing git command", "command", cmdStr, "args", args)
 
-	// nolint:gosec // intentional subprocess for git operation
 	out, err := exec.Command("git", args...).Output()
 	if err != nil {
 		logger.Error("git command failed",
@@ -31,32 +30,8 @@ func Run(args []string) (string, error) {
 
 func IsRepo() bool {
 	logger.Debug("checking if directory is a git repository")
-	// nolint:gosec // intentional subprocess for git repository check
 	_, err := exec.Command("git", "rev-parse", "--is-inside-work-tree").Output()
 	isRepo := err == nil
 	logger.Debug("git repository check completed", "is_repo", isRepo)
 	return isRepo
 }
-
-func LatestTag() (string, error) {
-	out, err := Run([]string{"describe", "--tags", "--abbrev=0"})
-	if err != nil {
-		return "", err
-	}
-	return strings.TrimSpace(out), nil
-}
-
-// PreviousTag returns the tag immediately before the given one.
-func PreviousTag(current string) (string, error) {
-	out, err := Run([]string{"describe", "--tags", "--abbrev=0", current + "^"})
-	if err != nil {
-		return "", err
-	}
-	return strings.TrimSpace(out), nil
-}
-
-// LogSince returns formatted commit log since the given ref (exactly matches the todo spec).
-func LogSince(since string) (string, error) {
-	args := []string{"log", "--pretty=format:%s%n%b%n---", since + "..HEAD"}
-	return Run(args)
-}

internal/grok/client.go

@@ -76,23 +76,20 @@ func (c *Client) streamInternal(messages []map[string]string, model string, temp
 		"stream":      true,
 	}
 
-	ctx, cancel := context.WithTimeout(context.Background(), 60*time.Second)
-	// Manual cancel before os.Exit; otherwise defer is fine for the main path.
-	defer cancel()
-
 	body, err := json.Marshal(payload)
 	if err != nil {
 		logger.Error("failed to marshal API request", "error", err)
 		color.Red("Failed to marshal request: %v", err)
-		cancel()
 		os.Exit(1)
 	}
 
-	req, err := http.NewRequestWithContext(ctx, http.MethodPost, url, bytes.NewReader(body))
+	ctx, cancel := context.WithTimeout(context.Background(), 60*time.Second)
+	defer cancel()
+
+	req, err := http.NewRequestWithContext(ctx, "POST", url, bytes.NewReader(body))
 	if err != nil {
 		logger.Error("failed to create HTTP request", "error", err, "url", url)
 		color.Red("Failed to create request: %v", err)
-		cancel()
 		os.Exit(1)
 	}
 	req.Header.Set("Authorization", "Bearer "+c.APIKey)
@@ -107,7 +104,6 @@ func (c *Client) streamInternal(messages []map[string]string, model string, temp
 			"model", model,
 			"duration_ms", time.Since(startTime).Milliseconds())
 		color.Red("Request failed: %v", err)
-		cancel()
 		os.Exit(1)
 	}
 	defer func() { _ = resp.Body.Close() }()

internal/linter/linter.go

@@ -1,7 +1,6 @@
 package linter
 
 import (
-	"errors"
 	"fmt"
 	"os"
 	"os/exec"
@@ -215,7 +214,7 @@ func FindAvailableLinter(lang *Language) (*Linter, error) {
 	}
 
 	// Build install instructions
-	installOptions := make([]string, 0, len(lang.Linters))
+	var installOptions []string
 	for _, linter := range lang.Linters {
 		installOptions = append(installOptions,
 			fmt.Sprintf("  - %s: %s", linter.Name, linter.InstallInfo))
@@ -233,12 +232,10 @@ func RunLinter(filePath string, linter Linter) (*LintResult, error) {
 		"command", linter.Command)
 
 	// Build command arguments
-	linterArgs := append([]string{}, linter.Args...)
-	linterArgs = append(linterArgs, filePath)
+	args := append(linter.Args, filePath)
 
 	// Execute linter
-	// nolint:gosec // intentional subprocess for linter
-	cmd := exec.Command(linter.Command, linterArgs...)
+	cmd := exec.Command(linter.Command, args...)
 	output, err := cmd.CombinedOutput()
 
 	result := &LintResult{
@@ -248,8 +245,7 @@ func RunLinter(filePath string, linter Linter) (*LintResult, error) {
 	}
 
 	if err != nil {
-		var exitErr *exec.ExitError
-		if errors.As(err, &exitErr) {
+		if exitErr, ok := err.(*exec.ExitError); ok {
 			result.ExitCode = exitErr.ExitCode()
 			result.HasIssues = true
 			logger.Info("linter found issues",
@@ -306,7 +302,7 @@ func LintFile(filePath string) (*LintResult, error) {
 
 // GetSupportedLanguages returns a list of all supported languages
 func GetSupportedLanguages() []string {
-	langs := make([]string, 0, len(languages))
+	var langs []string
 	for _, lang := range languages {
 		langs = append(langs, fmt.Sprintf("%s (%s)", lang.Name, strings.Join(lang.Extensions, ", ")))
 	}

internal/linter/linter_test.go

@@ -151,7 +151,7 @@ func main() {
 	fmt.Println("Hello, World!")
 }
 `
-	if err := os.WriteFile(testFile, []byte(validCode), 0600); err != nil {
+	if err := os.WriteFile(testFile, []byte(validCode), 0644); err != nil {
 		t.Fatalf("Failed to create test file: %v", err)
 	}
 
@@ -207,7 +207,7 @@ func main() {
 	fmt.Println("Hello, World!")
 }
 `
-		if err := os.WriteFile(testFile, []byte(validCode), 0600); err != nil {
+		if err := os.WriteFile(testFile, []byte(validCode), 0644); err != nil {
 			t.Fatalf("Failed to create test file: %v", err)
 		}
 
@@ -237,7 +237,7 @@ func main() {
 
 	t.Run("Lint unsupported file type", func(t *testing.T) {
 		testFile := filepath.Join(tmpDir, "test.txt")
-		if err := os.WriteFile(testFile, []byte("hello"), 0600); err != nil {
+		if err := os.WriteFile(testFile, []byte("hello"), 0644); err != nil {
 			t.Fatalf("Failed to create test file: %v", err)
 		}
 

internal/logger/logger.go

@@ -21,12 +21,12 @@ func Init(logLevel string) error {
 	}
 
 	logDir := filepath.Join(home, ".config", "grokkit")
-	if err := os.MkdirAll(logDir, 0750); err != nil {
+	if err := os.MkdirAll(logDir, 0755); err != nil {
 		return err
 	}
 
 	logFile := filepath.Join(logDir, "grokkit.log")
-	file, err := os.OpenFile(logFile, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0600)
+	file, err := os.OpenFile(logFile, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0644)
 	if err != nil {
 		return err
 	}

internal/version/version_test.go

@@ -33,7 +33,6 @@ func TestVersionInfo(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			t.Parallel()
 			tt.check(t)
 		})
 	}

release.sh

@@ -1,65 +0,0 @@
-#!/bin/bash
-# release.sh — One-command release driver for Grokkit
-# Usage: ./release.sh v0.2.3
-
-set -euo pipefail
-
-VERSION="${1:-}"
-
-if [[ -z "$VERSION" || ! "$VERSION" =~ ^v[0-9]+\.[0-9]+\.[0-9]+ ]]; then
-  echo "❌ Usage: $0 vX.Y.Z"
-  echo "   Example: $0 v0.2.3"
-  exit 1
-fi
-
-echo "🚀 Starting release process for $VERSION..."
-
-# Safety check: clean working tree
-if [[ -n $(git status --porcelain) ]]; then
-  echo "❌ Working tree is dirty. Commit or stash changes first."
-  exit 1
-fi
-
-# Final human confirmation before anything touches git
-echo ""
-echo "This will:"
-echo "  1. Create git tag $VERSION"
-echo "  2. Run grokkit changelog (with preview + your confirmation)"
-echo "  3. Stage CHANGELOG.md and run grokkit commit (AI message + confirmation)"
-echo "  4. Push commit + tag"
-echo ""
-read -p "Proceed with release $VERSION? (y/N) " -n 1 -r
-echo
-if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-  echo "Aborted."
-  exit 1
-fi
-
-# 1. Tag first (so changelog can see it as LatestTag)
-echo "🏷️  Creating tag $VERSION..."
-git tag "$VERSION"
-
-# 2. Generate changelog (interactive preview + confirm inside the command)
-echo "📝 Generating CHANGELOG.md section..."
-grokkit changelog --version "$VERSION"
-
-# 3. Commit with Grok (uses the exact same safety flow you already love)
-echo "📦 Staging and committing changelog..."
-git add CHANGELOG.md
-grokkit commit
-
-# 4. Push everything
-echo "📤 Pushing commit + tag..."
-git push
-git push --tags
-
-# 5. Show the exact notes ready for Gitea release page
-echo ""
-echo "✅ Release $VERSION complete!"
-echo ""
-echo "📋 Copy-paste this into the Gitea release notes:"
-echo "------------------------------------------------"
-grokkit changelog --stdout --version "$VERSION"
-echo "------------------------------------------------"
-echo ""
-echo "🎉 Done! Go create the release on Gitea now."

todo/README.md

@@ -1,27 +0,0 @@
-# Grokkit TODO List
-
-This document provides a table of contents for all tasks and features currently tracked in the `todo/` directory.
-
-## Queued
-
-* [1] [interactive-agent.md](./queued/interactive-agent.md) : Grokkit Interactive Agent
-* [2] [rg.md](./queued/rg.md) : grokkit agent ripgrep (rg) integration
-* [3] [gotools.md](./queued/gotools.md) : grokkit agent Go tools integration
-* [4] [make.md](./queued/make.md) : grokkit agent make integration
-* [5] [tea.md](./queued/tea.md) : grokkit agent tea (Gitea CLI) integration
-* [6] [cnotes.md](./queued/cnotes.md) : grokkit agent cnotes integration
-* [7] [profile.md](./queued/profile.md) : grokkit profile
-* [8] [pprof.md](./queued/pprof.md) : grokkit agent pprof integration
-* [9] [audit.md](./queued/audit.md) : grokkit audit
-* [10] [git-chglog.md](./queued/git-chglog.md) : grokkit agent git-chglog integration
-* [11] [admin.md](./queued/admin.md) : grokkit admin tool (to show token usage and other admin-only features)
-* [99] [TODO_ITEM.md](./queued/TODO_ITEM.md) : TODO ITEM template
-
-## Completed
-
-* [non-interactive-query](./completed/non-interactive-query.md) : grokkit query *(done - v0.1.9)*
-* [changelog.md](./completed/changelog.md) : grokkit changelog *(done — v0.1.8+)*
-* [3-new-feature-suggestions.md](./completed/3-new-feature-suggestions.md) : 3 New AI-Enhanced Feature Suggestions for Grokkit
-* [MODEL_ENFORCEMENT.md](./completed/MODEL_ENFORCEMENT.md) : Model Enforcement
-* [scaffold.md](./completed/scaffold.md) : grokkit scaffold
-* [testgen.md](./completed/testgen.md) : grokkit testgen [path...]

todo/completed/non-interactive-query.md

@@ -1,138 +0,0 @@
-# `grokkit query` Simple Query Tool
-## Priority: x of 13
-
-**Description**: A "one shot" prompt/answer query tool. Use it to ask StackOverflow style questions, like "how do I do x? Or What library do I use for y?". The sweet spot is concise, actionable answers with just enough context, a code snippet if relevant, and no unnecessary fluff. Think: problem solved in 100-300 words, tops. However, a "wordy" flag will be included to allow for full "textbook" style answers, as well.
-
-## Problem It Solves
-
-Reduces the amount of context switching needed to answer basic programming questions.
-
-### Example Short Query 1: 
-
-```
-grokkit query "What is the difference between a slice and a map in Go?"
-
-Slices are dynamic arrays for ordered sequences of elements (e.g., `[]int`), with length/capacity and fast index access. Use for lists or queues; append with `append(s, val)`.
-
-Maps are unordered key-value stores (e.g., `map[string]int`), for fast lookups/deletes via hashing. Use for dictionaries or caches; access with `m[key]`.
-
-Key diffs: Slices are indexed by integers and ordered; maps use arbitrary keys and are unordered. Both are reference types, initialized with `make`.
-
-Example:
-    ```go
-    s := []int{1, 2, 3} // Slice
-    m := map[string]int{"a": 1} // Map
-    ```
-```
-### Example Short Query 2: 
-```
-"$ grokkit query "Are there any other languages with a concept like Python list comprehensions?"
-
-Yes, several languages have similar concise syntax for creating collections via expressions:
-
-- **Haskell**: List comprehensions, e.g., `[x*2 | x <- [1..5]]` (outputs `[2,4,6,8,10]`).
-- **Scala**: For-comprehensions, e.g., `for (x <- 1 to 5) yield x*2`.
-- **JavaScript (ES6+)**: Array methods like `map`/`filter`, e.g., `[1,2,3,4,5].map(x => x*2)`. Not identical, but functional equivalent.
-- **C#**: LINQ queries, e.g., `from x in Enumerable.Range(1,5) select x*2`.
-- **Rust**: Iterators with `map`/`collect`, e.g., `(1..=5).map(|x| x*2).collect::<Vec<i32>>()`.
-
-Python's are inspired by Haskell. For exact matches, Haskell is closest.
-```
-
-### Example Long Query: 
-```
-$ grokkit query "What is the difference between a slice and a map in Go?" --wordy
-
-Ah, slices and maps in Go—two of the language's built-in data structures that make life easier (or at least less array-tastic) for developers. They're both reference types, meaning they're lightweight and point to underlying data, but they serve very different purposes. I'll break it down step by step, with some examples, and keep it straightforward. If Go had a sense of humor, it'd say slices are for cutting things up neatly, while maps are for when you lose your keys... again.
-
-### What is a Slice?
-A slice is essentially a dynamic, resizable view of an array. Think of it as a flexible array that can grow or shrink without you having to manage the underlying memory manually. Slices are backed by an array under the hood, but they're more convenient because:
-- They have a length (how many elements are currently in it) and a capacity (how much room is allocated before it needs to resize).
-- You can append to them, slice them further (hence the name), and they're zero-initialized by default.
-- They're ordered: Elements are accessed by integer indices, starting from 0.
-
-**Key traits:**
-- Type: `[]T` (e.g., `[]int` for a slice of integers).
-- Ordered? Yes.
-- Use case: Lists, stacks, queues, or any sequence of items where order matters and you need fast random access.
-
-**Example:**
-
-    ```go
-    package main
-    
-    import "fmt"
-    
-    func main() {
-        // Create a slice with make (initial length 3, capacity 5)
-        s := make([]string, 3, 5)
-        s[0] = "apple"
-        s[1] = "banana"
-        s[2] = "cherry"
-    
-        // Append to it (grows if needed)
-        s = append(s, "date")
-    
-        fmt.Println(s)       // Output: [apple banana cherry date]
-        fmt.Println(s[1:3])  // Slicing: [banana cherry]
-    }
-    ```
-
-Slices are great for when you want array-like behavior without the fixed-size hassle of actual arrays in Go.
-
-### What is a Map?
-A map is Go's version of a hash table or dictionary: an unordered collection of key-value pairs. You use it to associate unique keys with values, and it's super efficient for lookups, inserts, and deletes (average O(1) time complexity, thanks to hashing).
-- Keys must be comparable (e.g., strings, ints, structs without pointers), but values can be anything.
-- Maps are unordered, so don't rely on iteration order—it's not guaranteed (though as of Go 1.12, it's pseudo-random but consistent per run for debugging).
-- They're also nil-safe: An uninitialized map is nil, and you can't add to it until you `make` it.
-
-**Key traits:**
-- Type: `map[K]V` (e.g., `map[string]int` for string keys and int values).
-- Ordered? No (unordered by design).
-- Use case: Lookup tables, caches, configurations, or anywhere you need fast key-based access without caring about order.
-
-**Example:**
-
-    ```go
-    package main
-    
-    import "fmt"
-    
-    func main() {
-        // Create a map with make
-        m := make(map[string]int)
-        m["apple"] = 1
-        m["banana"] = 2
-        m["cherry"] = 3
-    
-        // Access by key
-        fmt.Println(m["banana"])  // Output: 2
-    
-        // Check if a key exists
-        value, exists := m["date"]
-        if !exists {
-            fmt.Println("Date not found!")  // This will print
-        }
-    
-        // Delete a key
-        delete(m, "cherry")
-    }
-    ```
-Maps are your go-to when you need to map one thing to another without the linear search nonsense.
-
-### Key Differences
-- **Structure and Purpose**: Slices are sequences of elements (like a list or array), ideal for ordered collections. Maps are associative arrays for key-value storage, perfect for quick lookups by key.
-- **Access Method**: Slices use integer indices (e.g., `s[42]`). Maps use keys of any hashable type (e.g., `m["foo"]`).
-- **Ordering**: Slices preserve insertion order and allow indexing. Maps are unordered—iteration might give you keys in a seemingly random order.
-- **Resizing and Capacity**: Slices have explicit length and capacity; they can grow via `append`. Maps grow automatically as you add elements, but you don't control capacity directly.
-- **Initialization**: Both often use `make` for creation, but slices can be literals like `[]int{1, 2, 3}`, while maps use `map[string]int{"a": 1}`.
-- **Performance**: Slices are contiguous in memory (fast iteration and access), maps use hashing (fast lookups but more overhead for small sizes).
-- **Nil Behavior**: A nil slice can still be appended to (it acts like an empty slice). A nil map will panic if you try to read/write to it.
-- **When to Use Which**: Use a slice if you need a list (e.g., processing a queue of tasks). Use a map if you need to associate data (e.g., user IDs to profiles).
-
-In short, if slices are like a neatly organized toolbox drawer, maps are like a junk drawer where you can find stuff quickly by yelling its name—but don't expect it to be in any particular spot.
-```
-
-## ROI
-
-**Medium**

todo/completed/scaffold.md

@@ -1,87 +0,0 @@
-# `grokkit scaffold`
-## Priority: 1 of 12
-
-**Description**: AI-powered code generation that creates new files from natural language descriptions, learning from your existing codebase patterns to produce idiomatic, ready-to-compile output.
-
-## Problem It Solves
-
-Every existing grokkit command operates on *existing* files (`edit`, `agent`, `lint`, `docs`, `testgen`). There is no command to *create* a new file from scratch. Developers spend significant time writing boilerplate for new handlers, services, models, CLI commands, middleware, etc. `scaffold` fills this gap.
-
-## Benefits
-
-- **Pattern-aware**: Reads similar existing files for package names, import styles, error handling conventions, struct naming, and logger usage before generating.
-- **Idiomatic output**: Produces code that looks like it was written by the same developer — same patterns as the rest of the codebase, not generic AI output.
-- **Multi-language**: Works for any language the project already supports (Go, Python, JS/TS, etc.), with lang detected from the output path or `--lang` flag.
-- **Optional test companion**: With `--with-tests`, also generates a corresponding test file using the same testgen prompt logic.
-- **Safe preview**: Shows a diff/preview before writing, requires confirmation (or `--yes` to skip).
-- **No overwrites**: Refuses to overwrite an existing file unless `--force` is passed.
-
-## CLI Examples
-
-```bash
-# Generate a new Cobra CLI command file
-grokkit scaffold cmd/export.go "export command that writes chat history to JSON or CSV"
-
-# New internal package
-grokkit scaffold internal/cache/cache.go "simple in-memory LRU cache with TTL support"
-
-# Generate with a companion test file
-grokkit scaffold internal/git/tag.go "functions to list and create git tags" --with-tests
-
-# Python module
-grokkit scaffold src/auth/jwt.py "JWT encode/decode utilities using PyJWT"
-
-# Skip confirmation
-grokkit scaffold cmd/webhook.go "webhook command that posts PR descriptions to Slack" --yes
-
-# Preview only (no write)
-grokkit scaffold cmd/watch.go "file watcher that reruns lint on save" --dry-run
-```
-
-## High-Level Implementation
-
-1. **Input validation**: Check target path doesn't already exist (error unless `--force`). Detect language from file extension.
-2. **Context harvesting**: Walk sibling files in the same directory and the most structurally similar files in the project (e.g., other `cmd/*.go` if generating a command). Read them for patterns.
-3. **Prompt construction**:
-   - System: "You are an expert Go/Python/etc. engineer. Given existing code samples, generate a new file that matches the codebase's conventions exactly. Return only the file content — no explanations, no markdown."
-   - User: paste harvested sibling files as context + target path + description.
-4. **Streaming response**: Use `client.StreamSilent` (same as `edit`/`testgen`).
-5. **Clean output**: Run through `grok.CleanCodeResponse` to strip any markdown fences.
-6. **Preview**: Print file content with a header showing the target path. Prompt `(y/n)`.
-7. **Write**: Create parent directories if needed (`os.MkdirAll`), write file.
-8. **Optional test generation**: If `--with-tests`, reuse testgen logic to generate companion test file with the same preview/confirm flow.
-
-## Flags
-
-| Flag | Description |
-|---|---|
-| `--with-tests` | Also generate companion test file |
-| `--yes` / `-y` | Skip confirmation prompt |
-| `--dry-run` | Preview only, do not write |
-| `--force` | Overwrite if target file already exists |
-| `--lang` | Override language detection (go, python, js, ts, rust, ruby) |
-| `--model` / `-m` | Override model |
-| `--context-files` | Comma-separated paths to use as context (overrides auto-detection) |
-
-## Implementation Notes
-
-- **Context size**: Cap harvested context at ~4000 tokens to avoid prompt bloat. Prefer the most relevant files (same directory, same `package` name for Go).
-- **Go specifics**: Extract `package` name from sibling files and inject it at the top of the prompt so the AI uses the correct package declaration.
-- **Reuse existing patterns**: Share `internal/linter.DetectLanguage`, `grok.CleanCodeResponse`, and the standard preview/confirm UX from `edit.go`.
-- **Effort**: Medium (~250–350 LOC). Most complexity is in the context-harvesting logic.
-
-## ROI
-
-**High**. This closes the only remaining gap in grokkit's file lifecycle coverage:
-
-| Lifecycle stage | Command |
-|---|---|
-| Create new file | **scaffold** ← missing |
-| Edit existing file | `edit` |
-| Multi-file changes | `agent` |
-| Review changes | `review` |
-| Document code | `docs` |
-| Generate tests | `testgen` |
-| Commit | `commit` |
-
-Developers scaffold new files multiple times per day. Making that AI-assisted and pattern-aware is a force multiplier for the entire tool.

todo/queued/admin.md

@@ -1,19 +0,0 @@
-# `grokkit admin` administration tools
-
-### Category: Nice to Have
-
-Add an `admin` command that, if an `XAI_MGMT_KEY` is set in the environment, displays up-to-date token usage on the api account/team.
-
-### Details
-- **Command**: `grokkit admin`
-- **Subcommands**:
-    - `grokkit admin token-usage`    <-- default (shows token usage for current team)
-    - `grokkit admin credit-balance` <-- shows credit balance for current team
-    - `grokkit admin models`         <-- shows available models
-    - `grokkit admin api-keys`       <-- shows available API keys
-
-- **Condition**: Must check for `XAI_MGMT_KEY` env var.
-- **Functionality**: Fetch and display token usage stats from the [XAI MANAGEMENT API](https://docs.x.ai/developers/rest-api-reference/management.
-- **Goal**: Help me monitor my API costs and limits directly from the CLI.
-
-NOTE: If possible, it would be cool if this command was "hidden" if the XAI_MGMT_KEY is not set.

todo/queued/audit.md

@@ -1,26 +0,0 @@
-# `grokkit audit`
-**Description**: Comprehensive AI-powered code audit for security, performance, best practices, and potential bugs across single files or entire projects.
-
-**Benefits**:
-- Deep analysis beyond static linters: vulns (e.g., SQLi, race cond), perf hotspots, Go idioms violations.
-- Generates actionable report + diff previews for fixes.
-- Multi-language support (reuse testgen/lint patterns).
-- Boosts code quality/PR readiness.
-
-**High-level implementation**:
-- Detect lang/files (internal/linter), collect code snippets/context.
-- Prompt Grok: "Audit for security, perf, best practices, bugs. List issues prioritized + suggested code fixes."
-- Output: Markdown report (sections: Critical/High/Med/Low), optional `--fix` generates edit previews.
-- Reuse `edit` preview/apply workflow; add `--apply` flag.
-
-**CLI example**:
-```
-grokkit audit main.go                    # Single file report
-grokkit audit ./cmd --lang=go             # Dir, lang filter
-grokkit audit . --fix                     # Report + fix previews
-grokkit audit . --fix --yes               # Auto-apply fixes (dangerous, preview first)
-```
-
-**Similar to**: lint/review but deeper, proactive fixes.
-**Effort**: Medium (prompt tuning, multi-file handling ~300 LOC).
-**ROI**: High - daily dev essential, esp. before releases.

todo/queued/changelog.md

@@ -1,6 +1,5 @@
 # `grokkit changelog`
-## Priority: 1 of 12
-**Description**: AI-generated CHANGELOG.md updates from git history (commits/tags). The generated CHANGELOG.md should be used as the basis for the release notes for the release page of each release on gitea.
+**Description**: AI-generated CHANGELOG.md updates from git history (commits/tags).
 
 **Benefits**:
 - Automates semantic release notes (feat/fix/docs/etc.).

todo/queued/cnotes.md

@@ -1,73 +0,0 @@
-# `grokkit agent` cnotes integration
-
-**Description**: Wrappers for your `cnotes` CLI logging suite. Allows Grok to automatically log coding sessions, notes, progress during agent workflows (e.g., "start work on feature", "log bug found").
-
-## Problem It Solves
-
-Developers track time/effort manually. Integrate `cnotes` (cnadd/cndump/etc.) for AI-assisted logging: timestamps, descriptions, stats—seamless with code edits.
-
-## Benefits
-
-- **Automated logging**: Grok logs at key points (start/edit/commit).
-- **Queryable**: Search/dump stats via AI ("Show time on agent tasks today").
-- **Safe**: Append-only (`cnadd`), read-only queries (`cnfind`/`cncount`); no deletes.
-- **Terminal-native**: Fast C impl, no deps.
-- **Workflow boost**: "Refactor → log changes → stats".
-
-## Agent Tool Examples
-
-```
-grokkit agent "Start logging session, fix main.go, log completion"
-# Grok: cnadd "start: agent refactor main.go" → edits → cnadd "done: fixed errors"
-```
-
-```
-grokkit agent "Log today's coding stats before planning"
-# Grok: cncount → cnfind "today" → "2h on edits, 3 sessions"
-```
-
-## High-Level Implementation
-
-1. **Detect cnotes**: Run `cnhelp` or `which cnotes`.
-2. **Tool schemas**:
-   - `log_note(message: string) → ok`
-   - `list_notes(filter?: string) → string`
-   - `session_stats() → string`
-3. **Wrappers** in `internal/tools/cnotes.go`:
-   ```go
-   func LogNote(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Auto-log on milestones; user-prompted.
-5. **Safety**:
-   - Only `cnadd`/`cndump`/`cnfind`/`cncount`.
-   - Preview log message.
-   - Config: `[tools.cnotes.enabled]`.
-6. **Parsing**: Stdout JSON? Or simple text.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.cnotes.enabled` | Enable cnotes tools |
-| `--auto-log` | Log start/end automatically |
-
-## Implementation Notes
-
-- **Commands**: cnadd (log), cndump (all), cnfind (search), cncount (stats), cndel (archive—whitelist carefully).
-- **Extend agent**: Tool calls in loop.
-- **Errors**: Custom CnotesError.
-- **Tests**: Mock `exec.Command`, table-driven.
-- **Effort**: Low (~150 LOC). Simple exec wrappers.
-- **Prereq**: cnotes in PATH (`cnhelp` verifies).
-
-## ROI
-
-**High**. Personal logging elevates from code tool to **dev productivity hub**:
-
-| Stage | Covered |
-|-------|---------|
-| Logging | **cnotes** ← new |
-| Code | `agent` ✓ |
-| Repo | tea (sibling) |
-
-Quantifiable: Track hours/features automatically.

todo/queued/git-chglog.md

@@ -1,61 +0,0 @@
-# `grokkit agent` git-chglog integration
-
-**Description**: Wrapper for git-chglog CLI: generates CHANGELOG.md from conventional commits/tags. AI-enhanced parsing/validation.
-
-## Problem It Solves
-
-changelog.md is AI-from-git-log; git-chglog is standard tool—wrapper adds agent control/previews.
-
-## Benefits
-
-- **Standard**: git-chglog config + AI tweaks.
-- **Validate**: Check changelog before release.
-- **Safe**: Dry-run, output preview.
-- **Integrate**: "Generate changelog v0.2.0, commit".
-
-## Agent Tool Examples
-
-```
-grokkit agent "Generate changelog for next release from commits"
-# Grok: git-chglog --next-tag v0.2.0 → preview → append to CHANGELOG.md
-```
-
-## High-Level Implementation
-
-1. **Detect**: `git-chglog --version`.
-2. **Tool schemas**:
-   - `generate(tag: string) → changelog_md`
-   - `validate() → issues`
-3. **Wrappers** in `internal/tools/git-chglog.go`:
-   ```go
-   func Generate(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Release workflows.
-5. **Safety**:
-   - --dry-run flag.
-   - Config file respect.
-   - Config: `[tools.git-chglog.enabled]`.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.git-chglog.enabled` | Enable git-chglog |
-
-## Implementation Notes
-
-- **Commands**: `git-chglog --output CHANGELOG.md --next-tag vX.Y.Z`.
-- **Parsing**: Markdown output.
-- **Errors**: ChglogError.
-- **Tests**: Mock exec.
-- **Effort**: Low (~110 LOC). Complements changelog.md.
-- **Prereq**: git-chglog installed.
-
-## ROI
-
-**Medium-High**. Release polish:
-
-| Stage | Covered |
-|-------|---------|
-| Changelog | **git-chglog** ← new |
-| Release | changelog.md ✓ |

todo/queued/gotools.md

@@ -1,64 +0,0 @@
-# `grokkit agent` Go tools integration
-
-**Description**: Wrappers for `go` subcommands: mod tidy, generate, vet, fmt. Ensures hygiene post-agent edits.
-
-## Problem It Solves
-
-Agent changes may break deps/fmt/vet—manual fixes. Auto-run + fix loops.
-
-## Benefits
-
-- **Hygiene auto**: Tidy deps, fmt code, vet issues.
-- **Generate**: Run go:generate for boilerplate.
-- **Safe mutations**: Preview changes (git diff).
-- **Workflow**: Edit → gotools tidy → test.
-
-## Agent Tool Examples
-
-```
-grokkit agent "Add new import, tidy mods, vet all"
-# Grok: edits → go mod tidy → go vet → reports clean
-```
-
-## High-Level Implementation
-
-1. **Detect**: `go version`.
-2. **Tool schemas**:
-   - `tidy() → changes`
-   - `vet(path?) → issues`
-   - `generate() → output`
-   - `fmt_diff() → diff`
-3. **Wrappers** in `internal/tools/gotools.go`:
-   ```go
-   func Tidy(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Post-edit hygiene step.
-5. **Safety**:
-   - Dry-run where possible (go fmt -d).
-   - Whitelist subcmds.
-   - Config: `[tools.gotools.enabled]`.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.gotools.enabled` | Enable Go tools |
-
-## Implementation Notes
-
-- **Commands**: `go mod tidy`, `go vet ./...`, `go generate ./...`.
-- **Parsing**: Diff output, error lists.
-- **Errors**: GoToolError.
-- **Tests**: Mock, table-driven.
-- **Effort**: Low (~130 LOC).
-- **Prereq**: Go workspace.
-
-## ROI
-
-**High**. Go-specific polish:
-
-| Stage | Covered |
-|-------|---------|
-| Hygiene | **gotools** ← new |
-| Search | rg (sibling) |
-| Build | make (sibling) |

todo/queued/interactive-agent.md

@@ -1,53 +0,0 @@
-# Grokkit Interactive Agent
-
-
-## Goal
-
-Add a persistent, conversational **agent mode** to Grokkit so I can chat interactively in the terminal and have Grok directly edit code, run tests, commit, etc., without any copy-paste friction.
-
-This is the natural evolution of the current `chat` and `agent` commands — turning the terminal into a true agentic partner instead of a one-shot prompt tool.
-
-## Command
-
-```bash
-grokkit chat --agent # or new subcommand: grokkit agent-chat
-```
-
-(Keep the existing simple `chat` mode untouched; add an `--agent` flag or a dedicated `agent-chat` command.)
-
-## Core Features
-
-- Persistent session using the existing `chat_history.json` (with optional `--new-session` flag)
-- Full tool-calling loop: I (Grok) can invoke any existing Grokkit primitive:
-  - `edit`
-  - `scaffold`
-  - `testgen`
-  - `lint`
-  - `review`
-  - `commit` / `commit-msg`
-  - `docs`
-  - etc.
-- Safe-by-default workflow:
-  - Always preview changes (diff style, same as current `edit`/`scaffold`)
-  - Require explicit confirmation (`y/n` or `--yes`)
-  - Preview and confirm all changes before application
-- Ability to chain actions naturally in conversation:
-  - “The scaffold tests are flaky on Windows paths — fix it and add a test case.”
-  - “Refactor the context harvester to use rg, then run make test.”
-  - “Write a new command for todo prioritization and commit it.”
-- Optional `--dry-run` and `--model` flags (inherited from root)
-
-## Acceptance Criteria
-
-- `make test` remains fast (no new API cost in normal tests)
-- Live agent mode respects the same `-short` gating pattern we just built for scaffold
-- Session survives across multiple `grokkit` invocations (or can be started fresh)
-- All edits go through the existing safe preview/confirm flow — no silent changes
-- Works with the full Grok context window (no early compaction)
-- Can be extended later with vector memory or external tool plugins
-
-## Why This Matters
-
-This removes the last remaining copy-paste friction between me and the codebase. The 45-minute scaffold test iteration we just did showed the productivity gain; this turns that gain into a permanent workflow. Closes the “agentic chat” vision we discussed.
-
-

todo/queued/make.md

@@ -1,73 +0,0 @@
-# `grokkit agent` make integration
-
-**Description**: Wrappers for Makefile targets (test/lint/build/cover). Enables Grok to run/verify builds mid-agent workflow (e.g., "edit, test, fix loops").
-
-## Problem It Solves
-
-Agent edits code but can't auto-verify compilation/tests—manual `make test` context-switch.
-
-## Benefits
-
-- **Automated verification**: Post-edit `make test` + analyze failures.
-- **Dry-runs**: Preview `make build` output.
-- **Safe**: Whitelisted targets, timeout, project-dir.
-- **Parse results**: Extract pass/fail, coverage, errors for next agent step.
-- **Workflow**: "Refactor → test → fix → commit".
-
-## Agent Tool Examples
-
-```
-grokkit agent "Fix lint errors in cmd/, run make lint to verify, then test"
-# Grok: edits → make lint → "All green!" → make test → fixes failures
-```
-
-```
-grokkit agent "Benchmark changes before commit"
-# Grok: make test-cover → "Coverage drop 2%" → optimizations
-```
-
-## High-Level Implementation
-
-1. **Detect**: `test -f Makefile` or `make --version`.
-2. **Tool schemas**:
-   - `run_target(target: string) → {output: string, success: bool}`
-   - `dry_run(target: string) → simulated_output`
-3. **Wrappers** in `internal/tools/make.go`:
-   ```go
-   func RunTarget(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Tool call → parse stdout → feed to Grok ("Tests failed: fix?").
-5. **Safety**:
-   - Whitelist: test, lint, build, test-cover, install.
-   - 300s timeout.
-   - No sudo/privileged.
-   - Config: `[tools.make.enabled]`.
-6. **Parsing**: Grep for "PASS/FAIL", coverage %.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.make.enabled` | Enable make tools |
-| `tools.make.timeout` | Per-target timeout (s) |
-
-## Implementation Notes
-
-- **Commands**: `make TARGET` (no args).
-- **Extend agent**: Loop includes make calls post-edit.
-- **Errors**: MakeError with output.
-- **Tests**: Mock exec, table-driven (success/fail outputs).
-- **Effort**: Low (~120 LOC). Std exec + parsing.
-- **Prereq**: Makefile present.
-
-## ROI
-
-**High**. Agent verification loop:
-
-| Stage | Covered |
-|-------|---------|
-| Edit | `agent` ✓ |
-| Verify | **make** ← new |
-| Repo/log | tea/cnotes |
-
-Instant feedback elevates agent reliability.

todo/queued/pprof.md

@@ -1,62 +0,0 @@
-# `grokkit agent` pprof integration
-
-**Description**: Go pprof profiling wrappers (CPU/memory/allocs). Captures profiles during agent runs, AI-analyzes hotspots.
-
-## Problem It Solves
-
-Profile.md is high-level; pprof enables raw data + AI opts for slow cmds (agent loops).
-
-## Benefits
-
-- **Capture**: `go test -cpuprofile` or runtime/pprof.
-- **Analyze**: Feed pprof text/top to Grok ("Optimize this flamegraph").
-- **Safe**: Temp .pprof files, no web UI.
-- **Integrate**: Auto-profile long agents.
-
-## Agent Tool Examples
-
-```
-grokkit agent "Profile cmd/agent.go benchmark, suggest fixes"
-# Grok: pprof cpu → "30% in StreamSilent, batch calls"
-```
-
-## High-Level Implementation
-
-1. **Detect**: `go tool pprof --version`.
-2. **Tool schemas**:
-   - `cpu_profile(duration: int) → pprof_text`
-   - `analyze(profile_data: string) → suggestions`
-3. **Wrappers** in `internal/tools/pprof.go`:
-   ```go
-   func CPUProfile(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Flag-triggered profiling.
-5. **Safety**:
-   - Tempdir files (os.TempDir).
-   - No --http/--pdf.
-   - Config: `[tools.pprof.enabled]`.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.pprof.enabled` | Enable pprof tools |
-| `--profile-cpu` | Auto-cpu on agent |
-
-## Implementation Notes
-
-- **Capture**: runtime/pprof.StartCPUProfile, or `go test -cpuprofile`.
-- **Parse**: `go tool pprof -text`.
-- **Errors**: PprofError.
-- **Tests**: Mock profiles.
-- **Effort**: Medium (~180 LOC). Ties to profile.md.
-- **Prereq**: Go stdlib pprof.
-
-## ROI
-
-**High** for perf:
-
-| Stage | Covered |
-|-------|---------|
-| Profile | **pprof** ← new |
-| Analyze | profile.md ✓ |

todo/queued/rg.md

@@ -1,72 +0,0 @@
-# `grokkit agent` ripgrep (rg) integration
-
-**Description**: Fast search wrapper for ripgrep (`rg`). Enables agent to grep project for symbols/patterns/context before edits.
-
-## Problem It Solves
-
-Agent lacks quick codebase search—relies on full file scans or manual `grep`. rg is 10x faster, regex-aware.
-
-## Benefits
-
-- **Lightning search**: Multi-line, git-ignored, type-aware (`--type=go`).
-- **Context gathering**: "rg 'err handling' → snippets for prompt".
-- **Safe**: Read-only, project-only, no filespecs.
-- **Stats**: Count matches, paths.
-- **Workflow**: "Search todos → prioritize → edit".
-
-## Agent Tool Examples
-
-```
-grokkit agent "Find all error returns in cmd/, improve handling"
-# Grok: rg 'return err' cmd/ → analyzes → targeted edits
-```
-
-```
-grokkit agent "Count test coverage gaps: rg 't.Skip' or untested funcs"
-# Grok: rg --type=go '_test\.go' → stats → testgen suggestions
-```
-
-## High-Level Implementation
-
-1. **Detect**: `rg --version`.
-2. **Tool schemas**:
-   - `search(pattern: string, path?: string) → matches[]`
-   - `count(pattern: string) → int`
-3. **Wrappers** in `internal/tools/rg.go`:
-   ```go
-   func Search(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent integration**: Pre-plan search → richer context.
-5. **Safety**:
-   - Args whitelist: --type, --no-heading, project root.
-   - Max 100 matches.
-   - Config: `[tools.rg.enabled]`.
-6. **Parsing**: JSON output (`--json`), or lines → structured.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.rg.enabled` | Enable rg tools |
-| `tools.rg.max_matches` | Limit results |
-
-## Implementation Notes
-
-- **Commands**: `rg PATTERN [PATH] --no-heading --colors=never`.
-- **Extend agent**: Optional search phase in planning.
-- **Errors**: RgError.
-- **Tests**: Mock, table-driven patterns.
-- **Effort**: Low (~100 LOC).
-- **Prereq**: rg installed (`apt install ripgrep`).
-
-## ROI
-
-**High**. Search is dev superpower:
-
-| Stage | Covered |
-|-------|---------|
-| Search | **rg** ← new |
-| Edit | `agent` ✓ |
-| Verify | make (sibling) |
-
-Agent plans smarter with instant intel.

todo/queued/tea.md

@@ -1,74 +0,0 @@
-# `grokkit agent` tea integration
-
-**Description**: Safe, AI-orchestrated wrappers for Gitea `tea` CLI commands. Enables Grok to manage repos (list/create PRs/issues, comments) as part of agent workflows, with previews and confirmations.
-
-## Problem It Solves
-
-Grokkit's `agent` excels at code edits but lacks repo mgmt. Developers context-switch to browser/CLI for Gitea actions. Integrate `tea` for seamless: "Review PRs, create issue for bugs, log progress".
-
-## Benefits
-
-- **Safe execution**: Whitelisted commands/args, dry-run previews, confirm destructive ops (merge/push).
-- **AI reasoning**: Grok decides *when* to call `tea` (e.g., "After edits, create PR").
-- **Logging**: All calls logged to `~/.config/grokkit/tools.log`.
-- **Repo-aware**: Combines with git (`internal/git`), code edits.
-- **No new deps**: Uses `os/exec` for `tea` (assume installed).
-
-## Agent Tool Examples
-
-```
-grokkit agent "List open PRs, pick one to work on, create branch"
-# Grok: tea pr ls → Analyzes → tea branch create fix-123 → edits files
-```
-
-```
-grokkit agent "Fix lint, create issue if high-risk changes"
-# Grok: Runs lint → tea issue create "High-risk refactor"
-```
-
-## High-Level Implementation
-
-1. **Detect tea**: Check `exec.Command("tea", "version")`.
-2. **Tool schemas** (Grok tool-calling JSON):
-   - `list_prs(state: "open") → string`
-   - `create_pr(title, body, base) → pr_url`
-   - `comment_pr(pr_id, body)`
-   - `merge_pr(pr_id)`
-3. **Wrapper funcs** in `internal/tools/tea.go`:
-   ```go
-   func ListPRs(ctx context.Context, args map[string]any) (string, error)
-   ```
-4. **Agent loop** (`cmd/agent.go`): Parse tool calls → Execute → Feed output back.
-5. **Safety**:
-   - Whitelist: pr ls/create/comment/merge, issue create/comment.
-   - Preview: Print `tea ...` + expected output.
-   - Confirm: User y/n for each.
-6. **Config**: `[tools.tea.enabled]`, `[tools.tea.gitea_url]`.
-
-## Flags / Config
-
-| Key | Description |
-|-----|-------------|
-| `tools.tea.enabled` | Enable tea tools |
-| `tools.tea.gitea_url` | Override repo URL |
-| `--dry-run` | Simulate tool calls |
-
-## Implementation Notes
-
-- **Extend agent**: Modify plan phase to include tool suggestions.
-- **Error handling**: `internal/errors` + GitError-like TeaError.
-- **Tests**: Table-driven `TestTeaTools` with mocks (`exec.Command` patching).
-- **Effort**: Medium (~200 LOC). Reuse `grok` client, agent UX.
-- **Prereq**: User installs `tea` (`go install github.com/go-tea/tea/cmd/tea@latest`).
-
-## ROI
-
-**High**. Closes workflow gap:
-
-| Stage | Covered |
-|-------|---------|
-| Code edit | `agent` ✓ |
-| Repo mgmt | **tea tools** ← new |
-| Logging | cnotes (next) |
-
-Force-multiplies daily PR/issue workflows.