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

Inline comments:
In `@docs/architecture.md`:
- Line 78: The docs reference to internal/mcpserver/server.go is stale; update
the package map entry in docs/architecture.md to point to the actual
implementation file internal/mcpserver/mcpserver.go (replace server.go with
mcpserver.go) so links/navigation resolve correctly and the MCP server entry
matches the real symbol.

In `@internal/mcpserver/mcpserver.go`:
- Line 588: The code currently calls s.store.GetTopMemories(ctx, "_global", 50)
which only returns the top 50 global memories and truncates "all" cross-project
memories; update the handler to retrieve the full set instead of a fixed 50.
Replace the single GetTopMemories call with either (a) a call to a store method
that returns all global memories (e.g., s.store.GetAllMemories(ctx, "_global")
or similarly named method), or (b) implement pagination by repeatedly calling
s.store.GetTopMemories (or a paginated store API) and appending results to the
memories slice until no more results are returned; ensure the variable memories
still holds the complete set before further processing. Reference:
s.store.GetTopMemories and the memories variable in this handler.
- Around line 555-572: The handler currently calls s.buildProjectContext(ctx,
projectID) which swallows datastore/read errors and returns an empty string,
making real errors look like an empty project; change buildProjectContext to
return (string, error) and update the ReadResource handler(s) to check that
error and return a non-OK error (wrapped with context) instead of embedding an
empty Text field—specifically update the anonymous ReadResource func shown and
the analogous handler around lines 611-624 to call buildProjectContext(ctx,
projectID), handle a non-nil error by returning nil, fmt.Errorf("reading project
context %q: %w", projectID, err), and only populate ResourceContents.Text when
err == nil. Ensure you propagate the underlying store error (not mask it) and
keep resolveProjectID usage unchanged.

---

Nitpick comments:
In `@internal/mcpserver/mcpserver_test.go`:
- Around line 257-268: Update TestNew_RegistersResources to verify that New()
actually registers resources instead of only constructing the server: after
creating store and logger and calling New(store, logger) (symbol New and
variable srv), assert that known resource endpoints are discoverable by invoking
the server's discovery/read handlers or making test HTTP requests against the
server/router for the expected resource URIs (use the same URIs used elsewhere
in the package), and check responses/status codes or returned resource metadata
to confirm registration; fail the test if the discovery/read path for each
expected resource URI is missing or returns an error.