Commit 734fc6c
authored
fix: remove chat command and add CI coverage gate (#81)
The chat command advertised three conflicting default models (kimi-k2.6
in help text, openai/gpt-oss in the option builder, google/gemma-3-27b-it
in the service) and was effectively untested (1.21% line coverage).
Remove the command entirely per product direction.
Removed:
- src/commands/chat.ts, src/services/chat-service.ts (+ their tests)
- src/utils/markdown-renderer.ts (only used by chat)
- marked, marked-terminal runtime deps + @types/* dev deps (orphaned)
- CHAT entries from command-structure constants
- Chat Command section from README
Kept DefaultApiKeyManager: still used by auth/credentials/api-key.ts and
the api-keys set-default/get-default subcommands.
Add a CI coverage gate to prevent silent coverage regressions:
- vitest.config.ts: v8 coverage, include all src, exclude generated types/
schemas/__tests__, thresholds ratcheted just below the post-removal
baseline (50/52/48/50 stmt/branch/func/line)
- package.json: test:coverage script
- test.yml: run npm run test:coverage in CI (replaces plain test:run)
- .gitignore: ignore coverage/ (artifacts were previously committed)1 parent cc2ce25 commit 734fc6c
23 files changed
Lines changed: 24 additions & 2742 deletions
File tree
- .github/workflows
- coverage
- src
- commands
- constants
- services
- utils
- tests
- commands
- services
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
41 | | - | |
| 40 | + | |
| 41 | + | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
| 3 | + | |
3 | 4 | | |
4 | 5 | | |
5 | 6 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | 28 | | |
47 | 29 | | |
48 | 30 | | |
| |||
68 | 50 | | |
69 | 51 | | |
70 | 52 | | |
71 | | - | |
| 53 | + | |
72 | 54 | | |
73 | 55 | | |
74 | 56 | | |
75 | 57 | | |
76 | | - | |
77 | | - | |
78 | | - | |
79 | 58 | | |
80 | 59 | | |
81 | 60 | | |
| |||
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Binary file not shown.
0 commit comments