korekt-cli 0.13.2 → 0.13.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +1 -1
- package/src/git-logic.js +3 -1
- package/src/git-logic.test.js +2 -2
- package/src/index.js +20 -2
- package/src/index.test.js +29 -0
package/package.json
CHANGED
package/src/git-logic.js
CHANGED
|
@@ -18,7 +18,9 @@ export function truncateContent(content, maxLines = 2000) {
|
|
|
18
18
|
const halfMax = Math.floor(maxLines / 2);
|
|
19
19
|
const head = lines.slice(0, halfMax).join('\n');
|
|
20
20
|
const tail = lines.slice(-halfMax).join('\n');
|
|
21
|
-
|
|
21
|
+
const linesKept = 2 * halfMax;
|
|
22
|
+
const omittedLines = lines.length - linesKept;
|
|
23
|
+
return `${head}\n\n... [${omittedLines} lines truncated] ...\n\n${tail}`;
|
|
22
24
|
}
|
|
23
25
|
|
|
24
26
|
/**
|
package/src/git-logic.test.js
CHANGED
|
@@ -450,7 +450,7 @@ describe('truncateContent', () => {
|
|
|
450
450
|
// Should contain head and tail
|
|
451
451
|
expect(result).toContain('line 0');
|
|
452
452
|
expect(result).toContain('line 999'); // Last line of head (first 1000 lines)
|
|
453
|
-
expect(result).toContain('... [truncated] ...');
|
|
453
|
+
expect(result).toContain('... [1000 lines truncated] ...');
|
|
454
454
|
expect(result).toContain('line 2000'); // First line of tail (last 1000 lines)
|
|
455
455
|
expect(result).toContain('line 2999');
|
|
456
456
|
|
|
@@ -464,7 +464,7 @@ describe('truncateContent', () => {
|
|
|
464
464
|
|
|
465
465
|
expect(result).toContain('line 0');
|
|
466
466
|
expect(result).toContain('line 49'); // Last of first 50
|
|
467
|
-
expect(result).toContain('... [truncated] ...');
|
|
467
|
+
expect(result).toContain('... [100 lines truncated] ...');
|
|
468
468
|
expect(result).toContain('line 150'); // First of last 50
|
|
469
469
|
expect(result).toContain('line 199');
|
|
470
470
|
});
|
package/src/index.js
CHANGED
|
@@ -19,6 +19,9 @@ import { detectCIProvider, truncateFileData, formatErrorOutput } from './utils.j
|
|
|
19
19
|
// Re-export utilities for backward compatibility
|
|
20
20
|
export { detectCIProvider, truncateFileData, formatErrorOutput, getPrUrl } from './utils.js';
|
|
21
21
|
|
|
22
|
+
// Export for testing
|
|
23
|
+
export { GEMINI_MODELS, resolveModel };
|
|
24
|
+
|
|
22
25
|
const require = createRequire(import.meta.url);
|
|
23
26
|
const { version } = require('../package.json');
|
|
24
27
|
|
|
@@ -103,6 +106,19 @@ const GEMINI_MODELS = [
|
|
|
103
106
|
},
|
|
104
107
|
];
|
|
105
108
|
|
|
109
|
+
/**
|
|
110
|
+
* Resolve model input - supports numeric shortcuts (1-4) or full model names
|
|
111
|
+
* @param {string} input - Model input (number or name)
|
|
112
|
+
* @returns {string} - Resolved model name
|
|
113
|
+
*/
|
|
114
|
+
function resolveModel(input) {
|
|
115
|
+
const num = parseInt(input, 10);
|
|
116
|
+
if (!isNaN(num) && num >= 1 && num <= GEMINI_MODELS.length) {
|
|
117
|
+
return GEMINI_MODELS[num - 1].value;
|
|
118
|
+
}
|
|
119
|
+
return input;
|
|
120
|
+
}
|
|
121
|
+
|
|
106
122
|
/**
|
|
107
123
|
* Prompt user to select a Gemini model
|
|
108
124
|
* @returns {Promise<string>} - Selected model name
|
|
@@ -258,7 +274,8 @@ program
|
|
|
258
274
|
selectedModel = await selectModel();
|
|
259
275
|
log(chalk.green(`Using model: ${selectedModel}\n`));
|
|
260
276
|
} else if (typeof options.model === 'string') {
|
|
261
|
-
selectedModel = options.model;
|
|
277
|
+
selectedModel = resolveModel(options.model);
|
|
278
|
+
log(chalk.green(`Using model: ${selectedModel}\n`));
|
|
262
279
|
}
|
|
263
280
|
|
|
264
281
|
// Fetch file rules config from API
|
|
@@ -465,7 +482,8 @@ async function reviewUncommitted(mode, options) {
|
|
|
465
482
|
selectedModel = await selectModel();
|
|
466
483
|
log(chalk.green(`Using model: ${selectedModel}\n`));
|
|
467
484
|
} else if (typeof options.model === 'string') {
|
|
468
|
-
selectedModel = options.model;
|
|
485
|
+
selectedModel = resolveModel(options.model);
|
|
486
|
+
log(chalk.green(`Using model: ${selectedModel}\n`));
|
|
469
487
|
}
|
|
470
488
|
|
|
471
489
|
// Fetch file rules config from API
|
package/src/index.test.js
CHANGED
|
@@ -5,6 +5,7 @@ import {
|
|
|
5
5
|
detectCIProvider,
|
|
6
6
|
getPrUrl,
|
|
7
7
|
handleSkippedResponse,
|
|
8
|
+
resolveModel,
|
|
8
9
|
} from './index.js';
|
|
9
10
|
|
|
10
11
|
describe('CLI JSON output mode', () => {
|
|
@@ -553,6 +554,34 @@ describe('--model flag behavior', () => {
|
|
|
553
554
|
});
|
|
554
555
|
});
|
|
555
556
|
});
|
|
557
|
+
|
|
558
|
+
describe('numeric model shortcuts', () => {
|
|
559
|
+
it('should resolve "1" to gemini-2.5-pro', () => {
|
|
560
|
+
expect(resolveModel('1')).toBe('gemini-2.5-pro');
|
|
561
|
+
});
|
|
562
|
+
|
|
563
|
+
it('should resolve "2" to gemini-2.5-flash', () => {
|
|
564
|
+
expect(resolveModel('2')).toBe('gemini-2.5-flash');
|
|
565
|
+
});
|
|
566
|
+
|
|
567
|
+
it('should resolve "3" to gemini-3-pro-preview', () => {
|
|
568
|
+
expect(resolveModel('3')).toBe('gemini-3-pro-preview');
|
|
569
|
+
});
|
|
570
|
+
|
|
571
|
+
it('should resolve "4" to gemini-3-flash-preview', () => {
|
|
572
|
+
expect(resolveModel('4')).toBe('gemini-3-flash-preview');
|
|
573
|
+
});
|
|
574
|
+
|
|
575
|
+
it('should pass through full model names unchanged', () => {
|
|
576
|
+
expect(resolveModel('gemini-3-pro-preview')).toBe('gemini-3-pro-preview');
|
|
577
|
+
});
|
|
578
|
+
|
|
579
|
+
it('should pass through invalid input unchanged', () => {
|
|
580
|
+
expect(resolveModel('invalid')).toBe('invalid');
|
|
581
|
+
expect(resolveModel('0')).toBe('0');
|
|
582
|
+
expect(resolveModel('5')).toBe('5');
|
|
583
|
+
});
|
|
584
|
+
});
|
|
556
585
|
});
|
|
557
586
|
|
|
558
587
|
describe('skipped response handling', () => {
|