Add content classification for truncated prefixes - #10
Merged
Merged
Conversation
andrew
force-pushed
the
add-classify-prefix
branch
2 times, most recently
from
October 1, 2026 10:32
a74981c to
e4126e8
Compare
Export Result.Set so content results reach the path-based Set filters.
andrew
force-pushed
the
add-classify-prefix
branch
from
October 1, 2026 10:36
e4126e8 to
9d2a362
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
1 participant
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.
ClassifyBlobrequires complete contents, so passing a cached prefix can treat a cut-off comment as ending at EOF. AddClassifyPrefix(path, prefix)and a matching classifier method for callers sharing bounded inputs with tools such aspeek.Inspected prefixes set
HeaderLimitedand use the existing truncation handling for incomplete comments and split UTF-8 characters. The API preserves vendor-root context and the existing byte and line limits. Add coverage for cached 1 KiB inputs and document input completeness in content cache keys.