Bill/obsidian-mcp-server
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: remove console.error from graceful error handlers

Browse Source 
Removed console.error calls from error handlers that gracefully skip
problematic files and continue processing. These handlers catch errors
when reading or parsing files but successfully return fallback values,
so logging errors creates unnecessary noise during testing and deployment.

Changes:
- vault-tools.ts: Remove console.error from search and frontmatter extraction
- search-utils.ts: Remove console.error from file search handlers
- waypoint-utils.ts: Remove console.error from file read handler
- frontmatter-utils.ts: Remove console.error from YAML and Excalidraw parsing

Test updates:
- Remove test assertions checking for console.error calls since these
  are no longer emitted by graceful error handlers

All 709 tests pass with no console noise during error handling.
This commit is contained in:
Bill Ballou
2025-10-26 12:44:00 -04:00
parent 8957f852b8
commit 48e429d59e
7 changed files with 19 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
tests/frontmatter-utils.test.ts
View File

@@ -158,8 +158,7 @@ describe('FrontmatterUtils', () => {
describe('parse errors', () => {
test('handles parseYaml throwing error', () => {
const content = '---\ninvalid: yaml: content:\n---\nContent';
const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
mockParseYaml.mockImplementation(() => {
mockParseYaml.mockImplementation(() => {
throw new Error('Invalid YAML');
});
@@ -169,13 +168,8 @@ describe('FrontmatterUtils', () => {
expect(result.frontmatter).toBe('invalid: yaml: content:');
expect(result.parsedFrontmatter).toBe(null);
expect(result.contentWithoutFrontmatter).toBe('Content');
expect(consoleErrorSpy).toHaveBeenCalledWith(
'Failed to parse frontmatter:',
expect.any(Error)
);
consoleErrorSpy.mockRestore();
});
});
test('handles parseYaml returning null', () => {
const content = '---\ntitle: Test\n---\nContent';
@@ -806,21 +800,15 @@ excalidraw-plugin`;
\`\`\`compressed-json
N4KAkARALgngDgUwgLgAQQQDwMYEMA2AlgCYBOuA7hADTgQBuCpAzoQPYB2KqATL
\`\`\``;
const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
const result = FrontmatterUtils.parseExcalidrawMetadata(content);
expect(result.isExcalidraw).toBe(true);
expect(result.elementCount).toBe(0);
expect(result.hasCompressedData).toBe(true);
expect(result.metadata).toEqual({ compressed: true });
expect(consoleErrorSpy).toHaveBeenCalledWith(
'Failed to process compressed Excalidraw data:',
expect.anything()
);
consoleErrorSpy.mockRestore();
global.atob = originalAtob;
global.atob = originalAtob;
});
test('handles JSON parse error gracefully', () => {
@@ -828,28 +816,21 @@ N4KAkARALgngDgUwgLgAQQQDwMYEMA2AlgCYBOuA7hADTgQBuCpAzoQPYB2KqATL
\`\`\`json
{invalid json content}
\`\`\``;
const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
const result = FrontmatterUtils.parseExcalidrawMetadata(content);
expect(result.isExcalidraw).toBe(true);
expect(result.elementCount).toBe(0);
expect(result.hasCompressedData).toBe(false);
expect(result.metadata).toEqual({});
expect(consoleErrorSpy).toHaveBeenCalledWith(
'Excalidraw parsing error:',
expect.any(Error)
);
consoleErrorSpy.mockRestore();
});
});
test('handles error when no Excalidraw marker present', () => {
const content = `\`\`\`json
{invalid json}
\`\`\``;
const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
const result = FrontmatterUtils.parseExcalidrawMetadata(content);
expect(result.isExcalidraw).toBe(false);
@@ -857,21 +838,18 @@ N4KAkARALgngDgUwgLgAQQQDwMYEMA2AlgCYBOuA7hADTgQBuCpAzoQPYB2KqATL
expect(result.hasCompressedData).toBeUndefined();
expect(result.metadata).toBeUndefined();
consoleErrorSpy.mockRestore();
});
});
test('logs error but returns valid result structure', () => {
const content = 'excalidraw-plugin with error causing content';
const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
// Force an error by making content throw during processing
const result = FrontmatterUtils.parseExcalidrawMetadata(content);
// Should still return valid structure
expect(result).toHaveProperty('isExcalidraw');
consoleErrorSpy.mockRestore();
});
});
});
describe('edge cases', () => {