Skip to content

feat: add generic parser filters to parse and search - #152

Merged
digizeph merged 2 commits into
mainfrom
feat/generic-parser-filters
Aug 17, 2026
Merged

feat: add generic parser filters to parse and search#152
digizeph merged 2 commits into
mainfrom
feat/generic-parser-filters

Conversation

@digizeph

Copy link
Copy Markdown
Member

Summary

  • add repeatable non-time --filter key=value|key!=value expressions to parse and search
  • forward generic and extended element filters through remote-search client and SSE server
  • normalize remote start + duration searches to explicit UTC epoch bounds

Validation

  • cargo fmt -- --check
  • cargo clippy --all-features -- -D warnings
  • cargo test --all-features
  • exercised parse/search CLI help plus accepted ip_version and rejected generic time filters

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds generic bgpkit-parser filters to local and remote parse/search workflows.

Changes:

  • Adds repeatable generic filter expressions with validation.
  • Forwards extended filters through SSE remote search.
  • Normalizes remote duration-based searches to epoch bounds.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/lens/parse/mod.rs Parses and applies generic filters.
src/lens/search/mod.rs Validates parser filters before search.
src/server/search.rs Accepts generic filters over SSE.
src/bin/commands/search.rs Forwards filters and normalizes time bounds.
src/bin/commands/search_remote.rs Extends the remote wire DTO.
CHANGELOG.md Documents the feature.

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/lens/parse/mod.rs Outdated
@digizeph
digizeph merged commit 6835087 into main Aug 17, 2026
1 check passed
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.

2 participants