Fix Linting Issues in CLI Tests

Claude Code·Opus 4.8[1m]·Soph·3mo ago·2min·1 Checkpoint·1 file change·-3·8.1K tokens

can you fix linting

3mo ago·1m

Lint passes — 0 issues.

The failure was a gocritic exitAfterDefer warning in cmd/entire/cli/global_test.go: a defer os.RemoveAll(isolationDir) that could never run because TestMain ends with os.Exit(code). Since the same cleanup already runs explicitly on the line right before os.Exit, I removed the dead defer.

can you commit and push

3mo ago·31.5s

Committed and pushed to soph/cli-testmain-token-isolation.

28f560acli: drop dead defer in test TestMain that os.Exit skips The deferred os.RemoveAll never ran because TestMain ends with os.Exit(code); the same cleanup already runs explicitly just before the exit. Removing it clears the gocritic exitAfterDefer lint failure. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 438e4b46f73c-3