fix(mcp-server): retrieve collection id to send the right activity log

[Scra3]

Summary

• MCP activity logs now call /api/activity-logs-requests/by-collection-name instead of /api/activity-logs-requests
• The server-side endpoint resolves collection names to IDs before storing
• Adds resolveCollectionName option to ActivityLogsService and createPath to ActivityLogHttpOptions

Context: The MCP server sends collection names (e.g. "Address") because it doesn't have access to internal collection IDs. This caused the collection field to appear blank in the Activity Logs UI.

Depends on: ForestAdmin/forestadmin-server#8163 (new server endpoint)

Test plan

• forestadmin-client tests pass (269)
• mcp-server tests pass (535)
• E2E: create activity log via MCP, verify collection shows in UI

🤖 Generated with Claude Code

Note

Fix MCP server activity logs to send collection ID instead of collection name

• Adds getCollectionId(renderingId, collectionName) to ActivityLogsService and ForestHttpApi, fetching the collection ID from /liana/renderings/{renderingId}/collections/{name}/id.
• Updates createPendingActivityLog in the MCP server to resolve the collection name to its ID before sending the activity log, falling back to the name when the ID cannot be resolved.
• Caches resolved collection IDs per rendering in a module-level map to avoid redundant HTTP calls.
• Returns null on NotFoundError and propagates other errors.

^{Macroscope summarized db1d81f.}

[qltysh]

Coverage Impact

⬇️ Merging this pull request will decrease total coverage on main by 0.01%.

Modified Files with Diff Coverage (5)

Rating	File	% Diff	Uncovered Line #s
	packages/mcp-server/src/utils/activity-logs-creator.ts	100.0%
	packages/agent-testing/src/forest-admin-client-mock.ts	0.0%	62
	packages/mcp-server/src/http-client/mcp-http-client.ts	100.0%
	packages/forestadmin-client/src/activity-logs/index.ts	100.0%
	packages/forestadmin-client/src/permissions/forest-http-api.ts	0.0%	134-140
	Total	86.4%

🤖 Increase coverage with AI coding...

In the `fix/mcp-activity-log-collection-name` branch, add test coverage for this new code:

- `packages/agent-testing/src/forest-admin-client-mock.ts` -- Line 62
- `packages/forestadmin-client/src/permissions/forest-http-api.ts` -- Line 134-140

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[qltysh]

All good ✅