Skip to content

fix: append article subtopic content - #23

Merged
julianschiavo merged 1 commit into
mainfrom
julian/fix-include-article-subtopics
Mar 13, 2026
Merged

fix: append article subtopic content#23
julianschiavo merged 1 commit into
mainfrom
julian/fix-include-article-subtopics

Conversation

@julianschiavo

@julianschiavo julianschiavo commented Mar 13, 2026

Copy link
Copy Markdown
Member
  • Append child topic content onto parent article results in search and entry lookups.
  • Discover child topics through VectorSearch identifier filters instead of SQLite or payload inspection.
  • Cover the regression with live integration tests for expanded content and subtopic deduplication.

Note

Medium Risk
Changes vector-search result shaping by fetching and merging descendant topic content and filtering out subtopic hits, which can affect search/entry output and performance. Also adds a new Objective-C selector bridge (stringIdentifiersApplyingFilters) that could fail at runtime if the private API differs across OS versions.

Overview
Search and exact entry lookups now expand article content with descendant topic content (identifiers using the parent#child convention), so parent articles include subtopic text when omitContent is false.

This adds a new VSKClientObject.stringIdentifiers(applying:) bridge to fetch identifiers by attribute filters, uses it to discover topic identifiers, fetches descendant assets in bulk, appends their content onto the parent hit, and deduplicates search results so subtopics aren’t returned separately when the parent article is present.

Adds live integration tests covering content expansion and subtopic deduplication, plus new live-environment fixtures for an article with subtopics.

Written by Cursor Bugbot for commit 207c363. This will update automatically on new commits. Configure here.

@julianschiavo
julianschiavo merged commit f5834f8 into main Mar 13, 2026
3 checks passed
@julianschiavo
julianschiavo deleted the julian/fix-include-article-subtopics branch March 13, 2026 15:30
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