Skip to content

Commit a548ae0

Browse files
coderabbitai[bot]CodeRabbit
andauthored
fix: apply CodeRabbit auto-fixes (#97)
Fixed 1 file(s) based on 1 unresolved review comment. Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
1 parent 58621c7 commit a548ae0

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

src/routes/documents.ts

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -160,17 +160,12 @@ documentRoutes.post('/upload/batch', async (c) => {
160160
const parsed = JSON.parse(resultText);
161161
r2Key = parsed.r2_key ?? r2Key;
162162
} catch (parseErr) {
163-
console.error(`[documents] Batch: ChittyStorage JSON parse failed for ${safeName}:`, parseErr, 'MCP response:', mcp);
163+
console.error(`[documents] Batch: ChittyStorage mcp/error parsing for ${safeName}:`, parseErr, 'resultText:', resultText);
164164
results.push({ filename: safeName, status: 'error', error: 'ChittyStorage ingest failed' });
165165
continue;
166166
}
167-
} else if (!storageRes.ok || mcp?.error) {
168-
console.error(`[documents] Batch: ChittyStorage failed for ${safeName}:`, mcp?.error ?? storageRes.status);
169-
results.push({ filename: safeName, status: 'error', error: 'ChittyStorage ingest failed' });
170-
continue;
171167
} else {
172-
// HTTP 200 but resultText is missing/empty — silent fall-through bug
173-
console.error(`[documents] Batch: ChittyStorage returned OK but missing result.content[0].text for ${safeName}, MCP response:`, mcp);
168+
console.error(`[documents] Batch: ChittyStorage mcp/response missing resultText for ${safeName}:`, mcp);
174169
results.push({ filename: safeName, status: 'error', error: 'ChittyStorage ingest failed' });
175170
continue;
176171
}

0 commit comments

Comments
 (0)