Skip to content

fix: build system, version sync, and error logging improvements - #12

Open
bark1n0 wants to merge 1 commit into
Goldentrii:mainfrom
bark1n0:main
Open

fix: build system, version sync, and error logging improvements#12
bark1n0 wants to merge 1 commit into
Goldentrii:mainfrom
bark1n0:main

Conversation

@bark1n0

@bark1n0 bark1n0 commented May 31, 2026

Copy link
Copy Markdown

Changes

  • Fix build system: Add typescript as devDependency — tsc command was not found because the package was missing from node_modules
  • Fix version mismatch: Sync VERSION constant in types.ts (3.4.18 → 3.4.19) to match package.json
  • Fix dependency version: Update mcp-server core dependency from 3.4.18 → 3.4.19 for consistency
  • Improve error handling: Add process.stderr.write logging to empty catch blocks in critical code paths:
  • session-start.ts — backfill errors
  • session-end.ts — awareness and rooms read errors
  • check.ts — correction write errors
  • smart-remember.ts — conflict scan and consistency check errors
  • recall-backend.ts — backend loading errors

Testing

  • npm run build — passes ✅
  • npm test — 272/272 tests pass ✅

Notes
Previously, errors in these catch blocks were silently swallowed, making debugging difficult. Now they log to stderr without breaking the main flow (all blocks remain non-blocking).

- Install TypeScript dev dependency to fix tsc command not found
- Sync VERSION constant (3.4.18 → 3.4.19) with package.json
- Update mcp-server core dependency to match (3.4.18 → 3.4.19)
- Add stderr logging to empty catch blocks in critical paths:
  session-start, session-end, check, smart-remember, recall-backend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant