Skip to content

feat(memory/v2): add concept-frequency debug route#29618

Merged
siddseethepalli merged 1 commit into
mainfrom
do/memory-v2-frequency-log
May 5, 2026
Merged

feat(memory/v2): add concept-frequency debug route#29618
siddseethepalli merged 1 commit into
mainfrom
do/memory-v2-frequency-log

Conversation

@siddseethepalli
Copy link
Copy Markdown
Contributor

@siddseethepalli siddseethepalli commented May 5, 2026

Summary

  • New POST memory/v2/concept-frequency debug route. Aggregates the existing memory_v2_activation_logs table by (slug, status) using SQLite json_each and cross-references listPages() for the on-disk concept universe.
  • Returns per-slug counts (injected / in_context / not_injected / page_missing), lastInjectedAt, an onDisk flag, and a neverEvaluatedSlugs array. Optional filters: conversationId, sinceMs.
  • Pure derived view from existing logs — no new schema, no dual-write counter, no migration.

Original prompt

it


Open in Devin Review

Aggregates memory_v2_activation_logs by (slug, status) and cross-
references the on-disk concept universe so an operator can see which
concepts get injected often, which get scored but rejected, and which
on-disk pages never surface as candidates. Pure derived view — no new
persistence, no schema migration.
@siddseethepalli siddseethepalli merged commit 628827c into main May 5, 2026
@siddseethepalli siddseethepalli deleted the do/memory-v2-frequency-log branch May 5, 2026 07:25
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

siddseethepalli added a commit that referenced this pull request May 5, 2026
The memory/v2/concept-frequency debug endpoint was added in #29618 but no
matching scope policy was registered, causing guard-tests.test.ts to fail
in CI. Add settings.read (matches sibling read-only debug routes like
concept-page and explain-similarity).

Co-authored-by: Vellum Assistant <assistant@vellum.ai>
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