Skip to content

Commit 3677241

Browse files
committed
fix: resolve all 15 lint errors failing CI
- Remove unused imports: buildCorrectionMessage, formatRetry, ToolPolicy, StoredMessage, stat, extname, estimateMessageTokens, estimateTokens - Remove unnecessary type assertion in critic.ts - Prefix unused var _MAX_SUMMARY_TOKENS in memory.ts - Remove unused loop variable 'i' in memory.ts - Fix eslint-disable-next-line placement for no-control-regex in logo.ts
1 parent 5af8f48 commit 3677241

8 files changed

Lines changed: 8 additions & 14 deletions

File tree

src/commands/ask.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,10 @@ import {
1313
enrichUserPrompt,
1414
generateSearchPlan,
1515
buildSynthesisReminder,
16-
buildCorrectionMessage,
1716
formatToolCall,
1817
formatToolResult,
1918
formatSeparator,
2019
formatDuplicateSkip,
21-
formatRetry,
2220
formatHintInjection,
2321
formatUsage,
2422
formatPlanBlock,

src/commands/run.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,10 @@ import {
1212
enrichUserPrompt,
1313
generateSearchPlan,
1414
buildSynthesisReminder,
15-
buildCorrectionMessage,
1615
formatToolCall,
1716
formatToolResult,
1817
formatSeparator,
1918
formatDuplicateSkip,
20-
formatRetry,
2119
formatHintInjection,
2220
formatUsage,
2321
formatPlanBlock,
@@ -29,7 +27,7 @@ import { criticEvaluate, buildCriticCorrection } from '../utils/critic.js';
2927
import { searchPastSessions, formatPastExchanges } from '../utils/past-sessions.js';
3028
import { getOrBuildIndex, searchSymbols, formatSymbolResults } from '../utils/index-builder.js';
3129
import { updateContextWithUsage } from '../utils/context.js';
32-
import type { CliOverrides, ToolPolicy } from '../config/schema.js';
30+
import type { CliOverrides } from '../config/schema.js';
3331
import type { Message } from '../providers/base.js';
3432

3533
export interface RunOptions extends CliOverrides {

src/ui/chat.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import {
1515
enrichUserPrompt,
1616
generateSearchPlan,
1717
buildSynthesisReminder,
18-
buildCorrectionMessage,
1918
} from '../utils/agent.js';
2019
import { WorkingMemory } from '../utils/memory.js';
2120
import { ToolResultCache } from '../utils/cache.js';

src/ui/logo.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ function buildPlain(): string {
6161

6262
// Strip ANSI escape codes to get the visible (printable) length of a string.
6363
const visibleLength = (s: string): number =>
64+
// eslint-disable-next-line no-control-regex
6465
s.replace(/\x1b\[[0-9;]*m/g, '').length;
6566

6667
function buildColored(): string {

src/utils/context.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
* conventions, architectural decisions, etc.
1111
*/
1212

13-
import { readFile, writeFile, readdir, access, constants, stat } from 'node:fs/promises';
14-
import { basename, join, extname } from 'node:path';
13+
import { readFile, writeFile, readdir, access, constants } from 'node:fs/promises';
14+
import { basename, join } from 'node:path';
1515
import type { Dirent } from 'node:fs';
1616

1717
export const CONTEXT_FILENAME = 'JAM.md';

src/utils/critic.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ function parseCriticResponse(text: string): CriticVerdict {
118118
const confLine = lines.find(l => l.toUpperCase().startsWith('CONFIDENCE'));
119119
if (confLine) {
120120
const match = confLine.match(/[\d.]+/);
121-
if (match) confidence = Math.min(1, Math.max(0, parseFloat(match[0]!)));
121+
if (match) confidence = Math.min(1, Math.max(0, parseFloat(match[0])));
122122
}
123123

124124
// Extract reason

src/utils/memory.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@
1616
import type { Message } from '../providers/base.js';
1717
import type { ProviderAdapter } from '../providers/base.js';
1818
import {
19-
estimateMessageTokens,
20-
estimateTokens,
2119
checkBudget,
2220
truncateToolOutput,
2321
} from './tokens.js';
@@ -34,7 +32,7 @@ const COMPACTION_THRESHOLD = 0.70;
3432
export const MAX_TOOL_RESULT_TOKENS = 1500;
3533

3634
/** Max token budget for compacted summary. */
37-
const MAX_SUMMARY_TOKENS = 800;
35+
const _MAX_SUMMARY_TOKENS = 800;
3836

3937
// ── Tool result capping (P0-2) ────────────────────────────────────────────────
4038

@@ -127,7 +125,7 @@ export async function compactMessages(
127125

128126
// Build a condensed representation of middle messages for the summarizer
129127
const middleText = middleMessages
130-
.map((m, i) => `[${m.role}] ${m.content.slice(0, 500)}${m.content.length > 500 ? '…' : ''}`)
128+
.map((m) => `[${m.role}] ${m.content.slice(0, 500)}${m.content.length > 500 ? '…' : ''}`)
131129
.join('\n---\n');
132130

133131
// Try to summarize via LLM

src/utils/past-sessions.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
*/
88

99
import { listSessions, getSession } from '../storage/history.js';
10-
import type { SessionDetail, StoredMessage } from '../storage/history.js';
10+
import type { SessionDetail } from '../storage/history.js';
1111

1212
// ── Types ─────────────────────────────────────────────────────────────────────
1313

0 commit comments

Comments
 (0)