Skip to content

feat(admin)!: search analytics route reads core's recorder (slice 3b) - #16

Closed
zumbrunn wants to merge 2 commits into
feat/plugin-services-slice3from
feat/search-analytics
Closed

zumbrunn wants to merge 2 commits into
feat/plugin-services-slice3from
feat/search-analytics

Conversation

@zumbrunn

@zumbrunn zumbrunn commented Nov 5, 2026

Copy link
Copy Markdown
Member

Slice 3b of the @dune/plugin-api extraction (duneorg/dune#64). Stacked on #15 (slice 3); base is that branch. Draft until #12, #13, #14, #15 and core 0.36 are out.

Core now records search queries itself when system.search.analytics: true (opt-in; see the core PR). This restores GET /admin/api/search/analytics on top of that: it returns services.search.analytics.summarize() plus an enabled flag instead of reading the file directly, so admin imports nothing from @dune/core/search. This supersedes the route removal in #15; the changelog entry is rewritten to match.

Tests and deno task check pass (239, 2 new) against unreleased core 0.36 (scratch workspace). Not mergeable until @dune/core is ^0.36.0.

🤖 Generated with Claude Code

zumbrunn and others added 2 commits November 5, 2026 01:35
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@zumbrunn

Copy link
Copy Markdown
Member Author

Landed in 4.0.0 as part of #26 (one squash of the whole stack: its intermediate commits only type-check against the final contract). Verified against the published @dune/core 0.36.0 and @dune/testing 2.0.0.

@zumbrunn zumbrunn closed this Nov 10, 2026
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