Address code review feedback: fix exception handling, async lock initialization, and concurrency protection

[Copilot]

Addresses 5 review comments from PR #100 on exception handling, async lock initialization, and database concurrency protection.

Changes

• Exception handling: Changed bare except: to except Exception: in utils.py to avoid catching system-exiting exceptions (SystemExit, KeyboardInterrupt)

• Async lock initialization: Made _db_lock lazy-initialized via property to prevent event loop context issues when DocumentationIndexer is instantiated outside async contexts:

@property
def db_lock(self) -> asyncio.Lock:
    """Lazy-initialize and return the database lock."""
    if self._db_lock is None:
        self._db_lock = asyncio.Lock()
    return self._db_lock

• Concurrency protection: Wrapped index_documentation method in __init__ to ensure all invocations (including direct calls from update_index tool and run method) automatically acquire the database lock, preventing race conditions with concurrent search operations

• Code cleanup: Removed useless try-except block that caught and immediately re-raised exceptions, and removed unused param import from tests

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[MarcSkovMadsen]

Looks good