Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@internal/mcpinit/hook_test.go`:
- Line 19: The t.Cleanup callback is ignoring the error returned by db.Close(),
causing golangci-lint failure; update the cleanup closure to capture the
returned error from db.Close() and report it (e.g., if err := db.Close(); err !=
nil { t.Fatalf("closing db: %v", err) }) so the test reports the failure instead
of discarding it. Use the existing t.Cleanup(func() { ... }) and replace the
body to check and handle the error from db.Close().

In `@internal/mcpinit/hook.go`:
- Around line 229-234: The current combined SQL (the row := db.QueryRow(...)
query using cwd and cwdBase) lets a name-matching row win due to ORDER BY
LENGTH(path), and LENGTH(path) > 10 also blocks short exact path matches; fix by
preferring path matches first: run a path-only query using the ((? = path OR ?
LIKE path || '/%')) condition (remove the LENGTH(path) > 10 check so exact short
paths are allowed) and if that returns no row, run a separate fallback query for
name = ? using cwdBase; update the code around row := db.QueryRow(...) to
attempt the path lookup first (with cwd, cwd) and only query by name (cwdBase)
if the first query yields no result.

In `@internal/mcpserver/mcpserver_test.go`:
- Around line 593-627: The test TestTaskUpdate_EmptyStatusPreservesCurrentStatus
is re-implementing the handler logic instead of exercising the actual
ghost_task_update handler; update the test to invoke the registered handler path
(or refactor the handler body into a callable function) so an empty args.Status
is passed through ghost_task_update and the handler reads current :=
store.GetTask(...) and calls store.UpdateTask(...) with current.Status;
specifically, replace the direct call to store.UpdateTask with a call that posts
args to the MCP server's ghost_task_update handler (or call the extracted
handler function) and assert the resulting task still has Status "pending" and
Priority 1.

---

Outside diff comments:
In `@internal/mcpserver/mcpserver.go`:
- Around line 972-980: The resource Description in the s.mcp.AddResource call is
stale—its text says "Top 50 cross-project Ghost memories" while the handler
calls s.store.GetTopMemories(ctx, "_global", 15); update the Description string
used in the Resource (in the AddResource invocation) to reflect the actual limit
(e.g., "Top 15 cross-project Ghost memories") and ensure any adjacent wording
(like "Top 50" elsewhere in that Description) is changed consistently so the
user-facing text matches the s.store.GetTopMemories limit.
- Around line 1128-1144: parseProjectIDFromURI double-decodes the project_id
because it uses u.Path (already decoded) then calls url.PathUnescape; change the
function to use the raw/escaped path instead: obtain the escaped path via
u.RawPath when non-empty or u.EscapedPath(), split that escaped path
(strings.TrimPrefix(...), strings.SplitN) to extract the project segment, then
call url.PathUnescape exactly once on that segment so encoded values (including
%2F) are preserved and only decoded a single time.