Repository navigation
feat(tools/discourse): search forum topics by full text - #2018
Open
MakerFlyFly wants to merge 1 commit into
Open
MakerFlyFly wants to merge 1 commit into
MakerFlyFly wants to merge 1 commit into
Conversation
This was referenced Nov 6, 2026
Let callers find relevant forum discussions using native queries and follow the matching posts. Preserve forum ranking, deduplicate topics and distinguish output truncation from the server's search window. Generated-by: OpenAI Codex CLI 0.166.0-alpha.5.1 (GPT-6)
MakerFlyFly
force-pushed
the
feat/discourse-topic-search-upstream
branch
from
November 6, 2026 12:35
89d47d8 to
bc5dd1b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
magpie-discourse topic search <query> [--limit N]so callers can discover relevant forum discussions and open them withtopic view. The forum receives the original query, including native filters and ordering.matchescontains the distinct posts returned on the pages read, so its length cannot establish a total hit or contribution count.truncatedand Discourse's ten-page window with the independentserver_limit_reachedflag. Valid empty searches succeed. An explicit null group on page 1 gives a keyword hint, while a null later page identifies the failed page. Read failures produce no partial JSON.Type of change
Test plan
Current head
bc5dd1bce83c298381515d019f2c0704e772dc68is one commit onb6ccd434a54bdd4761fb487d7b55aaa8bd89dbc0. Its complete Git tree and canonical patch are identical to the prior publication89d47d81d9edbe4f4adb28abb4eeaf71e639ee9a.89d47d81includes 410 package tests on each of Linux/Python 3.12 and Windows/Python 3.14, ruff, format, mypy and five integration suites (vendor-neutrality-score, skill-and-tool-validator, dev, spec-validator and spec-inventory). These results retain their original SHA.86811931exercised a filter-only query and a normal two-topic search in two GETs. Publication README rendering and links were checked on the identical89d47d81file content. Owner auditRFC-AI-0004 compliance
Linked issues
Refs #307, following merged #1755 and #1841. Owner audit is recorded in fork PR #34 and fork issue #33.
Notes for reviewers
AI disclosure
AI-generated PR description. OpenAI Codex CLI 0.166.0-alpha.5.1 (GPT-6) assisted with implementation, tests, review follow-up and publication.
The author has manually reviewed the complete change through audited
868119319e05a0e213b613af76d4ca8e0c1eaa24and takes responsibility for this contribution. The published change is equivalent to the reviewed changes, with the existing publication tree preserved through the squash.