feat: NoAutoFetch parity for summarizePr/summarizeDiff + gitignore cleanup (#211)

* feat(query): NoAutoFetch parity for summarizePr and summarizeDiff

The 9.1.0 --no-auto-fetch opt-out was only wired through ReviewPROptions.
summarizePr (pr.go) and summarizeDiff (navigation.go) still auto-fetched
unconditionally, giving air-gapped callers inconsistent behavior across
the three review-adjacent MCP tools.

Add NoAutoFetch to both options structs; route through VerifyRef
(local-only) instead of EnsureRef when set.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* chore: gitignore ckb_fresh and *.test binaries in repo root

ckb_fresh is a dev-build binary that kept showing up as untracked;
*.test catches Go test binaries like scip.test.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: SimplyLiz

.gitignore

@@ -8,8 +8,10 @@ bin/
 /ckb
 /ckb-test
 /ckb-bench
+/ckb_fresh
 coverage.out
 *_test
+*.test
 *.scip
 
 # Registry / credential tokens

CHANGELOG.md

@@ -17,6 +17,10 @@ All notable changes to CKB will be documented in this file.
 
 ### Added
 
+- `NoAutoFetch` option on `SummarizePROptions` and `SummarizeDiffOptions`
+  for parity with `ReviewPROptions`. Previously the air-gapped opt-out
+  applied only to `ckb review`; MCP/HTTP callers of `summarizePr` and
+  `summarizeDiff` had no way to disable auto-fetch.
 - Troubleshooting section in `docs/plans/review-cicd.md` covering shallow
   CI clones, auth-failure remediation (`persistCredentials`), air-gapped
   pipelines, and depth-0 checkout alternatives.

internal/query/navigation.go

@@ -1879,6 +1879,7 @@ type SummarizeDiffOptions struct {
 	CommitRange *CommitRangeSelector `json:"commitRange,omitempty"`
 	Commit      string               `json:"commit,omitempty"`
 	TimeWindow  *TimeWindowSelector  `json:"timeWindow,omitempty"`
+	NoAutoFetch bool                 `json:"noAutoFetch,omitempty"` // Disable auto-fetch of missing refs (see ReviewPROptions)
 }
 
 // CommitRangeSelector specifies a base..head range.
@@ -2026,11 +2027,13 @@ func (e *Engine) SummarizeDiff(ctx context.Context, opts SummarizeDiffOptions) (
 		selector = DiffSelector{Type: "commitRange", Value: opts.CommitRange.Base + ".." + opts.CommitRange.Head}
 		base = opts.CommitRange.Base
 		head = opts.CommitRange.Head
-		if resolved, rerr := e.gitAdapter.EnsureRef(base); rerr == nil {
-			base = resolved
-		}
-		if resolved, rerr := e.gitAdapter.EnsureRef(head); rerr == nil {
-			head = resolved
+		if !opts.NoAutoFetch {
+			if resolved, rerr := e.gitAdapter.EnsureRef(base); rerr == nil {
+				base = resolved
+			}
+			if resolved, rerr := e.gitAdapter.EnsureRef(head); rerr == nil {
+				head = resolved
+			}
 		}
 		diffStats, err = e.gitAdapter.GetCommitRangeDiff(base, head)
 		if err != nil {

internal/query/pr.go

@@ -16,6 +16,7 @@ type SummarizePROptions struct {
 	BaseBranch       string `json:"baseBranch"`       // Base branch to compare against (default: "main")
 	HeadBranch       string `json:"headBranch"`       // Head branch (default: current branch)
 	IncludeOwnership bool   `json:"includeOwnership"` // Include ownership analysis (default: true)
+	NoAutoFetch      bool   `json:"noAutoFetch"`      // Disable auto-fetch of missing base ref (see ReviewPROptions)
 }
 
 // SummarizePRResponse is the response for summarize-pr.
@@ -102,11 +103,20 @@ func (e *Engine) SummarizePR(ctx context.Context, opts SummarizePROptions) (*Sum
 	if headRef == "" {
 		headRef = "HEAD"
 	}
-	baseRef, err := e.gitAdapter.EnsureRef(opts.BaseBranch)
-	if err != nil {
-		return nil, fmt.Errorf("failed to resolve base ref %q: %w", opts.BaseBranch, err)
+	var baseRef string
+	if opts.NoAutoFetch {
+		if verr := e.gitAdapter.VerifyRef(opts.BaseBranch); verr != nil {
+			return nil, fmt.Errorf("failed to resolve base ref %q: %w", opts.BaseBranch, verr)
+		}
+		baseRef = opts.BaseBranch
+	} else {
+		resolved, rerr := e.gitAdapter.EnsureRef(opts.BaseBranch)
+		if rerr != nil {
+			return nil, fmt.Errorf("failed to resolve base ref %q: %w", opts.BaseBranch, rerr)
+		}
+		baseRef = resolved
+		opts.BaseBranch = resolved
 	}
-	opts.BaseBranch = baseRef
 	diffStats, err := e.gitAdapter.GetCommitRangeDiff(baseRef, headRef)
 	if err != nil {
 		return nil, fmt.Errorf("failed to get diff: %w", err)