feat(v1.2.3): thinking feedback, git tools fix, authorize_path #34
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add evolving spinner messages during LLM thinking (Thinking → Analyzing → Planning → Preparing → elapsed time) - Fix git tools cwd bug for global npm installs (simpleGit object form, explicit cwd in execSync) - Add read-only git tools (status, diff, log, branch) to all subagent types - Create authorize_path tool so LLM can request path access conversationally - Highlight project folder name in welcome screen path display - Update system prompt to use authorize_path instead of manual /allow-path - Add .tape/.gif to .gitignore - Bump version to 1.2.3
…coverage checks - Add excludePaths to Pattern interface; skip console.log detection for CLI/REPL files - Make checkTestCoverage() verify test files on disk before flagging missing tests - Downgrade "no test" to info severity when test file exists but wasn't in the diff - Add no-AI-attribution guideline to system prompt
- Replace console.log+chalk rendering with markdown builder + renderMarkdown() - Review output now uses formatted tables (marked-terminal box-drawing) - Findings shown as scannable table rows: File | Severity | Category | Issue - Output also returned as markdown string in skill result for LLM consumption - Add JSDoc to exported analyzePatterns function
- Skip test-coverage finding entirely when test file exists on disk and source changes are < 15 additions (refactors/fixes covered by existing tests) - Only emit info-level nudge for large changes (>= 15 additions) with existing tests - Fix oxlint no-control-regex warning in renderer.ts (intentional ANSI escape match) - Linter now fully clean: 0 warnings, 0 errors
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
simpleGitto use{ baseDir }object form and added explicitcwd: process.cwd()to allexecSynccalls across git-simple.ts and git-enhanced.ts — fixes "Not in a git repository" error on global npm installs/allow-pathTest plan
pnpm typecheck— no errorspnpm lint— 0 errors (1 pre-existing warning)pnpm test— 172 test files, 4360 tests passedpnpm dev, ask a complex question, verify spinner evolves through messagespnpm dev, ask "review the PR of this branch vs main", verify git tools workgit statusworks